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

claude_mpm/services/agents/agent_builder.py

@@ -287,7 +287,7 @@ class AgentBuilderService:
 
         for template_file in self.templates_dir.glob("*.json"):
             try:
-                with open(template_file) as f:
+                with template_file.open() as f:
                     config = json.load(f)
 
                 # Use filename stem as ID if not specified in config
@@ -377,12 +377,14 @@ class AgentBuilderService:
             raise AgentDeploymentError(f"Template '{template_id}' not found")
 
         try:
-            with open(template_file) as f:
+            with template_file.open() as f:
                 config = json.load(f)
                 self._template_cache[template_id] = config
                 return config.copy()
         except Exception as e:
-            raise AgentDeploymentError(f"Failed to load template '{template_id}': {e}")
+            raise AgentDeploymentError(
+                f"Failed to load template '{template_id}': {e}"
+            ) from e
 
     def _load_instructions(self, agent_id: str) -> str:
         """Load agent instructions.
@@ -406,7 +408,7 @@ class AgentBuilderService:
         for instructions_file in possible_files:
             if instructions_file.exists():
                 try:
-                    with open(instructions_file) as f:
+                    with instructions_file.open() as f:
                         return f.read()
                 except Exception as e:
                     self.logger.warning(

claude_mpm/services/agents/deployment/agent_lifecycle_manager.py

@@ -272,7 +272,7 @@ class AgentLifecycleManager(BaseService):
             # First convert to JSON string with custom encoder, then save
             json_str = json.dumps(data, indent=2, default=str)
             records_file.parent.mkdir(parents=True, exist_ok=True)
-            with open(records_file, "w", encoding="utf-8") as f:
+            with records_file.open("w", encoding="utf-8") as f:
                 f.write(json_str)
 
             self.logger.debug(f"Saved {len(self.agent_records)} agent records")

claude_mpm/services/agents/deployment/agent_metrics_collector.py

@@ -227,7 +227,7 @@ class AgentMetricsCollector:
             return "validation_error"
         return "other_error"
 
-    def _extract_agent_type(self, agent_name: str) -> str:  # noqa: PLR0911
+    def _extract_agent_type(self, agent_name: str) -> str:
         """
         Extract agent type from agent name for categorization.
 

claude_mpm/services/agents/deployment/agent_record_service.py

@@ -80,7 +80,7 @@ class AgentRecordService(BaseService):
             json_str = json.dumps(data, indent=2, default=str)
             self.records_file.parent.mkdir(parents=True, exist_ok=True)
 
-            with open(self.records_file, "w", encoding="utf-8") as f:
+            with self.records_file.open("w", encoding="utf-8") as f:
                 f.write(json_str)
 
             self.logger.debug(f"Saved {len(records)} agent records")
@@ -149,7 +149,7 @@ class AgentRecordService(BaseService):
             json_str = json.dumps(data, indent=2, default=str)
             self.history_file.parent.mkdir(parents=True, exist_ok=True)
 
-            with open(self.history_file, "w", encoding="utf-8") as f:
+            with self.history_file.open("w", encoding="utf-8") as f:
                 f.write(json_str)
 
             self.logger.debug(f"Saved {len(history)} operation history entries")
@@ -255,7 +255,7 @@ class AgentRecordService(BaseService):
 
                 # Write to output path
                 json_str = json.dumps(data, indent=2, default=str)
-                with open(output_path, "w", encoding="utf-8") as f:
+                with output_path.open("w", encoding="utf-8") as f:
                     f.write(json_str)
 
             elif format == "csv":

claude_mpm/services/agents/deployment/deployment_wrapper.py

@@ -91,7 +91,7 @@ class DeploymentServiceWrapper:
 
                     # Read agent content if file exists
                     if agent_path.exists():
-                        with open(agent_path) as f:
+                        with agent_path.open() as f:
                             content = f.read()
 
                         # Parse metadata from content

claude_mpm/services/agents/deployment/pipeline/steps/target_directory_step.py

@@ -48,10 +48,10 @@ class TargetDirectorySetupStep(BaseDeploymentStep):
             try:
                 test_file.write_text("test")
                 test_file.unlink()
-            except Exception:
+            except Exception as e:
                 raise PermissionError(
                     f"Target directory is not writable: {context.actual_target_dir}"
-                )
+                ) from e
 
             self.logger.info(f"Target directory set up: {context.actual_target_dir}")
 

claude_mpm/services/agents/loading/agent_profile_loader.py

@@ -457,7 +457,7 @@ class AgentProfileLoader(BaseService):
 
         if prompt_file.exists():
             try:
-                with open(prompt_file) as f:
+                with prompt_file.open() as f:
                     data = json.load(f)
                     for prompt_data in data:
                         prompt = ImprovedPrompt(
@@ -500,7 +500,7 @@ class AgentProfileLoader(BaseService):
 
             # Save to file
             prompt_file = self.improved_prompts_path / f"{agent_name}_prompts.json"
-            with open(prompt_file, "w") as f:
+            with prompt_file.open("w") as f:
                 json.dump(
                     [
                         {

claude_mpm/services/agents/local_template_manager.py

@@ -195,7 +195,7 @@ class LocalAgentTemplateManager:
         """
         for template_file in directory.glob("*.json"):
             try:
-                with open(template_file) as f:
+                with template_file.open() as f:
                     data = json.load(f)
 
                 # Create LocalAgentTemplate
@@ -307,7 +307,7 @@ class LocalAgentTemplateManager:
 
         # Save to JSON file
         template_file = target_dir / f"{template.agent_id}.json"
-        with open(template_file, "w") as f:
+        with template_file.open("w") as f:
             json.dump(template.to_json(), f, indent=2)
 
         # Invalidate cache
@@ -628,7 +628,7 @@ class LocalAgentTemplateManager:
 
         # Save current version
         old_version_file = versions_dir / f"{template.agent_version}.json"
-        with open(old_version_file, "w") as f:
+        with old_version_file.open("w") as f:
             json.dump(template.to_json(), f, indent=2)
 
         # Update template version
@@ -691,7 +691,7 @@ class LocalAgentTemplateManager:
         count = 0
         for agent_id, template in templates.items():
             output_file = output_dir / f"{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)
             count += 1
 
@@ -715,7 +715,7 @@ class LocalAgentTemplateManager:
         count = 0
         for template_file in input_dir.glob("*.json"):
             try:
-                with open(template_file) as f:
+                with template_file.open() as f:
                     data = json.load(f)
 
                 template = LocalAgentTemplate.from_json(data)

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

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

claude_mpm/services/claude_session_logger.py

@@ -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,17 @@ 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