SuperAPI AI 工具集

review-pr

编程与开发

查看项目

更新于 10-15 21:55

技能介绍

用途:审查非本人编写的 gcx 拉取请求,由该技能统一负责报告的内容、顺序和停止条件。它自身不定义检查项,而是要求运行已有位置的检查。

主要能力: 一是正确性审查,需亲自阅读 diff,查找评审人应捕捉的缺陷,如返回错误值的 nil 或错误路径、被移动的边界、失败分支上的资源泄漏、引入的并发访问,以及改变行为却未更新调用方的调用。 二是运行针对该 diff 触发的检查项,外加合规层级检查。相关检查包括 .claude/skills/integrate-with-gcx/references/self-review.md,其中 T12 为过度设计标准,T5 覆盖重复仓库已有内容的代码;AGENTS.md 用于合规层级,需按顺序检查全部四个层级。

阻断合并的情况:违反 CONSTITUTION.md 或 DESIGN.md;对已发布命令表面的回归;在描述为重构的 diff 中改变行为;凭据泄漏。命名、结构、重复和测试形态类问题记为 nit。

不报告:CI 已强制执行的内容,如 mise run lint、测试套件、reference-drift、validate-skills 及一致性套件;docs/reference/cli/ 下的生成文件和 vendor/ 中的内容;diff 未触及文件的测试覆盖缺失。

报告形态:使用粗体文本作标题,而非 markdown 标题;各节最多两句。包含 Intent、Blocking、Other findings、Fix summary、Verdict。第 4 节按锁定成本排序,最多六条。

工作流调用时:不询问直接交付,绝不批准或请求更改,仅作为评论发布;最多三条阻断发现、共八条评论;无发现时明确说明。以结构化输出返回,格式为 {"review": {"body": "<the summary>", "comments": []}}。

发布到 GitHub:报告后提供发布选项,未经明确同意不得发布。使用 gh api repos/grafana/gcx/pulls/{n}/reviews -X POST --input - 并传入 JSON。每条评论需 path、line 和 side,范围还需 start_line 和 start_side,行号须对照 PR head 核对。

语言要求:报告和评论使用简明英语,短句、主动语态、避免习语。限制:官方文本为空,其他未提供信息暂未提供。