review-it

CC Review

コミットまたはリリースの前に、最終確認として自動コードレビューを実行します。複数のAIコーディングエージェントに対応しています。

次の場合に使用します。

• ユーザーがコードレビュー／review-it／autoreviewを依頼したとき
• 重要なコード変更の後、最終確認／コミット／リリースの前
• 修正後にローカルブランチまたはPRブランチをレビューするとき

対応エージェント

エージェント	レビューコマンド	備考
Claude Code	/review	組み込み機能です。未コミットの変更または差分をレビューできます
Codex	codex review	差分ファイルを渡すか、自動検出に任せます
OpenCode	/review	Claude Codeと同じです
DeepSeek TUI	/review または手動で差分をレビュー	分析用に差分の内容を渡します
Antigravity CLI	/code-review	組み込みのスラッシュコマンドで、差分を自動検出します

基本方針

• レビュー結果は参考情報として扱います。盲目的に適用してはいけません。
• すべての指摘について、実際のコードパスと関連ファイルを読んで検証します。
• 指摘が外部の動作に依存する場合は、依存関係のドキュメント／ソース／型定義を確認します。
• 非現実的なエッジケース、推測に基づくリスク、広範な書き換え、コードベースを過度に複雑にする修正は却下します。
• 適切な責任境界で小さな修正を優先します。バグの分類を明確に改善する場合を除き、リファクタリングは行いません。
• レビューで採用可能な対応必須の指摘がなくなるまで続けます。
• レビューを受けてコードを修正した場合は、対象を絞ったテストを再実行し、レビューも再実行します。
• 対応必須の指摘がなく、レビューが問題なしとなった時点で終了します。
• 指摘を意図的な仕様または修正する価値がないものとして却下する場合、将来のレビュアーが知るべき実際の不変条件や責任分担を説明するときに限り、短いインラインコードコメントを追加します。
• レビューだけを目的にプッシュしてはいけません。ユーザーからプッシュ／リリース／PR更新を依頼された場合に限り、プッシュします。

レビューの重点項目

現在の差分をレビューしてください。構文や明らかなバグだけでなく、以下の観点を重点的に確認し、最後に深刻度の高い順に並べてください。

1. 隠れた副作用 — 変更によって、見えにくい箇所で連鎖的な影響が生じないか。共有状態、グローバル変数、または外部依存の動作を変更していないか。
2. 互換性の破壊 — APIシグネチャ、データ構造、設定ファイル形式、またはコマンドラインインターフェースを変更していないか。既存の呼び出し元に影響しないか。
3. エッジケース — null／空値／空のコレクション、極端に大きい値／小さい値、並行処理／競合状態、例外経路が適切に処理されているか。
4. パフォーマンスリスク — 不要なループのネスト、N+1クエリ、大きなオブジェクトの割り当て、ブロッキングI/O、またはロック競合が発生していないか。
5. セキュリティリスク — インジェクション、権限の逸脱、機密情報の漏えい、安全でないデシリアライズ、または依存関係のバージョンに起因する脆弱性がないか。
6. 誤解を招く命名 — 変数／関数／型の名前が実際の動作と一致しているか。実態にそぐわない、または意味が曖昧な名前がないか。
7. テスト不足 — 重要な経路、境界条件、エラー処理のテストカバレッジが不足していないか。既存のテストで期待する動作を実際に検証できているか。
8. 将来の保守コスト — 不要な抽象化、重複コード、暗黙的な結合、または追跡しにくい制御フローが導入されていないか。後から変更する人が理解しやすく、修正しやすいか。

対象を選択

Claude Code／OpenCode／DeepSeek TUI

ローカルの未コミット変更（デフォルト — /review は未コミットの変更をレビューできます）：

/review

ブランチ／PRの作業 — ベースとの差分全体をレビューします。

まず差分を生成し、その後レビューします。

git diff origin/main...HEAD > /tmp/review-it.diff

次に、焦点を絞ったプロンプトを使って差分ファイルをレビューします。

/review the changes in /tmp/review-it.diff against origin/main

オープンなPRがある場合は、実際のベースを使用します。

base=$(gh pr view --json baseRefName --jq .baseRefName)
git diff "origin/$base"...HEAD > /tmp/review-it.diff

Antigravity CLI (agy)

ローカルの未コミット変更：

/code-review

ブランチ／PRの作業 — ベースとの差分全体をレビューします。

git diff origin/main...HEAD > /tmp/review-it.diff

次に、差分をレビューコマンドに渡します。

/code-review the changes in /tmp/review-it.diff against origin/main

オープンなPRがある場合は、実際のベースを使用します。

base=$(gh pr view --json baseRefName --jq .baseRefName)
git diff "origin/$base"...HEAD > /tmp/review-it.diff

Codex

# Review uncommitted changes
codex review

# Review branch diff
git diff origin/main...HEAD > /tmp/review-it.diff
codex review /tmp/review-it.diff

並行して行う最終確認

フォーマットによって行の位置が変わる可能性がある場合は、まずフォーマットを行います。その後、テストとレビューを並行して実行できます。

scripts/review-it --parallel-tests "<focused test command>"

注意点：テストによってコードを変更すると、レビュー結果が古くなることがあります。テストまたはレビューを受けてコードを編集した場合は、影響を受けるテストを再実行し、採用可能な対応必須の指摘がなくなるまでレビューを再実行します。

未コミット変更とブランチのレビュー

適切なモードを選択します。

• 未コミットの変更（ステージ済み／未ステージ）：/review を直接使用します（Antigravity：/code-review、Codex：codex review）
• コミット済み、未プッシュ：git diff origin/main...HEAD とレビューを使用します
• プッシュ済み／PR：コミット済みの場合と同様に、PRのベースに対してレビューします
• クリーンな作業ツリー：レビュー対象が本当に何もなければ、レビューをスキップします

ヘルパー

テストとレビューを並行して実行するためのバンドルされたヘルパースクリプト：

~/.claude/skills/review-it/scripts/review-it --help

ヘルパーの機能：

• --agent auto を使用して、実行中のエージェント（Claude Code、Antigravity CLI、Codex）を検出します
• 未コミットの変更をレビューするか、ブランチの差分をレビューするかを検出します
• ブランチモードの場合：origin/main（または PR のベース）との差分を生成してから、レビューを実行します
• --parallel-tests によるテストとレビューの並行実行をサポートします
• --dry-run による、使用されるコマンドの確認をサポートします
• レビューで問題が見つからなかった場合は、review-it clean: no accepted/actionable findings reported と出力します

最終レポート

以下を含めてください：

• レビュー対象（未コミットの変更／ブランチ／PR のベース）
• 実行したテスト／検証
• 採用または却下した指摘と、その簡潔な理由
• レビューで問題が見つからなかった結果、または残った指摘を意図的に却下した理由

最終レポートの文言を改善することだけを目的に、レビューを再実行しないでください。レビューが実行され、対応が必要な指摘がなかった場合は、問題なしとして報告してください。