SuperAPI AI 工具集

basic-machines-review

编程与开发

查看项目

更新于 10-15 17:24

技能介绍

用途:该技能用于 basic-memory / basic-memory-cloud 仓库内的本地代码审查,在常规代码审查之外补充 Basic Machines 的内部风格与架构判断。默认只报告发现的问题,不修改代码,除非用户明确要求修复具体问题。

审查范围:对比当前 diff 或指定文件与仓库的 AGENTS.md / CLAUDE.md、docs/ENGINEERING_STYLE.md 以及被改动的代码路径与测试。只应用当前仓库适用的规则:在 basic-memory 中优先检查本地优先的文件、数据库与 MCP 边界;在 basic-memory-cloud 中优先检查租户/工作区隔离、云端 worker 行为以及 web-v2 状态与运行时边界。

审查标准:只报告具体且可证伪的风险,包括认知负荷、变更传播、知识重复、意外复杂度、依赖方向、领域模型失真以及测试预言质量。

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

报告格式:按严重程度排序。high 表示可能的正确性、安全、数据丢失或租户/工作区隔离失败;medium 表示具体可维护性或边界风险;low 表示轻微的连贯性问题、模糊指引或仅审查层面的清理。每条发现包含 severity | file:line | 风险类别 | 断言,以及 Why(证明风险的具体行为或代码路径)与 Fix(最小实际改动,若需产品输入则写 none obvious)。若无发现需明确说明,并指出仍存在的验证缺口。

限制:资料未提供该技能的安装命令、收费情况、价格与具体平台支持信息。