SuperAPI AI 工具集

code-review-and-quality

测试与质量

查看项目

更新于 10-16 09:57

技能介绍

用途:在合并任何 PR 或变更前执行多维度代码审查,适用于审查自己、其他 agent 或人类编写的代码,也用于功能实现后、重构后、修复 bug 后(同时审查修复与回归测试)。

主要能力:覆盖五个轴——正确性(是否符合 spec、边界与错误路径、测试是否有效)、可读性与简单性(命名、控制流、抽象是否值得、死代码痕迹)、架构(是否遵循既有模式、模块边界、依赖方向、重构是否真正降低复杂度)、安全性(输入校验、密钥、认证授权、SQL 参数化、XSS、依赖可信度、外部数据视为不可信)、性能(N+1、无界循环、同步阻塞、重渲染、分页、热路径大对象)。

使用方式:先理解变更意图,再先审查测试,然后逐文件按五轴走查,给发现分级(无前缀=必改、Critical=阻塞合并、Nit=可选、Optional/Consider=建议、FYI=仅供参考),按影响力排序,最后核查验证叙事。支持多模型审查模式:一个模型写代码,另一个模型审查正确性与架构,作者处理反馈,人类做最终决定。

变更规模:约 100 行改动为宜,约 300 行可接受(单一逻辑变更),约 1000 行应拆分;同时关注文件总大小,约 1000 总行数是审视信号。重构与功能开发应分开提交。

限制与说明:批准标准是变更明确改善整体代码健康度即可,不追求完美;不接受“之后再做清理”;依赖升级需读 changelog、一次一个依赖、由测试验证并审查 lockfile diff。详细安全与性能检查项分别见 security-checklist.md 和 performance-checklist.md。具体命令与安装方式暂未提供。