name: 请求代码审查 description: 在完成任务、实现主要功能或合并之前使用,以验证工作符合要求 - 派遣超级能力:code-reviewer 子代理根据计划或要求在继续之前审查实现
请求代码审查
派遣超级能力:code-reviewer 子代理以在问题级联之前捕获问题。
核心原则: 早期审查,频繁审查。
何时请求审查
强制性:
- 在子代理驱动的开发中的每个任务之后
- 完成主要功能之后
- 合并到主分支之前
可选但有价值:
- 当卡住时(新鲜视角)
- 重构之前(基线检查)
- 修复复杂错误之后
如何请求
1. 获取 git SHAs:
BASE_SHA=$(git rev-parse HEAD~1) # 或 origin/main
HEAD_SHA=$(git rev-parse HEAD)
2. 派遣 code-reviewer 子代理:
使用 Task 工具和 superpowers:code-reviewer 类型,在 code-reviewer.md 处填写模板
占位符:
{WHAT_WAS_IMPLEMENTED}- 您刚刚构建的内容{PLAN_OR_REQUIREMENTS}- 它应该做什么{BASE_SHA}- 起始提交{HEAD_SHA}- 结束提交{DESCRIPTION}- 简要摘要
3. 根据反馈行动:
- 立即修复关键问题
- 在继续之前修复重要问题
- 标记次要问题以供稍后处理
- 如果审查者错误,提出异议(附理由)
示例
[刚刚完成任务 2: 添加验证函数]
您: 让我在继续之前请求代码审查。
BASE_SHA=$(git log --oneline | grep "Task 1" | head -1 | awk '{print $1}')
HEAD_SHA=$(git rev-parse HEAD)
[派遣超级能力:code-reviewer 子代理]
WHAT_WAS_IMPLEMENTED: 对话索引的验证和修复函数
PLAN_OR_REQUIREMENTS: 来自 docs/plans/deployment-plan.md 的任务 2
BASE_SHA: a7981ec
HEAD_SHA: 3df7661
DESCRIPTION: 添加了 verifyIndex() 和 repairIndex() 与 4 种问题类型
[子代理返回]:
优点: 干净的架构,真实的测试
问题:
重要: 缺少进度指示器
次要: 报告间隔的魔术数字 (100)
评估: 可以继续
您: [修复进度指示器]
[继续到任务 3]
与工作流的集成
子代理驱动开发:
- 在 EACH 任务之后审查
- 在问题复合之前捕获问题
- 在移动到下一个任务之前修复
执行计划:
- 在每批次(3 个任务)之后审查
- 获取反馈,应用,继续
临时开发:
- 合并之前审查
- 卡住时审查
红旗警告
从不:
- 跳过审查因为“它很简单”
- 忽略关键问题
- 在未修复重要问题的情况下继续
- 与有效的技术反馈争论
如果审查者错误:
- 用技术理由提出异议
- 显示证明其工作的代码/测试
- 请求澄清
参见模板:requesting-code-review/code-reviewer.md