自动化的 GitHub PR 代码审查,具备差异分析、lint 集成和结构化报告。适用于审查 pull request、检查安全问题等。
PR Reviewer : GitHub 拖动请求的自动代码审查
功能概述
PR Reviewer : GitHub 拖动请求的自动代码审查是一项面向实际任务的技能,主要用于分析 diffs 的安全问题、 错误处理漏洞、 样式问题和测试覆盖;Prerequistites.gh CLI 已安装和认证。
核心要点
- 它将相关步骤、工具调用和结果整理方式集中到统一流程中,帮助使用者更快完成目标并减少重复操作。
- 使用时应结合输入条件选择合适的执行方式,核对必要参数、依赖环境与输出内容,并按原始要求处理异常情况。
- 该技能适合需要稳定复用相关能力的场景,可作为自动化工作流的一部分,也便于后续检查、调整和扩展。
使用与执行
从功能定位来看,该技能强调把分散的操作要求整理成清晰、可复用的处理流程,使用户能够围绕既定目标快速准备输入、选择执行方式并获得结构化结果。实际使用前应先确认任务范围、数据来源、运行环境、必要权限和关键参数,再依据技能说明逐步执行;
结果检查与注意事项
若输入条件不完整,应先补齐信息或采用保守配置,避免因错误假设导致结果偏离需求。执行过程中需要关注工具调用是否成功、接口或依赖是否可用、输出格式是否符合预期,并对异常提示、缺失字段和边界情况进行处理;涉及批量任务时,还应保存进度,避免中断后重复操作。
PR Reviewer
面向 GitHub Pull Request 的自动化代码审查工具。分析代码差异(diff),识别安全问题、错误处理缺失、风格问题及测试覆盖率不足等情况。
前置条件
- 已安装并完成身份验证的
ghCLI(运行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(测试覆盖率) | 📊 | 源文件发生变更但未同步更新对应测试文件 |
智能重审(Smart Re-Review)
为每个 PR 记录其 HEAD SHA 值。仅当有新提交被推送时才触发重新审查。如需强制重新审查,请使用 review 命令。
报告格式
报告以 Markdown 文件形式保存在输出目录中。每份报告均包含以下内容:
- PR 元数据(作者、分支、变更摘要)
- 提交列表
- 按编程语言/类型归类的已变更文件
- 自动化 diff 分析结果,含文件路径、行号、问题类别与上下文代码片段
- 测试覆盖率分析
- 本地 lint 检查结果(仅当仓库已在本地检出时生效)
- 总结性结论:🔴 SECURITY / 🟡 NEEDS ATTENTION / 🔵 MINOR NOTES / ✅ LOOKS GOOD
心跳检测 / 定时任务(Heartbeat/Cron)集成
可将以下逻辑加入周期性检查流程(例如心跳服务、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'))
热门AI工具
相关专题
本专题介绍Gradle插件开发的基本流程,涵盖插件类编写、自定义任务注册、扩展参数设计和插件应用方法,帮助你理解插件如何组织构建逻辑,减少多个项目之间的重复配置。
0
2026.10.22
本专题介绍Gradle Wrapper的基本作用,涵盖文件组成、生成方法、版本配置和任务调用,帮助你理解项目如何使用指定的Gradle版本,减少不同开发环境带来的构建差异。
0
2026.10.22
本专题整理Golangci-lint常用Linters的用途与配置方法,覆盖错误处理、无效代码、安全、复杂度、命名和依赖检查,帮助项目合理选规则。
0
2026.10.22
本专题介绍Gradle Java项目构建方法,涵盖Java插件配置、依赖声明、源码编译、测试执行和打包检查,帮助你理解各阶段的任务关系,逐步完成项目构建。
0
2026.10.22
本专题汇总PHP常见时间戳漏洞,包括2038年溢出、时区漂移、时间戳伪造、重放攻击、时间盲注与日志审计失效等,并给出升级64位环境、使用 DateTimeImmutable、统一UTC、严格超时校验、预处理绑定、hash_equals 比对等修复方案,帮助开发者快速排查和加固时间相关代码。
0
2026.10.22
本专题聚焦Golangci-lint YAML配置文件,讲清运行参数、规则启停、目录排除、问题过滤及严重级别设置,便于团队统一检查标准。
0
2026.10.22
本专题讲解Golangci-lint的安装、初始化、基础命令和首次代码检查,覆盖本地环境、Docker及多平台用法,帮助Go开发者快速上手。
0
2026.10.22
PHP2Shell漏洞是什么,哪些PHP版本和运行环境会受到影响,服务器应该如何检测与修复?本专题提供版本自查脚本、Linux与Docker升级命令、PHP-FPM和Apache排查步骤,以及修复验证、临时缓解和应急响应方案。
0
2026.10.22
热门下载
相关下载
精品课程
共6课时 | 12.1万人学习
共79课时 | 161.5万人学习
共6课时 | 54.7万人学习
最新文章
