
agents24 agent-teams 插件的评审维度检查清单为并行代码审查建立可执行的多维评审标准【免费下载链接】agentsMulti-harness agentic plugin marketplace for Claude Code, Codex, Cursor, OpenCode, GitHub Copilot, and Google Antigravity项目地址: https://gitcode.com/GitHub_Trending/agents24/agents本文以 review-dimensions.md 中的五大评审维度检查清单为主体结合 agents24 仓库中 agent-teams 插件的 team-reviewer 智能体定义 与 /team-review 命令编排讲解多智能体并行代码审查中每个维度“查什么、按什么顺序查、发现如何结构化上报”。读完本文你可以理解该插件如何把 Security、Performance、Architecture、Testing、Accessibility 五个维度的审查拆解为可逐项核销的 Checklist并与发现去重、严重度校准机制配合产出一份可追溯的合并评审报告。检查清单在并行评审流程中的位置agents24 仓库的 agent-teams 插件基于 Claude Code 的实验性 Agent Teams 能力把一次代码审查拆成多个专职评审员并行执行。其分工链条是team-reviewer.md 定义了单一维度的评审员它被分配一个维度security / performance / architecture / testing / accessibility只在该维度内深挖产出带file:line定位、严重度等级和修复建议的结构化发现team-review.md 定义了编排流程解析目标文件、目录、git diff 区间或 PR 号→ 为每个维度生成一个{dimension}-reviewer→ 收集各维度的结构化发现 → 去重、按严重度归并 → 输出合并报告并清理团队资源本文的主题文件 review-dimensions.md 则是这套流程的“执行手册”它把每个维度展开为带- [ ]复选框的详细检查清单供评审员在并行评审时逐项核对。也就是说team-reviewer.md 描述的是“每个维度关注哪些主题”如 Security 维度关注输入校验、认证授权、SQL 注入/XSS/CSRF、密钥暴露、依赖 CVE 等而检查清单文件进一步把这些主题落成可逐项打勾的具体验证点。下面逐维度完整介绍这些清单及其设计意图。安全评审检查清单Security安全维度覆盖输入处理、认证授权、密钥与配置、依赖四个子域是评审用户输入或涉及认证的代码时必查的维度。输入处理所有用户输入都经过验证与净化SQL 查询使用参数化语句禁止字符串拼接HTML 输出正确转义以防止 XSS文件路径经过校验以防止路径穿越强制限制请求体大小认证与授权所有受保护端点都要求认证授权检查验证用户是否有权执行该操作JWT 令牌经过完整校验签名、过期时间、签发者密码哈希使用 bcrypt/argon2而非 MD5/SHA会话管理遵循最佳实践密钥与配置无硬编码的密钥、API Key 或密码密钥从环境变量或密钥管理器加载.gitignore包含敏感文件模式生产环境中禁用调试/开发端点依赖直接依赖中无已知 CVE依赖锁定到具体版本无增大攻击面的不必要依赖与 team-reviewer.md 中 Security 维度的主题描述对照可见清单把“不安全的加密使用”“API 安全限流、输入边界”等主题细化成了可操作的核对点例如“请求大小限制”“依赖版本锁定”。按该插件的严重度校准规则见 multi-reviewer-patterns 的 SKILL.md可被外部用户利用的安全漏洞一律定为 Critical 或 High因此这一维度产出的发现天然排在报告前列。性能评审检查清单Performance性能维度覆盖数据库、内存与资源、计算三个子域适合在修改数据访问层或热路径代码时启用。数据库不存在 N1 查询模式查询使用合适的索引大表上不做SELECT *列表端点实现了分页配置了连接池内存与资源无内存泄漏事件监听器被清理、流被关闭大数据集采用流式处理而非整体载入内存文件句柄与连接被正确关闭昂贵操作使用了缓存计算无不必要的重复计算或冗余操作算法复杂度与数据规模匹配I/O 密集场景使用了异步操作主线程上无阻塞操作对照 team-reviewer.md 中 Performance 维度的主题N1/缺索引/全表扫描、内存分配与潜在泄漏、缓存机会与失效、异步正确性、资源清理、算法复杂度、包体大小与懒加载检查清单将其中后端相关部分展开为具体核对点而“包体大小与懒加载”这类前端性能主题则由该维度的描述兜底体现了“清单为主、维度描述为辅”的分工。性能发现按校准规则在热路径上至少定为 Medium。架构评审检查清单Architecture架构维度覆盖设计原则、结构、模式三个子域适合结构性变更或新增模块时使用。设计原则单一职责每个模块/类只有一个变更原因开闭原则不修改即可扩展依赖倒置依赖抽象而非具体实现模块之间无循环依赖结构关注点分离清晰UI、业务逻辑、数据层全代码库的错误处理策略一致配置外部化而非硬编码API 契约定义清晰且带版本管理模式全代码库模式使用一致无模式混用抽象层级合适不过度设计也不欠设计模块边界与领域边界对齐共享工具真正被共享无重复实现值得注意的是清单把“配置外部化”放在架构维度而安全维度同时检查“密钥从环境变量/密钥管理器加载”——两个维度从不同角度覆盖同一处代码这正是并行评审后需要发现去重的原因。测试评审检查清单Testing测试维度覆盖覆盖率、质量、可维护性三个子域适合新增功能时启用。覆盖率关键路径有测试覆盖边界情况有测试空输入、null、边界值错误路径有测试失败时发生什么集成点有集成测试质量测试是确定性的无 flaky 测试测试是隔离的测试间无共享状态断言足够具体不满足于“没抛异常”测试命名清晰描述测试内容可维护性测试没有复制实现逻辑Mock/Stub 最少化且准确测试数据清晰且相关不看实现也能读懂测试与 team-reviewer.md 中 Testing 维度的八项主题关键路径覆盖缺口、测试隔离与确定性、Mock 恰当性、边界条件、集成测试完整性、命名与文档、断言质量、测试可维护性与脆弱性逐项对应可以推断该清单是由维度描述“落地化”而来每项主题都能找到对应的可勾选项。无障碍评审检查清单Accessibility无障碍维度覆盖结构、交互、内容三个子域适合 UI/前端变更时启用。该维度是五个维度中唯一有明确量化阈值的结构使用语义化 HTML 元素nav、main、article、button标题层级逻辑正确h1 → h2 → h3ARIA role 与属性使用正确地标Landmarks标识页面区域交互全部功能可通过键盘访问焦点顺序逻辑且可见无键盘陷阱触摸目标至少 44x44px内容图片具备有意义的 alt 文本颜色不是传达信息的唯一手段文本对比度充足正文 4.5:1大字 3:1内容在 200% 缩放下仍可读其中 44x44px 触摸目标、4.5:1 / 3:1 对比度阈值与 team-reviewer.md 中“WCAG 2.1 AA 合规”的要求一致使评审员可以直接对照 WCAG 级别给出结论。按严重度校准规则核心功能的无障碍违规至少定为 Medium。从清单到报告发现格式、去重与严重度校准检查清单的价值最终体现在合并报告中。三个机制保证多评审员结果不会互相矛盾或重复结构化发现格式team-reviewer.md 要求每条发现使用固定结构标题含严重度前缀如### [SEVERITY] Finding Title并包含 Locationpath/to/file.ts:42形式、Dimension、Severity、Evidence含代码片段、Impact、Recommended Fix 六个字段。行为准则还要求严格停留在被分配维度内、每条发现必须引用具体 file:line、严重度基于证据而非印象、区分“已确认问题”与“潜在隐患”、对无发现的维度如实报告而非凑数。发现去重规则当多个评审员在同一位置报告问题时multi-reviewer-patterns 的 SKILL.md 定义了合并规则同 file:line、同问题— 合并为一条发现署名所有评审员同 file:line、不同问题— 保留为两条独立发现同问题、不同位置— 分开保留但互相交叉引用严重度冲突— 采用更高等级修复建议冲突— 两条都保留并标注来源评审员。去重流程逐条检查各报告中的 file:line 是否重合重合时判断是否描述同一问题同一问题则合并并保留更详细的描述不同问题则都保留并打上 “co-located” 标签合并后的严重度取最高值。/team-review 命令 的 Phase 4 正是按这套规则执行去重 → 冲突取更高等级 → 按 Critical/High/Medium/Low 分组 → 交叉引用跨维度出现的发现。严重度校准标准严重度影响可能性典型示例Critical数据丢失、安全入侵、完全失效确定或极有可能SQL 注入、认证绕过、数据损坏High显著功能影响、性能退化很可能内存泄漏、缺失校验、流程中断Medium部分影响、存在绕过方案有可能N1 查询、缺失边界用例、错误信息不清晰Low影响极小、外观问题不太可能风格问题、次要优化、命名配套的校准规则把检查清单中常见命中项直接映射到等级可被外部用户利用的漏洞一律 Critical/High热路径性能问题至少 Medium关键路径缺测试至少 Medium核心功能无障碍违规至少 Medium无功能影响的风格问题为 Low。这解释了为何清单把“N1 查询”放在性能维度、“缺失边界用例”放在测试维度——它们在合并报告中的起始等级都已预设。合并报告模板与实操方式评审完成后按 SKILL.md 中的报告模板输出头部包含 Target、Reviewers各维度、Date、Files Reviewed正文按 Critical/High/Medium/Low 分节每条发现带编号如[CR-001]及 Location/Dimension/Description/Impact/Fix 字段结尾是一张按维度 x 严重度交叉统计的 Summary 表加总体建议。实操层面README 给出了完整的启动方式先设置环境变量export CLAUDE_CODE_EXPERIMENTAL_AGENT_TEAMS1在~/.claude/settings.json中配置teammateMode推荐tmux可选iterm2或默认的in-process安装插件后直接运行/team-review src/ --reviewers security,performance,architecture该命令的 argument-hint 为target [--reviewers security,performance,architecture,testing,accessibility] [--base-branch main]其中 target 可以是文件路径、目录、git diff 区间如main...HEAD或 PR 号如#123--reviewers缺省为security,performance,architecture即 team-spawn 的 review 预设默认组合。针对特定变更类型SKILL.md 还给出了推荐组合API 端点变更选 Security Performance Architecture前端组件选 Architecture Testing Accessibility数据库迁移选 Performance Architecture认证变更选 Security Testing完整功能评审则启用除 Accessibility 外的全部维度。小结review-dimensions.md 把 agent-teams 插件的多维评审从“维度主题描述”推进到了“逐项可核销的检查点”安全维度 18 项核对点覆盖输入、认证、密钥、依赖四层纵深性能维度 14 项聚焦数据库、资源与计算架构维度 12 项约束 SOLID、结构与模式一致性测试维度 12 项兼顾覆盖、质量与可维护无障碍维度 12 项以 WCAG 量化阈值收口。配合 team-reviewer 智能体的结构化发现格式与 SKILL.md 的去重、校准规则这套清单使并行评审的产出收敛为一份按严重度排序、可追溯到人评审员和位置file:line的合并报告这正是该插件“多维度并行审查”能力的执行层保障。【免费下载链接】agentsMulti-harness agentic plugin marketplace for Claude Code, Codex, Cursor, OpenCode, GitHub Copilot, and Google Antigravity项目地址: https://gitcode.com/GitHub_Trending/agents24/agents创作声明:本文部分内容由AI辅助生成(AIGC),仅供参考