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

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/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

@@ -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__}")

claude_mpm/core/file_utils.py

@@ -80,8 +80,8 @@ def safe_path_join(*parts: Union[str, Path]) -> Path:
     # Ensure the resolved path is under the base path
     try:
         resolved.relative_to(base)
-    except ValueError:
-        raise ValueError(f"Path traversal detected: {path}")
+    except ValueError as e:
+        raise ValueError(f"Path traversal detected: {path}") from e
 
     return resolved
 
@@ -142,7 +142,7 @@ def safe_read(
     """Safely read a file with error handling.
 
     Replaces the common pattern:
-        with open(file, 'r') as f:
+        with file.open('r') as f:
             content = f.read()
 
     Args:
@@ -641,7 +641,7 @@ def file_lock(filepath: Union[str, Path], timeout: float = 5.0):
                 if e.errno != errno.EAGAIN:
                     raise
                 if time.time() - start_time > timeout:
-                    raise TimeoutError(f"Could not acquire lock for {filepath}")
+                    raise TimeoutError(f"Could not acquire lock for {filepath}") from e
                 time.sleep(0.1)
 
         yield lock_handle
@@ -724,7 +724,7 @@ def get_file_hash(
 
     try:
         hasher = hashlib.new(algorithm)
-        with open(filepath, "rb") as f:
+        with filepath.open("rb") as f:
             for chunk in iter(lambda: f.read(8192), b""):
                 hasher.update(chunk)
         return hasher.hexdigest()

claude_mpm/core/framework/formatters/capability_generator.py

@@ -154,7 +154,7 @@ class CapabilityGenerator:
             Dictionary with agent metadata or None
         """
         try:
-            with open(agent_file) as f:
+            with agent_file.open() as f:
                 content = f.read()
 
             # Default values
@@ -244,7 +244,7 @@ class CapabilityGenerator:
             template_file = templates_dir / f"{agent_name}.json"
 
             if template_file.exists():
-                with open(template_file) as f:
+                with template_file.open() as f:
                     template_data = json.load(f)
                     return template_data.get("routing")
 
@@ -262,7 +262,7 @@ class CapabilityGenerator:
                 if alt_name != agent_name:
                     alt_file = templates_dir / f"{alt_name}.json"
                     if alt_file.exists():
-                        with open(alt_file) as f:
+                        with alt_file.open() as f:
                             template_data = json.load(f)
                             return template_data.get("routing")
 
@@ -310,7 +310,7 @@ class CapabilityGenerator:
             template_file = templates_dir / f"{agent_name}.json"
 
             if template_file.exists():
-                with open(template_file) as f:
+                with template_file.open() as f:
                     template_data = json.load(f)
                     return template_data.get("memory_routing")
 
@@ -330,7 +330,7 @@ class CapabilityGenerator:
                 if alt_name != agent_name:
                     alt_file = templates_dir / f"{alt_name}.json"
                     if alt_file.exists():
-                        with open(alt_file) as f:
+                        with alt_file.open() as f:
                             template_data = json.load(f)
                             return template_data.get("memory_routing")
 

claude_mpm/core/framework/loaders/agent_loader.py

@@ -134,7 +134,7 @@ class AgentLoader:
 
             for json_file in template_dir.glob("*.json"):
                 try:
-                    with open(json_file) as f:
+                    with json_file.open() as f:
                         template_data = json.load(f)
 
                     # Extract agent metadata

claude_mpm/core/framework/processors/metadata_processor.py

@@ -55,7 +55,7 @@ class MetadataProcessor:
             Dictionary with agent metadata or None
         """
         try:
-            with open(agent_file) as f:
+            with agent_file.open() as f:
                 content = f.read()
 
             # Default values

claude_mpm/core/framework/processors/template_processor.py

@@ -89,7 +89,7 @@ class TemplateProcessor:
         # Try exact match first
         template_file = templates_dir / f"{agent_name}.json"
         if template_file.exists():
-            with open(template_file) as f:
+            with template_file.open() as f:
                 return json.load(f)
 
         # Try alternative naming variations
@@ -97,7 +97,7 @@ class TemplateProcessor:
         for alt_name in alternative_names:
             alt_file = templates_dir / f"{alt_name}.json"
             if alt_file.exists():
-                with open(alt_file) as f:
+                with alt_file.open() as f:
                     return json.load(f)
 
         return None
@@ -218,7 +218,7 @@ class TemplateProcessor:
 
             for json_file in template_dir.glob("*.json"):
                 try:
-                    with open(json_file) as f:
+                    with json_file.open() as f:
                         template_data = json.load(f)
 
                     agent_metadata = self.extract_metadata(template_data)

claude_mpm/core/framework_loader.py

@@ -504,9 +504,9 @@ class FrameworkLoader:
             metadata["instructions_length"] = len(instructions)
 
             if loop.is_running():
-                asyncio.create_task(
+                _task = asyncio.create_task(  # noqa: RUF006
                     log_manager.log_prompt("system_prompt", instructions, metadata)
-                )
+                )  # Fire-and-forget logging
             else:
                 loop.run_until_complete(
                     log_manager.log_prompt("system_prompt", instructions, metadata)

claude_mpm/core/log_manager.py

@@ -542,12 +542,12 @@ class LogManager:
 
                 if extension == ".json":
                     # JSON files also get structured metadata for consistency
-                    with open(file_path, "w", encoding="utf-8") as f:
+                    with file_path.open("w", encoding="utf-8") as f:
                         json.dump(data, f, indent=2, ensure_ascii=False)
                 # For markdown or text files
                 elif isinstance(data, dict):
                     # Write as formatted markdown with metadata
-                    with open(file_path, "w", encoding="utf-8") as f:
+                    with file_path.open("w", encoding="utf-8") as f:
                         f.write("---\n")
                         f.write(f"timestamp: {data.get('timestamp', 'unknown')}\n")
                         f.write(f"type: {data.get('type', 'unknown')}\n")
@@ -559,7 +559,7 @@ class LogManager:
                         f.write(data.get("content", ""))
                 else:
                     # Write content directly
-                    with open(file_path, "w", encoding="utf-8") as f:
+                    with file_path.open("w", encoding="utf-8") as f:
                         f.write(str(data))
             except Exception as e:
                 logger.error(f"Failed to write {file_path}: {e}")
@@ -588,7 +588,7 @@ class LogManager:
 
         def write_task():
             try:
-                with open(log_file, "a", encoding="utf-8") as f:
+                with log_file.open("a", encoding="utf-8") as f:
                     f.write(log_entry)
             except Exception as e:
                 logger.error(f"Failed to write log: {e}")

claude_mpm/core/logger.py

@@ -526,7 +526,7 @@ class ProjectLogger:
             self.dirs["logs_system"]
             / f"{datetime.now(timezone.utc).strftime('%Y%m%d')}.jsonl"
         )
-        with open(log_file, "a") as f:
+        with log_file.open("a") as f:
             f.write(json.dumps(log_entry) + "\n")
 
     def log_agent_invocation(
@@ -580,7 +580,7 @@ class ProjectLogger:
         daily_log = (
             agent_log_dir / f"{datetime.now(timezone.utc).strftime('%Y%m%d')}.jsonl"
         )
-        with open(daily_log, "a") as f:
+        with daily_log.open("a") as f:
             f.write(json.dumps(log_entry) + "\n")
 
     def get_session_summary(self) -> Dict[str, Any]:

claude_mpm/core/optimized_startup.py

@@ -227,7 +227,7 @@ class OptimizedStartup:
         """Parse configuration file."""
         import yaml
 
-        with open(path) as f:
+        with path.open() as f:
             return yaml.safe_load(f) or {}
 
     def _store_config(self, config: Dict[str, Any]):

claude_mpm/core/output_style_manager.py

@@ -42,7 +42,7 @@ class OutputStyleManager:
             Path(__file__).parent.parent / "agents" / "OUTPUT_STYLE.md"
         )
 
-    def _detect_claude_version(self) -> Optional[str]:  # noqa: PLR0911
+    def _detect_claude_version(self) -> Optional[str]:
         """
         Detect Claude Code version by running 'claude --version'.
         Uses global cache to avoid duplicate detection and logging.

claude_mpm/core/service_registry.py

@@ -160,10 +160,10 @@ class ServiceRegistry:
             try:
                 # Use the enhanced container's named resolution
                 return self.container.get(BaseService, name=service_type)
-            except Exception:
+            except Exception as e:
                 # Fall back to looking up class and resolving
                 if service_type not in self._services:
-                    raise KeyError(f"Service '{service_type}' not registered")
+                    raise KeyError(f"Service '{service_type}' not registered") from e
                 service_class = self._services[service_type]
                 return self.container.get(service_class)
         else:

claude_mpm/core/session_manager.py

@@ -187,7 +187,7 @@ class SessionManager:
         """Save sessions to disk."""
         session_file = self.session_dir / "active_sessions.json"
         try:
-            with open(session_file, "w") as f:
+            with session_file.open("w") as f:
                 json.dump(self.active_sessions, f, indent=2)
         except Exception as e:
             logger.error(f"Failed to save sessions: {e}")
@@ -197,7 +197,7 @@ class SessionManager:
         session_file = self.session_dir / "active_sessions.json"
         if session_file.exists():
             try:
-                with open(session_file) as f:
+                with session_file.open() as f:
                     self.active_sessions = json.load(f)
 
                 # Clean up old sessions on load (archive by default)
@@ -286,7 +286,7 @@ class SessionManager:
             backup_path = archive_dir / backup_name
 
             # Compress and backup current file
-            with open(claude_json_path, "rb") as f_in:
+            with claude_json_path.open("rb") as f_in:
                 with gzip.open(backup_path, "wb") as f_out:
                     shutil.copyfileobj(f_in, f_out)
 

claude_mpm/core/shared/config_loader.py

@@ -228,7 +228,7 @@ class ConfigLoader:
         try:
             import yaml
 
-            with open(config_file) as f:
+            with config_file.open() as f:
                 if config_file.suffix.lower() in (".yaml", ".yml"):
                     return yaml.safe_load(f) or {}
                 # Try JSON as fallback

claude_mpm/core/socketio_pool.py

@@ -594,9 +594,9 @@ class SocketIOConnectionPool:
                 timeout=2.0,
             )
 
-        except asyncio.TimeoutError:
+        except asyncio.TimeoutError as e:
             self.logger.debug("Socket.IO connection timeout")
-            raise TimeoutError("Socket.IO connection timeout")
+            raise TimeoutError("Socket.IO connection timeout") from e
         except Exception as e:
             self.logger.debug(f"Client connection failed: {e}")
             raise

claude_mpm/core/unified_agent_registry.py

@@ -728,7 +728,7 @@ class UnifiedAgentRegistry:
             },
         }
 
-        with open(output_path, "w") as f:
+        with output_path.open("w") as f:
             json.dump(export_data, f, indent=2)
 
         logger.info(f"Exported {len(self.registry)} agents to {output_path}")
@@ -737,7 +737,7 @@ class UnifiedAgentRegistry:
         """Import registry from JSON file."""
         input_path = Path(input_path)
 
-        with open(input_path) as f:
+        with input_path.open() as f:
             data = json.load(f)
 
         # Clear current registry

claude_mpm/core/unified_config.py

@@ -436,7 +436,7 @@ class ConfigurationService:
                 if config_path.exists():
                     import yaml
 
-                    with open(config_path) as f:
+                    with config_path.open() as f:
                         file_config = yaml.safe_load(f) or {}
                     config_data.update(file_config)
                     break
@@ -448,7 +448,7 @@ class ConfigurationService:
             raise ConfigurationError(
                 f"Failed to load configuration: {e}",
                 context={"error_type": type(e).__name__},
-            )
+            ) from e
 
     @property
     def config(self) -> UnifiedConfig:
@@ -515,7 +515,7 @@ class ConfigurationService:
                 return True
             raise ConfigurationError("Invalid SocketIO port range")
         except Exception as e:
-            raise ConfigurationError(f"Configuration validation failed: {e}")
+            raise ConfigurationError(f"Configuration validation failed: {e}") from e
 
     def export_to_file(self, file_path: Union[str, Path], format: str = "yaml") -> None:
         """
@@ -531,12 +531,12 @@ class ConfigurationService:
             if format.lower() == "yaml":
                 import yaml
 
-                with open(file_path, "w") as f:
+                with file_path.open("w") as f:
                     yaml.dump(self._config.dict(), f, default_flow_style=False)
             elif format.lower() == "json":
                 import json
 
-                with open(file_path, "w") as f:
+                with file_path.open("w") as f:
                     json.dump(self._config.dict(), f, indent=2)
             else:
                 raise ConfigurationError(f"Unsupported export format: {format}")
@@ -545,4 +545,4 @@ class ConfigurationService:
             raise ConfigurationError(
                 f"Failed to export configuration: {e}",
                 context={"file_path": str(file_path), "format": format},
-            )
+            ) from e

claude_mpm/core/unified_paths.py

@@ -167,7 +167,7 @@ class PathContext:
 
     @staticmethod
     @lru_cache(maxsize=1)
-    def detect_deployment_context() -> DeploymentContext:  # noqa: PLR0911
+    def detect_deployment_context() -> DeploymentContext:
         """Detect the current deployment context.
 
         Priority order:
@@ -396,7 +396,7 @@ class UnifiedPathManager:
                     return current
                 current = current.parent
 
-            raise FileNotFoundError("Could not determine framework root")
+            raise FileNotFoundError("Could not determine framework root") from None
 
     @property
     @lru_cache(maxsize=1)

claude_mpm/dashboard/api/simple_directory.py

@@ -129,7 +129,7 @@ def has_code_files(directory_path, max_depth=5, current_depth=0):
     return False
 
 
-def should_show_item(item_name, item_path, is_directory):  # noqa: PLR0911
+def should_show_item(item_name, item_path, is_directory):
     """Determine if an item should be shown based on filtering rules"""
     # Always hide system files
     if item_name in {".DS_Store", "Thumbs.db", "desktop.ini"}:

claude_mpm/generators/agent_profile_generator.py

@@ -29,7 +29,7 @@ class AgentProfileGenerator:
         if not self.template_path.exists():
             raise FileNotFoundError(f"Template not found: {self.template_path}")
 
-        with open(self.template_path) as f:
+        with self.template_path.open() as f:
             return yaml.safe_load(f)
 
     def generate_profile(self, config: Dict[str, Any]) -> str:

claude_mpm/hooks/claude_hooks/event_handlers.py

@@ -284,11 +284,11 @@ class EventHandlers:
                 # Log the agent prompt asynchronously
                 try:
                     loop = asyncio.get_running_loop()
-                    asyncio.create_task(
+                    _task = asyncio.create_task(  # noqa: RUF006
                         log_manager.log_prompt(
                             f"agent_{agent_type}", prompt_content, metadata
                         )
-                    )
+                    )  # Fire-and-forget logging (ephemeral hook process)
                 except RuntimeError:
                     # No running loop, create one
                     loop = asyncio.new_event_loop()