コードレビューのリクエスト

superpowers: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: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:code-reviewer subagent]
  WHAT_WAS_IMPLEMENTED: Verification and repair functions for conversation index
  PLAN_OR_REQUIREMENTS: Task 2 from docs/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