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

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,7 @@ 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 +107,7 @@ 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 +138,7 @@ 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 +162,7 @@ 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 +228,7 @@ 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 +448,7 @@ 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 +470,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]]:
         """

claude_mpm/services/project_port_allocator.py

@@ -143,9 +143,8 @@ class ProjectPortAllocator(SyncBaseService):
 
         # Map to port range
         port_range = self.port_range_end - self.port_range_start + 1
-        port = self.port_range_start + (hash_int % port_range)
+        return self.port_range_start + (hash_int % port_range)
 
-        return port
 
     def _is_port_available(self, port: int) -> bool:
         """
@@ -177,10 +176,7 @@ class ProjectPortAllocator(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 _load_project_state(self) -> Dict[str, Any]:
         """
@@ -191,7 +187,7 @@ class ProjectPortAllocator(SyncBaseService):
         """
         try:
             if self.state_file.exists():
-                with open(self.state_file) as f:
+                with self.state_file.open() as f:
                     return json.load(f)
         except Exception as e:
             self.log_warning(f"Failed to load project state: {e}")
@@ -208,7 +204,7 @@ class ProjectPortAllocator(SyncBaseService):
         try:
             # Write to temporary file first
             temp_file = self.state_file.with_suffix(".tmp")
-            with open(temp_file, "w") as f:
+            with temp_file.open("w") as f:
                 json.dump(state, f, indent=2)
 
             # Atomic rename
@@ -227,7 +223,7 @@ class ProjectPortAllocator(SyncBaseService):
         """
         try:
             if self.global_registry_file.exists():
-                with open(self.global_registry_file) as f:
+                with self.global_registry_file.open() as f:
                     return json.load(f)
         except Exception as e:
             self.log_warning(f"Failed to load global registry: {e}")
@@ -247,7 +243,7 @@ class ProjectPortAllocator(SyncBaseService):
 
             # Write to temporary file first
             temp_file = self.global_registry_file.with_suffix(".tmp")
-            with open(temp_file, "w") as f:
+            with temp_file.open("w") as f:
                 json.dump(registry, f, indent=2)
 
             # Atomic rename
@@ -507,7 +503,7 @@ class ProjectPortAllocator(SyncBaseService):
             state_file = project_path / ".claude-mpm" / self.STATE_FILE_NAME
             if state_file.exists():
                 try:
-                    with open(state_file) as f:
+                    with state_file.open() as f:
                         project_state = json.load(f)
 
                     # Check if service still registered

claude_mpm/services/session_management_service.py

@@ -187,7 +187,7 @@ class SessionManagementService(BaseService, SessionManagementInterface):
             event_data["timestamp"] = datetime.now(timezone.utc).isoformat()
 
             # Append to log file as JSONL
-            with open(log_file, "a") as f:
+            with log_file.open("a") as f:
                 f.write(json.dumps(event_data) + "\n")
 
         except Exception as e:

claude_mpm/services/socketio/event_normalizer.py

@@ -342,7 +342,7 @@ class EventNormalizer:
 
         return "unknown"
 
-    def _map_event_name(self, event_name: str) -> Tuple[str, str]:  # noqa: PLR0911
+    def _map_event_name(self, event_name: str) -> Tuple[str, str]:
         """Map event name to (type, subtype) tuple.
 
         WHY: Consistent categorization helps clients filter and handle events.

claude_mpm/services/socketio/handlers/code_analysis.py

@@ -41,6 +41,14 @@ class CodeAnalysisEventHandler(BaseEventHandler):
         self.logger = get_logger(__name__)
         self.analysis_runner = None
         self.code_analyzer = None  # For lazy loading operations
+        self._emit_tasks: set = set()  # Track emit tasks to prevent GC
+
+    def _create_emit_task(self, coro):
+        """Create a tracked emit task to prevent garbage collection."""
+        task = asyncio.get_event_loop().create_task(coro)
+        self._emit_tasks.add(task)
+        task.add_done_callback(self._emit_tasks.discard)
+        return task
 
     def initialize(self):
         """Initialize the analysis runner."""
@@ -246,16 +254,14 @@ class CodeAnalysisEventHandler(BaseEventHandler):
                     # Special handling for 'info' events - they should be passed through directly
                     if event_type == "info":
                         # INFO events for granular tracking
-                        loop = asyncio.get_event_loop()
-                        loop.create_task(
+                        self._create_emit_task(
                             self.server.core.sio.emit(
                                 "info", {"request_id": request_id, **event_data}
                             )
                         )
                     else:
                         # Regular code analysis events
-                        loop = asyncio.get_event_loop()
-                        loop.create_task(
+                        self._create_emit_task(
                             self.server.core.sio.emit(
                                 event_type, {"request_id": request_id, **event_data}
                             )
@@ -437,16 +443,14 @@ class CodeAnalysisEventHandler(BaseEventHandler):
                     # Special handling for 'info' events - they should be passed through directly
                     if event_type == "info":
                         # INFO events for granular tracking
-                        loop = asyncio.get_event_loop()
-                        loop.create_task(
+                        self._create_emit_task(
                             self.server.core.sio.emit(
                                 "info", {"request_id": request_id, **event_data}
                             )
                         )
                     else:
                         # Regular code analysis events
-                        loop = asyncio.get_event_loop()
-                        loop.create_task(
+                        self._create_emit_task(
                             self.server.core.sio.emit(
                                 event_type, {"request_id": request_id, **event_data}
                             )
@@ -609,16 +613,14 @@ class CodeAnalysisEventHandler(BaseEventHandler):
                     # Special handling for 'info' events - they should be passed through directly
                     if event_type == "info":
                         # INFO events for granular tracking
-                        loop = asyncio.get_event_loop()
-                        loop.create_task(
+                        self._create_emit_task(
                             self.server.core.sio.emit(
                                 "info", {"request_id": request_id, **event_data}
                             )
                         )
                     else:
                         # Regular code analysis events
-                        loop = asyncio.get_event_loop()
-                        loop.create_task(
+                        self._create_emit_task(
                             self.server.core.sio.emit(
                                 event_type, {"request_id": request_id, **event_data}
                             )

claude_mpm/services/socketio/handlers/file.py

@@ -224,7 +224,7 @@ class FileEventHandler(BaseEventHandler):
         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/socketio/migration_utils.py

@@ -223,7 +223,7 @@ class EventTypeMapper:
     }
 
     @classmethod
-    def map_event_type(cls, old_type: str) -> Tuple[str, str]:  # noqa: PLR0911
+    def map_event_type(cls, old_type: str) -> Tuple[str, str]:
         """Map an old event type to new type/subtype.
 
         WHY: Provides consistent categorization for all events.

claude_mpm/services/socketio/server/core.py

@@ -428,7 +428,7 @@ class SocketIOServerCore:
         )
 
         # Add file reading endpoint for source viewer
-        async def file_read_handler(request):  # noqa: PLR0911
+        async def file_read_handler(request):
             """Handle GET /api/file/read for reading source files."""
             import os
 

claude_mpm/services/unified/analyzer_strategies/code_analyzer.py

@@ -305,7 +305,7 @@ class CodeAnalyzerStrategy(AnalyzerStrategy):
                 tree = ast.parse(content)
                 complexity["cyclomatic"] = self._calculate_cyclomatic_complexity(tree)
                 complexity["cognitive"] = self._calculate_cognitive_complexity(tree)
-            except:
+            except (SyntaxError, ValueError):
                 pass
 
         return complexity

claude_mpm/services/unified/analyzer_strategies/dependency_analyzer.py

@@ -372,7 +372,7 @@ class DependencyAnalyzerStrategy(AnalyzerStrategy):
     def _parse_package_json(self, path: Path) -> Dict[str, Any]:
         """Parse package.json file."""
         try:
-            with open(path) as f:
+            with path.open() as f:
                 data = json.load(f)
 
             return {
@@ -417,7 +417,7 @@ class DependencyAnalyzerStrategy(AnalyzerStrategy):
                 return {}
 
         try:
-            with open(path, "rb") as f:
+            with path.open("rb") as f:
                 data = tomllib.load(f)
 
             dependencies = {}
@@ -459,7 +459,7 @@ class DependencyAnalyzerStrategy(AnalyzerStrategy):
                 return {}
 
         try:
-            with open(path, "rb") as f:
+            with path.open("rb") as f:
                 data = tomllib.load(f)
 
             return {
@@ -481,7 +481,7 @@ class DependencyAnalyzerStrategy(AnalyzerStrategy):
                 return {}
 
         try:
-            with open(path, "rb") as f:
+            with path.open("rb") as f:
                 data = tomllib.load(f)
 
             return {

claude_mpm/services/unified/analyzer_strategies/structure_analyzer.py

@@ -292,7 +292,7 @@ class StructureAnalyzerStrategy(AnalyzerStrategy):
                     # Get file info
                     try:
                         size = item.stat().st_size
-                    except:
+                    except (OSError, PermissionError):
                         size = 0
 
                     child_node = {

claude_mpm/services/unified/config_strategies/config_schema.py

@@ -471,7 +471,7 @@ class SchemaValidator:
         try:
             datetime.strptime(value, format)
             return True
-        except:
+        except (ValueError, TypeError):
             return False
 
     def _validate_uuid(self, value: str) -> bool:
@@ -481,7 +481,7 @@ class SchemaValidator:
         try:
             uuid.UUID(value)
             return True
-        except:
+        except (ValueError, TypeError, AttributeError):
             return False
 
     def _validate_ipv4(self, value: str) -> bool:
@@ -491,7 +491,7 @@ class SchemaValidator:
         try:
             ipaddress.IPv4Address(value)
             return True
-        except:
+        except (ValueError, TypeError, ipaddress.AddressValueError):
             return False
 
     def _validate_ipv6(self, value: str) -> bool:
@@ -501,7 +501,7 @@ class SchemaValidator:
         try:
             ipaddress.IPv6Address(value)
             return True
-        except:
+        except (ValueError, TypeError, ipaddress.AddressValueError):
             return False
 
     def _validate_semver(self, value: str) -> bool:

claude_mpm/services/unified/config_strategies/context_strategy.py

@@ -9,7 +9,7 @@ from abc import ABC, abstractmethod
 from collections import OrderedDict
 from contextlib import contextmanager
 from dataclasses import dataclass, field
-from datetime import datetime, timedelta
+from datetime import datetime, timedelta, timezone
 from enum import Enum
 from pathlib import Path
 from typing import Any, Callable, Dict, List, Optional, Set, Union
@@ -124,11 +124,11 @@ class HierarchicalContextManager(BaseContextManager):
                 id=context_id,
                 scope=scope,
                 lifecycle=ContextLifecycle.CREATED,
-                created_at=datetime.now(),
-                updated_at=datetime.now(),
+                created_at=datetime.now(timezone.utc),
+                updated_at=datetime.now(timezone.utc),
                 parent_id=parent_id,
                 ttl=ttl,
-                expires_at=datetime.now() + ttl if ttl else None,
+                expires_at=datetime.now(timezone.utc) + ttl if ttl else None,
                 attributes=kwargs,
             )
 
@@ -158,7 +158,7 @@ class HierarchicalContextManager(BaseContextManager):
 
             # Check expiration
             if context and context.expires_at:
-                if datetime.now() > context.expires_at:
+                if datetime.now(timezone.utc) > context.expires_at:
                     self.close_context(context_id)
                     return None
 
@@ -299,7 +299,7 @@ class ScopedConfigManager:
 
             # Update context metadata
             if context_id in self.context_manager.contexts:
-                self.context_manager.contexts[context_id].updated_at = datetime.now()
+                self.context_manager.contexts[context_id].updated_at = datetime.now(timezone.utc)
 
     def _get_inherited_config(self, context_id: str) -> Dict[str, Any]:
         """Get merged configuration from context hierarchy"""

claude_mpm/services/unified/config_strategies/error_handling_strategy.py

@@ -206,7 +206,7 @@ class FileIOErrorHandler(BaseErrorHandler):
                 result.fallback_value = str(alt_path)
                 result.actions_taken.append(f"Using alternative location: {alt_path}")
 
-            except:
+            except (OSError, PermissionError):
                 result.should_escalate = True
 
         # Use read-only mode if applicable
@@ -293,7 +293,7 @@ class ParsingErrorHandler(BaseErrorHandler):
                 result.actions_taken.append(f"Fixed JSON with {fix_func.__name__}")
                 self.logger.info(f"Recovered from JSON error using {fix_func.__name__}")
                 return result
-            except:
+            except (json.JSONDecodeError, ValueError, TypeError):
                 continue
 
         # Use lenient parser if available
@@ -346,7 +346,7 @@ class ParsingErrorHandler(BaseErrorHandler):
             result.recovered = True
             result.fallback_value = parsed
             result.actions_taken.append("Parsed as Python literal")
-        except:
+        except (ValueError, SyntaxError, TypeError):
             # Return empty dict as last resort
             result.recovered = True
             result.fallback_value = {}
@@ -370,7 +370,7 @@ class ParsingErrorHandler(BaseErrorHandler):
             result.fallback_value = parsed
             result.actions_taken.append("Parsed with safe YAML loader")
 
-        except:
+        except (yaml.YAMLError, ValueError, AttributeError):
             # Try to fix tabs
             content = content.replace("\t", "    ")
             try:
@@ -378,7 +378,7 @@ class ParsingErrorHandler(BaseErrorHandler):
                 result.recovered = True
                 result.fallback_value = parsed
                 result.actions_taken.append("Fixed YAML tabs")
-            except:
+            except (yaml.YAMLError, ValueError, AttributeError):
                 result.fallback_value = {}
                 result.actions_taken.append("Used empty configuration as fallback")
 
@@ -406,7 +406,7 @@ class ParsingErrorHandler(BaseErrorHandler):
                     result.fallback_value = parsed
                     result.actions_taken.append(f"Parsed as {format_name}")
                     return result
-            except:
+            except (ValueError, TypeError, AttributeError, ImportError):
                 continue
 
         # Use default/empty config
@@ -533,7 +533,7 @@ class ValidationErrorHandler(BaseErrorHandler):
                 if isinstance(value, str):
                     return json.loads(value)
                 return dict(value)
-        except:
+        except (ValueError, TypeError, json.JSONDecodeError):
             return None
 
     def _handle_generic_validation(
@@ -661,7 +661,7 @@ class TypeConversionErrorHandler(BaseErrorHandler):
         if converter:
             try:
                 return converter(value)
-            except:
+            except (ValueError, TypeError, AttributeError):
                 pass
 
         return None
@@ -706,7 +706,7 @@ class TypeConversionErrorHandler(BaseErrorHandler):
             if value.startswith("["):
                 try:
                     return json.loads(value)
-                except:
+                except (json.JSONDecodeError, ValueError):
                     pass
             # Try comma-separated
             return [v.strip() for v in value.split(",")]
@@ -720,7 +720,7 @@ class TypeConversionErrorHandler(BaseErrorHandler):
             # Try JSON object
             try:
                 return json.loads(value)
-            except:
+            except (json.JSONDecodeError, ValueError):
                 pass
             # Try key=value pairs
             result = {}

claude_mpm/services/unified/config_strategies/file_loader_strategy.py

@@ -75,7 +75,7 @@ class BaseFileLoader(ABC):
                             f"Read {path} with fallback encoding: {enc}"
                         )
                         return f.read()
-                except:
+                except (UnicodeDecodeError, OSError):
                     continue
             raise
 
@@ -185,8 +185,8 @@ class StructuredFileLoader(BaseFileLoader):
                 import tomli
 
                 return tomli.loads(content)
-            except ImportError:
-                raise ImportError("Neither toml nor tomli package is installed")
+            except ImportError as e:
+                raise ImportError("Neither toml nor tomli package is installed") from e
         except Exception as e:
             if context.strict:
                 raise
@@ -209,7 +209,7 @@ class StructuredFileLoader(BaseFileLoader):
 
         try:
             return json.loads(content)
-        except:
+        except (json.JSONDecodeError, ValueError):
             return {}
 
     def _process_includes(
@@ -411,7 +411,7 @@ class EnvironmentFileLoader(BaseFileLoader):
         if value.startswith(("[", "{")):
             try:
                 return json.loads(value)
-            except:
+            except (json.JSONDecodeError, ValueError):
                 pass
 
         # Comma-separated list