claude-mpm 4.24.0__py3-none-any.whl → 5.4.41__py3-none-any.whl

claude_mpm/cli/commands/agents_cleanup.py

@@ -0,0 +1,210 @@
+"""Agent cleanup command for syncing, installing, and removing old underscore-named agents.
+
+WHY: After standardizing template filenames from underscores to dashes, users need
+a way to sync the latest agents, install them with correct names, and remove old
+underscore-named duplicates.
+
+DESIGN DECISION: Three-phase cleanup process:
+1. Sync agents from remote repository (download latest)
+2. Deploy synced agents to target directory (force overwrite)
+3. Remove old underscore-named agents that have dash equivalents
+
+IMPLEMENTATION NOTE: Uses name similarity matching to identify old agents that
+should be removed (e.g., python_engineer.md → python-engineer.md).
+"""
+
+import logging
+from pathlib import Path
+from typing import List
+
+from ...config.agent_sources import AgentSourceConfiguration
+from ...services.agents.sources.git_source_sync_service import GitSourceSyncService
+
+logger = logging.getLogger(__name__)
+
+
+def _normalize_agent_name(name: str) -> str:
+    """Normalize agent name by removing extensions and converting to lowercase.
+
+    Args:
+        name: Agent filename (e.g., "python_engineer.md" or "python-engineer.md")
+
+    Returns:
+        Normalized name (e.g., "pythonengineer")
+    """
+    # Remove file extension
+    name = name.replace(".md", "").replace(".json", "")
+    # Remove dashes and underscores for comparison
+    name = name.replace("-", "").replace("_", "")
+    return name.lower()
+
+
+def _find_old_underscore_agents(
+    deployed_agents: List[Path], new_agents: List[str]
+) -> List[Path]:
+    """Find old underscore-named agents that have dash-named equivalents.
+
+    Args:
+        deployed_agents: List of paths to deployed agent files
+        new_agents: List of new agent filenames (with dashes)
+
+    Returns:
+        List of paths to old agents that should be removed
+    """
+    # Normalize new agent names for comparison
+    normalized_new = {_normalize_agent_name(agent) for agent in new_agents}
+
+    old_agents = []
+    for agent_path in deployed_agents:
+        agent_name = agent_path.name
+
+        # Check if this is an underscore-named agent
+        if "_" in agent_name and "-" not in agent_name:
+            normalized = _normalize_agent_name(agent_name)
+
+            # If we have a dash-named equivalent, mark for removal
+            if normalized in normalized_new:
+                old_agents.append(agent_path)
+
+    return old_agents
+
+
+def handle_agents_cleanup(args) -> int:
+    """Handle the 'claude-mpm agents cleanup' command.
+
+    This command performs a complete agent cleanup:
+    1. Syncs agents from remote repository
+    2. Deploys agents with new naming convention (force overwrite)
+    3. Removes old underscore-named agents that have dash equivalents
+
+    Args:
+        args: Parsed command-line arguments with:
+            - dry_run: Show what would be done without doing it
+            - target: Target directory for deployment
+            - global_deployment: Deploy to global ~/.claude/agents/
+
+    Returns:
+        Exit code (0 for success, 1 for error)
+    """
+    try:
+        dry_run = args.dry_run
+
+        # Determine target directory
+        if args.global_deployment:
+            target_dir = Path.home() / ".claude" / "agents"
+        elif args.target:
+            target_dir = args.target
+        else:
+            target_dir = Path.cwd() / ".claude-mpm" / "agents"
+
+        print("\n🧹 Agent Cleanup")
+        print(f"{'=' * 60}")
+
+        if dry_run:
+            print("🔍 DRY RUN MODE - No changes will be made\n")
+
+        # Phase 1: Sync agents from remote
+        print("📥 Phase 1: Syncing agents from remote repository...")
+
+        config = AgentSourceConfiguration.load()
+        repositories = config.get_enabled_repositories()
+
+        if not repositories:
+            print(
+                "⚠️  No agent sources configured. Run 'claude-mpm agent-source add' first."
+            )
+            return 1
+
+        sync_service = GitSourceSyncService()
+        synced_count = 0
+
+        for repo in repositories:
+            print(f"  Syncing from {repo.url}...")
+            try:
+                result = sync_service.sync_agents(show_progress=False)
+                synced_count += result.get("total_downloaded", 0)
+                print(f"  ✓ Synced {result.get('total_downloaded', 0)} agents")
+            except Exception as e:
+                print(f"  ✗ Failed to sync: {e}")
+                continue
+
+        if synced_count == 0:
+            print(
+                "⚠️  No agents synced. Check your network connection or agent sources."
+            )
+            return 1
+
+        print(f"\n✓ Synced {synced_count} agents total")
+
+        # Phase 2: Deploy agents (force overwrite)
+        print(f"\n📦 Phase 2: Deploying agents to {target_dir}...")
+
+        if not dry_run:
+            target_dir.mkdir(parents=True, exist_ok=True)
+
+        # Get list of agents from cache
+        cache_dir = sync_service.get_cached_agents_dir()
+        cached_agents = list(cache_dir.glob("*.md"))
+        new_agent_names = [agent.name for agent in cached_agents]
+
+        if not dry_run:
+            result = sync_service.deploy_agents_to_project(Path.cwd(), force=True)
+            deployed_count = len(result.get("deployed", [])) + len(
+                result.get("updated", [])
+            )
+            print(f"✓ Deployed {deployed_count} agents")
+        else:
+            print(f"  Would deploy {len(cached_agents)} agents:")
+            for agent in cached_agents[:10]:  # Show first 10
+                print(f"    - {agent.name}")
+            if len(cached_agents) > 10:
+                print(f"    ... and {len(cached_agents) - 10} more")
+
+        # Phase 3: Remove old underscore-named agents
+        print("\n🗑️  Phase 3: Removing old underscore-named agents...")
+
+        # Find deployed agents
+        deployed_agents = list(target_dir.glob("*.md"))
+        old_agents = _find_old_underscore_agents(deployed_agents, new_agent_names)
+
+        if not old_agents:
+            print("✓ No old underscore-named agents found")
+        else:
+            print(f"  Found {len(old_agents)} old agents to remove:")
+            for agent in old_agents:
+                # Find the new equivalent name
+                normalized = _normalize_agent_name(agent.name)
+                dash_equiv = [
+                    name
+                    for name in new_agent_names
+                    if _normalize_agent_name(name) == normalized
+                ]
+                equiv_str = f" → {dash_equiv[0]}" if dash_equiv else ""
+
+                print(f"    - {agent.name}{equiv_str}")
+
+                if not dry_run:
+                    agent.unlink()
+
+            if not dry_run:
+                print(f"✓ Removed {len(old_agents)} old agents")
+            else:
+                print(f"  Would remove {len(old_agents)} old agents")
+
+        # Summary
+        print(f"\n{'=' * 60}")
+        print("✅ Cleanup complete!")
+        print("\nSummary:")
+        print(f"  • Synced: {synced_count} agents")
+        print(f"  • Deployed: {len(cached_agents)} agents")
+        print(f"  • Removed: {len(old_agents)} old agents")
+
+        if dry_run:
+            print("\n💡 Run without --dry-run to apply changes")
+
+        return 0
+
+    except Exception as e:
+        logger.exception("Agent cleanup failed")
+        print(f"\n❌ Error: {e}")
+        return 1

claude_mpm/cli/commands/agents_discover.py

@@ -0,0 +1,338 @@
+"""Agent discovery command for browsing available agents from configured sources.
+
+WHY: This module implements the 'agents discover' command, allowing users to
+browse and filter agents from configured Git sources. Provides rich filtering
+by category, language, framework, platform, and specialization.
+
+DESIGN DECISION: Rich console output using rich library
+
+Rationale: Users need beautiful, informative output when discovering agents.
+The rich library (already a dependency) provides excellent table formatting,
+color support, and tree views.
+
+Trade-offs:
+- User Experience: Beautiful output improves discoverability
+- Complexity: Rich table formatting adds some code overhead
+- Performance: Rendering is fast enough for CLI use (<50ms)
+
+Example Usage:
+    >>> claude-mpm agents discover --category engineer/backend
+    >>> claude-mpm agents discover --language python --verbose
+    >>> claude-mpm agents discover --framework react --format json
+"""
+
+import json
+import logging
+from typing import Any, Dict, List
+
+from rich.console import Console
+
+from ...config.agent_sources import AgentSourceConfiguration
+from ...services.agents.git_source_manager import GitSourceManager
+
+logger = logging.getLogger(__name__)
+console = Console()
+
+
+def _group_agents_by_category(
+    agents: List[Dict[str, Any]],
+) -> Dict[str, List[Dict[str, Any]]]:
+    """Group agents by their category for organized display.
+
+    Args:
+        agents: List of agent dictionaries
+
+    Returns:
+        Dictionary mapping category names to lists of agents
+
+    Example:
+        >>> agents = [
+        ...     {"agent_id": "engineer/backend/python-engineer", "category": "engineer/backend"},
+        ...     {"agent_id": "qa/qa", "category": "qa"}
+        ... ]
+        >>> grouped = _group_agents_by_category(agents)
+        >>> grouped.keys()
+        dict_keys(['engineer/backend', 'qa'])
+    """
+    grouped: Dict[str, List[Dict[str, Any]]] = {}
+
+    for agent in agents:
+        # Extract category from agent_id if not present
+        category = agent.get("category", "unknown")
+
+        if category == "unknown" and "agent_id" in agent:
+            agent_id = agent["agent_id"]
+            if "/" in agent_id:
+                category = agent_id.rsplit("/", 1)[0]
+
+        if category not in grouped:
+            grouped[category] = []
+
+        grouped[category].append(agent)
+
+    return grouped
+
+
+def _format_category_name(category: str) -> str:
+    """Format category name for display.
+
+    Args:
+        category: Raw category string (e.g., "engineer/backend")
+
+    Returns:
+        Formatted category name (e.g., "Engineering/Backend")
+
+    Example:
+        >>> _format_category_name("engineer/backend")
+        'Engineering/Backend'
+        >>> _format_category_name("qa")
+        'QA'
+    """
+    # Special cases
+    if category == "qa":
+        return "QA"
+    if category == "ops":
+        return "Ops"
+
+    # Capitalize each part
+    parts = category.split("/")
+    formatted_parts = []
+
+    for part in parts:
+        if part == "qa":
+            formatted_parts.append("QA")
+        elif part == "ops":
+            formatted_parts.append("Ops")
+        else:
+            formatted_parts.append(part.capitalize())
+
+    return "/".join(formatted_parts)
+
+
+def _output_table_format(agents: List[Dict[str, Any]], verbose: bool = False) -> None:
+    """Output agents in rich table format (default).
+
+    Design Decision: Group by category for better organization
+
+    Rationale: Agents grouped by category (engineer/backend, qa, ops) are
+    easier to scan than a flat list. Categories provide context about
+    agent purpose.
+
+    Args:
+        agents: List of agent dictionaries
+        verbose: Show descriptions and metadata
+    """
+    if not agents:
+        console.print("[yellow]No agents found matching filters.[/yellow]")
+        return
+
+    console.print(
+        f"\n[bold]📚 Agents from configured sources ({len(agents)} matching filters):[/bold]\n"
+    )
+
+    # Group by category
+    grouped = _group_agents_by_category(agents)
+
+    # Sort categories for consistent display
+    sorted_categories = sorted(grouped.keys())
+
+    for category in sorted_categories:
+        category_agents = grouped[category]
+
+        # Print category header
+        formatted_category = _format_category_name(category)
+        console.print(f"[bold cyan]{formatted_category}[/bold cyan]")
+
+        # Print agents in this category
+        for agent in sorted(category_agents, key=lambda a: a.get("agent_id", "")):
+            agent_id = agent.get("agent_id", "unknown")
+            source = agent.get("source", agent.get("repository", "unknown"))
+            priority = agent.get("priority", "unknown")
+
+            # Agent ID line
+            console.print(f"  • [bold]{agent_id}[/bold]")
+
+            if verbose:
+                # Description
+                description = agent.get("metadata", {}).get("description", "")
+                if not description:
+                    description = agent.get("description", "No description available")
+
+                console.print(f"    {description}")
+
+                # Version
+                version = agent.get(
+                    "version", agent.get("metadata", {}).get("version", "unknown")
+                )
+                console.print(f"    Version: {version}")
+
+            # Source attribution
+            console.print(f"    Source: [dim]{source}[/dim] (priority: {priority})")
+
+            console.print()  # Blank line between agents
+
+        console.print()  # Blank line between categories
+
+
+def _output_json_format(
+    agents: List[Dict[str, Any]], filters_applied: Dict[str, str]
+) -> None:
+    """Output agents in JSON format for scripting.
+
+    Args:
+        agents: List of agent dictionaries
+        filters_applied: Dictionary of filters that were applied
+    """
+    output = {
+        "total": len(agents),
+        "filters_applied": filters_applied,
+        "agents": [
+            {
+                "agent_id": agent.get("agent_id", "unknown"),
+                "source": agent.get("source", agent.get("repository", "unknown")),
+                "priority": agent.get("priority", "unknown"),
+                "category": agent.get("category", "unknown"),
+                "version": agent.get(
+                    "version", agent.get("metadata", {}).get("version", "unknown")
+                ),
+                "description": agent.get("metadata", {}).get(
+                    "description", agent.get("description", "")
+                ),
+            }
+            for agent in agents
+        ],
+    }
+
+    console.print(json.dumps(output, indent=2))
+
+
+def _output_simple_format(agents: List[Dict[str, Any]]) -> None:
+    """Output agents in simple format (agent IDs only).
+
+    Args:
+        agents: List of agent dictionaries
+    """
+    for agent in sorted(agents, key=lambda a: a.get("agent_id", "")):
+        agent_id = agent.get("agent_id", "unknown")
+        console.print(agent_id)
+
+
+def discover_command(args) -> int:
+    """Execute the 'agents discover' command.
+
+    This command discovers and lists available agents from all configured
+    Git sources, with optional filtering by category, language, framework,
+    platform, and source.
+
+    Design Decision: Load sources from configuration
+
+    Rationale: Use AgentSourceConfiguration to load all configured sources,
+    respecting enabled/disabled state and priority. This ensures consistent
+    behavior with other agent commands.
+
+    Error Handling:
+    - Graceful degradation if no sources configured
+    - Warning if cache not populated (suggest agent-source update)
+    - Empty result sets clearly communicated
+
+    Args:
+        args: Parsed command arguments with attributes:
+            - source: Optional source ID filter
+            - category: Optional category filter
+            - language: Optional language filter
+            - framework: Optional framework filter
+            - platform: Optional platform filter
+            - specialization: Optional specialization filter
+            - format: Output format (table, json, simple)
+            - verbose: Show descriptions and metadata
+
+    Returns:
+        Exit code (0 for success, 1 for error)
+
+    Example:
+        >>> args = argparse.Namespace(
+        ...     source=None,
+        ...     category="engineer/backend",
+        ...     language=None,
+        ...     framework=None,
+        ...     platform=None,
+        ...     specialization=None,
+        ...     format="table",
+        ...     verbose=False
+        ... )
+        >>> discover_command(args)
+        0
+    """
+    try:
+        # Load agent sources configuration
+        sources_config = AgentSourceConfiguration()
+
+        # Initialize Git source manager
+        manager = GitSourceManager()
+
+        # Build filters dictionary
+        filters = {}
+
+        if args.category:
+            filters["category"] = args.category
+
+        if args.language:
+            filters["language"] = args.language
+
+        if args.framework:
+            filters["framework"] = args.framework
+
+        if args.platform:
+            filters["platform"] = args.platform
+
+        if args.specialization:
+            filters["specialization"] = args.specialization
+
+        # Get agents with filters
+        agents = manager.list_cached_agents_with_filters(
+            repo_identifier=args.source, filters=filters if filters else None
+        )
+
+        # Check if any sources are configured
+        if not agents and not sources_config.list_sources():
+            console.print("[yellow]⚠️  No agent sources configured.[/yellow]")
+            console.print("\nAdd a source with:")
+            console.print("  claude-mpm agent-source add <git-url>")
+            return 1
+
+        # Check if cache might be empty
+        if not agents and not filters:
+            console.print("[yellow]⚠️  No agents found in cache.[/yellow]")
+            console.print("\nTry syncing sources first:")
+            console.print("  claude-mpm agent-source update")
+            return 1
+
+        # Output based on format
+        if args.format == "json":
+            filters_applied = {
+                k: v
+                for k, v in [
+                    ("source", args.source),
+                    ("category", args.category),
+                    ("language", args.language),
+                    ("framework", args.framework),
+                    ("platform", args.platform),
+                    ("specialization", args.specialization),
+                ]
+                if v is not None
+            }
+
+            _output_json_format(agents, filters_applied)
+
+        elif args.format == "simple":
+            _output_simple_format(agents)
+
+        else:  # table (default)
+            _output_table_format(agents, verbose=args.verbose)
+
+        return 0
+
+    except Exception as e:
+        logger.error(f"Error discovering agents: {e}", exc_info=True)
+        console.print(f"[bold red]Error:[/bold red] {e}")
+        return 1

claude_mpm/cli/commands/aggregate.py

@@ -365,7 +365,7 @@ def view_command_legacy(args):
             print(f"   Tools: {len(delegation.tool_operations)}")
             print(f"   Files: {len(delegation.file_changes)}")
             if delegation.duration_ms:
-                print(f"   Duration: {delegation.duration_ms/1000:.1f}s")
+                print(f"   Duration: {delegation.duration_ms / 1000:.1f}s")
             print(f"   Status: {'✅ Success' if delegation.success else '❌ Failed'}")
             if delegation.error:
                 print(f"   Error: {delegation.error}")

claude_mpm/cli/commands/analyze.py

@@ -350,7 +350,7 @@ class AnalyzeCommand(BaseCommand):
 
         for i, match in enumerate(matches):
             # Try to extract title from preceding line
-            title = f"diagram_{i+1}"
+            title = f"diagram_{i + 1}"
 
             # Look for a title pattern before the diagram
             title_pattern = r"(?:#+\s*)?([^\n]+)\n+```mermaid"
@@ -456,7 +456,7 @@ class AnalyzeCommand(BaseCommand):
             return output
 
         # text format
-        output = f"Code Analysis Report\n{'='*50}\n\n"
+        output = f"Code Analysis Report\n{'=' * 50}\n\n"
         output += f"Target: {result_data['target']}\n"
 
         if diagrams:
@@ -469,7 +469,7 @@ class AnalyzeCommand(BaseCommand):
             for filepath in result_data["saved_diagrams"]:
                 output += f"  • {filepath}\n"
 
-        output += f"\n{'-'*50}\nAnalysis Results:\n{'-'*50}\n"
+        output += f"\n{'-' * 50}\nAnalysis Results:\n{'-' * 50}\n"
         output += result_data.get("analysis", "No analysis results")
 
         return output