review-pr について
プルリクエストまたはブランチの全コミットをベースブランチと比較するコードレビュー用スキルです。リスク評価、呼び出し元やテストへの影響、高影響ファイルの確認を重視します。
主な機能
- 01変更のリスク評価
- 02呼び出し元とテストへの影響確認
- 03対象を絞ったレビューコンテキスト
- 04リスクと影響範囲を含むレビュー要約
活用シーン
- 01マージ前のブランチレビュー
- 02公開 API 変更で呼び出し元の更新漏れを確認
- 03テスト不足と影響を受けるフローの特定
MCP 対応
code-review-graph MCP ツールを使い、変更、依存関係、テスト、影響を受けるフローを分析します。
インストール
コマンドをコピーしてターミナルで実行します。
$
npx skills add https:/ / github.com/ tirth8205/ code-review-graph --skill review-prインストール後、Claude Code で/review-prを入力して読み込まれたか確認します。
詳細
PR のレビュー
プルリクエストまたはブランチのすべてのコミットを、ベースブランチと比較してレビューします。
手順
- 変更内容を特定します。ブランチ名がある場合は
git diff --name-only main...<branch>を実行します。ない場合は、現在のブランチをmain(またはmaster)と比較します。以下では、そのリファレンスをbaseとして使用します。 get_minimal_context_tool(task="review PR", base="main")を呼び出します。status: not_readyが返された場合は、build_or_update_graph_tool()を呼び出してから続行します。detect_changes_tool(base="main", detail_level="minimal")を呼び出し、ブランチ全体にわたるリスクスコア付きの変更関数、テストの欠落、影響を受けるフローを取得します。- 高リスクの関数については、
query_graph_tool(pattern="callers_of", target="<function>")とquery_graph_tool(pattern="tests_for", target="<function>")を呼び出します。呼び出し元が更新されていない公開 API の変更にはフラグを立てます。 - ソースのスニペットが必要な場合は
get_review_context_tool(base="main")を、依存先の完全な一覧が必要な場合はget_impact_radius_tool(base="main")を呼び出します。これは、手順 3 で高リスクの項目が不明確なままの場合に限ります。 - 影響度の高いファイルから順に、変更されたソースを読みます。変更に必要な場合を除き、ファイル全体は読み込まないでください。
出力
## PR Review: <title>
### Summary
<1-3 sentences>
### Risk
- Overall: Low / Medium / High
- Blast radius: <n> files, <m> functions
- Test coverage: <covered> of <changed> changed functions
### Files
#### <path>
- Changes: <what>
- Impact: <who depends on it>
- Issues: <bugs, style, concerns>
### Missing Tests
- <function> in <file>
### Recommendations
1. <action>
ヒント
semantic_search_nodes_toolを使って、PR で見落とされた可能性のある関連コードを見つけます。
コメント
Markdown 対応