SuperAPI AI 工具集

pre-push-review

通用与其它

查看项目

更新于 10-16 00:31

技能介绍

用途:在推送分支前运行与自动 review-pr-opened GitHub Action 相同的审查,提前发现 CI 会指出的问题。资料特别强调 amplifier-resolve 上最常见的模式:模块拆分后 mock patch 目标失效——把单体模块拆成包后,测试中的 patch("old.module._X") 或 from old.module import _X 必须改为 patch("new.module.submodule._X");对再导出打补丁会静默失效,因为真实模块内部使用自己的本地绑定。

主要能力与步骤:1) 用 git diff origin/main...HEAD --name-only 和 git diff origin/main...HEAD 了解改动,判断是否发生模块拆分(.py 被删除并创建同名目录)。2) 检查失效 patch 目标,对每个拆分模块在 tests/ 中 grep 旧路径,规则是 patch("a.b.symbol") 仅当 symbol 真正定义在 a.b 时才正确,正确目标应是调用该符号的模块。3) 检查带 # noqa: F401 的死导入,判断是否为向后兼容再导出或 TYPE_CHECKING 保护,否则标记。4) 检查 fixture 环境变量安全,grep os.environ[ 和 os.environ.pop,手动赋值在测试异常时会泄漏,应改用 monkeypatch.setenv。5) 获取 https://raw.githubusercontent.com/microsoft/amplifier-app-actions/main/context/pr-review.md 的审查标准,委托 superpowers:code-reviewer(model_role=critique)做完整审查,要求读完整文件而非仅 diff,并 grep 所有调用方与测试;核心标准包括必要性、层次契合、模式、正确性、校准,以及验证每个 unittest.mock.patch() 目标。6) 按严重程度输出 Required(会触发 CI CHANGES_REQUESTED)与 Advisory 发现,附 file:line 和修复建议,并给出 READY TO PUSH 或 NEEDS FIXES FIRST 的明确结论。

输入:分支名(默认当前分支),可选包含仓库路径。

限制:资料未提供安装方式、收费、平台支持或运行环境要求,暂未提供。