reviewing-code について
Codexを使ってコード品質、バグ、セキュリティ、実装パターンを確認するコードレビュー用スキルです。レビューの限界や、ノイズや見落としを減らす方法も説明します。
主な機能
- 01コード品質とパターンの確認
- 02バグや境界条件の検出
- 03セキュリティ上の問題の確認
- 04レビューの制約への対処
活用シーン
- 01Goコードの慣用的な書き方の確認
- 02コードのセキュリティ脆弱性の調査
- 03パフォーマンス上の問題のレビュー
MCP 対応
`mcp__codex__spawn_agent` を使ってコードレビューを実行します。
インストール
コマンドをコピーしてターミナルで実行します。
$
npx skills add https:/ / github.com/ julianobarbosa/ claude-code-skills --skill reviewing-codeインストール後、Claude Code で/reviewing-codeを入力して読み込まれたか確認します。
詳細
Codex によるコードレビュー
コードレビューには mcp__codex__spawn_agent を使用します。
使用する場面
- コードの品質とパターンをレビューする
- 潜在的なバグやエッジケースを見つける
- ベストプラクティスに沿っているか検証する
- セキュリティ上の問題を確認する
使用方法
{
"prompt": "Review this code for [quality/bugs/security]: [code or file]"
}
プロンプトの例
- 「コード品質と Go のベストプラクティスの観点からレビューしてください:[code]」
- 「セキュリティの脆弱性がないか分析してください:[code]」
- 「パフォーマンス上の問題がないかレビューしてください:[code]」
- 「これは慣用的なパターンに従っていますか?[code]」
注意点
- Codex のレビューは非決定的です — 同じコードでも呼び出し方が異なると優先順位が変わります。再現性が重要な場合は、モデルのバージョンを固定してください。
- Codex は、ステージ済みだが未コミットの変更をデフォルトでは認識しません —
--include-stagedまたはパイプで渡す diff を使って、明示的に含めてください。 - diff のみのレビューでは、ファイルをまたぐ影響を見落とします — 関数名を変更しても、別のファイルにあるその関数の呼び出し箇所は diff に含まれないため、変更の影響範囲が明らかになりません。
- レビュー結果は、ユーザーが定義した優先順位ではなく、Codex 内部の重大度ヒューリスティックに基づいて並べられます — プロジェクト固有の懸念に合わせて、手動で優先順位を付け直してください。
- 空白文字のみの diff でもレビューが実行されます —
--ignore-whitespaceを指定しないと、不要な結果が生じます。 - 大きな diff(500 LOC 超)は、行ごとにレビューされず要約されます — ファイル群ごとに焦点を絞ってレビューを分けてください。
コメント
Markdown 対応