thermo-nuclear-review

Thermo Nuclear Review 用于对一个已检出的分支进行全面的安全与正确性审计，也可用于深度审查或分支/PR 差异审计。它是一个审查技能，角色定位为安全专家，要极其彻底、严谨地检查分支及其改动中的 bug、破坏现有功能或特性的改动以及安全漏洞。\n\n主要能力与关注点：\n范围上只报告本 PR 中新增或修改的代码相关问题，聚焦 diff 中的改动，不报告未改动代码的既有漏洞。破坏功能方面，强调复杂代码库存在跨包、跨模块依赖，简单改动的副作用可能破坏他处功能，因此必须追踪改动的可能副作用。开发者体验方面，要捕获影响本地运行/构建的改动，例如修改密钥读取方式或来源、更新环境变量名或新增环境变量、重映射端口或网络、新增必须运行才能继续使用某些功能的脚本；通过包管理器新增依赖不算破坏 devex，除非要求用户做超出常规工作流的新操作（如从网站或应用商店手动安装软件）。功能泄漏方面，不得让本应处于功能开关或内部检查之后的特性泄漏。\n\n处理“有意破坏”的准则：如果高风险发现正是分支意图（例如破坏某功能、移除功能开关、移除防护），且改动范围受限，则不应浪费作者时间上报；但若作者可能不了解改动的全部影响、低估负面影响，或改动可能恶意，仍应上报。过度上报方面：不要把非高优先级问题当成高优先级，优先级别必须准确，否则会失去开发者信任。\n\n最终响应要求：若有中高优先级或风险发现，且该分支存在 PR，则用 gh/glab cli 查看 PR/MR 讨论中 BugBot 或其他人的评论，纳入其发现；若其发现你遗漏的问题，判断有效性并加入报告；若有相同问题，考虑吸收其补充；在报告中标注由 BugBot 或他人发现的问题。注意必须先完成审计再查看 PR/MR 讨论，以保持视角。关键规则还包括：不得在未完成研究时呈现问题，例如有后端代码可查就不能说“客户端有问题，若后端处理则没问题”。\n\n适用场景是分支或 PR 差异的深度安全与正确性审查。限制方面：资料未提供安装命令、价格、平台支持或具体操作参数等信息。