claude-mpm 4.14.8__py3-none-any.whl → 4.15.0__py3-none-any.whl

claude_mpm/services/agents/deployment/pipeline/steps/base_step.py

@@ -2,26 +2,17 @@
 
 from abc import ABC, abstractmethod
 from dataclasses import dataclass
-from enum import Enum
 from typing import Optional
 
+from claude_mpm.core.enums import OperationResult
 from claude_mpm.core.logger import get_logger
 
 
-class StepStatus(Enum):
-    """Status of a pipeline step execution."""
-
-    SUCCESS = "success"
-    FAILURE = "failure"
-    SKIPPED = "skipped"
-    WARNING = "warning"
-
-
 @dataclass
 class StepResult:
     """Result of executing a pipeline step."""
 
-    status: StepStatus
+    status: OperationResult
     message: Optional[str] = None
     error: Optional[Exception] = None
     execution_time: Optional[float] = None
@@ -29,22 +20,22 @@ class StepResult:
     @property
     def is_success(self) -> bool:
         """Check if the step was successful."""
-        return self.status == StepStatus.SUCCESS
+        return self.status == OperationResult.SUCCESS
 
     @property
     def is_failure(self) -> bool:
         """Check if the step failed."""
-        return self.status == StepStatus.FAILURE
+        return self.status == OperationResult.FAILED
 
     @property
     def is_skipped(self) -> bool:
         """Check if the step was skipped."""
-        return self.status == StepStatus.SKIPPED
+        return self.status == OperationResult.SKIPPED
 
     @property
     def is_warning(self) -> bool:
         """Check if the step completed with warnings."""
-        return self.status == StepStatus.WARNING
+        return self.status == OperationResult.WARNING
 
 
 class BaseDeploymentStep(ABC):
@@ -67,7 +58,7 @@ class BaseDeploymentStep(ABC):
         self.logger = get_logger(f"{__name__}.{self.__class__.__name__}")
 
     @abstractmethod
-    def execute(self, context) -> StepResult:
+    def execute(self, context) -> "StepResult":
         """Execute this deployment step.
 
         Args:

claude_mpm/services/agents/deployment/pipeline/steps/configuration_step.py

@@ -3,8 +3,9 @@
 import time
 
 from claude_mpm.core.config import Config
+from claude_mpm.core.enums import OperationResult
 
-from .base_step import BaseDeploymentStep, StepResult, StepStatus
+from .base_step import BaseDeploymentStep, StepResult
 
 
 class ConfigurationLoadStep(BaseDeploymentStep):
@@ -54,7 +55,7 @@ class ConfigurationLoadStep(BaseDeploymentStep):
             context.step_timings[self.name] = execution_time
 
             return StepResult(
-                status=StepStatus.SUCCESS,
+                status=OperationResult.SUCCESS,
                 message=f"Configuration loaded successfully in {execution_time:.3f}s",
                 execution_time=execution_time,
             )
@@ -68,7 +69,7 @@ class ConfigurationLoadStep(BaseDeploymentStep):
             context.add_error(error_msg)
 
             return StepResult(
-                status=StepStatus.FAILURE,
+                status=OperationResult.FAILED,
                 message=error_msg,
                 error=e,
                 execution_time=execution_time,

claude_mpm/services/agents/deployment/pipeline/steps/target_directory_step.py

@@ -3,7 +3,9 @@
 import time
 from pathlib import Path
 
-from .base_step import BaseDeploymentStep, StepResult, StepStatus
+from claude_mpm.core.enums import OperationResult
+
+from .base_step import BaseDeploymentStep, StepResult
 
 
 class TargetDirectorySetupStep(BaseDeploymentStep):
@@ -59,7 +61,7 @@ class TargetDirectorySetupStep(BaseDeploymentStep):
             context.step_timings[self.name] = execution_time
 
             return StepResult(
-                status=StepStatus.SUCCESS,
+                status=OperationResult.SUCCESS,
                 message=f"Target directory set up at {context.actual_target_dir} in {execution_time:.3f}s",
                 execution_time=execution_time,
             )
@@ -73,7 +75,7 @@ class TargetDirectorySetupStep(BaseDeploymentStep):
             context.add_error(error_msg)
 
             return StepResult(
-                status=StepStatus.FAILURE,
+                status=OperationResult.FAILED,
                 message=error_msg,
                 error=e,
                 execution_time=execution_time,

claude_mpm/services/agents/deployment/pipeline/steps/validation_step.py

@@ -2,9 +2,10 @@
 
 import time
 
+from claude_mpm.core.enums import OperationResult
 from claude_mpm.services.agents.deployment.validation import DeploymentValidator
 
-from .base_step import BaseDeploymentStep, StepResult, StepStatus
+from .base_step import BaseDeploymentStep, StepResult
 
 
 class ValidationStep(BaseDeploymentStep):
@@ -49,13 +50,13 @@ class ValidationStep(BaseDeploymentStep):
 
             # Determine step status
             if not validation_result.is_valid:
-                status = StepStatus.FAILURE
+                status = OperationResult.FAILED
                 message = f"Validation failed with {validation_result.error_count} errors in {execution_time:.3f}s"
             elif validation_result.has_warnings:
-                status = StepStatus.WARNING
+                status = OperationResult.WARNING
                 message = f"Validation completed with {validation_result.warning_count} warnings in {execution_time:.3f}s"
             else:
-                status = StepStatus.SUCCESS
+                status = OperationResult.SUCCESS
                 message = f"Validation passed successfully in {execution_time:.3f}s"
 
             self.logger.info(message)
@@ -73,7 +74,7 @@ class ValidationStep(BaseDeploymentStep):
             context.add_error(error_msg)
 
             return StepResult(
-                status=StepStatus.FAILURE,
+                status=OperationResult.FAILED,
                 message=error_msg,
                 error=e,
                 execution_time=execution_time,

claude_mpm/services/agents/deployment/refactored_agent_deployment_service.py

@@ -117,8 +117,10 @@ class RefactoredAgentDeploymentService(AgentDeploymentInterface):
             )
 
             # Ensure success field is present
-            if "success" not in results:
-                results["success"] = not bool(results.get("errors", []))
+            if OperationResult.SUCCESS.value not in results:
+                results[OperationResult.SUCCESS.value] = not bool(
+                    results.get("errors", [])
+                )
 
             self.logger.info(f"Deployment completed: {results.get('success', False)}")
             return results
@@ -126,7 +128,7 @@ class RefactoredAgentDeploymentService(AgentDeploymentInterface):
         except Exception as e:
             self.logger.error(f"Deployment failed: {e}", exc_info=True)
             return {
-                "success": False,
+                OperationResult.SUCCESS.value: False,
                 "error": str(e),
                 "deployed": [],
                 "updated": [],
@@ -302,7 +304,7 @@ class RefactoredAgentDeploymentService(AgentDeploymentInterface):
         except Exception as e:
             self.logger.error(f"Async deployment failed: {e}", exc_info=True)
             return {
-                "success": False,
+                OperationResult.SUCCESS.value: False,
                 "error": str(e),
                 "deployed": [],
                 "updated": [],

claude_mpm/services/agents/registry/modification_tracker.py

@@ -36,6 +36,7 @@ from watchdog.events import FileSystemEvent, FileSystemEventHandler
 from watchdog.observers import Observer
 
 from claude_mpm.core.base_service import BaseService
+from claude_mpm.core.enums import OperationResult
 from claude_mpm.core.logging_utils import get_logger
 from claude_mpm.core.unified_agent_registry import UnifiedAgentRegistry as AgentRegistry
 from claude_mpm.core.unified_paths import get_path_manager
@@ -551,12 +552,14 @@ class AgentModificationTracker(BaseService):
                     errors.append(f"File does not exist: {modification.file_path}")
 
             # Update validation status
-            modification.validation_status = "failed" if errors else "passed"
+            modification.validation_status = (
+                OperationResult.FAILED if errors else OperationResult.SUCCESS
+            )
             modification.validation_errors = errors
 
         except Exception as e:
             self.logger.error(f"Validation error: {e}")
-            modification.validation_status = "error"
+            modification.validation_status = OperationResult.ERROR
             modification.validation_errors.append(str(e))
 
     async def _invalidate_agent_cache(self, agent_name: str) -> None:

claude_mpm/services/command_handler_service.py

@@ -12,6 +12,7 @@ Extracted from ClaudeRunner to follow Single Responsibility Principle.
 from typing import Any, Dict, List
 
 from claude_mpm.core.base_service import BaseService
+from claude_mpm.core.enums import OperationResult
 from claude_mpm.services.core.interfaces import CommandHandlerInterface
 
 
@@ -169,7 +170,7 @@ class CommandHandlerService(BaseService, CommandHandlerInterface):
             if command == "test":
                 success = self._handle_test_command(args)
                 return {
-                    "success": success,
+                    OperationResult.SUCCESS.value: success,
                     "command": command,
                     "args": args,
                     "message": (
@@ -179,7 +180,7 @@ class CommandHandlerService(BaseService, CommandHandlerInterface):
             if command == "agents":
                 success = self._handle_agents_command(args)
                 return {
-                    "success": success,
+                    OperationResult.SUCCESS.value: success,
                     "command": command,
                     "args": args,
                     "message": (
@@ -189,14 +190,19 @@ class CommandHandlerService(BaseService, CommandHandlerInterface):
                     ),
                 }
             return {
-                "success": False,
+                OperationResult.SUCCESS.value: False,
                 "command": command,
                 "args": args,
-                "error": f"Unknown command: {command}",
+                OperationResult.ERROR.value: f"Unknown command: {command}",
                 "available_commands": self.get_available_commands(),
             }
         except Exception as e:
-            return {"success": False, "command": command, "args": args, "error": str(e)}
+            return {
+                OperationResult.SUCCESS.value: False,
+                "command": command,
+                "args": args,
+                OperationResult.ERROR.value: str(e),
+            }
 
     def get_command_help(self, command: str) -> str:
         """Get help text for a specific command.

claude_mpm/services/core/interfaces/process.py

@@ -32,10 +32,10 @@ USAGE:
 from abc import ABC, abstractmethod
 from typing import Dict, List, Optional
 
+from claude_mpm.core.enums import ServiceState
 from claude_mpm.services.core.models.process import (
     DeploymentState,
     ProcessInfo,
-    ProcessStatus,
     StartConfig,
 )
 
@@ -102,12 +102,12 @@ class IDeploymentStateManager(ABC):
         """
 
     @abstractmethod
-    def get_deployments_by_status(self, status: ProcessStatus) -> List[DeploymentState]:
+    def get_deployments_by_status(self, status: ServiceState) -> List[DeploymentState]:
         """
         Get all deployments with a specific status.
 
         Args:
-            status: ProcessStatus to filter by
+            status: ServiceState to filter by
 
         Returns:
             List of matching DeploymentState objects
@@ -168,14 +168,14 @@ class IDeploymentStateManager(ABC):
 
     @abstractmethod
     def update_deployment_status(
-        self, deployment_id: str, status: ProcessStatus
+        self, deployment_id: str, status: ServiceState
     ) -> bool:
         """
         Update the status of a deployment.
 
         Args:
             deployment_id: Unique deployment identifier
-            status: New ProcessStatus
+            status: New ServiceState
 
         Returns:
             True if updated, False if deployment not found
@@ -290,7 +290,7 @@ class ILocalProcessManager(ABC):
 
     @abstractmethod
     def list_processes(
-        self, status_filter: Optional[ProcessStatus] = None
+        self, status_filter: Optional[ServiceState] = None
     ) -> List[ProcessInfo]:
         """
         List all managed processes.

claude_mpm/services/core/models/__init__.py

@@ -24,7 +24,6 @@ from .process import (
     PROTECTED_PORT_RANGES,
     DeploymentState,
     ProcessInfo,
-    ProcessStatus,
     StartConfig,
     is_port_protected,
 )
@@ -63,7 +62,6 @@ __all__ = [  # noqa: RUF022 - Grouped by category with comments for clarity
     "ValidationResult",
     "ConfigurationPreview",
     # Process management models
-    "ProcessStatus",
     "DeploymentState",
     "ProcessInfo",
     "StartConfig",

claude_mpm/services/core/models/agent_config.py

@@ -17,6 +17,8 @@ from dataclasses import dataclass, field
 from enum import Enum
 from typing import Any, Dict, List, Optional
 
+from ....core.enums import OperationResult, ValidationSeverity
+
 
 class AgentSpecialization(str, Enum):
     """Agent specialization categories.
@@ -154,20 +156,6 @@ class AgentRecommendation:
         }
 
 
-class ConfigurationStatus(str, Enum):
-    """Status of configuration operation.
-
-    WHY: Configuration can succeed, fail, or partially succeed. This enum
-    provides a standardized way to communicate operation outcomes.
-    """
-
-    SUCCESS = "success"
-    PARTIAL_SUCCESS = "partial_success"
-    FAILURE = "failure"
-    VALIDATION_ERROR = "validation_error"
-    USER_CANCELLED = "user_cancelled"
-
-
 @dataclass
 class ConfigurationResult:
     """Result of automated configuration operation.
@@ -179,9 +167,17 @@ class ConfigurationResult:
     DESIGN DECISION: Separates successful and failed deployments to enable
     proper error handling. Includes validation results and user-facing
     messages for transparency.
+
+    NOTE: Uses core OperationResult enum (consolidated from ConfigurationStatus
+    in Phase 3A Batch 25). Mappings:
+    - SUCCESS → OperationResult.SUCCESS
+    - PARTIAL_SUCCESS → OperationResult.WARNING (partial success with issues)
+    - FAILURE → OperationResult.FAILED
+    - VALIDATION_ERROR → OperationResult.ERROR
+    - USER_CANCELLED → OperationResult.CANCELLED
     """
 
-    status: ConfigurationStatus
+    status: OperationResult
     deployed_agents: List[str] = field(default_factory=list)
     failed_agents: List[str] = field(default_factory=list)
     validation_warnings: List[str] = field(default_factory=list)
@@ -193,7 +189,7 @@ class ConfigurationResult:
     @property
     def is_successful(self) -> bool:
         """Check if configuration was completely successful."""
-        return self.status == ConfigurationStatus.SUCCESS
+        return self.status == OperationResult.SUCCESS
 
     @property
     def has_failures(self) -> bool:
@@ -223,18 +219,6 @@ class ConfigurationResult:
         }
 
 
-class ValidationSeverity(str, Enum):
-    """Severity level for validation issues.
-
-    WHY: Not all validation issues are equally critical. This enum enables
-    categorization of issues by severity to support appropriate handling.
-    """
-
-    ERROR = "error"  # Blocks deployment
-    WARNING = "warning"  # Should be reviewed but doesn't block
-    INFO = "info"  # Informational only
-
-
 @dataclass
 class ValidationIssue:
     """Represents a validation issue.

claude_mpm/services/core/models/process.py

@@ -3,56 +3,27 @@ Process Management Data Models for Claude MPM Framework
 ========================================================
 
 WHY: This module defines data structures for process management operations,
-including process status, deployment state, and runtime information.
+including deployment state and runtime information.
 
 DESIGN DECISION: Uses dataclasses for immutability and type safety. Provides
-serialization methods for state persistence.
+serialization methods for state persistence. Process status uses ServiceState
+enum from core.enums (consolidated in Phase 3A Batch 24).
 
 ARCHITECTURE:
-- ProcessStatus: Enum of process lifecycle states
 - DeploymentState: Complete deployment information for persistence
 - ProcessInfo: Runtime process information
 - StartConfig: Configuration for spawning new processes
+
+Note: ProcessStatus enum has been consolidated into ServiceState (core.enums).
 """
 
 import json
 from dataclasses import asdict, dataclass, field
 from datetime import datetime
-from enum import Enum
 from pathlib import Path
 from typing import Any, Dict, List, Optional
 
-
-class ProcessStatus(Enum):
-    """
-    Process lifecycle status.
-
-    WHY: Explicit status tracking enables proper state machine management
-    and prevents invalid state transitions.
-
-    States:
-        STARTING: Process is being spawned
-        RUNNING: Process is actively running
-        STOPPING: Process is shutting down
-        STOPPED: Process has stopped cleanly
-        CRASHED: Process terminated unexpectedly
-        UNKNOWN: Process state cannot be determined
-    """
-
-    STARTING = "starting"
-    RUNNING = "running"
-    STOPPING = "stopping"
-    STOPPED = "stopped"
-    CRASHED = "crashed"
-    UNKNOWN = "unknown"
-
-    def is_active(self) -> bool:
-        """Check if status represents an active process."""
-        return self in (ProcessStatus.STARTING, ProcessStatus.RUNNING)
-
-    def is_terminal(self) -> bool:
-        """Check if status represents a terminal state."""
-        return self in (ProcessStatus.STOPPED, ProcessStatus.CRASHED)
+from claude_mpm.core.enums import ServiceState
 
 
 @dataclass
@@ -71,7 +42,7 @@ class DeploymentState:
         environment: Environment variables (beyond inherited ones)
         port: Primary port used by the process (if applicable)
         started_at: Timestamp when process was started
-        status: Current ProcessStatus
+        status: Current ServiceState (process lifecycle state)
         metadata: Additional deployment-specific information
     """
 
@@ -82,7 +53,7 @@ class DeploymentState:
     environment: Dict[str, str] = field(default_factory=dict)
     port: Optional[int] = None
     started_at: datetime = field(default_factory=datetime.now)
-    status: ProcessStatus = ProcessStatus.STARTING
+    status: ServiceState = ServiceState.STARTING
     metadata: Dict[str, Any] = field(default_factory=dict)
 
     def to_dict(self) -> Dict[str, Any]:
@@ -114,7 +85,14 @@ class DeploymentState:
 
         # Convert status string to enum
         if isinstance(data.get("status"), str):
-            data["status"] = ProcessStatus(data["status"])
+            status_value = data["status"]
+            # Handle legacy "crashed" status -> map to ERROR
+            if status_value == "crashed":
+                data["status"] = (
+                    ServiceState.ERROR
+                )  # CRASHED semantically maps to ERROR state
+            else:
+                data["status"] = ServiceState(status_value)
 
         return cls(**data)
 
@@ -153,18 +131,18 @@ class ProcessInfo:
     Attributes:
         deployment_id: Unique deployment identifier
         process_id: OS process ID
-        status: Current ProcessStatus
+        status: Current ServiceState (process lifecycle state)
         port: Port the process is using
         uptime_seconds: How long the process has been running
         memory_mb: Current memory usage in megabytes
         cpu_percent: Current CPU usage percentage
         is_responding: Whether the process responds to health checks
-        error_message: Error message if status is CRASHED
+        error_message: Error message if status is ERROR (crashed)
     """
 
     deployment_id: str
     process_id: int
-    status: ProcessStatus
+    status: ServiceState
     port: Optional[int] = None
     uptime_seconds: float = 0.0
     memory_mb: float = 0.0
@@ -252,7 +230,6 @@ __all__ = [
     "PROTECTED_PORT_RANGES",
     "DeploymentState",
     "ProcessInfo",
-    "ProcessStatus",
     "StartConfig",
     "is_port_protected",
 ]

claude_mpm/services/local_ops/__init__.py

@@ -32,9 +32,9 @@ USAGE:
         DeploymentStateManager,
         HealthCheckManager,
         StartConfig,
-        ProcessStatus,
         HealthStatus,
     )
+    from claude_mpm.core.enums import ServiceState
 
     # Initialize managers
     state_manager = DeploymentStateManager(".claude-mpm/deployment-state.json")
@@ -54,6 +54,8 @@ USAGE:
 
     # Start background monitoring
     health_manager.start_monitoring()
+
+Note: ProcessStatus has been consolidated into ServiceState (core.enums) as of Phase 3A Batch 24.
 """
 
 # Re-export data models and interfaces for convenience
@@ -84,7 +86,6 @@ from claude_mpm.services.core.models.process import (
     PROTECTED_PORT_RANGES,
     DeploymentState,
     ProcessInfo,
-    ProcessStatus,
     StartConfig,
     is_port_protected,
 )
@@ -147,7 +148,6 @@ __all__ = [
     "ProcessInfo",
     "ProcessSpawnError",
     # Data models - Process
-    "ProcessStatus",
     "ResourceMonitor",
     "ResourceUsage",
     "RestartAttempt",

claude_mpm/services/local_ops/process_manager.py

@@ -42,6 +42,7 @@ from typing import List, Optional
 
 import psutil
 
+from claude_mpm.core.enums import ServiceState
 from claude_mpm.services.core.base import SyncBaseService
 from claude_mpm.services.core.interfaces.process import (
     IDeploymentStateManager,
@@ -50,7 +51,6 @@ from claude_mpm.services.core.interfaces.process import (
 from claude_mpm.services.core.models.process import (
     DeploymentState,
     ProcessInfo,
-    ProcessStatus,
     StartConfig,
     is_port_protected,
 )
@@ -203,7 +203,7 @@ class LocalProcessManager(SyncBaseService, ILocalProcessManager):
                 environment=config.environment,
                 port=allocated_port,
                 started_at=datetime.now(tz=timezone.utc),
-                status=ProcessStatus.RUNNING,
+                status=ServiceState.RUNNING,
                 metadata=config.metadata,
             )
 
@@ -248,13 +248,13 @@ class LocalProcessManager(SyncBaseService, ILocalProcessManager):
             # Process already dead, just update state
             self.log_info(f"Process {deployment.process_id} already dead")
             self.state_manager.update_deployment_status(
-                deployment_id, ProcessStatus.STOPPED
+                deployment_id, ServiceState.STOPPED
             )
             return True
 
         self.log_info(f"Stopping process {deployment.process_id} for {deployment_id}")
         self.state_manager.update_deployment_status(
-            deployment_id, ProcessStatus.STOPPING
+            deployment_id, ServiceState.STOPPING
         )
 
         try:
@@ -262,7 +262,7 @@ class LocalProcessManager(SyncBaseService, ILocalProcessManager):
                 # Force kill immediately
                 self._kill_process_group(process)
                 self.state_manager.update_deployment_status(
-                    deployment_id, ProcessStatus.STOPPED
+                    deployment_id, ServiceState.STOPPED
                 )
                 return True
 
@@ -275,7 +275,7 @@ class LocalProcessManager(SyncBaseService, ILocalProcessManager):
                 if not process.is_running():
                     self.log_info(f"Process {deployment.process_id} stopped gracefully")
                     self.state_manager.update_deployment_status(
-                        deployment_id, ProcessStatus.STOPPED
+                        deployment_id, ServiceState.STOPPED
                     )
                     return True
                 time.sleep(0.1)
@@ -286,14 +286,14 @@ class LocalProcessManager(SyncBaseService, ILocalProcessManager):
             )
             self._kill_process_group(process)
             self.state_manager.update_deployment_status(
-                deployment_id, ProcessStatus.STOPPED
+                deployment_id, ServiceState.STOPPED
             )
             return True
 
         except psutil.NoSuchProcess:
             # Process died during shutdown
             self.state_manager.update_deployment_status(
-                deployment_id, ProcessStatus.STOPPED
+                deployment_id, ServiceState.STOPPED
             )
             return True
 
@@ -370,9 +370,9 @@ class LocalProcessManager(SyncBaseService, ILocalProcessManager):
 
             # Determine status
             if process.is_running():
-                status = ProcessStatus.RUNNING
+                status = ServiceState.RUNNING
             else:
-                status = ProcessStatus.STOPPED
+                status = ServiceState.STOPPED
 
             return ProcessInfo(
                 deployment_id=deployment_id,
@@ -389,13 +389,13 @@ class LocalProcessManager(SyncBaseService, ILocalProcessManager):
             return ProcessInfo(
                 deployment_id=deployment_id,
                 process_id=deployment.process_id,
-                status=ProcessStatus.CRASHED,
+                status=ServiceState.ERROR,  # CRASHED semantically maps to ERROR state
                 port=deployment.port,
                 error_message="Process no longer exists",
             )
 
     def list_processes(
-        self, status_filter: Optional[ProcessStatus] = None
+        self, status_filter: Optional[ServiceState] = None
     ) -> List[ProcessInfo]:
         """
         List all managed processes.