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 行
- 涉及模块: [列出模块/目录]

文件分类

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

Step 3: 安全审查

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

安全检查清单

🔐 安全审查

**认证/授权 (Auth)**
- [ ] 是否有未授权的端点暴露？
- [ ] 权限检查是否完整？
- [ ] Token/Session 处理是否安全？

**数据暴露 (Data Exposure)**
- [ ] 是否有敏感数据泄露到日志？
- [ ] API 响应是否暴露内部信息？
- [ ] 错误消息是否泄露系统细节？

**注入风险 (Injection)**
- [ ] SQL 注入：是否使用参数化查询？
- [ ] XSS：用户输入是否正确转义？
- [ ] 命令注入：是否有未经验证的 shell 执行？
- [ ] 路径遍历：文件路径是否验证？

**加密与密钥 (Crypto)**
- [ ] 是否使用了弱加密算法？
- [ ] 密钥是否硬编码？
- [ ] 是否使用安全的随机数生成？

**依赖安全 (Dependencies)**
- [ ] 新增依赖是否有已知漏洞？
- [ ] 版本是否锁定？
- [ ] 是否引入了不必要的权限？

安全严重性分级

Step 4: 测试覆盖检查

测试分析清单

🧪 测试覆盖分析

**新增代码测试**
- [ ] 新增功能是否有对应测试？
- [ ] 测试是否覆盖正常流程？
- [ ] 测试是否覆盖异常/边界用例？

**测试质量**
- [ ] 断言是否有意义（非 trivial assert）？
- [ ] 测试是否独立（无共享状态）？
- [ ] Mock/Stub 使用是否合理？

**回归测试**
- [ ] 修改的逻辑是否有回归测试？
- [ ] 是否破坏了现有测试？

**覆盖率评估**
- 关键路径覆盖率: __%
- 建议覆盖率目标: 80%+

测试建议模板

## 🧪 测试建议

### 缺失测试
1. [文件路径] — [建议添加的测试]
2. ...

### 边界用例
1. [场景描述] — [建议的测试方法]
2. ...

### 测试改进
1. [现有测试] — [改进建议]

Step 5: 性能 & 质量评估

性能检查

⚡ 性能评估

**算法复杂度**
- [ ] 是否有 O(n²) 或更高复杂度的操作在热点路径？
- [ ] 是否有不必要的嵌套循环？

**数据库/IO**
- [ ] 是否有 N+1 查询问题？
- [ ] 是否有不必要的全表扫描？
- [ ] 文件操作是否有适当的缓冲？

**内存使用**
- [ ] 是否有内存泄漏风险？
- [ ] 大数据集是否分页/流式处理？

**缓存**
- [ ] 是否应该缓存的结果未缓存？
- [ ] 缓存失效策略是否正确？

代码质量

📝 代码质量

**可读性**
- [ ] 命名是否清晰、一致？
- [ ] 函数/方法是否职责单一？
- [ ] 是否有过度复杂的条件逻辑？

**可维护性**
- [ ] 是否遵循项目的代码规范？
- [ ] 是否有重复代码可以提取？
- [ ] 错误处理是否完整？

**架构**
- [ ] 变更是否符合项目架构模式？
- [ ] 是否引入了不当的耦合？
- [ ] 是否有向后兼容性问题？

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

审查报告模板

# 🔍 PR 审查报告

**PR**: # —