请求代码审查是一个用于在开发过程中派遣代码审查子代理、尽早发现问题的技能。其核心原则是“早审查,勤审查”。审查者获得的是精心组织的评估上下文,而不是你的会话历史,这样可以让审查者专注于工作成果而非你的思考过程,同时保留你自己的上下文以便继续工作。
何时请求审查
必须审查的情况包括:子代理驱动开发中每个任务完成后、完成重要功能后、合并到 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 问题记录下来稍后处理;如果审查者有误,用技术理由反驳。
示例
[刚完成任务 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 中的任务 2 BASE_SHA: a7981ec HEAD_SHA: 3df7661
问题: Important:缺少进度指示器 Minor:报告间隔使用了魔法数字 (100) 评估:可以继续
你:[修复进度指示器] [继续任务 3]
与工作流的集成
子代理驱动开发:每个任务完成后审查,在问题叠加之前发现它们,修复后再进入下一个任务。执行计划:每个任务完成后或在自然 checkpoint 审查,获取反馈,应用,继续。临时开发:合并前审查,卡住时审查。
红线
绝不要:因为“很简单”就跳过审查;忽略 Critical 问题;带着未修复的 Important 问题继续推进;对合理的技术反馈进行争辩。
如果审查者有误:用技术理由反驳;展示证明其可行的代码/测试;要求澄清。
参见模板:requesting-code-review/code-reviewer.md