claude-mpm 4.20.3__py3-none-any.whl → 5.1.8__py3-none-any.whl

claude_mpm/cli/commands/run.py

@@ -32,7 +32,7 @@ from ..startup_logging import (
     log_startup_status,
     setup_startup_logging,
 )
-from ..utils import get_user_input, list_agent_versions_at_startup
+from ..utils import get_user_input
 
 
 def filter_claude_mpm_args(claude_args):
@@ -167,10 +167,10 @@ def create_session_context(session_id, session_manager):
 # Session Resumption
 
 You are resuming session {session_id[:8]}... which was:
-- Created: {session_data.get('created_at', 'unknown')}
-- Last used: {session_data.get('last_used', 'unknown')}
-- Context: {session_data.get('context', 'default')}
-- Use count: {session_data.get('use_count', 0)}
+- Created: {session_data.get("created_at", "unknown")}
+- Last used: {session_data.get("last_used", "unknown")}
+- Context: {session_data.get("context", "default")}
+- Use count: {session_data.get("use_count", 0)}
 """
 
     # Add information about agents previously run in this session
@@ -361,7 +361,7 @@ class RunCommand(BaseCommand):
                 loader = AgentDependencyLoader(auto_install=False)
 
                 # Check if agents have changed
-                has_changed, deployment_hash = loader.has_agents_changed()
+                _has_changed, deployment_hash = loader.has_agents_changed()
 
                 # Determine if we should check dependencies
                 should_check, check_reason = smart_checker.should_check_dependencies(
@@ -443,7 +443,7 @@ class RunCommand(BaseCommand):
             else:
                 # Find available port and start server
                 websocket_port = dashboard_manager.find_available_port(8765)
-                success, server_info = dashboard_manager.start_server(
+                success, _server_info = dashboard_manager.start_server(
                     port=websocket_port
                 )
 
@@ -569,41 +569,6 @@ class RunCommand(BaseCommand):
             return False
 
 
-def _ensure_mcp_services_configured(logger):
-    """
-    Ensure MCP services are configured in .mcp.json on startup.
-
-    This function automatically configures the core MCP services
-    (mcp-vector-search, mcp-browser, mcp-ticketer) if they're not
-    already configured in the project's .mcp.json file.
-
-    Args:
-        logger: Logger instance for output
-    """
-    try:
-        from ...services.mcp_config_manager import MCPConfigManager
-
-        logger.debug("Checking MCP service configuration...")
-        manager = MCPConfigManager()
-
-        # Check and auto-configure missing MCP services
-        success, message = manager.ensure_mcp_services_configured()
-
-        if success:
-            if "already configured" not in message.lower():
-                logger.info(message)
-                print(f"✅ {message}")
-            else:
-                logger.debug(message)
-        else:
-            logger.warning(f"MCP auto-configuration issue: {message}")
-            # Don't fail the session, just warn
-
-    except Exception as e:
-        logger.debug(f"MCP auto-configuration skipped: {e}")
-        # Don't fail the session if auto-configuration fails
-
-
 def _handle_reload_agents(logger):
     """
     Handle the --reload-agents flag by deleting all local claude-mpm system agents.
@@ -703,8 +668,10 @@ def run_session_legacy(args):
     Args:
         args: Parsed command line arguments
     """
-    # Set up startup logging to file early in the process
-    setup_startup_logging(Path.cwd())
+    # Only setup startup logging if user wants logging
+    if args.logging != LogLevel.OFF.value:
+        # Set up startup logging to file early in the process
+        setup_startup_logging(Path.cwd())
 
     logger = get_logger("cli")
     if args.logging != LogLevel.OFF.value:
@@ -712,12 +679,13 @@ def run_session_legacy(args):
         # Log file already announced in startup_logging.py when created
 
     # Clean up old startup logs (using configured retention count)
-    try:
-        deleted_count = cleanup_old_startup_logs(Path.cwd())
-        if deleted_count > 0:
-            logger.debug(f"Cleaned up {deleted_count} old startup log files")
-    except Exception as e:
-        logger.debug(f"Failed to clean up old logs: {e}")
+    if args.logging != LogLevel.OFF.value:
+        try:
+            deleted_count = cleanup_old_startup_logs(Path.cwd())
+            if deleted_count > 0:
+                logger.debug(f"Cleaned up {deleted_count} old startup log files")
+        except Exception as e:
+            logger.debug(f"Failed to clean up old logs: {e}")
 
     # Log MCP and monitor startup status
     if args.logging != LogLevel.OFF.value:
@@ -735,10 +703,7 @@ def run_session_legacy(args):
     if getattr(args, "reload_agents", False):
         _handle_reload_agents(logger)
 
-    # Auto-configure MCP services on startup
-    _ensure_mcp_services_configured(logger)
-
-    # Trigger vector search indexing after MCP is configured
+    # Trigger vector search indexing
     try:
         from ...cli.startup_logging import start_vector_search_indexing
 
@@ -805,134 +770,127 @@ def run_session_legacy(args):
     if getattr(args, "no_native_agents", False):
         print("Native agents disabled")
     else:
-        # List deployed agent versions at startup
-        list_agent_versions_at_startup()
+        # Agent versions removed from startup display - use /mpm-agents to view
+        # list_agent_versions_at_startup()
+        pass
 
-        # Smart dependency checking - only when needed
-        if getattr(args, "check_dependencies", True):  # Default to checking
-            try:
-                from ...utils.agent_dependency_loader import AgentDependencyLoader
-                from ...utils.dependency_cache import SmartDependencyChecker
-                from ...utils.environment_context import should_prompt_for_dependencies
-
-                # Initialize smart checker
-                smart_checker = SmartDependencyChecker()
-                loader = AgentDependencyLoader(auto_install=False)
+    # Smart dependency checking - only when needed
+    if getattr(args, "check_dependencies", True):  # Default to checking
+        try:
+            from ...utils.agent_dependency_loader import AgentDependencyLoader
+            from ...utils.dependency_cache import SmartDependencyChecker
+            from ...utils.environment_context import should_prompt_for_dependencies
 
-                # Check if agents have changed
-                has_changed, deployment_hash = loader.has_agents_changed()
+            # Initialize smart checker
+            smart_checker = SmartDependencyChecker()
+            loader = AgentDependencyLoader(auto_install=False)
 
-                # Determine if we should check dependencies
-                should_check, check_reason = smart_checker.should_check_dependencies(
-                    force_check=getattr(args, "force_check_dependencies", False),
-                    deployment_hash=deployment_hash,
-                )
+            # Check if agents have changed
+            _has_changed, deployment_hash = loader.has_agents_changed()
 
-                if should_check:
-                    # Check if we're in an environment where prompting makes sense
-                    can_prompt, prompt_reason = should_prompt_for_dependencies(
-                        force_prompt=getattr(args, "force_prompt", False),
-                        force_skip=getattr(args, "no_prompt", False),
-                    )
+            # Determine if we should check dependencies
+            should_check, check_reason = smart_checker.should_check_dependencies(
+                force_check=getattr(args, "force_check_dependencies", False),
+                deployment_hash=deployment_hash,
+            )
 
-                    logger.debug(f"Dependency check needed: {check_reason}")
-                    logger.debug(
-                        f"Interactive prompting: {can_prompt} ({prompt_reason})"
-                    )
+            if should_check:
+                # Check if we're in an environment where prompting makes sense
+                can_prompt, prompt_reason = should_prompt_for_dependencies(
+                    force_prompt=getattr(args, "force_prompt", False),
+                    force_skip=getattr(args, "no_prompt", False),
+                )
 
-                    # Get or check dependencies
-                    results, was_cached = smart_checker.get_or_check_dependencies(
-                        loader=loader,
-                        force_check=getattr(args, "force_check_dependencies", False),
-                    )
+                logger.debug(f"Dependency check needed: {check_reason}")
+                logger.debug(f"Interactive prompting: {can_prompt} ({prompt_reason})")
 
-                    # Show summary if there are missing dependencies
-                    if results["summary"]["missing_python"]:
-                        missing_count = len(results["summary"]["missing_python"])
-                        print(f"⚠️  {missing_count} agent dependencies missing")
-
-                        if can_prompt and missing_count > 0:
-                            # Interactive prompt for installation
-                            print("\n📦 Missing dependencies detected:")
-                            for dep in results["summary"]["missing_python"][:5]:
-                                print(f"   - {dep}")
-                            if missing_count > 5:
-                                print(f"   ... and {missing_count - 5} more")
-
-                            print("\nWould you like to install them now?")
-                            print("  [y] Yes, install missing dependencies")
-                            print("  [n] No, continue without installing")
-                            print("  [q] Quit")
-
-                            sys.stdout.flush()  # Ensure prompt is displayed before input
-
-                            # Check if we're in a TTY environment for proper input handling
-                            if not sys.stdin.isatty():
-                                # In non-TTY environment (like pipes), use readline
-                                print("\nChoice [y/n/q]: ", end="", flush=True)
-                                try:
-                                    response = sys.stdin.readline().strip().lower()
-                                    # Handle various line endings and control characters
-                                    response = (
-                                        response.replace("\r", "")
-                                        .replace("\n", "")
-                                        .strip()
-                                    )
-                                except (EOFError, KeyboardInterrupt):
-                                    response = "q"
-                            else:
-                                # In TTY environment, use normal input()
-                                try:
-                                    response = (
-                                        input("\nChoice [y/n/q]: ").strip().lower()
-                                    )
-                                except (EOFError, KeyboardInterrupt):
-                                    response = "q"
+                # Get or check dependencies
+                results, was_cached = smart_checker.get_or_check_dependencies(
+                    loader=loader,
+                    force_check=getattr(args, "force_check_dependencies", False),
+                )
 
+                # Show summary if there are missing dependencies
+                if results["summary"]["missing_python"]:
+                    missing_count = len(results["summary"]["missing_python"])
+                    print(f"⚠️  {missing_count} agent dependencies missing")
+
+                    if can_prompt and missing_count > 0:
+                        # Interactive prompt for installation
+                        print("\n📦 Missing dependencies detected:")
+                        for dep in results["summary"]["missing_python"][:5]:
+                            print(f"   - {dep}")
+                        if missing_count > 5:
+                            print(f"   ... and {missing_count - 5} more")
+
+                        print("\nWould you like to install them now?")
+                        print("  [y] Yes, install missing dependencies")
+                        print("  [n] No, continue without installing")
+                        print("  [q] Quit")
+
+                        sys.stdout.flush()  # Ensure prompt is displayed before input
+
+                        # Check if we're in a TTY environment for proper input handling
+                        if not sys.stdin.isatty():
+                            # In non-TTY environment (like pipes), use readline
+                            print("\nChoice [y/n/q]: ", end="", flush=True)
                             try:
-                                if response == "y":
-                                    print("\n🔧 Installing missing dependencies...")
-                                    loader.auto_install = True
-                                    (
-                                        success,
-                                        error,
-                                    ) = loader.install_missing_dependencies(
-                                        results["summary"]["missing_python"]
-                                    )
-                                    if success:
-                                        print("✅ Dependencies installed successfully")
-                                        # Invalidate cache after installation
-                                        smart_checker.cache.invalidate(deployment_hash)
-                                    else:
-                                        print(f"❌ Installation failed: {error}")
-                                elif response == "q":
-                                    print("👋 Exiting...")
-                                    return
-                                else:
-                                    print(
-                                        "⏩ Continuing without installing dependencies"
-                                    )
+                                response = sys.stdin.readline().strip().lower()
+                                # Handle various line endings and control characters
+                                response = (
+                                    response.replace("\r", "").replace("\n", "").strip()
+                                )
                             except (EOFError, KeyboardInterrupt):
-                                print("\n⏩ Continuing without installing dependencies")
+                                response = "q"
                         else:
-                            # Non-interactive environment or prompting disabled
-                            print(
-                                "   Run 'pip install \"claude-mpm[agents]\"' to install all agent dependencies"
-                            )
-                            if not can_prompt:
-                                logger.debug(
-                                    f"Not prompting for installation: {prompt_reason}"
+                            # In TTY environment, use normal input()
+                            try:
+                                response = input("\nChoice [y/n/q]: ").strip().lower()
+                            except (EOFError, KeyboardInterrupt):
+                                response = "q"
+
+                        try:
+                            if response == "y":
+                                print("\n🔧 Installing missing dependencies...")
+                                loader.auto_install = True
+                                (
+                                    success,
+                                    error,
+                                ) = loader.install_missing_dependencies(
+                                    results["summary"]["missing_python"]
                                 )
-                    elif was_cached:
-                        logger.debug("Dependencies satisfied (cached result)")
+                                if success:
+                                    print("✅ Dependencies installed successfully")
+                                    # Invalidate cache after installation
+                                    smart_checker.cache.invalidate(deployment_hash)
+                                else:
+                                    print(f"❌ Installation failed: {error}")
+                            elif response == "q":
+                                print("👋 Exiting...")
+                                return
+                            else:
+                                print("⏩ Continuing without installing dependencies")
+                        except (EOFError, KeyboardInterrupt):
+                            print("\n⏩ Continuing without installing dependencies")
                     else:
-                        logger.debug("All dependencies satisfied")
+                        # Non-interactive environment or prompting disabled
+                        print(
+                            "   Run 'pip install \"claude-mpm[agents]\"' to install all agent dependencies"
+                        )
+                        if not can_prompt:
+                            logger.debug(
+                                f"Not prompting for installation: {prompt_reason}"
+                            )
+                elif was_cached:
+                    logger.debug("Dependencies satisfied (cached result)")
                 else:
-                    logger.debug(f"Skipping dependency check: {check_reason}")
+                    logger.debug("All dependencies satisfied")
+            else:
+                logger.debug(f"Skipping dependency check: {check_reason}")
 
-            except Exception as e:
-                if args.logging != LogLevel.OFF.value:
-                    logger.debug(f"Could not check agent dependencies: {e}")
+        except Exception as e:
+            if args.logging != LogLevel.OFF.value:
+                logger.debug(f"Could not check agent dependencies: {e}")
                 # Continue anyway - don't block execution
 
     # Create simple runner