请求代码审查

调用 superpowers:code-reviewer 子智能体，提前发现潜在问题，防止问题扩散。

核心原则：尽早审查，频繁审查。

何时请求审查

必须执行的情况：

• 子智能体驱动开发中，每完成一个任务后
• 完成主要功能后
• 合并至 main 分支前

可选但高价值的场景：

• 遇到卡点时（获取新视角）
• 重构前（建立基线检查）
• 修复复杂 Bug 后

如何发起请求

1. 获取 Git SHA 值：

BASE_SHA=$(git rev-parse HEAD~1)  # 或 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 级别问题记录下来，后续处理
• 若审查者判断有误，需基于技术理由提出异议

示例

[刚完成 Task 2：添加验证函数]

你：在继续之前，先发起一次代码审查。

BASE_SHA=$(git log --oneline | grep "Task 1" | head -1 | awk '{print $1}')
HEAD_SHA=$(git rev-parse HEAD)

[调用 superpowers:code-reviewer 子智能体]
  WHAT_WAS_IMPLEMENTED: 对话索引的验证与修复函数
  PLAN_OR_REQUIREMENTS: docs/plans/deployment-plan.md 中的 Task 2
  BASE_SHA: a7981ec
  HEAD_SHA: 3df7661
  DESCRIPTION: 新增 verifyIndex() 和 repairIndex()，支持 4 类问题类型

[子智能体返回结果]：
  优势：架构清晰，具备真实测试
  问题：
    Important：缺少进度指示器
    Minor：报告间隔中使用了魔法数字（100）
  评估结论：可继续推进

你：[修复进度指示器]
[进入 Task 3]

与工作流的集成

子智能体驱动开发：

• 每个任务完成后均需审查
• 在问题累积前及时捕获
• 修复完毕后再进入下一任务

执行计划类任务：

• 每完成一批（3 个任务）后进行一次审查
• 获取反馈 → 应用修改 → 继续执行

临时性开发：

• 合并前必须审查
• 卡点时主动请求审查

危险信号（Red Flags）

绝对禁止的行为：

• 以“代码很简单”为由跳过审查
• 忽略 Critical 级别问题
• 在 Important 级别问题未修复的情况下继续推进
• 对有效的技术反馈进行无依据争辩

当审查者判断有误时：

• 基于技术理由提出异议
• 提供相关代码或测试用例，证明其实现正确
• 请求进一步澄清

模板文件详见：requesting-code-review/code-reviewer.md