kryota-dev/cc-review

Claude Code レビュー

claude -p（非対話型モード）で別の Claude Code インスタンスを起動し、独立したコンテキストでコードレビューを実行する。 現在のセッションに蓄積されたバイアスのない、フレッシュな視点でレビューが得られる。 プロジェクトの CLAUDE.md は自動的に読み込まれるため、プロジェクト固有の規約を踏まえたレビューになる。

SSOT としての位置づけ

本 skill は コードレビュー用 claude -p 起動の Single Source of Truth。multi-review skill から並列呼び出しされる場合も、本ファイルの起動オプション・プロンプトテンプレート・コマンド例に従う。multi-review 側で重複定義しない。

起動オプション

| オプション | 値 | 理由 | |------------|-----|------| | --allowedTools | "Read,Glob,Grep" | 読み取り専用。周辺コードの参照を許可しつつ書き込みは禁止 | | --max-turns | 10 | 差分読解 + 周辺コード参照を許容。深掘り余地確保のため単独/並列共通で 10 | | --effort | max | レビュー品質を最大化 | | --output-format | text | 人間が読みやすいテキスト形式 |

注意: --bare は使わない（OAuth 認証が通らなくなる）。--model はデフォルト（ユーザー設定を継承）。

引数の解釈

$ARGUMENTS を以下の優先順で判定する:

1. PR番号 (^\d+$ または ^#\d+$): gh pr diff <番号> で差分取得
2. PR URL (github.com を含む URL): URL から PR 番号を抽出し、gh pr diff で差分取得
3. --staged: git diff --cached でステージング済みの差分を取得
4. ファイルパス (. を含む拡張子、または / を含む): cat <path> でファイル内容を取得
5. ブランチ名 (上記に該当しない文字列): git diff <branch>...HEAD で差分取得
6. 引数なし: デフォルトブランチとの差分を取得

プロンプトのルール

重要: claude -p に渡すプロンプトには、以下の指示を必ず含めること:

「確認や質問は不要です。具体的な指摘と修正案を自主的に出力してください。」

プロンプトテンプレート

あなたはシニアソフトウェアエンジニアです。以下のコード差分をレビューしてください。

## レビュー観点

1. バグ・論理エラー: 明らかなバグ、エッジケースの未処理、オフバイワンエラー
2. 設計・アーキテクチャ: 既存パターンとの一貫性、適切な抽象化、責務の分離
3. 可読性・保守性: 命名の適切さ、コメントの過不足、コード複雑度
4. エラーハンドリング: 例外処理の網羅性、エラーメッセージの適切さ
5. パフォーマンス: N+1問題、不要な計算・再描画、メモリリーク
6. テスト: テストの有無、カバレッジ、エッジケースのテスト

## 出力形式

各指摘を以下のカテゴリで分類:
- [MUST] 修正必須（バグ、セキュリティ、設計違反）
- [SHOULD] 修正推奨（品質向上、可読性改善）
- [NITS] 軽微な提案（命名、フォーマット）
- [GOOD] 良い実装（称賛すべき点）

各指摘に、ファイル名:行番号、問題の説明、具体的な修正案を含めてください。
最後にレビューサマリー（カテゴリ別件数 + 総合評価）を付けてください。

## 重要: 技術的主張の確実性

ライブラリ・フレームワーク・言語仕様について断定する場合（例: 「X は Y をサポートしていない」「API Z は deprecated」）、確信が持てないなら必ず本文に **「（未確認）」** または **「（要検証）」** と明示してください。学習データのカットオフ後の変更を見落とすリスクがあるため、自信満々に誤情報を出力するのは避けてください。

確認や質問は不要です。具体的な指摘と修正案を自主的に出力してください。

実行手順

1. 引数を解析してレビュー対象を特定する
2. 差分を取得する。差分が空の場合は「レビュー対象の差分がありません」と報告して終了
3. プロンプトを構築する（heredoc 使用。シェルのクォート問題を回避）
4. claude -p を実行する（Bash の timeout は 300000ms = 5分 に設定）
5. 結果をユーザーに表示する

コマンド例

# PR差分レビュー
gh pr diff 123 | claude -p \
  --allowedTools "Read,Glob,Grep" \
  --max-turns 10 \
  --effort max \
  --output-format text \
  "$(cat <<'PROMPT'
あなたはシニアソフトウェアエンジニアです。以下のコード差分をレビューしてください。
...（プロンプトテンプレート）
PROMPT
)"

# ブランチ差分レビュー
git diff main...HEAD | claude -p \
  --allowedTools "Read,Glob,Grep" \
  --max-turns 10 \
  --effort max \
  --output-format text \
  "$(cat <<'PROMPT'
...（プロンプトテンプレート）
PROMPT
)"

# デフォルトブランチとの差分（引数なし）
DEFAULT_BRANCH=$(git rev-parse --abbrev-ref origin/HEAD 2>/dev/null | sed 's|origin/||')
git diff "${DEFAULT_BRANCH:-main}...HEAD" | claude -p ...

注意事項

コスト管理

• デフォルトモデル（ユーザー設定継承）で実行される。opus は高品質だがコストが高い
• コストを抑えたい場合は --model sonnet を追加する。ただし CLAUDE.md が大きいプロジェクトでは "Prompt is too long" エラーが発生する場合がある（sonnet の context window は 200k）
• --max-turns を減らすことでもコスト削減可能

エラーハンドリング

• claude コマンドが見つからない場合: Claude Code CLI のインストールを案内
• 空の差分: 事前チェックで即座に報告
• タイムアウト: Bash の timeout 300000ms で保護。超過時はリトライまたは --max-turns 削減を提案
• "Error: Reached max turns": レビュー範囲が広すぎる。--max-turns を増やすか、対象を絞る