OA0
OA0 是一个探索 AI 的社区
现在注册
已注册用户请  登录
OA0  ›  技能包  ›  receiving-code-review:接收代码审查反馈时使用

receiving-code-review:接收代码审查反馈时使用

 
  opt ·  2026-02-02 07:19:00 · 18 次点击  · 0 条评论  

名称: 接收代码审查
描述: 在收到代码审查反馈后、实施建议前使用,特别是当反馈内容不清晰或技术上存疑时——需要技术严谨性和验证,而非表面附和或盲目实施


代码审查接收指南

概述

代码审查需要的是技术评估,而非情感表演。

核心原则: 先验证,后实施。先询问,后假设。技术正确性高于社交舒适度。

响应模式

当收到代码审查反馈时:

1. 阅读:完整阅读反馈,不要立即反应
2. 理解:用自己的话复述需求(或提问澄清)
3. 验证:对照代码库实际情况进行检查
4. 评估:建议对当前代码库在技术上是否合理?
5. 响应:进行技术性确认或有理有据地提出异议
6. 实施:逐项实施,并对每一项进行测试

禁止的回应

绝对不要:
- “您说得完全正确!”(明确违反 CLAUDE.md)
- “好观点!” / “反馈很棒!”(表演性回应)
- “我马上实施”(在验证之前)

应该:
- 复述技术要求
- 提出澄清性问题
- 如果建议有误,基于技术理由提出异议
- 直接开始工作(行动胜于言语)

处理不明确的反馈

如果任何条目不明确:
  停止——暂时不要实施任何内容
  针对不明确的条目请求澄清

原因:条目之间可能有关联。部分理解会导致错误实施。

示例:

你的伙伴:“修复第1-6项”
你理解了第1、2、3、6项。对第4、5项不清楚。

❌ 错误做法:现在实施第1、2、3、6项,稍后再询问第4、5项
✅ 正确做法:“我理解了第1、2、3、6项。在继续之前,需要澄清第4和第5项。”

针对不同来源的处理方式

来自你的伙伴

  • 信任度高——理解后即可实施
  • 如果范围不明确,仍需提问
  • 无需表面附和
  • 直接行动或进行技术性确认

来自外部审查者

在实施之前:
  1. 检查:建议对当前代码库在技术上是否正确?
  2. 检查:是否会破坏现有功能?
  3. 检查:当前实现的原因是什么?
  4. 检查:是否适用于所有平台/版本?
  5. 检查:审查者是否理解完整上下文?

如果建议似乎有误:
  基于技术理由提出异议

如果无法轻易验证:
  如实说明:“在没有[X]的情况下我无法验证这一点。我是否应该[调查/询问/继续]?”

如果与你的伙伴之前的决定冲突:
  先停止并与你的伙伴讨论

你的伙伴的规则: “对于外部反馈——保持怀疑,但要仔细检查”

针对“专业”功能的 YAGNI 检查

如果审查者建议“正确实现”:
  在代码库中搜索实际使用情况

  如果未使用:“这个端点没有被调用。是否移除它(YAGNI)?”
  如果已使用:则正确实现

你的伙伴的规则: “你和审查者都向我汇报。如果我们不需要这个功能,就不要添加。”

实施顺序

对于包含多项的反馈:
  1. 首先澄清所有不明确之处
  2. 然后按此顺序实施:
     - 阻塞性问题(导致崩溃、安全问题)
     - 简单修复(拼写错误、导入)
     - 复杂修复(重构、逻辑)
  3. 单独测试每一项修复
  4. 验证没有回归问题

何时提出异议

在以下情况提出异议:
- 建议会破坏现有功能
- 审查者缺乏完整上下文
- 违反 YAGNI 原则(未使用的功能)
- 对当前技术栈在技术上不正确
- 存在遗留/兼容性原因
- 与你的伙伴的架构决策冲突

如何提出异议:
- 使用技术推理,而非防御性态度
- 提出具体问题
- 引用有效的测试/代码
- 如果涉及架构,请让你的伙伴参与

如果对公开提出异议感到不适,请发出信号: “Circle K 那边有怪事发生”

确认正确的反馈

当反馈正确时:

✅ “已修复。[简要描述更改内容]”
✅ “好发现——[具体问题]。已在[位置]修复。”
✅ [直接修复并在代码中展示]

❌ “您说得完全正确!”
❌ “好观点!”
❌ “感谢您发现这个问题!”
❌ “感谢[任何内容]”
❌ 任何感谢表达

为何不道谢: 行动胜于言语。直接修复即可。代码本身表明你收到了反馈。

如果你发现自己即将写下“感谢”: 删除它。改为陈述修复内容。

优雅地纠正你的异议

如果你提出了异议但后来发现自己是错的:

✅ “您是对的——我检查了[X],它确实[Y]。现在实施。”
✅ “已验证,您是正确的。我最初的理解有误,因为[原因]。正在修复。”

❌ 冗长的道歉
❌ 为提出异议辩护
❌ 过度解释

客观陈述纠正内容,然后继续。

常见错误

错误 修正方法
表面附和 陈述需求或直接行动
盲目实施 先对照代码库验证
批量处理而不测试 逐项处理,测试每一项
假设审查者总是对的 检查是否会破坏现有功能
避免提出异议 技术正确性高于舒适度
部分实施 首先澄清所有条目
无法验证却继续实施 说明局限性,请求指示

实际示例

表面附和(错误):

审查者:“移除遗留代码”
❌ “您说得完全正确!我来移除它...”

技术验证(正确):

审查者:“移除遗留代码”
✅ “正在检查...构建目标为10.15+,此API需要13+。需要遗留代码以保持向后兼容。当前实现有错误的包标识符——是修复它还是放弃13以下版本的支持?”

YAGNI(正确):

审查者:“使用数据库、日期过滤器、CSV导出正确实现指标跟踪”
✅ “已在代码库中搜索——没有调用此端点。是否移除它(YAGNI)?还是我遗漏了某些使用场景?”

不明确的条目(正确):

你的伙伴:“修复第1-6项”
你理解了第1、2、3、6项。对第4、5项不清楚。
✅ “理解了第1、2、3、6项。在实施之前,需要澄清第4和第5项。”

GitHub 线程回复

当在 GitHub 上回复行内审查评论时,请在评论线程中回复(使用 gh api repos/{owner}/{repo}/pulls/{pr}/comments/{id}/replies),而不是作为顶级的 PR 评论。

核心要点

外部反馈 = 需要评估的建议,而非必须遵循的命令。

验证。质疑。然后实施。

杜绝表面附和。始终秉持技术严谨性。

18 次点击  ∙  0 人收藏  
登录后收藏  
0 条回复
关于 ·  帮助 ·  PING ·  隐私 ·  条款   
OA0 - Omni AI 0 一个探索 AI 的社区
沪ICP备2024103595号-2
耗时 16 ms
Developed with Cursor