comprehensive-review-full-review

使用此技能的场景

• 处理全面审查任务或工作流时
• 需要全面审查的指导、最佳实践或检查清单时

不要使用此技能的场景

• 任务与全面审查无关时
• 需要使用此范围之外的其他领域或工具时

说明

• 明确目标、约束和所需输入。
• 应用相关最佳实践并验证结果。
• 提供可执行的步骤和验证方法。

使用专门的审查 Agent 编排全面、多维度的代码审查

[扩展思考：此工作流通过按顺序分阶段编排多个专门的 Agent，执行彻底的代码审查。每个阶段都会基于前序阶段的发现，全面覆盖代码质量、安全性、性能、测试、文档和最佳实践。此工作流整合了现代 AI 辅助审查工具、静态分析、安全扫描和自动化质量指标。最终将结果汇总为可执行的反馈，并明确优先级和修复建议。分阶段的方法可确保审查全面，同时通过在适当情况下并行执行 Agent 来提高效率。]

审查配置选项

• --security-focus：优先关注安全漏洞和 OWASP 合规性
• --performance-critical：重点关注性能瓶颈和可扩展性问题
• --tdd-review：检查 TDD 合规性并验证测试优先原则
• --ai-assisted：启用 AI 驱动的审查工具（Copilot、Codium、Bito）
• --strict-mode：发现任何严重问题时判定审查失败
• --metrics-report：生成详细的质量指标仪表板
• --framework [name]：应用特定框架的最佳实践（React、Spring、Django 等）

阶段 1：代码质量与架构审查

使用 Task 工具并行编排质量和架构 Agent：

1A. 代码质量分析

• 使用 Task 工具，并设置 subagent_type="code-reviewer"
• 提示词："对以下内容执行全面的代码质量审查：$ARGUMENTS。分析代码复杂度、可维护性指数、技术债务、代码重复、命名规范，以及对 Clean Code 原则的遵循情况。将 SonarQube、CodeQL 和 Semgrep 集成到静态分析中。检查代码异味、反模式和违反 SOLID 原则的情况。生成圈复杂度指标并找出重构机会。"
• 预期输出：质量指标、代码异味清单、重构建议
• 上下文：初始代码库分析，不依赖其他阶段

1B. 架构与设计审查

• 使用 Task 工具，并设置 subagent_type="architect-review"
• 提示词："审查以下内容中的架构设计模式和结构完整性：$ARGUMENTS。评估微服务边界、API 设计、数据库架构、依赖管理，以及对领域驱动设计原则的遵循情况。检查循环依赖、不当耦合、缺失的抽象和架构偏移。验证其是否符合企业架构标准和云原生模式。"
• 预期输出：架构评估、设计模式分析、结构建议
• 上下文：与代码质量分析并行运行

阶段 2：安全与性能审查

使用安全和性能 Agent，并结合阶段 1 的发现：

2A. 安全漏洞评估

• 使用 Task 工具，subagent_type="security-auditor"
• 提示词："对以下内容执行全面的安全审计：$ARGUMENTS。执行 OWASP Top 10 分析，使用 Snyk/Trivy 进行依赖漏洞扫描，使用 GitLeaks 进行密钥检测，进行输入验证审查、认证/授权评估以及加密实现审查。包含来自第 1 阶段架构审查的发现：{phase1_architecture_context}。检查 SQL 注入、XSS、CSRF、不安全反序列化以及配置安全问题。"
• 预期输出：漏洞报告、CVE 列表、安全风险矩阵、修复步骤
• 上下文：纳入在第 1B 阶段识别出的架构漏洞

2B. 性能与可扩展性分析

• 使用 Task 工具，subagent_type="application-performance::performance-engineer"
• 提示词："对以下内容进行性能分析和可扩展性评估：$ARGUMENTS。对代码进行 CPU/内存热点分析，分析数据库查询性能，审查缓存策略，识别 N+1 问题，评估连接池，并评估异步处理模式。考虑来自第 1 阶段的架构发现：{phase1_architecture_context}。检查内存泄漏、资源争用以及负载下的瓶颈。"
• 预期输出：性能指标、瓶颈分析、优化建议
• 上下文：利用架构洞察识别系统性性能问题

第 3 阶段：测试与文档审查

使用 Task 工具进行测试和文档质量评估：

3A. 测试覆盖率与质量分析

• 使用 Task 工具，subagent_type="unit-testing::test-automator"
• 提示词："评估以下内容的测试策略和实现：$ARGUMENTS。分析单元测试覆盖率、集成测试完整性、端到端测试场景、测试金字塔遵循情况以及测试可维护性。审查测试质量指标，包括断言密度、测试隔离、mock 使用以及不稳定性。考虑来自第 2 阶段的安全和性能测试要求：{phase2_security_context}、{phase2_performance_context}。如果设置了 --tdd-review 标志，则验证 TDD 实践。"
• 预期输出：覆盖率报告、测试质量指标、测试缺口分析
• 上下文：纳入来自第 2 阶段的安全和性能测试要求

3B. 文档与 API 规范审查

• 使用 Task 工具，subagent_type="code-documentation::docs-architect"
• 提示词："审查以下内容的文档完整性和质量：$ARGUMENTS。评估内联代码文档、API 文档（OpenAPI/Swagger）、架构决策记录（ADR）、README 完整性、部署指南以及运行手册。基于所有先前阶段的发现验证文档是否反映实际实现：{phase1_context}、{phase2_context}。检查过时的文档、缺失的示例以及不清晰的解释。"
• 预期输出：文档覆盖率报告、不一致列表、改进建议
• 上下文：交叉引用所有先前的发现，以确保文档准确性

第 4 阶段：最佳实践与标准合规性

使用 Task 工具验证框架特定和行业最佳实践：

4A. 框架与语言最佳实践

• 使用 Task 工具，subagent_type="framework-migration::legacy-modernizer"
• 提示词："验证以下内容是否遵循框架和语言最佳实践：$ARGUMENTS。检查现代 JavaScript/TypeScript 模式、React hooks 最佳实践、Python PEP 合规性、Java 企业模式、Go 惯用代码或框架特定约定（基于 --framework 标志）。审查包管理、构建配置、环境处理以及部署实践。包含来自所有先前阶段的所有质量问题：{all_previous_contexts}。"
• 预期输出：最佳实践合规报告、现代化建议
• 上下文：综合所有先前的发现，提供框架特定的指导

4B. CI/CD 与 DevOps 实践审查

• 使用 Task 工具，设置 subagent_type="cicd-automation::deployment-engineer"
• 提示词："审查以下内容的 CI/CD 管道和 DevOps 实践：$ARGUMENTS。评估构建自动化、测试自动化集成、部署策略（蓝绿部署、金丝雀发布）、基础设施即代码、监控/可观测性配置以及事件响应流程。评估管道安全、制品管理和回滚能力。考虑前几个阶段中所有会影响部署的问题：{all_critical_issues}。"
• 预期输出：管道评估、DevOps 成熟度评估、自动化建议
• 背景：重点是落实针对所有已识别问题的修复措施

综合报告生成

将各阶段的输出汇总为一份全面的审查报告：

严重问题（P0 - 必须立即修复）

• CVSS > 7.0 的安全漏洞
• 数据丢失或损坏风险
• 身份验证/授权绕过
• 生产稳定性威胁
• 合规性违规（GDPR、PCI DSS、SOC2）

高优先级（P1 - 下次发布前修复）

• 影响用户体验的性能瓶颈
• 缺少关键测试覆盖
• 导致技术债务的架构反模式
• 存在已知漏洞的过时依赖项
• 影响可维护性的代码质量问题

中优先级（P2 - 计划在下个 Sprint 处理）

• 非关键性能优化
• 文档缺失和不一致
• 代码重构机会
• 测试质量改进
• DevOps 自动化增强

低优先级（P3 - 纳入待办事项）

• 违反风格指南
• 轻微代码异味问题
• 非必需的文档更新
• 外观改进

成功标准

满足以下条件时，即视为审查成功：

• 已识别并记录所有严重安全漏洞
• 已分析性能瓶颈，并给出修复方案
• 已梳理测试覆盖缺口，并给出优先级建议
• 已评估架构风险，并制定缓解策略
• 文档反映实际的实现状态
• 已确认符合框架最佳实践
• CI/CD 管道支持安全部署经过审查的代码
• 针对所有发现提供清晰、可执行的反馈
• 指标仪表板显示改进趋势
• 团队制定了清晰且按优先级排序的修复行动计划

目标：$ARGUMENTS