Pr Review Analysis

PR Review 分析 Skill

本 Skill 提供 PR Code Review 评论分析的知识库，用于支持 /fix-pr-review 工作流。

1. 置信度评估体系

1.1 评分因素

置信度表示评论的"可操作性"，分数范围 0-100。

1.2 评分指标

明确性 (Clarity) - 权重 40%

具体性 (Specificity) - 权重 30%

上下文 (Context) - 权重 20%

可复现 (Reproducibility) - 权重 10%

1.3 置信度等级

2. 优先级分类体系

2.1 优先级定义

2.2 分类关键词

P0 (Blocker) 关键词

安全相关（自动升级 2 个优先级）：

• security, vulnerability, injection
• XSS, CSRF, leak, exposed
• sensitive, password, token, secret
• 安全, 漏洞, 泄露, 暴露

关键缺陷：

• crash, data loss, downtime
• blocker, production, urgent
• 崩溃, 数据丢失, 紧急, 阻塞

P1 (Critical) 关键词

• bug, broken, fail, error
• incorrect, doesn't work, not working
• wrong, invalid, missing
• 错误, 失败, 不正确, 缺失

P2 (Major) 关键词

• should, better, improve
• optimize, refactor, performance
• cleanup, simplify
• 应该, 改进, 优化, 重构

P3 (Minor) 关键词

• consider, maybe, could
• nit, style, minor, typo
• nitpick, suggestion
• 建议, 风格, 小问题

2.3 优先级提升规则

3. 技术栈识别

3.1 路径模式匹配

Backend

patterns:
  - "src/api/**"
  - "src/models/**"
  - "src/services/**"
  - "app/**"
  - "tests/backend/**"
  - "tests/unit/**"
  - "**/*.py"

Frontend

patterns:
  - "src/components/**"
  - "src/pages/**"
  - "src/hooks/**"
  - "src/stores/**"
  - "tests/frontend/**"
  - "**/*.tsx"
  - "**/*.jsx"

E2E

patterns:
  - "tests/e2e/**"
  - "e2e/**"
  - "playwright/**"
  - "cypress/**"

3.2 文件扩展名推断

4. 常见评论模式

4.1 Backend 常见评论

数据库相关

API 相关

认证相关

4.2 Frontend 常见评论

React 相关

测试相关

4.3 E2E 常见评论

5. 回复最佳实践

5.1 回复原则

1. 感谢 Reviewer：始终表示感谢
2. 说明行动：清楚描述做了什么
3. 提供证据：链接到修复代码/测试
4. 开放沟通：邀请进一步讨论

5.2 回复模板

已修复

✅ 已修复

感谢指出！已在 `{commit}` 中完成修复。

**变更**：
- {变更描述}

**测试**：
- ✅ {测试名称} 通过

需要澄清

⏸️ 需要更多信息

感谢建议！为了更好地理解，能否提供：
1. {问题 1}
2. {问题 2}

不采纳（有理由）

ℹ️ 暂不修改

感谢建议！经过分析，当前实现是预期行为，原因：
- {原因}

如果您有不同看法，欢迎继续讨论。

5.3 避免的回复

• ❌ "你的建议不对"
• ❌ "代码已经这样写了"
• ❌ 不提供任何解释的 "已修复"
• ❌ 防御性语气

6. 时间窗口过滤

6.1 过滤规则

有效评论条件：

• 评论创建时间 > 最后 commit 时间
• 或评论更新时间 > 最后 commit 时间（有新回复）

6.2 时区处理

• 所有时间使用 UTC
• GitHub API 返回的时间已是 UTC
• 比较前确保时区一致

6.3 边界情况

7. TDD 集成

7.1 修复流程

所有 PR Review 修复必须遵循 TDD：

1. RED：编写能复现评论问题的测试
2. GREEN：最小实现使测试通过
3. REFACTOR：优化代码

7.2 测试命名

# 格式: test_{功能}_{评论描述}
def test_token_validation_returns_401_when_expired():
    """
    PR Review: rc_123456
    Reviewer: @alice_dev
    """
    pass

7.3 覆盖率要求

• 新增代码：100% 覆盖
• 修改代码：不低于原覆盖率
• 总体覆盖：>= 90%

8. 知识沉淀

8.1 何时沉淀

• P0/P1 评论的修复
• 置信度 >= 85 的评论
• 新发现的问题模式

8.2 沉淀内容

## {问题模式名称}

**频率**: ★★★☆☆
**技术栈**: Backend/Frontend/E2E
**关键词**: token, expire, validation

### 问题描述
{描述}

### 解决方案
{TDD 修复代码示例}

### 检查清单
- [ ] 检查项 1
- [ ] 检查项 2

8.3 沉淀位置

• 通用模式：docs/best-practices/pr-review-patterns.md
• 技术栈特定：docs/best-practices/{stack}/*.md