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

claude_mpm/services/monitor/handlers/file.py

@@ -12,7 +12,6 @@ DESIGN DECISIONS:
 - Compatible with UnifiedMonitorServer architecture
 """
 
-import os
 from pathlib import Path
 from typing import Any, Dict, Optional
 
@@ -56,7 +55,7 @@ class FileHandler:
 
             try:
                 file_path = data.get("file_path")
-                working_dir = data.get("working_dir", os.getcwd())
+                working_dir = data.get("working_dir", Path.cwd())
                 max_size = data.get("max_size", 1024 * 1024)  # 1MB default limit
 
                 if not file_path:
@@ -126,7 +125,7 @@ class FileHandler:
         """
         try:
             if working_dir is None:
-                working_dir = os.getcwd()
+                working_dir = Path.cwd()
 
             # Resolve absolute path based on working directory
             file_path_obj = Path(file_path)
@@ -192,7 +191,9 @@ class FileHandler:
 
             # Read file content
             try:
-                with open(real_path, encoding="utf-8") as f:
+                with Path(real_path).open(
+                    encoding="utf-8",
+                ) as f:
                     content = f.read()
 
                 # Get file extension for syntax highlighting hint

claude_mpm/services/monitor/management/lifecycle.py

@@ -121,7 +121,7 @@ class DaemonLifecycle:
             sys.stderr.flush()
 
             # Redirect stdin to /dev/null
-            with open("/dev/null") as null_in:
+            with Path("/dev/null").open() as null_in:
                 os.dup2(null_in.fileno(), sys.stdin.fileno())
 
             # Redirect stdout and stderr

claude_mpm/services/monitor/server.py

@@ -368,25 +368,24 @@ class UnifiedMonitorServer:
             async def api_file_handler(request):
                 """Handle file content requests."""
                 import json
-                import os
 
                 try:
                     data = await request.json()
                     file_path = data.get("path", "")
 
                     # Security check: ensure path is absolute and exists
-                    if not file_path or not os.path.isabs(file_path):
+                    if not file_path or not Path(file_path).is_absolute():
                         return web.json_response(
                             {"success": False, "error": "Invalid file path"}, status=400
                         )
 
                     # Check if file exists and is readable
-                    if not os.path.exists(file_path):
+                    if not Path(file_path).exists():
                         return web.json_response(
                             {"success": False, "error": "File not found"}, status=404
                         )
 
-                    if not os.path.isfile(file_path):
+                    if not Path(file_path).is_file():
                         return web.json_response(
                             {"success": False, "error": "Path is not a file"},
                             status=400,
@@ -394,7 +393,7 @@ class UnifiedMonitorServer:
 
                     # Read file content (with size limit for safety)
                     max_size = 10 * 1024 * 1024  # 10MB limit
-                    file_size = os.path.getsize(file_path)
+                    file_size = Path(file_path).stat().st_size
 
                     if file_size > max_size:
                         return web.json_response(
@@ -406,7 +405,9 @@ class UnifiedMonitorServer:
                         )
 
                     try:
-                        with open(file_path, encoding="utf-8") as f:
+                        with Path(file_path).open(
+                            encoding="utf-8",
+                        ) as f:
                             content = f.read()
                             lines = content.count("\n") + 1
                     except UnicodeDecodeError:
@@ -417,7 +418,7 @@ class UnifiedMonitorServer:
                         )
 
                     # Get file extension for type detection
-                    file_ext = os.path.splitext(file_path)[1].lstrip(".")
+                    file_ext = Path(file_path).suffix.lstrip(".")
 
                     return web.json_response(
                         {
@@ -475,11 +476,10 @@ class UnifiedMonitorServer:
             # Configuration endpoint for dashboard initialization
             async def config_handler(request):
                 """Return configuration for dashboard initialization."""
-                import os
                 import subprocess
 
                 config = {
-                    "workingDirectory": os.getcwd(),
+                    "workingDirectory": Path.cwd(),
                     "gitBranch": "Unknown",
                     "serverTime": datetime.now(timezone.utc).isoformat() + "Z",
                     "service": "unified-monitor",
@@ -492,7 +492,7 @@ class UnifiedMonitorServer:
                         capture_output=True,
                         text=True,
                         timeout=2,
-                        cwd=os.getcwd(),
+                        cwd=Path.cwd(),
                         check=False,
                     )
                     if result.returncode == 0 and result.stdout.strip():
@@ -506,7 +506,7 @@ class UnifiedMonitorServer:
             async def working_directory_handler(request):
                 """Return the current working directory."""
                 return web.json_response(
-                    {"working_directory": os.getcwd(), "success": True}
+                    {"working_directory": Path.cwd(), "success": True}
                 )
 
             # Monitor page routes
@@ -517,7 +517,9 @@ class UnifiedMonitorServer:
                 file_path = static_dir / f"{page_name}.html"
 
                 if file_path.exists() and file_path.is_file():
-                    with open(file_path, encoding="utf-8") as f:
+                    with Path(file_path).open(
+                        encoding="utf-8",
+                    ) as f:
                         content = f.read()
                     return web.Response(text=content, content_type="text/html")
                 return web.Response(text="Page not found", status=404)

claude_mpm/services/project/architecture_analyzer.py

@@ -15,7 +15,7 @@ import logging
 import re
 from dataclasses import asdict, dataclass
 from pathlib import Path
-from typing import Dict, List
+from typing import ClassVar, Dict, List
 
 
 @dataclass
@@ -44,7 +44,7 @@ class ArchitectureAnalyzerService:
     """
 
     # Common architectural directories
-    ARCHITECTURE_INDICATORS = {
+    ARCHITECTURE_INDICATORS: ClassVar[dict] = {
         "mvc": ["models", "views", "controllers"],
         "mvvm": ["models", "views", "viewmodels"],
         "layered": ["presentation", "business", "data", "domain"],
@@ -58,7 +58,7 @@ class ArchitectureAnalyzerService:
     }
 
     # Entry point patterns by language/framework
-    ENTRY_POINT_PATTERNS = {
+    ENTRY_POINT_PATTERNS: ClassVar[dict] = {
         "python": [
             "main.py",
             "app.py",
@@ -77,7 +77,7 @@ class ArchitectureAnalyzerService:
     }
 
     # API pattern indicators
-    API_INDICATORS = {
+    API_INDICATORS: ClassVar[dict] = {
         "rest": ["routes", "endpoints", "resources", "api/v", "/api/"],
         "graphql": ["schema.graphql", "resolvers", "typeDefs", "graphql"],
         "grpc": [".proto", "grpc", "protobuf", "rpc"],
@@ -86,7 +86,7 @@ class ArchitectureAnalyzerService:
     }
 
     # Configuration file patterns
-    CONFIG_PATTERNS = {
+    CONFIG_PATTERNS: ClassVar[dict] = {
         "yaml": [".yaml", ".yml"],
         "json": [".json", "config.json", "settings.json"],
         "toml": [".toml", "pyproject.toml", "Cargo.toml"],

claude_mpm/services/project/metrics_collector.py

@@ -15,7 +15,7 @@ import logging
 from collections import Counter
 from dataclasses import asdict, dataclass
 from pathlib import Path
-from typing import Dict, List, Tuple
+from typing import ClassVar, Dict, List, Tuple
 
 
 @dataclass
@@ -68,7 +68,7 @@ class MetricsCollectorService:
     """
 
     # File extensions to analyze
-    CODE_EXTENSIONS = {
+    CODE_EXTENSIONS: ClassVar[set] = {
         ".py",
         ".js",
         ".ts",
@@ -90,7 +90,7 @@ class MetricsCollectorService:
     }
 
     # Test file patterns
-    TEST_PATTERNS = [
+    TEST_PATTERNS: ClassVar[list] = [
         "test_",
         "_test.",
         ".test.",
@@ -103,7 +103,7 @@ class MetricsCollectorService:
     ]
 
     # Directories to exclude from analysis
-    EXCLUDE_DIRS = {
+    EXCLUDE_DIRS: ClassVar[set] = {
         ".git",
         "node_modules",
         "vendor",

claude_mpm/services/project/project_organizer.py

@@ -17,7 +17,7 @@ Created: 2025-01-26
 """
 
 from pathlib import Path
-from typing import Dict, List, Optional, Tuple
+from typing import ClassVar, Dict, List, Optional, Tuple
 
 from rich.console import Console
 
@@ -31,7 +31,7 @@ class ProjectOrganizer:
     """Manages project directory structure and organization."""
 
     # Standard directory structure for Claude MPM projects
-    STANDARD_DIRECTORIES = {
+    STANDARD_DIRECTORIES: ClassVar[dict] = {
         "tmp": "Temporary files, test outputs, and experiments",
         "scripts": "Project scripts and automation tools",
         "docs": "Project documentation",
@@ -44,7 +44,7 @@ class ProjectOrganizer:
     }
 
     # Comprehensive gitignore patterns for Claude MPM projects
-    GITIGNORE_DIRS = {
+    GITIGNORE_DIRS: ClassVar[dict] = {
         # Temporary and cache directories
         "tmp/",
         "temp/",
@@ -160,7 +160,7 @@ class ProjectOrganizer:
     }
 
     # Project type specific structures
-    PROJECT_STRUCTURES = {
+    PROJECT_STRUCTURES: ClassVar[dict] = {
         "web": ["public", "src/components", "src/pages", "src/styles"],
         "api": ["src/routes", "src/models", "src/middleware", "src/services"],
         "cli": ["src/commands", "src/utils", "src/config"],

claude_mpm/services/project/registry.py

@@ -124,7 +124,9 @@ class ProjectRegistry:
             # Search all registry files
             for registry_file in self.registry_dir.glob("*.yaml"):
                 try:
-                    with open(registry_file, encoding="utf-8") as f:
+                    with Path(registry_file).open(
+                        encoding="utf-8",
+                    ) as f:
                         data = yaml.safe_load(f) or {}
 
                     # Check if project_path matches
@@ -517,7 +519,9 @@ class ProjectRegistry:
         try:
             for registry_file in self.registry_dir.glob("*.yaml"):
                 try:
-                    with open(registry_file, encoding="utf-8") as f:
+                    with Path(registry_file).open(
+                        encoding="utf-8",
+                    ) as f:
                         data = yaml.safe_load(f) or {}
                     projects.append(data)
                 except Exception as e:
@@ -553,7 +557,9 @@ class ProjectRegistry:
         try:
             for registry_file in self.registry_dir.glob("*.yaml"):
                 try:
-                    with open(registry_file, encoding="utf-8") as f:
+                    with Path(registry_file).open(
+                        encoding="utf-8",
+                    ) as f:
                         data = yaml.safe_load(f) or {}
 
                     # Check last accessed time

claude_mpm/services/shared/config_service_base.py

@@ -99,9 +99,8 @@ class ConfigServiceBase(LoggerMixin, ABC):
             raise ValueError(f"Required configuration value missing: {full_key}")
 
         # Type validation
-        if config_type is not None and value is not None:
-            if not isinstance(value, config_type):
-                try:
+        if config_type is not None and value is not None and not isinstance(value, config_type):
+            try:
                     # Try to convert
                     if config_type == bool and isinstance(value, str):
                         value = value.lower() in ("true", "1", "yes", "on")

claude_mpm/services/socketio/handlers/file.py

@@ -5,7 +5,6 @@ safely with security checks. Separating file operations improves security
 auditing and makes it easier to add file-related features.
 """
 
-import os
 from pathlib import Path
 from typing import Any, Dict, Optional
 
@@ -44,7 +43,7 @@ class FileEventHandler(BaseEventHandler):
             )
             try:
                 file_path = data.get("file_path")
-                working_dir = data.get("working_dir", os.getcwd())
+                working_dir = data.get("working_dir", Path.cwd())
                 max_size = data.get("max_size", 1024 * 1024)  # 1MB default limit
 
                 if not file_path:
@@ -125,7 +124,7 @@ class FileEventHandler(BaseEventHandler):
         """
         try:
             if working_dir is None:
-                working_dir = os.getcwd()
+                working_dir = Path.cwd()
 
             # Resolve absolute path based on working directory
             file_path_obj = Path(file_path)
@@ -191,7 +190,9 @@ class FileEventHandler(BaseEventHandler):
 
             # Read file content
             try:
-                with open(real_path, encoding="utf-8") as f:
+                with Path(real_path).open(
+                    encoding="utf-8",
+                ) as f:
                     content = f.read()
 
                 # Get file extension for syntax highlighting hint

claude_mpm/services/socketio/handlers/git.py

@@ -106,7 +106,7 @@ class GitEventHandler(BaseEventHandler):
             """
             try:
                 file_path = data.get("file_path")
-                working_dir = data.get("working_dir", os.getcwd())
+                working_dir = data.get("working_dir", Path.cwd())
 
                 if not file_path:
                     await self.emit_to_client(
@@ -162,7 +162,7 @@ class GitEventHandler(BaseEventHandler):
             """
             try:
                 file_path = data.get("file_path")
-                working_dir = data.get("working_dir", os.getcwd())
+                working_dir = data.get("working_dir", Path.cwd())
 
                 # Debug: check_git_status called
 
@@ -287,7 +287,7 @@ class GitEventHandler(BaseEventHandler):
             """
             try:
                 file_path = data.get("file_path")
-                working_dir = data.get("working_dir", os.getcwd())
+                working_dir = data.get("working_dir", Path.cwd())
 
                 # Debug: git_add_file called
 
@@ -389,7 +389,7 @@ class GitEventHandler(BaseEventHandler):
         if working_dir in invalid_states or (
             isinstance(working_dir, str) and working_dir.strip() == ""
         ):
-            working_dir = os.getcwd()
+            working_dir = Path.cwd()
             self.logger.info(
                 f"[{operation}] working_dir was invalid ({original_working_dir!r}), using cwd: {working_dir}"
             )
@@ -404,7 +404,7 @@ class GitEventHandler(BaseEventHandler):
                 self.logger.warning(
                     f"[{operation}] working_dir contains null bytes, using cwd instead"
                 )
-                working_dir = os.getcwd()
+                working_dir = Path.cwd()
 
         return working_dir
 
@@ -640,7 +640,7 @@ class GitEventHandler(BaseEventHandler):
 
             # Handle case where working_dir is None, empty string, or 'Unknown'
             if not working_dir or working_dir == "Unknown" or working_dir.strip() == "":
-                working_dir = os.getcwd()
+                working_dir = Path.cwd()
                 # Debug: working_dir was invalid, using cwd
             else:
                 # Debug: Using provided working_dir
@@ -648,7 +648,7 @@ class GitEventHandler(BaseEventHandler):
 
             # For read-only git operations, we can work from any directory
             # by passing the -C flag to git commands instead of changing directories
-            original_cwd = os.getcwd()
+            original_cwd = Path.cwd()
             try:
                 # We'll use git -C <working_dir> for all commands instead of chdir
 

claude_mpm/services/socketio/server/core.py

@@ -396,11 +396,10 @@ class SocketIOServerCore:
         # Add working directory endpoint
         async def working_directory_handler(request):
             """Handle GET /api/working-directory to provide current working directory."""
-            import os
             from pathlib import Path
 
             try:
-                working_dir = os.getcwd()
+                working_dir = Path.cwd()
                 home_dir = str(Path.home())
 
                 return web.json_response(
@@ -430,27 +429,26 @@ class SocketIOServerCore:
         # Add file reading endpoint for source viewer
         async def file_read_handler(request):
             """Handle GET /api/file/read for reading source files."""
-            import os
 
             file_path = request.query.get("path", "")
 
             if not file_path:
                 return web.json_response({"error": "No path provided"}, status=400)
 
-            abs_path = os.path.abspath(os.path.expanduser(file_path))
+            abs_path = Path(Path(file_path).resolve().expanduser())
 
             # Security check - ensure file is within the project
             try:
-                project_root = os.getcwd()
+                project_root = Path.cwd()
                 if not abs_path.startswith(project_root):
                     return web.json_response({"error": "Access denied"}, status=403)
             except Exception:
                 pass
 
-            if not os.path.exists(abs_path):
+            if not Path(abs_path).exists():
                 return web.json_response({"error": "File not found"}, status=404)
 
-            if not os.path.isfile(abs_path):
+            if not Path(abs_path).is_file():
                 return web.json_response({"error": "Not a file"}, status=400)
 
             try:
@@ -460,7 +458,9 @@ class SocketIOServerCore:
 
                 for encoding in encodings:
                     try:
-                        with open(abs_path, encoding=encoding) as f:
+                        with Path(abs_path).open(
+                            encoding=encoding,
+                        ) as f:
                             content = f.read()
                         break
                     except UnicodeDecodeError:
@@ -474,10 +474,10 @@ class SocketIOServerCore:
                 return web.json_response(
                     {
                         "path": abs_path,
-                        "name": os.path.basename(abs_path),
+                        "name": Path(abs_path).name,
                         "content": content,
                         "lines": len(content.splitlines()),
-                        "size": os.path.getsize(abs_path),
+                        "size": Path(abs_path).stat().st_size,
                     }
                 )
 

claude_mpm/services/subprocess_launcher_service.py

@@ -268,16 +268,11 @@ class SubprocessLauncherService(BaseService, SubprocessLauncherInterface):
         Returns:
             True if subprocess mode with PTY is available
         """
-        try:
-            # Check if we can import required modules
-            import pty
-            import select
-            import termios
-            import tty
-
-            return True
-        except ImportError:
-            return False
+        import importlib.util
+
+        # Check if we can import required modules
+        required = ["pty", "select", "termios", "tty"]
+        return all(importlib.util.find_spec(mod) is not None for mod in required)
 
     def create_subprocess_command(
         self, base_cmd: List[str], additional_args: Optional[List[str]] = None

claude_mpm/services/ticket_services/formatter_service.py

@@ -278,7 +278,7 @@ class TicketFormatterService:
         Returns:
             Formatted info message
         """
-        return f"ℹ️ {info}"
+        return f"[INFO]️ {info}"
 
     def format_warning(self, warning: str) -> str:
         """

claude_mpm/services/ticket_services/validation_service.py

@@ -11,17 +11,17 @@ DESIGN DECISIONS:
 - Provides sanitization for user inputs
 """
 
-from typing import Any, Dict, List, Optional, Tuple
+from typing import Any, ClassVar, Dict, List, Optional, Tuple
 
 
 class TicketValidationService:
     """Service for validating ticket inputs."""
 
     # Valid ticket types
-    VALID_TYPES = ["task", "issue", "epic", "bug", "feature", "story"]
+    VALID_TYPES: ClassVar[list] = ["task", "issue", "epic", "bug", "feature", "story"]
 
     # Valid ticket statuses
-    VALID_STATUSES = [
+    VALID_STATUSES: ClassVar[list] = [
         "open",
         "in_progress",
         "ready",
@@ -34,10 +34,10 @@ class TicketValidationService:
     ]
 
     # Valid priorities
-    VALID_PRIORITIES = ["low", "medium", "high", "critical"]
+    VALID_PRIORITIES: ClassVar[list] = ["low", "medium", "high", "critical"]
 
     # Valid workflow states
-    VALID_WORKFLOW_STATES = [
+    VALID_WORKFLOW_STATES: ClassVar[list] = [
         "todo",
         "in_progress",
         "ready",

claude_mpm/services/unified/analyzer_strategies/dependency_analyzer.py

@@ -12,7 +12,7 @@ Created: 2025-01-26
 import json
 import re
 from pathlib import Path
-from typing import Any, Dict, List, Optional
+from typing import Any, ClassVar, Dict, List, Optional
 
 from claude_mpm.core.logging_utils import get_logger
 
@@ -39,7 +39,7 @@ class DependencyAnalyzerStrategy(AnalyzerStrategy):
     """
 
     # Package manager configurations
-    PACKAGE_MANAGERS = {
+    PACKAGE_MANAGERS: ClassVar[dict] = {
         "package.json": "npm",
         "yarn.lock": "yarn",
         "pnpm-lock.yaml": "pnpm",
@@ -59,7 +59,7 @@ class DependencyAnalyzerStrategy(AnalyzerStrategy):
     }
 
     # Database-related dependencies
-    DATABASE_PACKAGES = {
+    DATABASE_PACKAGES: ClassVar[dict] = {
         "postgresql": ["psycopg2", "pg", "postgres", "postgresql", "node-postgres"],
         "mysql": ["mysql", "mysql2", "mysqlclient", "mysql-connector"],
         "sqlite": ["sqlite3", "better-sqlite3"],
@@ -71,7 +71,7 @@ class DependencyAnalyzerStrategy(AnalyzerStrategy):
     }
 
     # Testing framework packages
-    TESTING_PACKAGES = {
+    TESTING_PACKAGES: ClassVar[dict] = {
         "python": ["pytest", "unittest", "nose", "nose2", "tox", "coverage"],
         "javascript": [
             "jest",
@@ -89,7 +89,7 @@ class DependencyAnalyzerStrategy(AnalyzerStrategy):
     }
 
     # Web framework packages
-    FRAMEWORK_PACKAGES = {
+    FRAMEWORK_PACKAGES: ClassVar[dict] = {
         "python": ["django", "flask", "fastapi", "pyramid", "tornado", "aiohttp"],
         "javascript": [
             "express",

claude_mpm/services/unified/analyzer_strategies/performance_analyzer.py

@@ -12,7 +12,7 @@ Created: 2025-01-26
 import ast
 import re
 from pathlib import Path
-from typing import Any, Dict, List, Optional
+from typing import Any, ClassVar, Dict, List, Optional
 
 from claude_mpm.core.logging_utils import get_logger
 
@@ -39,7 +39,7 @@ class PerformanceAnalyzerStrategy(AnalyzerStrategy):
     """
 
     # Performance anti-patterns
-    PERFORMANCE_PATTERNS = {
+    PERFORMANCE_PATTERNS: ClassVar[dict] = {
         "n_plus_one_query": {
             "patterns": [
                 r"for .* in .*:\s*\n.*\.(get|filter|select|find)",
@@ -89,7 +89,7 @@ class PerformanceAnalyzerStrategy(AnalyzerStrategy):
     }
 
     # Algorithm complexity indicators
-    COMPLEXITY_INDICATORS = {
+    COMPLEXITY_INDICATORS: ClassVar[dict] = {
         "quadratic": ["nested_loops", "bubble_sort", "selection_sort"],
         "exponential": ["recursive_fibonacci", "recursive_factorial"],
         "linear": ["single_loop", "map", "filter"],
@@ -98,7 +98,7 @@ class PerformanceAnalyzerStrategy(AnalyzerStrategy):
     }
 
     # Memory usage patterns
-    MEMORY_PATTERNS = {
+    MEMORY_PATTERNS: ClassVar[dict] = {
         "memory_leak": {
             "patterns": [
                 r"global\s+\w+\s*=",

claude_mpm/services/unified/analyzer_strategies/security_analyzer.py

@@ -12,7 +12,7 @@ Created: 2025-01-26
 import ast
 import re
 from pathlib import Path
-from typing import Any, Dict, List, Optional
+from typing import Any, ClassVar, Dict, List, Optional
 
 from claude_mpm.core.logging_utils import get_logger
 
@@ -39,7 +39,7 @@ class SecurityAnalyzerStrategy(AnalyzerStrategy):
     """
 
     # Common security vulnerability patterns
-    VULNERABILITY_PATTERNS = {
+    VULNERABILITY_PATTERNS: ClassVar[dict] = {
         "sql_injection": {
             "patterns": [
                 r'(execute|query)\s*\(\s*["\'].*%[s|d].*["\'].*%',
@@ -97,7 +97,7 @@ class SecurityAnalyzerStrategy(AnalyzerStrategy):
     }
 
     # Insecure configuration patterns
-    CONFIG_ISSUES = {
+    CONFIG_ISSUES: ClassVar[dict] = {
         "debug_enabled": {
             "patterns": [
                 r"DEBUG\s*=\s*True",
@@ -128,7 +128,7 @@ class SecurityAnalyzerStrategy(AnalyzerStrategy):
     }
 
     # Security headers to check
-    SECURITY_HEADERS = [
+    SECURITY_HEADERS: ClassVar[list] = [
         "Content-Security-Policy",
         "X-Content-Type-Options",
         "X-Frame-Options",