Multi Reviewer Patterns

作者 wshobson46891e7e60da無授權條款收錄於 2026年10月8日更新於 2026年10月8日

Coordinate parallel code reviews across multiple quality dimensions with finding deduplication, severity calibration, and consolidated reporting. Use this skill when organizing multi-reviewer code reviews, calibrating finding severity, or consolidating review results.

AI 產生的概覽

用於協調多維度平行程式碼審查、去除重複發現、校準嚴重程度並產出彙總報告的模式。

功能
提供組織多審查人程式碼審查的指引,涵蓋安全性、效能、架構、測試與無障礙等維度。它定義了同一位置發現的合併規則與去重流程、嚴重程度標準與校準規則,以及包含彙總表的合併報告範本。
適用情境
適用於組織多維度程式碼審查、決定指派哪些審查維度、對多位審查人的發現進行去重、一致地校準嚴重程度評級,或產出合併審查報告。適合處理使用者輸入或驗證的程式碼、資料存取或熱路徑變更、結構性變更、新功能,以及 UI 或前端變更。不用於撰寫或除錯程式碼本身。
執行需求
無需腳本或工具;僅為說明性內容,並附帶一份關於審查維度的參考檔案。

Multi-Reviewer Patterns

Patterns for coordinating parallel code reviews across multiple quality dimensions, deduplicating findings, calibrating severity, and producing consolidated reports.

When to Use This Skill

  • Organizing a multi-dimensional code review
  • Deciding which review dimensions to assign
  • Deduplicating findings from multiple reviewers
  • Calibrating severity ratings consistently
  • Producing a consolidated review report

Review Dimension Allocation

Available Dimensions

DimensionFocusWhen to Include
SecurityVulnerabilities, auth, input validationAlways for code handling user input or auth
PerformanceQuery efficiency, memory, cachingWhen changing data access or hot paths
ArchitectureSOLID, coupling, patternsFor structural changes or new modules
TestingCoverage, quality, edge casesWhen adding new functionality
AccessibilityWCAG, ARIA, keyboard navFor UI/frontend changes

Recommended Combinations

ScenarioDimensions
API endpoint changesSecurity, Performance, Architecture
Frontend componentArchitecture, Testing, Accessibility
Database migrationPerformance, Architecture
Authentication changesSecurity, Testing
Full feature reviewSecurity, Performance, Architecture, Testing

Finding Deduplication

When multiple reviewers report issues at the same location:

Merge Rules

  1. Same file:line, same issue — Merge into one finding, credit all reviewers
  2. Same file:line, different issues — Keep as separate findings
  3. Same issue, different locations — Keep separate but cross-reference
  4. Conflicting severity — Use the higher severity rating
  5. Conflicting recommendations — Include both with reviewer attribution

Deduplication Process

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

Severity Calibration

Severity Criteria

SeverityImpactLikelihoodExamples
CriticalData loss, security breach, complete failureCertain or very likelySQL injection, auth bypass, data corruption
HighSignificant functionality impact, degradationLikelyMemory leak, missing validation, broken flow
MediumPartial impact, workaround existsPossibleN+1 query, missing edge case, unclear error
LowMinimal impact, cosmeticUnlikelyStyle issue, minor optimization, naming

Calibration Rules

  • Security vulnerabilities exploitable by external users: always Critical or High
  • Performance issues in hot paths: at least Medium
  • Missing tests for critical paths: at least Medium
  • Accessibility violations for core functionality: at least Medium
  • Code style issues with no functional impact: Low

Consolidated Report Template

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

來源與署名

來源:wshobson/agents位於plugins/agent-teams/skills/multi-reviewer-patterns提交46891e7

授權條款: 無授權條款

內容歸原作者所有。SourceWeft 從公開儲存庫中收錄這些內容。

檢舉或申請下架