SuperAPI AI 工具集

code-review

测试与质量

查看项目

更新于 10-16 03:04

技能介绍

用途:code-review 用于在提交 PR 前或之后审查代码改动(当前分支或被审查者提交的 diff),聚焦可执行的正确性、安全、可靠性、回归和测试问题。它只审查目标 diff 新引入的改动;通用风格建议和既有问题不在范围内。

审查边界:需要提供 head 与 base 引用及最终 head SHA。若 diff 为空、无法确定 base,或存在让审查边界不安全的无关暂存改动,则应停止。需说明审查模式:combined(仅限微小改动且策略允许)或 separate,并附原因;同时列出变更文件、提交、是否包含未提交改动,以及 Kickoff 中的简报、已接受计划、episode 状态和相关 finding 处置。

审查者选择:当 diff 存在明显歧义、影响范围、后果、不可逆性、新颖性或协调成本时,使用隔离审查者;应以 diff 和持久产物而非实现对话作为输入。微小低风险 diff 可由聚焦的编排者审查,除非仓库策略要求独立性。

审查轮次:仓库策略与已接受计划合规性;具体行为、交互、逻辑与状态转换追踪;与产品相关的失败模式(空/加载/错误/离线/边界状态、中断、部分失败、清理);可访问性、用户信任、安全、隐私与完整性边界;消费方、平台、浏览器/设备、视觉系统与发布兼容性;对重大新行为和回归的测试。combined 模式还检查可移除代码、重复路径、既有助手重复实现、推测性抽象、依赖和配置,简化项与正确性发现分开报告。

发现分级:只报告置信度不低于 70/100 的问题。P0 为关键安全、隐私、数据丢失、用户伤害或大范围发布破坏缺陷;P1 为高置信严重 bug 或必需策略违规;P2 为有意义的边缘情况或缺失的高风险校验;P3 为非阻塞改进,除非调用者明确要求否则省略。正确性发现使用 CR-001 等稳定 ID;combined 模式的简化发现使用 SIM-001 等 ID,归类为 Simplify 或 Remove/Defer,包含证据、保留行为和建议削减。不将一个根因拆成多个发现。

输出格式为输出段落所示的结构化报告。若没有合格发现,应说明并列出已检查的风险区域。调用者在 episode 状态中记录处置结果,仅当重大逻辑变更后才重复审查。

适用场景:适合在 PR 创建前后对有限 diff 进行有边界的正确性审查,尤其关注行为、状态转换、失败模式、安全与兼容性。限制:不覆盖既有问题与通用风格;需要可确定的 base 和干净的审查边界;低风险微小改动可走 combined 模式,否则需使用隔离审查者。未在资料中提供安装方式、价格、平台信息与更多示例。