codex-review-workflow

Codex レビューワークフロー

概要

OpenAI Codex CLI を使用した自動コードレビューワークフローです。反復的な修正とレビューのサイクルを実施し、自動検証によってコード品質を確保します。

使用する場面： 自動コードレビュー、反復的な改善サイクル、または Codex CLI を使った特定の品質基準に対する検証が必要な機能を構築するとき。

このスキルを使用する場面

✅ 次の場合にこのスキルを使用してください：

• ユーザーが Codex CLI によるレビューを明示的に依頼している場合（例：「これを Codex でレビューしてください」）
• 自動コード検証が必要な機能を実装する場合
• 特定の品質基準を満たす必要があるコードを構築する場合
• 反復的なレビューと改善が必要な場合
• セキュリティ、バグ、ベストプラクティスを自動的に検証する場合

❌ 次の場合はこのスキルを使用しないでください：

• ユーザーが手動でのコードレビューのみを希望している場合
• 環境で Codex CLI を利用できない場合
• タスクが純粋に探索的、または調査を目的としている場合
• 正式なレビューを必要としない単純なコードの場合

前提条件

• Codex CLI がインストールされ、PATH から利用できること
• Git リポジトリ（または --skip-git-repo-check フラグを使用）
• インストールを確認：codex --version

基本的なワークフロー

このスキルでは、次の 6 つのステップからなる構造化されたプロセスに従います。

1. コーディングタスクを完了する

標準的なベストプラクティスに従って、ユーザーが依頼した機能を実装します。レビューに提出する前に、コードが適切に構成されていることを確認してください。

TodoWrite で進捗を追跡します：

• 依頼された機能または修正を実装する
• Codex CLI による初回レビューを実行する
• レビューで見つかった問題を修正する（あれば）
• Codex CLI による最終レビューを実行する
• 最終ステータスを報告する

2. Codex CLI による初回レビューを実行する

Git の要件： Codex CLI を使用するには Git リポジトリが必要です。Git リポジトリ内にいない場合は、まず git init を実行するか、--skip-git-repo-check フラグを使用してください（本番環境では推奨されません）。

codex exec を使用して Codex CLI によるレビューを実行します（codex review は使用しないでください）：

# For a specific file
codex exec "Review the code in <file_name> for bugs, security issues, best practices, and potential improvements. Provide specific, actionable feedback with line numbers and examples."

# For multiple files
codex exec "Review the files auth.py, user.py, and session.py for bugs, security issues, best practices, and potential improvements. Provide specific feedback for each file."

# With working directory context
codex exec "Review the code in email_validator.py for bugs, security issues, best practices, and potential improvements. Provide specific feedback." -C /path/to/project

# With specific model
codex exec "Review <file_name>..." -m gpt-5-codex

# With custom configuration
codex exec "Review <file_name>..." -c model="o3"

重要なポイント：

• レビュー対象をプロンプトで具体的に指定します
• 行番号と具体例を求めます
• 適切なタイムアウトを設定します（120000ms（推奨：2 分））

3. レビュー結果を分析する

Codex CLI は、形式が一定ではない構造化された Markdown 出力を返します。次の項目を確認してください。

重大な問題の指標（必ず修正）：

• セクション：バグ、セキュリティ、主な問題、主な指摘事項
• 重大度の表記：「High:」、「Medium:」、「critical」、「vulnerability」

品質向上の提案（優先度は低め）：

• セクション：保守性、ユーザビリティ、ベストプラクティス、提案
• 重大度の表記：「Low:」

確認の指標（成功）：

• セクション：解決済みのチェック項目、レビュー、レビューでの指摘事項
• フレーズ：「未解決の指摘事項はありません」、「すべての問題が解決しました」、「[N] 件すべての問題が解決したようです」
• チェックマーク（✅）または確認を示す表現

判断基準：

• 完了： バグ／セキュリティ／主な問題のセクションがなく、提案のみが残っている
• 完了： 解決済みのチェック項目で、以前の問題がすべて修正済みと確認されている
• 完了： 「未解決の指摘事項はありません」や「すべての問題が解決しました」などのフレーズがある
• 続行： バグ／セキュリティ／主な問題のセクションがある → ステップ 4 に進む

4. 特定された問題を修正する

特定された問題ごとに、次の手順を実施します。

1. 問題のあるコードを特定する
2. 問題の内容を理解する
3. Edit ツールを使用して修正する
4. 何を、なぜ変更したのかを記録する

ベストプラクティス：

• 再レビューする前に、1 回の反復ですべての問題を修正します
• 警告よりも重大なエラーを優先します
• 各修正についてユーザーに明確に説明します
• 問題に対処しながら、機能を維持します

5. Codex CLI によるフォローアップレビューを実行する

修正を適用した後、対象を絞ったレビューを実行します。

codex exec "Review the updated <file_name> code. Check if the previous issues have been fixed: 1) issue description, 2) issue description, etc. Report any remaining issues or confirm the code is now correct."

これにより Codex は次のことができます。

• 特定の問題が解決されたかどうかに焦点を当てる
• 新たに生じた問題を特定する
• 合格／不合格を明確に確認する

結果を分析します。

• すべての問題を修正済み： ワークフローを完了としてマークし、成功を報告する
• 問題が残っている： 反復を続けるか判断する（制限を確認）

6. 反復回数の上限を設定する

最大反復回数： 2 回のレビューサイクル（初回＋再レビュー 1 回）

上限を設ける理由：

• 無限ループを防ぎます
• 人間の判断が必要な問題もあります
• 複雑な問題にはアーキテクチャの変更が必要です

最大回数に達した場合：

1. TodoWrite で現在のタスクのステータスを記録します
2. 修正した内容と残っている問題を要約します
3. Codex CLI の最終出力をユーザーに表示します
4. 次のどれを行うか、ユーザーに尋ねます：
   • 手動での修正を続ける
   • アプローチを調整する
   • 現在の状態を受け入れる

例

例 1：シンプルな修正ワークフロー

User: "Add a function to calculate fibonacci numbers and review it with Codex"

Claude:
1. Creates fibonacci function
2. Checks for git repo, runs git init if needed
3. Runs: codex exec "Review fibonacci.py for bugs, edge cases, and best practices."
4. Codex output:
   - **Bug** `fibonacci.py:5`: Missing input validation for negative numbers
   - **Maintainability** `fibonacci.py:10`: Consider adding type hints
5. Adds input validation and type hints
6. Runs: codex exec "Review updated fibonacci.py. Check if input validation added."
7. Codex output:
   **Resolved Checks**
   - `fibonacci.py:5` Input validation is now present
   - `fibonacci.py:10` Type hints added
8. Reports completion

例 2：複数回の反復（セキュリティ）

User: "Implement user authentication and run through Codex review"

Claude:
1. Implements auth system across multiple files
2. Runs: codex exec "Review auth.py, user.py, session.py for security vulnerabilities."
3. Codex output:
   - **Security** `auth.py:23`: Password stored without hashing
   - **Security** `session.py:45`: Session tokens don't expire
   - **Bug** `user.py:67`: SQL injection vulnerability
   - **Security** `auth.py:89`: Missing CSRF protection
   - **Bug** `user.py:12`: No input validation
4. Fixes all 5 issues
5. Runs targeted review: "Check if these issues are fixed: password hashing, session expiration, SQL injection, CSRF, input validation."
6. Codex output:
   **Resolved Checks**
   - All 5 issues confirmed fixed
   **Findings**
   - Minor style suggestions
7. Reached iteration limit (2 reviews)
8. Reports: "All critical security issues fixed. Minor style suggestions remain."
9. Asks user about style fixes

例 3：初回で成功

User: "Write a date formatter helper, review with Codex"

Claude:
1. Creates date formatting function with error handling
2. Runs: codex exec "Review utils/date_formatter.py for bugs, edge cases."
3. Codex output:
   No bugs or critical issues. Code properly handles:
   - None values with error messages
   - Invalid format strings
   - Timezone conversions
   **Suggestions**
   - Could add more detailed error messages
4. Reviews suggestion, decides optional for current use case
5. Marks task complete

統合に関する注意事項

Codex CLI コマンド

# Basic review
codex exec "Review <file_name> for bugs, security issues, best practices. Provide specific feedback with line numbers."

# With working directory
codex exec "Review <file_name>..." -C /path/to/project

# With specific model
codex exec "Review <file_name>..." -m gpt-5-codex

# Skip git check (not recommended)
codex exec "Review <file_name>..." --skip-git-repo-check

Git リポジトリの要件

# Check if in git repo
git status

# Initialize if needed
git init

# Alternative: skip check
codex exec "..." --skip-git-repo-check

エラー処理

Codex CLI が見つからない場合：

• 確認方法：which codex または codex --version
• Codex CLI を利用できないことをユーザーに伝えます
• 自動レビューを行わずにタスクを完了する方法を提案します

Git リポジトリのエラー：

• エラー：「Not inside trusted directory and --skip-git-repo-check not specified」
• 解決方法：git init を実行します
• 代替方法：--skip-git-repo-check を追加します

Codex CLI のエラー：

• よくあるエラー：
  • unexpected argument - 構文を確認し、codex review ではなく codex exec を使用します
  • 認証エラー - codex login が必要な場合があります
• パラメーターを修正して、もう一度試します
• エラーが続く場合は、ユーザーに指示を求めます

結果が曖昧な場合：

• 合格か不合格か判断できない場合は、慎重な側に判断を寄せます
• 「Key Issues」と「Suggestions」のセクションを確認します
• 出力をユーザーに表示し、説明を求めます

レビューに時間がかかる場合：

• Codex による複雑なレビューには 30～120 秒かかることがあります
• 適切なタイムアウトを設定します（120000ms を推奨）

ベストプラクティス

1. ワークフローの各ステップの追跡には、必ず TodoWrite を使用します
2. 各レビュー段階で Codex の出力を表示します
3. 修正内容を明確に説明します - 黙って修正することは避けます
4. 反復回数の上限を守ります - 無限ループを避けます
5. 機能を維持します - 機能を損なわずに問題に対処します
6. 不確かな場合は尋ねます - フィードバックが曖昧な場合はユーザーに相談します

カスタマイズオプション

• 反復回数の上限を調整する（デフォルト：レビュー 2 回）
• Codex CLI のカスタムコマンドを指定する
• Codex のルールを記述した設定ファイルを指定する
• レビューの対象に含める／除外するファイルを定義する
• 重大度のしきい値を設定する（エラーのみ、または警告も対象）

関連スキル

• testing-strategist：コードレビューを補完するテストスイートの作成
• security-engineer：手動でのセキュリティレビューと脅威モデリング
• quality-auditor：包括的な品質評価
• technical-writer：レビュー結果と改善内容の文書化

ツールと依存関係

必須：

• Codex CLI（OpenAI）
• Git（リポジトリのコンテキスト用）

推奨：

• TodoWrite ツール（進捗の追跡）
• Edit ツール（修正の適用）

成功のヒント

1. 最初のコードを丁寧に書きます - 出発点の品質が高いほど、反復回数を減らせます
2. レビューのプロンプトを具体的にします - 「これをレビューして」ではなく「ログイン関数の SQL インジェクションを確認して」と指定します
3. 関連するファイルをまとめます - ファイルごとではなく、認証システム全体をレビューします
4. 問題を一度にすべて修正します - 1 件ずつ修正するより効率的です
5. 対象を絞った追加のプロンプトを使います - 全般的なレビューではなく、特定の修正について尋ねます
6. いつ停止するかを見極めます - 人間の判断やアーキテクチャの変更が必要な問題もあります

---

スキルタイプ： 自動化 難易度： 中級 推定時間： タスクによって異なります（レビュー：1～2 分、修正：反復 1 回あたり 5～30 分） 統合： Codex CLI、Git