用途
Requesting Code Review 用于在完成开发任务、实现重大功能或合并前,验证工作是否符合需求。其核心原则是:尽早审查、频繁审查(Review early, review often)。
主要能力
该技能会派发一个代码审查子代理,在问题级联扩散之前将其捕获。审查者获得的是为评估精确构造的上下文,而不是当前会话的历史记录。这样可以让审查者专注于工作产出,而不是你的思考过程,同时也保留你自己的上下文以便继续工作。
何时请求审查
强制性的时机:
- 子代理驱动开发中,每个任务完成后
- 完成重大功能后
- 合并到主分支(main)前
可选但有价值的时机:
- 卡住时(获取新视角)
- 重构前(基线检查)
- 修复复杂 Bug 后
使用方式
第一步,获取 git SHA:
BASE_SHA=$(git rev-parse HEAD~1) # 或 origin/main
HEAD_SHA=$(git rev-parse HEAD)
第二步,派发代码审查子代理:
使用 Task 工具,类型为 general-purpose,填写 code-reviewer.md 中的模板。
占位符:
- {DESCRIPTION} - 对所构建内容的简要总结
- {PLAN_OR_REQUIREMENTS} - 它应该做什么
- {BASE_SHA} - 起始提交
- {HEAD_SHA} - 结束提交
第三步,根据反馈行动:
- 立即修复 Critical 问题
- 继续之前先修复 Important 问题
- 记录 Minor 问题以便后续处理
- 如果审查者有误,带理由进行反驳
示例
[刚完成 Task 2:添加验证函数]
你:让我继续之前先请求代码审查。
BASE_SHA=$(git log --oneline | grep "Task 1" | head -1 | awk '{print $1}')
HEAD_SHA=$(git rev-parse HEAD)
[派发代码审查子代理]
- DESCRIPTION: 添加了 verifyIndex() 和 repairIndex(),包含 4 种问题类型
- PLAN_OR_REQUIREMENTS: 来自 docs/superpowers/plans/deployment-plan.md 的 Task 2
- BASE_SHA: a7981ec
- HEAD_SHA: 3df7661
[子代理返回]:
- 优点:架构清晰,有真实的测试
- 问题:
- Important:缺少进度指示器
- Minor:报告间隔使用了魔法数字(100)
- 评估:可以继续推进
你:[修复进度指示器] [继续到 Task 3]
与工作流的集成
子代理驱动开发:
- 在每一个任务之后审查
- 在问题叠加之前捕获它们
- 在进入下一个任务之前修复
执行计划:
- 在每个任务之后或自然检查点进行审查
- 获取反馈、应用、继续
临时开发:
- 合并前审查
- 卡住时审查
危险信号(Red Flags)
绝不要:
- 因为“它很简单”就跳过审查
- 忽略 Critical 问题
- 在 Important 问题未修复的情况下继续
- 与有效的技术反馈争辩
如果审查者有误:
- 带技术理由进行反驳
- 展示能证明其可用的代码/测试
- 请求澄清
限制
模板位于:requesting-code-review/code-reviewer.md。关于具体安装方式、支持的平台细节、收费与价格信息,现有资料暂未提供。