全面代码审查编排

要解决什么问题

全面代码审查往往被拆成「看风格」「扫漏洞」「跑覆盖率」等零散动作，各工具各说各话，缺少统一优先级，架构层面的问题也很难和安全、性能结论对齐。本技能把审查当成可编排的多阶段工作流：先并行做质量与架构，再带着上下文做安全与性能，最后覆盖测试、文档、框架约定与 CI/CD，并汇总成 P0–P3 分级报告。

工作方式

核心思路是用 Task 工具 调度不同 subagent_type 的专门代理，阶段之间通过上下文变量传递发现（例如架构结论进入安全审计、性能分析）。

• 阶段 1：code-reviewer 做复杂度、异味、SOLID 与静态分析取向的质量审查；architect-review 并行评估边界、耦合、DDD 与云原生结构。
• 阶段 2：security-auditor 做 OWASP、依赖与密钥类检查；application-performance::performance-engineer 做热点、查询、缓存与 N+1 等可扩展性分析。
• 阶段 3：unit-testing::test-automator 评估测试金字塔与覆盖率缺口；code-documentation::docs-architect 核对 README、OpenAPI、ADR 与实现是否一致。
• 阶段 4：框架/语言最佳实践与 cicd-automation::deployment-engineer 对流水线、部署与可观测性的审查。

可通过 --security-focus、--performance-critical、--tdd-review、--strict-mode、--framework [name] 等选项收窄或加重某一维度；详细示例见 resources/implementation-playbook.md。

适用边界

任务与全面代码审查无关时不要启用。技能描述的是代理编排与检查清单，不会替代你方实际的 SonarQube、Snyk、Trivy 等工具接入；--strict-mode 会在发现严重问题时令审查失败，需团队事先约定门槛。成功标准包括：严重安全问题已记录、性能与测试缺口有修复路径、文档与实现一致，且团队拿到可执行的优先级行动计划。