SuperAPI AI 工具集

receiving-code-review

编程与开发

查看项目

更新于 10-24 16:15

技能介绍

用途

「接收代码审查反馈」是一个指导 AI 编码智能体处理代码审查意见的行为准则技能。它适用于智能体收到代码审查反馈、准备实施建议之前,尤其是反馈含糊或技术上存疑的情形。该技能强调:代码审查需要技术评估,而不是情绪表演。

主要能力与响应流程

技能定义了一套收到反馈后的标准动作顺序:

  1. 阅读:完整读完反馈,不做即时反应。
  2. 理解:用自己的话复述要求,或向对方提问确认。
  3. 核实:对照代码库实际情况检查。
  4. 评估:判断该建议对当前这个代码库在技术上是否成立。
  5. 回应:作出技术性确认,或给出有理由的反驳。
  6. 实施:一次只处理一条,每条单独测试。

禁止的回应方式

技能明确禁止以下表达:「你完全正确」「好观点」「精彩反馈」等表演式附和,以及在核实之前就说「我现在就实施」。替代做法是:复述技术要求、提出澄清问题、以技术理由反驳、或直接动手(行动优于言辞)。同时,技能要求不出现任何感谢类表达,认为代码本身就表明已听到反馈。

处理不明确的反馈

若任一条目不清楚,技能要求立即停止,不实施任何内容,先就不清楚的条目请求澄清。原因是条目之间可能相互关联,部分理解会导致错误实现。示例:人类伙伴说「修复第 1–6 项」,智能体理解第 1、2、3、6 项、对第 4、5 项不清楚,正确做法是说明已理解哪些、需先澄清 4 和 5 再继续,而不是先做清楚的几项。

区分反馈来源

  • 来自人类伙伴:视为可信,理解后实施;范围不清仍需询问;不做表演式附和;直接进入行动或技术性确认。
  • 来自外部审查者:实施前需检查:对该代码库是否技术正确、是否破坏现有功能、当前实现的原因、是否在所有平台或版本上可用、审查者是否了解完整上下文。若建议看似错误,以技术理由反驳;若难以核实,说明「没有 X 我无法验证,是否应先调查/询问/继续」;若与人类伙伴此前的决策冲突,先停下与人类伙伴讨论。

YAGNI 检查

当审查者建议「正规实现」某功能时,技能要求先在代码库中搜索该功能的实际使用情况:若无使用,则提出按 YAGNI 移除;若有使用,再正规实现。

实施顺序与验证

对多条反馈:先澄清所有不清楚条目;然后按阻塞性问题(破坏、安全)、简单修复(拼写、导入)、复杂修复(重构、逻辑)的顺序实施;每条单独测试;最后确认无回归。

何时反驳

出现以下情况应反驳:建议破坏现有功能、审查者缺乏完整上下文、违反 YAGNI、对该技术栈不正确、存在遗留或兼容性原因、与人类伙伴的架构决策冲突。反驳方式为技术推理而非防御姿态,提出具体问题,引用可运行的测试或代码,涉及架构时让人类伙伴介入。

确认正确反馈与纠正自身错误

反馈正确时,采用「已修复。[简述改动]」「抓得好——[具体问题],已在[位置]修复」或直接修复并在代码中体现。若此前反驳有误,则以事实陈述纠正,例如「你是对的——我检查了[X],它确实[Y],现在实施」,不写长篇道歉、不为反驳辩解、不过度解释。

GitHub 评论回复

回复 GitHub 内联审查评论时,应在评论线程中回复(gh api repos/{owner}/{repo}/pulls/{pr}/comments/{id}/replies),而不是作为顶层 PR 评论。

适用场景与限制

适用场景:AI 编码智能体在代码审查环节中的回应与实施决策,尤其是外部审查意见、含糊反馈、疑似过度设计的情形。限制:该技能为行为准则类内容,不提供具体代码实现;官方页面未提供收费、平台支持范围等信息,相关事实暂未提供。