empathy-framework 5.2.1__py3-none-any.whl → 5.4.0__py3-none-any.whl

empathy_os/workflows/code_review.py

@@ -1,1048 +0,0 @@
-"""Code Review Workflow
-
-A tiered code analysis pipeline:
-1. Haiku: Classify change type (cheap, fast)
-2. Sonnet: Security scan + bug pattern matching
-3. Opus: Architectural review (conditional on complexity)
-
-Copyright 2025 Smart-AI-Memory
-Licensed under Fair Source License 0.9
-"""
-
-from typing import Any
-
-from .base import BaseWorkflow, ModelTier
-from .step_config import WorkflowStepConfig
-
-# Define step configurations for executor-based execution
-CODE_REVIEW_STEPS = {
-    "architect_review": WorkflowStepConfig(
-        name="architect_review",
-        task_type="architectural_decision",  # Premium tier task
-        tier_hint="premium",
-        description="Comprehensive architectural code review",
-        max_tokens=3000,
-    ),
-}
-
-
-class CodeReviewWorkflow(BaseWorkflow):
-    """Multi-tier code review workflow.
-
-    Uses cheap models for classification, capable models for security
-    and bug scanning, and premium models only for complex architectural
-    reviews (10+ files or core module changes).
-
-    Usage:
-        workflow = CodeReviewWorkflow()
-        result = await workflow.execute(
-            diff="...",
-            files_changed=["src/main.py", "tests/test_main.py"],
-            is_core_module=False
-        )
-    """
-
-    name = "code-review"
-    description = "Tiered code analysis with conditional premium review"
-    stages = ["classify", "scan", "architect_review"]
-    tier_map = {
-        "classify": ModelTier.CHEAP,
-        "scan": ModelTier.CAPABLE,
-        "architect_review": ModelTier.PREMIUM,
-    }
-
-    def __init__(
-        self,
-        file_threshold: int = 10,
-        core_modules: list[str] | None = None,
-        use_crew: bool = True,
-        crew_config: dict | None = None,
-        enable_auth_strategy: bool = True,
-        **kwargs: Any,
-    ):
-        """Initialize workflow.
-
-        Args:
-            file_threshold: Number of files above which premium review is used.
-            core_modules: List of module paths considered "core" (trigger premium).
-            use_crew: Enable CodeReviewCrew for comprehensive 5-agent analysis (default: True).
-            crew_config: Configuration dict for CodeReviewCrew.
-            enable_auth_strategy: If True, use intelligent subscription vs API routing
-                based on module size (default True).
-
-        """
-        super().__init__(**kwargs)
-        self.file_threshold = file_threshold
-        self.core_modules = core_modules or [
-            "src/core/",
-            "src/security/",
-            "src/auth/",
-            "empathy_os/core.py",
-            "empathy_os/security/",
-        ]
-        self.use_crew = use_crew
-        self.crew_config = crew_config or {}
-        self.enable_auth_strategy = enable_auth_strategy
-        self._needs_architect_review: bool = False
-        self._change_type: str = "unknown"
-        self._crew: Any = None
-        self._crew_available = False
-        self._auth_mode_used: str | None = None
-
-        # Dynamically configure stages based on crew setting
-        if use_crew:
-            self.stages = ["classify", "crew_review", "scan", "architect_review"]
-            self.tier_map = {
-                "classify": ModelTier.CHEAP,
-                "crew_review": ModelTier.CAPABLE,  # Changed from PREMIUM to CAPABLE
-                "scan": ModelTier.CAPABLE,
-                "architect_review": ModelTier.PREMIUM,
-            }
-
-    async def _initialize_crew(self) -> None:
-        """Initialize the CodeReviewCrew."""
-        if self._crew is not None:
-            return
-
-        try:
-            import logging
-
-            from empathy_llm_toolkit.agent_factory.crews.code_review import CodeReviewCrew
-
-            self._crew = CodeReviewCrew()
-            self._crew_available = True
-            logging.getLogger(__name__).info("CodeReviewCrew initialized successfully")
-        except ImportError as e:
-            import logging
-
-            logging.getLogger(__name__).warning(f"CodeReviewCrew not available: {e}")
-            self._crew_available = False
-
-    def should_skip_stage(self, stage_name: str, input_data: Any) -> tuple[bool, str | None]:
-        """Skip stages when appropriate."""
-        # Skip all stages after classify if there was an input error
-        if isinstance(input_data, dict) and input_data.get("error"):
-            if stage_name != "classify":
-                return True, "Skipped due to input validation error"
-
-        # Skip crew review if crew is not available
-        if stage_name == "crew_review" and not self._crew_available:
-            return True, "CodeReviewCrew not available"
-
-        # Skip architectural review if change is simple
-        if stage_name == "architect_review" and not self._needs_architect_review:
-            return True, "Simple change - architectural review not needed"
-        return False, None
-
-    def _gather_project_context(self) -> str:
-        """Gather project context for project-level reviews.
-
-        Reads project metadata and key files to provide context to the LLM.
-        Returns formatted project context string, or empty string if no context found.
-        """
-        import os
-        from pathlib import Path
-
-        context_parts = []
-        cwd = Path.cwd()
-
-        # Get project name from directory or config files
-        project_name = cwd.name
-        context_parts.append(f"# Project: {project_name}")
-        context_parts.append(f"# Path: {cwd}")
-        context_parts.append("")
-
-        # Check for pyproject.toml
-        pyproject = cwd / "pyproject.toml"
-        if pyproject.exists():
-            try:
-                content = pyproject.read_text()[:2000]
-                context_parts.append("## pyproject.toml")
-                context_parts.append("```toml")
-                context_parts.append(content)
-                context_parts.append("```")
-                context_parts.append("")
-            except OSError:
-                pass
-
-        # Check for package.json
-        package_json = cwd / "package.json"
-        if package_json.exists():
-            try:
-                content = package_json.read_text()[:2000]
-                context_parts.append("## package.json")
-                context_parts.append("```json")
-                context_parts.append(content)
-                context_parts.append("```")
-                context_parts.append("")
-            except OSError:
-                pass
-
-        # Check for README
-        for readme_name in ["README.md", "README.rst", "README.txt", "README"]:
-            readme = cwd / readme_name
-            if readme.exists():
-                try:
-                    content = readme.read_text()[:3000]
-                    context_parts.append(f"## {readme_name}")
-                    context_parts.append(content)
-                    context_parts.append("")
-                    break
-                except OSError:
-                    pass
-
-        # Get directory structure (top 2 levels)
-        context_parts.append("## Project Structure")
-        context_parts.append("```")
-        try:
-            for root, dirs, files in os.walk(cwd):
-                # Skip hidden and common ignored directories
-                dirs[:] = [
-                    d
-                    for d in dirs
-                    if not d.startswith(".")
-                    and d
-                    not in (
-                        "node_modules",
-                        "__pycache__",
-                        "venv",
-                        ".venv",
-                        "dist",
-                        "build",
-                        ".git",
-                        ".tox",
-                        ".pytest_cache",
-                        ".mypy_cache",
-                        "htmlcov",
-                    )
-                ]
-                level = root.replace(str(cwd), "").count(os.sep)
-                if level < 2:
-                    indent = "  " * level
-                    folder_name = os.path.basename(root) or project_name
-                    context_parts.append(f"{indent}{folder_name}/")
-                    # Show key files at this level
-                    key_files = [
-                        f
-                        for f in files
-                        if f.endswith(
-                            (".py", ".ts", ".js", ".json", ".yaml", ".yml", ".toml", ".md"),
-                        )
-                        and not f.startswith(".")
-                    ][:10]
-                    for f in key_files:
-                        context_parts.append(f"{indent}  {f}")
-                if level >= 2:
-                    break
-        except OSError:
-            context_parts.append("(Unable to read directory structure)")
-        context_parts.append("```")
-
-        # Return empty if we only have the header
-        if len(context_parts) <= 3:
-            return ""
-
-        return "\n".join(context_parts)
-
-    async def run_stage(
-        self,
-        stage_name: str,
-        tier: ModelTier,
-        input_data: Any,
-    ) -> tuple[Any, int, int]:
-        """Execute a code review stage."""
-        if stage_name == "classify":
-            return await self._classify(input_data, tier)
-        if stage_name == "crew_review":
-            return await self._crew_review(input_data, tier)
-        if stage_name == "scan":
-            return await self._scan(input_data, tier)
-        if stage_name == "architect_review":
-            return await self._architect_review(input_data, tier)
-        raise ValueError(f"Unknown stage: {stage_name}")
-
-    async def _classify(self, input_data: dict, tier: ModelTier) -> tuple[dict, int, int]:
-        """Classify the type of change."""
-        diff = input_data.get("diff", "")
-        target = input_data.get("target", "")
-        files_changed = input_data.get("files_changed", [])
-
-        # If target provided instead of diff, use it as the code to review
-        code_to_review = diff or target
-
-        # Handle project-level review when target is "." or empty
-        if not code_to_review or code_to_review.strip() in (".", "", "./"):
-            # Gather project context for project-level review
-            project_context = self._gather_project_context()
-            if not project_context:
-                # Return early with helpful error message if no context found
-                return (
-                    {
-                        "classification": "ERROR: No code provided for review",
-                        "error": True,
-                        "error_message": (
-                            "No code was provided for review. Please ensure you:\n"
-                            "1. Have a file open in the editor, OR\n"
-                            "2. Select a specific file to review, OR\n"
-                            '3. Provide code content directly via --input \'{"diff": "..."}\'\n\n'
-                            "Tip: Use 'Select File...' option in the workflow picker."
-                        ),
-                        "change_type": "none",
-                        "files_changed": [],
-                        "file_count": 0,
-                        "needs_architect_review": False,
-                        "is_core_module": False,
-                        "code_to_review": "",
-                    },
-                    0,
-                    0,
-                )
-            code_to_review = project_context
-            # Mark as project-level review
-            input_data["is_project_review"] = True
-
-        # === AUTH STRATEGY INTEGRATION ===
-        if self.enable_auth_strategy:
-            try:
-                import logging
-                from pathlib import Path
-
-                from empathy_os.models import (
-                    count_lines_of_code,
-                    get_auth_strategy,
-                    get_module_size_category,
-                )
-
-                logger = logging.getLogger(__name__)
-
-                # Calculate module size (for file) or total LOC (for directory)
-                target_path = target or diff
-                total_lines = 0
-                if target_path:
-                    target_obj = Path(target_path)
-                    if target_obj.exists():
-                        if target_obj.is_file():
-                            total_lines = count_lines_of_code(target_obj)
-                        elif target_obj.is_dir():
-                            for py_file in target_obj.rglob("*.py"):
-                                try:
-                                    total_lines += count_lines_of_code(py_file)
-                                except Exception:
-                                    pass
-
-                if total_lines > 0:
-                    strategy = get_auth_strategy()
-                    recommended_mode = strategy.get_recommended_mode(total_lines)
-                    self._auth_mode_used = recommended_mode.value
-
-                    size_category = get_module_size_category(total_lines)
-                    logger.info(
-                        f"Code review target: {target_path} ({total_lines:,} LOC, {size_category})"
-                    )
-                    logger.info(f"Recommended auth mode: {recommended_mode.value}")
-
-                    cost_estimate = strategy.estimate_cost(total_lines, recommended_mode)
-                    if recommended_mode.value == "subscription":
-                        logger.info(f"Cost: {cost_estimate['quota_cost']}")
-                    else:
-                        logger.info(f"Cost: ~${cost_estimate['monetary_cost']:.4f}")
-
-            except Exception as e:
-                logger = logging.getLogger(__name__)
-                logger.warning(f"Auth strategy detection failed: {e}")
-
-        system = """You are a code review classifier. Analyze the code and classify:
-1. Change type: bug_fix, feature, refactor, docs, test, config, or security
-2. Complexity: low, medium, high
-3. Risk level: low, medium, high
-
-Respond with a brief classification summary."""
-
-        user_message = f"""Classify this code change:
-
-Files: {", ".join(files_changed) if files_changed else "Not specified"}
-
-Code:
-{code_to_review[:4000]}"""
-
-        response, input_tokens, output_tokens = await self._call_llm(
-            tier,
-            system,
-            user_message,
-            max_tokens=500,
-        )
-
-        # Parse response to determine if architect review needed
-        is_high_complexity = "high" in response.lower() and (
-            "complexity" in response.lower() or "risk" in response.lower()
-        )
-        is_core = (
-            any(any(core in f for core in self.core_modules) for f in files_changed)
-            if files_changed
-            else False
-        )
-
-        self._needs_architect_review = (
-            len(files_changed) >= self.file_threshold
-            or is_core
-            or is_high_complexity
-            or input_data.get("is_core_module", False)
-        )
-
-        return (
-            {
-                "classification": response,
-                "change_type": "feature",  # Will be refined by LLM
-                "files_changed": files_changed,
-                "file_count": len(files_changed),
-                "needs_architect_review": self._needs_architect_review,
-                "is_core_module": is_core,
-                "code_to_review": code_to_review,
-            },
-            input_tokens,
-            output_tokens,
-        )
-
-    async def _crew_review(self, input_data: dict, tier: ModelTier) -> tuple[dict, int, int]:
-        """Run CodeReviewCrew for comprehensive 5-agent analysis.
-
-        This stage uses the CodeReviewCrew (Review Lead, Security Analyst,
-        Architecture Reviewer, Quality Analyst, Performance Reviewer) for
-        deep code analysis with memory graph integration.
-
-        Falls back gracefully if CodeReviewCrew is not available.
-        """
-        await self._initialize_crew()
-
-        try:
-            from .code_review_adapters import (
-                _check_crew_available,
-                _get_crew_review,
-                crew_report_to_workflow_format,
-            )
-        except ImportError:
-            # Crew adapters removed - return fallback
-            return (
-                {
-                    "crew_review": {
-                        "available": False,
-                        "fallback": True,
-                        "reason": "Crew adapters not installed",
-                    },
-                    **input_data,
-                },
-                0,
-                0,
-            )
-
-        # Get code to review
-        diff = input_data.get("diff", "") or input_data.get("code_to_review", "")
-        files_changed = input_data.get("files_changed", [])
-
-        # Check if crew is available
-        if not self._crew_available or not _check_crew_available():
-            return (
-                {
-                    "crew_review": {
-                        "available": False,
-                        "fallback": True,
-                        "reason": "CodeReviewCrew not installed or failed to initialize",
-                    },
-                    **input_data,
-                },
-                0,
-                0,
-            )
-
-        # Run CodeReviewCrew
-        report = await _get_crew_review(
-            diff=diff,
-            files_changed=files_changed,
-            config=self.crew_config,
-        )
-
-        if report is None:
-            return (
-                {
-                    "crew_review": {
-                        "available": True,
-                        "fallback": True,
-                        "reason": "CodeReviewCrew review failed or timed out",
-                    },
-                    **input_data,
-                },
-                0,
-                0,
-            )
-
-        # Convert crew report to workflow format
-        crew_results = crew_report_to_workflow_format(report)
-
-        # Update needs_architect_review based on crew findings
-        has_blocking = crew_results.get("has_blocking_issues", False)
-        critical_count = len(crew_results.get("assessment", {}).get("critical_findings", []))
-        high_count = len(crew_results.get("assessment", {}).get("high_findings", []))
-
-        if has_blocking or critical_count > 0 or high_count > 2:
-            self._needs_architect_review = True
-
-        crew_review_result = {
-            "available": True,
-            "fallback": False,
-            "findings": crew_results.get("findings", []),
-            "finding_count": crew_results.get("finding_count", 0),
-            "verdict": crew_results.get("verdict", "approve"),
-            "quality_score": crew_results.get("quality_score", 100),
-            "has_blocking_issues": has_blocking,
-            "critical_count": critical_count,
-            "high_count": high_count,
-            "summary": crew_results.get("summary", ""),
-            "agents_used": crew_results.get("agents_used", []),
-            "memory_graph_hits": crew_results.get("memory_graph_hits", 0),
-            "review_duration_seconds": crew_results.get("review_duration_seconds", 0),
-        }
-
-        # Estimate tokens (crew uses internal LLM calls)
-        input_tokens = len(diff) // 4
-        output_tokens = len(str(crew_review_result)) // 4
-
-        return (
-            {
-                "crew_review": crew_review_result,
-                "needs_architect_review": self._needs_architect_review,
-                **input_data,
-            },
-            input_tokens,
-            output_tokens,
-        )
-
-    async def _scan(self, input_data: dict, tier: ModelTier) -> tuple[dict, int, int]:
-        """Security scan and bug pattern matching.
-
-        When external_audit_results is provided in input_data (e.g., from
-        SecurityAuditCrew), these findings are merged with the LLM analysis
-        and can trigger architect_review if critical issues are found.
-        """
-        code_to_review = input_data.get("code_to_review", input_data.get("diff", ""))
-        classification = input_data.get("classification", "")
-        files_changed = input_data.get("files_changed", input_data.get("files", []))
-
-        # Check for external audit results (e.g., from SecurityAuditCrew)
-        external_audit = input_data.get("external_audit_results")
-
-        system = """You are a security and code quality expert. Analyze the code for:
-
-1. SECURITY ISSUES (OWASP Top 10):
-   - SQL Injection, XSS, Command Injection
-   - Hardcoded secrets, API keys, passwords
-   - Insecure deserialization
-   - Authentication/authorization flaws
-
-2. BUG PATTERNS:
-   - Null/undefined references
-   - Resource leaks
-   - Race conditions
-   - Error handling issues
-
-3. CODE QUALITY:
-   - Code smells
-   - Maintainability issues
-   - Performance concerns
-
-For each issue found, provide:
-- Severity (critical/high/medium/low)
-- Location (if identifiable)
-- Description
-- Recommendation
-
-Be thorough but focused on actionable findings."""
-
-        # If external audit provided, include it in the prompt for context
-        external_context = ""
-        if external_audit:
-            external_summary = external_audit.get("summary", "")
-            external_findings = external_audit.get("findings", [])
-            if external_summary or external_findings:
-                # Build findings list efficiently (avoid O(n²) string concat)
-                finding_lines = []
-                for finding in external_findings[:10]:  # Top 10
-                    sev = finding.get("severity", "unknown").upper()
-                    title = finding.get("title", "N/A")
-                    desc = finding.get("description", "")[:100]
-                    finding_lines.append(f"- [{sev}] {title}: {desc}")
-
-                external_context = f"""
-
-## External Security Audit Results
-Summary: {external_summary}
-
-Findings ({len(external_findings)} total):
-{chr(10).join(finding_lines)}
-
-Verify these findings and identify additional issues."""
-
-        user_message = f"""Review this code for security and quality issues:
-
-Previous classification: {classification}
-{external_context}
-Code to review:
-{code_to_review[:6000]}"""
-
-        response, input_tokens, output_tokens = await self._call_llm(
-            tier,
-            system,
-            user_message,
-            max_tokens=2048,
-        )
-
-        # Extract structured findings from LLM response
-        llm_findings = self._extract_findings_from_response(
-            response=response,
-            files_changed=files_changed or [],
-            code_context=code_to_review[:1000],  # First 1000 chars for context
-        )
-
-        # Check if critical issues found in LLM response
-        has_critical = "critical" in response.lower() or "high" in response.lower()
-
-        # Merge external audit findings if provided
-        security_findings: list[dict] = []
-        external_has_critical = False
-
-        if external_audit:
-            merged_response, security_findings, external_has_critical = self._merge_external_audit(
-                response,
-                external_audit,
-            )
-            response = merged_response
-            has_critical = has_critical or external_has_critical
-
-        # Combine LLM findings with security findings
-        all_findings = llm_findings + security_findings
-
-        # Calculate summary statistics
-        summary: dict[str, Any] = {
-            "total_findings": len(all_findings),
-            "by_severity": {},
-            "by_category": {},
-            "files_affected": list({f.get("file", "") for f in all_findings if f.get("file")}),
-        }
-
-        # Count by severity
-        for finding in all_findings:
-            sev = finding.get("severity", "info")
-            summary["by_severity"][sev] = summary["by_severity"].get(sev, 0) + 1
-
-        # Count by category
-        for finding in all_findings:
-            cat = finding.get("category", "other")
-            summary["by_category"][cat] = summary["by_category"].get(cat, 0) + 1
-
-        # Add helpful message if no findings
-        if len(all_findings) == 0:
-            summary["message"] = (
-                "No security or quality issues found in scan. "
-                "Code will proceed to architectural review."
-            )
-
-        # Calculate security score
-        security_score = 70 if has_critical else 90
-
-        # Determine preliminary verdict based on scan
-        if has_critical:
-            preliminary_verdict = "request_changes"
-        elif security_score >= 90:
-            preliminary_verdict = "approve"
-        else:
-            preliminary_verdict = "approve_with_suggestions"
-
-        result = {
-            "scan_results": response,
-            "findings": all_findings,  # NEW: structured findings for UI
-            "summary": summary,  # NEW: summary statistics
-            "security_findings": security_findings,  # Keep for backward compat
-            "bug_patterns": [],
-            "quality_issues": [],
-            "has_critical_issues": has_critical,
-            "security_score": security_score,
-            "verdict": preliminary_verdict,  # Add verdict for when architect_review is skipped
-            "needs_architect_review": input_data.get("needs_architect_review", False)
-            or has_critical,
-            "code_to_review": code_to_review,
-            "classification": classification,
-            "external_audit_included": external_audit is not None,
-            "external_audit_risk_score": (
-                external_audit.get("risk_score", 0) if external_audit else 0
-            ),
-            "auth_mode_used": self._auth_mode_used,  # Track auth mode
-            "model_tier_used": tier.value,  # Track model tier
-        }
-
-        # Generate formatted report (for when architect_review is skipped)
-        formatted_report = format_code_review_report(result, input_data)
-        result["formatted_report"] = formatted_report
-        result["display_output"] = formatted_report
-
-        return (result, input_tokens, output_tokens)
-
-    def _merge_external_audit(
-        self,
-        llm_response: str,
-        external_audit: dict,
-    ) -> tuple[str, list, bool]:
-        """Merge external SecurityAuditCrew results into scan output.
-
-        Args:
-            llm_response: Response from LLM security scan
-            external_audit: External audit dict (from SecurityAuditCrew.to_dict())
-
-        Returns:
-            Tuple of (merged_response, security_findings, has_critical)
-
-        """
-        findings = external_audit.get("findings", [])
-        summary = external_audit.get("summary", "")
-        risk_score = external_audit.get("risk_score", 0)
-
-        # Check for critical/high findings
-        has_critical = any(f.get("severity") in ("critical", "high") for f in findings)
-
-        # Build merged response
-        merged_sections = [llm_response]
-
-        if summary or findings:
-            # Build crew section efficiently (avoid O(n²) string concat)
-            parts = ["\n\n## SecurityAuditCrew Analysis\n"]
-            if summary:
-                parts.append(f"\n{summary}\n")
-
-            parts.append(f"\n**Risk Score**: {risk_score}/100\n")
-
-            if findings:
-                critical = [f for f in findings if f.get("severity") == "critical"]
-                high = [f for f in findings if f.get("severity") == "high"]
-
-                if critical:
-                    parts.append("\n### Critical Findings\n")
-                    for f in critical:
-                        title = f"- **{f.get('title', 'N/A')}**"
-                        if f.get("file"):
-                            title += f" ({f.get('file')}:{f.get('line', '?')})"
-                        parts.append(title)
-                        parts.append(f"\n  {f.get('description', '')[:200]}\n")
-                        if f.get("remediation"):
-                            parts.append(f"  *Fix*: {f.get('remediation')[:150]}\n")
-
-                if high:
-                    parts.append("\n### High Severity Findings\n")
-                    for f in high[:5]:  # Top 5
-                        title = f"- **{f.get('title', 'N/A')}**"
-                        if f.get("file"):
-                            title += f" ({f.get('file')}:{f.get('line', '?')})"
-                        parts.append(title)
-                        parts.append(f"\n  {f.get('description', '')[:150]}\n")
-
-            merged_sections.append("".join(parts))
-
-        return "\n".join(merged_sections), findings, has_critical
-
-    async def _architect_review(self, input_data: dict, tier: ModelTier) -> tuple[dict, int, int]:
-        """Deep architectural review.
-
-        Supports XML-enhanced prompts when enabled in workflow config.
-        """
-        code_to_review = input_data.get("code_to_review", "")
-        scan_results = input_data.get("scan_results", "")
-        classification = input_data.get("classification", "")
-
-        # Build input payload
-        input_payload = f"""Classification: {classification}
-
-Security Scan Results:
-{scan_results[:2000]}
-
-Code:
-{code_to_review[:4000]}"""
-
-        # Check if XML prompts are enabled
-        if self._is_xml_enabled():
-            user_message = self._render_xml_prompt(
-                role="senior software architect",
-                goal="Perform comprehensive code review with architectural assessment",
-                instructions=[
-                    "Assess design patterns used (or missing)",
-                    "Evaluate SOLID principles compliance",
-                    "Check separation of concerns",
-                    "Analyze coupling and cohesion",
-                    "Provide specific improvement recommendations with examples",
-                    "Suggest refactoring and testing improvements",
-                    "Provide verdict: approve, approve_with_suggestions, or reject",
-                ],
-                constraints=[
-                    "Be specific and actionable",
-                    "Reference file locations where possible",
-                    "Prioritize issues by impact",
-                ],
-                input_type="code",
-                input_payload=input_payload,
-            )
-            system = None
-        else:
-            system = """You are a senior software architect. Provide a comprehensive review:
-
-1. ARCHITECTURAL ASSESSMENT:
-   - Design patterns used (or missing)
-   - SOLID principles compliance
-   - Separation of concerns
-   - Coupling and cohesion
-
-2. RECOMMENDATIONS:
-   - Specific improvements with examples
-   - Refactoring suggestions
-   - Testing recommendations
-
-3. VERDICT:
-   - APPROVE: Code is production-ready
-   - APPROVE_WITH_SUGGESTIONS: Minor improvements recommended
-   - REQUEST_CHANGES: Issues must be addressed
-   - REJECT: Fundamental problems
-
-Provide actionable, specific feedback."""
-
-            user_message = f"""Perform an architectural review:
-
-{input_payload}"""
-
-        # Try executor-based execution first (Phase 3 pattern)
-        if self._executor is not None or self._api_key:
-            try:
-                step = CODE_REVIEW_STEPS["architect_review"]
-                response, input_tokens, output_tokens, cost = await self.run_step_with_executor(
-                    step=step,
-                    prompt=user_message,
-                    system=system,
-                )
-            except Exception:
-                # Fall back to legacy _call_llm if executor fails
-                response, input_tokens, output_tokens = await self._call_llm(
-                    tier,
-                    system or "",
-                    user_message,
-                    max_tokens=3000,
-                )
-        else:
-            # Legacy path for backward compatibility
-            response, input_tokens, output_tokens = await self._call_llm(
-                tier,
-                system or "",
-                user_message,
-                max_tokens=3000,
-            )
-
-        # Parse XML response if enforcement is enabled
-        parsed_data = self._parse_xml_response(response)
-
-        # Determine verdict from response or parsed data
-        verdict = "approve_with_suggestions"
-        if parsed_data.get("xml_parsed"):
-            extra = parsed_data.get("_parsed_response")
-            if extra and hasattr(extra, "extra"):
-                parsed_verdict = extra.extra.get("verdict", "").lower()
-                if parsed_verdict in [
-                    "approve",
-                    "approve_with_suggestions",
-                    "request_changes",
-                    "reject",
-                ]:
-                    verdict = parsed_verdict
-
-        if verdict == "approve_with_suggestions":
-            # Fall back to text parsing
-            if "REQUEST_CHANGES" in response.upper() or "REJECT" in response.upper():
-                verdict = "request_changes"
-            elif "APPROVE" in response.upper() and "SUGGESTIONS" not in response.upper():
-                verdict = "approve"
-
-        result: dict = {
-            "architectural_review": response,
-            "verdict": verdict,
-            "recommendations": [],
-            "model_tier_used": tier.value,
-            "auth_mode_used": self._auth_mode_used,
-        }
-
-        # Merge parsed XML data if available
-        if parsed_data.get("xml_parsed"):
-            result.update(
-                {
-                    "xml_parsed": True,
-                    "summary": parsed_data.get("summary"),
-                    "findings": parsed_data.get("findings", []),
-                    "checklist": parsed_data.get("checklist", []),
-                },
-            )
-
-        # Add formatted report for human readability
-        formatted_report = format_code_review_report(result, input_data)
-        result["formatted_report"] = formatted_report
-
-        # Also add as top-level display_output for better UX
-        result["display_output"] = formatted_report
-
-        return (result, input_tokens, output_tokens)
-
-
-def format_code_review_report(result: dict, input_data: dict) -> str:
-    """Format code review output as a human-readable report.
-
-    Args:
-        result: The architect_review stage result
-        input_data: Input data from previous stages
-
-    Returns:
-        Formatted report string
-
-    """
-    lines = []
-
-    # Check for input validation error
-    if input_data.get("error"):
-        lines.append("=" * 60)
-        lines.append("CODE REVIEW - INPUT ERROR")
-        lines.append("=" * 60)
-        lines.append("")
-        lines.append(input_data.get("error_message", "No code provided for review."))
-        lines.append("")
-        lines.append("=" * 60)
-        return "\n".join(lines)
-
-    # Header
-    verdict = result.get("verdict", "unknown").upper().replace("_", " ")
-    verdict_icon = {
-        "APPROVE": "✅",
-        "APPROVE WITH SUGGESTIONS": "🔶",
-        "REQUEST CHANGES": "⚠️",
-        "REJECT": "❌",
-    }.get(verdict, "❓")
-
-    lines.append("=" * 60)
-    lines.append("CODE REVIEW REPORT")
-    lines.append("=" * 60)
-    lines.append("")
-    lines.append(f"Verdict: {verdict_icon} {verdict}")
-    lines.append("")
-
-    # Classification summary
-    classification = input_data.get("classification", "")
-    if classification:
-        lines.append("-" * 60)
-        lines.append("CLASSIFICATION")
-        lines.append("-" * 60)
-        lines.append(classification[:500])
-        lines.append("")
-
-    # Security scan results
-    has_critical = input_data.get("has_critical_issues", False)
-    security_score = input_data.get("security_score", 100)
-    security_icon = "🔴" if has_critical else ("🟡" if security_score < 90 else "🟢")
-
-    lines.append("-" * 60)
-    lines.append("SECURITY ANALYSIS")
-    lines.append("-" * 60)
-    lines.append(f"Security Score: {security_icon} {security_score}/100")
-    lines.append(f"Critical Issues: {'Yes' if has_critical else 'No'}")
-    lines.append("")
-
-    # Security findings
-    security_findings = input_data.get("security_findings", [])
-    if security_findings:
-        lines.append("Security Findings:")
-        for finding in security_findings[:10]:
-            severity = finding.get("severity", "unknown").upper()
-            title = finding.get("title", "N/A")
-            sev_icon = {"CRITICAL": "🔴", "HIGH": "🟠", "MEDIUM": "🟡", "LOW": "🟢"}.get(
-                severity,
-                "⚪",
-            )
-            lines.append(f"  {sev_icon} [{severity}] {title}")
-        lines.append("")
-
-    # Scan results summary
-    scan_results = input_data.get("scan_results", "")
-    if scan_results:
-        lines.append("Scan Summary:")
-        # Truncate scan results for readability
-        summary = scan_results[:800]
-        if len(scan_results) > 800:
-            summary += "..."
-        lines.append(summary)
-        lines.append("")
-
-    # Architectural review
-    arch_review = result.get("architectural_review", "")
-    if arch_review:
-        lines.append("-" * 60)
-        lines.append("ARCHITECTURAL REVIEW")
-        lines.append("-" * 60)
-        lines.append(arch_review)
-        lines.append("")
-
-    # Recommendations
-    recommendations = result.get("recommendations", [])
-    if recommendations:
-        lines.append("-" * 60)
-        lines.append("RECOMMENDATIONS")
-        lines.append("-" * 60)
-        for i, rec in enumerate(recommendations, 1):
-            lines.append(f"{i}. {rec}")
-        lines.append("")
-
-    # Crew review results (if available)
-    crew_review = input_data.get("crew_review", {})
-    if crew_review and crew_review.get("available") and not crew_review.get("fallback"):
-        lines.append("-" * 60)
-        lines.append("CREW REVIEW ANALYSIS")
-        lines.append("-" * 60)
-        lines.append(f"Quality Score: {crew_review.get('quality_score', 'N/A')}/100")
-        lines.append(f"Finding Count: {crew_review.get('finding_count', 0)}")
-        agents = crew_review.get("agents_used", [])
-        if agents:
-            lines.append(f"Agents Used: {', '.join(agents)}")
-        summary = crew_review.get("summary", "")
-        if summary:
-            lines.append(f"Summary: {summary[:300]}")
-        lines.append("")
-
-    # Check if we have any meaningful content to show
-    content_sections = [
-        input_data.get("classification"),
-        input_data.get("security_findings"),
-        input_data.get("scan_results"),
-        result.get("architectural_review"),
-        result.get("recommendations"),
-    ]
-    has_content = any(content_sections)
-
-    # If no content was generated, add a helpful message
-    if not has_content and len(lines) < 15:  # Just header/footer, no real content
-        lines.append("-" * 60)
-        lines.append("NO ISSUES FOUND")
-        lines.append("-" * 60)
-        lines.append("")
-        lines.append("The code review workflow completed but found no issues to report.")
-        lines.append("This could mean:")
-        lines.append("  • No code was provided for review (check input parameters)")
-        lines.append("  • The code is clean and follows best practices")
-        lines.append("  • The workflow needs configuration (check .empathy/workflows.yaml)")
-        lines.append("")
-        lines.append("Tip: Try running with a specific file or diff:")
-        lines.append('  empathy workflow run code-review --input \'{"target": "path/to/file.py"}\'')
-        lines.append("")
-
-    # Footer
-    lines.append("=" * 60)
-    model_tier = result.get("model_tier_used", "unknown")
-    lines.append(f"Review completed using {model_tier} tier model")
-    lines.append("=" * 60)
-
-    return "\n".join(lines)