Working on comprehensive review full review tasks or workflows
Needing guidance, best practices, or checklists for comprehensive review full review
Do not use this skill when
The task is unrelated to comprehensive review full review
You need a different domain or tool outside this scope
Instructions
Clarify goals, constraints, and required inputs.
Apply relevant best practices and validate outcomes.
Provide actionable steps and verification.
If detailed examples are required, open resources/implementation-playbook.md.
Orchestrate comprehensive multi-dimensional code review using specialized review agents
[Extended thinking: This workflow performs an exhaustive code review by orchestrating multiple specialized agents in sequential phases. Each phase builds upon previous findings to create a comprehensive review that covers code quality, security, performance, testing, documentation, and best practices. The workflow integrates modern AI-assisted review tools, static analysis, security scanning, and automated quality metrics. Results are consolidated into actionable feedback with clear prioritization and remediation guidance. The phased approach ensures thorough coverage while maintaining efficiency through parallel agent execution where appropriate.]
Review Configuration Options
--security-focus: Prioritize security vulnerabilities and OWASP compliance
--performance-critical: Emphasize performance bottlenecks and scalability issues
--tdd-review: Include TDD compliance and test-first verification
Context: Uses architecture insights to identify systemic performance issues
Phase 3: Testing & Documentation Review
Use Task tool for test and documentation quality assessment:
3A. Test Coverage & Quality Analysis
Use Task tool with subagent_type="unit-testing::test-automator"
Prompt: "Evaluate testing strategy and implementation for: $ARGUMENTS. Analyze unit test coverage, integration test completeness, end-to-end test scenarios, test pyramid adherence, and test maintainability. Review test quality metrics including assertion density, test isolation, mock usage, and flakiness. Consider security and performance test requirements from Phase 2: {phase2_security_context}, {phase2_performance_context}. Verify TDD practices if --tdd-review flag is set."
Expected output: Coverage report, test quality metrics, testing gap analysis
Context: Incorporates security and performance testing requirements from Phase 2
3B. Documentation & API Specification Review
Use Task tool with subagent_type="code-documentation::docs-architect"
Prompt: "Review documentation completeness and quality for: $ARGUMENTS. Assess inline code documentation, API documentation (OpenAPI/Swagger), architecture decision records (ADRs), README completeness, deployment guides, and runbooks. Verify documentation reflects actual implementation based on all previous phase findings: {phase1_context}, {phase2_context}. Check for outdated documentation, missing examples, and unclear explanations."
Context: Cross-references all previous findings to ensure documentation accuracy
Phase 4: Best Practices & Standards Compliance
Use Task tool to verify framework-specific and industry best practices:
4A. Framework & Language Best Practices
Use Task tool with subagent_type="framework-migration::legacy-modernizer"
Prompt: "Verify adherence to framework and language best practices for: $ARGUMENTS. Check modern JavaScript/TypeScript patterns, React hooks best practices, Python PEP compliance, Java enterprise patterns, Go idiomatic code, or framework-specific conventions (based on --framework flag). Review package management, build configuration, environment handling, and deployment practices. Include all quality issues from previous phases: {all_previous_contexts}."
Expected output: Best practices compliance report, modernization recommendations
Context: Synthesizes all previous findings for framework-specific guidance
4B. CI/CD & DevOps Practices Review
Use Task tool with subagent_type="cicd-automation::deployment-engineer"
Prompt: "Review CI/CD pipeline and DevOps practices for: $ARGUMENTS. Evaluate build automation, test automation integration, deployment strategies (blue-green, canary), infrastructure as code, monitoring/observability setup, and incident response procedures. Assess pipeline security, artifact management, and rollback capabilities. Consider all issues identified in previous phases that impact deployment: {all_critical_issues}."
