SuperAPI AI 工具集

implement-review-pr

安全与审计

查看项目

更新于 10-15 21:40

技能介绍

用途:对 GitHub PR 进行只读式审查,覆盖 CI 状态、提交标题规范、规格符合度、代码质量与安全(OWASP Top 10),并可选地在用户确认后向 GitHub 提交审查意见。

前提条件:已安装并认证 gh CLI。

主要流程:

  1. 获取 PR 信息:gh pr view <number-or-url>、gh pr diff <number-or-url>。
  2. 检查 CI:gh pr checks <number-or-url>,确认全部通过,失败则先向用户报告。
  3. 校验标题是否符合 Conventional Commits(type(scope): subject,有效 type 含 feat/fix/docs/refactor/test/chore/style/build/ci/perf,subject 72 字以内)。
  4. ①规格符合度审查:是否满足关联 Issue 的需求与验收条件、是否混入 out-of-scope 实现、标题与 body 是否与改动一致。
  5. ②代码质量审查(含安全):架构与命名规范、import 规则、前端不必要的重渲染风险、测试覆盖;安全项包括 OWASP Top 10、密钥泄露、输入校验、认证授权、敏感数据处置。
  6. 生成评审报告,含 CI 状态、标题合规、①规格符合、②代码质量与安全(安全作为②内的小节)等部分。
  7. 可选发布评论:gh pr review <number> --comment --body "..."、--approve 或 --request-changes --body "..."。

out-of-scope 处理:默认只读;仅当发现超出当前范围的改进、后续工作等事项时,经用户批准后例外执行写入,可用 gh issue list --state open --search "${KEYWORD}" 查找已有 Issue,再评论或新建。关键词与正文用 "${var}" 引用并以 HEREDOC 传入以防注入。报告阶段未知的 Issue 编号标注为 TBD。

限制:需要网络访问 GitHub,相关命令需按命令粒度关闭 sandbox;网络被阻断的环境无法运行。存在 HIGH 级安全问题或 CI 失败时建议阻塞评审。验证要求报告中同时具备①规格符合与②代码质量/安全两节,且不得在无证据时声称完成。常见失败包括跳过规格审查、把安全审查拆成独立章节、未确认 CI 就进入代码审查。