SuperAPI AI 工具集

multi-reviewer-patterns

编程与开发

查看项目

更新于 10-15 11:10

技能介绍

用途

Multi-Reviewer Patterns 是一套用于组织多维度并行代码审查的模式集合。它解决的核心问题是:当代码审查需要覆盖安全、性能、架构、测试、无障碍等多个质量维度时,如何分配审查任务、共享上下文、合并重复发现、统一严重度,并最终产出一份可用的合并报告。

主要能力

审查维度分配

模式列出了可用的审查维度及其关注点与纳入时机:

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

并给出推荐组合,例如 API 端点变更用安全、性能、架构;前端组件用架构、测试、无障碍;数据库迁移用性能、架构;认证变更用安全、测试;完整功能审查用安全、性能、架构、测试。

上下文共享

在流水线模式(未使用 --skip-interconnect)下,审查者不只收到原始代码,还会收到两类上下文产物:深挖输出(.deep-dive/ 下的结构、接口、风险等文件)和互连映射(.team-review/02-interconnect.md,包含契约、不变量、领域规则、假设、集成热点、变更影响半径)。审查者不应通读整个上下文文件,而应依据互连映射底部的 Reviewer Hints 按维度只读取相关锚点。模式给出了默认的锚点路由表,例如安全维度关注集成热点(入站)、未验证假设、隐式契约与输入校验;架构维度关注不变量、结构性与隐式契约、调用图等。

模式还提出一个质量指标:上下文利用率,即引用互连映射锚点的发现占比。高(>=30%)表示上下文被有效利用;中(10-30%)表示使用不一致;低(<10%)表示审查者忽略映射或映射过于笼统。logic-integrity-auditor 维度应达到 >=70%。

回退模式 --skip-interconnect:跳过流水线时,审查者只收到目标与 diff;不生成 logic-integrity-auditor;其他审查者回退到流水线前行为;提示中不出现 .deep-dive/ 或互连映射引用。

流水线约定

每个作为 Phase 2 运行的审查者都带有四条跨切面规则:范围预算(约 15 次文件读取无发现后返回 scope-off-topic 报告)、无发现协议(允许合法返回“已检查 X、Y、Z,无问题”)、跨审查者备注(把属于其他维度的观察写入 Cross-Reviewer Notes,供合并阶段路由)、互连锚点引用。

发现去重

合并规则:同一 file:line 同一问题合并并记功所有审查者;同一位置不同问题保留为独立发现;同一问题不同位置保留但交叉引用;严重度冲突取更高者;建议冲突则两者都保留并标注来源。去重流程按 file:line 比对、判断是否同一问题、合并保留更详细描述、不同问题标记共址、取最高严重度。

严重度校准

严重度分为 Critical、High、Medium、Low,各自对应影响、可能性与示例。校准规则包括:外部用户可利用的安全漏洞始终为 Critical 或 High;热路径性能问题至少 Medium;关键路径缺测试至少 Medium;核心功能无障碍违规至少 Medium;无功能影响的代码风格问题为 Low。

合并报告模板

模板包含目标、审查者、日期、审查文件数,按 Critical/High/Medium/Low 分组列出发现(编号、位置、维度、描述、影响、修复建议),并附按维度统计的汇总表与总体建议。

对抗式验证面板

发现合并后,每条被选中的发现由 3 个并行验证者从不同视角判定:Lens 1 可达性/正确性(判断缺陷是否真实且可达)、Lens 2 误报成因(主动反驳,默认不确定即判误报)、Lens 3 严重度校准(假定真实,只投票严重度)。存活规则:Lens 1-2 中至少 2 票 REAL 则存活;至少 2 票 FALSE_POSITIVE 则丢弃并计入 filtered;平局或不确定则存活并标记 contested。最终严重度取 Lens 3 的投票。验证者出错或返回畸形结果视为弃权,面板不会崩溃也不会静默丢弃发现。

进入面板的选择:默认所有置信度 >=50% 且通过去重的发现;成本护栏下(超过 25 条存活发现且未设 --rigorous)收窄到高风险与不确定带;--rigorous 忽略上限;--fast 跳过整个门控。成本护栏以发现数量(阈值 25)为代理,而非真实 token 消耗。

完整性批评者

验证之后,一个批评者代理检查审查遗漏了什么,按固定分类输出 Coverage Gaps:范围需要但未运行的维度、范围内无任何发现引用的文件、互连映射中未被任何发现处理的未验证假设、零发现的高风险热点。若且仅若某个缺口是高风险的未覆盖区域,则给出唯一一条最有价值的后续建议。

使用方式

该模式以技能形式提供,页面地址为 https://www.skills.sh/acaprino/daodan/multi-reviewer-patterns,来源为 acaprino/daodan。它配合 /team-review 与 /senior-review:code-review 等命令使用,涉及 Phase 1 上下文生成、Phase 2 审查、Phase 3 合并、Phase 4b 验证面板、Phase 4c 完整性批评者等阶段。具体安装命令与调用方式在提供的资料中暂未说明。

适用场景

适用于需要多维度并行代码审查的团队或流水线,尤其是涉及用户输入、认证、数据访问、结构性改动、前端 UI 等场景。也适用于希望减少重复发现、统一严重度口径、并系统化暴露审查盲区的流程。

限制

资料未提供该模式的收费、价格、平台支持范围或实际效果数据。成本护栏基于发现数量而非真实 token 消耗,文档要求明确说明以避免虚假精度。回退模式会禁用 logic-integrity-auditor 并移除上下文引用。