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

claude_mpm/services/diagnostics/checks/common_issues_check.py

@@ -114,7 +114,7 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
             # Try to count conversations
             conversation_count = 0
             try:
-                with open(claude_json_path) as f:
+                with claude_json_path.open() as f:
                     data = json.load(f)
                     if isinstance(data, dict) and "conversations" in data:
                         conversation_count = len(data["conversations"])
@@ -283,10 +283,10 @@ class CommonIssuesCheck(BaseDiagnosticCheck):
             try:
                 import yaml
 
-                with open(user_config) as f:
+                with user_config.open() as f:
                     user_data = yaml.safe_load(f) or {}
 
-                with open(project_config) as f:
+                with project_config.open() as f:
                     project_data = yaml.safe_load(f) or {}
 
                 # Check for conflicting keys

claude_mpm/services/diagnostics/checks/configuration_check.py

@@ -91,7 +91,7 @@ class ConfigurationCheck(BaseDiagnosticCheck):
             )
 
         try:
-            with open(config_path) as f:
+            with config_path.open() as f:
                 config = yaml.safe_load(f)
 
             issues = self._validate_config_structure(config)
@@ -143,7 +143,7 @@ class ConfigurationCheck(BaseDiagnosticCheck):
             )
 
         try:
-            with open(config_path) as f:
+            with config_path.open() as f:
                 config = yaml.safe_load(f)
 
             issues = self._validate_config_structure(config)

claude_mpm/services/diagnostics/checks/installation_check.py

@@ -138,7 +138,7 @@ class InstallationCheck(BaseDiagnosticCheck):
                 details={"error": str(e)},
             )
 
-    def _check_installation_method(self) -> DiagnosticResult:  # noqa: PLR0911
+    def _check_installation_method(self) -> DiagnosticResult:
         """Detect how claude-mpm was installed."""
         methods_found = []
         details = {}

claude_mpm/services/diagnostics/checks/mcp_check.py

@@ -154,7 +154,7 @@ class MCPCheck(BaseDiagnosticCheck):
             )
 
         try:
-            with open(config_path) as f:
+            with config_path.open() as f:
                 config = json.load(f)
 
                 mcp_servers = config.get("mcpServers", {})

claude_mpm/services/diagnostics/checks/mcp_services_check.py

@@ -369,7 +369,7 @@ class MCPServicesCheck(BaseDiagnosticCheck):
                             stderr_output = stderr_data.decode(
                                 "utf-8", errors="ignore"
                             )[:200]
-                    except:
+                    except (asyncio.TimeoutError, OSError):
                         pass
 
                 if stderr_output:
@@ -391,7 +391,7 @@ class MCPServicesCheck(BaseDiagnosticCheck):
                             stderr_output = stderr_data.decode(
                                 "utf-8", errors="ignore"
                             )[:200]
-                    except:
+                    except (asyncio.TimeoutError, OSError):
                         pass
 
                 if stderr_output:
@@ -778,7 +778,7 @@ class MCPServicesCheck(BaseDiagnosticCheck):
             return None
 
         try:
-            with open(claude_config_path) as f:
+            with claude_config_path.open() as f:
                 config = json.load(f)
 
             mcp_servers = config.get("mcpServers", {})
@@ -869,7 +869,7 @@ class MCPServicesCheck(BaseDiagnosticCheck):
 
             # Create backup
             backup_path = config_path.with_suffix(".json.backup")
-            with open(backup_path, "w") as f:
+            with backup_path.open("w") as f:
                 json.dump(config, f, indent=2)
 
             # Update the configuration - ensure we're setting the exact new_args
@@ -884,11 +884,11 @@ class MCPServicesCheck(BaseDiagnosticCheck):
                 return False
 
             # Write updated configuration
-            with open(config_path, "w") as f:
+            with config_path.open("w") as f:
                 json.dump(config, f, indent=2)
 
             # Verify the file was written correctly
-            with open(config_path) as f:
+            with config_path.open() as f:
                 verify_config = json.load(f)
                 verify_args = (
                     verify_config.get("mcpServers", {})
@@ -902,9 +902,9 @@ class MCPServicesCheck(BaseDiagnosticCheck):
                         f"Expected {new_args}, got {verify_args}"
                     )
                     # Restore backup
-                    with open(backup_path) as bf:
+                    with backup_path.open() as bf:
                         backup_config = json.load(bf)
-                    with open(config_path, "w") as f:
+                    with config_path.open("w") as f:
                         json.dump(backup_config, f, indent=2)
                     return False
 
@@ -936,7 +936,7 @@ class MCPServicesCheck(BaseDiagnosticCheck):
                     fix_description="Initialize Claude configuration",
                 )
 
-            with open(config_file) as f:
+            with config_file.open() as f:
                 config = json.load(f)
 
             # Get the current project configuration

claude_mpm/services/diagnostics/checks/monitor_check.py

@@ -193,7 +193,7 @@ class MonitorCheck(BaseDiagnosticCheck):
         for config_path in config_paths:
             if config_path.exists():
                 try:
-                    with open(config_path) as f:
+                    with config_path.open() as f:
                         config = yaml.safe_load(f)
                         if config and isinstance(config, dict):
                             response_config = config.get("response_logging", {})

claude_mpm/services/diagnostics/doctor_reporter.py

@@ -248,7 +248,7 @@ class DoctorReporter:
         # Header with timestamp
         print("# Claude MPM Doctor Report")
         print(
-            f"\n**Generated:** {datetime.datetime.now().strftime('%Y-%m-%d %H:%M:%S')}"
+            f"\n**Generated:** {datetime.datetime.now(datetime.timezone.utc).strftime('%Y-%m-%d %H:%M:%S')}"
         )
         print(f"**Version:** {self._get_version()}\n")
         print("---\n")

claude_mpm/services/event_aggregator.py

@@ -491,7 +491,7 @@ class EventAggregator:
         for filepath in session_files:
             try:
                 # Load just the metadata, not the full session
-                with open(filepath) as f:
+                with filepath.open() as f:
                     data = json.load(f)
 
                 sessions.append(

claude_mpm/services/event_bus/event_bus.py

@@ -65,6 +65,9 @@ class EventBus:
         self._event_history: List[Dict[str, Any]] = []
         self._max_history_size = 100
 
+        # Track async handler tasks to prevent garbage collection
+        self._handler_tasks: Set[asyncio.Task] = set()
+
         logger.info("EventBus initialized")
 
     @classmethod
@@ -190,13 +193,15 @@ class EventBus:
                             try:
                                 # Call with event_type and data for wildcard handlers
                                 if asyncio.iscoroutinefunction(handler):
-                                    # Schedule async handlers
+                                    # Schedule async handlers with tracking
                                     try:
                                         loop = asyncio.get_event_loop()
                                         if loop.is_running():
-                                            asyncio.create_task(
+                                            task = asyncio.create_task(
                                                 handler(event_type, data)
                                             )
+                                            self._handler_tasks.add(task)
+                                            task.add_done_callback(self._handler_tasks.discard)
                                         else:
                                             loop.run_until_complete(
                                                 handler(event_type, data)

claude_mpm/services/events/consumers/dead_letter.py

@@ -109,7 +109,7 @@ class DeadLetterConsumer(IEventConsumer):
                 self._rotate_file()
 
             # Write to file
-            with open(self._current_file, "a") as f:
+            with self._current_file.open("a") as f:
                 f.write(event_json)
 
             # Update metrics
@@ -165,7 +165,7 @@ class DeadLetterConsumer(IEventConsumer):
         # Find files in time range
         for file_path in sorted(self.output_dir.glob("dead-letter-*.jsonl")):
             try:
-                with open(file_path) as f:
+                with file_path.open() as f:
                     for line in f:
                         event_data = json.loads(line)
 

claude_mpm/services/framework_claude_md_generator/__init__.py

@@ -141,7 +141,7 @@ class FrameworkClaudeMdGenerator:
         current_content = None
 
         if target_file.exists():
-            with open(target_file) as f:
+            with target_file.open() as f:
                 current_content = f.read()
 
         # Generate new content

claude_mpm/services/framework_claude_md_generator/deployment_manager.py

@@ -95,7 +95,7 @@ class DeploymentManager:
 
         # Check if file exists and compare versions
         if target_file.exists() and not force:
-            with open(target_file) as f:
+            with target_file.open() as f:
                 existing_content = f.read()
                 existing_fw_ver = self.version_manager.parse_current_version(
                     existing_content
@@ -110,7 +110,7 @@ class DeploymentManager:
             parent_path.mkdir(parents=True, exist_ok=True)
 
             # Write content
-            with open(target_file, "w") as f:
+            with target_file.open("w") as f:
                 f.write(content)
 
             # Get version info for success message
@@ -137,7 +137,7 @@ class DeploymentManager:
             return True, f"{self.target_filename} does not exist"
 
         try:
-            with open(target_file) as f:
+            with target_file.open() as f:
                 existing_content = f.read()
                 existing_fw_ver = self.version_manager.parse_current_version(
                     existing_content

claude_mpm/services/framework_claude_md_generator/version_manager.py

@@ -38,7 +38,7 @@ class VersionManager:
             version_path = package_path.parent / "framework" / "VERSION"
 
         if version_path.exists():
-            with open(version_path) as f:
+            with version_path.open() as f:
                 version_content = f.read().strip()
                 # Framework VERSION file contains just the framework version number
                 try:

claude_mpm/services/hook_installer_service.py

@@ -24,7 +24,7 @@ class HookInstallerService:
         self.claude_dir = Path.home() / ".claude"
         self.settings_file = self.claude_dir / "settings.json"
 
-    def is_hooks_configured(self) -> bool:  # noqa: PLR0911
+    def is_hooks_configured(self) -> bool:
         """Check if hooks are configured in Claude settings.
 
         Returns:
@@ -35,7 +35,7 @@ class HookInstallerService:
                 self.logger.debug("Claude settings file does not exist")
                 return False
 
-            with open(self.settings_file) as f:
+            with self.settings_file.open() as f:
                 settings = json.load(f)
 
             # Check if hooks section exists
@@ -292,7 +292,7 @@ class HookInstallerService:
 
             # Load existing settings or create new
             if self.settings_file.exists():
-                with open(self.settings_file) as f:
+                with self.settings_file.open() as f:
                     settings = json.load(f)
                 self.logger.debug("Found existing Claude settings")
             else:
@@ -320,7 +320,7 @@ class HookInstallerService:
                 settings["hooks"][event_type] = [hook_config]
 
             # Write settings
-            with open(self.settings_file, "w") as f:
+            with self.settings_file.open("w") as f:
                 json.dump(settings, f, indent=2)
 
             self.logger.info(f"Updated Claude settings at: {self.settings_file}")
@@ -380,7 +380,7 @@ class HookInstallerService:
 
             self.logger.info("Removing Claude Code hooks...")
 
-            with open(self.settings_file) as f:
+            with self.settings_file.open() as f:
                 settings = json.load(f)
 
             hooks_removed = 0
@@ -422,7 +422,7 @@ class HookInstallerService:
                     del settings["hooks"]
 
             # Write updated settings
-            with open(self.settings_file, "w") as f:
+            with self.settings_file.open("w") as f:
                 json.dump(settings, f, indent=2)
 
             if hooks_removed > 0:
@@ -469,7 +469,7 @@ class HookInstallerService:
 
         try:
             if self.settings_file.exists():
-                with open(self.settings_file) as f:
+                with self.settings_file.open() as f:
                     settings = json.load(f)
 
                 if "hooks" in settings:

claude_mpm/services/infrastructure/context_preservation.py

@@ -151,7 +151,7 @@ class ContextPreservationService(BaseService):
                 return None
 
             # Use streaming to find active conversation
-            with open(self.claude_json_path, "rb") as f:
+            with self.claude_json_path.open("rb") as f:
                 parser = ijson.parse(f)
 
                 active_conv_id = None
@@ -207,7 +207,7 @@ class ContextPreservationService(BaseService):
                 keep_recent_days * 86400
             )
 
-            with open(self.claude_json_path) as f:
+            with self.claude_json_path.open() as f:
                 data = json.load(f)
 
             original_count = len(data.get("conversations", []))
@@ -239,7 +239,7 @@ class ContextPreservationService(BaseService):
 
             # Write compressed version
             temp_path = self.claude_json_path.with_suffix(".tmp")
-            with open(temp_path, "w") as f:
+            with temp_path.open("w") as f:
                 json.dump(data, f, separators=(",", ":"))  # Compact format
 
             # Replace original
@@ -316,7 +316,7 @@ class ContextPreservationService(BaseService):
             # Use streaming to extract preferences
             preferences = {}
 
-            with open(self.claude_json_path, "rb") as f:
+            with self.claude_json_path.open("rb") as f:
                 parser = ijson.parse(f)
 
                 for prefix, event, value in parser:
@@ -339,7 +339,7 @@ class ContextPreservationService(BaseService):
     ) -> ConversationState:
         """Parse Claude JSON using standard JSON parser."""
         try:
-            with open(self.claude_json_path) as f:
+            with self.claude_json_path.open() as f:
                 data = json.load(f)
 
             return await self._extract_conversation_state(data, extract_full)
@@ -357,7 +357,7 @@ class ContextPreservationService(BaseService):
             preferences = {}
             open_files = []
 
-            with open(self.claude_json_path, "rb") as f:
+            with self.claude_json_path.open("rb") as f:
                 parser = ijson.parse(f)
 
                 conversation_count = 0
@@ -527,7 +527,7 @@ class ContextPreservationService(BaseService):
                 backup_name += ".gz"
                 backup_path = self.claude_backup_dir / backup_name
 
-                with open(self.claude_json_path, "rb") as f_in:
+                with self.claude_json_path.open("rb") as f_in:
                     with gzip.open(backup_path, "wb") as f_out:
                         shutil.copyfileobj(f_in, f_out)
             else:

claude_mpm/services/infrastructure/daemon_manager.py

@@ -80,7 +80,7 @@ class SocketIODaemonManager:
             return False
 
         try:
-            with open(self.pid_file) as f:
+            with self.pid_file.open() as f:
                 pid = int(f.read().strip())
 
             # Check if process exists and is running
@@ -135,7 +135,7 @@ class SocketIODaemonManager:
             pid = os.fork()
             if pid > 0:
                 # Parent process - save PID and exit
-                with open(self.pid_file, "w") as f:
+                with self.pid_file.open("w") as f:
                     f.write(str(pid))
                 logger.info(f"Socket.IO server started as daemon (PID: {pid})")
                 return True
@@ -155,7 +155,7 @@ class SocketIODaemonManager:
         os.umask(0)
 
         # Redirect stdout/stderr to log file
-        with open(self.log_file, "a") as log:
+        with self.log_file.open("a") as log:
             os.dup2(log.fileno(), sys.stdout.fileno())
             os.dup2(log.fileno(), sys.stderr.fileno())
 
@@ -199,7 +199,7 @@ class SocketIODaemonManager:
             return False
 
         try:
-            with open(self.pid_file) as f:
+            with self.pid_file.open() as f:
                 pid = int(f.read().strip())
 
             logger.info(f"Stopping Socket.IO server (PID: {pid})")
@@ -254,7 +254,7 @@ class SocketIODaemonManager:
         }
 
         if status_info["running"]:
-            with open(self.pid_file) as f:
+            with self.pid_file.open() as f:
                 status_info["pid"] = int(f.read().strip())
 
             # Check port accessibility

claude_mpm/services/mcp_config_manager.py

@@ -199,7 +199,7 @@ class MCPConfigManager:
                             f"Found kuzu-memory with MCP support at {path}"
                         )
                         return path
-                except:
+                except (subprocess.SubprocessError, subprocess.TimeoutExpired, OSError):
                     pass
 
             # If no MCP-capable version found, log warning but return None
@@ -571,7 +571,7 @@ class MCPConfigManager:
                 if result.returncode == 0 or "version" in result.stdout.lower():
                     use_pipx_run = True
                     self.logger.debug(f"Will use 'pipx run' for {service_name}")
-            except:
+            except (subprocess.SubprocessError, subprocess.TimeoutExpired, OSError):
                 pass
 
         # Try uvx if pipx run not available
@@ -587,7 +587,7 @@ class MCPConfigManager:
                 if result.returncode == 0 or "version" in result.stdout.lower():
                     use_uvx = True
                     self.logger.debug(f"Will use 'uvx' for {service_name}")
-            except:
+            except (subprocess.SubprocessError, subprocess.TimeoutExpired, OSError):
                 pass
 
         # If neither work, try to find direct path
@@ -719,7 +719,7 @@ class MCPConfigManager:
         claude_config = {}
         if self.claude_config_path.exists():
             try:
-                with open(self.claude_config_path) as f:
+                with self.claude_config_path.open() as f:
                     claude_config = json.load(f)
             except Exception as e:
                 self.logger.error(f"Error reading {self.claude_config_path}: {e}")
@@ -832,7 +832,7 @@ class MCPConfigManager:
                         self.logger.debug(f"Created backup: {backup_path}")
 
                 # Write updated config
-                with open(self.claude_config_path, "w") as f:
+                with self.claude_config_path.open("w") as f:
                     json.dump(claude_config, f, indent=2)
 
                 messages = []
@@ -884,7 +884,7 @@ class MCPConfigManager:
         existing_config = {}
         if mcp_config_path.exists():
             try:
-                with open(mcp_config_path) as f:
+                with mcp_config_path.open() as f:
                     existing_config = json.load(f)
             except Exception as e:
                 self.logger.error(f"Error reading existing config: {e}")
@@ -912,7 +912,7 @@ class MCPConfigManager:
 
         # Write the updated configuration
         try:
-            with open(mcp_config_path, "w") as f:
+            with mcp_config_path.open("w") as f:
                 json.dump(new_config, f, indent=2)
 
             if missing_services:
@@ -937,7 +937,7 @@ class MCPConfigManager:
             mcp_config_path = self.project_root / ConfigLocation.PROJECT_MCP.value
             if mcp_config_path.exists():
                 try:
-                    with open(mcp_config_path) as f:
+                    with mcp_config_path.open() as f:
                         config = json.load(f)
                         results = {}
                         for service_name, service_config in config.get(
@@ -951,7 +951,7 @@ class MCPConfigManager:
             return {}
 
         try:
-            with open(self.claude_config_path) as f:
+            with self.claude_config_path.open() as f:
                 claude_config = json.load(f)
 
             # Get project's MCP servers
@@ -1638,7 +1638,7 @@ class MCPConfigManager:
                     if result.returncode == 0 or "version" in result.stdout.lower():
                         self.logger.debug(f"{service_name} accessible via 'pipx run'")
                         return True
-                except:
+                except (subprocess.SubprocessError, subprocess.TimeoutExpired, OSError):
                     pass
             return False
 

claude_mpm/services/mcp_gateway/auto_configure.py

@@ -78,7 +78,7 @@ class MCPAutoConfigurator:
             return False
 
         try:
-            with open(self.claude_config_path) as f:
+            with self.claude_config_path.open() as f:
                 config = json.load(f)
 
             # Check if claude-mpm-gateway is configured
@@ -131,7 +131,7 @@ class MCPAutoConfigurator:
             return False
 
         try:
-            with open(self.preference_file) as f:
+            with self.preference_file.open() as f:
                 prefs = json.load(f)
             return prefs.get("asked", False)
         except (OSError, json.JSONDecodeError):
@@ -148,7 +148,7 @@ class MCPAutoConfigurator:
         }
 
         try:
-            with open(self.preference_file, "w") as f:
+            with self.preference_file.open("w") as f:
                 json.dump(prefs, f, indent=2)
         except Exception as e:
             self.logger.debug(f"Could not save preference: {e}")
@@ -244,7 +244,7 @@ class MCPAutoConfigurator:
             }
 
             # Save configuration
-            with open(self.claude_config_path, "w") as f:
+            with self.claude_config_path.open("w") as f:
                 json.dump(config, f, indent=2)
 
             print(f"✅ Configuration saved to: {self.claude_config_path}")
@@ -284,7 +284,7 @@ class MCPAutoConfigurator:
         """Load existing config or create new one."""
         if self.claude_config_path.exists():
             try:
-                with open(self.claude_config_path) as f:
+                with self.claude_config_path.open() as f:
                     return json.load(f)
             except json.JSONDecodeError:
                 self.logger.warning("Existing config is invalid JSON, creating new")

claude_mpm/services/mcp_gateway/config/config_loader.py

@@ -97,7 +97,7 @@ class MCPConfigLoader:
                 self.logger.error(f"Configuration file not found: {expanded_path}")
                 return None
 
-            with open(expanded_path) as f:
+            with expanded_path.open() as f:
                 config = yaml.safe_load(f)
 
             self.logger.info(f"Configuration loaded from {expanded_path}")
@@ -271,7 +271,7 @@ class MCPConfigLoader:
             expanded_path = path.expanduser()
             expanded_path.parent.mkdir(parents=True, exist_ok=True)
 
-            with open(expanded_path, "w") as f:
+            with expanded_path.open("w") as f:
                 yaml.dump(
                     MCPConfiguration.DEFAULT_CONFIG,
                     f,

claude_mpm/services/mcp_gateway/config/configuration.py

@@ -155,7 +155,7 @@ class MCPConfiguration(BaseMCPService, IMCPConfiguration):
                 self.log_warning(f"Configuration file not found: {config_path}")
                 return True  # Not an error, use defaults
 
-            with open(config_path) as f:
+            with config_path.open() as f:
                 if config_path.suffix in [".yaml", ".yml"]:
                     loaded_config = yaml.safe_load(f) or {}
                 else:
@@ -171,7 +171,7 @@ class MCPConfiguration(BaseMCPService, IMCPConfiguration):
             return True
 
         except yaml.YAMLError as e:
-            raise MCPConfigurationError(f"Failed to parse YAML configuration: {e}")
+            raise MCPConfigurationError(f"Failed to parse YAML configuration: {e}") from e
         except Exception as e:
             self.log_error(f"Failed to load configuration: {e}")
             return False
@@ -364,7 +364,7 @@ class MCPConfiguration(BaseMCPService, IMCPConfiguration):
             save_path = Path(save_path).expanduser()
             save_path.parent.mkdir(parents=True, exist_ok=True)
 
-            with open(save_path, "w") as f:
+            with save_path.open("w") as f:
                 yaml.dump(
                     self._config_data, f, default_flow_style=False, sort_keys=True
                 )

claude_mpm/services/mcp_gateway/core/process_pool.py

@@ -190,7 +190,7 @@ class MCPProcessPool:
 
             # Write process info to file for debugging
             info_file = self.pool_dir / f"{server_name}_{process.pid}.json"
-            with open(info_file, "w") as f:
+            with info_file.open("w") as f:
                 json.dump(self._process_info[server_name], f, indent=2)
 
             return process
@@ -934,7 +934,7 @@ async def pre_warm_mcp_servers():
     #
     # if claude_config_path.exists():
     #     try:
-    #         with open(claude_config_path) as f:
+    #         with claude_config_path.open() as f:
     #             config_data = json.load(f)
     #             mcp_servers = config_data.get("mcpServers", {})
     #             configs.update(mcp_servers)
@@ -945,7 +945,7 @@ async def pre_warm_mcp_servers():
     # mcp_config_path = Path.cwd() / ".mcp.json"
     # if mcp_config_path.exists():
     #     try:
-    #         with open(mcp_config_path) as f:
+    #         with mcp_config_path.open() as f:
     #             config_data = json.load(f)
     #             mcp_servers = config_data.get("mcpServers", {})
     #             configs.update(mcp_servers)

claude_mpm/services/mcp_gateway/core/singleton_manager.py

@@ -149,7 +149,7 @@ class MCPGatewayManager:
             return None
 
         try:
-            with open(self.instance_file) as f:
+            with self.instance_file.open() as f:
                 instance_info = json.load(f)
 
             # Validate PID is still running
@@ -195,7 +195,7 @@ class MCPGatewayManager:
                 "lock_file": str(self.lock_file),
             }
 
-            with open(self.instance_file, "w") as f:
+            with self.instance_file.open("w") as f:
                 json.dump(instance_info, f, indent=2)
 
             self._current_instance = instance_info

claude_mpm/services/mcp_gateway/core/startup_verification.py

@@ -125,7 +125,7 @@ class MCPGatewayStartupVerifier:
         try:
             import json
 
-            with open(self.config_file, "w") as f:
+            with self.config_file.open("w") as f:
                 json.dump(default_config, f, indent=2)
 
             self.logger.info(