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

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.

claude_mpm/services/infrastructure/monitoring/service.py

@@ -6,7 +6,8 @@ Monitors service-specific metrics like client connections, event processing, and
 import time
 from typing import Any, Dict, List
 
-from .base import BaseMonitoringService, HealthMetric, HealthStatus
+from ....core.enums import HealthStatus
+from .base import BaseMonitoringService, HealthMetric
 
 
 class ServiceHealthService(BaseMonitoringService):
@@ -79,9 +80,9 @@ class ServiceHealthService(BaseMonitoringService):
 
             # Determine status based on thresholds
             if client_count > self.max_clients:
-                client_status = HealthStatus.CRITICAL
+                client_status = HealthStatus.UNHEALTHY
             elif client_count > self.max_clients * 0.8:
-                client_status = HealthStatus.WARNING
+                client_status = HealthStatus.DEGRADED
             else:
                 client_status = HealthStatus.HEALTHY
 
@@ -129,7 +130,7 @@ class ServiceHealthService(BaseMonitoringService):
                 # Determine status based on rate
                 rate_status = HealthStatus.HEALTHY
                 if event_rate == 0 and events_processed > 0:
-                    rate_status = HealthStatus.WARNING  # Processing stopped
+                    rate_status = HealthStatus.DEGRADED  # Processing stopped
 
                 metrics.append(
                     HealthMetric(
@@ -157,9 +158,9 @@ class ServiceHealthService(BaseMonitoringService):
                 queue_size = self.service_stats["event_queue_size"]
                 queue_status = HealthStatus.HEALTHY
                 if queue_size > 1000:
-                    queue_status = HealthStatus.WARNING
+                    queue_status = HealthStatus.DEGRADED
                 if queue_size > 5000:
-                    queue_status = HealthStatus.CRITICAL
+                    queue_status = HealthStatus.UNHEALTHY
 
                 metrics.append(
                     HealthMetric(
@@ -191,9 +192,9 @@ class ServiceHealthService(BaseMonitoringService):
 
             # Determine status based on rate
             if error_rate > self.max_error_rate:
-                error_status = HealthStatus.CRITICAL
+                error_status = HealthStatus.UNHEALTHY
             elif error_rate > self.max_error_rate * 0.5:
-                error_status = HealthStatus.WARNING
+                error_status = HealthStatus.DEGRADED
             else:
                 error_status = HealthStatus.HEALTHY
 
@@ -213,7 +214,7 @@ class ServiceHealthService(BaseMonitoringService):
                     name="total_errors",
                     value=errors,
                     status=(
-                        HealthStatus.HEALTHY if errors == 0 else HealthStatus.WARNING
+                        HealthStatus.HEALTHY if errors == 0 else HealthStatus.DEGRADED
                     ),
                 )
             )
@@ -228,7 +229,7 @@ class ServiceHealthService(BaseMonitoringService):
                         status=(
                             HealthStatus.HEALTHY
                             if recent_errors == 0
-                            else HealthStatus.WARNING
+                            else HealthStatus.DEGRADED
                         ),
                     )
                 )
@@ -263,9 +264,9 @@ class ServiceHealthService(BaseMonitoringService):
 
                 # Determine status based on staleness
                 if time_since_activity > self.stale_activity_seconds * 2:
-                    activity_status = HealthStatus.CRITICAL
+                    activity_status = HealthStatus.UNHEALTHY
                 elif time_since_activity > self.stale_activity_seconds:
-                    activity_status = HealthStatus.WARNING
+                    activity_status = HealthStatus.DEGRADED
                 else:
                     activity_status = HealthStatus.HEALTHY
 
@@ -282,7 +283,7 @@ class ServiceHealthService(BaseMonitoringService):
                     HealthMetric(
                         name="time_since_last_activity",
                         value=-1,
-                        status=HealthStatus.WARNING,
+                        status=HealthStatus.DEGRADED,
                         message="No last activity recorded",
                     )
                 )
@@ -307,9 +308,9 @@ class ServiceHealthService(BaseMonitoringService):
 
             # Determine status based on response time
             if avg_time > 1000:  # > 1 second
-                time_status = HealthStatus.CRITICAL
+                time_status = HealthStatus.UNHEALTHY
             elif avg_time > 500:  # > 500ms
-                time_status = HealthStatus.WARNING
+                time_status = HealthStatus.DEGRADED
             else:
                 time_status = HealthStatus.HEALTHY