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

claude_mpm/services/project_port_allocator.py

@@ -143,9 +143,7 @@ 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 port
+        return self.port_range_start + (hash_int % port_range)
 
     def _is_port_available(self, port: int) -> bool:
         """
@@ -177,10 +175,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 +186,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 +203,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 +222,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 +242,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 +502,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,9 @@ 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

claude_mpm/services/unified/config_strategies/unified_config_service.py

@@ -12,7 +12,7 @@ import threading
 from abc import ABC, abstractmethod
 from collections import defaultdict
 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, TypeVar, Union
@@ -216,7 +216,7 @@ class UnifiedConfigService:
                     source=str(source),
                     format=format,
                     context=context,
-                    loaded_at=datetime.now(),
+                    loaded_at=datetime.now(timezone.utc),
                     checksum=self._calculate_checksum(config),
                     hot_reload=hot_reload,
                     ttl=ttl,
@@ -439,7 +439,7 @@ class UnifiedConfigService:
 
         path = Path(source)
         if path.exists():
-            with open(path) as f:
+            with path.open() as f:
                 return json.load(f)
 
         # Try to parse as JSON string
@@ -452,7 +452,7 @@ class UnifiedConfigService:
 
         path = Path(source)
         if path.exists():
-            with open(path) as f:
+            with path.open() as f:
                 return yaml.safe_load(f)
 
         # Try to parse as YAML string
@@ -476,7 +476,7 @@ class UnifiedConfigService:
             # Try to parse value
             try:
                 config[clean_key] = json.loads(value)
-            except:
+            except (json.JSONDecodeError, ValueError):
                 config[clean_key] = value
 
         return config
@@ -508,7 +508,7 @@ class UnifiedConfigService:
 
         path = Path(source)
         if path.exists():
-            with open(path) as f:
+            with path.open() as f:
                 return toml.load(f)
 
         return toml.loads(str(source))
@@ -706,7 +706,7 @@ class UnifiedConfigService:
         """Check if cached configuration is still valid"""
         if metadata.ttl:
             expiry = metadata.loaded_at + metadata.ttl
-            if datetime.now() > expiry:
+            if datetime.now(timezone.utc) > expiry:
                 return False
         return True
 
@@ -742,7 +742,7 @@ class UnifiedConfigService:
                 result = handler(error, source, context)
                 if result is not None:
                     return result
-            except:
+            except Exception:
                 continue
 
         self.logger.error(f"Failed to load config from {source}: {error}")

claude_mpm/services/unified/config_strategies/validation_strategy.py

@@ -402,7 +402,7 @@ class FormatValidator(BaseValidator):
         try:
             result = urllib.parse.urlparse(value)
             return all([result.scheme, result.netloc])
-        except:
+        except (ValueError, AttributeError, TypeError):
             return False
 
     @staticmethod
@@ -410,7 +410,7 @@ class FormatValidator(BaseValidator):
         try:
             result = urllib.parse.urlparse(value)
             return bool(result.scheme)
-        except:
+        except (ValueError, AttributeError, TypeError):
             return False
 
     @staticmethod
@@ -420,7 +420,7 @@ class FormatValidator(BaseValidator):
         try:
             uuid.UUID(value)
             return True
-        except:
+        except (ValueError, AttributeError, TypeError):
             return False
 
     @staticmethod
@@ -428,7 +428,7 @@ class FormatValidator(BaseValidator):
         try:
             ipaddress.IPv4Address(value)
             return True
-        except:
+        except (ValueError, ipaddress.AddressValueError):
             return False
 
     @staticmethod
@@ -436,7 +436,7 @@ class FormatValidator(BaseValidator):
         try:
             ipaddress.IPv6Address(value)
             return True
-        except:
+        except (ValueError, ipaddress.AddressValueError):
             return False
 
     @staticmethod
@@ -444,7 +444,7 @@ class FormatValidator(BaseValidator):
         try:
             ipaddress.ip_address(value)
             return True
-        except:
+        except (ValueError, ipaddress.AddressValueError):
             return False
 
     @staticmethod
@@ -461,7 +461,7 @@ class FormatValidator(BaseValidator):
         try:
             datetime.strptime(value, "%Y-%m-%d")
             return True
-        except:
+        except (ValueError, TypeError):
             return False
 
     @staticmethod
@@ -469,11 +469,11 @@ class FormatValidator(BaseValidator):
         try:
             datetime.strptime(value, "%H:%M:%S")
             return True
-        except:
+        except (ValueError, TypeError):
             try:
                 datetime.strptime(value, "%H:%M")
                 return True
-            except:
+            except (ValueError, TypeError):
                 return False
 
     @staticmethod
@@ -488,7 +488,7 @@ class FormatValidator(BaseValidator):
             try:
                 datetime.strptime(value, fmt)
                 return True
-            except:
+            except (ValueError, TypeError):
                 continue
         return False
 
@@ -499,7 +499,7 @@ class FormatValidator(BaseValidator):
         try:
             json.loads(value)
             return True
-        except:
+        except (json.JSONDecodeError, ValueError, TypeError):
             return False
 
     @staticmethod
@@ -509,7 +509,7 @@ class FormatValidator(BaseValidator):
         try:
             base64.b64decode(value, validate=True)
             return True
-        except:
+        except (ValueError, base64.binascii.Error):
             return False
 
     @staticmethod
@@ -517,7 +517,7 @@ class FormatValidator(BaseValidator):
         try:
             Path(value)
             return True
-        except:
+        except (ValueError, TypeError, OSError):
             return False
 
     @staticmethod
@@ -709,7 +709,7 @@ class ConditionalValidator(BaseValidator):
 
         try:
             return comparator(actual, expected)
-        except:
+        except (TypeError, ValueError, AttributeError, KeyError):
             return False
 
     def _apply_rule(
@@ -910,7 +910,7 @@ class CrossFieldValidator(BaseValidator):
         if comparator:
             try:
                 return comparator(val1, val2)
-            except:
+            except (TypeError, ValueError, AttributeError, KeyError):
                 return False
 
         return False