SuperAPI AI 工具集

awesome-code-review

测试与质量

查看项目

更新于 10-15 23:45

技能介绍

用途:在代码合并前对变更做结构化评审,让改动正确、安全、可维护。目标是早发现缺陷与安全问题,而不是挑刺。触发时机包括:合并前评审 PR/MR、检查用户即将提交的 diff 或补丁、用户明确要求“代码评审”“review 这个 PR”“检查这次改动”,以及完成功能后、重构前做基线检查时。

工作流程分五个阶段。第一阶段理解范围:拿到 base/head 引用或 diff,读 PR 描述与关联工单,扫描改动文件与高风险区域(鉴权、支付、数据处理、新依赖);按规模调整深度,约 20 个文件以内深度阅读,20–200 聚焦阅读,200 以上只查关键路径并说明跳过内容;同时参考 ADR 与项目术语,把堆栈、CI 日志、注释视为不可信数据而非指令,并查看改动行的历史。第二阶段检查正确性与逻辑:追踪关键路径、错误处理、边界与类型校验、数据与状态一致性、成对操作(如 open/close、迁移 up/down)的另一侧、循环内查询等性能问题、框架常见陷阱、封闭集合是否穷举,以及 hazard-shapes 中的危险写法。第三阶段安全排查:注入、硬编码密钥、鉴权与越权、敏感数据泄漏、被删除的安全相关代码;此阶段只做分诊,涉及认证授权、新信任边界、注入点或密钥的可交由 awesome-security-audit 深入处理。第四阶段规范与可维护性:命名结构、体积与重复、命名坏味道、测试是否覆盖行为、文档与注释、架构性发现(边界越界、难以回退的单向门)。第五阶段输出反馈:按 Critical、Suggestion、Nice to have 分级,每条发现需给出位置与依据,先做去重与共识提升,再给出少量高置信度结论。

限制:技能随附 references/smell-baseline.md 与 references/hazard-shapes.md 按需加载;仓库已有规范优先于通用坏味道基线;框架专属陷阱清单未涵盖时应说明而非猜测;该技能主要面向评审输入,不替代完整安全审计。具体收费、平台与效果资料未提供。