请求代码审查

调用 code-reviewer 子代理，在问题扩散前及时发现缺陷。审查者获得的是专为评估而精心构建的上下文——而非您的会话历史。这确保审查者聚焦于交付成果本身，而非您的思考过程，同时完整保留您自身的上下文，以便持续开展后续工作。

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

何时请求审查

必须执行的情形：

• 子代理驱动开发中，每完成一项任务后
• 完成重大功能开发后
• 合并至 main 分支前

可选但极具价值的情形：

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

如何发起审查请求

1. 获取 SVN 版本号：

BASE_REV=$(svn info | grep 'Revision:' | awk '{print $2}')  # 或上一版本号
HEAD_REV=$(svn info | grep 'Revision:' | awk '{print $2}')  # 当前版本号
# 示例：BASE_REV=100，HEAD_REV=105

2. 调用 code-reviewer 子代理：

使用 Task 工具，指定类型为 superpowers:code-reviewer，并填写模板文件 code-reviewer.md

占位符说明：

• {WHAT_WAS_IMPLEMENTED} — 您刚刚实现的内容
• {PLAN_OR_REQUIREMENTS} — 其应满足的设计方案或需求
• {BASE_REV} — 起始版本号
• {HEAD_REV} — 结束版本号
• {DESCRIPTION} — 简要摘要

3. 响应审查反馈：

• 立即修复 Critical 级别问题
• 在继续下一步前修复 Important 级别问题
• 将 Minor 级别问题记录下来，留待后续处理
• 若审查意见有误，请基于技术理由提出异议

示例

[刚完成任务 2：添加校验函数]

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

BASE_REV=$(svn info | grep 'Revision:' | awk '{print $2}')  # 上一版本号
HEAD_REV=$(svn info | grep 'Revision:' | awk '{print $2}')  # 当前版本号

[调用 superpowers:code-reviewer 子代理]
  WHAT_WAS_IMPLEMENTED: 对话索引的校验与修复函数
  PLAN_OR_REQUIREMENTS: docs/superpowers/plans/deployment-plan.md 中的任务 2
  BASE_REV: 100
  HEAD_REV: 105
  DESCRIPTION: 新增 verifyIndex() 和 repairIndex()，支持 4 类问题类型

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

您：[修复进度指示器]
[进入任务 3]

与工作流的集成

子代理驱动开发：

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

执行计划（Plans）：

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

临时性开发（Ad-Hoc Development）：

• 合并前必须审查
• 遇到阻塞时可主动发起审查

危险信号（Red Flags）

绝对禁止的行为：

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

若审查者判断有误：

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

模板详见：code-reviewer.md