SuperAPI AI 工具集

code-review-standards

测试与质量

查看项目

更新于 10-15 20:07

技能介绍

用途:该技能定义 code-critic 智能体审查实现时使用的结构化、带严重度标签的检查清单,使审查在不同派发之间保持确定性,让 PM 与工程师都清楚哪些问题会阻断交付、哪些只是建议。工程师也可在请求 critic 审查前自行加载用于自审。

主要能力:按四级严重度组织检查项。CRITICAL(必须修复、阻断交付)包括:不得硬编码密钥、API key 或凭据;不得存在注入向量(仅用参数化查询,不做不安全的 shell/模板插值);不得有任意代码执行路径(eval、exec、不受限反序列化);认证/授权不可绕过;不得有数据丢失或静默损坏路径。HIGH(必须修复、阻断交付)包括:公共函数与类需有类型提示/静态类型且类型检查器干净通过;测试零失败且新代码有有意义覆盖;不得裸 except、不得吞掉错误、错误分支需显式处理;不得有可变默认参数、无保护的全局可变状态;异步函数内不得有同步 I/O、不得有 N+1 查询。MEDIUM(标记并在报告中记录后继续)包括:函数规模合理且职责单一;用哈希表替代嵌套循环主导的复杂度;异步操作有显式超时;公共方法有 docstring、生产路径无多余 Any。LOW(仅记录):格式化/静态检查干净(由工具处理)、命名清晰、无注释掉的代码、导入整洁。

使用方式:critic 回复的第一行必须是结论,随后是问题表。结论规则:APPROVE 表示零 CRITICAL 且零 HIGH,PM 继续进入安全审查;WARN 表示零 CRITICAL 但有一个及以上 HIGH,继续但将问题表记录到交接文档;BLOCK 表示存在任何 CRITICAL,暂停、向用户呈现问题并等待指示。问题表列为 Severity | File | Line | Issue | Required Fix。

限制:采用 80% 置信度过滤——干净的审查也是有效审查,不得制造问题;只报告有 80% 把握确属真实问题的项,低于该阈值的问题应作为摘要段落中的提问而非表格行。每条表格问题必须可操作,工程师无需追问即可修复。该技能的具体安装命令、平台支持与收费信息暂未提供。