SuperAPI AI 工具集

Code Review Reception

编程与开发

查看项目

更新于 10-15 12:53

技能介绍

用途

Code Review Reception 是一份用于指导开发者如何接收并处理代码评审反馈的技能规范。其核心主张是:代码评审需要的是技术评估,而不是情绪表演。核心原则为:先验证再实现,先询问再假设,技术正确性优先于社交舒适度。

主要能力与响应模式

该规范定义了一套接收反馈时的响应流程:

  • READ:完整读完反馈,不做反应
  • UNDERSTAND:用自己的话复述需求(或提问)
  • VERIFY:对照代码库实际情况检查
  • EVALUATE:判断该建议对当前代码库是否技术上成立
  • RESPOND:做出技术性确认,或有理由的反驳
  • IMPLEMENT:一次实现一项,逐项测试

禁止的回应

规范明确列出了禁止使用的回应方式,包括:“你说得完全对(explicit CLAUDE.md violation)”“好观点”“很棒的反馈”等表演式赞同,以及在验证之前就说“我现在就实现它”。替代做法是:复述技术要求、提出澄清问题、在建议错误时以技术理由反驳,或直接开始工作(行动优先于言辞)。

处理不清晰的反馈

如果有任何一项不清楚,应停止,不要先实现任何内容,而是就不清楚的项目请求澄清。原因是各项之间可能存在关联,理解不完整会导致错误实现。示例中,人类合作者要求“修复 1-6 项”,若你理解 1、2、3、6,但不清楚 4、5,错误做法是先实现 1、2、3、6 再问 4、5,正确做法是说明理解情况并请求澄清 4 和 5 后再继续。

区分反馈来源

来自人类合作者的反馈被视为可信,理解后即可实现,但范围不清时仍需询问,且不使用表演式赞同,可直接行动或做技术性确认。

来自外部评审者的反馈,在实现前需要检查:对当前代码库技术上是否正确、是否破坏现有功能、当前实现的原因是什么、是否在所有平台与版本上可用、评审者是否理解完整上下文。如果建议看起来是错的,应基于技术理由反驳;如果难以验证,应说明无法在缺少某项信息的情况下验证,并询问是否调查、询问或继续;如果与人类合作者的既往决策冲突,应先停止并与人类合作者讨论。文档引用的人类合作者规则是:“外部反馈——保持怀疑,但仔细核查”。

YAGNI 检查

当评审者建议“正确地实现”某个功能时,应在代码库中搜索实际使用情况。若未被使用,可提议移除(YAGNI);若被使用,则应正确实现。文档引用的人类合作者规则是:“你和评审者都向我汇报。如果我们不需要这个功能,就不要添加它。”

实现顺序

对于多项反馈:先澄清所有不清楚之处,然后按以下顺序实现:阻塞性问题(破坏、安全)、简单修复(拼写错误、导入)、复杂修复(重构、逻辑)。每项修复单独测试,并验证没有回归。

何时应当反驳

当建议会破坏现有功能、评审者缺乏完整上下文、违反 YAGNI(未使用的功能)、对当前技术栈而言技术上不正确、存在遗留或兼容性原因,或与人类合作者的架构决策冲突时,应当反驳。反驳方式包括:使用技术理由而非防御性态度、提出具体问题、引用可用的测试或代码、涉及架构问题时让人类合作者参与。文档还给出了在难以当面反驳时可使用的信号语:“Strange things are afoot at the Circle K”。

确认正确反馈的方式

当反馈确实正确时,可回应:“已修复。[简要描述改动]”“抓得好——[具体问题]。已在[位置]修复。”或直接修复并在代码中体现。禁止使用“你说得完全对”“好观点”“感谢指出”以及任何感谢表达。文档解释不用感谢的原因是:行动胜于言辞,直接修复即可,代码本身就表明你听到了反馈。如果发现自己即将写“谢谢”,应删除它并改为陈述修复内容。

优雅地纠正自己的反驳

如果反驳后发现自己错了,可回应:“你是对的——我检查了 [X],它确实会 [Y]。现在实现。”或“已验证,你是正确的。我最初的理解有误,因为[原因]。正在修复。”应避免冗长道歉、为自己当初的反驳辩护或过度解释。

常见错误对照表

文档列出了常见错误及修复方式:表演式赞同应改为陈述需求或直接行动;盲目实现应先对照代码库验证;未测试就批量处理应改为一次一项并逐项测试;假设评审者正确应检查是否会破坏功能;回避反驳应牢记技术正确性优于舒适度;部分实现应先澄清所有项目;无法验证却继续推进应说明限制并请求指示。

实际示例

文档给出了表演式赞同(差)与技术验证(好)的对比,例如评审者要求移除遗留代码时,好的回应是检查构建目标为 10.15+、该 API 需要 13+、为向后兼容需要保留遗留代码,并指出当前实现的 bundle ID 有误,询问是修复还是放弃 13 以下支持。YAGNI 示例中,评审者要求实现包含数据库、日期筛选、CSV 导出的完整指标追踪,好的回应是说明搜索代码库后没有调用该端点,询问是否移除或是否有遗漏的使用场景。不清晰项示例中,好的回应是说明理解 1、2、3、6,并请求澄清 4 和 5 后再实现。

底线

外部反馈是待评估的建议,而非必须服从的命令。验证、质疑,然后实现。不要表演式赞同,始终坚持技术严谨性。

限制

目前资料未提供该技能对具体编程语言、平台或工具链的适配限制,也未提供安装方法、代码片段或收费方式,相关事实暂未提供。