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

claude_mpm/services/monitor/daemon_manager.py

@@ -302,7 +302,7 @@ class DaemonManager:
             if not self.pid_file.exists():
                 return True
 
-            with open(self.pid_file) as f:
+            with self.pid_file.open() as f:
                 pid = int(f.read().strip())
 
             self.logger.info(f"Found PID {pid} in PID file")
@@ -397,7 +397,7 @@ class DaemonManager:
             # First check PID file
             if self.pid_file.exists():
                 try:
-                    with open(self.pid_file) as f:
+                    with self.pid_file.open() as f:
                         pid = int(f.read().strip())
 
                     # Verify process exists
@@ -604,7 +604,7 @@ class DaemonManager:
                     # Check if PID file was written
                     if self.pid_file.exists():
                         try:
-                            with open(self.pid_file) as f:
+                            with self.pid_file.open() as f:
                                 written_pid = int(f.read().strip())
                             if written_pid == pid:
                                 # PID file written correctly, check port
@@ -798,7 +798,7 @@ class DaemonManager:
             if not self.pid_file.exists():
                 return None
 
-            with open(self.pid_file) as f:
+            with self.pid_file.open() as f:
                 return int(f.read().strip())
 
         except Exception as e:
@@ -809,7 +809,7 @@ class DaemonManager:
         """Write current PID to PID file."""
         try:
             self.pid_file.parent.mkdir(parents=True, exist_ok=True)
-            with open(self.pid_file, "w") as f:
+            with self.pid_file.open("w") as f:
                 f.write(str(os.getpid()))
             self.logger.debug(f"PID file written: {self.pid_file}")
         except Exception as e:
@@ -863,7 +863,7 @@ class DaemonManager:
 
             # Redirect stdout and stderr to log file
             self.log_file.parent.mkdir(parents=True, exist_ok=True)
-            with open(self.log_file, "a") as log_out:
+            with self.log_file.open("a") as log_out:
                 os.dup2(log_out.fileno(), sys.stdout.fileno())
                 os.dup2(log_out.fileno(), sys.stderr.fileno())
 
@@ -903,7 +903,7 @@ class DaemonManager:
                     continue
 
                 try:
-                    with open(self.startup_status_file) as f:
+                    with self.startup_status_file.open() as f:
                         status = f.read().strip()
 
                     if status == "success":
@@ -935,7 +935,7 @@ class DaemonManager:
             try:
                 # Don't check if file exists - we need to write to it regardless
                 # The parent created it and is waiting for us to update it
-                with open(self.startup_status_file, "w") as f:
+                with self.startup_status_file.open("w") as f:
                     f.write("success")
                     f.flush()  # Ensure it's written immediately
                     os.fsync(f.fileno())  # Force write to disk
@@ -948,7 +948,7 @@ class DaemonManager:
         if self.startup_status_file:
             try:
                 # Don't check if file exists - we need to write to it regardless
-                with open(self.startup_status_file, "w") as f:
+                with self.startup_status_file.open("w") as f:
                     f.write(f"error:{error}")
                     f.flush()  # Ensure it's written immediately
                     os.fsync(f.fileno())  # Force write to disk

claude_mpm/services/monitor/handlers/file.py

@@ -225,7 +225,7 @@ class FileHandler:
         be displayed as text.
         """
         try:
-            with open(real_path, "rb") as f:
+            with real_path.open("rb") as f:
                 binary_content = f.read()
 
             # Check if it's a text file by looking for common text patterns

claude_mpm/services/monitor/handlers/hooks.py

@@ -236,10 +236,10 @@ class HookHandler:
                     },
                 )
 
-                # Remove from active sessions after a delay
-                asyncio.create_task(
+                # Remove from active sessions after a delay (5 minutes)
+                _task = asyncio.create_task(  # noqa: RUF006
                     self._cleanup_session(session_id, delay=300)
-                )  # 5 minutes
+                )  # Fire-and-forget cleanup task
 
                 self.logger.info(f"Claude Code session ended: {session_id}")
             else:

claude_mpm/services/monitor/management/lifecycle.py

@@ -127,14 +127,14 @@ class DaemonLifecycle:
             # Redirect stdout and stderr
             if self.log_file:
                 # Redirect to log file
-                with open(self.log_file, "a") as log_out:
+                with self.log_file.open("a") as log_out:
                     os.dup2(log_out.fileno(), sys.stdout.fileno())
                     os.dup2(log_out.fileno(), sys.stderr.fileno())
             else:
                 # Default to a daemon log file instead of /dev/null for errors
                 default_log = Path.home() / ".claude-mpm" / "monitor-daemon.log"
                 default_log.parent.mkdir(parents=True, exist_ok=True)
-                with open(default_log, "a") as log_out:
+                with default_log.open("a") as log_out:
                     os.dup2(log_out.fileno(), sys.stdout.fileno())
                     os.dup2(log_out.fileno(), sys.stderr.fileno())
 
@@ -149,7 +149,7 @@ class DaemonLifecycle:
             self.pid_file.parent.mkdir(parents=True, exist_ok=True)
 
             # Write PID
-            with open(self.pid_file, "w") as f:
+            with self.pid_file.open("w") as f:
                 f.write(str(os.getpid()))
 
             self.logger.debug(f"PID file written: {self.pid_file}")
@@ -199,7 +199,7 @@ class DaemonLifecycle:
             if not self.pid_file.exists():
                 return None
 
-            with open(self.pid_file) as f:
+            with self.pid_file.open() as f:
                 pid_str = f.read().strip()
                 return int(pid_str) if pid_str else None
 
@@ -385,7 +385,7 @@ class DaemonLifecycle:
             try:
                 # Check if status file exists and read it
                 if self.startup_status_file and Path(self.startup_status_file).exists():
-                    with open(self.startup_status_file) as f:
+                    with self.startup_status_file.open() as f:
                         status = f.read().strip()
 
                     if status == "success":
@@ -437,7 +437,7 @@ class DaemonLifecycle:
         """Report successful startup to parent process."""
         if self.startup_status_file:
             try:
-                with open(self.startup_status_file, "w") as f:
+                with self.startup_status_file.open("w") as f:
                     f.write("success")
             except Exception as e:
                 self.logger.error(f"Failed to report startup success: {e}")
@@ -450,7 +450,7 @@ class DaemonLifecycle:
         """
         if self.startup_status_file:
             try:
-                with open(self.startup_status_file, "w") as f:
+                with self.startup_status_file.open("w") as f:
                     f.write(f"error:{error_msg}")
             except Exception:
                 pass  # Can't report if file write fails

claude_mpm/services/monitor/server.py

@@ -313,7 +313,7 @@ class UnifiedMonitorServer:
             async def dashboard_index(request):
                 template_path = dashboard_dir / "templates" / "index.html"
                 if template_path.exists():
-                    with open(template_path) as f:
+                    with template_path.open() as f:
                         content = f.read()
                     return web.Response(text=content, content_type="text/html")
                 return web.Response(text="Dashboard not found", status=404)
@@ -365,7 +365,7 @@ class UnifiedMonitorServer:
                     return web.Response(text=f"Error: {e!s}", status=500)
 
             # File content endpoint for file viewer
-            async def api_file_handler(request):  # noqa: PLR0911
+            async def api_file_handler(request):
                 """Handle file content requests."""
                 import json
                 import os

claude_mpm/services/orphan_detection.py

@@ -199,10 +199,7 @@ class OrphanDetectionService(SyncBaseService):
         Returns:
             True if port is protected
         """
-        for start, end in self.PROTECTED_PORT_RANGES:
-            if start <= port <= end:
-                return True
-        return False
+        return any(start <= port <= end for start, end in self.PROTECTED_PORT_RANGES)
 
     def _get_process_age(self, pid: int) -> Optional[float]:
         """
@@ -262,7 +259,7 @@ class OrphanDetectionService(SyncBaseService):
             if not self.state_file.exists():
                 return orphans
 
-            with open(self.state_file) as f:
+            with self.state_file.open() as f:
                 state = json.load(f)
 
             deployments = state.get("deployments", {})
@@ -308,7 +305,7 @@ class OrphanDetectionService(SyncBaseService):
             if not self.global_registry_file.exists():
                 return orphans
 
-            with open(self.global_registry_file) as f:
+            with self.global_registry_file.open() as f:
                 registry = json.load(f)
 
             allocations = registry.get("allocations", {})
@@ -350,11 +347,9 @@ class OrphanDetectionService(SyncBaseService):
             # Load global registry to know which ports are managed
             managed_ports = set()
             if self.global_registry_file.exists():
-                with open(self.global_registry_file) as f:
+                with self.global_registry_file.open() as f:
                     registry = json.load(f)
-                    managed_ports = set(
-                        int(p) for p in registry.get("allocations", {}).keys()
-                    )
+                    managed_ports = {int(p) for p in registry.get("allocations", {})}
 
             # Scan all network connections
             for conn in psutil.net_connections(kind="inet"):
@@ -557,7 +552,7 @@ class OrphanDetectionService(SyncBaseService):
         # Check project state
         if self.state_file.exists():
             try:
-                with open(self.state_file) as f:
+                with self.state_file.open() as f:
                     state = json.load(f)
 
                 for deployment in state.get("deployments", {}).values():
@@ -585,7 +580,7 @@ class OrphanDetectionService(SyncBaseService):
         # Check project state
         if self.state_file.exists():
             try:
-                with open(self.state_file) as f:
+                with self.state_file.open() as f:
                     state = json.load(f)
 
                 for deployment in state.get("deployments", {}).values():
@@ -667,14 +662,14 @@ class OrphanDetectionService(SyncBaseService):
     def _cleanup_dead_pid(self, orphan: OrphanInfo) -> Tuple[bool, str]:
         """Clean up dead PID entry from state file."""
         try:
-            with open(self.state_file) as f:
+            with self.state_file.open() as f:
                 state = json.load(f)
 
             service_name = orphan.details.get("service_name")
             if service_name in state.get("deployments", {}):
                 del state["deployments"][service_name]
 
-                with open(self.state_file, "w") as f:
+                with self.state_file.open("w") as f:
                     json.dump(state, f, indent=2)
 
                 return True, f"Removed dead PID entry for {service_name}"
@@ -687,14 +682,14 @@ class OrphanDetectionService(SyncBaseService):
     def _cleanup_deleted_project(self, orphan: OrphanInfo) -> Tuple[bool, str]:
         """Clean up deleted project entry from global registry."""
         try:
-            with open(self.global_registry_file) as f:
+            with self.global_registry_file.open() as f:
                 registry = json.load(f)
 
             port = str(orphan.details.get("port"))
             if port in registry.get("allocations", {}):
                 del registry["allocations"][port]
 
-                with open(self.global_registry_file, "w") as f:
+                with self.global_registry_file.open("w") as f:
                     json.dump(registry, f, indent=2)
 
                 return True, f"Removed deleted project entry for port {port}"

claude_mpm/services/port_manager.py

@@ -474,7 +474,7 @@ class PortManager:
         """Load registered instances from file."""
         try:
             if self.instances_file.exists():
-                with open(self.instances_file) as f:
+                with self.instances_file.open() as f:
                     return json.load(f)
         except Exception as e:
             self.logger.warning(f"Failed to load instances file: {e}")
@@ -484,7 +484,7 @@ class PortManager:
     def save_instances(self, instances: Dict) -> None:
         """Save registered instances to file."""
         try:
-            with open(self.instances_file, "w") as f:
+            with self.instances_file.open("w") as f:
                 json.dump(instances, f, indent=2)
         except Exception as e:
             self.logger.error(f"Failed to save instances file: {e}")

claude_mpm/services/project/analyzer.py

@@ -313,7 +313,7 @@ class ProjectAnalyzer(ProjectAnalyzerInterface):
     ) -> None:
         """Parse package.json for Node.js project details."""
         try:
-            with open(package_path) as f:
+            with package_path.open() as f:
                 package_data = json.load(f)
 
             # Extract dependencies
@@ -382,7 +382,7 @@ class ProjectAnalyzer(ProjectAnalyzerInterface):
                             f"TOML parsing not available for {deps_path}"
                         )
                         return
-                with open(deps_path, "rb") as f:
+                with deps_path.open("rb") as f:
                     data = tomllib.load(f)
                 deps = list(data.get("project", {}).get("dependencies", []))
                 deps.extend(
@@ -430,7 +430,7 @@ class ProjectAnalyzer(ProjectAnalyzerInterface):
                 except ImportError:
                     self.logger.warning(f"TOML parsing not available for {cargo_path}")
                     return
-            with open(cargo_path, "rb") as f:
+            with cargo_path.open("rb") as f:
                 cargo_data = tomllib.load(f)
 
             deps = cargo_data.get("dependencies", {})

claude_mpm/services/project/archive_manager.py

@@ -24,7 +24,7 @@ import json
 import re
 import shutil
 import subprocess
-from datetime import datetime, timedelta
+from datetime import datetime, timedelta, timezone
 from pathlib import Path
 from typing import Dict, List, Optional, Tuple
 
@@ -116,7 +116,7 @@ Generated by Claude MPM Archive Manager
 
         try:
             # Generate archive filename
-            timestamp = datetime.now().strftime("%Y-%m-%dT%H-%M-%S")
+            timestamp = datetime.now(timezone.utc).strftime("%Y-%m-%dT%H-%M-%S")
             archive_name = f"{file_path.name}.{timestamp}{file_path.suffix}"
             archive_file_path = self.archive_path / archive_name
 
@@ -130,7 +130,7 @@ Generated by Claude MPM Archive Manager
                 meta_data.update(
                     {
                         "original_path": str(file_path),
-                        "archived_at": datetime.now().isoformat(),
+                        "archived_at": datetime.now(timezone.utc).isoformat(),
                         "reason": reason or "Manual archive",
                         "file_size": file_path.stat().st_size,
                         "file_hash": self._calculate_file_hash(file_path),
@@ -152,7 +152,7 @@ Generated by Claude MPM Archive Manager
     def _calculate_file_hash(self, file_path: Path) -> str:
         """Calculate MD5 hash of file."""
         hasher = hashlib.md5()
-        with open(file_path, "rb") as f:
+        with file_path.open("rb") as f:
             for chunk in iter(lambda: f.read(4096), b""):
                 hasher.update(chunk)
         return hasher.hexdigest()
@@ -172,7 +172,9 @@ Generated by Claude MPM Archive Manager
                 logger.info(f"Removed old archive: {archive.name}")
 
         # Compress old archives
-        cutoff_compress = datetime.now() - timedelta(days=self.COMPRESS_AFTER_DAYS)
+        cutoff_compress = datetime.now(timezone.utc) - timedelta(
+            days=self.COMPRESS_AFTER_DAYS
+        )
         for archive in archives:
             if archive.suffix != ".gz":
                 mtime = datetime.fromtimestamp(archive.stat().st_mtime)
@@ -180,7 +182,9 @@ Generated by Claude MPM Archive Manager
                     self._compress_archive(archive)
 
         # Delete very old archives
-        cutoff_delete = datetime.now() - timedelta(days=self.DELETE_AFTER_DAYS)
+        cutoff_delete = datetime.now(timezone.utc) - timedelta(
+            days=self.DELETE_AFTER_DAYS
+        )
         for archive in archives:
             mtime = datetime.fromtimestamp(archive.stat().st_mtime)
             if mtime < cutoff_delete:
@@ -204,7 +208,7 @@ Generated by Claude MPM Archive Manager
         try:
             compressed_path = archive_path.with_suffix(archive_path.suffix + ".gz")
 
-            with open(archive_path, "rb") as f_in:
+            with archive_path.open("rb") as f_in:
                 with gzip.open(compressed_path, "wb") as f_out:
                     shutil.copyfileobj(f_in, f_out)
 
@@ -321,7 +325,7 @@ Generated by Claude MPM Archive Manager
             if archive_file.suffix == ".gz":
                 # Decompress first
                 with gzip.open(archive_file, "rb") as f_in:
-                    with open(target_path, "wb") as f_out:
+                    with target_path.open("wb") as f_out:
                         shutil.copyfileobj(f_in, f_out)
             else:
                 shutil.copy2(archive_file, target_path)
@@ -519,7 +523,7 @@ Generated by Claude MPM Archive Manager
     def review_documentation(self, check_git: bool = True) -> Dict:
         """Comprehensive documentation review with outdated detection."""
         report = {
-            "timestamp": datetime.now().isoformat(),
+            "timestamp": datetime.now(timezone.utc).isoformat(),
             "files_reviewed": {},
             "outdated_sections": [],
             "synchronization_issues": [],
@@ -698,7 +702,7 @@ Generated by Claude MPM Archive Manager
         if changelog_path.exists():
             last_modified = self.get_file_last_modified(changelog_path)
             if last_modified:
-                days_old = (datetime.now() - last_modified).days
+                days_old = (datetime.now(timezone.utc) - last_modified).days
                 if days_old > 30:
                     issues.append(
                         {
@@ -800,7 +804,7 @@ Generated by Claude MPM Archive Manager
             # Check if file hasn't been updated in git for long time
             if file_report.get("last_git_update"):
                 last_update = datetime.fromisoformat(file_report["last_git_update"])
-                days_old = (datetime.now() - last_update).days
+                days_old = (datetime.now(timezone.utc) - last_update).days
                 if days_old > 90:
                     should_archive = True
                     archive_reason.append(f"No updates for {days_old} days")
@@ -913,7 +917,7 @@ Generated by Claude MPM Archive Manager
                 changelog_content = changelog_path.read_text()
                 if f"## [{current_version}]" not in changelog_content:
                     # Add new version section
-                    today = datetime.now().strftime("%Y-%m-%d")
+                    today = datetime.now(timezone.utc).strftime("%Y-%m-%d")
                     new_section = f"\n## [{current_version}] - {today}\n\n### Added\n\n### Changed\n\n### Fixed\n\n"
 
                     # Insert after header
@@ -981,7 +985,7 @@ Generated by Claude MPM Archive Manager
             last_updated = "Unknown"
             if report.get("last_git_update"):
                 last_date = datetime.fromisoformat(report["last_git_update"])
-                days_ago = (datetime.now() - last_date).days
+                days_ago = (datetime.now(timezone.utc) - last_date).days
                 last_updated = f"{days_ago} days ago"
 
             table.add_row(filename, status, str(issues), last_updated)

claude_mpm/services/project/dependency_analyzer.py

@@ -279,7 +279,7 @@ class DependencyAnalyzerService:
             package_json_path = self.working_directory / "package.json"
             if package_json_path.exists():
                 try:
-                    with open(package_json_path) as f:
+                    with package_json_path.open() as f:
                         package_data = json.load(f)
                 except Exception as e:
                     self.logger.warning(f"Error reading package.json: {e}")
@@ -324,7 +324,7 @@ class DependencyAnalyzerService:
     ) -> None:
         """Parse package.json for dependencies."""
         try:
-            with open(path) as f:
+            with path.open() as f:
                 data = json.load(f)
 
             # Production dependencies
@@ -382,7 +382,7 @@ class DependencyAnalyzerService:
                     except ImportError:
                         return
 
-                with open(path, "rb") as f:
+                with path.open("rb") as f:
                     data = tomllib.load(f)
 
                 # PEP 621 dependencies
@@ -433,7 +433,7 @@ class DependencyAnalyzerService:
                 except ImportError:
                     return
 
-            with open(path, "rb") as f:
+            with path.open("rb") as f:
                 data = tomllib.load(f)
 
             # Production dependencies

claude_mpm/services/project/documentation_manager.py

@@ -19,7 +19,7 @@ Created: 2025-01-26
 import difflib
 import hashlib
 import re
-from datetime import datetime
+from datetime import datetime, timezone
 from pathlib import Path
 from typing import Dict, List, Tuple
 
@@ -384,7 +384,7 @@ class DocumentationManager:
 
     def _add_metadata(self, content: str) -> str:
         """Add metadata to the document."""
-        timestamp = datetime.now().isoformat()
+        timestamp = datetime.now(timezone.utc).isoformat()
 
         # Check if meta section exists
         if "## 📝 Meta:" not in content and "## Meta:" not in content:
@@ -417,7 +417,7 @@ class DocumentationManager:
     def generate_update_report(self, old_content: str, new_content: str) -> Dict:
         """Generate a report of changes between old and new content."""
         report = {
-            "timestamp": datetime.now().isoformat(),
+            "timestamp": datetime.now(timezone.utc).isoformat(),
             "changes": [],
             "additions": [],
             "deletions": [],
@@ -547,7 +547,7 @@ class DocumentationManager:
 
 ## 📝 Meta: Maintaining This Document
 
-- **Last Updated**: {datetime.now().isoformat()}
+- **Last Updated**: {datetime.now(timezone.utc).isoformat()}
 - **Created By**: Claude MPM /mpm-init
 - **Update Frequency**: As needed when requirements change
 """

claude_mpm/services/project/enhanced_analyzer.py

@@ -17,7 +17,7 @@ Created: 2025-01-26
 """
 
 import subprocess
-from datetime import datetime, timedelta
+from datetime import datetime, timedelta, timezone
 from pathlib import Path
 from typing import Dict, List, Optional
 
@@ -74,7 +74,9 @@ class EnhancedProjectAnalyzer:
 
     def _get_recent_commits(self, days: int) -> List[Dict]:
         """Get recent commits within specified days."""
-        since_date = (datetime.now() - timedelta(days=days)).strftime("%Y-%m-%d")
+        since_date = (datetime.now(timezone.utc) - timedelta(days=days)).strftime(
+            "%Y-%m-%d"
+        )
 
         # Get commit log with structured format
         output = self._run_git_command(
@@ -107,7 +109,9 @@ class EnhancedProjectAnalyzer:
 
     def _get_changed_files(self, days: int) -> Dict:
         """Get files changed in recent commits."""
-        since_date = (datetime.now() - timedelta(days=days)).strftime("%Y-%m-%d")
+        since_date = (datetime.now(timezone.utc) - timedelta(days=days)).strftime(
+            "%Y-%m-%d"
+        )
 
         output = self._run_git_command(
             [
@@ -138,7 +142,9 @@ class EnhancedProjectAnalyzer:
 
     def _get_recently_added_files(self, days: int) -> List[str]:
         """Get files added in recent commits."""
-        since_date = (datetime.now() - timedelta(days=days)).strftime("%Y-%m-%d")
+        since_date = (datetime.now(timezone.utc) - timedelta(days=days)).strftime(
+            "%Y-%m-%d"
+        )
 
         output = self._run_git_command(
             [
@@ -162,7 +168,9 @@ class EnhancedProjectAnalyzer:
 
     def _get_author_stats(self, days: int) -> Dict:
         """Get author contribution statistics."""
-        since_date = (datetime.now() - timedelta(days=days)).strftime("%Y-%m-%d")
+        since_date = (datetime.now(timezone.utc) - timedelta(days=days)).strftime(
+            "%Y-%m-%d"
+        )
 
         output = self._run_git_command(
             [
@@ -228,7 +236,9 @@ class EnhancedProjectAnalyzer:
 
     def _get_documentation_changes(self, days: int) -> Dict:
         """Track changes to documentation files."""
-        since_date = (datetime.now() - timedelta(days=days)).strftime("%Y-%m-%d")
+        since_date = (datetime.now(timezone.utc) - timedelta(days=days)).strftime(
+            "%Y-%m-%d"
+        )
 
         # Get changes to documentation files
         doc_patterns = ["*.md", "*.rst", "*.txt", "docs/*", "README*", "CLAUDE*"]
@@ -448,7 +458,8 @@ class EnhancedProjectAnalyzer:
             )
             if first_commit:
                 age_days = (
-                    datetime.now() - datetime.fromtimestamp(int(first_commit))
+                    datetime.now(timezone.utc)
+                    - datetime.fromtimestamp(int(first_commit))
                 ).days
                 indicators.append(f"{age_days} days old")
 
@@ -470,7 +481,7 @@ class EnhancedProjectAnalyzer:
         """Generate comprehensive project analysis report."""
         report = {
             "project_path": str(self.project_path),
-            "timestamp": datetime.now().isoformat(),
+            "timestamp": datetime.now(timezone.utc).isoformat(),
         }
 
         # Basic project info

claude_mpm/services/project/registry.py

@@ -69,7 +69,7 @@ class ProjectRegistry:
             self.registry_dir.mkdir(parents=True, exist_ok=True)
         except Exception as e:
             self.logger.error(f"Failed to create registry directory: {e}")
-            raise ProjectRegistryError(f"Cannot create registry directory: {e}")
+            raise ProjectRegistryError(f"Cannot create registry directory: {e}") from e
 
     def get_or_create_project_entry(self) -> Dict[str, Any]:
         """
@@ -104,7 +104,7 @@ class ProjectRegistry:
 
         except Exception as e:
             self.logger.error(f"Failed to get or create project entry: {e}")
-            raise ProjectRegistryError(f"Registry operation failed: {e}")
+            raise ProjectRegistryError(f"Registry operation failed: {e}") from e
 
     def _find_existing_entry(self) -> Optional[Dict[str, Any]]:
         """
@@ -491,7 +491,7 @@ class ProjectRegistry:
             # Remove internal fields before saving
             save_data = {k: v for k, v in data.items() if not k.startswith("_")}
 
-            with open(registry_file, "w", encoding="utf-8") as f:
+            with registry_file.open("w", encoding="utf-8") as f:
                 yaml.dump(
                     save_data, f, default_flow_style=False, sort_keys=False, indent=2
                 )
@@ -500,7 +500,7 @@ class ProjectRegistry:
 
         except Exception as e:
             self.logger.error(f"Failed to save registry data: {e}")
-            raise ProjectRegistryError(f"Failed to save registry: {e}")
+            raise ProjectRegistryError(f"Failed to save registry: {e}") from e
 
     def list_projects(self) -> List[Dict[str, Any]]:
         """