| name | receiving-code-review |
| description | 接收代码审查反馈后的正确处理流程。分类问题、确认理解、逐项修复、请求重新审查。触发词:审查反馈、review comments、修改意见、改一下。 |
| agent_created | true |
接收代码审查
概述
收到审查反馈后,正确分类、确认、逐项处理。不要让审查意见堆成山再一次性改。
处理流程
接收审查 → 分类问题 → 确认理解 → 逐项修复 → 请求重新审查
步骤 1:分类问题
将每个问题按严重程度分类:
| 严重程度 | 示例 | 处理 |
|---|
| 严重 | 逻辑错误、安全漏洞、数据丢失风险 | 立即修复 |
| 重要 | 性能问题、可维护性问题、测试缺失 | 优先修复 |
| 次要 | 命名建议、风格偏好 | 评估后决定 |
| 讨论 | 设计分歧、替代方案 | 先讨论再决定 |
步骤 2:确认理解
对每个你不确定的问题,先确认理解再动手:
- "确认一下,你是说 [你的理解],对吗?"
- 不要假设自己理解了就改
步骤 3:逐项修复
- 一次修复一个类别的问题
- 每项修复后验证测试
- 用清晰提交信息
git commit -m "fix: address review - [简要描述修改]"
步骤 4:请求重新审查
修复完成后:
- 汇总修改清单
- 说明每个问题如何解决或为什么不改
- 派发
requesting-code-review 重新审查变更部分
回复模板
审查反馈处理完毕:
严重问题 (N 个):
- [问题1]: [如何修复的]
- [问题2]: [如何修复的]
重要问题 (N 个):
- [问题1]: [如何修复的]
次要问题 (N 个):
- [问题1]: [采纳/不采纳及原因]
讨论问题 (N 个):
- [问题1]: [我的看法,等你确认]
请重新审查 [commit range] 的变更。