claude-mpm 4.3.12__py3-none-any.whl → 4.3.14__py3-none-any.whl

claude_mpm/cli/commands/mcp_external_commands.py

@@ -4,8 +4,6 @@ This module provides commands for managing external MCP services
 like mcp-vector-search and mcp-browser.
 """
 
-import sys
-
 
 class MCPExternalCommands:
     """Handles MCP external service commands."""
@@ -14,7 +12,7 @@ class MCPExternalCommands:
         """Initialize the MCP external commands handler."""
         self.logger = logger
 
-    def manage_external(self, args):
+    def manage_external(self, args):  # noqa: PLR0911
         """Manage external MCP services.
 
         Args:
@@ -24,7 +22,7 @@ class MCPExternalCommands:
             int: Exit code (0 for success, non-zero for failure)
         """
         # Get the external subcommand if it exists
-        external_action = getattr(args, 'external_action', None)
+        external_action = getattr(args, "external_action", None)
 
         if not external_action:
             # No subcommand provided, show help
@@ -34,18 +32,17 @@ class MCPExternalCommands:
         # Route to appropriate handler
         if external_action == "setup":
             return self._setup_external(args)
-        elif external_action == "list":
+        if external_action == "list":
             return self._list_external(args)
-        elif external_action == "check":
+        if external_action == "check":
             return self._check_external(args)
-        elif external_action == "fix-browser":
+        if external_action == "fix-browser":
             return self._fix_browser(args)
-        elif external_action == "detect":
+        if external_action == "detect":
             return self._detect_and_update(args)
-        else:
-            print(f"Unknown external subcommand: {external_action}")
-            self._show_help()
-            return 1
+        print(f"Unknown external subcommand: {external_action}")
+        self._show_help()
+        return 1
 
     def _setup_external(self, args):
         """Setup external MCP services in Claude Desktop.
@@ -72,7 +69,7 @@ class MCPExternalCommands:
 
         # Then configure in Claude Desktop
         print("\n2️⃣  Configuring Claude Desktop...")
-        force = getattr(args, 'force', False)
+        force = getattr(args, "force", False)
         if setup.setup_external_services(force=force):
             print("\n✅ External services setup completed successfully!")
             print("\nNext steps:")
@@ -80,10 +77,9 @@ class MCPExternalCommands:
             print("2. Check status with: claude-mpm mcp external list")
             print("3. The services will be available in Claude as separate MCP servers")
             return 0
-        else:
-            print("\n❌ Failed to setup external services")
-            print("Please check the error messages above and try again")
-            return 1
+        print("\n❌ Failed to setup external services")
+        print("Please check the error messages above and try again")
+        return 1
 
     def _list_external(self, args):
         """List external MCP services and their status.
@@ -112,16 +108,24 @@ class MCPExternalCommands:
         print("🔍 Checking External MCP Services Configuration")
         print("=" * 50)
 
-        from pathlib import Path
         import json
+        from pathlib import Path
 
         # Check Claude Desktop configuration
         config_paths = [
-            Path.home() / "Library" / "Application Support" / "Claude" / "claude_desktop_config.json",  # macOS
+            Path.home()
+            / "Library"
+            / "Application Support"
+            / "Claude"
+            / "claude_desktop_config.json",  # macOS
             Path.home() / ".config" / "Claude" / "claude_desktop_config.json",  # Linux
-            Path.home() / "AppData" / "Roaming" / "Claude" / "claude_desktop_config.json",  # Windows
+            Path.home()
+            / "AppData"
+            / "Roaming"
+            / "Claude"
+            / "claude_desktop_config.json",  # Windows
             Path.home() / ".claude" / "claude_desktop_config.json",  # Alternative
-            Path.home() / ".claude.json"  # Legacy
+            Path.home() / ".claude.json",  # Legacy
         ]
 
         config_found = False
@@ -164,7 +168,7 @@ class MCPExternalCommands:
 
         packages = [
             ("mcp-vector-search", "mcp_vector_search"),
-            ("mcp-browser", "mcp_browser")
+            ("mcp-browser", "mcp_browser"),
         ]
 
         for package_name, module_name in packages:
@@ -189,8 +193,7 @@ class MCPExternalCommands:
         setup = MCPExternalServicesSetup(self.logger)
         if setup.fix_browser_configuration():
             return 0
-        else:
-            return 1
+        return 1
 
     def _detect_and_update(self, args):
         """Auto-detect MCP service installations and update configuration.
@@ -206,7 +209,7 @@ class MCPExternalCommands:
         from .mcp_setup_external import MCPExternalServicesSetup
 
         setup = MCPExternalServicesSetup(self.logger)
-        force = getattr(args, 'force', False)
+        force = getattr(args, "force", False)
 
         if setup.update_mcp_json_with_detected(force=force):
             print("\n✅ Configuration updated successfully!")
@@ -214,28 +217,33 @@ class MCPExternalCommands:
             print("1. Review the .mcp.json file to verify the configuration")
             print("2. Restart Claude Desktop to load the updated services")
             return 0
-        else:
-            print("\n❌ Failed to update configuration")
-            return 1
+        print("\n❌ Failed to update configuration")
+        return 1
 
     def _show_help(self):
         """Show help for external commands."""
         print("\nMCP External Services Management")
         print("=" * 40)
         print("\nAvailable commands:")
-        print("  setup       - Setup external MCP services (mcp-vector-search, mcp-browser)")
+        print(
+            "  setup       - Setup external MCP services (mcp-vector-search, mcp-browser)"
+        )
         print("  list        - List available external services and their status")
         print("  check       - Check configuration and installation status")
-        print("  detect      - Auto-detect installations and update .mcp.json (prioritizes local dev)")
+        print(
+            "  detect      - Auto-detect installations and update .mcp.json (prioritizes local dev)"
+        )
         print("  fix-browser - Fix mcp-browser configuration to use pipx installation")
         print("\nUsage:")
         print("  claude-mpm mcp external setup         # Interactive setup")
         print("  claude-mpm mcp external setup --force # Force reconfiguration")
         print("  claude-mpm mcp external detect        # Auto-detect and update config")
-        print("  claude-mpm mcp external detect --force # Force update even if configured")
+        print(
+            "  claude-mpm mcp external detect --force # Force update even if configured"
+        )
         print("  claude-mpm mcp external list          # Show service status")
         print("  claude-mpm mcp external check         # Detailed configuration check")
         print("  claude-mpm mcp external fix-browser   # Fix mcp-browser to use pipx")
         print("\nExternal services provide additional capabilities:")
         print("  - mcp-vector-search: Semantic code search with embeddings")
-        print("  - mcp-browser: Web browsing and content extraction")
+        print("  - mcp-browser: Web browsing and content extraction")

claude_mpm/cli/commands/mcp_install_commands.py

@@ -61,7 +61,13 @@ class MCPInstallCommands:
             external_setup = MCPExternalServicesSetup(self.logger)
 
             # Check if user wants to set up external services
-            response = input("\nDo you want to set up external MCP services (mcp-vector-search, mcp-browser)? (Y/n): ").strip().lower()
+            response = (
+                input(
+                    "\nDo you want to set up external MCP services (mcp-vector-search, mcp-browser)? (Y/n): "
+                )
+                .strip()
+                .lower()
+            )
             if response in ["", "y", "yes"]:
                 # Install Python packages for external services
                 external_setup.check_and_install_pip_packages()
@@ -73,7 +79,9 @@ class MCPInstallCommands:
                     print("⚠️ Some external services may not have been configured")
             else:
                 print("⏭️ Skipping external services setup")
-                print("   You can set them up later with: claude-mpm mcp external setup")
+                print(
+                    "   You can set them up later with: claude-mpm mcp external setup"
+                )
 
             print("\n✅ Configuration completed successfully")
             print("\n🎉 MCP Gateway is ready to use!")
@@ -167,9 +175,17 @@ class MCPInstallCommands:
 
         # Try multiple possible locations for Claude Desktop config
         possible_paths = [
-            Path.home() / "Library" / "Application Support" / "Claude" / "claude_desktop_config.json",  # macOS
+            Path.home()
+            / "Library"
+            / "Application Support"
+            / "Claude"
+            / "claude_desktop_config.json",  # macOS
             Path.home() / ".config" / "Claude" / "claude_desktop_config.json",  # Linux
-            Path.home() / "AppData" / "Roaming" / "Claude" / "claude_desktop_config.json",  # Windows
+            Path.home()
+            / "AppData"
+            / "Roaming"
+            / "Claude"
+            / "claude_desktop_config.json",  # Windows
             Path.home() / ".claude" / "claude_desktop_config.json",  # Alternative
             Path.home() / ".claude.json",  # Legacy
         ]
@@ -181,11 +197,23 @@ class MCPInstallCommands:
         # If none exist, return the platform-appropriate default
         system = platform.system()
         if system == "Darwin":  # macOS
-            return Path.home() / "Library" / "Application Support" / "Claude" / "claude_desktop_config.json"
-        elif system == "Windows":
-            return Path.home() / "AppData" / "Roaming" / "Claude" / "claude_desktop_config.json"
-        else:  # Linux and others
-            return Path.home() / ".config" / "Claude" / "claude_desktop_config.json"
+            return (
+                Path.home()
+                / "Library"
+                / "Application Support"
+                / "Claude"
+                / "claude_desktop_config.json"
+            )
+        if system == "Windows":
+            return (
+                Path.home()
+                / "AppData"
+                / "Roaming"
+                / "Claude"
+                / "claude_desktop_config.json"
+            )
+        # Linux and others
+        return Path.home() / ".config" / "Claude" / "claude_desktop_config.json"
 
     def _find_claude_mpm_executable(self):
         """Find the claude-mpm executable path.
@@ -281,7 +309,7 @@ class MCPInstallCommands:
                         existing_config = json.load(f)
                     config = existing_config
                     print("   Force mode: Overwriting existing configuration")
-                except:
+                except Exception:
                     pass  # File doesn't exist or is invalid, start fresh
 
         # Ensure mcpServers section exists