qe-pr-review について
qe-pr-reviewは、AQE/QEスキルを扱う貢献者向けのGitHubプルリクエストレビュー手順です。対象範囲や信頼ティア、コード品質を確認し、実行可能なレビューコメントの作成も案内します。
主な機能
- 01差分全体とPR説明の確認
- 02AQE/QEの対象範囲チェック
- 03信頼ティア割り当ての検証
- 04コード品質とセキュリティの確認
- 05建設的なレビュー連絡の指針
活用シーン
- 01AQE中心のGitHubプルリクエストのレビュー
- 02スキルの範囲と信頼ティアの確認
- 03テスト・セキュリティ・本番環境の懸念の特定
インストール
コマンドをコピーしてターミナルで実行します。
$
npx skills add https:/ / github.com/ proffesor-for-testing/ agentic-qe --skill qe-pr-reviewインストール後、Claude Code で/qe-pr-reviewを入力して読み込まれたか確認します。
詳細
PR レビューワークフロー
適切な AQE のスコープ境界を守り、明確なコミュニケーションと実行可能なフィードバックを添えて、プルリクエストをレビューします。
引数
<pr-number>— レビューする GitHub PR 番号です。省略された場合は、ユーザーに確認してください。
手順
1. 差分全体を読む
gh pr diff <pr-number>
gh pr view <pr-number>
差分全体と PR の説明を読みます。流し読みせず、変更されたファイルをすべて読んでください。
2. スコープの確認
- AQE/QE スキルのみを分析します(Claude Flow プラットフォームスキルは対象外)。
- 除外するプラットフォームスキル:v3-, flow-nexus-, agentdb-, reasoningbank-, swarm-, github-, hive-mind-advanced, hooks-automation, iterative-loop, stream-chain, skill-builder, sparc-methodology, pair-programming, release, debug-loop, aqe-v2-v3-migration
- PR がスキルに触れている場合、件数とスコープが想定(約78個の AQE スキル)と一致することを確認します。
- AQE に焦点を当てた PR にプラットフォームスキルの変更が混入している可能性があれば、指摘します。
3. 変更内容を要約する
変更内容とその理由を、ユーザーに分かりやすく要約します。
- 実装の詳細ではなく、成果を重視します。
- 過度に専門的な用語は避けます。
- 箇条書き3〜5項目にまとめます。
4. Trust Tier を検証する
スキルに変更がある場合、trust_tier の割り当てを検証します。
- tier 3 = 評価インフラ(evals/, schemas/, scripts/)がある
- tier 2 = テスト済みだが、評価フレームワークはない
- tier 1 = 未テスト
- 不整合を指摘します(例:evals があるスキルが tier 2 なら、tier 3 であるべきです)。
5. コード品質をレビューする
以下を確認します。
- ハードコードされたバージョン文字列
- 本番環境の安全性に関する懸念(アダプターの変更、破壊的変更)
- 新しいコードに対するテストカバレッジの不足
- セキュリティ上の問題(秘密情報の露出、インジェクションのリスク)
6. レビューを投稿する
gh pr review <pr-number> --body "review comments"
コミュニケーションのルール
- 建設的で実行可能なフィードバックを心がけます。
- 成果を重視し、問題点ではなく、作成者が何をすべきかを伝えます。
- 細かなコメントを多数投稿するのではなく、関連するコメントをまとめます。
- 軽微な提案を添えて承認する場合は、REQUEST_CHANGES ではなく、コメント付きの APPROVE を使用します。
コメント
Markdown 対応