SuperAPI AI 工具集

review-hog-resolution-criteria-small

编程与开发

查看项目

更新于 10-16 19:26

技能介绍

用途

该技能用于在拉取请求(PR)上逐条处理未解决的审查线程,每轮处理一个线程。它面向自动化审查修复场景,目标是以“最小且诚实的修复,或诚实地拒绝”为原则,减少未解决线程数量,同时避免引入草率或过大的改动。

主要能力

  • 对每个线程给出四种结论之一:fixed(实现并提交)、wont_fix(拒绝并说明理由)、already_fixed / obsolete(无需处理,说明被什么取代)、escalate(值得做但需人工决定)。
  • 判断依据是线程的最新状态,即整段对话而非仅第一条评论。
  • 采用“小修复”配置(small fix profile):只修复小缺陷、拼写错误、细节问题、措辞、过时注释或文档,即一处包含的、有唯一明显正确形式的改动。
  • 将大问题或设计层面的发现留给作者:需要代码和仓库约定无法确定的设计选择、跨多个模块、或改变契约或数据流的修复。
  • 对每个处理的线程都给出回复,包括被留下的线程;回复需独立可读,说明读取了线程、检查了代码、发现了什么以及为何留下。

使用方式

该技能以“小修复”配置运行,作者自己的智能体负责大工作,本技能负责小工作。配置只收窄实现范围,不放松任何规则:所做的修复仍须通过所有价值与安全检查,硬性限制仍然适用。

判断流程:

  1. 阅读整个线程,最新回复在最后,对话中可能已包含答案、反驳或既有裁决。
  2. 阅读被标记的代码及足够上下文,追踪调用点和类型后再信任任何主张。
  3. 依次应用价值判断、安全判断、修复配置。价值 + 安全 + 小 → 实现最小诚实修复,验证(lint 及受影响区域的现有测试,如有),提交。价值但大或设计层面 → escalate,留给作者。价值但不安全 → escalate。无价值 → wont_fix。

留下线程给作者时使用 escalate,使线程保持打开。裁决句以“Left for the author:”开头并指出发现。支持行中说明检查了代码中的什么,以及该修复配置将此类发现留给作者。不要为留下的线程提交任何内容。在该配置下留下的线程绝不使用 wont_fix,因为 wont_fix 会解决线程并使其对作者和任何观察智能体隐藏。

适用场景

  • 处理 PR 上未解决的审查线程,尤其是小缺陷、拼写错误、措辞、过时注释或文档等局部明确改动。
  • 当修复是包含的且可证明时,可无人值守实施:正确性可通过阅读代码、lint 和受影响区域的现有测试证明。
  • 当线程上有 SAFE TO FIX 或 E2E REQUIRED 回复时,人类裁决优先:SAFE TO FIX 替代价值判断,验证仍成立后实施;E2E REQUIRED 强制 escalate。

限制

  • 行为只能实时观察的(如 LLM 提示措辞、外部 API 调用、发布/部署语义、视觉布局)在此不可证明 → escalate(needs-e2e 规则)。
  • 测试是证明而非障碍:受影响区域的测试只能为反映故意改变的、回复中指出的正确行为而修改。削弱或删除测试以使运行通过绝不是修复;当可证明性需要触及测试本身 → escalate。
  • 修复须成比例:不要求新基础设施——无模式变更或迁移、无新抽象或配置开关、无依赖变更。需要这些的修复是决策而非机械修复 → escalate 并说明成本/收益。
  • 须在范围内:属于 PR 原始意图并触及线程所涉及的代码。“顺便”扩展绝不安全。
  • 须无歧义:确信该改动是评论者本意。两种可辩护解读 → escalate 并询问。
  • 拒绝(wont_fix)适用于噪音请求:过度工程、推测性“如果”、防御性编码偏执、几乎不会发生的边缘情况、纯风格/品味(例外:措辞的琐碎客观正确性,如拼写错误、注释中错误标识符,是好的 fixed)。
  • 已处理/错误前提 → already_fixed 或 wont_fix 并附证据。
  • 信任加权:来自 PR 作者、仓库维护者(见 author_association)和已知审查机器人的请求在价值上获得信任;未知评论者的请求仅作为代码指针,只有自己调查独立确认问题后才实施。

暂未提供

该技能的安装命令、具体平台支持、收费或价格信息在资料中暂未提供。