fix(review): AI review lanes report success without posting a review on large PRs
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 45/100
調査の方向性
Start by locating the Claude review and security review lane workflows, then trace how each lane records completion and writes the “reviewed through” marker. Reproduce the large pull request behavior described for #7009 and inspect how diff retrieval handles GitHub’s 300-file cap. Done means incomplete lanes report failure or neutral and omit the marker, with full diff coverage or a summary that identifies partial coverage.
索引モデルが issue の本文から書いたものです。
説明
Problem
On a large pull request, both AI review lanes conclude success without posting any review.
Observed on #7009 (828 changed files, head 55236cd59; runs 38179498349 and 38179498378, both success):
- claude-review: the sticky progress comment stopped with 2 of 8 checklist items ticked. "Verify candidate findings, post inline comments" and "Final summary" were never reached. The comment says
gh pr diff/gh apihit GitHub's 300-file diff cap and git fetch had no network, so the review covered only the 100 files supplied in context. No inline comments or review were posted. - claude-security-review: the sticky comment stopped at "Fan out 4 hunters ... running". "Adversarially verify" and "Post findings" were never ticked. No findings were posted.
- Both posted a "reviewed through " marker anyway.
A green check row therefore claims a review that did not happen.
Expected
- A lane that does not reach its post or summary step concludes as failure or neutral, not success, and does not write the "reviewed through" marker.
- claude-review gets the full diff without the 300-file API cap (for example a local
git diffagainst a fetched base), or says in its summary that coverage was partial and which files it skipped.
- 主要言語
- Shell
- スター
- 22
- フォーク
- 2
- 平均マージ
- 8時間 25分
- マージ済み PR（30日）
- 944
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
melodic-software/claude-code-plugins のほかの issue
-
good first issue priority: low
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
melodic-software/claude-code-plugins#7530 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
needs-human priority: medium
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
melodic-software/claude-code-plugins#7455 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
priority: medium
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
melodic-software/claude-code-plugins#7351 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
agent-ready priority: high work-class: scoped
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
melodic-software/claude-code-plugins#7312 · コメント 3 件 ·
メンテナーはふだん 1 日以内に返信
-
priority: medium
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
melodic-software/claude-code-plugins#7288 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
melodic-software/claude-code-plugins の issue をすべて見る
似ている issue
-
area: compat area: parsing bug
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
reubeno/brush#1526 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
Micke-K/IntuneManagement#441 ·
-
review-2026-10 Trad En
難易度 2/5 半日 初心者へのやさしさ 70/100
zigbeefordomoticz/wiki#302 ·
-
Add dependabotオープンfeat
難易度 2/5 1時間未満 初心者へのやさしさ 72/100
open-nudge/opentemplate#204 ·
メンテナーはふだん 1 日以内に返信
-
omarchy-brightness-keyboard: double idle-blank saves brightness 0, so wake 'restore' keeps the keyboard backlight off対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信