名称:请求代码审查 描述:在继续之前,分派代码审查子代理以根据计划或要求审查实现 使用时机:完成任务、实施主要功能或合并之前,以验证工作符合要求 版本:1.2.0 渐进披露: 级别:1 参考资料: - 路径:references/code-reviewer-template.md 标题:代码审查模板 描述:完整的子代理模板,包含占位符和审查清单 - 路径:references/review-examples.md 标题:审查示例和工作流 描述:好与坏的审查示例,严重性指南,完整的工作流示例
请求代码审查
分派代码审查子代理以在问题级联前捕获它们。
核心原则: 早审查,常审查。
何时请求审查
强制性:
- 在子代理驱动开发中的每个任务后
- 完成主要功能后
- 合并到主分支前
可选但有价值:
- 当卡住时(新视角)
- 重构前(基线检查)
- 修复复杂bug后
快速开始
1. 获取git SHAs:
BASE_SHA=$(git rev-parse HEAD~1) # 或 origin/main
HEAD_SHA=$(git rev-parse HEAD)
2. 分派代码审查子代理:
使用代码审查类型的任务工具,填写模板在代码审查模板
必需的占位符:
{WHAT_WAS_IMPLEMENTED}- 您刚刚构建的内容{PLAN_OR_REQUIREMENTS}- 它应该做什么{BASE_SHA}- 起始提交{HEAD_SHA}- 结束提交{DESCRIPTION}- 简要摘要
3. 根据反馈行动:
| 严重性 | 行动 |
|---|---|
| 关键 | 立即修复,不要继续 |
| 重要 | 在下个主要任务前修复 |
| 次要 | 记录以备后用,可以继续 |
详情见严重性指南 对于细节。
与工作流集成
子代理驱动开发:
- 在每个任务后审查
- 在问题复合前捕获
- 在移动到下个任务前修复
执行计划:
- 每批(3个任务)后审查
- 获取反馈,应用,继续
临时开发:
- 合并前审查
- 卡住时审查
对审查提出异议
如果审查员错误:
- 用技术推理提出异议
- 显示证明它有效的代码/测试
- 参考计划要求
- 请求澄清
见提出异议示例 对于示例。
常见错误
绝不:
- 因为“它简单”而跳过审查
- 忽略关键问题
- 继续未修复的重要问题
- 没有技术理由的争论
总是:
- 在审查请求中提供完整上下文
- 立即修复关键问题
- 记录您不同意的原因(如果有)
- 在修复关键问题后重新审查
示例
需要示例? 见审查示例和工作流 对于:
- 完整的审查输出示例
- 好与坏的审查请求
- 不同场景的审查工作流
- 如何处理不同严重性级别
- 何时以及如何提出异议
需要模板? 见代码审查模板 对于完整的子代理分派模板。