実装の差分をレビューする

PR作成前に少なくとも1回、実装とは別セッションで実行する。実装の経緯は渡さず、差分と判定の基準を渡す。

1. 対象を固定する

AGENTS.md、agents/rules/review.mdと該当ルールを読む。対象の差分、対応する要件とプランを特定する。

未コミット変更はgit diff HEADで確認し、未追跡ファイルも別途読む。ブランチは合意した比較元との差分を使う。対象が不明なら確認する。

2. 観点を分けて照合する

観点	確認する内容
プランとの整合	受け入れ条件の不足、範囲外の変更、設計判断との矛盾
異常系と境界値	0・上限・空の値、失敗経路、削除箇所の参照、対応するテスト
セキュリティ	認可、外部入力、秘密情報、危険なデータ操作

例えば削除処理なら、「存在するIDを削除できる」だけでなく、「存在しないID」「他の利用者のID」「保存処理の失敗」を確認する。差分で判断できない部分は周辺コードを読む。

観点別の3体を使う場合はdesign-reviewer、edge-case-reviewer、security-reviewerへ同じ成果物を渡し、並列でレビューする。実装セッションの対話履歴は渡さない。ツール固有の起動方法に合わせ、各担当の指摘を根拠ごとに整理する。

3. 指摘を報告する

P0/P1/P2、実際のファイルと行、問題を起こす条件、根拠、期待する結果を書く。推測は「要確認」と分ける。

レビュー中はファイルを編集しない。検証を実行した場合はコマンドと結果を示す。静的に読んだだけなら、実行時の挙動を確認した扱いにしない。

4. 終了条件を示す

対象と観点、指摘、未検証の範囲を報告する。指摘がなくても何を確認したかを示す。P0・P1修正後の再レビューでは、既存指摘の解消と修正による新しい問題を確認する。

エラー時

差分や資料を取得できない場合は、不足する情報を報告する。確認できない対象を問題なしと判定しない。