review-pr について
プルリクエストの修正可能なバグ、リグレッション、セキュリティリスク、テスト不足を調べるスキルです。変更を取得して重要な問題に絞り、レビューの送信は保留します。
主な機能
- 01PR番号またはURLの解析
- 02PR差分と変更ファイルの取得
- 03修正が必要な変更に注目
- 04生成物とベンダーファイルの除外
- 05保留中のレビューコメント作成
活用シーン
- 01PRのリグレッション確認
- 02セキュリティや性能リスクの特定
- 03変更コードのテスト不足の発見
インストール
コマンドをコピーしてターミナルで実行します。
$
npx skills add https:/ / github.com/ fcakyon/ claude-codex-settings --skill review-prインストール後、Claude Code で/review-prを入力して読み込まれたか確認します。
詳細
PR をレビューする
プルリクエストをレビューし、バグ、リグレッション、テスト不足、リスクの高い変更を確認します。
追加のテキストとともに明示的に呼び出された場合、そのテキストを PR 番号または URL として扱います。PR の参照が指定されていない場合は、現在のブランチから特定します。
プロセス
-
優先する実行方法
- サブエージェントを利用できる場合は、
github-dev:pr-reviewerを使用し、PR の参照を渡します。 - それ以外の場合は、以下の手動手順に従います。
- サブエージェントを利用できる場合は、
-
PR の参照を解析する
- PR 番号または URL が指定されている場合は、owner、repo、PR 番号を抽出します。
- 指定されていない場合は、
gh pr view --json number,headRefNameを使って現在のブランチから PR を自動検出します。
-
PR のデータを取得する
- 完全な diff には
gh pr diff <number>を使用します。 - 変更されたファイルの一覧には
gh pr view <number> --json filesを使用します。 .lock、.min.js、.min.css、dist/、build/、vendor/、node_modules/、_pb2.py、画像などの生成ファイルやベンダー提供ファイルは対象外にします。
- 完全な diff には
-
レビューの重点項目
- 修正が必要な問題のみ報告します。
- PR の変更のみをレビューし、変更されていないコードに既存する問題は報告しません。
- バグ、セキュリティ上の問題、破壊的変更、パフォーマンスの問題、エッジケース、テスト不足を優先します。
- 同じ根本原因に起因する関連問題はまとめます。
- 報告は簡潔で、重要度の高いものに絞ります。
-
レビューコメントのルール
- 保留中の PR コメントのみ作成し、レビューを自動的に送信または確定しないでください。
- GitHub の操作には
ghを使用します。 - コメントは小文字で始め、簡潔にし、可能な場合は文末の句読点を避けます。
- ボット向け、人間向けのどちらのコメントも平易な言葉で記述します。
出力形式
問題が見つかった場合は、重要度の高い順に、ファイルの参照と最終的な推奨事項 NEEDS_CHANGES を添えて報告します。
問題が見つからなかった場合は、APPROVE - No fixes required を返します。
コメント
Markdown 対応