SuperAPI AI 工具集

code-review

测试与质量

查看项目

更新于 10-15 17:24

技能介绍

用途:在 Basic Machines 仓库中进行本地化代码审查时使用,普通代码审查之外还需判断仓库风格与架构风险;只报告发现,除用户明确要求修复具体问题外不改代码。

审查范围:针对当前 diff 或指定文件,对照仓库的 AGENTS.md / CLAUDE.md、docs/ENGINEERING_STYLE.md 以及涉及的代码路径与测试;只应用当前活跃仓库的指引。在 basic-memory 中优先关注本地优先的文件/数据库/MCP 边界;在 basic-memory-cloud 中优先关注租户/工作区隔离、云 worker 行为及 web-v2 状态/运行时边界。

审查要点:只报告具体、可证伪的风险,包括认知负荷(改动是否比问题本身更难理解)、变更传播(一个产品改动是否迫使无关层修改)、知识重复(同一规则多处编码可能漂移)、偶然复杂度(是否无必要地增加抽象、回退或状态)、依赖方向(API/MCP/CLI、服务、仓库、UI store 是否遵守预期边界)、领域模型失真(命名与类型是否仍匹配产品概念,传输/存储细节是否泄漏进领域)、测试预言质量(测试是否会针对该改动声称防护的缺陷或回归失败)。

需显式检查的仓库规则:不得为未知模型形状使用推测性的 getattr(obj, "attr", default);不得宽泛吞异常、仅告警的失败路径或隐藏回退行为;不得用 cast 或 Any 掩盖不清晰的类型关系;内部值/结果对象用 dataclasses,验证/序列化边界用 Pydantic;仅在需要某项能力时使用窄 Protocol;显式管理 async/资源所有权、取消与清理;高风险改动需有有意义的回归测试或验证;注释解释 why 而非 what。

报告格式:按严重程度排序。每条发现包含严重级别(high 为可能的正确性、安全、数据丢失或租户/工作区隔离失败;medium 为具体可维护性或边界风险;low 为轻微一致性、指引模糊或仅清理)、位置 file:line、风险类别、结论,并给出 Why(证明风险的具体行为或代码路径)与 Fix(最小可行改动,若无明显方案则写“none obvious”)。如无发现需明确说明,并指出仍存在的验证缺口。

限制:具体仓库配置、命令与依赖细节资料未提供。