用途
Review All 用于在准备提交 PR 或完成一个阶段的工作时,执行一次全谱代码审查。它通过并行运行多个专用审查器,覆盖安全、API 和变更范围等维度,最终综合成一份统一的审查报告。整个流程是只读的,不会对代码做任何修改。
主要能力
该技能会并行启动多个审查代理,包括:
- 安全审查器(Security Reviewer):仅针对变更文件,检查 OWASP Top 10、IDOR、注入、认证缺口等问题。
- API 路由审计器(API Route Auditor):检查路由与类型目录,关注 schema 覆盖、类型漂移、认证钩子。
- 范围审计器(Scope Auditor):检查 git diff 输出,识别超出范围的修改和仅格式化的噪音。
此外,还可以按需添加项目特定的审查器,例如数据库查询审查器或框架审查器。
使用方式
流程分为四步:
- 识别变更文件:使用
git diff --name-only main...HEAD,在 main 分支上可用git diff --name-only HEAD~5,或运行bash scripts/detect-review-range.sh。 - 并行启动审查代理:通过 Task 工具同时启动所有代理。
- 综合报告:生成包含摘要表格、关键问题、警告和干净区域的报告。摘要表格列出各审查器的发现数、关键数和警告数。
- 给出结论:以一句明确的话结束,例如 READY TO MERGE(可合并)、NEEDS FIXES(需修复)或 NEEDS DISCUSSION(需讨论)。
参数方面:无参数时审查相对 main 的变更;HEAD~3 审查最近 3 次提交;--security-only 仅运行安全审查器。
适用场景
适用于准备 PR 或完成阶段性工作、需要全面代码审查时。它应在 lint、类型检查和测试之后运行,作为这些检查的补充。
限制
该技能有明确的禁止事项:任何审查器都不得在流程中编辑文件,只读是硬性要求;当两个审查器标记同一行时,不得分别报告重复发现,必须合并为一条;当用户只改了很小的 diff 时,不得审查整个仓库,应限定在变更文件;不得用它替代 lint、类型检查或测试;也不适用于实现任务或自动修复请求。