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