SuperAPI AI 工具集

review-all

编程与开发

查看项目

更新于 10-15 08:17

技能介绍

用途

Review All 用于在准备提交 PR 或完成一个阶段的工作时,执行一次全谱代码审查。它通过并行运行多个专用审查器,覆盖安全、API 和变更范围等维度,最终综合成一份统一的审查报告。整个流程是只读的,不会对代码做任何修改。

主要能力

该技能会并行启动多个审查代理,包括:

  • 安全审查器(Security Reviewer):仅针对变更文件,检查 OWASP Top 10、IDOR、注入、认证缺口等问题。
  • API 路由审计器(API Route Auditor):检查路由与类型目录,关注 schema 覆盖、类型漂移、认证钩子。
  • 范围审计器(Scope Auditor):检查 git diff 输出,识别超出范围的修改和仅格式化的噪音。

此外,还可以按需添加项目特定的审查器,例如数据库查询审查器或框架审查器。

使用方式

流程分为四步:

  1. 识别变更文件:使用 git diff --name-only main...HEAD,在 main 分支上可用 git diff --name-only HEAD~5,或运行 bash scripts/detect-review-range.sh。
  2. 并行启动审查代理:通过 Task 工具同时启动所有代理。
  3. 综合报告:生成包含摘要表格、关键问题、警告和干净区域的报告。摘要表格列出各审查器的发现数、关键数和警告数。
  4. 给出结论:以一句明确的话结束,例如 READY TO MERGE(可合并)、NEEDS FIXES(需修复)或 NEEDS DISCUSSION(需讨论)。

参数方面:无参数时审查相对 main 的变更;HEAD~3 审查最近 3 次提交;--security-only 仅运行安全审查器。

适用场景

适用于准备 PR 或完成阶段性工作、需要全面代码审查时。它应在 lint、类型检查和测试之后运行,作为这些检查的补充。

限制

该技能有明确的禁止事项:任何审查器都不得在流程中编辑文件,只读是硬性要求;当两个审查器标记同一行时,不得分别报告重复发现,必须合并为一条;当用户只改了很小的 diff 时,不得审查整个仓库,应限定在变更文件;不得用它替代 lint、类型检查或测试;也不适用于实现任务或自动修复请求。