SuperAPI AI 工具集

thermo-nuclear-code-quality-review

编程与开发

查看项目

更新于 10-15 08:32

技能介绍

用途

Thermo-Nuclear Code Quality Review 是一个严格的代码可维护性审查技能。它的核心使命不是判断代码“是否能工作”,而是追问“这次改动是否让下一次修改更容易”。审查者被要求以“对接下来六个月改动负责”的心态来审视代码,删除偶然复杂性,同时保留领域边界、正确性、可审计性、可观测性、安全性、失败处理和真正的不变量。

主要能力

该技能围绕一组核心问题展开审查:这段代码能否被删除?能否更简单?这是偶然复杂性还是本质复杂性?分支能否转化为模型?重复能否收敛为单一事实来源?垂直切片是否清晰(输入 → 领域 → 副作用 → 输出 → 测试)?功能是否有清晰的归属,还是到处泄漏?测试是否在验证行为而非实现?文件是否超过 1000 行或变得难以扫描?相关更新能否原子化?

技能内置 8 个审查视角(lenses),并附带预设的 blocker/serious 清单,详见 REFERENCE.md。

使用方式

快速开始流程:先阅读 PR 描述、提交信息和行内注释;将没有关联 issue 编号的“推迟技术债”理由标记为发现项;运行各审查视角;按指定格式输出发现项,并附上最终裁决。

如果环境支持并行代理,可采用子代理模式:Agent A 负责代码柔道与 DRY;Agent B 负责垂直切片与归属;Agent C 负责测试与未来可测试性;Agent D 负责文件形态与编排。父代理合并发现项、去重,只保留最强的根因。若无法使用子代理,则顺序模拟各视角。

发现项输出格式包含编号、严重级别(Blocker | High | Medium | Low)、一行描述(附具体文件/函数/分支证据)以及建议修复方案。严重级别映射规则为:blocker → Blocker,serious → High,consider → Medium 或 Low。要求具体,禁止泛泛而谈。

适用场景

适用于代码合并前的质量把关,尤其是需要评估结构可维护性、控制技术债、确保垂直切片清晰、测试保护有效接缝的场景。最终裁决规则:存在任何 Blocker 发现则 Block;存在应在合并前修复的 High 发现则 Request changes;有真实顾虑但可接受则 Approve with concerns;结构合理、切片清晰、测试保护有用接缝则 Approve。只有在代码变得更简单或保持简单、复杂性被删除而非转移、垂直切片清晰、边界干净、文件形态可维护时,才给出 Approve。

限制

该技能聚焦于可维护性与结构质量,不替代功能正确性验证。其效果依赖审查者对代码库上下文的理解。子代理模式需要环境支持并行代理,否则需顺序执行。关于具体安装命令、平台支持范围、收费方式等信息,资料中暂未提供。