← ClaudeAtlas

pr-reviewlisted

Review a PR diff, or the branch diff with --branch, and post an approve or request-changes verdict. Use when reviewing a PR(PRレビュー・マージ前チェック・セルフレビュー).
devbasex/ai-plugins · ★ 1 · Code & Development · score 71
Install: claude install-skill devbasex/ai-plugins
# コードレビューコマンド PR 差分、または `--branch` 指定時は現在のブランチの差分を、専門家としてレビューする。 ## 引数 | 引数 | 意味 | 既定 | |---|---|---| | `[PR番号]` | レビュー対象の PR | 直前の PR | | `--branch` | PR ではなく **ローカルブランチの差分**をレビューする(PR 作成前のセルフレビュー) | OFF | | `[AIエージェント]` | `codex` / `gemini` に委譲。省略時は Claude 自身 | Claude | | `--focus AREA` | 重点観点(`security` / `performance` / `tests` / 任意の文字列) | なし | ``` /ndf:pr-review # 直前 PR をレビュー /ndf:pr-review 9352 # PR 番号を指定 /ndf:pr-review 9352 codex # Codex CLI に委譲 /ndf:pr-review --branch # ローカルブランチをセルフレビュー /ndf:pr-review --branch security # セキュリティに焦点を当ててセルフレビュー ``` ## 2 つのモード | 観点 | PR モード(既定) | `--branch` モード | |---|---|---| | 対象 | GitHub 上の PR 差分 | `git diff <既定ブランチ>` の差分 | | 出力先 | **PR 上にインラインコメント + 総評を投稿** | セッション上の報告のみ(投稿しない) | | 判定 | `APPROVE` / `REQUEST_CHANGES` | 判定を出さず改善提案を返す | | 用途 | PR 作成後のレビュー | PR 作成前のセルフレビュー | **どちらのモードでもコード��正は行わない**(分析と指摘のみ。修正は `/ndf:fix`)。 ## 観点は 2 段に分ける **第 1 段(仕様適合)を先に通す。** コード品質だけを見ると、きれいに書かれた「仕様を 満たさない実装」を通してしまう。 ### 第 1 段: 仕様適合 | 確認 | 見るもの | | --- | --- | | 受け入れ条件を満たすか | プラン・PR 本文の受け入れ条件と、対応するテスト | | ドメインの不変条件を破っていないか | 状態遷移・数量・期限・権限の扱い。上位層から迂回して不整合な状態を作れないか | | 対象範囲外の変更がないか | 依頼・プランの範囲と差分の一致。無関係な整形や改名の混入 | | テストが仕様を表しているか | テスト名と検証内容が受け入れ条件に対応しているか。実装の写しになっていないか | 第 1 段で**満たさない項目があれば、その時点で `REQUEST_CHANGES` とする**。第 2 段の 指摘を積み上げても、仕様を満たさない実装は直しようがない。 受け入れ条件が PR 本文にもプランにも書かれていない場合は、その不在自体を指摘する (条件がなければ第 1 段のレビューが成立しない)。 ### 第 2 段: コード品質 | 確認 | 見るもの | | --- | --- | | 責務・