tanabee/codex-review

Codex CLI にコードレビューを依頼する。Claude Code は codex exec で自己完結したレビュープロンプトを渡すだけで、レビュー本体は Codex CLI 側で実施される。

引数

$ARGUMENTS は PR 番号/URL（省略可）。123、#123、PR URL のいずれか。

手順

1. Codex CLI へレビューを依頼

以下の codex exec コマンドを Bash ツールで実行する。<ARGS> は $ARGUMENTS をそのまま展開する（空なら空文字列のまま）:

codex exec "$(cat <<'EOF'
以下の手順でコードレビューを実施してください。

## 手順

### 1. 差分の取得
- PR 番号/URL が指定された場合: `gh pr view` で PR 情報を取得し、`gh pr diff` で差分を取得（PR モード）
- PR 番号/URL が空の場合: `gh pr view` を試み、現在のブランチに PR が存在するか確認する
  - PR が存在する → PR モードで差分を取得
  - PR が存在しない → `git diff main...HEAD` でメインブランチとの差分を取得（ローカルモード）

### 2. 情報収集
- PR モードの場合、PR の情報（タイトル、説明、関連 issue）を把握する
- 関連する issue がある場合は `gh issue view` で issue の目的・要件を把握する
- ローカルモードの場合、`tmp/issues/<issue番号>/` 配下の既存成果物を確認する:
  - `plan.md` — 実装計画。意図した設計や変更方針との整合性を確認
  - `report.md` — 実装レポート。実装者が認識している懸念点や追加変更を把握
  - `checklist.md` — 受け入れテストチェックリスト。テストの網羅性を検証する基準
  - `pr.md` — PR テキスト。PR の説明と実際の差分に乖離がないか確認

### 3. レビュー実施
変更ファイルごとにコードベースの該当箇所を読み、以下の観点でレビューする。出力フォーマットは `skills/codex-review/assets/template.md` を参照:

- **正確性**: ロジックにバグや抜け漏れがないか
- **設計**: 責務分離、命名、既存パターンとの一貫性
- **副作用 / 影響範囲**: 変更・削除・リネームしたシンボル（関数・型・定数・ファイルパス・設定キー等）の参照元が壊れていないか。シグネチャ変更による型エラー、削除した機能の呼び出し元、挙動変更による既存フローへの影響を `grep` 等で網羅的に確認する
- **セキュリティ**: インジェクション、認証・認可の不備、機密情報の漏洩がないか
- **パフォーマンス**: N+1 クエリ、不要な再レンダリング、計算量の問題がないか
- **テスト**: テストの網羅性、境界値・異常系のカバレッジ
- **可読性**: 複雑すぎるロジック、不明瞭な命名、過剰な抽象化がないか

### 4. 結果の保存
- PR モード: `tmp/prs/<PR 番号>/review-codex.md`
- ローカルモード: `tmp/issues/<issue 番号>/review-codex.md`
- `mkdir -p` で出力先ディレクトリを作成してから書き込む

## 注意事項
- 軽微なスタイルの指摘（空白、改行など）はリンターに任せ、レビューでは扱わない
- 指摘には必ず該当ファイルと行番号を含める
- 指摘ごとに重要度（must / should / nit）を付与する
- 良い点も積極的にコメントする

## 引数
<ARGS>
EOF
)"

• codex exec は非対話モードで動作する。対話的に止まる場合は --yolo 等の自動承認フラグを追加する
• 起動はフォアグラウンド実行で構わない（Codex CLI の完了を待つだけ）

2. 完了検証

Codex CLI の実行完了後、ls で以下のいずれかが存在することを確認する:

• PR モード: tmp/prs/<PR 番号>/review-codex.md
• ローカルモード: tmp/issues/<issue 番号>/review-codex.md

ファイルが生成されていない場合は codex exec を再実行する（最大 2 回まで）。

3. ユーザーへの提示

生成された review-codex.md の内容を要約してユーザーに提示する。