pr-reviewlisted
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 段: コード品質
| 確認 | 見るもの |
| --- | --- |
| 責務・