请求代码评审(Requesting Code Review)
概述
派遣代码评审子代理(subagent)在问题扩散前捕获它们。
核心原则: 尽早评审,频繁评审。
何时请求评审
必须:
- 子代理驱动开发中每个任务完成后
- 完成主要功能后
- 合并到 main 分支前
可选但有价值:
- 遇到困难时(获取新视角)
- 重构前(基线检查)
- 修复复杂 bug 后
如何请求
- 获取 git SHA:
BASE_SHA=$(git rev-parse HEAD~1) # 或 origin/main
HEAD_SHA=$(git rev-parse HEAD)
- 派遣代码评审子代理,填充模板占位符:
{WHAT_WAS_IMPLEMENTED}- 刚构建的内容{PLAN_OR_REQUIREMENTS}- 应该做什么{BASE_SHA}- 起始提交{HEAD_SHA}- 结束提交
- 根据反馈采取行动:
- Critical(严重)问题:立即修复
- Important(重要)问题:继续前修复
- Minor(次要)问题:记录待后续处理
- 如果评审者错了:用技术理由反驳
评审代理模板
评审代理应按以下格式输出:
优点
[具体说明做得好的地方]
问题
Critical(必须修复)
[Bug、安全问题、数据丢失风险]
Important(应该修复)
[架构问题、缺失功能、测试缺口]
Minor(锦上添花)
[代码风格、优化机会]
每个问题包含: 文件:行号引用、问题描述、影响原因、修复建议
评估
可以合并吗? [是/否/修复后可以]