requesting-code-review について
作業内容とコミット範囲を示して、superpowers-ruby:code-reviewer サブエージェントにレビューを依頼するスキルです。依頼するタイミングと、指摘に対応してから開発を続けたりマージしたりする手順を説明します。
主な機能
- 01タスク、機能の完了後とマージ前にレビュー
- 02実装内容とコミット SHA の提示
- 03指摘事項への優先順位付けと対応
- 04デバッグやリファクタリング時のレビュー依頼
活用シーン
- 01サブエージェント駆動開発で各タスクを確認
- 02main へのマージ前に機能をレビュー
- 03複雑なバグについて新たな視点を得る
インストール
コマンドをコピーしてターミナルで実行します。
$
npx skills add https:/ / github.com/ lucianghinda/ superpowers-ruby --skill requesting-code-reviewインストール後、Claude Code で/requesting-code-reviewを入力して読み込まれたか確認します。
同梱ファイルcode-reviewer.md
詳細
コードレビューを依頼する
superpowers-ruby:code-reviewer サブエージェントを呼び出して、問題が連鎖する前に検出します。レビュー担当者には評価に必要なコンテキストだけを正確に渡し、セッションの履歴は一切渡しません。これにより、レビュー担当者はあなたの思考過程ではなく成果物に集中でき、あなた自身のコンテキストもその後の作業に保つことができます。
基本原則: 早めに、頻繁にレビューします。
レビューを依頼するタイミング
必須:
- サブエージェント駆動開発では、各タスクの完了後
- 大きな機能の完成後
- main にマージする前
任意ですが有益:
- 行き詰まったとき(新たな視点を得る)
- リファクタリングの前(ベースラインを確認する)
- 複雑なバグの修正後
依頼方法
1. git SHA を取得します:
BASE_SHA=$(git rev-parse HEAD~1) # or origin/main
HEAD_SHA=$(git rev-parse HEAD)
2. code-reviewer サブエージェントを呼び出します:
Task ツールで superpowers-ruby:code-reviewer タイプを使用し、code-reviewer.md のテンプレートに記入します。
プレースホルダー:
{WHAT_WAS_IMPLEMENTED}- 直近で実装した内容{PLAN_OR_REQUIREMENTS}- 実装で満たすべき内容{BASE_SHA}- 開始時のコミット{HEAD_SHA}- 終了時のコミット{DESCRIPTION}- 簡単な概要
3. フィードバックに対応します:
- Critical の問題は直ちに修正します
- Important の問題は先に進む前に修正します
- Minor の問題は後で対応できるよう記録します
- レビュー担当者が間違っている場合は、理由を示して反論します
例
[Just completed Task 2: Add verification function]
You: Let me request code review before proceeding.
BASE_SHA=$(git log --oneline | grep "Task 1" | head -1 | awk '{print $1}')
HEAD_SHA=$(git rev-parse HEAD)
[Dispatch superpowers-ruby:code-reviewer subagent]
WHAT_WAS_IMPLEMENTED: Verification and repair functions for conversation index
PLAN_OR_REQUIREMENTS: Task 2 from docs/superpowers/plans/deployment-plan.md
BASE_SHA: a7981ec
HEAD_SHA: 3df7661
DESCRIPTION: Added verifyIndex() and repairIndex() with 4 issue types
[Subagent returns]:
Strengths: Clean architecture, real tests
Issues:
Important: Missing progress indicators
Minor: Magic number (100) for reporting interval
Assessment: Ready to proceed
You: [Fix progress indicators]
[Continue to Task 3]
ワークフローとの連携
サブエージェント駆動開発:
- 各タスクの後にレビューします
- 問題が積み重なる前に検出します
- 次のタスクに進む前に修正します
計画の実行:
- 各バッチ(3 タスク)の後にレビューします
- フィードバックを得て、反映してから続行します
アドホック開発:
- マージ前にレビューします
- 行き詰まったときにレビューします
危険信号
次のことは決してしないでください:
- 「簡単だから」とレビューを省略する
- Critical の問題を無視する
- 未修正の Important な問題がある状態で先に進む
- 妥当な技術的フィードバックに反論する
レビュー担当者が間違っている場合:
- 技術的な根拠を示して反論します
- 正しく動作することを証明するコードやテストを示します
- 説明を求めます
テンプレートについては、requesting-code-review/code-reviewer.md を参照してください。
コメント
Markdown 対応