claude-mpm 4.1.0__py3-none-any.whl → 4.1.2__py3-none-any.whl

claude_mpm/cli/commands/doctor.py

@@ -20,7 +20,7 @@ from ...services.diagnostics import DiagnosticRunner, DoctorReporter
 
 def add_doctor_parser(subparsers):
     """Add doctor command parser.
-    
+
     WHY: This command helps users diagnose and fix issues with their
     claude-mpm installation, providing clear actionable feedback.
     """
@@ -28,72 +28,67 @@ def add_doctor_parser(subparsers):
         "doctor",
         aliases=["diagnose", "check-health"],
         help="Run comprehensive diagnostics on claude-mpm installation",
-        description="Run comprehensive health checks on your claude-mpm installation and configuration"
+        description="Run comprehensive health checks on your claude-mpm installation and configuration",
     )
-    
+
     parser.add_argument(
         "--verbose",
         "-v",
         action="store_true",
-        help="Show detailed diagnostic information"
+        help="Show detailed diagnostic information",
     )
-    
+
     parser.add_argument(
-        "--json",
-        action="store_true",
-        help="Output results in JSON format"
+        "--json", action="store_true", help="Output results in JSON format"
     )
-    
+
     parser.add_argument(
-        "--markdown",
-        action="store_true",
-        help="Output results in Markdown format"
+        "--markdown", action="store_true", help="Output results in Markdown format"
     )
-    
+
     parser.add_argument(
         "--fix",
         action="store_true",
-        help="Attempt to fix issues automatically (experimental)"
+        help="Attempt to fix issues automatically (experimental)",
     )
-    
+
     parser.add_argument(
         "--checks",
         nargs="+",
         choices=[
-            "installation", "configuration", "filesystem",
-            "claude", "agents", "mcp", "monitor", "common"
+            "installation",
+            "configuration",
+            "filesystem",
+            "claude",
+            "agents",
+            "mcp",
+            "monitor",
+            "common",
         ],
-        help="Run only specific checks"
+        help="Run only specific checks",
     )
-    
+
     parser.add_argument(
         "--parallel",
         action="store_true",
-        help="Run checks in parallel for faster execution"
+        help="Run checks in parallel for faster execution",
     )
-    
-    parser.add_argument(
-        "--no-color",
-        action="store_true",
-        help="Disable colored output"
-    )
-    
+
     parser.add_argument(
-        "--output",
-        "-o",
-        type=Path,
-        help="Save output to file"
+        "--no-color", action="store_true", help="Disable colored output"
     )
-    
+
+    parser.add_argument("--output", "-o", type=Path, help="Save output to file")
+
     parser.set_defaults(func=doctor_command)
 
 
 def run_doctor(args):
     """Main entry point for doctor command (used by CLI).
-    
+
     Args:
         args: Parsed command-line arguments
-        
+
     Returns:
         Exit code (0 for success, 1 for warnings, 2 for errors)
     """
@@ -102,19 +97,19 @@ def run_doctor(args):
 
 def doctor_command(args):
     """Execute the doctor command.
-    
+
     WHY: Provides a single entry point for system diagnostics, helping users
     quickly identify and resolve issues with their claude-mpm setup.
-    
+
     Args:
         args: Parsed command-line arguments
-        
+
     Returns:
         Exit code (0 for success, 1 for warnings, 2 for errors)
     """
     # Configure logging
     logger = logging.getLogger(__name__)
-    
+
     # Determine output format
     if args.json:
         output_format = "json"
@@ -122,13 +117,10 @@ def doctor_command(args):
         output_format = "markdown"
     else:
         output_format = "terminal"
-    
+
     # Create diagnostic runner
-    runner = DiagnosticRunner(
-        verbose=args.verbose,
-        fix=args.fix
-    )
-    
+    runner = DiagnosticRunner(verbose=args.verbose, fix=args.fix)
+
     # Run diagnostics
     try:
         if args.checks:
@@ -143,57 +135,55 @@ def doctor_command(args):
             # Run all checks sequentially
             logger.info("Running comprehensive diagnostics")
             summary = runner.run_diagnostics()
-    
+
     except KeyboardInterrupt:
         print("\nDiagnostics interrupted by user")
         return 130
     except Exception as e:
         logger.error(f"Diagnostic failed: {e}")
-        print(f"\n❌ Diagnostic failed: {str(e)}")
+        print(f"\n❌ Diagnostic failed: {e!s}")
         if args.verbose:
             import traceback
+
             traceback.print_exc()
         return 2
-    
+
     # Create reporter
-    reporter = DoctorReporter(
-        use_color=not args.no_color,
-        verbose=args.verbose
-    )
-    
+    reporter = DoctorReporter(use_color=not args.no_color, verbose=args.verbose)
+
     # Output results
     if args.output:
         # Save to file
         try:
             import sys
+
             original_stdout = sys.stdout
-            with open(args.output, 'w') as f:
+            with open(args.output, "w") as f:
                 sys.stdout = f
                 reporter.report(summary, format=output_format)
             sys.stdout = original_stdout
             print(f"Report saved to: {args.output}")
         except Exception as e:
             logger.error(f"Failed to save report: {e}")
-            print(f"❌ Failed to save report: {str(e)}")
+            print(f"❌ Failed to save report: {e!s}")
             # Still output to terminal
             reporter.report(summary, format=output_format)
     else:
         # Output to terminal
         reporter.report(summary, format=output_format)
-    
+
     # Determine exit code based on results
     if summary.error_count > 0:
         return 2  # Errors found
-    elif summary.warning_count > 0:
+    if summary.warning_count > 0:
         return 1  # Warnings found
-    else:
-        return 0  # All OK
+    return 0  # All OK
 
 
 # Optional: Standalone execution for testing
 if __name__ == "__main__":
     import argparse
-    
+
     parser = argparse.ArgumentParser(description="Claude MPM Doctor")
     parser.add_argument("--verbose", "-v", action="store_true")
     parser.add_argument("--json", action="store_true")
@@ -201,9 +191,9 @@ if __name__ == "__main__":
     parser.add_argument("--no-color", action="store_true")
     parser.add_argument("--checks", nargs="+")
     parser.add_argument("--parallel", action="store_true")
-    
+
     args = parser.parse_args()
     args.markdown = False
     args.output = None
-    
-    sys.exit(doctor_command(args))
+
+    sys.exit(doctor_command(args))

claude_mpm/cli/commands/info.py

@@ -13,7 +13,7 @@ DESIGN DECISIONS:
 
 import shutil
 from pathlib import Path
-from typing import Dict, Any, List
+from typing import Any, Dict
 
 from ..shared import BaseCommand, CommandResult
 
@@ -35,15 +35,16 @@ class InfoCommand(BaseCommand):
             # Gather system information
             info_data = self._gather_system_info(args)
 
-            output_format = getattr(args, 'format', 'text')
+            output_format = getattr(args, "format", "text")
 
-            if output_format in ['json', 'yaml']:
+            if output_format in ["json", "yaml"]:
                 # Structured output
-                return CommandResult.success_result("System information retrieved", data=info_data)
-            else:
-                # Text output
-                self._display_text_info(info_data)
-                return CommandResult.success_result("System information displayed")
+                return CommandResult.success_result(
+                    "System information retrieved", data=info_data
+                )
+            # Text output
+            self._display_text_info(info_data)
+            return CommandResult.success_result("System information displayed")
 
         except Exception as e:
             self.logger.error(f"Error gathering system info: {e}", exc_info=True)
@@ -57,20 +58,26 @@ class InfoCommand(BaseCommand):
             from claude_mpm.core.framework_loader import FrameworkLoader
 
         # Framework information
-        framework_path = getattr(args, 'framework_path', None)
+        framework_path = getattr(args, "framework_path", None)
         loader = FrameworkLoader(framework_path)
 
         framework_info = {
             "loaded": loader.framework_content["loaded"],
-            "name": "claude-multiagent-pm" if loader.framework_content["loaded"] else "Not found",
-            "version": loader.framework_content.get('version', 'unknown'),
+            "name": (
+                "claude-multiagent-pm"
+                if loader.framework_content["loaded"]
+                else "Not found"
+            ),
+            "version": loader.framework_content.get("version", "unknown"),
             "path": str(loader.framework_path) if loader.framework_path else None,
-            "agents": loader.get_agent_list() if loader.framework_content["loaded"] else []
+            "agents": (
+                loader.get_agent_list() if loader.framework_content["loaded"] else []
+            ),
         }
 
         # Configuration information
         config_info = {
-            "log_directory": getattr(args, 'log_dir', None) or '~/.claude-mpm/logs'
+            "log_directory": getattr(args, "log_dir", None) or "~/.claude-mpm/logs"
         }
 
         # Agent hierarchy
@@ -79,12 +86,12 @@ class InfoCommand(BaseCommand):
         if loader.agent_registry:
             hierarchy = loader.agent_registry.get_agent_hierarchy()
             agent_hierarchy = {
-                "project_agents": len(hierarchy['project']),
-                "user_agents": len(hierarchy['user']),
-                "system_agents": len(hierarchy['system']),
-                "project_agent_list": hierarchy['project'],
-                "user_agent_list": hierarchy['user'],
-                "system_agent_list": hierarchy['system']
+                "project_agents": len(hierarchy["project"]),
+                "user_agents": len(hierarchy["user"]),
+                "system_agents": len(hierarchy["system"]),
+                "project_agent_list": hierarchy["project"],
+                "user_agent_list": hierarchy["user"],
+                "system_agent_list": hierarchy["system"],
             }
             core_agents = loader.agent_registry.get_core_agents()
 
@@ -96,7 +103,7 @@ class InfoCommand(BaseCommand):
             "configuration": config_info,
             "agent_hierarchy": agent_hierarchy,
             "core_agents": core_agents,
-            "dependencies": dependencies
+            "dependencies": dependencies,
         }
 
     def _check_dependencies(self) -> Dict[str, Any]:
@@ -108,20 +115,21 @@ class InfoCommand(BaseCommand):
         dependencies["claude_cli"] = {
             "installed": bool(claude_path),
             "path": claude_path,
-            "status": "✓ Installed" if claude_path else "✗ Not found in PATH"
+            "status": "✓ Installed" if claude_path else "✗ Not found in PATH",
         }
 
         # Check ai-trackdown-pytools
         try:
             import ai_trackdown_pytools
+
             dependencies["ai_trackdown_pytools"] = {
                 "installed": True,
-                "status": "✓ Installed"
+                "status": "✓ Installed",
             }
         except ImportError:
             dependencies["ai_trackdown_pytools"] = {
                 "installed": False,
-                "status": "✗ Not installed"
+                "status": "✗ Not installed",
             }
 
         # Check Claude Code hooks
@@ -130,7 +138,11 @@ class InfoCommand(BaseCommand):
             "installed": claude_settings.exists(),
             "settings_path": str(claude_settings),
             "status": "✓ Installed" if claude_settings.exists() else "✗ Not installed",
-            "install_command": "python scripts/install_hooks.py" if not claude_settings.exists() else None
+            "install_command": (
+                "python scripts/install_hooks.py"
+                if not claude_settings.exists()
+                else None
+            ),
         }
 
         return dependencies
@@ -193,7 +205,7 @@ def show_info(args):
     result = command.execute(args)
 
     # Print result if structured output format is requested
-    if hasattr(args, 'format') and args.format in ['json', 'yaml']:
+    if hasattr(args, "format") and args.format in ["json", "yaml"]:
         command.print_result(result, args)
 
     return result.exit_code

claude_mpm/cli/commands/mcp.py

@@ -1,5 +1,3 @@
-from pathlib import Path
-
 """
 MCP Gateway command implementation for claude-mpm.
 
@@ -11,11 +9,9 @@ that dispatches to specific subcommand handlers, maintaining consistency
 with other command modules like agents.py and memory.py.
 """
 
-import asyncio
 import json
 import sys
 from pathlib import Path
-from typing import Any, Dict, List, Optional
 
 from ...constants import MCPCommands
 from ...core.logger import get_logger
@@ -70,7 +66,7 @@ def manage_mcp(args):
                 MCPGateway = None
             else:
                 print(
-                    f"\nError: MCP Gateway services not fully available",
+                    "\nError: MCP Gateway services not fully available",
                     file=sys.stderr,
                 )
                 print(f"Details: {e}", file=sys.stderr)
@@ -185,9 +181,9 @@ def _show_status(
     # Show available tools count
     if ToolRegistry:
         try:
-            registry = ToolRegistry()
+            ToolRegistry()
             # Don't initialize fully, just check
-            print(f"\n🔧 Tools: Check with 'claude-mpm mcp tools'")
+            print("\n🔧 Tools: Check with 'claude-mpm mcp tools'")
         except:
             print("\n🔧 Tools: Registry not available")
 

claude_mpm/cli/commands/mcp_command_router.py

@@ -5,9 +5,6 @@ Extracted from mcp.py to reduce complexity and improve maintainability.
 """
 
 import asyncio
-import os
-import sys
-from typing import Any
 
 from ...constants import MCPCommands
 
@@ -24,38 +21,37 @@ class MCPCommandRouter:
         if args.mcp_command == MCPCommands.START.value:
             return asyncio.run(self._start_server(args))
 
-        elif args.mcp_command == MCPCommands.STOP.value:
+        if args.mcp_command == MCPCommands.STOP.value:
             return self._stop_server(args)
 
-        elif args.mcp_command == MCPCommands.STATUS.value:
+        if args.mcp_command == MCPCommands.STATUS.value:
             return self._show_status(args)
 
-        elif args.mcp_command == MCPCommands.TOOLS.value:
+        if args.mcp_command == MCPCommands.TOOLS.value:
             return self._manage_tools(args)
 
-        elif args.mcp_command == MCPCommands.REGISTER.value:
+        if args.mcp_command == MCPCommands.REGISTER.value:
             return self._register_tool(args)
 
-        elif args.mcp_command == MCPCommands.TEST.value:
+        if args.mcp_command == MCPCommands.TEST.value:
             return self._test_tool(args)
 
-        elif args.mcp_command == MCPCommands.INSTALL.value:
+        if args.mcp_command == MCPCommands.INSTALL.value:
             return self._install_gateway(args)
 
-        elif args.mcp_command == MCPCommands.CONFIG.value:
+        if args.mcp_command == MCPCommands.CONFIG.value:
             return self._manage_config(args)
 
-        elif args.mcp_command == MCPCommands.SERVER.value:
+        if args.mcp_command == MCPCommands.SERVER.value:
             return self._run_server(args)
 
-        elif args.mcp_command == "cleanup":
+        if args.mcp_command == "cleanup":
             return self._cleanup_locks(args)
 
-        else:
-            self.logger.error(f"Unknown MCP command: {args.mcp_command}")
-            print(f"Unknown MCP command: {args.mcp_command}")
-            self._show_help()
-            return 1
+        self.logger.error(f"Unknown MCP command: {args.mcp_command}")
+        print(f"Unknown MCP command: {args.mcp_command}")
+        self._show_help()
+        return 1
 
     async def _start_server(self, args) -> int:
         """Start server command handler."""
@@ -124,7 +120,7 @@ class MCPCommandRouter:
         """Run server command handler - direct server execution."""
         # Simply delegate to the async start_server method using asyncio.run
         from .mcp_server_commands import MCPServerCommands
-        
+
         handler = MCPServerCommands(self.logger)
         return asyncio.run(handler.start_server(args))
 

claude_mpm/cli/commands/mcp_install_commands.py

@@ -33,6 +33,7 @@ class MCPInstallCommands:
         print("\n1️⃣  Checking MCP package installation...")
         try:
             import mcp
+
             print("✅ MCP package already installed")
         except ImportError:
             print("📦 Installing MCP package...")
@@ -56,9 +57,8 @@ class MCPInstallCommands:
                 print("2. Test the server: claude-mpm mcp server --test")
                 print("3. Check status: claude-mpm mcp status")
                 return 0
-            else:
-                print("❌ Configuration failed")
-                return 1
+            print("❌ Configuration failed")
+            return 1
 
         except Exception as e:
             print(f"❌ Error during configuration: {e}")
@@ -66,7 +66,7 @@ class MCPInstallCommands:
 
     def _configure_claude_desktop(self, force=False):
         """Configure Claude Code to use the MCP gateway via CLI command.
-        
+
         WHY: Claude Code reads MCP server configurations from ~/.claude.json
         (not ~/.claude/settings.local.json). This method updates that file
         to include the claude-mpm-gateway server configuration.
@@ -77,10 +77,6 @@ class MCPInstallCommands:
         Returns:
             bool: True if configuration was successful
         """
-        import json
-        import platform
-        from pathlib import Path
-        from datetime import datetime
 
         # Determine Claude Code config path
         config_path = self._get_claude_config_path()
@@ -102,15 +98,15 @@ class MCPInstallCommands:
             return False
 
         # Determine if we need to use -m claude_mpm or direct command
-        if claude_mpm_path.endswith(('python', 'python3', 'python.exe', 'python3.exe')):
+        if claude_mpm_path.endswith(("python", "python3", "python.exe", "python3.exe")):
             # Using Python interpreter directly
             mcp_config = {
                 "command": claude_mpm_path,
                 "args": ["-m", "claude_mpm", "mcp", "server"],
                 "env": {
                     "PYTHONPATH": str(Path(__file__).parent.parent.parent.parent),
-                    "MCP_MODE": "production"
-                }
+                    "MCP_MODE": "production",
+                },
             }
         else:
             # Using installed claude-mpm command
@@ -119,8 +115,8 @@ class MCPInstallCommands:
                 "args": ["mcp", "server"],
                 "env": {
                     "PYTHONPATH": str(Path(__file__).parent.parent.parent.parent),
-                    "MCP_MODE": "production"
-                }
+                    "MCP_MODE": "production",
+                },
             }
 
         # Update configuration
@@ -143,7 +139,6 @@ class MCPInstallCommands:
         Returns:
             Path or None: Path to Claude Code config file
         """
-        from pathlib import Path
 
         # Claude Code reads MCP server configurations from ~/.claude.json
         # This is the actual file that Claude Code uses for MCP servers
@@ -167,6 +162,7 @@ class MCPInstallCommands:
             str or None: Path to claude-mpm executable
         """
         import sys
+
         from ...core.unified_paths import get_executable_path
 
         # Use the enhanced unified path manager for executable detection
@@ -178,6 +174,7 @@ class MCPInstallCommands:
         # Fallback: Use Python module invocation if no executable found
         try:
             import claude_mpm
+
             print(f"   Using Python module: {sys.executable} -m claude_mpm")
             return sys.executable
         except ImportError:
@@ -204,25 +201,33 @@ class MCPInstallCommands:
             if not force:
                 # Check if claude-mpm-gateway already exists
                 try:
-                    with open(config_path, 'r') as f:
+                    with open(config_path) as f:
                         existing_config = json.load(f)
 
-                    if (existing_config.get("mcpServers", {}).get("claude-mpm-gateway") and
-                        not force):
+                    if (
+                        existing_config.get("mcpServers", {}).get("claude-mpm-gateway")
+                        and not force
+                    ):
                         print("⚠️  claude-mpm-gateway is already configured")
-                        response = input("Do you want to overwrite it? (y/N): ").strip().lower()
-                        if response not in ['y', 'yes']:
+                        response = (
+                            input("Do you want to overwrite it? (y/N): ")
+                            .strip()
+                            .lower()
+                        )
+                        if response not in ["y", "yes"]:
                             print("❌ Configuration cancelled")
                             return None
 
                     config = existing_config
 
-                except (json.JSONDecodeError, IOError) as e:
+                except (OSError, json.JSONDecodeError) as e:
                     print(f"⚠️  Error reading existing config: {e}")
                     print("Creating backup and starting fresh...")
 
                     # Create backup
-                    backup_path = config_path.with_suffix(f'.backup.{datetime.now().strftime("%Y%m%d_%H%M%S")}.json')
+                    backup_path = config_path.with_suffix(
+                        f'.backup.{datetime.now().strftime("%Y%m%d_%H%M%S")}.json'
+                    )
                     try:
                         config_path.rename(backup_path)
                         print(f"   Backup created: {backup_path}")
@@ -231,7 +236,7 @@ class MCPInstallCommands:
             else:
                 # Force mode - create backup but proceed
                 try:
-                    with open(config_path, 'r') as f:
+                    with open(config_path) as f:
                         existing_config = json.load(f)
                     config = existing_config
                     print("   Force mode: Overwriting existing configuration")
@@ -261,7 +266,7 @@ class MCPInstallCommands:
             config_path.parent.mkdir(parents=True, exist_ok=True)
 
             # Write configuration with nice formatting
-            with open(config_path, 'w') as f:
+            with open(config_path, "w") as f:
                 json.dump(config, f, indent=2)
 
             print(f"\n✅ Configuration saved to {config_path}")