SuperAPI AI 工具集

review-pr

安全与审计

查看项目

更新于 10-15 21:08

技能介绍

该技能用于审查 denoland/deno 仓库的 Pull Request,围绕正确性、测试、安全和项目规范给出评审意见。触发条件是:被要求审查某个 PR,或提供了 PR 编号/URL。

工作流程分为五步。第一步用 gh 命令收集 PR 上下文,包括元数据、diff、评论与 CI 检查状态。第二步做前置门槛检查:CI 必须全部通过;PR 标题须符合 type(scope): description 格式(类型如 feat、fix、perf、refactor、chore、docs、test、revert、BREAKING);不允许 force push;改动范围要聚焦,不带无关清理;疑似 AI 生成需有声明;外部贡献者的 PR 必须关联 issue。任一门槛不通过需在评审顶部显著标出且不得批准。

第三步逐文件审查代码。Rust 方面关注边界情况、错误处理、性能、unsafe 使用、权限系统接入、依赖引入合理性。JS/TS 方面关注 ext/node 的 Node.js 兼容性、primordials 使用、Web 标准与 WPT 覆盖、懒加载。测试要求每个 bug 修复配回归测试,功能需覆盖正常路径和边界情况。安全敏感区域包括 runtime/permissions.rs、ext/net、ext/fs、ext/node、cli/tools/compile.rs 及任何调用外部命令或处理用户路径/URL 的代码。

第四步按 PR 类型补充检查,如 Node.js 兼容、性能基准、依赖更新、WPT 变更、CI/发布工具链(需转交维护者,不得自行批准)。第五步用 gh pr review 或 gh api 提交评审,给出摘要、阻塞问题、具体行内意见和结论;语气直接、友善且简洁。

限制:该技能不具备合并权限,PR 准备好后只能评论请维护者签署;不得批准 CI 失败的 PR,不得批准绕过权限系统的 PR,不得批准未标记讨论的大型架构改动;对通过 lint 的代码不做风格挑剔;对自动化已强制的内容不要求改动;向 GitHub 发布任何评审评论前必须先与用户确认。以上均以资料为准,未提供的细节暂不明确。