SuperAPI AI 工具集

pr-review

编程与开发

查看项目

更新于 10-16 01:16

技能介绍

PR Review 用于对拉取请求做聚焦、分角度的代码审查。它先确定审查视角(security、perf、tests、arch);若用户未指定,则询问或默认采用风险最高的安全角度。审查前需获取完整文件上下文,因为仅凭 diff 常会漏掉改动行之外的缺陷;若只有 diff,应先索取相关文件再下结论。

主要能力:按各角度清单逐项检查。安全角度关注认证/授权缺失、IDOR、输入校验、注入(SQL/NoSQL/命令/提示/模板)、硬编码密钥与日志泄露、输出编码与 XSS、加密与随机数误用、PII 暴露。性能角度关注 N+1、热路径分配、无界工作、错误异步、缓存误用、隐藏的 O(n²)。测试角度关注新分支覆盖、边界情况、断言强度、mock 纪律、确定性、测试命名。架构角度关注边界漂移、过早抽象、耦合、可扩展性、可逆性、命名。

使用方式:每条发现须标注文件路径与行号,说明具体缺陷,按 blocker、important、nit 分级;不确定时明确说明并索取更多代码;最后单独一行给出结论:Safe to merge | needs changes | reject(架构角度为 Architecturally sound | needs trim | re-think before merging)。若要求“完整审查”,应依次运行四个清单并分别给出各自标题与结论,不混合;时间有限时优先级为安全、测试、性能、架构。

限制:仅依据 diff 不足以确信时应请求上下文而非猜测;不应把测试通过当作正确性证明;不应把与本次改动无关的既有问题作为阻塞项;未看到足够上下文不应给出结论。