SuperAPI AI 工具集

workflows-review

办公与效率

查看项目

更新于 10-16 09:30

技能介绍

用途:对明确要求的大型或高风险改动执行详尽代码审查。需遵循 RUNTIME_TOOLS.md 及共享的角色与授权策略,并按 closeout.md 处理严重级别、范围与审查循环上限。该技能只增加覆盖度,不授予编辑、推送、发布评论或创建跟踪文件的权限。

确定目标:先解析请求的工作区改动、提交、分支、PR 或计划,并阅读仓库说明和状态。脏工作区可作为有效审查输入并须保留;审查其他 ref 时用 diff 或隔离 worktree,不切换用户当前分支。PR 场景可用 gh 读取元数据、diff、检查与既有反馈;本地审查不要求 GitHub 认证。计划场景读取其声明文档并把标准映射到实现。需记录目标版本或工作区状态以界定范围。

选择覆盖:从当前运行时可调用的 agent 与技能目录出发,仅在需要时发现额外已配置角色;磁盘文件不证明角色可调用,不扫描全部插件或启动整个测试台。按实际改动选择独立检查:认证/密钥/权限/不可信数据对应安全与滥用路径;边界/API/数据归属对应架构与集成;热路径/查询/资源使用对应实测性能与可扩展性;类型/错误处理/并发对应正确性与失败行为;UI/交互/动画对应用户流程、无障碍与设计要求;测试与迁移对应覆盖、可重复性、上线与恢复;复杂或重复模式对应简洁与一致性。通常运行二到四个相关专家,受运行时槽位与独立范围限制,使用已配置模型,不在工作流中写模型版本。一名主审负责综合;专家只返回发现,不编辑、不嵌套派生审查者,需给出精确范围、相关上下文并要求引用当前代码。无法委派时在本地执行选定检查并披露没有独立 agent 审查;不得声称失败的工作者返回了干净审查。

收集证据:阅读足够周边代码以追踪改动行之外的行为,跟进调用方、数据模型、测试与配置;不熟悉的当前 API 行为用权威文档,意图不清时参考历史。只加载与目标相关的技能,本地与插件技能重叠时选一份,保留用户视觉风格与项目约定。计划含 breadboard 时用 breadboard-reflection 比对其位置、可供性与接线同实际实现。对每个适用场景追踪输入、状态与可观察输出:预期使用、空与畸形输入、边界值;部分失败、取消、重试、超时与恢复;并发请求、重复操作、陈旧数据与资源限制;授权边界与敏感数据处理;部署、迁移及任务仍要求的向后交互;用户可见的加载、错误、无障碍与减少动效行为。必要处用聚焦复现或测试确立缺陷,不报告没有受影响路径与理由的推测性规模问题。

综合:收集每个已启动检查的结果,或明确记录其失败;对每个候选发现针对当前代码确认,去重,并区分既有问题与请求范围内的回归。优先级:P0 需立即处理的严重失败;P1 阻碍所请求改动发布的确定缺陷;P2 应处理的重要问题,须说明影响与约束;P3 可选改进。每个发现需短标题、当前文件与紧凑行范围、触发条件、具体后果与可执行修正。省略无依据的主张与用户未要求的风格偏好。审查者分歧以证据解决,不投票、不自动改方向。

返回审查:先按严重级别列出可执行发现,再写已执行的验证与重大覆盖限制。若无发现通过验证,说明并列出检查内容,不暗示穷尽证明正确性。默认输出于对话;仅在被要求时才更新 prd.json 或创建 file-todos,不悄悄改变故事完成状态或接受新范围。除非明确要求评论,否则不在 PR 上发帖。若同一任务要求修复,实现者应用已验证的范围内修复并按 closeout 契约重跑受影响检查;仅审查请求在交付报告后停止。