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

claude_mpm/services/diagnostics/checks/mcp_check.py

@@ -9,7 +9,9 @@ import json
 import subprocess
 from pathlib import Path
 
-from ..models import DiagnosticResult, DiagnosticStatus
+from claude_mpm.core.enums import OperationResult, ServiceState, ValidationSeverity
+
+from ..models import DiagnosticResult
 from .base_check import BaseDiagnosticCheck
 
 
@@ -34,13 +36,13 @@ class MCPCheck(BaseDiagnosticCheck):
             # Check if MCP is installed
             install_result = self._check_installation()
             sub_results.append(install_result)
-            details["installed"] = install_result.status == DiagnosticStatus.OK
+            details["installed"] = install_result.status == OperationResult.SUCCESS
 
-            if install_result.status == DiagnosticStatus.OK:
+            if install_result.status == OperationResult.SUCCESS:
                 # Check MCP configuration
                 config_result = self._check_configuration()
                 sub_results.append(config_result)
-                details["configured"] = config_result.status == DiagnosticStatus.OK
+                details["configured"] = config_result.status == OperationResult.SUCCESS
 
                 # Check MCP server status
                 status_result = self._check_server_status()
@@ -52,17 +54,17 @@ class MCPCheck(BaseDiagnosticCheck):
                 sub_results.append(startup_result)
 
             # Determine overall status
-            if any(r.status == DiagnosticStatus.ERROR for r in sub_results):
-                status = DiagnosticStatus.ERROR
+            if any(r.status == ValidationSeverity.ERROR for r in sub_results):
+                status = ValidationSeverity.ERROR
                 message = "MCP server has critical issues"
             elif not details.get("installed", False):
-                status = DiagnosticStatus.WARNING
+                status = ValidationSeverity.WARNING
                 message = "MCP server not installed"
-            elif any(r.status == DiagnosticStatus.WARNING for r in sub_results):
-                status = DiagnosticStatus.WARNING
+            elif any(r.status == ValidationSeverity.WARNING for r in sub_results):
+                status = ValidationSeverity.WARNING
                 message = "MCP server needs configuration"
             else:
-                status = DiagnosticStatus.OK
+                status = OperationResult.SUCCESS
                 message = "MCP server properly configured"
 
             return DiagnosticResult(
@@ -76,7 +78,7 @@ class MCPCheck(BaseDiagnosticCheck):
         except Exception as e:
             return DiagnosticResult(
                 category=self.category,
-                status=DiagnosticStatus.ERROR,
+                status=ValidationSeverity.ERROR,
                 message=f"MCP check failed: {e!s}",
                 details={"error": str(e)},
             )
@@ -94,7 +96,7 @@ class MCPCheck(BaseDiagnosticCheck):
             if mcp_path.exists():
                 return DiagnosticResult(
                     category="MCP Installation",
-                    status=DiagnosticStatus.OK,
+                    status=OperationResult.SUCCESS,
                     message="MCP server installed",
                     details={"path": str(mcp_path), "installed": True},
                 )
@@ -112,7 +114,7 @@ class MCPCheck(BaseDiagnosticCheck):
                 path = result.stdout.strip()
                 return DiagnosticResult(
                     category="MCP Installation",
-                    status=DiagnosticStatus.OK,
+                    status=OperationResult.SUCCESS,
                     message="MCP server installed",
                     details={"path": path, "installed": True},
                 )
@@ -121,7 +123,7 @@ class MCPCheck(BaseDiagnosticCheck):
 
         return DiagnosticResult(
             category="MCP Installation",
-            status=DiagnosticStatus.WARNING,
+            status=ValidationSeverity.WARNING,
             message="MCP server not installed",
             details={"installed": False},
             fix_command="claude-mpm mcp install",
@@ -146,7 +148,7 @@ class MCPCheck(BaseDiagnosticCheck):
         if not config_path:
             return DiagnosticResult(
                 category="MCP Configuration",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message="Claude Code config not found",
                 details={"configured": False},
                 fix_command="claude-mpm mcp config",
@@ -163,7 +165,7 @@ class MCPCheck(BaseDiagnosticCheck):
                 if not gateway:
                     return DiagnosticResult(
                         category="MCP Configuration",
-                        status=DiagnosticStatus.WARNING,
+                        status=ValidationSeverity.WARNING,
                         message="MCP gateway not configured",
                         details={"configured": False, "config_path": str(config_path)},
                         fix_command="claude-mpm mcp config",
@@ -175,7 +177,7 @@ class MCPCheck(BaseDiagnosticCheck):
                 if not command:
                     return DiagnosticResult(
                         category="MCP Configuration",
-                        status=DiagnosticStatus.ERROR,
+                        status=ValidationSeverity.ERROR,
                         message="MCP gateway misconfigured (no command)",
                         details={
                             "configured": True,
@@ -188,7 +190,7 @@ class MCPCheck(BaseDiagnosticCheck):
 
                 return DiagnosticResult(
                     category="MCP Configuration",
-                    status=DiagnosticStatus.OK,
+                    status=OperationResult.SUCCESS,
                     message="MCP gateway configured",
                     details={
                         "configured": True,
@@ -200,7 +202,7 @@ class MCPCheck(BaseDiagnosticCheck):
         except json.JSONDecodeError as e:
             return DiagnosticResult(
                 category="MCP Configuration",
-                status=DiagnosticStatus.ERROR,
+                status=ValidationSeverity.ERROR,
                 message="Invalid JSON in config file",
                 details={"error": str(e), "config_path": str(config_path)},
                 fix_description="Fix JSON syntax in Claude Code config",
@@ -208,7 +210,7 @@ class MCPCheck(BaseDiagnosticCheck):
         except Exception as e:
             return DiagnosticResult(
                 category="MCP Configuration",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message=f"Could not check configuration: {e!s}",
                 details={"error": str(e)},
             )
@@ -226,41 +228,53 @@ class MCPCheck(BaseDiagnosticCheck):
             )
 
             if result.returncode == 0:
-                if "running" in result.stdout.lower():
+                if ServiceState.RUNNING.value in result.stdout.lower():
                     return DiagnosticResult(
                         category="MCP Server Status",
-                        status=DiagnosticStatus.OK,
+                        status=OperationResult.SUCCESS,
                         message="MCP server is running",
-                        details={"running": True},
+                        details={"running": True, "state": ServiceState.RUNNING},
                     )
                 return DiagnosticResult(
                     category="MCP Server Status",
-                    status=DiagnosticStatus.WARNING,
+                    status=ValidationSeverity.WARNING,
                     message="MCP server not running",
-                    details={"running": False},
+                    details={"running": False, "state": ServiceState.STOPPED},
                     fix_command="claude-mpm mcp start",
                     fix_description="Start the MCP server",
                 )
             return DiagnosticResult(
                 category="MCP Server Status",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message="Could not determine server status",
-                details={"running": "unknown", "error": result.stderr},
+                details={
+                    "running": "unknown",
+                    "state": ServiceState.UNKNOWN,
+                    "error": result.stderr,
+                },
             )
 
         except subprocess.TimeoutExpired:
             return DiagnosticResult(
                 category="MCP Server Status",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message="Server status check timed out",
-                details={"running": "unknown", "error": "timeout"},
+                details={
+                    "running": "unknown",
+                    "state": ServiceState.UNKNOWN,
+                    "error": "timeout",
+                },
             )
         except Exception as e:
             return DiagnosticResult(
                 category="MCP Server Status",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message=f"Could not check server status: {e!s}",
-                details={"running": "unknown", "error": str(e)},
+                details={
+                    "running": "unknown",
+                    "state": ServiceState.UNKNOWN,
+                    "error": str(e),
+                },
             )
 
     def _check_startup_verification(self) -> DiagnosticResult:
@@ -276,7 +290,7 @@ class MCPCheck(BaseDiagnosticCheck):
             if not issues:
                 return DiagnosticResult(
                     category="MCP Startup Verification",
-                    status=DiagnosticStatus.OK,
+                    status=OperationResult.SUCCESS,
                     message="Startup verification passed",
                     details={"issues": []},
                 )
@@ -290,20 +304,20 @@ class MCPCheck(BaseDiagnosticCheck):
             if errors:
                 return DiagnosticResult(
                     category="MCP Startup Verification",
-                    status=DiagnosticStatus.ERROR,
+                    status=ValidationSeverity.ERROR,
                     message=f"{len(errors)} critical issue(s) found",
                     details={"errors": errors, "warnings": warnings},
                 )
             if warnings:
                 return DiagnosticResult(
                     category="MCP Startup Verification",
-                    status=DiagnosticStatus.WARNING,
+                    status=ValidationSeverity.WARNING,
                     message=f"{len(warnings)} warning(s) found",
                     details={"warnings": warnings},
                 )
             return DiagnosticResult(
                 category="MCP Startup Verification",
-                status=DiagnosticStatus.OK,
+                status=OperationResult.SUCCESS,
                 message="Startup verification passed",
                 details={"issues": []},
             )
@@ -311,7 +325,7 @@ class MCPCheck(BaseDiagnosticCheck):
         except Exception as e:
             return DiagnosticResult(
                 category="MCP Startup Verification",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message=f"Could not verify startup: {e!s}",
                 details={"error": str(e)},
             )

claude_mpm/services/diagnostics/checks/mcp_services_check.py

@@ -14,7 +14,8 @@ from typing import Dict, List, Optional, Tuple
 
 from claude_mpm.core.logger import get_logger
 
-from ..models import DiagnosticResult, DiagnosticStatus
+from ....core.enums import OperationResult, ValidationSeverity
+from ..models import DiagnosticResult
 from .base_check import BaseDiagnosticCheck
 
 
@@ -116,7 +117,9 @@ class MCPServicesCheck(BaseDiagnosticCheck):
                 fix_result = DiagnosticResult(
                     category="MCP Service Fixes",
                     status=(
-                        DiagnosticStatus.OK if fix_success else DiagnosticStatus.WARNING
+                        OperationResult.SUCCESS
+                        if fix_success
+                        else ValidationSeverity.WARNING
                     ),
                     message=fix_message,
                     details={"auto_fix_applied": True},
@@ -134,9 +137,9 @@ class MCPServicesCheck(BaseDiagnosticCheck):
                 config_result = DiagnosticResult(
                     category="MCP Configuration Update",
                     status=(
-                        DiagnosticStatus.OK
+                        OperationResult.SUCCESS
                         if config_success
-                        else DiagnosticStatus.WARNING
+                        else ValidationSeverity.WARNING
                     ),
                     message=config_message,
                     details={"auto_config_applied": True},
@@ -191,32 +194,34 @@ class MCPServicesCheck(BaseDiagnosticCheck):
             details["connected_count"] = connected_count
             details["total_services"] = total_services
             details["total_tools_discovered"] = total_tools
-            details["gateway_configured"] = gateway_result.status == DiagnosticStatus.OK
+            details["gateway_configured"] = (
+                gateway_result.status == OperationResult.SUCCESS
+            )
 
             # Determine overall status
-            errors = [r for r in sub_results if r.status == DiagnosticStatus.ERROR]
-            [r for r in sub_results if r.status == DiagnosticStatus.WARNING]
+            errors = [r for r in sub_results if r.status == ValidationSeverity.ERROR]
+            [r for r in sub_results if r.status == ValidationSeverity.WARNING]
 
             if errors:
-                status = DiagnosticStatus.ERROR
+                status = ValidationSeverity.ERROR
                 message = f"Critical issues with {len(errors)} MCP service(s)"
             elif installed_count == 0:
-                status = DiagnosticStatus.WARNING
+                status = ValidationSeverity.WARNING
                 message = "No MCP services installed"
             elif connected_count == total_services:
-                status = DiagnosticStatus.OK
+                status = OperationResult.SUCCESS
                 message = f"All {total_services} MCP services connected ({total_tools} tools available)"
             elif connected_count > 0:
-                status = DiagnosticStatus.WARNING
+                status = ValidationSeverity.WARNING
                 message = f"{connected_count}/{total_services} MCP services connected, {installed_count} installed"
             elif accessible_count < installed_count:
-                status = DiagnosticStatus.WARNING
+                status = ValidationSeverity.WARNING
                 message = f"{installed_count}/{total_services} services installed, {accessible_count} accessible"
             elif installed_count < total_services:
-                status = DiagnosticStatus.WARNING
+                status = ValidationSeverity.WARNING
                 message = f"{installed_count}/{total_services} MCP services installed"
             else:
-                status = DiagnosticStatus.WARNING
+                status = ValidationSeverity.WARNING
                 message = f"All {total_services} MCP services installed but connections not tested"
 
             return DiagnosticResult(
@@ -230,7 +235,7 @@ class MCPServicesCheck(BaseDiagnosticCheck):
         except Exception as e:
             return DiagnosticResult(
                 category=self.category,
-                status=DiagnosticStatus.ERROR,
+                status=ValidationSeverity.ERROR,
                 message=f"MCP services check failed: {e!s}",
                 details={"error": str(e)},
             )
@@ -495,7 +500,7 @@ class MCPServicesCheck(BaseDiagnosticCheck):
                     base_cmd = config["mcp_command"]
                     if len(base_cmd) > 0 and base_cmd[0] == config["package"]:
                         # Simple case where first command is the package name
-                        mcp_command = ["pipx", "run", config["package"]] + base_cmd[1:]
+                        mcp_command = ["pipx", "run", config["package"], *base_cmd[1:]]
                     else:
                         # Complex case - just try running the package with mcp arg
                         mcp_command = ["pipx", "run", config["package"], "mcp"]
@@ -509,7 +514,7 @@ class MCPServicesCheck(BaseDiagnosticCheck):
                     base_cmd = config["mcp_command"]
                     if service_name == "kuzu-memory":
                         # Special case for kuzu-memory with args
-                        mcp_command = ["pipx", "run", base_cmd[0]] + base_cmd[1:]
+                        mcp_command = ["pipx", "run", base_cmd[0], *base_cmd[1:]]
                     else:
                         mcp_command = ["pipx", "run", *base_cmd]
             else:
@@ -543,7 +548,7 @@ class MCPServicesCheck(BaseDiagnosticCheck):
         if not (pipx_installed or accessible):
             return DiagnosticResult(
                 category=f"MCP Service: {service_name}",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message=f"Not installed: {config['description']}",
                 details=details,
                 fix_command=f"pipx install {config['package']}",
@@ -565,16 +570,16 @@ class MCPServicesCheck(BaseDiagnosticCheck):
                 return DiagnosticResult(
                     category=f"MCP Service: {service_name}",
                     status=(
-                        DiagnosticStatus.OK
+                        OperationResult.SUCCESS
                         if connection_info.get("connected")
-                        else DiagnosticStatus.WARNING
+                        else ValidationSeverity.WARNING
                     ),
                     message=message,
                     details=details,
                 )
             return DiagnosticResult(
                 category=f"MCP Service: {service_name}",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message="Installed via pipx but not in PATH",
                 details=details,
                 fix_command="pipx ensurepath",
@@ -588,14 +593,14 @@ class MCPServicesCheck(BaseDiagnosticCheck):
                 response_time = connection_info.get("response_time_ms")
                 tools_count = connection_info.get("tools_discovered", 0)
                 message = f"Installed, accessible, connection OK ({tools_count} tools, {response_time}ms)"
-                status = DiagnosticStatus.OK
+                status = OperationResult.SUCCESS
             else:
                 error = connection_info.get("error", "Unknown error")
                 message = f"Installed but connection failed: {error}"
-                status = DiagnosticStatus.WARNING
+                status = ValidationSeverity.WARNING
         else:
             message = "Installed and accessible"
-            status = DiagnosticStatus.OK
+            status = OperationResult.SUCCESS
 
         return DiagnosticResult(
             category=f"MCP Service: {service_name}",
@@ -826,7 +831,7 @@ class MCPServicesCheck(BaseDiagnosticCheck):
                 if fixed:
                     return DiagnosticResult(
                         category="kuzu-memory Configuration Fix",
-                        status=DiagnosticStatus.OK,
+                        status=OperationResult.SUCCESS,
                         message="Fixed kuzu-memory configuration to use correct args",
                         details={
                             "old_args": args,
@@ -837,7 +842,7 @@ class MCPServicesCheck(BaseDiagnosticCheck):
                     )
                 return DiagnosticResult(
                     category="kuzu-memory Configuration",
-                    status=DiagnosticStatus.WARNING,
+                    status=ValidationSeverity.WARNING,
                     message="kuzu-memory has incorrect configuration",
                     details={
                         "current_args": args,
@@ -931,7 +936,7 @@ class MCPServicesCheck(BaseDiagnosticCheck):
             if not config_file.exists():
                 return DiagnosticResult(
                     category="MCP Gateway Configuration",
-                    status=DiagnosticStatus.WARNING,
+                    status=ValidationSeverity.WARNING,
                     message="Claude configuration file not found",
                     details={"config_path": str(config_file), "exists": False},
                     fix_command="claude-mpm configure --mcp",
@@ -951,7 +956,7 @@ class MCPServicesCheck(BaseDiagnosticCheck):
             if current_project not in projects:
                 return DiagnosticResult(
                     category="MCP Gateway Configuration",
-                    status=DiagnosticStatus.WARNING,
+                    status=ValidationSeverity.WARNING,
                     message="Current project not configured in Claude",
                     details={
                         "config_path": str(config_file),
@@ -982,7 +987,7 @@ class MCPServicesCheck(BaseDiagnosticCheck):
             if not configured_services:
                 return DiagnosticResult(
                     category="MCP Gateway Configuration",
-                    status=DiagnosticStatus.WARNING,
+                    status=ValidationSeverity.WARNING,
                     message="No MCP services configured in gateway",
                     details=details,
                     fix_command="claude-mpm configure --mcp --add-services",
@@ -992,14 +997,14 @@ class MCPServicesCheck(BaseDiagnosticCheck):
             if missing_services:
                 return DiagnosticResult(
                     category="MCP Gateway Configuration",
-                    status=DiagnosticStatus.WARNING,
+                    status=ValidationSeverity.WARNING,
                     message=f"{len(configured_services)} services configured, {len(missing_services)} missing",
                     details=details,
                 )
 
             return DiagnosticResult(
                 category="MCP Gateway Configuration",
-                status=DiagnosticStatus.OK,
+                status=OperationResult.SUCCESS,
                 message=f"All {len(configured_services)} services configured",
                 details=details,
             )
@@ -1007,14 +1012,14 @@ class MCPServicesCheck(BaseDiagnosticCheck):
         except json.JSONDecodeError as e:
             return DiagnosticResult(
                 category="MCP Gateway Configuration",
-                status=DiagnosticStatus.ERROR,
+                status=ValidationSeverity.ERROR,
                 message="Invalid JSON in MCP configuration",
                 details={"error": str(e)},
             )
         except Exception as e:
             return DiagnosticResult(
                 category="MCP Gateway Configuration",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message=f"Could not check configuration: {e!s}",
                 details={"error": str(e)},
             )

claude_mpm/services/diagnostics/checks/monitor_check.py

@@ -7,7 +7,8 @@ properly configured and accessible for real-time updates.
 
 import socket
 
-from ..models import DiagnosticResult, DiagnosticStatus
+from ....core.enums import OperationResult, ValidationSeverity
+from ..models import DiagnosticResult
 from .base_check import BaseDiagnosticCheck
 
 
@@ -33,7 +34,7 @@ class MonitorCheck(BaseDiagnosticCheck):
             socketio_result = self._check_socketio()
             sub_results.append(socketio_result)
             details["socketio_available"] = (
-                socketio_result.status == DiagnosticStatus.OK
+                socketio_result.status == OperationResult.SUCCESS
             )
 
             # Check port availability
@@ -49,17 +50,17 @@ class MonitorCheck(BaseDiagnosticCheck):
             # Check hook service
             hook_result = self._check_hook_service()
             sub_results.append(hook_result)
-            details["hooks_enabled"] = hook_result.status == DiagnosticStatus.OK
+            details["hooks_enabled"] = hook_result.status == OperationResult.SUCCESS
 
             # Determine overall status
-            if any(r.status == DiagnosticStatus.ERROR for r in sub_results):
-                status = DiagnosticStatus.ERROR
+            if any(r.status == ValidationSeverity.ERROR for r in sub_results):
+                status = ValidationSeverity.ERROR
                 message = "Monitoring has critical issues"
-            elif any(r.status == DiagnosticStatus.WARNING for r in sub_results):
-                status = DiagnosticStatus.WARNING
+            elif any(r.status == ValidationSeverity.WARNING for r in sub_results):
+                status = ValidationSeverity.WARNING
                 message = "Monitoring has minor issues"
             else:
-                status = DiagnosticStatus.OK
+                status = OperationResult.SUCCESS
                 message = "Monitoring properly configured"
 
             return DiagnosticResult(
@@ -73,7 +74,7 @@ class MonitorCheck(BaseDiagnosticCheck):
         except Exception as e:
             return DiagnosticResult(
                 category=self.category,
-                status=DiagnosticStatus.ERROR,
+                status=ValidationSeverity.ERROR,
                 message=f"Monitor check failed: {e!s}",
                 details={"error": str(e)},
             )
@@ -85,7 +86,7 @@ class MonitorCheck(BaseDiagnosticCheck):
 
             return DiagnosticResult(
                 category="SocketIO",
-                status=DiagnosticStatus.OK,
+                status=OperationResult.SUCCESS,
                 message="SocketIO library available",
                 details={
                     "available": True,
@@ -95,7 +96,7 @@ class MonitorCheck(BaseDiagnosticCheck):
         except ImportError:
             return DiagnosticResult(
                 category="SocketIO",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message="SocketIO not installed",
                 details={"available": False},
                 fix_command="pip install python-socketio[asyncio]",
@@ -122,7 +123,7 @@ class MonitorCheck(BaseDiagnosticCheck):
             if not available_ports:
                 return DiagnosticResult(
                     category="Port Availability",
-                    status=DiagnosticStatus.ERROR,
+                    status=ValidationSeverity.ERROR,
                     message="No monitoring ports available (8765-8785)",
                     details={
                         "available": [],
@@ -137,7 +138,7 @@ class MonitorCheck(BaseDiagnosticCheck):
             if default_port not in available_ports:
                 return DiagnosticResult(
                     category="Port Availability",
-                    status=DiagnosticStatus.WARNING,
+                    status=ValidationSeverity.WARNING,
                     message=f"Default port {default_port} in use, but alternatives available",
                     details={
                         "available": available_ports,
@@ -148,7 +149,7 @@ class MonitorCheck(BaseDiagnosticCheck):
 
             return DiagnosticResult(
                 category="Port Availability",
-                status=DiagnosticStatus.OK,
+                status=OperationResult.SUCCESS,
                 message=f"{len(available_ports)} monitoring port(s) available",
                 details={
                     "available": available_ports,
@@ -160,7 +161,7 @@ class MonitorCheck(BaseDiagnosticCheck):
         except Exception as e:
             return DiagnosticResult(
                 category="Port Availability",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message=f"Could not check ports: {e!s}",
                 details={"error": str(e)},
             )
@@ -209,7 +210,7 @@ class MonitorCheck(BaseDiagnosticCheck):
         if not response_logging_enabled:
             return DiagnosticResult(
                 category="Response Logging",
-                status=DiagnosticStatus.OK,
+                status=OperationResult.SUCCESS,
                 message="Response logging disabled (default)",
                 details={"enabled": False},
             )
@@ -219,7 +220,7 @@ class MonitorCheck(BaseDiagnosticCheck):
         if not log_dir.exists():
             return DiagnosticResult(
                 category="Response Logging",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message="Response logging enabled but directory missing",
                 details={"enabled": True, "path": str(log_dir), "exists": False},
                 fix_command=f"mkdir -p {log_dir}",
@@ -231,7 +232,7 @@ class MonitorCheck(BaseDiagnosticCheck):
         if not os.access(log_dir, os.W_OK):
             return DiagnosticResult(
                 category="Response Logging",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message="Response logging directory not writable",
                 details={"enabled": True, "path": str(log_dir), "writable": False},
                 fix_command=f"chmod 755 {log_dir}",
@@ -240,7 +241,7 @@ class MonitorCheck(BaseDiagnosticCheck):
 
         return DiagnosticResult(
             category="Response Logging",
-            status=DiagnosticStatus.OK,
+            status=OperationResult.SUCCESS,
             message="Response logging enabled and configured",
             details={
                 "enabled": True,
@@ -260,7 +261,7 @@ class MonitorCheck(BaseDiagnosticCheck):
 
             return DiagnosticResult(
                 category="Hook Service",
-                status=DiagnosticStatus.OK,
+                status=OperationResult.SUCCESS,
                 message="Hook service available",
                 details={"available": True},
             )
@@ -268,14 +269,14 @@ class MonitorCheck(BaseDiagnosticCheck):
         except ImportError:
             return DiagnosticResult(
                 category="Hook Service",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message="Hook service not available",
                 details={"available": False},
             )
         except Exception as e:
             return DiagnosticResult(
                 category="Hook Service",
-                status=DiagnosticStatus.WARNING,
+                status=ValidationSeverity.WARNING,
                 message=f"Hook service error: {e!s}",
                 details={"available": False, "error": str(e)},
             )