分层式代码审查Skill hierarchical

分层式代码审查技能是一个全面的代码审查工具,通过使用专用子代理进行多阶段评估,专门处理具有复杂架构影响、安全顾虑的Pull Request审查。它整合技术领导评估、代码逻辑、安全性和用户体验审查,提供优先级排序的建议,并支持优化实施。关键词:代码审查、架构设计、安全评估、多代理系统、PR审查、软件开发质量保证、分层式评估、代码优化。

架构设计 0 次安装 0 次浏览 更新于 3/16/2026

name: hierarchical user-invocable: true description: 使用专用子代理进行全面多阶段代码审查。在审查具有复杂架构影响、安全问题或需要彻底多视角分析的PR时使用。 argument-hint: [文件或目录] allowed-tools: Task

分层式代码审查

上下文

  • 当前分支:!git branch --show-current
  • Git状态:!git status --porcelain
  • 基础分支:!(git show-branch | grep '*' | grep -v "$(git rev-parse --abbrev-ref HEAD)" | head -1 | sed 's/.*\[\([^]]*\)\].*/\1/' | sed 's/\^.*//' 2>/dev/null) || echo "develop"
  • 自基础以来的变更:!BASE=$(git merge-base HEAD develop 2>/dev/null || git merge-base HEAD main 2>/dev/null) && git log --oneline $BASE..HEAD
  • 自基础以来更改的文件:!BASE=$(git merge-base HEAD develop 2>/dev/null || git merge-base HEAD main 2>/dev/null) && git diff --name-only $BASE..HEAD
  • 可用测试命令:!([ -f package.json ] && echo "npm/pnpm/yarn test") || ([ -f Cargo.toml ] && echo "cargo test") || ([ -f pyproject.toml ] && echo "pytest/uv run pytest") || ([ -f go.mod ] && echo "go test") || echo "no standard test framework detected"

要求

  • 使用 @tech-lead-reviewer — 架构影响评估 — 在启动专业审查之前评估架构风险范围。
  • 并行运行审查:
    • @code-reviewer — 逻辑正确性、测试、错误处理。
    • @security-reviewer — 认证、数据保护、验证。
    • @ux-reviewer — 可用性和可访问性(如果纯后端/CLI则跳过)。
  • 按优先级(关键 → 高 → 中 → 低)和置信度(高 → 中 → 低)合并发现。
  • 提供可选实施支持,并确保提交遵循Git约定(详见 ${CLAUDE_PLUGIN_ROOT}/skills/references/git-commit-conventions.md)。

您的任务

重要:您必须使用Task工具完成所有任务。

  1. 使用 @tech-lead-reviewer — 架构影响评估 — 进行领导力评估,映射风险区域并确定涉及哪些专业代理。
  2. 通过Task工具并行启动所需专业审查,收集结果并解决冲突反馈。
  3. 提出一个合并报告,包含优先推荐,询问用户是否希望实施修复,如果是,则在总结结果之前执行优化和测试。

审查流程

  • 技术领导力评估:评估架构、技术债务、可扩展性和可维护性影响。
  • 并行专业审查
    • @code-reviewer — 逻辑正确性、测试、错误处理。
    • @security-reviewer — 认证、数据保护、验证。
    • @ux-reviewer — 可用性和可访问性(如果纯后端/CLI则跳过)。
  • 合并分析:合并发现,按影响/置信度优先排序,并生成可操作的改进。
  • 可选实施:根据请求解决安全、质量或UX问题,然后运行测试和验证。
  • 最终优化:使用 @code-simplifier — 代码简化和优化 — 重构实施的修复,移除冗余,并在最终总结前验证符合SOLID原则。