自动化的 GitHub PR 代码审查,具备差异分析、lint 集成和结构化报告。适用于审查 pull request、检查安全问题等。
PR Reviewer : GitHub 拖动请求的自动代码审查是一项面向实际任务的技能,主要用于分析 diffs 的安全问题、 错误处理漏洞、 样式问题和测试覆盖;Prerequistites.gh CLI 已安装和认证。
从功能定位来看,该技能强调把分散的操作要求整理成清晰、可复用的处理流程,使用户能够围绕既定目标快速准备输入、选择执行方式并获得结构化结果。实际使用前应先确认任务范围、数据来源、运行环境、必要权限和关键参数,再依据技能说明逐步执行;
若输入条件不完整,应先补齐信息或采用保守配置,避免因错误假设导致结果偏离需求。执行过程中需要关注工具调用是否成功、接口或依赖是否可用、输出格式是否符合预期,并对异常提示、缺失字段和边界情况进行处理;涉及批量任务时,还应保存进度,避免中断后重复操作。
面向 GitHub Pull Request 的自动化代码审查工具。分析代码差异(diff),识别安全问题、错误处理缺失、风格问题及测试覆盖率不足等情况。
gh CLI(运行 gh auth status 可确认)golangci-lint 用于 Go 语言 lint 检查,或 ruff 用于 Python 语言 lint 检查# 审查当前仓库中所有打开状态的 PR scripts/github/pr-reviewer.sh check # 审查指定编号的 PR scripts/github/pr-reviewer.sh review 42 # 将审查结果以 GitHub 评论形式发布 scripts/github/pr-reviewer.sh post 42 # 查看所有打开状态 PR 的审查状态 scripts/github/pr-reviewer.sh status # 列出尚未审查的 PR(适用于心跳检测/定时任务集成) scripts/github/pr-reviewer.sh list-unreviewed
可通过设置以下环境变量进行配置;若未设置,脚本将自动从当前 Git 仓库中推断:
PR_REVIEW_REPO — GitHub 仓库名,格式为 owner/repo(默认值:由 gh repo view 自动检测)PR_REVIEW_DIR — 本地代码检出路径,用于执行 lint 检查(默认值:当前工作目录所在的 Git 仓库根目录)PR_REVIEW_STATE — 状态文件路径(默认值:./data/pr-reviews.json)PR_REVIEW_OUTDIR — 报告输出目录(默认值:./data/pr-reviews/)PR_REVIEW_STATE(默认值:./data/pr-reviews.json)—— 记录已被审查的 PR 及其 HEAD SHA 值PR_REVIEW_OUTDIR(默认值:./data/pr-reviews/)—— 存放 Markdown 格式的审查报告| 类别 | 图标 | 示例 |
|---|---|---|
| Security(安全) | 🔴 | 硬编码凭证、AWS 密钥、代码中明文存储密钥等 |
| Error Handling(错误处理) | 🟡 | 忽略错误(Go 中使用 _ :=)、裸 except:(Python)、未检查的 Close() 调用 |
| Risk(风险) | 🟠 | panic() 调用、process.exit() |
| Style(风格) | 🔵 | 生产环境中使用 fmt.Print/print()/console.log、超长行等 |
| TODOs(待办标记) | 📝 | 包含 TODO、FIXME、HACK、XXX 等标记的注释 |
| Test Coverage(测试覆盖率) | 📊 | 源文件发生变更但未同步更新对应测试文件 |
为每个 PR 记录其 HEAD SHA 值。仅当有新提交被推送时才触发重新审查。如需强制重新审查,请使用 review 命令。
报告以 Markdown 文件形式保存在输出目录中。每份报告均包含以下内容:
可将以下逻辑加入周期性检查流程(例如心跳服务、cron 任务或 CI 流程):
UNREVIEWED=$(scripts/github/pr-reviewer.sh list-unreviewed) if [ -n "$UNREVIEWED" ]; then scripts/github/pr-reviewer.sh check fi
脚本中的分析规则按编程语言组织。如需新增规则,请在 analyze_diff() 函数中对应语言的 pattern 列表末尾追加新条目:
# 添加一条新的 Go 语言规则
go_patterns.append((r'^+.*os.Exit(', 'RISK', 'Direct os.Exit() — consider returning error'))
相关专题
热门下载
相关下载
精品课程
共6课时 | 54.7万人学习
共89课时 | 134万人学习
共49课时 | 82.5万人学习
最新文章