comprehensive-review-full-review

Comprehensive Review - 全面代码审查工作流

技能概述

Comprehensive Review 是一个多阶段代码审查工作流，协调多个专业代理对代码进行全面、多维度的质量评估，覆盖架构、安全、性能、测试和文档等多个方面。

适用场景

上线前全面代码检查

在发布到生产环境前，执行完整的代码质量、安全性和性能检查，确保代码符合企业标准和最佳实践。

代码重构前的基线评估

在进行大规模重构或现代化改造前，全面评估现有代码库的技术债务、架构风险和改进机会。

企业级项目质量审计

为企业级项目提供标准化的代码质量评估，生成包含优先级和可执行建议的综合审查报告。

核心功能

多阶段审查工作流

通过四个独立阶段（代码质量、安全性能、测试文档、最佳实践）顺序执行审查，每个阶段的结果都为下一阶段提供上下文，确保审查的系统性和全面性。

可配置的审查模式

支持多种审查配置选项，包括安全重点模式（--security-focus）、性能关键模式（--performance-critical）、TDD 审查模式（--tdd-review）、严格模式（--strict-mode）和指标报告（--metrics-report）等。

综合报告生成

将所有阶段的审查结果整合成一份按优先级（P0-P3）分类的可执行报告，包括安全漏洞、性能瓶颈、测试覆盖率和架构风险的详细分析和修复建议。

常见问题

全面代码审查工作流包含哪些阶段？

工作流分为四个阶段：第一阶段进行代码质量和架构审查；第二阶段执行安全和性能分析；第三阶段评估测试覆盖率和文档完整性；第四阶段验证框架最佳实践和 CI/CD 流程。每个阶段的结果都会传递给后续阶段，确保审查的连贯性。

如何使用不同的审查配置模式？

通过命令行参数指定模式，例如 --security-focus 优先检查安全漏洞和 OWASP 合规性，--performance-critical 侧重性能瓶颈和可扩展性问题，--framework [name] 应用特定框架（如 React、Spring、Django）的最佳实践。

审查报告中的优先级是如何分类的？

报告将问题分为四个优先级：P0（必须立即修复）包括严重安全漏洞和数据丢失风险；P1（下个版本前修复）包括影响用户体验的性能问题和关键测试缺失；P2（计划下个迭代）包括非关键优化和文档改进；P3（待办跟踪）包括风格问题和次要优化。