claude-mpm 4.6.1__py3-none-any.whl → 4.7.1__py3-none-any.whl

claude_mpm/services/cli/startup_checker.py

@@ -246,7 +246,7 @@ class StartupCheckerService(IStartupChecker):
 
         # Display info last
         for warning in info:
-            print(f"ℹ️  {warning.message}")
+            print(f"[INFO]️  {warning.message}")
             if warning.suggestion:
                 print(f"   {warning.suggestion}")
 
@@ -314,16 +314,19 @@ class StartupCheckerService(IStartupChecker):
         try:
             # Try to get config file path from config service
             config_file = getattr(self.config_service, "config_file", None)
-            if config_file and Path(config_file).exists():
-                if not os.access(config_file, os.R_OK):
-                    warnings.append(
-                        StartupWarning(
-                            category="config",
-                            message=f"Configuration file is not readable: {config_file}",
-                            suggestion=f"Run: chmod 644 {config_file}",
-                            severity="warning",
-                        )
+            if (
+                config_file
+                and Path(config_file).exists()
+                and not os.access(config_file, os.R_OK)
+            ):
+                warnings.append(
+                    StartupWarning(
+                        category="config",
+                        message=f"Configuration file is not readable: {config_file}",
+                        suggestion=f"Run: chmod 644 {config_file}",
+                        severity="warning",
                     )
+                )
         except Exception as e:
             self.logger.debug(f"Config file access check failed: {e}")
 

claude_mpm/services/cli/unified_dashboard_manager.py

@@ -334,14 +334,18 @@ class UnifiedDashboardManager(IUnifiedDashboardManager):
         Returns:
             Tuple of (dependencies_ok, error_message)
         """
-        try:
-            import aiohttp
-            import socketio
+        import importlib.util
+
+        missing = []
+        if importlib.util.find_spec("aiohttp") is None:
+            missing.append("aiohttp")
+        if importlib.util.find_spec("socketio") is None:
+            missing.append("socketio")
 
-            return True, None
-        except ImportError as e:
-            error_msg = f"Required dependencies missing: {e}"
+        if missing:
+            error_msg = f"Required dependencies missing: {', '.join(missing)}"
             return False, error_msg
+        return True, None
 
     def find_available_port(self, preferred_port: int = 8765) -> int:
         """

claude_mpm/services/command_deployment_service.py

@@ -84,13 +84,15 @@ class CommandDeploymentService(BaseService):
 
                 try:
                     # Check if file exists and if we should overwrite
-                    if target_file.exists() and not force:
-                        # Check if source is newer
-                        if source_file.stat().st_mtime <= target_file.stat().st_mtime:
-                            self.logger.debug(
-                                f"Skipping {source_file.name} - target is up to date"
-                            )
-                            continue
+                    if (
+                        target_file.exists()
+                        and not force
+                        and source_file.stat().st_mtime <= target_file.stat().st_mtime
+                    ):
+                        self.logger.debug(
+                            f"Skipping {source_file.name} - target is up to date"
+                        )
+                        continue
 
                     # Copy the file
                     shutil.copy2(source_file, target_file)

claude_mpm/services/core/path_resolver.py

@@ -431,11 +431,14 @@ class PathResolver(IPathResolver):
         ]
 
         for candidate in candidates:
-            if candidate and candidate.exists():
-                # Check for claude-mpm agents directory
-                if (candidate / "src" / "claude_mpm" / "agents").exists():
-                    self.logger.info(f"Found claude-mpm at: {candidate}")
-                    return candidate
+            if (
+                candidate
+                and candidate.exists()
+                and (candidate / "src" / "claude_mpm" / "agents").exists()
+            ):
+                # Found claude-mpm agents directory
+                self.logger.info(f"Found claude-mpm at: {candidate}")
+                return candidate
 
         return None
 

claude_mpm/services/diagnostics/checks/agent_check.py

@@ -111,10 +111,9 @@ class AgentCheck(BaseDiagnosticCheck):
 
     def _check_deployed_agents(self) -> DiagnosticResult:
         """Check deployed agents in both project and user directories."""
-        import os
 
         # Check project-level agents first (preferred in development)
-        project_agents_dir = Path(os.getcwd()) / ".claude" / "agents"
+        project_agents_dir = Path(Path.cwd()) / ".claude" / "agents"
         user_agents_dir = Path.home() / ".claude" / "agents"
 
         # Determine which directory to check
@@ -188,7 +187,6 @@ class AgentCheck(BaseDiagnosticCheck):
     def _check_agent_versions(self) -> DiagnosticResult:
         """Check if deployed agents are up-to-date."""
         try:
-            import os
 
             from ....services.agents.deployment.agent_version_manager import (
                 AgentVersionManager,
@@ -197,7 +195,7 @@ class AgentCheck(BaseDiagnosticCheck):
             version_manager = AgentVersionManager()
 
             # Check both project and user directories
-            project_agents_dir = Path(os.getcwd()) / ".claude" / "agents"
+            project_agents_dir = Path(Path.cwd()) / ".claude" / "agents"
             user_agents_dir = Path.home() / ".claude" / "agents"
 
             if project_agents_dir.exists():
@@ -259,14 +257,13 @@ class AgentCheck(BaseDiagnosticCheck):
     def _validate_agents(self) -> DiagnosticResult:
         """Validate agent configurations."""
         try:
-            import os
 
             from ....services.agents.deployment.agent_validator import AgentValidator
 
             AgentValidator()
 
             # Check both project and user directories
-            project_agents_dir = Path(os.getcwd()) / ".claude" / "agents"
+            project_agents_dir = Path(Path.cwd()) / ".claude" / "agents"
             user_agents_dir = Path.home() / ".claude" / "agents"
 
             if project_agents_dir.exists():
@@ -331,7 +328,7 @@ class AgentCheck(BaseDiagnosticCheck):
         issues = []
 
         # Check both project and user directories
-        project_agents_dir = Path(os.getcwd()) / ".claude" / "agents"
+        project_agents_dir = Path(Path.cwd()) / ".claude" / "agents"
         user_agents_dir = Path.home() / ".claude" / "agents"
 
         if project_agents_dir.exists():

claude_mpm/services/diagnostics/checks/installation_check.py

@@ -176,10 +176,11 @@ class InstallationCheck(BaseDiagnosticCheck):
 
         # 5. Check if running from source (development mode)
         claude_mpm_path = Path(__file__).parent.parent.parent.parent.parent
-        if (claude_mpm_path / "pyproject.toml").exists():
-            if (claude_mpm_path / ".git").exists():
-                methods_found.append("development")
-                details["source_path"] = str(claude_mpm_path)
+        if (claude_mpm_path / "pyproject.toml").exists() and (
+            claude_mpm_path / ".git"
+        ).exists():
+            methods_found.append("development")
+            details["source_path"] = str(claude_mpm_path)
 
         # 6. Check Homebrew Python
         if not in_venv and "/opt/homebrew" in exe_path:
@@ -191,21 +192,23 @@ class InstallationCheck(BaseDiagnosticCheck):
             details["homebrew_python"] = exe_path
 
         # 7. Check for system Python
-        if not in_venv and not methods_found:
-            if "/usr/bin/python" in exe_path or "/usr/local/bin/python" in exe_path:
-                methods_found.append("system")
-                details["system_python"] = exe_path
+        if (
+            not in_venv
+            and not methods_found
+            and ("/usr/bin/python" in exe_path or "/usr/local/bin/python" in exe_path)
+        ):
+            methods_found.append("system")
+            details["system_python"] = exe_path
 
         # 8. Additional check for pipx if not detected via venv
         if "pipx" not in methods_found:
             pipx_check = self._check_pipx_installation_status()
-            if pipx_check:
-                if not is_pipx_venv:
-                    # Pipx is installed but we're not running from it
-                    details["pipx_installed"] = True
-                    details["pipx_not_active"] = (
-                        "claude-mpm is installed via pipx but not currently running from pipx environment"
-                    )
+            if pipx_check and not is_pipx_venv:
+                # Pipx is installed but we're not running from it
+                details["pipx_installed"] = True
+                details["pipx_not_active"] = (
+                    "claude-mpm is installed via pipx but not currently running from pipx environment"
+                )
 
         # 9. Check pip installation status
         try:
@@ -445,7 +448,7 @@ class InstallationCheck(BaseDiagnosticCheck):
 
         # Check cgroup for container indicators
         try:
-            with open("/proc/1/cgroup") as f:
+            with Path("/proc/1/cgroup").open() as f:
                 cgroup = f.read()
                 if "docker" in cgroup:
                     return "Docker"

claude_mpm/services/diagnostics/checks/mcp_services_check.py

@@ -444,11 +444,14 @@ class MCPServicesCheck(BaseDiagnosticCheck):
             details["command_path"] = command_path
 
         # If not directly accessible, try pipx run command
-        if not accessible and "pipx_run_command" in config:
-            if self._verify_command_works(config["pipx_run_command"]):
-                accessible = True
-                details["accessible_via_pipx_run"] = True
-                details["pipx_run_available"] = True
+        if (
+            not accessible
+            and "pipx_run_command" in config
+            and self._verify_command_works(config["pipx_run_command"])
+        ):
+            accessible = True
+            details["accessible_via_pipx_run"] = True
+            details["pipx_run_available"] = True
 
         # Check for installation in various locations
         if not pipx_installed and not accessible:
@@ -690,35 +693,34 @@ class MCPServicesCheck(BaseDiagnosticCheck):
                 if any(
                     keyword in output
                     for keyword in ["version", "v1.", "v0.", "1.", "0."]
+                ) and not any(
+                    error in output
+                    for error in [
+                        "error",
+                        "not found",
+                        "no such",
+                        "command not found",
+                    ]
                 ):
-                    # But reject if it's an error message
-                    if not any(
-                        error in output
-                        for error in [
-                            "error",
-                            "not found",
-                            "no such",
-                            "command not found",
-                        ]
-                    ):
-                        return True
+                    return True
 
             # For some tools, non-zero return code is OK if version is shown
             elif "--version" in command or "--help" in command:
                 output = (result.stdout + result.stderr).lower()
                 # Must have version info and no error indicators
-                if "version" in output or "v1." in output or "v0." in output:
-                    if not any(
-                        error in output
-                        for error in [
-                            "error",
-                            "not found",
-                            "no such",
-                            "command not found",
-                            "traceback",
-                        ]
-                    ):
-                        return True
+                if (
+                    "version" in output or "v1." in output or "v0." in output
+                ) and not any(
+                    error in output
+                    for error in [
+                        "error",
+                        "not found",
+                        "no such",
+                        "command not found",
+                        "traceback",
+                    ]
+                ):
+                    return True
 
         except (subprocess.SubprocessError, FileNotFoundError, OSError):
             pass

claude_mpm/services/diagnostics/checks/startup_log_check.py

@@ -15,7 +15,7 @@ DESIGN DECISIONS:
 import re
 from datetime import datetime
 from pathlib import Path
-from typing import Any, Dict, Optional
+from typing import Any, ClassVar, Dict, Optional
 
 from ..models import DiagnosticResult, DiagnosticStatus
 from .base_check import BaseDiagnosticCheck
@@ -25,7 +25,7 @@ class StartupLogCheck(BaseDiagnosticCheck):
     """Analyze startup logs for errors and issues."""
 
     # Common error patterns and their fixes
-    ERROR_PATTERNS = {
+    ERROR_PATTERNS: ClassVar[dict[str, dict[str, Any]]] = {
         r"Agent deployment.*failed": (
             "Agent deployment failure",
             "Check agent configuration in .claude/agents/ and run 'claude-mpm deploy'",
@@ -186,7 +186,9 @@ class StartupLogCheck(BaseDiagnosticCheck):
         }
 
         try:
-            with open(log_file, encoding="utf-8") as f:
+            with Path(log_file).open(
+                encoding="utf-8",
+            ) as f:
                 lines = f.readlines()
 
             # Extract timestamp from filename or first line

claude_mpm/services/events/core.py

@@ -412,9 +412,8 @@ class EventBus(IEventBus):
         """
         try:
             # Apply filter if configured
-            if consumer.config.filter_func:
-                if not consumer.config.filter_func(event):
-                    return
+            if consumer.config.filter_func and not consumer.config.filter_func(event):
+                return
 
             # Apply transformation if configured
             if consumer.config.transform_func:

claude_mpm/services/framework_claude_md_generator/content_validator.py

@@ -5,14 +5,14 @@ Validates generated content structure and completeness.
 """
 
 import re
-from typing import List, Tuple
+from typing import ClassVar, List, Tuple
 
 
 class ContentValidator:
     """Validates framework CLAUDE.md content for completeness and correctness."""
 
     # Required sections that must be present
-    REQUIRED_SECTIONS = [
+    REQUIRED_SECTIONS: ClassVar[list[str]] = [
         (r"CLAUDE_MD_VERSION:", "Version metadata"),
         (r"## 🤖 AI ASSISTANT ROLE DESIGNATION", "Role designation section"),
         (r"## A\) AGENTS", "Agents section"),

claude_mpm/services/hook_installer_service.py

@@ -5,7 +5,6 @@ to enable monitor event forwarding via Socket.IO.
 """
 
 import json
-import os
 import shutil
 import stat
 import sys
@@ -280,8 +279,8 @@ class HookInstallerService:
                 return False
 
             # Make sure the script is executable
-            st = os.stat(hook_script)
-            os.chmod(hook_script, st.st_mode | stat.S_IEXEC)
+            st = Path(hook_script).stat()
+            Path(hook_script).chmod(st.st_mode | stat.S_IEXEC)
             self.logger.debug(f"Made hook script executable: {hook_script}")
 
             hook_script_path = str(hook_script.absolute())

claude_mpm/services/hook_service.py

@@ -278,12 +278,11 @@ class HookService(HookServiceInterface):
         if hook_type in ["pre_delegation", "post_delegation"]:
             # If any registered hooks are memory-related, check memory.enabled
             for hook in getattr(self, f"{hook_type}_hooks", []):
-                if "memory" in hook.name.lower():
-                    if not self.config.get("memory.enabled", True):
-                        self.logger.debug(
-                            "Memory hooks disabled via memory.enabled config"
-                        )
-                        return False
+                if "memory" in hook.name.lower() and not self.config.get(
+                    "memory.enabled", True
+                ):
+                    self.logger.debug("Memory hooks disabled via memory.enabled config")
+                    return False
 
         return True
 

claude_mpm/services/mcp_gateway/auto_configure.py

@@ -176,13 +176,12 @@ class MCPAutoConfigurator:
         print(f"(Auto-declining in {timeout} seconds)")
 
         # Use threading for cross-platform timeout support
+        # Python 3.7+ has queue built-in - no need to check, we require 3.10+
+        import importlib.util
         import threading
 
-        try:
-            # Python 3.7+ has queue built-in
-            import queue
-        except ImportError:
-            # Python 2.x fallback
+        if importlib.util.find_spec("queue") is None:
+            # Extremely unlikely in Python 3.10+, but for completeness
             pass
 
         user_input = None

claude_mpm/services/mcp_gateway/main.py

@@ -179,10 +179,13 @@ class MCPGatewayOrchestrator:
 
             # Load configuration
             self.configuration = MCPConfiguration()
-            if self.config_path and self.config_path.exists():
-                if not self.configuration.load_config(self.config_path):
-                    self.logger.error("Failed to load configuration")
-                    return False
+            if (
+                self.config_path
+                and self.config_path.exists()
+                and not self.configuration.load_config(self.config_path)
+            ):
+                self.logger.error("Failed to load configuration")
+                return False
 
             # Initialize tool registry with error handling
             try:

claude_mpm/services/mcp_gateway/server/stdio_server.py

@@ -18,6 +18,7 @@ import os
 import sys
 import time
 from datetime import timezone
+from pathlib import Path
 from typing import Any, Dict
 
 # Import MCP SDK components
@@ -541,12 +542,10 @@ class SimpleMCPServer:
 
                         result = f"Python: {sys.version}"
                     elif info_type == "cwd":
-                        import os
 
-                        result = f"Working Directory: {os.getcwd()}"
+                        result = f"Working Directory: {Path.cwd()}"
                     elif info_type == "all":
                         import datetime
-                        import os
                         import platform
                         import sys
 
@@ -554,7 +553,7 @@ class SimpleMCPServer:
                             f"=== System Status ===\n"
                             f"Platform: {platform.system()} {platform.release()}\n"
                             f"Python: {sys.version.split()[0]}\n"
-                            f"Working Directory: {os.getcwd()}\n"
+                            f"Working Directory: {Path.cwd()}\n"
                             f"Server: {self.name} v{self.version}\n"
                             f"Timestamp: {datetime.datetime.now(timezone.utc).isoformat()}\n"
                             f"Tools Available: status, document_summarizer{', ticket' if self.unified_ticket_tool else ''}"

claude_mpm/services/mcp_gateway/tools/document_summarizer.py

@@ -10,7 +10,6 @@ Part of ISS-0037: Document Summarizer Tool - Intelligent Document Processing
 
 import hashlib
 import mimetypes
-import os
 import re
 from collections import OrderedDict
 from datetime import datetime, timezone
@@ -240,7 +239,7 @@ class DocumentSummarizerTool(BaseToolAdapter):
 
     def _get_file_hash(self, file_path: str) -> str:
         """Generate hash for file caching."""
-        stat = os.stat(file_path)
+        stat = Path(file_path).stat()
         hash_input = f"{file_path}:{stat.st_size}:{stat.st_mtime}"
         return hashlib.md5(hash_input.encode()).hexdigest()
 

claude_mpm/services/mcp_service_verifier.py

@@ -328,13 +328,11 @@ class MCPServiceVerifier:
             if any(
                 word in output
                 for word in ["version", "usage", "help", service_name.lower()]
+            ) and not any(
+                error in output
+                for error in ["error", "not found", "traceback", "no module"]
             ):
-                # Make sure it's not an error
-                if not any(
-                    error in output
-                    for error in ["error", "not found", "traceback", "no module"]
-                ):
-                    return True
+                return True
 
             # Try pipx run as fallback
             if shutil.which("pipx"):
@@ -485,16 +483,19 @@ class MCPServiceVerifier:
                         }
             else:
                 # Direct execution - command should be a valid path
-                if not Path(command).exists() and command != installed_path:
-                    # Allow for relative paths that might resolve differently
-                    if not shutil.which(command):
-                        return {
-                            "configured": True,
-                            "correct": False,
-                            "command": command,
-                            "args": args,
-                            "issue": f"Command path does not exist: {command}",
-                        }
+                if (
+                    not Path(command).exists()
+                    and command != installed_path
+                    and not shutil.which(command)
+                ):
+                    # Command path does not exist and cannot be found
+                    return {
+                        "configured": True,
+                        "correct": False,
+                        "command": command,
+                        "args": args,
+                        "issue": f"Command path does not exist: {command}",
+                    }
 
                 # Check required args
                 for req_arg in required_args:
@@ -615,7 +616,7 @@ class MCPServiceVerifier:
             if diagnostic.fix_command.startswith("chmod "):
                 # Fix permissions
                 path = diagnostic.fix_command.replace("chmod +x ", "")
-                os.chmod(path, 0o755)
+                Path(path).chmod(0o755)
                 return True
 
             if "claude-mpm configure" in diagnostic.fix_command:

claude_mpm/services/memory/builder.py

@@ -22,7 +22,6 @@ documentation verbatim. Creates concise learnings that fit memory constraints
 while preserving essential information.
 """
 
-import os
 import re
 from datetime import datetime, timezone
 from pathlib import Path
@@ -118,7 +117,7 @@ class MemoryBuilder(LoggerMixin):
             self.config = config_loader.load_main_config()
         self.project_root = get_path_manager().project_root
         # Use current working directory by default, not project root
-        self.working_directory = working_directory or Path(os.getcwd())
+        self.working_directory = working_directory or Path(Path.cwd())
         self.memories_dir = self.working_directory / ".claude-mpm" / "memories"
         self.router = MemoryRouter(config)
         self.project_analyzer = ProjectAnalyzer(config, self.working_directory)

claude_mpm/services/memory/indexed_memory.py

@@ -573,7 +573,7 @@ class IndexedMemoryService:
         self.text_index.save(self.data_dir / "text_index.pkl")
 
         # Save other indexes
-        with open(self.data_dir / "indexes.pkl", "wb") as f:
+        with Path(self.data_dir / "indexes.pkl").open("wb") as f:
             pickle.dump(
                 {
                     "memories": self.memories,

claude_mpm/services/memory/optimizer.py

@@ -22,7 +22,6 @@ rather than aggressively removing content. Better to keep potentially useful
 information than lose important insights.
 """
 
-import os
 import re
 from datetime import datetime, timezone
 from difflib import SequenceMatcher
@@ -79,7 +78,7 @@ class MemoryOptimizer(LoggerMixin):
         self.config = config or Config()
         self.project_root = get_path_manager().get_project_root()
         # Use current working directory by default, not project root
-        self.working_directory = working_directory or Path(os.getcwd())
+        self.working_directory = working_directory or Path(Path.cwd())
         self.memories_dir = self.working_directory / ".claude-mpm" / "memories"
 
     def optimize_agent_memory(self, agent_id: str) -> Dict[str, Any]:

claude_mpm/services/monitor/daemon_manager.py

@@ -561,7 +561,7 @@ class DaemonManager:
             # Open log file for output redirection
             log_file_handle = None
             if self.log_file:
-                log_file_handle = open(self.log_file, "a")
+                log_file_handle = Path(self.log_file).open("a")
                 log_file = log_file_handle
             else:
                 log_file = subprocess.DEVNULL
@@ -858,7 +858,7 @@ class DaemonManager:
             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 to log file
@@ -955,7 +955,7 @@ class DaemonManager:
             except Exception as e:
                 # Try to write error to a debug file since logging might not work
                 try:
-                    with open("/tmp/daemon_debug_error.txt", "a") as debug:
+                    with Path("/tmp/daemon_debug_error.txt").open("a") as debug:
                         debug.write(f"Error reporting error: {e}\n")
                         debug.write(f"Status file: {self.startup_status_file}\n")
                 except Exception: