claude-mpm 4.6.0__py3-none-any.whl → 4.7.0__py3-none-any.whl

claude_mpm/services/unified/analyzer_strategies/structure_analyzer.py

@@ -11,7 +11,7 @@ Created: 2025-01-26
 
 import fnmatch
 from pathlib import Path
-from typing import Any, Dict, List, Optional, Tuple
+from typing import Any, ClassVar, Dict, List, Optional, Tuple
 
 from claude_mpm.core.logging_utils import get_logger
 
@@ -38,7 +38,7 @@ class StructureAnalyzerStrategy(AnalyzerStrategy):
     """
 
     # Common project patterns
-    PROJECT_PATTERNS = {
+    PROJECT_PATTERNS: ClassVar[dict] = {
         "mvc": {
             "dirs": ["models", "views", "controllers"],
             "confidence": 0.8,
@@ -62,7 +62,7 @@ class StructureAnalyzerStrategy(AnalyzerStrategy):
     }
 
     # Language-specific structure patterns
-    LANGUAGE_STRUCTURES = {
+    LANGUAGE_STRUCTURES: ClassVar[dict] = {
         "python": {
             "src_patterns": ["src", "lib", "app"],
             "test_patterns": ["tests", "test", "spec"],
@@ -90,7 +90,7 @@ class StructureAnalyzerStrategy(AnalyzerStrategy):
     }
 
     # Common ignore patterns
-    IGNORE_PATTERNS = [
+    IGNORE_PATTERNS: ClassVar[list] = [
         "*.pyc",
         "__pycache__",
         ".git",

claude_mpm/services/unified/config_strategies/error_handling_strategy.py

@@ -10,7 +10,7 @@ from dataclasses import dataclass, field
 from datetime import datetime
 from enum import Enum
 from pathlib import Path
-from typing import Any, Callable, Dict, List, Optional, Type, Union
+from typing import Any, Callable, ClassVar, Dict, List, Optional, Type, Union
 
 from claude_mpm.core.logging_utils import get_logger
 
@@ -105,7 +105,7 @@ class BaseErrorHandler(ABC):
 class FileIOErrorHandler(BaseErrorHandler):
     """Handles file I/O errors - consolidates 18 file error patterns"""
 
-    ERROR_MAPPING = {
+    ERROR_MAPPING: ClassVar[dict] = {
         FileNotFoundError: "File not found",
         PermissionError: "Permission denied",
         IsADirectoryError: "Path is a directory",
@@ -241,7 +241,7 @@ class FileIOErrorHandler(BaseErrorHandler):
 class ParsingErrorHandler(BaseErrorHandler):
     """Handles parsing errors - consolidates 22 parsing error patterns"""
 
-    PARSER_ERRORS = {
+    PARSER_ERRORS: ClassVar[dict] = {
         json.JSONDecodeError: ErrorCategory.PARSING,
         ValueError: ErrorCategory.PARSING,  # Common for parsing
         SyntaxError: ErrorCategory.PARSING,
@@ -559,7 +559,7 @@ class ValidationErrorHandler(BaseErrorHandler):
 class NetworkErrorHandler(BaseErrorHandler):
     """Handles network-related errors - consolidates 12 network error patterns"""
 
-    NETWORK_ERRORS = [
+    NETWORK_ERRORS: ClassVar[list] = [
         ConnectionError,
         TimeoutError,
         ConnectionRefusedError,

claude_mpm/services/unified/config_strategies/file_loader_strategy.py

@@ -64,13 +64,17 @@ class BaseFileLoader(ABC):
     def _read_file(self, path: Path, encoding: str = "utf-8") -> str:
         """Read file with proper error handling"""
         try:
-            with open(path, encoding=encoding) as f:
+            with Path(path).open(
+                encoding=encoding,
+            ) as f:
                 return f.read()
         except UnicodeDecodeError:
             # Try with different encodings
             for enc in ["latin-1", "cp1252", "utf-16"]:
                 try:
-                    with open(path, encoding=enc) as f:
+                    with Path(path).open(
+                        encoding=enc,
+                    ) as f:
                         self.logger.warning(
                             f"Read {path} with fallback encoding: {enc}"
                         )

claude_mpm/services/unified/config_strategies/unified_config_service.py

@@ -254,9 +254,10 @@ class UnifiedConfigService:
             # Apply specified validators or use schema-defined ones
             if validators:
                 for validator_name in validators:
-                    if validator_name in self._validators:
-                        if not self._validators[validator_name](config, schema):
-                            return False
+                    if validator_name in self._validators and not self._validators[
+                        validator_name
+                    ](config, schema):
+                        return False
             else:
                 # Use schema to determine validators
                 return self._validate_schema(config, schema)
@@ -563,9 +564,12 @@ class UnifiedConfigService:
         """Validate enum values"""
         properties = schema.get("properties", {})
         for key, value in config.items():
-            if key in properties and "enum" in properties[key]:
-                if value not in properties[key]["enum"]:
-                    return False
+            if (
+                key in properties
+                and "enum" in properties[key]
+                and value not in properties[key]["enum"]
+            ):
+                return False
         return True
 
     def _validate_schema(self, config: Dict[str, Any], schema: Dict) -> bool:
@@ -596,9 +600,13 @@ class UnifiedConfigService:
         """Validate unique values in arrays"""
         properties = schema.get("properties", {})
         for key, value in config.items():
-            if key in properties and properties[key].get("uniqueItems"):
-                if isinstance(value, list) and len(value) != len(set(map(str, value))):
-                    return False
+            if (
+                key in properties
+                and properties[key].get("uniqueItems")
+                and isinstance(value, list)
+                and len(value) != len(set(map(str, value)))
+            ):
+                return False
         return True
 
     def _validate_format(self, config: Dict[str, Any], schema: Dict) -> bool:
@@ -641,10 +649,13 @@ class UnifiedConfigService:
         """Recursively validate nested structures"""
         properties = schema.get("properties", {})
         for key, value in config.items():
-            if key in properties and isinstance(value, dict):
-                if "properties" in properties[key]:
-                    if not self._validate_schema(value, properties[key]):
-                        return False
+            if (
+                key in properties
+                and isinstance(value, dict)
+                and "properties" in properties[key]
+                and not self._validate_schema(value, properties[key])
+            ):
+                return False
         return True
 
     def _validate_cross_field(self, config: Dict[str, Any], schema: Dict) -> bool:

claude_mpm/services/version_control/conflict_resolution.py

@@ -274,7 +274,9 @@ class ConflictResolutionManager:
             )
 
         try:
-            with open(full_path, encoding="utf-8") as f:
+            with Path(full_path).open(
+                encoding="utf-8",
+            ) as f:
                 content = f.read()
 
             # Parse conflict markers
@@ -587,7 +589,9 @@ class ConflictResolutionManager:
 
         try:
             # Read current content
-            with open(file_path, encoding="utf-8") as f:
+            with Path(file_path).open(
+                encoding="utf-8",
+            ) as f:
                 content = f.read()
 
             # Apply resolution strategy

claude_mpm/services/version_control/git_operations.py

@@ -333,7 +333,7 @@ class GitOperationsManager:
         """
         try:
             # Convert to relative path if absolute
-            if os.path.isabs(file_path):
+            if Path(file_path).is_absolute():
                 try:
                     file_path = os.path.relpath(file_path, self.project_root)
                 except ValueError:

claude_mpm/services/version_control/version_parser.py

@@ -113,7 +113,7 @@ class EnhancedVersionParser:
             r"(\d+)\.(\d+)\.(\d+)(?:-([a-zA-Z0-9\-\.]+))?(?:\+([a-zA-Z0-9\-\.]+))?"
         )
         self._changelog_version_pattern = re.compile(
-            r"##\s*\[?([0-9]+\.[0-9]+\.[0-9]+[^\]]*)\]?\s*[-–]\s*(\d{4}-\d{2}-\d{2})?"
+            r"##\s*\[?([0-9]+\.[0-9]+\.[0-9]+[^\]]*)\]?\s*[--]\s*(\d{4}-\d{2}-\d{2})?"
         )
 
     def _get_cached(self, key: str) -> Optional[any]:

claude_mpm/storage/state_storage.py

@@ -268,7 +268,7 @@ class StateStorage:
             finally:
                 # Clean up temp file if it still exists
                 if Path(temp_path).exists():
-                    os.unlink(temp_path)
+                    Path(temp_path).unlink()
 
         except Exception as e:
             self.logger.error(f"Atomic write failed for {file_path}: {e}")
@@ -288,14 +288,14 @@ class StateStorage:
         """
         if not self.supports_locking:
             # No locking on Windows
-            with open(file_path, mode) as f:
+            with Path(file_path).open(mode) as f:
                 yield f
             return
 
         # Unix-like systems with fcntl
         f = None
         try:
-            f = open(file_path, mode)
+            f = Path(file_path).open(mode)
 
             # Try to acquire lock (non-blocking)
             max_attempts = 50

claude_mpm/tools/__main__.py

@@ -118,7 +118,7 @@ def main():
     output_file = None
     if args.output_file:
         try:
-            output_file = open(args.output_file, "w")
+            output_file = Path(args.output_file).open("w")
         except Exception as e:
             emit_json_event(
                 "code:analysis:error", {"message": f"Failed to open output file: {e}"}

claude_mpm/tools/code_tree_analyzer.py

@@ -21,7 +21,7 @@ import time
 from dataclasses import dataclass
 from datetime import timezone
 from pathlib import Path
-from typing import Any, Dict, List, Optional
+from typing import Any, ClassVar, Dict, List, Optional
 
 try:
     import pathspec
@@ -31,14 +31,13 @@ except ImportError:
     PATHSPEC_AVAILABLE = False
     pathspec = None
 
-try:
+import importlib.util
+
+if importlib.util.find_spec("tree_sitter"):
     import tree_sitter
-    import tree_sitter_javascript
-    import tree_sitter_python
-    import tree_sitter_typescript
 
     TREE_SITTER_AVAILABLE = True
-except ImportError:
+else:
     TREE_SITTER_AVAILABLE = False
     tree_sitter = None
 
@@ -54,7 +53,7 @@ class GitignoreManager:
     """
 
     # Default patterns that should always be ignored
-    DEFAULT_PATTERNS = [
+    DEFAULT_PATTERNS: ClassVar[list] = [
         ".git/",
         "__pycache__/",
         "*.pyc",
@@ -88,13 +87,13 @@ class GitignoreManager:
     ]
 
     # Additional patterns to hide dotfiles (when enabled)
-    DOTFILE_PATTERNS = [
+    DOTFILE_PATTERNS: ClassVar[list] = [
         ".*",  # All dotfiles
         ".*/",  # All dot directories
     ]
 
     # Important files/directories to always show
-    DOTFILE_EXCEPTIONS = {
+    DOTFILE_EXCEPTIONS: ClassVar[set] = {
         # Removed .gitignore from exceptions - it should be hidden by default
         ".env.example",
         ".env.sample",
@@ -260,7 +259,9 @@ class GitignoreManager:
 
         patterns = []
         try:
-            with open(gitignore_path, encoding="utf-8") as f:
+            with Path(gitignore_path).open(
+                encoding="utf-8",
+            ) as f:
                 for line in f:
                     line = line.strip()
                     # Skip empty lines and comments
@@ -383,7 +384,9 @@ class PythonAnalyzer:
         nodes = []
 
         try:
-            with open(file_path, encoding="utf-8") as f:
+            with Path(file_path).open(
+                encoding="utf-8",
+            ) as f:
                 source = f.read()
 
             tree = ast.parse(source, filename=str(file_path))
@@ -635,7 +638,7 @@ class MultiLanguageAnalyzer:
     allowing us to support JavaScript, TypeScript, and other languages.
     """
 
-    LANGUAGE_PARSERS = {
+    LANGUAGE_PARSERS: ClassVar[dict] = {
         "python": "tree_sitter_python",
         "javascript": "tree_sitter_javascript",
         "typescript": "tree_sitter_typescript",
@@ -837,7 +840,7 @@ class CodeTreeAnalyzer:
     """
 
     # Define code file extensions at class level for directory filtering
-    CODE_EXTENSIONS = {
+    CODE_EXTENSIONS: ClassVar[set] = {
         ".py",
         ".js",
         ".jsx",
@@ -888,7 +891,7 @@ class CodeTreeAnalyzer:
     }
 
     # File extensions to language mapping
-    LANGUAGE_MAP = {
+    LANGUAGE_MAP: ClassVar[dict] = {
         ".py": "python",
         ".js": "javascript",
         ".jsx": "javascript",

claude_mpm/tools/socketio_debug.py

@@ -8,6 +8,9 @@ for developers working on the claude-mpm dashboard.
 
 import argparse
 import asyncio
+
+# Try to import Rich for enhanced output
+import importlib.util
 import json
 import signal
 import sys
@@ -20,18 +23,15 @@ from typing import Any, Dict, List, Optional, Set
 
 import socketio
 
-# Try to import Rich for enhanced output
-try:
+if importlib.util.find_spec("rich"):
     from rich.console import Console
-    from rich.layout import Layout
-    from rich.live import Live
     from rich.panel import Panel
     from rich.table import Table
     from rich.text import Text
 
     RICH_AVAILABLE = True
     console = Console()
-except ImportError:
+else:
     RICH_AVAILABLE = False
     console = None
 
@@ -266,7 +266,7 @@ class SocketIODebugger:
             "PostToolUse": "✅",
             "Error": "❌",
             "Warning": "⚠️",
-            "Info": "ℹ️",
+            "Info": "[INFO]️",
             "MemoryUpdate": "🧠",
             "ConfigChange": "⚙️",
         }
@@ -463,7 +463,7 @@ class SocketIODebugger:
             console.print(f"[{timestamp}] {message}", style=style)
         else:
             prefixes = {
-                "info": "ℹ️",
+                "info": "[INFO]️",
                 "success": "✅",
                 "warning": "⚠️",
                 "error": "❌",

claude_mpm/utils/common.py

@@ -46,7 +46,9 @@ def load_json_safe(
     file_path = Path(file_path)
 
     try:
-        with open(file_path, encoding=encoding) as f:
+        with Path(file_path).open(
+            encoding=encoding,
+        ) as f:
             return json.load(f)
     except FileNotFoundError:
         logger.debug(f"JSON file not found: {file_path}")
@@ -112,7 +114,9 @@ def load_yaml_safe(
     file_path = Path(file_path)
 
     try:
-        with open(file_path, encoding=encoding) as f:
+        with Path(file_path).open(
+            encoding=encoding,
+        ) as f:
             return yaml.safe_load(f) or default or {}
     except FileNotFoundError:
         logger.debug(f"YAML file not found: {file_path}")

claude_mpm/utils/config_manager.py

@@ -99,7 +99,9 @@ class ConfigurationManager:
 
         logger.debug(f"Loading JSON configuration from {file_path}")
         try:
-            with open(file_path, encoding="utf-8") as f:
+            with Path(file_path).open(
+                encoding="utf-8",
+            ) as f:
                 config = json.load(f)
             self._update_cache(file_path, config)
             return config
@@ -142,7 +144,9 @@ class ConfigurationManager:
 
         logger.debug(f"Loading YAML configuration from {file_path}")
         try:
-            with open(file_path, encoding="utf-8") as f:
+            with Path(file_path).open(
+                encoding="utf-8",
+            ) as f:
                 config = yaml.safe_load(f) or {}
             self._update_cache(file_path, config)
             return config
@@ -185,7 +189,9 @@ class ConfigurationManager:
 
         logger.debug(f"Loading TOML configuration from {file_path}")
         try:
-            with open(file_path, encoding="utf-8") as f:
+            with Path(file_path).open(
+                encoding="utf-8",
+            ) as f:
                 config = toml.load(f)
             self._update_cache(file_path, config)
             return config

claude_mpm/utils/database_connector.py

@@ -10,7 +10,7 @@ DESIGN DECISION: We prioritize pure Python implementations over native ones
 for better cross-platform compatibility, even if they might be slightly slower.
 """
 
-from typing import Any, Dict, Optional, Tuple
+from typing import Any, ClassVar, Dict, Optional, Tuple
 
 from ..core.logger import get_logger
 
@@ -26,19 +26,19 @@ class DatabaseConnector:
     """
 
     # Database drivers in order of preference (first is preferred)
-    MYSQL_DRIVERS = [
+    MYSQL_DRIVERS: ClassVar[list] = [
         ("pymysql", "pymysql"),  # Pure Python, no compilation required
         ("mysqlclient", "MySQLdb"),  # Faster but requires MySQL dev headers
         ("mysql-connector-python", "mysql.connector"),  # Oracle's pure Python driver
     ]
 
-    POSTGRESQL_DRIVERS = [
+    POSTGRESQL_DRIVERS: ClassVar[list] = [
         ("psycopg2-binary", "psycopg2"),  # Binary wheel, no compilation
         ("psycopg2", "psycopg2"),  # Requires PostgreSQL dev headers
         ("pg8000", "pg8000"),  # Pure Python alternative
     ]
 
-    ORACLE_DRIVERS = [
+    ORACLE_DRIVERS: ClassVar[list] = [
         ("cx_Oracle", "cx_Oracle"),  # Requires Oracle client libraries
         ("oracledb", "oracledb"),  # Pure Python Oracle driver (newer)
     ]

claude_mpm/utils/dependency_strategies.py

@@ -106,7 +106,7 @@ class DependencyStrategy:
     def _is_docker(self) -> bool:
         """Check if running inside Docker container."""
         return (
-            os.path.exists("/.dockerenv")
+            Path("/.dockerenv").exists()
             or os.environ.get("KUBERNETES_SERVICE_HOST") is not None
         )
 

claude_mpm/utils/environment_context.py

@@ -7,6 +7,7 @@ prompting is appropriate for dependency installation.
 
 import os
 import sys
+from pathlib import Path
 from typing import Dict, Tuple
 
 from ..core.logger import get_logger
@@ -139,7 +140,7 @@ class EnvironmentContext:
         """
         # Check for Docker-specific files
         for indicator_file in cls.CONTAINER_INDICATORS:
-            if os.path.exists(indicator_file):
+            if Path(indicator_file).exists():
                 logger.debug(f"Container detected via {indicator_file}")
                 return True
 
@@ -149,7 +150,7 @@ class EnvironmentContext:
 
         # Check cgroup for docker/containerd references
         try:
-            with open("/proc/1/cgroup") as f:
+            with Path("/proc/1/cgroup").open() as f:
                 cgroup_content = f.read()
                 if "docker" in cgroup_content or "containerd" in cgroup_content:
                     return True

claude_mpm/utils/file_utils.py

@@ -7,7 +7,6 @@ error handling, and directory management.
 """
 
 import json
-import os
 import shutil
 import tempfile
 from pathlib import Path
@@ -149,7 +148,7 @@ def atomic_write(
         # Clean up temporary file if it exists
         try:
             if "temp_path" in locals():
-                os.unlink(temp_path)
+                Path(temp_path).unlink()
         except Exception:
             pass
         raise FileOperationError(f"Failed to atomically write file {path}: {e}") from e

claude_mpm/utils/path_operations.py

@@ -136,7 +136,9 @@ class PathOperations:
                 logger.warning(f"File not readable: {path}")
                 return default
 
-            with open(path_obj, encoding=encoding) as f:
+            with Path(path_obj).open(
+                encoding=encoding,
+            ) as f:
                 return f.read()
         except Exception as e:
             logger.error(f"Error reading file {path}: {e}")

claude_mpm/utils/robust_installer.py

@@ -100,10 +100,9 @@ class RobustPackageInstaller:
             Tuple of (success, error_message)
         """
         # Check success cache first
-        if package_spec in self.success_cache:
-            if self.success_cache[package_spec]:
-                logger.debug(f"Package {package_spec} already successfully installed")
-                return True, None
+        if self.success_cache.get(package_spec):
+            logger.debug(f"Package {package_spec} already successfully installed")
+            return True, None
 
         # Default strategy order
         if strategies is None:

claude_mpm/validation/frontmatter_validator.py

@@ -8,7 +8,7 @@ Critical for ensuring agents work correctly with Claude Code.
 """
 
 import re
-from typing import Dict, List, Optional, Tuple
+from typing import ClassVar, Dict, List, Optional, Tuple
 
 import yaml
 
@@ -21,7 +21,7 @@ class FrontmatterValidator:
     NAME_PATTERN = re.compile(r"^[a-z0-9]+(-[a-z0-9]+)*$")
 
     # Valid tool names (from Claude Code spec)
-    VALID_TOOLS = {
+    VALID_TOOLS: ClassVar[set] = {
         "Read",
         "Write",
         "Edit",
@@ -41,10 +41,10 @@ class FrontmatterValidator:
     }
 
     # Valid model tiers
-    VALID_MODELS = {"opus", "sonnet", "haiku"}
+    VALID_MODELS: ClassVar[set] = {"opus", "sonnet", "haiku"}
 
     # Required fields in frontmatter
-    REQUIRED_FIELDS = {"name", "description", "tools"}
+    REQUIRED_FIELDS: ClassVar[set] = {"name", "description", "tools"}
 
     @classmethod
     def validate_name(cls, name: str) -> Tuple[bool, Optional[str]]:

{claude_mpm-4.6.0.dist-info → claude_mpm-4.7.0.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: claude-mpm
-Version: 4.6.0
+Version: 4.7.0
 Summary: Claude Multi-Agent Project Manager - Orchestrate Claude with agent delegation and ticket tracking
 Author-email: Bob Matsuoka <bob@matsuoka.com>
 Maintainer: Claude MPM Team