用途
该技能用于在软件开发过程中请求代码审查,目的是在问题扩散之前发现并修复它们。它强调“尽早审查、频繁审查”的核心原则,通过调度 superpowers:code-reviewer 子代理来执行审查任务。
主要能力
- 明确规定了必须请求审查的时机:子代理驱动开发中每个任务完成后、完成重大功能后、合并到主分支前。
- 列出了可选但有价值的审查时机:遇到困难时(获取新视角)、重构前(基线检查)、修复复杂 bug 后。
- 提供了请求审查的具体步骤:获取 git SHA、调度 code-reviewer 子代理、根据反馈采取行动。
- 定义了反馈处理优先级:立即修复 Critical 问题、在继续前修复 Important 问题、记录 Minor 问题供后续处理、如果审查者有误则有理有据地反驳。
- 给出了与不同工作流(子代理驱动开发、执行计划、临时开发)的集成方式。
- 列出了需要避免的红旗行为,如因“简单”而跳过审查、忽略 Critical 问题、带着未修复的 Important 问题继续、与有效的技术反馈争论。
使用方式
-
获取 git SHA: BASE_SHA=$(git rev-parse HEAD~1) # 或 origin/main HEAD_SHA=$(git rev-parse HEAD)
-
调度 code-reviewer 子代理:使用 Task 工具,类型为 superpowers:code-reviewer,填写 code-reviewer.md 模板。模板占位符包括:
- {WHAT_WAS_IMPLEMENTED} - 刚刚构建的内容
- {PLAN_OR_REQUIREMENTS} - 应该实现的功能
- {BASE_SHA} - 起始提交
- {HEAD_SHA} - 结束提交
- {DESCRIPTION} - 简要总结
-
根据反馈采取行动:
- 立即修复 Critical 问题
- 在继续前修复 Important 问题
- 记录 Minor 问题供后续处理
- 如果审查者有误,有理有据地反驳
示例流程:完成某个任务后,获取 BASE_SHA 和 HEAD_SHA,调度子代理并填写模板,子代理返回审查结果(包括优点、问题、评估),然后修复问题并继续。
适用场景
- 子代理驱动开发:每个任务后审查,防止问题累积,在进入下一个任务前修复。
- 执行计划:每批任务(3个)后审查,获取反馈、应用、继续。
- 临时开发:合并前审查,遇到困难时审查。
限制
- 该技能依赖于 superpowers:code-reviewer 子代理和 Task 工具。
- 需要 git 环境来获取 SHA。
- 模板文件位于 requesting-code-review/code-reviewer.md。
- 如果审查者有误,需要提供技术推理、展示代码/测试证明其工作,或请求澄清。
- 避免的行为包括:因“简单”而跳过审查、忽略 Critical 问题、带着未修复的 Important 问题继续、与有效的技术反馈争论。