multi-reviewer-patterns

多审查者模式

用于跨多个质量维度协调并行代码审查、对发现项去重、校准严重性并生成整合报告的模式。

何时使用本技能

• 组织多维度的代码审查
• 决定分配哪些审查维度
• 对来自多个审查者的发现项去重
• 一致地校准严重性评级
• 生成整合的审查报告

审查维度分配

可用维度

维度	关注点	何时纳入
安全	漏洞、认证、输入验证	处理用户输入或认证的代码始终纳入
性能	查询效率、内存、缓存	更改数据访问或热路径时
架构	SOLID、耦合、模式	结构性更改或新模块
测试	覆盖率、质量、边界情况	添加新功能时
无障碍	WCAG、ARIA、键盘导航	UI/前端更改

推荐组合

场景	维度
API 端点更改	安全、性能、架构
前端组件	架构、测试、无障碍
数据库迁移	性能、架构
认证更改	安全、测试
完整功能审查	安全、性能、架构、测试

发现项去重

当多个审查者在同一位置报告问题时：

合并规则

1. 同一 file:line，同一问题 — 合并为一个发现项，记功于所有审查者
2. 同一 file:line，不同问题 — 保留为单独的发现项
3. 同一问题，不同位置 — 保持独立但相互交叉引用
4. 严重性冲突 — 采用更高的严重性评级
5. 建议冲突 — 两者都包含，并注明审查者归属

去重流程

For each finding in all reviewer reports:
  1. Check if another finding references the same file:line
  2. If yes, check if they describe the same issue
  3. If same issue: merge, keeping the more detailed description
  4. If different issue: keep both, tag as "co-located"
  5. Use highest severity among merged findings

严重性校准

严重性标准

严重性	影响	可能性	示例
严重	数据丢失、安全漏洞、完全失效	确定或极可能	SQL 注入、认证绕过、数据损坏
高	重大功能影响、降级	可能	内存泄漏、缺少验证、流程中断
中	部分影响、存在变通方法	可能	N+1 查询、缺少边界情况、错误不清晰
低	影响极小、外观性	不太可能	风格问题、次要优化、命名

校准规则

• 外部用户可利用的安全漏洞：始终为严重或高
• 热路径中的性能问题：至少为中
• 关键路径缺少测试：至少为中
• 核心功能存在无障碍违规：至少为中
• 无功能影响的代码风格问题：低

整合报告模板

## Code Review Report

**Target**: {files/PR/directory}
**Reviewers**: {dimension-1}, {dimension-2}, {dimension-3}
**Date**: {date}
**Files Reviewed**: {count}

### Critical Findings ({count})

#### [CR-001] {Title}

**Location**: `{file}:{line}`
**Dimension**: {Security/Performance/etc.}
**Description**: {what was found}
**Impact**: {what could happen}
**Fix**: {recommended remediation}

### High Findings ({count})

...

### Medium Findings ({count})

...

### Low Findings ({count})

...

### Summary

| Dimension    | Critical | High  | Medium | Low   | Total  |
| ------------ | -------- | ----- | ------ | ----- | ------ |
| Security     | 1        | 2     | 3      | 0     | 6      |
| Performance  | 0        | 1     | 4      | 2     | 7      |
| Architecture | 0        | 0     | 2      | 3     | 5      |
| **Total**    | **1**    | **3** | **9**  | **5** | **18** |

### Recommendation

{Overall assessment and prioritized action items}