claude-mpm 4.1.0__py3-none-any.whl → 4.1.2__py3-none-any.whl

claude_mpm/services/command_handler_service.py

@@ -29,11 +29,9 @@ class CommandHandlerService(BaseService, CommandHandlerInterface):
 
     async def _initialize(self) -> None:
         """Initialize the service. No special initialization needed."""
-        pass
 
     async def _cleanup(self) -> None:
         """Cleanup service resources. No cleanup needed."""
-        pass
 
     def handle_mpm_command(self, prompt: str) -> bool:
         """Handle /mpm: commands directly without going to Claude.
@@ -59,12 +57,11 @@ class CommandHandlerService(BaseService, CommandHandlerInterface):
             # Route to appropriate handler
             if command == "test":
                 return self._handle_test_command(args)
-            elif command == "agents":
+            if command == "agents":
                 return self._handle_agents_command(args)
-            else:
-                print(f"Unknown command: {command}")
-                print("Available commands: test, agents")
-                return True
+            print(f"Unknown command: {command}")
+            print("Available commands: test, agents")
+            return True
 
         except KeyboardInterrupt:
             print("\nCommand interrupted")
@@ -175,28 +172,29 @@ class CommandHandlerService(BaseService, CommandHandlerInterface):
                     "success": success,
                     "command": command,
                     "args": args,
-                    "message": "Test command executed"
-                    if success
-                    else "Test command failed",
+                    "message": (
+                        "Test command executed" if success else "Test command failed"
+                    ),
                 }
-            elif command == "agents":
+            if command == "agents":
                 success = self._handle_agents_command(args)
                 return {
                     "success": success,
                     "command": command,
                     "args": args,
-                    "message": "Agents command executed"
-                    if success
-                    else "Agents command failed",
-                }
-            else:
-                return {
-                    "success": False,
-                    "command": command,
-                    "args": args,
-                    "error": f"Unknown command: {command}",
-                    "available_commands": self.get_available_commands(),
+                    "message": (
+                        "Agents command executed"
+                        if success
+                        else "Agents command failed"
+                    ),
                 }
+            return {
+                "success": False,
+                "command": command,
+                "args": args,
+                "error": f"Unknown command: {command}",
+                "available_commands": self.get_available_commands(),
+            }
         except Exception as e:
             return {"success": False, "command": command, "args": args, "error": str(e)}
 

claude_mpm/services/core/__init__.py

@@ -40,37 +40,37 @@ from .interfaces import (  # Core dependency injection; Configuration management
 )
 
 __all__ = [
-    # Core interfaces
-    "IServiceContainer",
-    "ServiceType",
-    "IConfigurationService",
-    "IConfigurationManager",
-    "IAgentRegistry",
+    # Service interfaces
+    "AgentDeploymentInterface",
     "AgentMetadata",
-    "IHealthMonitor",
+    # Base classes
+    "BaseService",
+    "CacheEntry",
     "HealthStatus",
+    "HookServiceInterface",
+    "IAgentRegistry",
+    "ICacheService",
+    "IConfigurationManager",
+    "IConfigurationService",
+    "IErrorHandler",
+    "IEventBus",
+    "IHealthMonitor",
+    "IPerformanceMonitor",
     "IPromptCache",
-    "CacheEntry",
-    "ITemplateManager",
-    "TemplateRenderContext",
+    # Core interfaces
+    "IServiceContainer",
     "IServiceFactory",
-    "IEventBus",
-    "IStructuredLogger",
     "IServiceLifecycle",
-    "IErrorHandler",
-    "IPerformanceMonitor",
-    "ICacheService",
-    # Service interfaces
-    "AgentDeploymentInterface",
-    "MemoryServiceInterface",
-    "HookServiceInterface",
-    "SocketIOServiceInterface",
-    "ProjectAnalyzerInterface",
-    "TicketManagerInterface",
+    "IStructuredLogger",
+    "ITemplateManager",
     # Registry
     "InterfaceRegistry",
-    # Base classes
-    "BaseService",
-    "SyncBaseService",
+    "MemoryServiceInterface",
+    "ProjectAnalyzerInterface",
+    "ServiceType",
     "SingletonService",
+    "SocketIOServiceInterface",
+    "SyncBaseService",
+    "TemplateRenderContext",
+    "TicketManagerInterface",
 ]

claude_mpm/services/core/base.py

@@ -9,7 +9,6 @@ and lifecycle management.
 Part of TSK-0046: Service Layer Architecture Reorganization
 """
 
-import logging
 from abc import ABC, abstractmethod
 from typing import Any, Dict, Optional
 
@@ -56,7 +55,6 @@ class BaseService(ABC):
         Returns:
             True if initialization successful, False otherwise
         """
-        pass
 
     @abstractmethod
     async def shutdown(self) -> None:
@@ -65,7 +63,6 @@ class BaseService(ABC):
 
         This method should clean up any resources held by the service.
         """
-        pass
 
     def get_config(self, key: str, default: Any = None) -> Any:
         """
@@ -157,7 +154,6 @@ class SyncBaseService(ABC):
         Returns:
             True if initialization successful, False otherwise
         """
-        pass
 
     @abstractmethod
     def shutdown(self) -> None:
@@ -166,7 +162,6 @@ class SyncBaseService(ABC):
 
         This method should clean up any resources held by the service.
         """
-        pass
 
     def get_config(self, key: str, default: Any = None) -> Any:
         """

claude_mpm/services/core/interfaces/__init__.py

@@ -143,46 +143,46 @@ class InterfaceRegistry:
 
 # Re-export everything for backward compatibility
 __all__ = [
-    # Type variables
-    "T",
-    "ServiceType",
-    # Infrastructure interfaces
-    "IServiceContainer",
-    "IConfigurationService",
-    "IConfigurationManager",
-    "ICacheService",
-    "IPromptCache",
+    "AgentCapabilitiesInterface",
+    "AgentDeploymentInterface",
+    "AgentMetadata",
     "CacheEntry",
-    "IHealthMonitor",
+    "CommandHandlerInterface",
     "HealthStatus",
-    "ITemplateManager",
-    "TemplateRenderContext",
-    "IServiceFactory",
-    "IStructuredLogger",
-    "IServiceLifecycle",
-    "IErrorHandler",
-    "IPerformanceMonitor",
-    "IEventBus",
+    "HookServiceInterface",
     # Agent interfaces
     "IAgentRegistry",
-    "AgentMetadata",
-    "AgentDeploymentInterface",
-    "AgentCapabilitiesInterface",
-    "SystemInstructionsInterface",
-    "SubprocessLauncherInterface",
-    "RunnerConfigurationInterface",
-    # Service interfaces
-    "VersionServiceInterface",
-    "CommandHandlerInterface",
+    "ICacheService",
+    "IConfigurationManager",
+    "IConfigurationService",
+    "IErrorHandler",
+    "IEventBus",
+    "IHealthMonitor",
+    "IPerformanceMonitor",
+    "IPromptCache",
+    # Infrastructure interfaces
+    "IServiceContainer",
+    "IServiceFactory",
+    "IServiceLifecycle",
+    "IStructuredLogger",
+    "ITemplateManager",
+    # Registry
+    "InterfaceRegistry",
     "MemoryHookInterface",
     "MemoryServiceInterface",
+    "ProjectAnalyzerInterface",
+    "RunnerConfigurationInterface",
+    "ServiceType",
     "SessionManagementInterface",
-    "UtilityServiceInterface",
-    "HookServiceInterface",
     # Communication interfaces
     "SocketIOServiceInterface",
-    "ProjectAnalyzerInterface",
+    "SubprocessLauncherInterface",
+    "SystemInstructionsInterface",
+    # Type variables
+    "T",
+    "TemplateRenderContext",
     "TicketManagerInterface",
-    # Registry
-    "InterfaceRegistry",
+    "UtilityServiceInterface",
+    # Service interfaces
+    "VersionServiceInterface",
 ]

claude_mpm/services/core/interfaces/agent.py

@@ -68,29 +68,24 @@ class IAgentRegistry(ABC):
         self, force_refresh: bool = False
     ) -> Dict[str, AgentMetadata]:
         """Discover all available agents"""
-        pass
 
     @abstractmethod
     async def get_agent(self, agent_name: str) -> Optional[AgentMetadata]:
         """Get specific agent metadata"""
-        pass
 
     @abstractmethod
     async def list_agents(
         self, agent_type: Optional[str] = None, tier: Optional[str] = None
     ) -> List[AgentMetadata]:
         """List agents with optional filtering"""
-        pass
 
     @abstractmethod
     async def get_specialized_agents(self, agent_type: str) -> List[AgentMetadata]:
         """Get agents of a specific specialized type"""
-        pass
 
     @abstractmethod
     async def refresh_agent_cache(self) -> None:
         """Refresh the agent metadata cache"""
-        pass
 
 
 # Agent deployment interface
@@ -118,7 +113,6 @@ class AgentDeploymentInterface(ABC):
         Returns:
             Dictionary with deployment results and status
         """
-        pass
 
     @abstractmethod
     def validate_agent(self, agent_path: Path) -> Tuple[bool, List[str]]:
@@ -130,7 +124,6 @@ class AgentDeploymentInterface(ABC):
         Returns:
             Tuple of (is_valid, list_of_errors)
         """
-        pass
 
     @abstractmethod
     def get_deployment_status(self, agent_name: str) -> Dict[str, Any]:
@@ -142,7 +135,6 @@ class AgentDeploymentInterface(ABC):
         Returns:
             Dictionary with deployment status information
         """
-        pass
 
 
 # Agent capabilities interface
@@ -168,7 +160,6 @@ class AgentCapabilitiesInterface(ABC):
         Returns:
             Formatted capabilities string for Claude consumption
         """
-        pass
 
 
 # System instructions interface
@@ -194,7 +185,6 @@ class SystemInstructionsInterface(ABC):
         Returns:
             Processed system instructions string
         """
-        pass
 
     @abstractmethod
     def get_available_instruction_types(self) -> List[str]:
@@ -203,7 +193,6 @@ class SystemInstructionsInterface(ABC):
         Returns:
             List of available instruction type names
         """
-        pass
 
     @abstractmethod
     def validate_instructions(self, instructions: str) -> Tuple[bool, List[str]]:
@@ -215,7 +204,6 @@ class SystemInstructionsInterface(ABC):
         Returns:
             Tuple of (is_valid, list_of_errors)
         """
-        pass
 
 
 # Subprocess launcher interface
@@ -241,7 +229,6 @@ class SubprocessLauncherInterface(ABC):
         Returns:
             Dictionary with subprocess information and handles
         """
-        pass
 
     @abstractmethod
     async def launch_subprocess_async(
@@ -256,7 +243,6 @@ class SubprocessLauncherInterface(ABC):
         Returns:
             Dictionary with subprocess information and handles
         """
-        pass
 
     @abstractmethod
     def terminate_subprocess(self, process_id: str) -> bool:
@@ -268,7 +254,6 @@ class SubprocessLauncherInterface(ABC):
         Returns:
             True if termination successful
         """
-        pass
 
     @abstractmethod
     def get_subprocess_status(self, process_id: str) -> Dict[str, Any]:
@@ -280,7 +265,6 @@ class SubprocessLauncherInterface(ABC):
         Returns:
             Dictionary with process status information
         """
-        pass
 
 
 # Runner configuration interface
@@ -306,7 +290,6 @@ class RunnerConfigurationInterface(ABC):
         Returns:
             Dictionary with initialization results
         """
-        pass
 
     @abstractmethod
     def register_services(self, service_container) -> None:
@@ -315,7 +298,6 @@ class RunnerConfigurationInterface(ABC):
         Args:
             service_container: Service container for registration
         """
-        pass
 
     @abstractmethod
     def load_configuration(self, config_path: Optional[Path] = None) -> Dict[str, Any]:
@@ -327,7 +309,6 @@ class RunnerConfigurationInterface(ABC):
         Returns:
             Loaded configuration dictionary
         """
-        pass
 
     @abstractmethod
     def validate_configuration(self, config: Dict[str, Any]) -> Tuple[bool, List[str]]:
@@ -339,7 +320,6 @@ class RunnerConfigurationInterface(ABC):
         Returns:
             Tuple of (is_valid, list_of_errors)
         """
-        pass
 
     @abstractmethod
     def setup_logging(self, config: Dict[str, Any]) -> None:
@@ -348,4 +328,3 @@ class RunnerConfigurationInterface(ABC):
         Args:
             config: Logging configuration
         """
-        pass

claude_mpm/services/core/interfaces/communication.py

@@ -36,12 +36,10 @@ class SocketIOServiceInterface(ABC):
     @abstractmethod
     def start_sync(self) -> None:
         """Start the WebSocket server synchronously."""
-        pass
 
     @abstractmethod
     def stop_sync(self) -> None:
         """Stop the WebSocket server synchronously."""
-        pass
 
     @abstractmethod
     def broadcast_event(self, event_type: str, data: Dict[str, Any]) -> None:
@@ -51,7 +49,6 @@ class SocketIOServiceInterface(ABC):
             event_type: Type of event to broadcast
             data: Event data to send
         """
-        pass
 
     @abstractmethod
     def send_to_client(
@@ -67,7 +64,6 @@ class SocketIOServiceInterface(ABC):
         Returns:
             True if message sent successfully
         """
-        pass
 
     @abstractmethod
     def get_connection_count(self) -> int:
@@ -76,7 +72,6 @@ class SocketIOServiceInterface(ABC):
         Returns:
             Number of connected clients
         """
-        pass
 
     @abstractmethod
     def is_running(self) -> bool:
@@ -85,7 +80,6 @@ class SocketIOServiceInterface(ABC):
         Returns:
             True if server is active
         """
-        pass
 
     @abstractmethod
     def session_started(
@@ -98,12 +92,10 @@ class SocketIOServiceInterface(ABC):
             launch_method: Method used to launch the session
             working_dir: Working directory of the session
         """
-        pass
 
     @abstractmethod
     def session_ended(self) -> None:
         """Notify that a session has ended."""
-        pass
 
     @abstractmethod
     def claude_status_changed(
@@ -116,7 +108,6 @@ class SocketIOServiceInterface(ABC):
             pid: Process ID if applicable
             message: Optional status message
         """
-        pass
 
     @abstractmethod
     def agent_delegated(self, agent: str, task: str, status: str = "started") -> None:
@@ -127,7 +118,6 @@ class SocketIOServiceInterface(ABC):
             task: Task assigned to the agent
             status: Status of the delegation
         """
-        pass
 
     @abstractmethod
     def todo_updated(self, todos: List[Dict[str, Any]]) -> None:
@@ -136,7 +126,6 @@ class SocketIOServiceInterface(ABC):
         Args:
             todos: Updated list of todo items
         """
-        pass
 
 
 # Project analyzer interface
@@ -161,7 +150,6 @@ class ProjectAnalyzerInterface(ABC):
         Returns:
             Dictionary with project analysis results
         """
-        pass
 
     @abstractmethod
     def detect_technology_stack(self) -> List[str]:
@@ -170,7 +158,6 @@ class ProjectAnalyzerInterface(ABC):
         Returns:
             List of detected technologies
         """
-        pass
 
     @abstractmethod
     def analyze_code_patterns(self) -> Dict[str, Any]:
@@ -179,7 +166,6 @@ class ProjectAnalyzerInterface(ABC):
         Returns:
             Dictionary of pattern analysis results
         """
-        pass
 
     @abstractmethod
     def get_project_structure(self) -> Dict[str, Any]:
@@ -188,7 +174,6 @@ class ProjectAnalyzerInterface(ABC):
         Returns:
             Dictionary representing project structure
         """
-        pass
 
     @abstractmethod
     def identify_entry_points(self) -> List[Path]:
@@ -197,7 +182,6 @@ class ProjectAnalyzerInterface(ABC):
         Returns:
             List of entry point paths
         """
-        pass
 
     @abstractmethod
     def get_dependencies(self) -> Dict[str, List[str]]:
@@ -206,7 +190,6 @@ class ProjectAnalyzerInterface(ABC):
         Returns:
             Dictionary mapping dependency types to lists of dependencies
         """
-        pass
 
     @abstractmethod
     def analyze_test_coverage(self) -> Dict[str, Any]:
@@ -215,7 +198,6 @@ class ProjectAnalyzerInterface(ABC):
         Returns:
             Dictionary with test coverage information
         """
-        pass
 
     @abstractmethod
     def get_build_configuration(self) -> Dict[str, Any]:
@@ -224,7 +206,6 @@ class ProjectAnalyzerInterface(ABC):
         Returns:
             Dictionary with build configuration details
         """
-        pass
 
 
 # Ticket manager interface
@@ -253,7 +234,6 @@ class TicketManagerInterface(ABC):
         Returns:
             Ticket ID
         """
-        pass
 
     @abstractmethod
     def get_ticket(self, ticket_id: str) -> Optional[Dict[str, Any]]:
@@ -265,7 +245,6 @@ class TicketManagerInterface(ABC):
         Returns:
             Ticket data or None if not found
         """
-        pass
 
     @abstractmethod
     def update_ticket(self, ticket_id: str, updates: Dict[str, Any]) -> bool:
@@ -278,7 +257,6 @@ class TicketManagerInterface(ABC):
         Returns:
             True if update successful
         """
-        pass
 
     @abstractmethod
     def list_tickets(
@@ -293,7 +271,6 @@ class TicketManagerInterface(ABC):
         Returns:
             List of ticket data
         """
-        pass
 
     @abstractmethod
     def assign_ticket(self, ticket_id: str, assignee: str) -> bool:
@@ -306,7 +283,6 @@ class TicketManagerInterface(ABC):
         Returns:
             True if assignment successful
         """
-        pass
 
     @abstractmethod
     def close_ticket(self, ticket_id: str, resolution: Optional[str] = None) -> bool:
@@ -319,7 +295,6 @@ class TicketManagerInterface(ABC):
         Returns:
             True if close successful
         """
-        pass
 
     @abstractmethod
     def search_tickets(self, query: str) -> List[Dict[str, Any]]:
@@ -331,7 +306,6 @@ class TicketManagerInterface(ABC):
         Returns:
             List of matching ticket data
         """
-        pass
 
     @abstractmethod
     def get_ticket_statistics(self) -> Dict[str, Any]:
@@ -340,4 +314,3 @@ class TicketManagerInterface(ABC):
         Returns:
             Dictionary with ticket statistics
         """
-        pass