multi-reviewer-patterns

マルチレビュアーパターン

複数の品質観点にわたる並列コードレビューを調整し、指摘事項を重複排除し、重大度を調整し、統合レポートを作成するためのパターンです。

このスキルを使う場面

• 多次元のコードレビューを整理するとき
• どのレビュー観点を割り当てるか決めるとき
• 複数のレビュアーからの指摘事項を重複排除するとき
• 重大度評価を一貫して調整するとき
• 統合されたレビューレポートを作成するとき

レビュー観点の割り当て

利用可能な観点

観点	焦点	含めるべき場面
セキュリティ	脆弱性、認証、入力検証	ユーザー入力や認証を扱うコードでは常に
パフォーマンス	クエリ効率、メモリ、キャッシュ	データアクセスやホットパスを変更するとき
アーキテクチャ	SOLID、結合度、パターン	構造変更や新規モジュールのとき
テスト	カバレッジ、品質、エッジケース	新機能を追加するとき
アクセシビリティ	WCAG、ARIA、キーボードナビゲーション	UI／フロントエンドの変更のとき

推奨される組み合わせ

シナリオ	観点
API エンドポイントの変更	セキュリティ、パフォーマンス、アーキテクチャ
フロントエンドコンポーネント	アーキテクチャ、テスト、アクセシビリティ
データベースマイグレーション	パフォーマンス、アーキテクチャ
認証の変更	セキュリティ、テスト
機能全体のレビュー	セキュリティ、パフォーマンス、アーキテクチャ、テスト

指摘事項の重複排除

複数のレビュアーが同じ箇所で問題を報告した場合：

マージルール

1. 同じ file:line、同じ問題 — 1 つの指摘事項にマージし、すべてのレビュアーの貢献を記録する
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

重大度の調整

重大度の基準

重大度	影響	発生可能性	例
Critical	データ損失、セキュリティ侵害、完全な障害	確実またはきわめて高い	SQL インジェクション、認証バイパス、データ破損
High	重大な機能影響、劣化	高い	メモリリーク、検証の欠如、フローの破綻
Medium	部分的な影響、回避策が存在する	起こりうる	N+1 クエリ、エッジケースの欠如、不明瞭なエラー
Low	最小限の影響、見た目上の問題	低い	スタイルの問題、軽微な最適化、命名

調整ルール

• 外部ユーザーが悪用可能なセキュリティ脆弱性：常に Critical または High
• ホットパスにおけるパフォーマンス問題：少なくとも Medium
• クリティカルパスに対するテストの欠如：少なくとも Medium
• 中核機能に対するアクセシビリティ違反：少なくとも Medium
• 機能的な影響がないコードスタイルの問題：Low

統合レポートのテンプレート

## 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}