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