Codex Review Skill

Codex CLI を使って 外部 AI（OpenAI）にコードレビューを依頼 するスラッシュコマンド。レビュー本体は必ず codex に書かせる（呼び出し側が diff を読んで自前でレビューを書くのは禁止）。

絶対遵守ルール

1. 下記の引数パターンで決まる codex exec review コマンドを 1 回だけ 実行する。失敗しても 同一コマンドの再実行も別コマンドへの切替も行わない（--base / --commit への fallback、リトライ、いずれも禁止）。必ず codex exec review を使うこと。裸の codex review（exec なし）は対話TUIを起動し、非TTY環境（このスキルの実行環境）ではハングして起動しない。
2. レビュー対象のリポジトリを改変しない (git add / git commit / git stash 等の禁止)。git status / git diff 等の読み取り系のみ許可。テスト/再現用に別ディレクトリで git init する分には対象外。
3. codex 出力が失敗（後述の判定基準）なら、自前でレビューを書かず「Codex レビュー失敗」として失敗フォーマットで報告する。
4. sandbox オプション（-c sandbox_mode=danger-full-access）を省略・変更しない。これがないと bwrap が user namespace を作れない環境で codex が diff を読めず空出力になる。
5. MCP 無効化フラグ（$MCPOFF、後述）を省略しない。 MCP サーバが有効なままだと codex が起動直後のツール呼び出しで無反応になり、リトライ禁止のため 1 回のタイムアウトで工程ごと落ちる。

使い方

codex exec review は --uncommitted / --base / --commit の各 scope フラグと自由文 PROMPT を 排他 で扱う。カスタム指示を渡す場合は scope フラグを外し、scope を PROMPT 本文に含めて指示する。

引数パターン判定

• $ARGUMENTS が空 → uncommitted (scope フラグ形)
• $ARGUMENTS が --base <branch> で始まる → base 差分
• $ARGUMENTS が --commit <sha> で始まる → commit
• それ以外 → uncommitted + カスタム指示（scope フラグを付けず、scope と追加指示を PROMPT 本文に含める）

MCP 無効化フラグ $MCPOFF

codex 実行前に必ず組み立てる。-c mcp_servers={} は効かない（設定がマージされて残る）。

MCPOFF=$(grep -oP '^\[mcp_servers\.\K[a-zA-Z0-9_-]+(?=\])' ~/.codex/config.toml \
  | sort -u | sed 's|^|-c mcp_servers.|; s|$|.enabled=false|' | tr '\n' ' ')

command を /bin/false に差し替える方法は使わない。url ベース（streamable HTTP）のサーバが 設定されていると Error loading config.toml: url is not supported for stdio で config 読み込みごと失敗する。enabled=false は stdio と url の両方に効く。

~/.codex/config.toml に [mcp_servers.*] が 1 つも無ければ $MCPOFF は空になる。それで正しい。

実行手順

1. git status --short で変更状況を確認。uncommitted パターンで差分ゼロなら codex を呼ばずに「レビュー対象なし」フォーマット（後述）で返す。
2. $MCPOFF を組み立てる（上記）。
3. 上記の表から 1 コマンドを選んで実行。タイムアウトは 300 秒程度見込む（reasoning effort が高い設定では 180 秒では足りないことがある）。
4. 成功判定（下記）を行い、成功/失敗フォーマットで報告。

成功判定（機械的に）

codex 出力が 下記のいずれかを含めば失敗:

• 空または codex 本体応答（ヘッダ行以降、codex\n ラベル以下の本文）が 1 行のみ
• unable to inspect / could not inspect のいずれかの文字列
• Sandbox(Denied / bwrap: のいずれかの文字列（bwrap 迂回失敗の実エラー）
• Error: で始まる行

また、codex プロセス自体が起動しない場合（Claude Code 側の permission 拒否、codex バイナリ未インストール、認証エラーで即時 exit 等）も 失敗扱い とする。この場合は codex 出力が無いため、原因として Claude Code / 環境側のエラーメッセージを引用する。

注意: codex は正常時も sandbox: <mode> をヘッダに出力するため、裸の sandbox 文字列を判定に使ってはいけない。上記の具体パターンのみを使うこと。

上記に該当しなければ成功。判定は重複除去前の原文に対して行う（重複除去は報告フォーマット適用時のみ）。

報告フォーマット

レビュー対象なし時（手順 1 で短絡した場合）:

## レビュー対象なし

作業ディレクトリに未コミット変更がないため、Codex は呼び出していません。
変更を加えてから再実行するか、`--base <branch>` / `--commit <sha>` で対象を指定してください。

成功時:

## Codex レビュー結果

### 指摘事項
- [codex 出力からそのまま抽出]

### 推奨事項
- [codex 出力からそのまま抽出]

### 評価
[codex 出力の総評部分]

codex 出力がこの 3 節構造と合わない場合の扱い:

• codex が Review comment: 形式の単一段落、または codex\n ラベル直後に裸の段落で返すことがある。3 節構造（指摘事項 / 推奨事項 / 評価）に分かれていない場合は一律 codex 出力の本文を「指摘事項」に丸ごと原文のまま貼る（勝手に日本語訳・要約・配分しない）。
• 「推奨事項」「評価」に該当する内容が codex 出力に無ければ、そのセクションは - なし と書くか、セクションごと省略する。
• codex の出力構造を 改造してはいけない。「推奨事項」が空だからといって「指摘の裏返し」を勝手に書き起こすのは自前補完の一種で禁止。
• codex が 同一文または同一段落 を 2 回以上連続で繰り返した場合（codex 側の既知の挙動）は 1 回にまとめてよい（重複除去のみ許可、内容は一字一句変えない）。文単位の重複と段落単位の重複の両方が起きた場合は段落単位を優先（より大きい単位で集約）。

失敗時:

## Codex レビュー失敗

### 原因
[codex 出力の該当部分を短く引用]

### 対処案
- `~/.codex/config.toml` で sandbox_mode を確認
- bwrap が user namespace を作れない環境では `-c sandbox_mode=danger-full-access` 必須
- タイムアウトで無反応だった場合は `$MCPOFF` が付いていたか確認（MCP サーバ起動でハングする）
- 未認証の可能性があれば `codex login` を案内

絶対にやってはいけない: 失敗時に自分で diff を読み直して「追加で気づいた点」を書くこと。

注意

• Codex CLI が認証済みであること（未認証なら codex login）
• 作業ディレクトリが git リポジトリであること