代码审查应对

技能简介

代码审查应对技能提供了一套结构化流程，用于接收、评估和处理代码审查反馈。强调 技术验证先于实施 ， 技术正确性高于社交舒适 ，避免盲目接受或表演式认同。

业务背景

代码审查应对帮助开发团队建立技术驱动的反馈处理机制，避免'为改而改'的低效沟通。通过系统化评估每条建议的技术合理性，优先处理阻塞性问题，减少不必要的功能堆砌，让代码审查真正提升代码质量而非消耗团队精力。

落地案例：收到审查者建议'完善未使用的API端点'时，先执行YAGNI检查：搜索代码库确认该端点是否被调用。若未使用，则提议直接删除而非投入开发资源完善。面对模糊反馈主动追问具体技术问题，对明显错误的技术建议基于代码上下文有理有据地反驳，确保修改决策建立在技术验证之上。

能做什么

• 系统化评估代码审查反馈的技术合理性
• 识别并拒绝表演式回应(如过度感谢)
• 在反馈不明确时主动寻求澄清
• 对技术错误的建议进行有理有据的反驳
• 实施YAGNI检查，避免添加未使用的功能
• 按优先级处理多项目反馈(阻塞问题→简单修复→复杂重构)
• 区分人类伙伴反馈与外部审查者反馈的处理方式

使用说明

安装指令：

clawhub install receiving-code-review

核心流程：

1. 阅读：完整阅读反馈，不做即时反应
2. 理解：用自己的话重述需求，或提出疑问
3. 验证：对照代码库现实进行检查
4. 评估：对该代码库而言技术上是否合理
5. 回应：技术确认或有理有据的反驳
6. 实施：一次处理一项，每项都进行测试

禁止的回应方式：

• 表演式认同：”你说得完全正确！”、”好观点！”
• 未经验证就承诺：”我马上去实现”
• 部分理解就实施：理解1-3项就动手，稍后再问4-5项

正确的回应方式：

• 重述技术需求
• 提出澄清问题
• 如错误则提供技术理由反驳
• 直接开始工作(行动胜于言语)

何时反驳：

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

YAGNI检查：如果审查者建议”正确实现”，先搜索代码库确认该端点是否被调用。如未使用，提议删除而非完善。

输入与输出

见下方输入与输出表格。

见下方输入输出表格。

风险提示

• 需要勇气对错误反馈进行技术反驳
• 过度谨慎可能导致实施延迟
• 需要良好的代码库熟悉度才能有效验证
• 原始页面信息不足，缺少具体安装命令和自动化脚本

来源信息

原始链接：https://github.com/openclaw/skills/tree/main/skills/chenleiyanquan/receiving-code-review/SKILL.md
来源类型：github