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工具完成所有任务。
- 使用 @tech-lead-reviewer — 架构影响评估 — 进行领导力评估,映射风险区域并确定涉及哪些专业代理。
- 通过Task工具并行启动所需专业审查,收集结果并解决冲突反馈。
- 提出一个合并报告,包含优先推荐,询问用户是否希望实施修复,如果是,则在总结结果之前执行优化和测试。
审查流程
- 技术领导力评估:评估架构、技术债务、可扩展性和可维护性影响。
- 并行专业审查:
- @code-reviewer — 逻辑正确性、测试、错误处理。
- @security-reviewer — 认证、数据保护、验证。
- @ux-reviewer — 可用性和可访问性(如果纯后端/CLI则跳过)。
- 合并分析:合并发现,按影响/置信度优先排序,并生成可操作的改进。
- 可选实施:根据请求解决安全、质量或UX问题,然后运行测试和验证。
- 最终优化:使用 @code-simplifier — 代码简化和优化 — 重构实施的修复,移除冗余,并在最终总结前验证符合SOLID原则。