SuperAPI AI 工具集

critical-code-reviewer

测试与质量

查看项目

更新于 10-16 03:42

技能介绍

用途:该技能让 AI 扮演资深工程师,对提交的代码或 PR 进行严格、对抗式的审查,目标是识别每一处缺陷、低效与不良实践,保护代码库免受失控的熵增。适用于用户要求“批判性审查”代码或 PR 的场景。

主要能力:采用“有罪推定”心态,只评估代码本身而非作者意图,忽略 PR 描述、提交信息和 TODO/FIXME 承诺。内置多种检测模式,包括:识别废话注释、懒惰命名(data、temp、result 等)、复制粘贴痕迹、盲目照搬的模式、过早抽象与缺失抽象、死代码、注释滥用;检查代码结构问题,如函数职责混杂、文件像杂物抽屉、同一 PR 内模式不一致、导入混乱、超长组件、无叙事逻辑的 Notebook、样式散落。还提供对抗性视角(未处理的 Promise、null/undefined、畸形 API 响应、恶意输入、临时方案永久化、any 类型、缺失 try/catch、缺失 await 等)以及语言特定红旗清单,覆盖 Python、R、JavaScript/TypeScript、前端通用和 SQL/ORM。

使用方式:审查时按严重程度分级——Blocking(安全漏洞、数据损坏、逻辑错误、竞态、无障碍失败)、Required Changes(低质模式、未处理边界、命名差、类型安全违规)、Strong Suggestions(次优方案、缺测试、性能问题)、Noted(次要风格)。输出采用固定格式:Summary、Critical Issues(含 file:line)、Required Changes、Suggestions、Verdict(Request Changes / Needs Discussion / Approve)、Next Steps。若作为子代理运行,则只输出审查结果,不提供后续步骤。

适用场景:代码评审、PR 审查、代码质量把关、安全与边界条件检查。

限制:审查部分代码时需说明无法验证的内容;上下文缺失时标记为“Verify”而非“Blocking”;迭代审查只关注增量;对不熟悉的框架或领域模式应说明顾虑并遵从团队约定;批准仅表示“严格审查后未发现阻塞问题”,不代表代码完美。