requesting-code-review

1# 请求代码审查
2 
3派遣代码审查子代理，在问题扩散之前发现它们。审查者获得的是精心组织的评估上下文——绝不是你的会话历史。
4 
5**核心原则：** 早审查，勤审查。
6 
7## 何时请求审查
8 
9**必须审查：**
10- 子代理驱动开发中每个任务完成后
11- 完成重要功能后
12- 合并到 main 之前
13 
14**可选但有价值：**
15- 卡住时（换个视角）
16- 重构之前（建立基线）
17- 修复复杂 bug 之后
18 
19## 如何请求
20 
21**1. 获取 git SHA：**
22```bash
23BASE_SHA=$(git rev-parse HEAD~1)  # 或：git merge-base origin/main HEAD
24HEAD_SHA=$(git rev-parse HEAD)
25```
26 
27**2. 派遣代码审查子代理：**
28 
29使用 Task 工具，指定 `general-purpose` 类型，填写 `code-reviewer.md` 中的模板
30 
31**占位符说明：**
32- `{DESCRIPTION}` - 你刚完成的内容简要说明
33- `{PLAN_OR_REQUIREMENTS}` - 预期功能
34- `{BASE_SHA}` - 起始提交
35- `{HEAD_SHA}` - 结束提交
36 
37**3. 处理反馈：**
38- Critical 问题立即修复
39- Important 问题在继续之前修复
40- Minor 问题记录下来稍后处理
41- 如果审查者有误，用技术理由反驳
42 
43## 示例
44 
45```
46[刚完成任务 2：添加验证功能]
47 
48你：让我在继续之前请求代码审查。
49 
50BASE_SHA=$(git log --oneline | grep "Task 1" | head -1 | cut -d' ' -f1)
51HEAD_SHA=$(git rev-parse HEAD)
52 
53[派遣代码审查子代理]
54  DESCRIPTION: 添加了 verifyIndex() 和 repairIndex()，支持 4 种问题类型
55  PLAN_OR_REQUIREMENTS: docs/superpowers/plans/deployment-plan.md 中的任务 2
56  BASE_SHA: a7981ec
57  HEAD_SHA: 3df7661
58 
59[子代理返回]:
60  优点：架构清晰，测试真实
61  问题：
62    Important：缺少进度指示器
63    Minor：报告间隔使用了魔法数字 (100)
64  评估：可以继续
65 
66你：[修复进度指示器]
67[继续任务 3]
68```
69 
70## 常见的合理化借口
71 
72| 借口 | 现实 |
73|------|------|
74| "我自己看一下 diff 就行了，不用专门派审查者" | 你是协调者——在自己的会话里读 diff 会烧掉你继续推进工作所需的上下文窗口。派一个审查子智能体：diff 和评估过程都待在它的上下文里，只有结论回到你这里。 |
75| "审查者需要我的全部会话历史才能理解这次改动" | 给它精心组织的上下文，绝不给会话历史。这样审查者才会盯着工作成果，而不是你的思考过程。 |
76 
77## 红线
78 
79**绝不要：**
80- 因为"很简单"就跳过审查
81- 忽略 Critical 问题
82- 带着未修复的 Important 问题继续推进
83- 对合理的技术反馈进行争辩
84 
85**如果审查者有误：**
86- 用技术理由反驳
87- 展示证明其可行的代码/测试
88- 要求澄清
89…