code-review

コードレビュー

ローカルの変更、PR の変更、またはレビューの反復に使用します。まず references/repository.md を読み、その後、変更箇所に適用される指示を確認してください。レビュー担当レーンは指摘事項のみを報告し、修正は変更の担当者が行います。

レビュー回数の上限

max-turns=<positive integer> を任意で指定できます。デフォルトは 10 です。最初のレビュー前に、指定がない場合、値が 0 以下の場合、整数でない場合、または形式が不正な場合は拒否してください。上限はありません。レビューを1回完了するごとに、この上限に加算してください。上限に達しても RESULT: PASSED になっていない場合は、現在の証拠を示して停止し、新たに明示的な承認を得た実行を開始するか尋ねてください。自動で上限を延長してはいけません。

1. 実効レビュー範囲の選択

HEAD だけでなく、現在意図されている変更をレビューしてください。

1. 明示されたターゲット、PR のメタデータ、またはリポジトリのデフォルトからベースを特定します。不明確な場合は確認してください。
2. コミット範囲 base...HEAD を算出します。
3. ステージ済みの変更（git diff --cached）、未ステージの変更（git diff）、および意図された未追跡ファイルを合わせます。
4. 各レビューの前、および修正の後に、この統合範囲を再計算します。

変更されていないコードは、実効差分によってリスクが生じる場合に限り、証拠として扱います。無関係な作業ツリーの変更は保持してください。

2. リポジトリのベースライン

AGENTS.md、適用されるコントリビューター向け指示、フォーマット・リント・テストの実行コマンド、アーキテクチャやドメインのドキュメント、変更箇所に適用される規約を確認してください。重大度と必須の検証方法はリポジトリのルールに従います。生成器の入力、テンプレート、または設定が変更されていない場合は、生成された出力を確認対象から除外してください。

3. レビュー

機械的レビュー

変更されたすべてのファイルを周辺の文脈とともに読みます。正確性、セキュリティ／プライバシー、データ損失、認可／検証、互換性／契約、エラー処理、並行性、マイグレーション／設定、動作テストを確認してください。review.md を参照してください。

構造レビュー

明示的に依頼された場合、またはリポジトリで必須の場合にのみ実施してください。structural-review.md に従って、重大な複雑さ、所有権、不変条件、境界、または重複する正規ヘルパーの回帰を探します。見た目、行数、推測に基づくリファクタリングを理由にブロックしてはいけません。

検証レビュー

実行可能な動作、API／契約、権限、ジョブ／イベント、マイグレーション、生成クライアント、またはブラウザーに表示される変更については、具体的な正常系、異常系、権限、境界のケースを導き出してください。validation.md に従ってください。ブラウザー、ライブ環境、作業ツリーのツールを利用できるとは限らず、条件に応じて使用してください。

指摘事項の基準

重大で、対処可能かつ差分に起因するリスクのみを報告してください。重大度は次のとおりです。

• Critical： セキュリティ／プライバシー侵害、データ破損／損失、公開契約の破壊、または主要経路の停止を引き起こす可能性が高いもの。
• Important： 正確性、認可、永続化、互換性、カバレッジ、アーキテクチャ、または変更されたホットパスのパフォーマンスに関する重大なリスク。
• Summary： 具体的な保守性に関する所見。重大なリスクに該当しない限り、ブロック要因にはなりません。

根本原因ごとに指摘事項を1件にまとめ、最適な変更箇所の path:line を示してください。具体的なリスク、リポジトリ上の根拠やルール、および必要な対応を記述します。称賛、好み、古い／重複したコメント、推測に基づく整理は省いてください。

上限付きレビューの反復

loop.md を使用してください。変更の担当者による修正後、コーディネーターは決定論的なチェックを実行し、実効範囲全体を再計算して、指摘事項のみの新たなレビューを依頼します。推測で指摘事項を解決済みにしてはいけません。重大な指摘事項が残っている場合、範囲が拡大した場合、または上限に達した場合は、現在の証拠を示して停止し、次の指示を求めてください。

出力の契約

レビュー担当者は指摘事項のみを返し、編集、コミット、プッシュ、投稿は行いません。各レビューの最後には次を記載してください。

CODE REVIEW
Scope: <base>...<head plus staged/unstaged/untracked intended overlay>
Depth: <standard|structural>
Findings: <none|severity summaries with path:line>
Checks: <not run|required commands and outcomes>
Residual risk: <none|summary>
RESULT: PASSED|FAILED

最終出力の最後の空でない行は、必ず RESULT: PASSED または RESULT: FAILED と完全に一致させてください。すべての重大な差分リスクが分類され、未解決の Critical／Important の指摘事項がなく、必須として選択された検証が合格しているか明確に実施不能と示され、現在の実効範囲が改めてレビューされている場合に限り、合格とします。