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

claude_mpm/services/agents/registry/deployed_agent_discovery.py

@@ -166,7 +166,7 @@ class DeployedAgentDiscovery(ConfigServiceBase):
         try:
             path = Path(agent_path)
             if path.exists() and path.suffix == ".json":
-                with open(path) as f:
+                with path.open() as f:
                     return json.load(f)
         except Exception as e:
             logger.warning(f"Failed to load full agent data from {agent_path}: {e}")

claude_mpm/services/agents/registry/modification_tracker.py

@@ -156,12 +156,19 @@ class AgentFileSystemHandler(FileSystemEventHandler):
     def __init__(self, tracker: "AgentModificationTracker"):
         self.tracker = tracker
 
+    def _create_tracked_task(self, coro):
+        """Create a task with automatic tracking and cleanup."""
+        task = asyncio.create_task(coro)
+        self.tracker._file_event_tasks.add(task)
+        task.add_done_callback(self.tracker._file_event_tasks.discard)
+        return task
+
     def on_created(self, event: FileSystemEvent) -> None:
         """Handle file creation events."""
         if not event.is_directory and event.src_path.endswith(
             (".md", ".json", ".yaml")
         ):
-            asyncio.create_task(
+            self._create_tracked_task(
                 self.tracker._handle_file_modification(
                     event.src_path, ModificationType.CREATE
                 )
@@ -172,7 +179,7 @@ class AgentFileSystemHandler(FileSystemEventHandler):
         if not event.is_directory and event.src_path.endswith(
             (".md", ".json", ".yaml")
         ):
-            asyncio.create_task(
+            self._create_tracked_task(
                 self.tracker._handle_file_modification(
                     event.src_path, ModificationType.MODIFY
                 )
@@ -183,7 +190,7 @@ class AgentFileSystemHandler(FileSystemEventHandler):
         if not event.is_directory and event.src_path.endswith(
             (".md", ".json", ".yaml")
         ):
-            asyncio.create_task(
+            self._create_tracked_task(
                 self.tracker._handle_file_modification(
                     event.src_path, ModificationType.DELETE
                 )
@@ -194,7 +201,7 @@ class AgentFileSystemHandler(FileSystemEventHandler):
         if not event.is_directory and event.src_path.endswith(
             (".md", ".json", ".yaml")
         ):
-            asyncio.create_task(
+            self._create_tracked_task(
                 self.tracker._handle_file_move(event.src_path, event.dest_path)
             )
 
@@ -248,6 +255,7 @@ class AgentModificationTracker(BaseService):
         # Background tasks
         self._persistence_task: Optional[asyncio.Task] = None
         self._cleanup_task: Optional[asyncio.Task] = None
+        self._file_event_tasks: Set[asyncio.Task] = set()  # Track file event tasks
 
         # Callbacks
         self.modification_callbacks: List[Callable[[AgentModification], None]] = []
@@ -473,7 +481,7 @@ class AgentModificationTracker(BaseService):
                 metadata["file_size_after"] = path.stat().st_size
 
                 # File hash
-                with open(path, "rb") as f:
+                with path.open("rb") as f:
                     metadata["file_hash_after"] = hashlib.sha256(f.read()).hexdigest()
 
                 # File type
@@ -512,7 +520,7 @@ class AgentModificationTracker(BaseService):
             }
 
             metadata_path = backup_dir / "metadata.json"
-            with open(metadata_path, "w") as f:
+            with metadata_path.open("w") as f:
                 json.dump(metadata, f, indent=2)
 
             return str(backup_path)
@@ -661,7 +669,7 @@ class AgentModificationTracker(BaseService):
             # Load active modifications
             active_path = self.persistence_root / "active_modifications.json"
             if active_path.exists():
-                with open(active_path) as f:
+                with active_path.open() as f:
                     data = json.load(f)
                     self.active_modifications = {
                         k: AgentModification.from_dict(v) for k, v in data.items()
@@ -669,7 +677,7 @@ class AgentModificationTracker(BaseService):
 
             # Load modification history
             for history_file in self.history_root.glob("*.json"):
-                with open(history_file) as f:
+                with history_file.open() as f:
                     data = json.load(f)
                     agent_name = data["agent_name"]
                     history = ModificationHistory(agent_name=agent_name)
@@ -699,7 +707,7 @@ class AgentModificationTracker(BaseService):
             active_data = {k: v.to_dict() for k, v in self.active_modifications.items()}
             active_path = self.persistence_root / "active_modifications.json"
 
-            with open(active_path, "w") as f:
+            with active_path.open("w") as f:
                 json.dump(active_data, f, indent=2)
 
             # Save modification history
@@ -714,7 +722,7 @@ class AgentModificationTracker(BaseService):
                 }
 
                 history_path = self.history_root / f"{agent_name}_history.json"
-                with open(history_path, "w") as f:
+                with history_path.open("w") as f:
                     json.dump(history_data, f, indent=2)
 
             self.logger.debug("Saved modification history to disk")
@@ -767,7 +775,7 @@ class AgentModificationTracker(BaseService):
                 if backup_dir.is_dir():
                     metadata_path = backup_dir / "metadata.json"
                     if metadata_path.exists():
-                        with open(metadata_path) as f:
+                        with metadata_path.open() as f:
                             metadata = json.load(f)
                             if metadata.get("backup_time", 0) < cutoff_time:
                                 shutil.rmtree(backup_dir)

claude_mpm/services/async_session_logger.py

@@ -190,7 +190,7 @@ class AsyncSessionLogger:
             self._initialized = True
 
             # Log initialization status
-            logger.info(
+            logger.debug(
                 f"AsyncSessionLogger initialized with SessionManager: session_id={self.session_id}, async={self.enable_async}, format={self.log_format.value}"
             )
 
@@ -336,7 +336,7 @@ class AsyncSessionLogger:
             with gzip.open(file_path, "wt", encoding="utf-8") as f:
                 json.dump(data, f, indent=2, ensure_ascii=False)
         else:
-            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)
 
         logger.debug(f"Wrote log entry to {file_path}")
@@ -537,7 +537,7 @@ class AsyncSessionLogger:
         # Also update SessionManager to keep consistency
         session_manager = get_session_manager()
         session_manager.set_session_id(session_id)
-        logger.info(f"Session ID updated to: {session_id}")
+        logger.debug(f"Session ID updated to: {session_id}")
 
     def is_enabled(self) -> bool:
         """Check if logging is enabled."""

claude_mpm/services/claude_session_logger.py

@@ -88,7 +88,7 @@ class ClaudeSessionLogger:
             # Use centralized SessionManager for session ID
             session_manager = get_session_manager()
             self.session_id = session_manager.get_session_id()
-            logger.info(
+            logger.debug(
                 f"ClaudeSessionLogger using session ID from SessionManager: {self.session_id}"
             )
 
@@ -125,11 +125,11 @@ class ClaudeSessionLogger:
                 # Synchronize session IDs - use the one we already generated
                 if self.session_id and hasattr(self._async_logger, "set_session_id"):
                     self._async_logger.set_session_id(self.session_id)
-                    logger.info(
+                    logger.debug(
                         f"Using async logger with session ID: {self.session_id}"
                     )
                 else:
-                    logger.info("Using async logger for improved performance")
+                    logger.debug("Using async logger for improved performance")
             except Exception as e:
                 logger.warning(
                     f"Failed to initialize async logger, falling back to sync: {e}"
@@ -224,7 +224,7 @@ class ClaudeSessionLogger:
 
         # Save response
         try:
-            with open(file_path, "w", encoding="utf-8") as f:
+            with file_path.open("w", encoding="utf-8") as f:
                 json.dump(response_data, f, indent=2, ensure_ascii=False)
 
             logger.debug(f"Logged response to {filename} for session {self.session_id}")

claude_mpm/services/cli/agent_listing_service.py

@@ -169,8 +169,8 @@ class AgentListingService(IAgentListingService):
 
                 base_service = AgentDeploymentService()
                 self._deployment_service = DeploymentServiceWrapper(base_service)
-            except ImportError:
-                raise ImportError("Agent deployment service not available")
+            except ImportError as e:
+                raise ImportError("Agent deployment service not available") from e
         return self._deployment_service
 
     @property
@@ -364,7 +364,7 @@ class AgentListingService(IAgentListingService):
             if not agent_path.exists():
                 return None
 
-            with open(agent_path) as f:
+            with agent_path.open() as f:
                 content = f.read()
 
             details = {

claude_mpm/services/cli/agent_validation_service.py

@@ -72,7 +72,7 @@ class AgentValidationService(IAgentValidationService):
                 self._registry = adapter.registry
             except Exception as e:
                 self.logger.error(f"Failed to initialize agent registry: {e}")
-                raise RuntimeError(f"Could not initialize agent registry: {e}")
+                raise RuntimeError(f"Could not initialize agent registry: {e}") from e
         return self._registry
 
     def validate_agent(self, agent_name: str) -> Dict[str, Any]:

claude_mpm/services/cli/session_manager.py

@@ -449,7 +449,7 @@ class SessionManager(ISessionManager):
             sessions_dict = {
                 sid: session.to_dict() for sid, session in self._sessions_cache.items()
             }
-            with open(session_file, "w") as f:
+            with session_file.open("w") as f:
                 json.dump(sessions_dict, f, indent=2)
         except Exception as e:
             self.logger.error(f"Failed to save sessions: {e}")
@@ -464,7 +464,7 @@ class SessionManager(ISessionManager):
             return
 
         try:
-            with open(session_file) as f:
+            with session_file.open() as f:
                 sessions_dict = json.load(f)
 
             self._sessions_cache = {

claude_mpm/services/core/path_resolver.py

@@ -166,7 +166,7 @@ class PathResolver(IPathResolver):
         self.logger.debug(f"No project root found from {start_path}")
         return None
 
-    def detect_framework_path(self) -> Optional[Path]:  # noqa: PLR0911
+    def detect_framework_path(self) -> Optional[Path]:
         """
         Auto-detect claude-mpm framework using unified path management.
 

claude_mpm/services/diagnostics/checks/agent_check.py

@@ -289,7 +289,7 @@ class AgentCheck(BaseDiagnosticCheck):
 
                 # Basic validation
                 try:
-                    with open(agent_file) as f:
+                    with agent_file.open() as f:
                         content = f.read()
 
                         # Check for required sections

claude_mpm/services/diagnostics/checks/claude_code_check.py

@@ -187,7 +187,7 @@ class ClaudeCodeCheck(BaseDiagnosticCheck):
 
         # Check if it's up to date
         try:
-            with open(style_path) as f:
+            with style_path.open() as f:
                 content = f.read()
                 if "Claude MPM Output Style" in content:
                     return DiagnosticResult(
@@ -232,7 +232,7 @@ class ClaudeCodeCheck(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/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: