Codex レビュー
Codex に組み込まれたコードレビューをクローズアウトチェックとして実行します。これはコードレビュー（codex review）であり、Guardian の auto_review 承認ルーティングではありません。
以下の場合に使用します:
- ユーザーがCodexレビュー / autoreview / セカンドモデルレビューを依頼したとき
- 重要なコード編集後、最終決定/コミット/出荷前
- 修正後のローカルブランチまたはPRブランチのレビュー時
契約事項
- レビュー出力は参考として扱います。盲目的に適用しないでください。
- すべての指摘は、実際のコードパスと隣接ファイルを読んで検証してください。
- 指摘が外部の振る舞いに依存する場合は、依存関係のドキュメント/ソース/型を読んでください。
- 非現実的なエッジケース、推測に基づくリスク、大規模な書き換え、コードベースを過度に複雑化する修正は却下してください。
- 適切な所有権の境界で小さな修正を優先し、バグのクラスを明確に改善しない限りリファクタリングは行わないでください。
- Codex レビューが受理可能/実行可能な指摘を返さなくなるまで続けてください。
- レビューでトリガーされた修正がコードを変更した場合は、対象のテストを再実行し、Codex レビューも再実行してください。
- レビューモデルを切り替えたり上書きしたりしないでください。レビューがモデルの容量に達した場合は、同じコマンドを同じモデルで数回再試行してください。ヘルパーはデフォルトでネストされたレビューを yolo/フルアクセスモードで実行します。サンドボックスの動作を意図的にテストする場合にのみ
--no-yoloを使用してください。 - レビューコマンド/ヘルパーが終了コード0で、受理可能/実行可能な指摘がないと判断した時点で停止してください。より良い「クリーン」な行、セカンドオピニオン、またはより明確なクローズアウト文言を得るために、追加で直接
codex reviewを実行しないでください。 - ヘルパーの正常終了と実行可能な指摘がないことを、基礎となる Codex CLI の出力が簡潔であっても、クリーンなレビュー結果として扱ってください。
- 指摘を意図的/修正不要として却下する場合は、将来のレビュアーが知るべき実際の不変条件や所有権の決定を説明する場合にのみ、簡潔なインラインコードコメントを追加してください。
- レビューだけのためにプッシュしないでください。ユーザーがプッシュ/出荷/PR更新をリクエストした場合にのみプッシュしてください。
対象の選択
ローカルのダーティな作業:
codex review --uncommitted
これは、パッチが現在のチェックアウトで実際にアンステージ/ステージ/アントラックされている場合にのみ使用してください。コミット済み、プッシュ済み、またはPRの作業の場合は、代わりにコミットまたはブランチ差分をCodexに指定してください。ヘルパードキュメントが最初にダーティな作業を挙げているからといって、--mode local / --uncommitted を強制しないでください。クリーンな --uncommitted レビューは、ローカルパッチがないことだけを証明します。
ブランチ/PRの作業:
git fetch origin
codex review --base origin/main
--base でインラインプロンプトを渡さないでください。現在のCLIは、ヘルプテキストが曖昧であっても、--base + [PROMPT] を拒否します。カスタム指示が必要な場合は、最初にプレーンなベースレビューを実行し、その後、ローカル/手動でフォローアップパスを行ってください。
オープンなPRが存在する場合は、その実際のベースを使用します:
base=$(gh pr view --json baseRefName --jq .baseRefName)
codex review --base "origin/$base"
コミット済みの単一変更:
codex review --commit HEAD
またはヘルパーを使用:
/Users/steipete/Projects/agent-scripts/skills/codex-review/scripts/codex-review --mode commit --commit HEAD
main に既にマージ済みまたはプッシュ済みの作業にはコミットレビューを使用してください。プッシュ後にクリーンな main を origin/main に対してレビューしても、通常は空の差分になります。小さなスタックの場合は、各コミットを明示的にレビューするか、マージ前に --base を使用してブランチをレビューしてください。
並行クローズアウト
フォーマットが行位置を変更する可能性がある場合は、最初にフォーマットしてください。その後、テストとレビューを並行して実行しても問題ありません:
scripts/codex-review --parallel-tests "<重点テストコマンド>"
トレードオフ: テストによりコード変更が強制され、レビューが古くなる可能性があります。テストまたはレビューがコード編集につながった場合は、影響を受けるテストを再実行し、受理可能/実行可能な指摘がなくなるまでレビューを再実行してください。その再実行が正常に終了したら停止してください。冗長な確認のために別の長いレビューサイクルを費やさないでください。
コンテキスト効率
Codex レビューは通常、ノイズが多いです。サブエージェントが利用可能な場合は、デフォルトでサブエージェントフィルタを使用してください。サブエージェントにレビューを実行させ、以下だけを返すように依頼してください:
- 受理する実行可能な指摘
- 却下する指摘（理由を一行で）
- 再実行する正確なファイル/テスト
インライン実行は、小さな変更の場合、またはサブエージェントが利用できない場合のみ使用してください。
ヘルパー
バンドルされているヘルパー:
~/.codex/skills/codex-review/scripts/codex-review --help
agent-scripts からインストールした場合、パスは:
/Users/steipete/Projects/agent-scripts/skills/codex-review/scripts/codex-review --help
ヘルパーは:
- まずダーティな
--uncommittedを選択します - そうでない場合、
gh pr viewが機能すれば現在のPRベースを使用します - そうでない場合、main以外のブランチには
origin/mainを使用します - 既にコミット済みの作業、特にマージ後のクリーンな
mainには--mode commit --commit <ref>を使用します - PR/ブランチの作業には
--mode autoのままにするか、--mode branchを強制します。コミット後に--mode localを強制しないでください --outputまたはCODEX_REVIEW_OUTPUTが設定されていない限り、stdout にのみ書き込みます--dry-run、--parallel-tests、およびコミット参照をサポートします- デフォルトで
--dangerously-bypass-approvals-and-sandboxを使用してネストされたレビューを実行します --full-accessを引き続き受け入れます。オプトアウトするには--no-yoloまたはCODEX_REVIEW_YOLO=0を使用します- 選択したレビューコマンドが終了コード0で終了した場合、
codex-review clean: no accepted/actionable findings reportedを出力します
最終報告
以下を含めてください:
- 使用したレビューコマンド
- 実行したテスト/証拠
- 受理/却下した指摘とその簡単な理由
- 最終的なヘルパー/レビュー実行からのクリーンなレビュー結果、または残った指摘を意識的に却下した理由
最終報告の文言を改善するためだけに別のCodexレビューを実行しないでください。最終的なヘルパー実行が終了コード0で、受理可能/実行可能な指摘を生成しなかった場合は、その正確な実行をクリーンとして報告してください。