用途
Thermo Nuclear Review 用于对已检出的分支执行一次全面的安全与正确性审计。它面向需要深度审查分支或 PR diff 的场景,例如在合并前排查新增或修改代码中的缺陷、破坏性变更和安全漏洞。
主要能力
该技能以安全专家提示词为核心,要求审查者极其彻底、严谨、细致地审计分支及其改动,目标是“不让任何问题漏过”。其审查范围明确限定为本次 PR 中新增或修改的代码,聚焦 diff 中的改动,不报告未被改动的既有代码中的漏洞。
审查重点包括:
- 破坏功能:代码库复杂、跨包/跨模块依赖多,简单改动可能在其他位置产生微妙交互并破坏功能,必须彻底追踪可能的副作用。
- 破坏开发者体验:识别会影响开发者本地运行或构建代码的改动,例如修改密钥读取方式或读取位置、更新环境变量名或新增环境变量、重映射端口或网络、新增必须运行的脚本等。通过包管理器添加依赖不算破坏 devex,除非要求用户执行非常规的新操作(如从网站或应用商店手动安装软件)。
- 功能泄漏:代码库可能通过功能开关或仅内部检查来谨慎控制功能,必须防止本应处于功能门控之后的功能泄漏,这类泄漏往往很隐蔽。
- 有意破坏:如果高风险发现正是分支意图引入的(例如破坏某功能、移除功能开关、移除保护措施),且改动范围约束良好,则不应浪费作者时间报告;但如果作者可能未意识到改动的全部影响、低估负面影响,或改动可能具有恶意,则仍应报告。
- 避免过度报告:不得把非高优先级问题误报为高优先级,否则会失去开发者信任;必须端到端彻底追踪,在完全确信后再报告。
使用方式
该技能针对已检出的分支使用。审查者需先完成审计,之后如果存在中高优先级或风险发现,并且该分支有对应 PR,则使用 gh/glab CLI 检查 PR/MR 讨论中是否有 BugBot 或其他人的评论。若有,应将其发现纳入考量:若他们发现了你遗漏的问题,评估其有效性并纳入报告;若与你的发现相同,看是否有值得补充的内容。报告中需标注来自 BugBot 或 PR/MR 讨论中他人的发现。
关键规则:
- 绝不在研究未完成时呈现问题。例如,如果有权访问后端代码可以自行检查,就不要说“客户端有问题 X,但如果在后端处理就没问题”。
- 必须在完成审计之后再去检查 PR/MR 讨论,以便审查时保持新鲜视角。
- 必须极其彻底、严谨、细致、积极、专注,不让任何问题漏过。
适用场景
适用于需要“thermo nuclear”“thermonuclear”或深度审查请求的场景,也适用于分支或 PR diff 审计。
限制
资料未提供该技能的安装方式、运行环境要求、支持平台、收费或价格信息,暂未提供。