SuperAPI AI 工具集

pr-code-review

测试与质量

查看项目

更新于 10-15 21:47

技能介绍

用途:对开放的 PR/MR 做合并前代码审查,也可由 merge-discipline Part R 调用。安装:npx skills add FuDesign2008/open-skills -g --skill pr-code-review --yes。

适用与跳过:用户要求审查开放 PR/MR,或合并纪律要求预合并审查时使用;PR 已关闭、草稿、琐碎/自动化、或本会话该 tip 已审查过则跳过并说明原因,不评论。

深度:调用方可传 depth=full 或 depth=light,默认 full。full 采用 Standards∥Spec 双轴,宿主支持时优先并行多视角;light 仍强制在固定 tip 上做双轴审查,但不要求并行多视角,置信度≥80 的 Critical/Important 放行与发布规则不变。独立用户触发无 depth 参数时必须用 full。

流程:1) 资格检查;2) 解析深度;3) 固定基点(PR/MR base ref、用户指定 commit/分支或 origin/默认分支),确认 ref 可解析且三点 diff 非空,空 diff 则中止;4) 解析 Spec 来源(PR/issue 正文与关联工单→用户路径→OpenSpec 变更增量/相关 specs→询问用户→否则跳过 Spec 轴);5) 解析 Standards 来源(AGENTS.md/CLAUDE.md 及 PR 涉及目录,编码规范文档,可选 reference.md 气味基线);6) 简短变更摘要;7) 双轴审查,不跨轴合并排序:Standards 轴记录有据可依的规范违背(引用文件+规则),Spec 轴记录缺失/部分实现、无理由范围蔓延、需求实现错误(引用 spec/plan 行),无 Spec 来源则整轴跳过;8) 校准:每条候选给 Critical/Important/Minor 与 0–100 置信度,低于 80 丢弃;9) 先列优点;10) 发布前复查资格;11) 通过 gh/glab 评论(宿主禁止则会话内报告),评论含双轴分节与完整 SHA 永久链接,模板见 reference.md。

置信度:0 误报/既有/经不起推敲;25 未验证/风格/不在规范;50 真实但琐碎或罕见;75 很可能真实且重要,或规范/计划明确要求;100 确凿。应丢弃:既有问题、形似非 bug、吹毛求疵、linter/类型检查/编译器可捕获项、无规范或计划依据的泛泛“多测试/文档/安全”建议、未改动行、PR 范围内的有意行为。

宿主契约:Part R 通过标准为两轴均无≥80 的 Critical 或 Important(仅 Minor 或全部低于 80 即通过),light 不得放宽;不通过则阻止合并直至修复或用户显式跳过留痕;CI 绿/覆盖率跳过不能替代;不要求完整接收审查循环。

限制:依赖 merge-discipline 时缺失则中止并提示安装命令;不要改名为 code-review(外部名称冲突)。