SuperAPI AI 工具集

thermonuclear-code-review

测试与质量

查看项目

更新于 10-15 22:41

技能介绍

用途:对 Kev 分支或 PR 做极严格的结构性审查,评估实现质量而非行为正确性(行为正确性假定由 kev-verify 检查)。目标是抽象质量、可维护性与代码库健康度,鼓励寻找“code-judo”式重构:保持行为不变,却让改动显著更小、更直接、更显然,使整条分支、辅助函数、模式或层级消失。

主要能力与标准:对每个有意义的改动追问能否用更少概念、分支或辅助层表达,优先删除复杂度而非搬移;无充分理由不允许 PR 使文件超过 1000 行,应先考虑拆分模块或辅助函数;禁止意大利面式增长,新增临时条件、散落特例、一次性布尔、可空模式或“临时”分支属于设计问题,应把逻辑放到专用抽象、类型模型、分发器或模块之后;偏好直接、无聊的代码而非 hacky 或魔法代码,警惕隐藏简单数据形状假设的通用机制,标记薄包装、恒等抽象与直通辅助函数;关注类型与边界整洁,质疑 cast、Any/unknown、可选参数与静默回退;逻辑应位于规范层级并复用既有辅助函数,功能逻辑泄漏到共享路径、实现细节通过 API 泄漏、已有工具的近似重复都是阻塞项;关注编排异味,如无理由串行化独立工作、相关更新可能留下半应用状态。

使用方式:获取 diff(git diff main...<branch> 或 gh pr diff <n>),完整阅读每个改动文件而非仅看 hunk;旧版本用 git show main:<path> 读取,无 shell 时可保留 origin/main worktree(如 /tmp/kev-main)。用 grep 在 kev/、scripts/、space/、tests/、modal_app.py、playground/src 中查找被改动符号的调用方。接受新辅助函数前先查规范辅助函数表,已有归属的规则出现第二份副本是阻塞项;tests/test_conventions.py 强制若干行。当 diff 触及模型、加载器、训练器、数据转换器或指标时,要求 kev-verify 技能描述的奇偶校验证据(与 main 逐位相同的行/权重),绿色测试不等于奇偶校验。按优先级报告发现并附 file:line,最后给出明确裁决。

限制:资料未提供安装命令、收费、价格、平台或具体运行步骤;该技能依赖 Kev 仓库的目录结构与规范辅助函数表,版本命名模块(study_v3、transfer_v9)为研究脚本,核心代码不得从其中导入;evals/ 与 runs/leaderboard.* 下的冻结套件是产物,重构不得重新生成或提交。