用途
本技能用于在完成任务、实现主要功能或合并到主分支之前发起代码审查,借助 superpowers:code-reviewer 子智能体在问题扩散之前将其捕获。核心原则是:尽早审查,频繁审查。
何时请求审查
强制场景:
- 子智能体驱动开发中的每个任务完成后
- 完成主要功能后
- 合并到 main 之前
可选但有价值的场景:
- 卡住时(获取新视角)
- 重构前(基线检查)
- 修复复杂 bug 后
使用方式
- 获取 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 问题以便稍后处理
- 如果审查者有误,可带有理由地反驳
示例
[刚完成任务 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 中的任务 2 BASE_SHA: a7981ec HEAD_SHA: 3df7661 DESCRIPTION: 添加了 verifyIndex() 和 repairIndex(),包含 4 种问题类型
[子智能体返回]: 优点:架构清晰,有真实测试 问题: Important: 缺少进度指示器 Minor: 报告间隔使用了魔法数字(100) 评估:可以继续
你:[修复进度指示器] [继续任务 3]
与工作流的集成
子智能体驱动开发:
- 每个任务之后都审查
- 在问题累积之前将其捕获
- 在进入下一个任务之前修复
执行计划:
- 每批(3 个任务)之后审查
- 获取反馈、应用、继续
临时开发:
- 合并前审查
- 卡住时审查
红旗行为
绝不:
- 因为“很简单”就跳过审查
- 忽略 Critical 问题
- 带着未修复的 Important 问题继续推进
- 与有效的技术反馈争辩
如果审查者有误:
- 以技术理由反驳
- 展示能证明其可行的代码/测试
- 请求澄清
模板见:requesting-code-review/code-reviewer.md