PR Doctor

PR全流程质量医生 — 自动化代码审查、测试分析、问题追踪和持续改进的完整流水线

PR Doctor - PR全流程质量医生

概述

PR Doctor 是一个完整的 Pull Request 质量保障流水线，同时调用多个专业 Skill 完成从代码审查到问题追踪的全流程。

解决问题：开发者花费大量时间手动审查 PR，人工追踪问题和跟进效率低下。

核心工作流

[PR URL/Repo] 
    │
    ├─→ code-review-skill    (代码审查 + 质量评分)
    │
    ├─→ testing-patterns     (测试覆盖分析 + 建议)
    │
    ├─→ github-issues-skill  (自动创建跟进issue)
    │
    └─→ self-improvement     (记录学习，持续改进)
            │
            ▼
      [综合审查报告]

使用方法

触发词

• 审查PR <PR_URL>
• PR质量检查 <owner/repo> <PR号>
• 检查我的PR
• PR健康检查

输入

提供以下任一信息：

• PR URL，例如：https://github.com/owner/repo/pull/123
• 或提供：owner/repo + PR 编号

执行流程

Step 1: 代码审查 调用 code-review-skill，使用 gh pr diff 获取 PR 差异，逐文件审查：

• 代码逻辑问题
• 安全漏洞
• 性能隐患
• 代码风格
• 边界条件

Step 2: 测试覆盖分析 调用 testing-patterns，分析：

• 新增代码的测试覆盖情况
• 是否有测试缺失的关键路径
• 是否违反 TDD 原则
• 测试命名和结构质量

Step 3: 创建跟进 Issue 调用 github-issues-skill，对于每个发现的问题：

• 按严重程度分级（critical/major/minor）
• 自动创建 GitHub Issue 描述问题
• 添加适当的标签（bug, security, performance, style）
• 关联到原始 PR

Step 4: 持续改进 调用 self-improvement：

• 记录本次审查中发现的新型问题模式
• 更新质量检查清单
• 促进后续审查效率提升

输出

综合审查报告，包含：

• PR 总体质量评分（1-10）
• 问题清单（按严重程度排序）
• 测试覆盖评估
• 建议行动项
• 相关 GitHub Issue 链接

质量评分标准

评分	等级	说明
9-10	🟢 Excellent	可直接合并
7-8	🔵 Good	小问题修复后可合并
4-6	🟡 Needs Work	需要重大修改
1-3	🔴 Poor	建议重新设计

审查维度

代码审查 (code-review-skill)

• ✅ 逻辑正确性
• ✅ 安全漏洞检测
• ✅ 性能影响评估
• ✅ 错误处理完整性
• ✅ 代码可读性
• ✅ 边界条件覆盖

测试分析 (testing-patterns)

• ✅ 测试覆盖率
• ✅ 边界条件测试
• ✅ Mock/Stub 使用合理性
• ✅ 测试命名描述性
• ✅ 测试结构（AAA模式）
• ✅ 回归测试完整性

示例对话

用户：审查这个PR: https://github.com/owner/project/pull/42

AI：正在启动 PR Doctor 流水线...

🔍 Step 1: 代码审查中... 📊 Step 2: 测试覆盖分析中... 📋 Step 3: 创建跟进Issue中... 📚 Step 4: 记录学习点... ✅ 审查完成！

综合报告：

• 质量评分：7.5/10 🟢
• 发现问题：3个（1个Critical, 2个Minor）
• 测试覆盖：良好，缺失边界测试
• 已创建Issue：#143, #144
• 建议：修复 Critical 问题后合并

注意事项

• 使用 gh CLI 进行 GitHub API 调用，需要先完成 gh auth login
• 代码审查为辅助性质，最终决策由人工做出
• 创建 Issue 前先搜索是否已有重复问题
• self-improvement 日志保存在 .learnings/ 目录