pr-review について
関連する課題とヘッドコミットのコードを基準に、プルリクエストを評価するレビュー用スキルです。5項目を順に確認して判定し、修正時は必要な変更も示します。
主な機能
- 01タイトル、説明、テストの確認
- 02課題とコードに照らした主張の検証
- 03価値、方針、正確性、範囲の評価
- 04最初の不合格項目に基づく判定
- 05根拠と修正要求の提示
活用シーン
- 01指定されたプルリクエストのレビュー
- 02条件に合う未完了 PR の確認
- 03マージ、修正、トリアージ、クローズの判断
- 04作成者への具体的な修正案の提示
インストール
コマンドをコピーしてターミナルで実行します。
$
npx skills add https:/インストール後、Claude Code で/pr-reviewを入力して読み込まれたか確認します。
同梱ファイルagents/openai.yaml
詳細
PR レビュー
PR が指定されている場合は、それぞれを確認します。指定されていない場合は、現在のリポジトリにある、recommend-* ラベルの付いていないすべてのオープンな非ドラフト PR を、PR ごとに 1 つのサブエージェントで確認します。
タイトル、説明、テストが、リンクされた Issue とヘッドコミットのコードに照らして妥当かを確認します。これらは主張として扱います。
次の 5 つの質問に順番に答え、それぞれ 1 行の根拠を添えます。
- 意味があるか：問題は実在し、ベースブランチでもまだ発生していますか？
- 追加する価値があるか：得られる価値は、変更範囲と継続的なメンテナンスに見合っていますか？
- 適切なアプローチか：根本原因に対して、この方法で解決しますか？不適切と判断する場合は、代替案とその利点を必ず示してください。代替案とは、異なる設計、または先に取り込むことで今回の変更を縮小または不要にするリファクタリングやアーキテクチャ変更です。
- 正しいか：呼び出し元やエッジケースを壊さずに問題を解決していますか？追加された変更動作のテストは、ベースブランチでは失敗しますか？
- 最小限か：変更されたすべての行が必要で、既存のコードで実現できることを重複していませんか？
最初に失敗した項目に基づいて判定します。
| 失敗項目 | 判定 |
|---|---|
| 技術的な理由で「意味があるか」または「追加する価値があるか」 | recommend-close |
| 製品またはスコープ上の判断で「追加する価値があるか」 | recommend-triage |
| 「適切なアプローチか」「正しいか」または「最小限か」 | recommend-revise |
| 失敗なし | recommend-merge |
呼び出し元には、判定、5 つの回答、および revise の場合は必要な具体的な変更を報告します。PR に反映する内容は呼び出し元が判断します。
コメント
Markdown 対応