OC PR Review

---

name: pr-review description: "PR 审查技能 — 完整的 Pull Request 审查流程，集成 GitHub CLI，支持安全分析、测试覆盖、性能评估、代码质量检查，并可推送飞书通知。Use when: (1) 审查 PR，(2) 检查 PR 安全性，(3) 分析测试覆盖，(4) 评估代码质量，(5) 生成审查报告。触发词：'review PR', '审查PR', '代码审查', 'pr review', '检查PR'。" metadata: { "openclaw": { "emoji": "🔍", "requires": { "bins": ["gh"] }, "install": [ { "id": "brew", "kind": "brew", "formula": "gh", "bins": ["gh"], "label": "Install GitHub CLI (brew)", }, ], }, }

---

PR Review Skill

完整的 Pull Request 审查工作流，涵盖安全分析、测试覆盖、性能评估、代码质量检查。

When to Use

✅ USE this skill when:

审查 Pull Request 的代码变更

检查 PR 的安全漏洞和风险

分析测试覆盖情况

评估代码变更的性能影响

生成审查报告并通知到飞书

❌ DON'T use this skill when:

本地 git 操作（commit、push、pull）→ 直接用 git

非 GitHub 仓库（GitLab、Bitbucket）→ 不同的 CLI

简单查看 PR 状态 → 直接用 gh pr view

写代码或修复 → 用 coding-agent skill

Prerequisites

# 确认 gh 已认证
gh auth status
如果未认证，执行
gh auth login

审查流程（6 步）

Step 1: 获取 PR 信息
    ↓
Step 2: 分析变更内容
    ↓
Step 3: 安全审查
    ↓
Step 4: 测试覆盖检查
    ↓
Step 5: 性能 & 质量评估
    ↓
Step 6: 生成报告 + 飞书通知

---

Step 1: 获取 PR 信息

确定 PR 目标

用户可能提供以下格式之一：

PR URL: https://github.com/owner/repo/pull/123

PR 编号: #123（需要在 git 仓库中）

分支名: feature/xxx（需查找对应的 PR）

# 从 URL 解析
owner/repo: 从 URL 提取
pr_number: 从 URL 提取
获取 PR 详情
gh pr view  --repo  --json title,body,author,state,additions,deletions,changedFiles,files,reviews,comments,labels,mergeable,headRefName,baseRefName
获取 PR 的文件变更列表
gh pr diff  --repo  --stat
获取完整 diff
gh pr diff  --repo 

如果用户只提供了分支名

# 查找分支对应的 PR
gh pr list --repo  --head  --json number,title

如果在 git 仓库内且未指定 PR

# 获取当前分支的 PR
gh pr view --json number,title,body,state

---

Step 2: 分析变更内容

获取 diff 后，按以下维度分析：

变更概览

📊 变更概览
文件数: X 个文件变更
新增: +X 行
删除: -X 行
涉及模块: [列出模块/目录]

文件分类

将变更文件分为以下类别：

| 类别 | 说明 | 关注点 | |------|------|--------| | 🔐 安全相关 | auth、crypto、permission、config | 注入、暴露、权限绕过 | | 🧪 测试文件 | test、spec、__tests__ | 覆盖率、边界用例 | | 📦 依赖变更 | package.json、requirements.txt | 版本锁定、已知漏洞 | | 🏗️ 核心逻辑 | 业务代码、API、数据库 | 逻辑正确性、性能 | | 📝 文档 | README、docs | 是否同步更新 | | ⚙️ 配置 | CI/CD、Dockerfile、环境变量 | 安全配置、敏感信息 |

---

Step 3: 安全审查

对每个变更文件检查以下安全维度：

安全检查清单

🔐 安全审查
认证/授权 (Auth)
[ ] 是否有未授权的端点暴露？
[ ] 权限检查是否完整？
[ ] Token/Session 处理是否安全？
数据暴露 (Data Exposure)
[ ] 是否有敏感数据泄露到日志？
[ ] API 响应是否暴露内部信息？
[ ] 错误消息是否泄露系统细节？
注入风险 (Injection)
[ ] SQL 注入：是否使用参数化查询？
[ ] XSS：用户输入是否正确转义？
[ ] 命令注入：是否有未经验证的 shell 执行？
[ ] 路径遍历：文件路径是否验证？
加密与密钥 (Crypto)
[ ] 是否使用了弱加密算法？
[ ] 密钥是否硬编码？
[ ] 是否使用安全的随机数生成？
依赖安全 (Dependencies)
[ ] 新增依赖是否有已知漏洞？
[ ] 版本是否锁定？
[ ] 是否引入了不必要的权限？

安全严重性分级

| 级别 | 标记 | 说明 | 处理 | |------|------|------|------| | 🔴 Critical | CRITICAL | 可被远程利用，影响数据安全 | 必须修复才能合并 | | 🟠 High | HIGH | 潜在安全风险，需要关注 | 建议修复后合并 | | 🟡 Medium | MEDIUM | 最佳实践问题 | 建议改进 | | 🟢 Low | LOW | 代码风格/建议 | 可选改进 |

---

Step 4: 测试覆盖检查

测试分析清单

🧪 测试覆盖分析
新增代码测试
[ ] 新增功能是否有对应测试？
[ ] 测试是否覆盖正常流程？
[ ] 测试是否覆盖异常/边界用例？
测试质量
[ ] 断言是否有意义（非 trivial assert）？
[ ] 测试是否独立（无共享状态）？
[ ] Mock/Stub 使用是否合理？
回归测试
[ ] 修改的逻辑是否有回归测试？
[ ] 是否破坏了现有测试？
覆盖率评估
关键路径覆盖率: __%
建议覆盖率目标: 80%+

测试建议模板

## 🧪 测试建议
缺失测试
1. [文件路径] — [建议添加的测试]
2. ...
边界用例
1. [场景描述] — [建议的测试方法]
2. ...
测试改进
1. [现有测试] — [改进建议]

---

Step 5: 性能 & 质量评估

性能检查

⚡ 性能评估
算法复杂度
[ ] 是否有 O(n²) 或更高复杂度的操作在热点路径？
[ ] 是否有不必要的嵌套循环？
数据库/IO
[ ] 是否有 N+1 查询问题？
[ ] 是否有不必要的全表扫描？
[ ] 文件操作是否有适当的缓冲？
内存使用
[ ] 是否有内存泄漏风险？
[ ] 大数据集是否分页/流式处理？
缓存
[ ] 是否应该缓存的结果未缓存？
[ ] 缓存失效策略是否正确？

代码质量

📝 代码质量
可读性
[ ] 命名是否清晰、一致？
[ ] 函数/方法是否职责单一？
[ ] 是否有过度复杂的条件逻辑？
可维护性
[ ] 是否遵循项目的代码规范？
[ ] 是否有重复代码可以提取？
[ ] 错误处理是否完整？
架构
[ ] 变更是否符合项目架构模式？
[ ] 是否引入了不当的耦合？
[ ] 是否有向后兼容性问题？

---

Step 6: 生成报告 + 飞书通知

审查报告模板

# 🔍 PR 审查报告
PR: # —