SuperAPI AI 工具集

pre-landing-review

通用与其它

查看项目

更新于 10-16 06:45

技能介绍

该技能用于代码变更落地前的安全审计,定位为“关卡”而非通用代码质量评审:它依赖外部检查清单,采用两轮严重性分级,并对问题给出阻断/非阻断分类。与 pr-review 的区别在于,后者是多维度代码质量评审;与 Claude Code 的 /ultrareview 的区别在于,/ultrareview 是偏向单次变更找 bug 的轻量原生评审,而本技能侧重清单驱动、面向关卡、带明确分级协议的阻断判定。

使用流程:先确定审查范围,若在功能分支则与默认分支对比(git symbolic-ref refs/remotes/origin/HEAD),若给出 PR 编号则获取该 PR 的 diff,若给出具体文件则审查这些文件。随后必须读取 references/checklist.md,若清单不可读则停止并报告错误。

第一轮为 CRITICAL(阻断):检查 SQL 与数据安全(未参数化裸 SQL、缺少迁移安全的 schema 变更、无 WHERE 校验的批量更新删除、绕过模型校验的直写列)、竞态与并发(读后写无锁、仅靠唯一约束、无同步的共享可变状态、队列任务幂等性)、信任边界(LLM/AI 输出未经净化用于 SQL、shell 或 HTML;用户输入直达特权操作;外部 API 响应未做 schema 校验;反序列化不可信数据)。每个 CRITICAL 问题需引用具体文件与行号、说明风险,并通过 AskUserQuestion 提供三个选项:立即修复、确认风险、误报;选择立即修复则修复后复查。

第二轮为 INFORMATIONAL(非阻断):条件副作用、魔法数字、死代码、测试缺口、加密与熵、时间窗口安全、类型强制转换,汇总为单个表格呈现。

关卡判定:第一轮全部解决则 CLEAR TO LAND,否则 BLOCKED;第二轮仅作建议,不阻断。不予标记的范围包括测试文件使用夹具/工厂、遵循框架惯例的迁移文件、说明意图的注释、有文档值的配置文件、类型存根或接口定义。默认只读,仅当用户明确选择立即修复时才改代码。相关收费、平台与效果等细节暂未提供。