claude-mpm 5.1.9__py3-none-any.whl → 5.4.48__py3-none-any.whl

claude_mpm/cli/chrome_devtools_installer.py

@@ -0,0 +1,175 @@
+"""
+Chrome DevTools MCP Auto-Installer
+===================================
+
+Automatically installs and configures chrome-devtools-mcp on startup
+to enable browser automation and web interaction capabilities in Claude Code.
+
+WHY: Users should have browser automation capabilities out-of-the-box without
+manual MCP server configuration.
+
+DESIGN DECISION: Non-blocking installation that doesn't prevent startup if it fails.
+Uses the `claude mcp add` command for native Claude Code integration.
+"""
+
+import json
+import subprocess
+from pathlib import Path
+from typing import Optional, Tuple
+
+from ..core.logger import get_logger
+
+
+class ChromeDevToolsInstaller:
+    """Manages automatic installation of chrome-devtools-mcp."""
+
+    MCP_SERVER_NAME = "chrome-devtools"
+    INSTALL_COMMAND = ["npx", "chrome-devtools-mcp@latest"]
+
+    def __init__(self):
+        """Initialize the chrome-devtools installer."""
+        self.logger = get_logger(__name__)
+        self.claude_config_path = Path.home() / ".claude.json"
+
+    def is_already_configured(self) -> bool:
+        """
+        Check if chrome-devtools-mcp is already configured in Claude.
+
+        Returns:
+            True if already configured, False otherwise
+        """
+        try:
+            if not self.claude_config_path.exists():
+                self.logger.debug("Claude config file not found")
+                return False
+
+            with open(self.claude_config_path) as f:
+                config = json.load(f)
+
+            # Check if chrome-devtools is in mcpServers
+            mcp_servers = config.get("mcpServers", {})
+            is_configured = self.MCP_SERVER_NAME in mcp_servers
+
+            if is_configured:
+                self.logger.debug(
+                    f"{self.MCP_SERVER_NAME} already configured in Claude"
+                )
+            else:
+                self.logger.debug(f"{self.MCP_SERVER_NAME} not found in Claude config")
+
+            return is_configured
+
+        except json.JSONDecodeError as e:
+            self.logger.warning(f"Failed to parse Claude config: {e}")
+            return False
+        except Exception as e:
+            self.logger.debug(f"Error checking Chrome DevTools configuration: {e}")
+            return False
+
+    def install_mcp_server(self) -> Tuple[bool, Optional[str]]:
+        """
+        Install chrome-devtools-mcp using the claude CLI.
+
+        Returns:
+            Tuple of (success: bool, error_message: Optional[str])
+        """
+        try:
+            # Build the command: claude mcp add chrome-devtools -- npx chrome-devtools-mcp@latest
+            command = [
+                "claude",
+                "mcp",
+                "add",
+                self.MCP_SERVER_NAME,
+                "--",
+            ] + self.INSTALL_COMMAND
+
+            self.logger.debug(f"Running: {' '.join(command)}")
+
+            # Run the command with timeout
+            result = subprocess.run(
+                command,
+                capture_output=True,
+                text=True,
+                timeout=30,  # 30 second timeout
+                check=False,
+            )
+
+            if result.returncode == 0:
+                self.logger.info(
+                    f"Successfully installed {self.MCP_SERVER_NAME} MCP server"
+                )
+                return True, None
+
+            # Command failed
+            error_msg = (
+                f"Failed to install {self.MCP_SERVER_NAME}: {result.stderr.strip()}"
+            )
+            self.logger.warning(error_msg)
+            return False, error_msg
+
+        except subprocess.TimeoutExpired:
+            error_msg = (
+                f"Installation of {self.MCP_SERVER_NAME} timed out after 30 seconds"
+            )
+            self.logger.warning(error_msg)
+            return False, error_msg
+
+        except FileNotFoundError:
+            error_msg = "Claude CLI not found. Is Claude Code installed?"
+            self.logger.warning(error_msg)
+            return False, error_msg
+
+        except Exception as e:
+            error_msg = f"Unexpected error installing {self.MCP_SERVER_NAME}: {e}"
+            self.logger.warning(error_msg)
+            return False, error_msg
+
+    def auto_install(self, quiet: bool = False) -> bool:
+        """
+        Automatically install chrome-devtools-mcp if not already configured.
+
+        Args:
+            quiet: If True, suppress console output
+
+        Returns:
+            True if installation succeeded or already configured, False otherwise
+        """
+        # Check if already configured
+        if self.is_already_configured():
+            if not quiet:
+                print("✓ Chrome DevTools MCP ready", flush=True)
+            return True
+
+        # Show progress feedback
+        if not quiet:
+            print("Installing Chrome DevTools MCP...", end=" ", flush=True)
+
+        # Install the MCP server
+        success, error = self.install_mcp_server()
+
+        if success:
+            if not quiet:
+                print("✓", flush=True)
+            return True
+
+        # Installation failed
+        if not quiet:
+            print("(skipped)", flush=True)
+
+        # Log the error but don't fail startup
+        self.logger.debug(f"Chrome DevTools installation skipped: {error}")
+        return False
+
+
+def auto_install_chrome_devtools(quiet: bool = False) -> bool:
+    """
+    Convenience function to auto-install chrome-devtools-mcp.
+
+    Args:
+        quiet: If True, suppress console output
+
+    Returns:
+        True if installation succeeded or already configured, False otherwise
+    """
+    installer = ChromeDevToolsInstaller()
+    return installer.auto_install(quiet=quiet)

claude_mpm/cli/commands/agent_state_manager.py

@@ -111,14 +111,14 @@ class SimpleAgentManager:
         local_agents = self._discover_local_template_agents()
         agents.extend(local_agents)
 
-        # Discover remote agents if requested
+        # Discover Git-sourced agents if requested
         if include_remote:
             try:
-                remote_agents = self._discover_remote_agents()
-                agents.extend(remote_agents)
-                self.logger.info(f"Discovered {len(remote_agents)} remote agents")
+                git_agents = self._discover_git_agents()
+                agents.extend(git_agents)
+                self.logger.info(f"Discovered {len(git_agents)} Git-sourced agents")
             except Exception as e:
-                self.logger.warning(f"Failed to discover remote agents: {e}")
+                self.logger.warning(f"Failed to discover Git-sourced agents: {e}")
 
         # Sort agents by name for consistent display
         agents.sort(key=lambda a: a.name)
@@ -152,7 +152,7 @@ class SimpleAgentManager:
 
                     # Get metadata for display info
                     metadata = template_data.get("metadata", {})
-                    metadata.get("name", agent_id)
+                    display_name = metadata.get("name", agent_id)
                     description = metadata.get(
                         "description", "No description available"
                     )
@@ -182,9 +182,10 @@ class SimpleAgentManager:
                         dependencies=display_tools,
                     )
 
-                    # Set deployment status
+                    # Set deployment status and display name
                     agent_config.is_deployed = is_deployed
                     agent_config.source_type = "local"
+                    agent_config.display_name = display_name
 
                     agents.append(agent_config)
 
@@ -207,20 +208,20 @@ class SimpleAgentManager:
 
         return agents
 
-    def _discover_remote_agents(self) -> List[AgentConfig]:
-        """Discover agents from remote Git sources using GitSourceManager."""
+    def _discover_git_agents(self) -> List[AgentConfig]:
+        """Discover agents from Git sources using GitSourceManager."""
         try:
             from claude_mpm.services.agents.git_source_manager import GitSourceManager
 
-            # Initialize source manager (uses ~/.claude-mpm/cache/remote-agents by default)
+            # Initialize source manager (uses ~/.claude-mpm/cache/agents by default)
             source_manager = GitSourceManager()
 
             # Discover all cached agents from all repositories
-            remote_agent_dicts = source_manager.list_cached_agents()
+            git_agent_dicts = source_manager.list_cached_agents()
 
             # Convert to AgentConfig objects for UI display
             agents = []
-            for agent_dict in remote_agent_dicts:
+            for agent_dict in git_agent_dicts:
                 # Extract metadata
                 metadata = agent_dict.get("metadata", {})
                 agent_id = agent_dict.get("agent_id", "unknown")
@@ -235,7 +236,7 @@ class SimpleAgentManager:
                 # Create AgentConfig with source information
                 # Store full agent_dict for later use in deployment
                 agent_config = AgentConfig(
-                    name=agent_id,  # Use agent_id as name for uniqueness
+                    name=name,  # Use display name for UI
                     description=(
                         f"[{category}] {description[:60]}..."
                         if len(description) > 60
@@ -248,6 +249,7 @@ class SimpleAgentManager:
                 agent_config.source_type = "remote"
                 agent_config.is_deployed = is_deployed
                 agent_config.display_name = name
+                agent_config.agent_id = agent_id  # Store technical ID for reference
                 agent_config.full_agent_id = agent_id
                 agent_config.source_dict = agent_dict  # Store full dict for deployment
 
@@ -272,9 +274,9 @@ class SimpleAgentManager:
             True if agent is deployed, False otherwise
         """
         # Check virtual deployment state (primary method)
+        # Only checking project-level deployment in simplified architecture
         deployment_state_paths = [
             Path.cwd() / ".claude" / "agents" / ".mpm_deployment_state",
-            Path.home() / ".claude" / "agents" / ".mpm_deployment_state",
         ]
 
         for state_path in deployment_state_paths:
@@ -319,8 +321,8 @@ class SimpleAgentManager:
             leaf_name = agent_id.split("/")[-1]
             agent_file_names.append(f"{leaf_name}.md")
 
-        # Check .claude-mpm/agents/ directory (project level)
-        project_agents_dir = Path.cwd() / ".claude-mpm" / "agents"
+        # Check .claude/agents/ directory (project deployment)
+        project_agents_dir = Path.cwd() / ".claude" / "agents"
         if project_agents_dir.exists():
             for agent_file_name in agent_file_names:
                 agent_file = project_agents_dir / agent_file_name
@@ -330,15 +332,4 @@ class SimpleAgentManager:
                     )
                     return True
 
-        # Check ~/.claude/agents/ directory (user level)
-        user_agents_dir = Path.home() / ".claude" / "agents"
-        if user_agents_dir.exists():
-            for agent_file_name in agent_file_names:
-                agent_file = user_agents_dir / agent_file_name
-                if agent_file.exists():
-                    self.logger.debug(
-                        f"Agent {agent_id} found as physical file: {agent_file}"
-                    )
-                    return True
-
         return False

claude_mpm/cli/commands/agents.py

@@ -157,9 +157,6 @@ class AgentsCommand(AgentCommand):
                 "configure": self._configure_deployment,
                 # Migration command (DEPRECATION support)
                 "migrate-to-project": self._migrate_to_project,
-                # Auto-configuration commands (TSK-0054 Phase 5)
-                "detect": self._detect_toolchain,
-                "recommend": self._recommend_agents,
                 # Agent selection modes (Phase 3: 1M-382)
                 "deploy-minimal": self._deploy_minimal_configuration,
                 "deploy-auto": self._deploy_auto_configure,
@@ -564,7 +561,7 @@ class AgentsCommand(AgentCommand):
         """Deploy agents using two-phase sync: cache → deploy.
 
         Phase 3 Integration (1M-486): Uses Git sync service for deployment.
-        - Phase 1: Sync agents to ~/.claude-mpm/cache/remote-agents/ (if needed)
+        - Phase 1: Sync agents to ~/.claude-mpm/cache/agents/ (if needed)
         - Phase 2: Deploy from cache to project .claude-mpm/agents/
 
         This replaces the old single-tier deployment with a multi-project
@@ -1825,34 +1822,6 @@ class AgentsCommand(AgentCommand):
                 f"Error in interactive configuration: {e}"
             )
 
-    def _detect_toolchain(self, args) -> CommandResult:
-        """Detect project toolchain without deploying agents.
-
-        Part of TSK-0054 Phase 5: Auto-configuration CLI integration.
-        """
-        try:
-            from .agents_detect import AgentsDetectCommand
-
-            cmd = AgentsDetectCommand()
-            return cmd.run(args)
-        except Exception as e:
-            self.logger.error(f"Error detecting toolchain: {e}", exc_info=True)
-            return CommandResult.error_result(f"Error detecting toolchain: {e}")
-
-    def _recommend_agents(self, args) -> CommandResult:
-        """Recommend agents based on project toolchain.
-
-        Part of TSK-0054 Phase 5: Auto-configuration CLI integration.
-        """
-        try:
-            from .agents_recommend import AgentsRecommendCommand
-
-            cmd = AgentsRecommendCommand()
-            return cmd.run(args)
-        except Exception as e:
-            self.logger.error(f"Error recommending agents: {e}", exc_info=True)
-            return CommandResult.error_result(f"Error recommending agents: {e}")
-
     def _migrate_to_project(self, args) -> CommandResult:
         """Migrate user-level agents to project-level.
 
@@ -2175,7 +2144,7 @@ class AgentsCommand(AgentCommand):
             )
 
             # Get remote agents cache directory
-            cache_dir = Path.home() / ".claude-mpm" / "cache" / "remote-agents"
+            cache_dir = Path.home() / ".claude-mpm" / "cache" / "agents"
 
             if not cache_dir.exists():
                 return CommandResult.error_result(
@@ -2223,7 +2192,7 @@ class AgentsCommand(AgentCommand):
 
             # Get agents from collection
             service = MultiSourceAgentDeploymentService()
-            cache_dir = Path.home() / ".claude-mpm" / "cache" / "remote-agents"
+            cache_dir = Path.home() / ".claude-mpm" / "cache" / "agents"
             agents = service.get_agents_by_collection(collection_id, cache_dir)
 
             if not agents:
@@ -2280,7 +2249,7 @@ class AgentsCommand(AgentCommand):
 
             # Get agents from collection
             service = MultiSourceAgentDeploymentService()
-            cache_dir = Path.home() / ".claude-mpm" / "cache" / "remote-agents"
+            cache_dir = Path.home() / ".claude-mpm" / "cache" / "agents"
             agents = service.get_agents_by_collection(collection_id, cache_dir)
 
             if not agents:
@@ -2335,7 +2304,7 @@ class AgentsCommand(AgentCommand):
         try:
             from ...services.agents.cache_git_manager import CacheGitManager
 
-            cache_dir = Path.home() / ".claude-mpm" / "cache" / "remote-agents"
+            cache_dir = Path.home() / ".claude-mpm" / "cache" / "agents"
             manager = CacheGitManager(cache_dir)
 
             if not manager.is_git_repo():
@@ -2408,7 +2377,7 @@ class AgentsCommand(AgentCommand):
         try:
             from ...services.agents.cache_git_manager import CacheGitManager
 
-            cache_dir = Path.home() / ".claude-mpm" / "cache" / "remote-agents"
+            cache_dir = Path.home() / ".claude-mpm" / "cache" / "agents"
             manager = CacheGitManager(cache_dir)
 
             if not manager.is_git_repo():
@@ -2435,7 +2404,7 @@ class AgentsCommand(AgentCommand):
         try:
             from ...services.agents.cache_git_manager import CacheGitManager
 
-            cache_dir = Path.home() / ".claude-mpm" / "cache" / "remote-agents"
+            cache_dir = Path.home() / ".claude-mpm" / "cache" / "agents"
             manager = CacheGitManager(cache_dir)
 
             if not manager.is_git_repo():
@@ -2467,7 +2436,7 @@ class AgentsCommand(AgentCommand):
         try:
             from ...services.agents.cache_git_manager import CacheGitManager
 
-            cache_dir = Path.home() / ".claude-mpm" / "cache" / "remote-agents"
+            cache_dir = Path.home() / ".claude-mpm" / "cache" / "agents"
             manager = CacheGitManager(cache_dir)
 
             if not manager.is_git_repo():
@@ -2515,7 +2484,7 @@ class AgentsCommand(AgentCommand):
         try:
             from ...services.agents.cache_git_manager import CacheGitManager
 
-            cache_dir = Path.home() / ".claude-mpm" / "cache" / "remote-agents"
+            cache_dir = Path.home() / ".claude-mpm" / "cache" / "agents"
             manager = CacheGitManager(cache_dir)
 
             if not manager.is_git_repo():