claude-mpm 4.5.8__py3-none-any.whl → 4.5.12__py3-none-any.whl

claude_mpm/cli/commands/dashboard.py

@@ -65,7 +65,7 @@ class DashboardCommand(BaseCommand):
             self.logger.error(f"Error executing dashboard command: {e}", exc_info=True)
             return CommandResult.error_result(f"Error executing dashboard command: {e}")
 
-    def _start_dashboard(self, args) -> CommandResult:  # noqa: PLR0911
+    def _start_dashboard(self, args) -> CommandResult:
         """Start the dashboard server."""
         port = getattr(args, "port", 8765)
         host = getattr(args, "host", "localhost")

claude_mpm/cli/commands/debug.py

@@ -21,7 +21,7 @@ from typing import Any, Dict
 from ...core.logger import get_logger
 
 
-def manage_debug(args):  # noqa: PLR0911
+def manage_debug(args):
     """
     Main entry point for debug commands.
 
@@ -446,7 +446,7 @@ def debug_agents(args, logger):
 
                         # Try to load and analyze memory
                         try:
-                            with open(mem_file) as f:
+                            with mem_file.open() as f:
                                 memory_data = json.load(f)
 
                             entry_count = (
@@ -509,7 +509,7 @@ def debug_agents(args, logger):
                     print(f"   Size: {agent_file.stat().st_size:,} bytes")
 
                     # Read first few lines for type detection
-                    with open(agent_file) as f:
+                    with agent_file.open() as f:
                         lines = f.readlines()[:10]
                         for line in lines:
                             if "role:" in line.lower():

claude_mpm/cli/commands/doctor.py

@@ -181,7 +181,7 @@ def doctor_command(args):
             import sys
 
             original_stdout = sys.stdout
-            with open(output_file, "w") as f:
+            with output_file.open("w") as f:
                 sys.stdout = f
                 reporter.report(summary, format=output_format)
             sys.stdout = original_stdout

claude_mpm/cli/commands/mcp.py

@@ -61,10 +61,10 @@ def manage_mcp(args):
 
             # Allow install command to proceed
             if args.mcp_command == MCPCommands.INSTALL.value:
-                MCPConfiguration = None  # noqa: N806
-                MCPServiceRegistry = None  # noqa: N806
-                ToolRegistry = None  # noqa: N806
-                MCPGateway = None  # noqa: N806
+                MCPConfiguration = None
+                MCPServiceRegistry = None
+                ToolRegistry = None
+                MCPGateway = None
             else:
                 print(
                     "\nError: MCP Gateway services not fully available",
@@ -120,7 +120,7 @@ def _show_status(
     MCPConfiguration,
     MCPServiceRegistry,
     ToolRegistry,
-    MCPGateway,  # noqa: N803
+    MCPGateway,
 ):
     """
     Show MCP Gateway status when no subcommand is provided.
@@ -151,7 +151,7 @@ def _show_status(
     if config_path.exists():
         print(f"   Config file: {config_path}")
         try:
-            with open(config_path) as f:
+            with config_path.open() as f:
                 config = json.load(f)
                 if "servers" in config:
                     print(f"   Configured servers: {len(config.get('servers', {}))}")
@@ -171,7 +171,7 @@ def _show_status(
     if claude_config.exists():
         print(f"\n🖥️  Claude Code Config: {claude_config}")
         try:
-            with open(claude_config) as f:
+            with claude_config.open() as f:
                 config = json.load(f)
                 mcp_servers = config.get("mcpServers", {})
                 if "claude-mpm" in mcp_servers:

claude_mpm/cli/commands/mcp_command_router.py

@@ -16,7 +16,7 @@ class MCPCommandRouter:
         """Initialize the command router."""
         self.logger = logger
 
-    def route_command(self, args) -> int:  # noqa: PLR0911
+    def route_command(self, args) -> int:
         """Route command to appropriate handler."""
         if args.mcp_command == MCPCommands.START.value:
             return asyncio.run(self._start_server(args))

claude_mpm/cli/commands/mcp_config.py

@@ -62,7 +62,7 @@ class MCPConfigCommand(BaseCommand):
             # Show the updated configuration
             config_path = Path.cwd() / ".mcp.json"
             if config_path.exists():
-                with open(config_path) as f:
+                with config_path.open() as f:
                     config = json.load(f)
                 return CommandResult(
                     success=True,
@@ -120,7 +120,7 @@ class MCPConfigCommand(BaseCommand):
         current_config = {}
         if config_path.exists():
             try:
-                with open(config_path) as f:
+                with config_path.open() as f:
                     current_config = json.load(f)
             except Exception:
                 pass

claude_mpm/cli/commands/mcp_external_commands.py

@@ -12,7 +12,7 @@ class MCPExternalCommands:
         """Initialize the MCP external commands handler."""
         self.logger = logger
 
-    def manage_external(self, args):  # noqa: PLR0911
+    def manage_external(self, args):
         """Manage external MCP services.
 
         Args:
@@ -135,7 +135,7 @@ class MCPExternalCommands:
                 print(f"\n📄 Found config: {config_path}")
 
                 try:
-                    with open(config_path) as f:
+                    with config_path.open() as f:
                         config = json.load(f)
 
                     mcp_servers = config.get("mcpServers", {})

claude_mpm/cli/commands/mcp_install_commands.py

@@ -270,7 +270,7 @@ class MCPInstallCommands:
             if not force:
                 # Check if claude-mpm-gateway already exists
                 try:
-                    with open(config_path) as f:
+                    with config_path.open() as f:
                         existing_config = json.load(f)
 
                     if (
@@ -305,7 +305,7 @@ class MCPInstallCommands:
             else:
                 # Force mode - create backup but proceed
                 try:
-                    with open(config_path) as f:
+                    with config_path.open() as f:
                         existing_config = json.load(f)
                     config = existing_config
                     print("   Force mode: Overwriting existing configuration")
@@ -335,7 +335,7 @@ class MCPInstallCommands:
             config_path.parent.mkdir(parents=True, exist_ok=True)
 
             # Write configuration with nice formatting
-            with open(config_path, "w") as f:
+            with config_path.open("w") as f:
                 json.dump(config, f, indent=2)
 
             print(f"\n✅ Configuration saved to {config_path}")

claude_mpm/cli/commands/mcp_pipx_config.py

@@ -110,7 +110,7 @@ def configure_mcp_for_pipx(args) -> int:
     existing_config = {}
     if config_path.exists():
         try:
-            with open(config_path) as f:
+            with config_path.open() as f:
                 existing_config = json.load(f)
             print("✅ Existing config loaded")
         except json.JSONDecodeError:
@@ -145,7 +145,7 @@ def configure_mcp_for_pipx(args) -> int:
     # Write config
     if not args.dry_run:
         try:
-            with open(config_path, "w") as f:
+            with config_path.open("w") as f:
                 json.dump(existing_config, f, indent=2)
             print(f"\n✅ Configuration written to: {config_path}")
         except Exception as e:

claude_mpm/cli/commands/mcp_setup_external.py

@@ -621,7 +621,7 @@ class MCPExternalServicesSetup:
         """
         try:
             if config_path.exists():
-                with open(config_path) as f:
+                with config_path.open() as f:
                     config = json.load(f)
                     # Ensure mcpServers key exists
                     if "mcpServers" not in config:
@@ -664,7 +664,7 @@ class MCPExternalServicesSetup:
                 print(f"   📁 Created backup: {backup_path}")
 
             # Write configuration with proper formatting
-            with open(config_path, "w") as f:
+            with config_path.open("w") as f:
                 json.dump(config, f, indent=2)
                 f.write("\n")  # Add newline at end of file
 
@@ -830,7 +830,7 @@ class MCPExternalServicesSetup:
 
         if mcp_config_path.exists():
             try:
-                with open(mcp_config_path) as f:
+                with mcp_config_path.open() as f:
                     mcp_config = json.load(f)
                 print(f"\n📁 Project MCP config: {mcp_config_path}")
             except Exception:

claude_mpm/cli/commands/monitor.py

@@ -37,7 +37,7 @@ class MonitorCommand(BaseCommand):
 
         return None
 
-    def run(self, args) -> CommandResult:  # noqa: PLR0911
+    def run(self, args) -> CommandResult:
         """Execute the monitor command using unified monitoring daemon."""
         try:
             self.logger.info("Monitor command using unified monitoring daemon")

claude_mpm/cli/commands/mpm_init_handler.py

@@ -11,7 +11,7 @@ from rich.console import Console
 console = Console()
 
 
-def manage_mpm_init(args):  # noqa: PLR0911
+def manage_mpm_init(args):
     """
     Handle mpm-init command execution.
 

claude_mpm/cli/interactive/agent_wizard.py

@@ -769,7 +769,7 @@ class AgentWizard:
 
         output_file = output_dir / f"{template.agent_id}.json"
 
-        with open(output_file, "w") as f:
+        with output_file.open("w") as f:
             json.dump(template.to_json(), f, indent=2)
 
         return True, f"Agent exported to {output_file}"

claude_mpm/cli/parsers/configure_parser.py

@@ -58,6 +58,11 @@ def add_configure_subparser(subparsers) -> argparse.ArgumentParser:
         action="store_true",
         help="Jump directly to behavior file management",
     )
+    nav_group.add_argument(
+        "--startup",
+        action="store_true",
+        help="Configure startup services and agents",
+    )
     nav_group.add_argument(
         "--version-info",
         action="store_true",

claude_mpm/cli/parsers/search_parser.py

@@ -190,7 +190,7 @@ Examples:
     return search_parser
 
 
-def validate_search_args(args: argparse.Namespace) -> Optional[str]:  # noqa: PLR0911
+def validate_search_args(args: argparse.Namespace) -> Optional[str]:
     """
     Validate search command arguments.
 

claude_mpm/cli/shared/argument_patterns.py

@@ -7,14 +7,14 @@ across multiple CLI commands.
 
 import argparse
 from pathlib import Path
-from typing import Any, Dict, List, Optional
+from typing import Any, ClassVar, Dict, List, Optional
 
 
 class CommonArguments:
     """Registry of common argument patterns used across CLI commands."""
 
     # Logging arguments
-    VERBOSE = {
+    VERBOSE: ClassVar[Dict[str, Any]] = {
         "flags": ["-v", "--verbose"],
         "action": "store_true",
         "help": "Enable verbose output",

claude_mpm/cli/shared/base_command.py

@@ -183,7 +183,7 @@ class BaseCommand(ABC):
 
         if hasattr(args, "output") and args.output:
             # Write to file
-            with open(args.output, "w") as f:
+            with args.output.open("w") as f:
                 f.write(formatted_output)
             self.logger.info(f"Output written to {args.output}")
         else:

claude_mpm/cli/startup_logging.py

@@ -303,7 +303,7 @@ class StartupStatusLogger:
 
             import json
 
-            with open(claude_json_path) as f:
+            with claude_json_path.open() as f:
                 config = json.load(f)
 
             result["found"] = True
@@ -716,9 +716,9 @@ def start_vector_search_indexing(project_root: Optional[Path] = None) -> None:
     try:
         # Try to get the current event loop
         loop = asyncio.get_running_loop()
-        # If we're in an event loop, create a task
-        # Store reference to avoid RUF006 warning
-        _ = loop.create_task(trigger_vector_search_indexing(project_root))
+        # If we're in an event loop, create a task (fire-and-forget)
+        _task = loop.create_task(trigger_vector_search_indexing(project_root))  # noqa: RUF006
+        # Fire-and-forget: task will complete in background
     except RuntimeError:
         # No event loop running - use subprocess directly to avoid event loop lifecycle issues
         # The async approach with asyncio.run() creates and closes a loop which causes

claude_mpm/config/experimental_features.py

@@ -64,7 +64,7 @@ class ExperimentalFeatures:
         """
         if self._config_file and self._config_file.exists():
             try:
-                with open(self._config_file) as f:
+                with self._config_file.open() as f:
                     config = json.load(f)
                     experimental = config.get("experimental_features", {})
                     self._features.update(experimental)
@@ -125,7 +125,7 @@ class ExperimentalFeatures:
         accepted_file = Path.home() / ".claude-mpm" / ".experimental_accepted"
         if accepted_file.exists():
             try:
-                with open(accepted_file) as f:
+                with accepted_file.open() as f:
                     accepted = json.load(f)
                     if feature in accepted.get("features", []):
                         return False
@@ -148,7 +148,7 @@ class ExperimentalFeatures:
 
         try:
             if accepted_file.exists():
-                with open(accepted_file) as f:
+                with accepted_file.open() as f:
                     data = json.load(f)
             else:
                 data = {"features": [], "timestamp": {}}
@@ -159,7 +159,7 @@ class ExperimentalFeatures:
                     "CLAUDE_MPM_TIMESTAMP", str(Path.cwd())
                 )
 
-            with open(accepted_file, "w") as f:
+            with accepted_file.open("w") as f:
                 json.dump(data, f, indent=2)
         except Exception:
             # Silently ignore errors in acceptance tracking

claude_mpm/config/socketio_config.py

@@ -249,7 +249,7 @@ class ConfigManager:
         for config_path in self.config_search_paths:
             if config_path.exists():
                 try:
-                    with open(config_path) as f:
+                    with config_path.open() as f:
                         return json.load(f)
                 except Exception as e:
                     print(f"Warning: Failed to load config from {config_path}: {e}")
@@ -267,7 +267,7 @@ class ConfigManager:
             path = config_dir / self.config_file_name
 
         try:
-            with open(path, "w") as f:
+            with path.open("w") as f:
                 json.dump(config.to_dict(), f, indent=2)
             return True
         except Exception as e:

claude_mpm/core/__init__.py

@@ -1,20 +1,56 @@
-"""Core components for Claude MPM."""
-
-from .claude_runner import ClaudeRunner
-from .config import Config
-
-# Import DI components
-from .container import DIContainer, ServiceLifetime, get_container
-from .factories import (
-    AgentServiceFactory,
-    ConfigurationFactory,
-    ServiceFactory,
-    SessionManagerFactory,
-    get_factory_registry,
-)
-from .injectable_service import InjectableService
-from .mixins import LoggerMixin
-from .service_registry import ServiceRegistry, get_service_registry, initialize_services
+"""Core components for Claude MPM.
+
+Lazy imports for all components to avoid loading heavy dependencies
+when only importing lightweight utilities (constants, logging_utils, etc).
+"""
+
+
+# Lazy imports via __getattr__ to prevent loading heavy dependencies
+# when hooks only need lightweight utilities
+def __getattr__(name):
+    """Lazy load core components only when accessed using dictionary-based mapping."""
+    from importlib import import_module
+
+    # Dictionary mapping: name -> (module_path, attribute_name)
+    _LAZY_IMPORTS = {
+        # Core components
+        "ClaudeRunner": ("claude_mpm.core.claude_runner", "ClaudeRunner"),
+        "Config": ("claude_mpm.core.config", "Config"),
+        # Dependency injection
+        "DIContainer": ("claude_mpm.core.container", "DIContainer"),
+        "ServiceLifetime": ("claude_mpm.core.container", "ServiceLifetime"),
+        "get_container": ("claude_mpm.core.container", "get_container"),
+        # Factories
+        "AgentServiceFactory": ("claude_mpm.core.factories", "AgentServiceFactory"),
+        "ConfigurationFactory": ("claude_mpm.core.factories", "ConfigurationFactory"),
+        "ServiceFactory": ("claude_mpm.core.factories", "ServiceFactory"),
+        "SessionManagerFactory": ("claude_mpm.core.factories", "SessionManagerFactory"),
+        "get_factory_registry": ("claude_mpm.core.factories", "get_factory_registry"),
+        # Services and utilities
+        "InjectableService": (
+            "claude_mpm.core.injectable_service",
+            "InjectableService",
+        ),
+        "LoggerMixin": ("claude_mpm.core.mixins", "LoggerMixin"),
+        # Service registry
+        "ServiceRegistry": ("claude_mpm.core.service_registry", "ServiceRegistry"),
+        "get_service_registry": (
+            "claude_mpm.core.service_registry",
+            "get_service_registry",
+        ),
+        "initialize_services": (
+            "claude_mpm.core.service_registry",
+            "initialize_services",
+        ),
+    }
+
+    if name in _LAZY_IMPORTS:
+        module_path, attr_name = _LAZY_IMPORTS[name]
+        module = import_module(module_path)
+        return getattr(module, attr_name)
+
+    raise AttributeError(f"module '{__name__}' has no attribute '{name}'")
+
 
 __all__ = [
     "AgentServiceFactory",

claude_mpm/core/agent_session_manager.py

@@ -209,7 +209,7 @@ class AgentSessionManager:
                 "agent_sessions": dict(self.agent_sessions),
                 "updated_at": datetime.now(timezone.utc).isoformat(),
             }
-            with open(session_file, "w") as f:
+            with session_file.open("w") as f:
                 json.dump(data, f, indent=2)
         except Exception as e:
             logger.error(f"Failed to save agent sessions: {e}")
@@ -219,7 +219,7 @@ class AgentSessionManager:
         session_file = self.session_dir / "agent_sessions.json"
         if session_file.exists():
             try:
-                with open(session_file) as f:
+                with session_file.open() as f:
                     data = json.load(f)
                     self.agent_sessions = defaultdict(
                         dict, data.get("agent_sessions", {})

claude_mpm/core/api_validator.py

@@ -84,7 +84,7 @@ class APIKeyValidator:
 
         return not bool(self.errors), self.errors, self.warnings
 
-    def _validate_openai_key(self, api_key: str) -> bool:  # noqa: PLR0911
+    def _validate_openai_key(self, api_key: str) -> bool:
         """Validate OpenAI API key.
 
         Args:
@@ -133,7 +133,7 @@ class APIKeyValidator:
             self.errors.append(f"❌ OpenAI API validation failed with error: {e}")
             return False
 
-    def _validate_anthropic_key(self, api_key: str) -> bool:  # noqa: PLR0911
+    def _validate_anthropic_key(self, api_key: str) -> bool:
         """Validate Anthropic API key.
 
         Args:
@@ -196,7 +196,7 @@ class APIKeyValidator:
             self.errors.append(f"❌ Anthropic API validation failed with error: {e}")
             return False
 
-    def _validate_github_token(self, token: str) -> bool:  # noqa: PLR0911
+    def _validate_github_token(self, token: str) -> bool:
         """Validate GitHub personal access token.
 
         Args:

claude_mpm/core/base_service.py

@@ -428,7 +428,16 @@ class BaseService(LoggerMixin, ABC):
             self.logger.info(
                 f"Received signal {signum}, initiating graceful shutdown..."
             )
-            asyncio.create_task(self.stop())
+            # Get the event loop and create a tracked shutdown task
+            try:
+                loop = asyncio.get_event_loop()
+                task = loop.create_task(self.stop())
+                # Store reference to prevent GC during shutdown
+                if not hasattr(self, '_shutdown_task'):
+                    self._shutdown_task = task
+            except RuntimeError:
+                # No event loop, call stop synchronously
+                self.logger.warning("No event loop available for graceful shutdown")
 
         signal.signal(signal.SIGINT, signal_handler)
         signal.signal(signal.SIGTERM, signal_handler)

claude_mpm/core/cache.py

@@ -391,7 +391,7 @@ class FileSystemCache:
 
         try:
             self.persist_path.parent.mkdir(parents=True, exist_ok=True)
-            with open(self.persist_path, "wb") as f:
+            with self.persist_path.open("wb") as f:
                 pickle.dump(self._cache, f)
             self._logger.debug(f"Cache persisted to {self.persist_path}")
         except Exception as e:
@@ -403,7 +403,7 @@ class FileSystemCache:
             return
 
         try:
-            with open(self.persist_path, "rb") as f:
+            with self.persist_path.open("rb") as f:
                 loaded_cache = pickle.load(f)
 
             # Rebuild cache with validation

claude_mpm/core/config.py

@@ -201,7 +201,7 @@ class Config:
                         # Set flag IMMEDIATELY before logging to prevent any possibility of duplicate
                         # messages. No lock needed here since we're already inside __init__ lock
                         Config._success_logged = True
-                        logger.info(
+                        logger.debug(
                             f"✓ Successfully loaded configuration from {file_path}"
                         )
                     else:
@@ -253,7 +253,7 @@ class Config:
                     "operation": "read",
                     "error_type": type(e).__name__,
                 },
-            )
+            ) from e
         except Exception as e:
             # Catch any remaining unexpected errors and wrap them as configuration errors
             raise ConfigurationError(
@@ -263,7 +263,7 @@ class Config:
                     "error_type": type(e).__name__,
                     "original_error": str(e),
                 },
-            )
+            ) from e
 
     def _load_env_vars(self) -> None:
         """Load configuration from environment variables."""
@@ -651,7 +651,7 @@ class Config:
                     "format": format,
                     "error_type": type(e).__name__,
                 },
-            )
+            ) from e
         except Exception as e:
             # Re-raise ConfigurationError as-is, wrap others
             if isinstance(e, ConfigurationError):
@@ -663,7 +663,7 @@ class Config:
                     "format": format,
                     "error_type": type(e).__name__,
                 },
-            )
+            ) from e
 
     def validate(self, schema: Dict[str, Any]) -> bool:
         """

claude_mpm/core/config_aliases.py

@@ -88,7 +88,7 @@ class ConfigAliasManager:
             self.config_mgr.save_json(aliases, self.aliases_file, sort_keys=True)
         except Exception as e:
             logger.error(f"Failed to save aliases: {e}")
-            raise ConfigAliasError(f"Failed to save aliases: {e}")
+            raise ConfigAliasError(f"Failed to save aliases: {e}") from e
 
     def create_alias(self, alias_name: str, directory_path: str) -> None:
         """
@@ -211,7 +211,7 @@ class ConfigAliasManager:
                 except Exception as e:
                     raise InvalidDirectoryError(
                         f"Cannot create directory '{directory_path}': {e}"
-                    )
+                    ) from e
 
             # Verify we can write to the directory
             test_file = directory_path / ".claude_pm_test"
@@ -221,14 +221,14 @@ class ConfigAliasManager:
             except Exception as e:
                 raise InvalidDirectoryError(
                     f"Directory '{directory_path}' is not writable: {e}"
-                )
+                ) from e
 
             return directory_path
 
         except InvalidDirectoryError:
             raise
         except Exception as e:
-            raise InvalidDirectoryError(f"Invalid directory path '{path}': {e}")
+            raise InvalidDirectoryError(f"Invalid directory path '{path}': {e}") from e
 
     def get_alias(self, alias_name: str) -> Optional[str]:
         """

claude_mpm/core/config_constants.py

@@ -99,7 +99,7 @@ class ConfigConstants:
         cls._config_service = config_service
 
     @classmethod
-    def get_timeout(cls, timeout_type: str) -> int:  # noqa: PLR0911
+    def get_timeout(cls, timeout_type: str) -> int:
         """
         Get timeout value by type.
 

claude_mpm/core/error_handler.py

@@ -218,7 +218,7 @@ class ErrorHandler:
                 return handler(error, error_context)
             except Exception as recovery_error:
                 self.logger.error(f"Recovery failed: {recovery_error}")
-                raise error
+                raise error from recovery_error
 
         # No recovery handler available
         self.logger.warning(f"No recovery handler for {error_type.__name__}")