ARTICLE DETAIL

资讯详情

深耕网站视觉设计与运营推广的一线实战洞察。

pstack的thermo-nuclear-code-quality-review:终极可维护性审计详解

pstack的thermo-nuclear-code-quality-review:终极可维护性审计详解 pstack的thermo-nuclear-code-quality-review终极可维护性审计详解【免费下载链接】pstack-claudeClaude Code, Codex, Copilot, Pi, OpenCode, Gemini, and Prime Agent versions of Potetos pstack. Rigorous agent workflows with Cursor primitives translated for other harnesses.项目地址: https://gitcode.com/GitHub_Trending/ps/pstack-claudepstack是面向 Claude Code、Codex、Pi 等 AI 编码智能体的技能栈而其中的thermo-nuclear-code-quality-review热核代码质量审查是一个极致严格的代码可维护性审计工具它会以近乎苛刻的标准审查你当前分支的改动专治巨型文件、面条式条件分支和腐化的抽象层并推动 Agent 主动做代码柔道式的结构重构而不是只挑几处小毛病就交差。它是什么一次不留情面的代码质量审计普通 code review 技能大多满足于功能对不对而 thermo-nuclear-code-quality-review 的定位完全不同——它的官方描述是Run an extremely strict maintainability review运行一次极其严格的可维护性审查聚焦三件事抽象质量这层封装真的值得存在吗还是只是套娃巨型文件一个 PR 有没有让文件突破 1000 行红线面条式增长新的 if 分支是不是被随手插进了不相干的流程它的核心指令要求审查者大胆ambitious不只是找局部清理点而是主动寻找能让实现大幅变简单、变小、更直接的重构路径。完整规则定义在 plugins/pstack/skills/thermo-nuclear-code-quality-review/SKILL.md。如何触发一条斜杠命令启动热核审查安装 pstack 插件后直接输入/thermo-nuclear-code-quality-review即可调用官方文档将其登记为extremely strict maintainability audit见 docs/reference.md 的斜杠命令表。也可以用自然语言触发比如对这个分支做一次深度代码质量审计或来一次最严厉的可维护性审查。它的审查基线可以概括为一句话对当前分支的改动做一次深度代码质量审计。在不改变行为的前提下重新思考如何组织与实现这些改动大胆重构把抽象、模块化和可读性提上去。三思而后行Measure twice, cut once。七大铁律审计时不可妥协的审查标准技能文件中列出了 7 条不可协商的硬性标准这是理解该技能的关键 #铁律一句话解释0结构简化要大胆别停在这里可以稍微干净点要寻找让整个分支、辅助函数、模式彻底消失的代码柔道解法11000 行红线PR 不应把文件从 1000 行以下推到以上除非有极强的结构性理由否则必须先拆分2禁止面条式蔓延往无关流程里插临时 if、散落特例是设计问题而非风格问题3设计优先于能跑就行行为不变但结构更干净时坚决推动更干净的版本4直白优于魔法警惕隐藏简单数据形状假设的通用魔法机制和薄封装5类型与边界要干净质疑不必要的any、unknown、可选参数和类型断言6逻辑放在规范层复用现有工具函数反对重复造轮子和架构漂移第 7 条则关注编排质量无意义的串行化、可能留下半更新状态的非原子操作都是设计坏味道。审查者会问你的 13 个问题对每一处有意义的改动审查者都会过一遍这套问题清单其中几个最扎心的是否存在一个代码柔道动作能让实现变得简单得多这次改动是改善了还是恶化了局部架构一个此前内聚的模块是否变得更耦合、更有状态、更难扫读重复出现的条件分支是不是暗示缺失了一个模型这个抽象真的在创造价值还是只是一个 wrapper会被重点打击的 15 类坏味道技能文件明确列出了要激进上报的问题包括文件因 PR 突破 1000 行、特例分支被焊死在不相关代码路径上、功能逻辑泄漏进通用模块、复制粘贴而非抽取、临时分支大概率变成永久债务、以及通过测试但让代码更不模块化的重构等。与之配套它还规定了一组优先修复方向删除整层间接性而不是打磨它、重设状态模型让条件分支消失、把特例逻辑变成更少例外的默认流程、用类型化模型或显式分发器替换条件链、把独立工作并行化、把相关更新重构成更原子的流程。原文强调不要满足于也许可以改个名这种反馈当真正的问题在结构层面时。审计输出一份分层报告而不是一锅乱炖审查的可交付物有严格格式 一个独立目录例如/tmp/project-thermo/内含一份总结报告 每个子系统的详细报告01_subsystem.md、02_subsystem.md……总结报告控制在约 200 行内一眼读完结论、每条发现的简短叙述、修复顺序建议、指向详细报告的链接详细文件承载深度测量数据、产生数据的命令、验证状态、可操作的代码柔道提案发现项用散文段落书写——点名文件和证据用句子解释问题与解法禁止碎片行和标签堆砌发现项按优先级排序结构性回归 错失的大幅简化机会 分支复杂度增长 边界/抽象/类型契约问题 文件体积 模块化 可读性。原则是少量高置信度的意见好过长篇大论的表面小疵。审批门槛不通过它似乎能跑就批准这是该技能最狠的地方。它明确拒绝仅因行为似乎正确就批准并列出若干推定阻断项presumptive blockers❌ 文件从 1000 行以下被推到以上❌ 在已有流程中加入临时分支使其更纠缠❌ 添加不必要的抽象、wrapper 或类型断言密集契约❌ 在已有规范工具函数存在时重复造轮子❌ 存在可行代码柔道路径却保留了大量偶然复杂度作者若不能清晰论证就必须接受推动更干净的拆分这一结论。组合技与 swarm 并行审查的威力thermo-nuclear 审查不是孤岛。当它借助 pstack 的swarm技能plugins/pstack/skills/swarm/SKILL.md扇出并行审查者时技能文件特别规定各审查者的独立报告就是交付物的详细文件而非可丢弃的原始输出——判断汇总进总结细节保留在磁盘上并互相链接。它与 pstack 的其他审查类技能形成互补interrogate多模型对抗式审查靠模型多样性找盲点偏挑错thermo-nuclear-code-quality-review单视角但极严偏可维护性与结构健康deslop提交前给 diff 做一轮去噪清理一键安装在你的智能体里启用热核审查在 Claude Code 中执行两条命令即可安装 pstack 插件含本技能/plugin marketplace add michael-denyer/pstack-claude /plugin install pstackpstack-claudeCodex 和 Pi 用户则分别运行codex plugin marketplace add/pi install对应命令详见 README.md 的 Install 章节。需要说明的是该技能源自 Cursor 的 cursor-team-kit见 NOTICE-skills.mdpstack 项目将其移植为跨运行时可用的技能规则本身保持原样。写在最后为什么需要这么狠的审查AI 编码智能体写代码的速度越快代码腐化的速度也越快——临时分支、膨胀的文件、层层套娃的抽象正是临时方案永久化的高发区。thermo-nuclear-code-quality-review 的价值在于它把资深工程师才会坚持的结构洁癖固化成了可重复执行的流程让 Agent 在合并前主动回答一个问题——这次改动是让代码库变干净了还是变脏了【免费下载链接】pstack-claudeClaude Code, Codex, Copilot, Pi, OpenCode, Gemini, and Prime Agent versions of Potetos pstack. Rigorous agent workflows with Cursor primitives translated for other harnesses.项目地址: https://gitcode.com/GitHub_Trending/ps/pstack-claude创作声明:本文部分内容由AI辅助生成(AIGC),仅供参考
返回列表