claude-mpm 4.7.4__py3-none-any.whl → 4.18.2__py3-none-any.whl

claude_mpm/services/diagnostics/checks/startup_log_check.py

@@ -17,7 +17,8 @@ from datetime import datetime
 from pathlib import Path
 from typing import Any, ClassVar, Dict, Optional
 
-from ..models import DiagnosticResult, DiagnosticStatus
+from ....core.enums import OperationResult, ValidationSeverity
+from ..models import DiagnosticResult
 from .base_check import BaseDiagnosticCheck
 
 
@@ -94,7 +95,7 @@ class StartupLogCheck(BaseDiagnosticCheck):
             if not log_file:
                 return DiagnosticResult(
                     category=self.category,
-                    status=DiagnosticStatus.WARNING,
+                    status=ValidationSeverity.WARNING,
                     message="No startup logs found",
                     details={
                         "recommendation": "Startup logging will be created on next run"
@@ -135,7 +136,7 @@ class StartupLogCheck(BaseDiagnosticCheck):
                     sub_results.append(
                         DiagnosticResult(
                             category="Error",
-                            status=DiagnosticStatus.ERROR,
+                            status=ValidationSeverity.ERROR,
                             message=error_type,
                             details={"fix": fix},
                         )
@@ -153,7 +154,7 @@ class StartupLogCheck(BaseDiagnosticCheck):
         except Exception as e:
             return DiagnosticResult(
                 category=self.category,
-                status=DiagnosticStatus.ERROR,
+                status=ValidationSeverity.ERROR,
                 message=f"Startup log check failed: {e!s}",
                 details={"error": str(e)},
             )
@@ -280,13 +281,13 @@ class StartupLogCheck(BaseDiagnosticCheck):
 
         analysis["recommendations"] = recommendations
 
-    def _determine_status(self, analysis: Dict[str, Any]) -> DiagnosticStatus:
+    def _determine_status(self, analysis: Dict[str, Any]):
         """Determine overall status based on analysis."""
         if analysis["error_count"] > 0:
-            return DiagnosticStatus.ERROR
+            return ValidationSeverity.ERROR
         if analysis["warning_count"] > 3 or analysis["warning_count"] > 0:
-            return DiagnosticStatus.WARNING
-        return DiagnosticStatus.OK
+            return ValidationSeverity.WARNING
+        return OperationResult.SUCCESS
 
     def _create_message(self, analysis: Dict[str, Any]) -> str:
         """Create summary message based on analysis."""

claude_mpm/services/diagnostics/diagnostic_runner.py

@@ -9,6 +9,7 @@ import asyncio
 from concurrent.futures import ThreadPoolExecutor, as_completed
 from typing import List, Type
 
+from claude_mpm.core.enums import ValidationSeverity
 from claude_mpm.core.logging_utils import get_logger
 
 from .checks import (
@@ -25,7 +26,7 @@ from .checks import (
     MonitorCheck,
     StartupLogCheck,
 )
-from .models import DiagnosticResult, DiagnosticStatus, DiagnosticSummary
+from .models import DiagnosticResult, DiagnosticSummary
 
 logger = get_logger(__name__)
 
@@ -92,7 +93,7 @@ class DiagnosticRunner:
                 self.logger.error(f"Check {check_class.__name__} failed: {e}")
                 error_result = DiagnosticResult(
                     category=check_class.__name__.replace("Check", ""),
-                    status=DiagnosticStatus.ERROR,
+                    status=ValidationSeverity.ERROR,
                     message=f"Check failed: {e!s}",
                     details={"error": str(e)},
                 )
@@ -167,7 +168,7 @@ class DiagnosticRunner:
                     results.append(
                         DiagnosticResult(
                             category=check_class.__name__.replace("Check", ""),
-                            status=DiagnosticStatus.ERROR,
+                            status=ValidationSeverity.ERROR,
                             message=f"Check initialization failed: {e!s}",
                             details={"error": str(e)},
                         )
@@ -183,7 +184,7 @@ class DiagnosticRunner:
                     results.append(
                         DiagnosticResult(
                             category=check_name.replace("Check", ""),
-                            status=DiagnosticStatus.ERROR,
+                            status=ValidationSeverity.ERROR,
                             message=f"Check execution failed: {e!s}",
                             details={"error": str(e)},
                         )
@@ -238,7 +239,7 @@ class DiagnosticRunner:
                 self.logger.error(f"Check {name} failed: {e}")
                 error_result = DiagnosticResult(
                     category=check_class.__name__.replace("Check", ""),
-                    status=DiagnosticStatus.ERROR,
+                    status=ValidationSeverity.ERROR,
                     message=f"Check failed: {e!s}",
                     details={"error": str(e)},
                 )

claude_mpm/services/diagnostics/doctor_reporter.py

@@ -8,7 +8,9 @@ formatting for terminal display and JSON export.
 import json
 import sys
 
-from .models import DiagnosticResult, DiagnosticStatus, DiagnosticSummary
+from claude_mpm.core.enums import OperationResult, ValidationSeverity
+
+from .models import DiagnosticResult, DiagnosticSummary
 
 
 class DoctorReporter:
@@ -20,10 +22,10 @@ class DoctorReporter:
 
     # Status symbols and colors
     STATUS_SYMBOLS = {
-        DiagnosticStatus.OK: "✅",
-        DiagnosticStatus.WARNING: "⚠️ ",
-        DiagnosticStatus.ERROR: "❌",
-        DiagnosticStatus.SKIPPED: "⏭️ ",
+        OperationResult.SUCCESS: "✅",
+        ValidationSeverity.WARNING: "⚠️ ",
+        ValidationSeverity.ERROR: "❌",
+        OperationResult.SKIPPED: "⏭️ ",
     }
 
     # ANSI color codes
@@ -94,11 +96,11 @@ class DoctorReporter:
         # Main result line
         line = f"{indent_str}{symbol} {result.category}: "
 
-        if result.status == DiagnosticStatus.OK:
+        if result.status == OperationResult.SUCCESS:
             line += self._color("OK", color)
-        elif result.status == DiagnosticStatus.WARNING:
+        elif result.status == ValidationSeverity.WARNING:
             line += self._color("Warning", color)
-        elif result.status == DiagnosticStatus.ERROR:
+        elif result.status == ValidationSeverity.ERROR:
             line += self._color("Error", color)
         else:
             line += self._color("Skipped", color)
@@ -163,9 +165,9 @@ class DoctorReporter:
 
         # Overall health
         overall = summary.overall_status
-        if overall == DiagnosticStatus.OK:
+        if overall == OperationResult.SUCCESS:
             print(self._color("\n✅ System is healthy!", "green"))
-        elif overall == DiagnosticStatus.WARNING:
+        elif overall == ValidationSeverity.WARNING:
             print(
                 self._color(
                     "\n⚠️  System has minor issues that should be addressed.", "yellow"
@@ -283,10 +285,10 @@ class DoctorReporter:
 
         # Overall Health Status
         overall = summary.overall_status
-        if overall == DiagnosticStatus.OK:
+        if overall == OperationResult.SUCCESS:
             print("### 🎉 Overall Status: **Healthy**")
             print("Your Claude MPM installation is functioning properly.\n")
-        elif overall == DiagnosticStatus.WARNING:
+        elif overall == ValidationSeverity.WARNING:
             print("### ⚠️ Overall Status: **Needs Attention**")
             print("Your installation has minor issues that should be addressed.\n")
         else:
@@ -344,13 +346,13 @@ class DoctorReporter:
         reset_code = self.COLORS["reset"]
         return f"{color_code}{text}{reset_code}"
 
-    def _get_status_color(self, status: DiagnosticStatus) -> str:
+    def _get_status_color(self, status) -> str:
         """Get color for a status."""
         color_map = {
-            DiagnosticStatus.OK: "green",
-            DiagnosticStatus.WARNING: "yellow",
-            DiagnosticStatus.ERROR: "red",
-            DiagnosticStatus.SKIPPED: "gray",
+            OperationResult.SUCCESS: "green",
+            ValidationSeverity.WARNING: "yellow",
+            ValidationSeverity.ERROR: "red",
+            OperationResult.SKIPPED: "gray",
         }
         return color_map.get(status, "reset")
 
@@ -475,12 +477,13 @@ class DoctorReporter:
         print(f"### {symbol} {result.category}\n")
 
         # Status badge
-        status_badge = {
-            DiagnosticStatus.OK: "![OK](https://img.shields.io/badge/status-OK-green)",
-            DiagnosticStatus.WARNING: "![Warning](https://img.shields.io/badge/status-Warning-yellow)",
-            DiagnosticStatus.ERROR: "![Error](https://img.shields.io/badge/status-Error-red)",
-            DiagnosticStatus.SKIPPED: "![Skipped](https://img.shields.io/badge/status-Skipped-gray)",
-        }.get(result.status, "")
+        status_badge_map = {
+            OperationResult.SUCCESS: "![OK](https://img.shields.io/badge/status-OK-green)",
+            ValidationSeverity.WARNING: "![Warning](https://img.shields.io/badge/status-Warning-yellow)",
+            ValidationSeverity.ERROR: "![Error](https://img.shields.io/badge/status-Error-red)",
+            OperationResult.SKIPPED: "![Skipped](https://img.shields.io/badge/status-Skipped-gray)",
+        }
+        status_badge = status_badge_map.get(result.status, "")
 
         print(f"{status_badge}")
         print(f"\n**Message:** {result.message}\n")
@@ -513,7 +516,7 @@ class DoctorReporter:
         for result in summary.results:
             if (
                 result.category == "Installation"
-                and result.status != DiagnosticStatus.OK
+                and result.status != OperationResult.SUCCESS
             ):
                 if "pipx" not in str(result.details.get("installation_method", "")):
                     recommendations.append(
@@ -532,7 +535,7 @@ class DoctorReporter:
 
             if (
                 result.category == "Claude Code"
-                and result.status == DiagnosticStatus.WARNING
+                and result.status == ValidationSeverity.WARNING
             ):
                 recommendations.append(
                     "Update Claude Code (CLI) to the latest version for best compatibility"

claude_mpm/services/diagnostics/models.py

@@ -6,17 +6,9 @@ consistency across all checks and reporting.
 """
 
 from dataclasses import dataclass, field
-from enum import Enum
-from typing import Any, Dict, List, Optional
+from typing import Any, Dict, List, Optional, Union
 
-
-class DiagnosticStatus(Enum):
-    """Status levels for diagnostic results."""
-
-    OK = "ok"
-    WARNING = "warning"
-    ERROR = "error"
-    SKIPPED = "skipped"
+from ...core.enums import OperationResult, ValidationSeverity
 
 
 @dataclass
@@ -25,10 +17,13 @@ class DiagnosticResult:
 
     WHY: Standardized result format ensures consistent reporting
     and makes it easy to aggregate and display results.
+
+    Note: status uses Union[OperationResult, ValidationSeverity] to support both
+    operation results (SUCCESS, SKIPPED) and validation results (WARNING, ERROR).
     """
 
     category: str  # e.g., "Installation", "Agents", "MCP Server"
-    status: DiagnosticStatus
+    status: Union[OperationResult, ValidationSeverity]
     message: str
     details: Dict[str, Any] = field(default_factory=dict)
     fix_command: Optional[str] = None
@@ -50,16 +45,16 @@ class DiagnosticResult:
     @property
     def has_issues(self) -> bool:
         """Check if this result indicates any issues."""
-        return self.status in (DiagnosticStatus.WARNING, DiagnosticStatus.ERROR)
+        return self.status in (ValidationSeverity.WARNING, ValidationSeverity.ERROR)
 
     @property
     def severity_level(self) -> int:
         """Get numeric severity level for sorting."""
         severity_map = {
-            DiagnosticStatus.OK: 0,
-            DiagnosticStatus.SKIPPED: 1,
-            DiagnosticStatus.WARNING: 2,
-            DiagnosticStatus.ERROR: 3,
+            OperationResult.SUCCESS: 0,
+            OperationResult.SKIPPED: 1,
+            ValidationSeverity.WARNING: 2,
+            ValidationSeverity.ERROR: 3,
         }
         return severity_map.get(self.status, 0)
 
@@ -84,13 +79,13 @@ class DiagnosticSummary:
         self.results.append(result)
         self.total_checks += 1
 
-        if result.status == DiagnosticStatus.OK:
+        if result.status == OperationResult.SUCCESS:
             self.ok_count += 1
-        elif result.status == DiagnosticStatus.WARNING:
+        elif result.status == ValidationSeverity.WARNING:
             self.warning_count += 1
-        elif result.status == DiagnosticStatus.ERROR:
+        elif result.status == ValidationSeverity.ERROR:
             self.error_count += 1
-        elif result.status == DiagnosticStatus.SKIPPED:
+        elif result.status == OperationResult.SKIPPED:
             self.skipped_count += 1
 
     @property
@@ -99,13 +94,13 @@ class DiagnosticSummary:
         return self.warning_count > 0 or self.error_count > 0
 
     @property
-    def overall_status(self) -> DiagnosticStatus:
+    def overall_status(self) -> Union[OperationResult, ValidationSeverity]:
         """Get overall system status."""
         if self.error_count > 0:
-            return DiagnosticStatus.ERROR
+            return ValidationSeverity.ERROR
         if self.warning_count > 0:
-            return DiagnosticStatus.WARNING
-        return DiagnosticStatus.OK
+            return ValidationSeverity.WARNING
+        return OperationResult.SUCCESS
 
     def to_dict(self) -> Dict[str, Any]:
         """Convert to dictionary for JSON serialization."""

claude_mpm/services/infrastructure/monitoring/__init__.py

@@ -3,12 +3,12 @@
 Exports main monitoring components for backward compatibility.
 """
 
+from ....core.enums import HealthStatus
 from .aggregator import MonitoringAggregatorService
 from .base import (
     HealthChecker,
     HealthCheckResult,
     HealthMetric,
-    HealthStatus,
 )
 
 # Legacy exports for backward compatibility

claude_mpm/services/infrastructure/monitoring/aggregator.py

@@ -9,12 +9,12 @@ import time
 from collections import deque
 from typing import Any, Callable, Dict, List, Optional
 
+from ....core.enums import HealthStatus
 from .base import (
     BaseMonitoringService,
     HealthChecker,
     HealthCheckResult,
     HealthMetric,
-    HealthStatus,
 )
 
 
@@ -212,14 +212,14 @@ class MonitoringAggregatorService(BaseMonitoringService):
 
         total_metrics = len(metrics)
 
-        # Critical if any critical metrics
-        if status_counts[HealthStatus.CRITICAL] > 0:
-            return HealthStatus.CRITICAL
+        # Unhealthy if any unhealthy metrics
+        if status_counts[HealthStatus.UNHEALTHY] > 0:
+            return HealthStatus.UNHEALTHY
 
-        # Warning if >30% warning metrics
-        warning_ratio = status_counts[HealthStatus.WARNING] / total_metrics
-        if warning_ratio > 0.3:
-            return HealthStatus.WARNING
+        # Degraded if >30% degraded metrics
+        degraded_ratio = status_counts[HealthStatus.DEGRADED] / total_metrics
+        if degraded_ratio > 0.3:
+            return HealthStatus.DEGRADED
 
         # Unknown if >50% unknown metrics
         unknown_ratio = status_counts[HealthStatus.UNKNOWN] / total_metrics
@@ -375,10 +375,10 @@ class MonitoringAggregatorService(BaseMonitoringService):
         checks_count = len(recent_results)
 
         # Determine aggregated status
-        if status_counts[HealthStatus.CRITICAL] > 0:
-            aggregated_status = HealthStatus.CRITICAL
-        elif status_counts[HealthStatus.WARNING] > checks_count * 0.3:
-            aggregated_status = HealthStatus.WARNING
+        if status_counts[HealthStatus.UNHEALTHY] > 0:
+            aggregated_status = HealthStatus.UNHEALTHY
+        elif status_counts[HealthStatus.DEGRADED] > checks_count * 0.3:
+            aggregated_status = HealthStatus.DEGRADED
         elif status_counts[HealthStatus.UNKNOWN] > checks_count * 0.5:
             aggregated_status = HealthStatus.UNKNOWN
         else:

claude_mpm/services/infrastructure/monitoring/base.py

@@ -7,17 +7,9 @@ import time
 from abc import ABC, abstractmethod
 from dataclasses import asdict, dataclass
 from datetime import datetime, timezone
-from enum import Enum
 from typing import Any, Dict, List, Optional, Union
 
-
-class HealthStatus(Enum):
-    """Health status levels for monitoring."""
-
-    HEALTHY = "healthy"
-    WARNING = "warning"
-    CRITICAL = "critical"
-    UNKNOWN = "unknown"
+from ....core.enums import HealthStatus
 
 
 @dataclass
@@ -75,11 +67,11 @@ class HealthCheckResult:
             "healthy_metrics": len(
                 [m for m in self.metrics if m.status == HealthStatus.HEALTHY]
             ),
-            "warning_metrics": len(
-                [m for m in self.metrics if m.status == HealthStatus.WARNING]
+            "degraded_metrics": len(
+                [m for m in self.metrics if m.status == HealthStatus.DEGRADED]
             ),
-            "critical_metrics": len(
-                [m for m in self.metrics if m.status == HealthStatus.CRITICAL]
+            "unhealthy_metrics": len(
+                [m for m in self.metrics if m.status == HealthStatus.UNHEALTHY]
             ),
         }
 

claude_mpm/services/infrastructure/monitoring/network.py

@@ -6,7 +6,8 @@ Monitors network connectivity, port availability, and socket health.
 import socket
 from typing import Dict, List, Optional
 
-from .base import BaseMonitoringService, HealthMetric, HealthStatus
+from ....core.enums import HealthStatus
+from .base import BaseMonitoringService, HealthMetric
 
 
 class NetworkHealthService(BaseMonitoringService):
@@ -98,7 +99,7 @@ class NetworkHealthService(BaseMonitoringService):
                 HealthMetric(
                     name="socket_creation",
                     value=False,
-                    status=HealthStatus.CRITICAL,
+                    status=HealthStatus.UNHEALTHY,
                     message=f"Failed to create socket: {e}",
                 )
             )
@@ -142,11 +143,11 @@ class NetworkHealthService(BaseMonitoringService):
                     )
                 )
             else:
-                # Determine if this is critical or warning based on endpoint type
+                # Determine if this is unhealthy or degraded based on endpoint type
                 status = (
-                    HealthStatus.WARNING
+                    HealthStatus.DEGRADED
                     if "optional" in name.lower()
-                    else HealthStatus.CRITICAL
+                    else HealthStatus.UNHEALTHY
                 )
                 metrics.append(
                     HealthMetric(
@@ -161,7 +162,7 @@ class NetworkHealthService(BaseMonitoringService):
                 HealthMetric(
                     name=metric_name,
                     value=False,
-                    status=HealthStatus.WARNING,
+                    status=HealthStatus.DEGRADED,
                     message=f"Connection to {host}:{port} timed out after {timeout}s",
                 )
             )

claude_mpm/services/infrastructure/monitoring/process.py

@@ -6,8 +6,9 @@ Monitors individual process health including CPU, memory, file descriptors, and
 from typing import List
 
 from claude_mpm.core.constants import ResourceLimits, TimeoutConfig
+from claude_mpm.core.enums import HealthStatus
 
-from .base import BaseMonitoringService, HealthMetric, HealthStatus
+from .base import BaseMonitoringService, HealthMetric
 
 try:
     import psutil
@@ -66,7 +67,7 @@ class ProcessHealthService(BaseMonitoringService):
                 HealthMetric(
                     name="psutil_availability",
                     value=False,
-                    status=HealthStatus.WARNING,
+                    status=HealthStatus.DEGRADED,
                     message="psutil not available for process monitoring",
                 )
             )
@@ -77,7 +78,7 @@ class ProcessHealthService(BaseMonitoringService):
                 HealthMetric(
                     name="process_exists",
                     value=False,
-                    status=HealthStatus.CRITICAL,
+                    status=HealthStatus.UNHEALTHY,
                     message=f"Process {self.pid} not found",
                 )
             )
@@ -90,7 +91,7 @@ class ProcessHealthService(BaseMonitoringService):
                     HealthMetric(
                         name="process_exists",
                         value=False,
-                        status=HealthStatus.CRITICAL,
+                        status=HealthStatus.UNHEALTHY,
                         message=f"Process {self.pid} is no longer running",
                     )
                 )
@@ -119,7 +120,7 @@ class ProcessHealthService(BaseMonitoringService):
                 HealthMetric(
                     name="process_exists",
                     value=False,
-                    status=HealthStatus.CRITICAL,
+                    status=HealthStatus.UNHEALTHY,
                     message=f"Process {self.pid} no longer exists",
                 )
             )
@@ -153,7 +154,7 @@ class ProcessHealthService(BaseMonitoringService):
                     status=(
                         HealthStatus.HEALTHY
                         if process_healthy
-                        else HealthStatus.CRITICAL
+                        else HealthStatus.UNHEALTHY
                     ),
                     message=f"Process status: {status}",
                 )
@@ -179,9 +180,9 @@ class ProcessHealthService(BaseMonitoringService):
             cpu_status = HealthStatus.HEALTHY
             if cpu_percent > self.cpu_threshold:
                 cpu_status = (
-                    HealthStatus.WARNING
+                    HealthStatus.DEGRADED
                     if cpu_percent < self.cpu_threshold * 1.2
-                    else HealthStatus.CRITICAL
+                    else HealthStatus.UNHEALTHY
                 )
 
             metrics.append(
@@ -213,9 +214,9 @@ class ProcessHealthService(BaseMonitoringService):
             memory_status = HealthStatus.HEALTHY
             if memory_mb > self.memory_threshold_mb:
                 memory_status = (
-                    HealthStatus.WARNING
+                    HealthStatus.DEGRADED
                     if memory_mb < self.memory_threshold_mb * 1.2
-                    else HealthStatus.CRITICAL
+                    else HealthStatus.UNHEALTHY
                 )
 
             metrics.append(
@@ -256,9 +257,9 @@ class ProcessHealthService(BaseMonitoringService):
                 fd_status = HealthStatus.HEALTHY
                 if fd_count > self.fd_threshold:
                     fd_status = (
-                        HealthStatus.WARNING
+                        HealthStatus.DEGRADED
                         if fd_count < self.fd_threshold * 1.2
-                        else HealthStatus.CRITICAL
+                        else HealthStatus.UNHEALTHY
                     )
 
                 metrics.append(

claude_mpm/services/infrastructure/monitoring/resources.py

@@ -5,7 +5,8 @@ Monitors system-wide resource usage including CPU, memory, and disk utilization.
 
 from typing import Dict, List, Optional
 
-from .base import BaseMonitoringService, HealthMetric, HealthStatus
+from ....core.enums import HealthStatus
+from .base import BaseMonitoringService, HealthMetric
 
 try:
     import psutil
@@ -53,7 +54,7 @@ class ResourceMonitorService(BaseMonitoringService):
                 HealthMetric(
                     name="psutil_availability",
                     value=False,
-                    status=HealthStatus.WARNING,
+                    status=HealthStatus.DEGRADED,
                     message="psutil not available for resource monitoring",
                 )
             )
@@ -182,9 +183,9 @@ class ResourceMonitorService(BaseMonitoringService):
                 # Load is concerning if > cpu_count
                 load_status = HealthStatus.HEALTHY
                 if load1 > cpu_count:
-                    load_status = HealthStatus.WARNING
+                    load_status = HealthStatus.DEGRADED
                 if load1 > cpu_count * 1.5:
-                    load_status = HealthStatus.CRITICAL
+                    load_status = HealthStatus.UNHEALTHY
 
                 metrics.append(
                     HealthMetric(
@@ -220,8 +221,8 @@ class ResourceMonitorService(BaseMonitoringService):
         if value < threshold:
             return HealthStatus.HEALTHY
         if value < threshold * 1.1:  # 10% above threshold
-            return HealthStatus.WARNING
-        return HealthStatus.CRITICAL
+            return HealthStatus.DEGRADED
+        return HealthStatus.UNHEALTHY
 
     def get_resource_summary(self) -> Optional[Dict[str, float]]:
         """Get quick resource summary without full health check.