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 从公开仓库中收录这些内容。

举报或申请下架