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

claude_mpm/hooks/claude_hooks/installer.py

@@ -428,7 +428,7 @@ main "$@"
             return
 
         try:
-            with open(self.old_settings_file) as f:
+            with self.old_settings_file.open() as f:
                 old_settings = json.load(f)
 
             # Remove hooks section if present
@@ -437,7 +437,7 @@ main "$@"
                 self.logger.info(f"Removing hooks from {self.old_settings_file}")
 
                 # Write back the cleaned settings
-                with open(self.old_settings_file, "w") as f:
+                with self.old_settings_file.open("w") as f:
                     json.dump(old_settings, f, indent=2)
 
                 self.logger.info(f"Cleaned up hooks from {self.old_settings_file}")
@@ -450,7 +450,7 @@ main "$@"
 
         # Load existing settings.json 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.info(f"Found existing Claude settings at {self.settings_file}")
         else:
@@ -490,7 +490,7 @@ main "$@"
             ]
 
         # Write settings to settings.json
-        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}")
@@ -564,7 +564,7 @@ main "$@"
             issues.append(f"Claude settings file not found at {self.settings_file}")
         else:
             try:
-                with open(self.settings_file) as f:
+                with self.settings_file.open() as f:
                     settings = json.load(f)
 
                 if "hooks" not in settings:
@@ -615,7 +615,7 @@ main "$@"
             # Remove from Claude settings (both old and new locations)
             for settings_path in [self.settings_file, self.old_settings_file]:
                 if settings_path.exists():
-                    with open(settings_path) as f:
+                    with settings_path.open() as f:
                         settings = json.load(f)
 
                     if "hooks" in settings:
@@ -655,7 +655,7 @@ main "$@"
                             del settings["hooks"]
 
                         # Write back settings
-                        with open(settings_path, "w") as f:
+                        with settings_path.open("w") as f:
                             json.dump(settings, f, indent=2)
 
                         self.logger.info(f"Removed hooks from {settings_path}")
@@ -709,7 +709,7 @@ main "$@"
 
         if self.settings_file.exists():
             try:
-                with open(self.settings_file) as f:
+                with self.settings_file.open() as f:
                     settings = json.load(f)
                     if "hooks" in settings:
                         status["configured_events"] = list(settings["hooks"].keys())
@@ -720,7 +720,7 @@ main "$@"
         # Also check old settings file
         if self.old_settings_file.exists():
             try:
-                with open(self.old_settings_file) as f:
+                with self.old_settings_file.open() as f:
                     old_settings = json.load(f)
                     if "hooks" in old_settings:
                         status["old_file_has_hooks"] = True

claude_mpm/hooks/claude_hooks/services/connection_manager_http.py

@@ -76,6 +76,9 @@ class ConnectionManagerService:
         # For backward compatibility with tests
         self.connection_pool = None  # No longer used
 
+        # Track async emit tasks to prevent garbage collection
+        self._emit_tasks: set = set()
+
         if DEBUG:
             print(
                 f"✅ HTTP connection manager initialized - endpoint: {self.http_endpoint}",
@@ -141,8 +144,10 @@ class ConnectionManagerService:
                 pass
 
             if loop:
-                # We're in an async context, create a task
-                loop.create_task(self._async_emit(namespace, event, data))
+                # We're in an async context, create a task with tracking
+                task = loop.create_task(self._async_emit(namespace, event, data))
+                self._emit_tasks.add(task)
+                task.add_done_callback(self._emit_tasks.discard)
                 # Don't wait for completion to maintain low latency
                 if DEBUG:
                     print(f"✅ Async emit scheduled: {event}", file=sys.stderr)

claude_mpm/hooks/claude_hooks/tool_analysis.py

@@ -138,7 +138,7 @@ def summarize_todos(todos: list) -> dict:
     }
 
 
-def classify_tool_operation(tool_name: str, tool_input: dict) -> str:  # noqa: PLR0911
+def classify_tool_operation(tool_name: str, tool_input: dict) -> str:
     """Classify the type of operation being performed."""
     if tool_name in ["Read", "LS", "Glob", "Grep", "NotebookRead"]:
         return "read"
@@ -155,7 +155,7 @@ def classify_tool_operation(tool_name: str, tool_input: dict) -> str:  # noqa: P
     return "other"
 
 
-def assess_security_risk(tool_name: str, tool_input: dict) -> str:  # noqa: PLR0911
+def assess_security_risk(tool_name: str, tool_input: dict) -> str:
     """Assess the security risk level of the tool operation."""
     if tool_name == "Bash":
         command = tool_input.get("command", "").lower()

claude_mpm/hooks/memory_integration_hook.py

@@ -234,7 +234,7 @@ class MemoryPostDelegationHook(PostDelegationHook):
             "context": "context",  # Current Technical Context
         }
 
-    def execute(self, context: HookContext) -> HookResult:  # noqa: PLR0911
+    def execute(self, context: HookContext) -> HookResult:
         """Extract and store learnings from delegation result.
 
         WHY: Capturing learnings immediately after task completion ensures we

claude_mpm/hooks/validation_hooks.py

@@ -118,7 +118,7 @@ async def validate_agent_dependencies(profile_path: Path) -> ValidationResult:
     result = ValidationResult(is_valid=True)
 
     try:
-        with open(profile_path) as f:
+        with profile_path.open() as f:
             profile_data = yaml.safe_load(f)
 
         # Check for circular dependencies

claude_mpm/init.py

@@ -270,11 +270,11 @@ class ProjectInitializer:
         """
         try:
             # Read existing JSON configuration
-            with open(old_file) as f:
+            with old_file.open() as f:
                 config = json.load(f)
 
             # Write as YAML
-            with open(new_file, "w") as f:
+            with new_file.open("w") as f:
                 yaml.dump(config, f, default_flow_style=False, sort_keys=False)
 
             self.logger.info(
@@ -310,7 +310,7 @@ class ProjectInitializer:
             },
         }
 
-        with open(config_file, "w") as f:
+        with config_file.open("w") as f:
             yaml.dump(default_config, f, default_flow_style=False, sort_keys=False)
 
     def _create_project_config(self, config_file: Path):
@@ -322,7 +322,7 @@ class ProjectInitializer:
             "tickets": {"auto_create": True, "prefix": "TSK"},
         }
 
-        with open(config_file, "w") as f:
+        with config_file.open("w") as f:
             json.dump(project_config, f, indent=2)
 
     def _copy_agent_templates(self):

claude_mpm/models/agent_session.py

@@ -448,7 +448,7 @@ class AgentSession:
         filepath = directory / filename
 
         # Save to file
-        with open(filepath, "w", encoding="utf-8") as f:
+        with filepath.open("w", encoding="utf-8") as f:
             json.dump(self.to_dict(), f, indent=2, ensure_ascii=False)
 
         return str(filepath)

claude_mpm/scripts/socketio_daemon.py

@@ -37,7 +37,7 @@ def is_running(pid_file: Path) -> bool:
         return False
 
     try:
-        with open(pid_file) as f:
+        with pid_file.open() as f:
             pid = int(f.read().strip())
 
         # Check if process exists
@@ -85,7 +85,7 @@ def start_server(port: int = DEFAULT_PORT, daemon: bool = True) -> bool:
         logger.info(f"Socket.IO daemon started on port {actual_port}")
         # Save the port for clients to discover
         port_file = pid_file.parent / "socketio-port"
-        with open(port_file, "w") as f:
+        with port_file.open("w") as f:
             f.write(str(actual_port))
     else:
         logger.error("Failed to start Socket.IO daemon")
@@ -102,7 +102,7 @@ def stop_server() -> bool:
         return False
 
     try:
-        with open(pid_file) as f:
+        with pid_file.open() as f:
             pid = int(f.read().strip())
 
         # Send SIGTERM for graceful shutdown
@@ -155,12 +155,12 @@ def status_server() -> bool:
 
     if is_running(pid_file):
         try:
-            with open(pid_file) as f:
+            with pid_file.open() as f:
                 pid = int(f.read().strip())
 
             port = DEFAULT_PORT
             if port_file.exists():
-                with open(port_file) as f:
+                with port_file.open() as f:
                     port = int(f.read().strip())
 
             print(f"Socket.IO daemon is running (PID: {pid}, Port: {port})")

claude_mpm/services/__init__.py

@@ -144,8 +144,8 @@ def __getattr__(name):
         try:
             module = import_module("claude_mpm.services.recovery_manager")
             return module.RecoveryManager
-        except ImportError:
-            raise AttributeError(f"Recovery management not available: {name}")
+        except ImportError as e:
+            raise AttributeError(f"Recovery management not available: {name}") from e
 
     # Handle MCP interfaces (names starting with "IMCP")
     if name.startswith("IMCP"):

claude_mpm/services/agent_capabilities_service.py

@@ -215,7 +215,7 @@ class AgentCapabilitiesService(BaseService, AgentCapabilitiesInterface):
                 self.logger.debug(f"Could not parse agent {agent_file}: {e}")
                 continue
 
-    def _categorize_agent(self, agent_id: str, content: str) -> str:  # noqa: PLR0911
+    def _categorize_agent(self, agent_id: str, content: str) -> str:
         """Categorize an agent based on its ID and content."""
         agent_id_lower = agent_id.lower()
         content_lower = content.lower()

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,12 @@ 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 +406,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", {})