claude-mpm 5.4.55__py3-none-any.whl → 5.4.85__py3-none-any.whl

claude_mpm/agents/PM_INSTRUCTIONS.md

@@ -7,6 +7,17 @@
 
 The Project Manager (PM) agent coordinates work across specialized agents in the Claude MPM framework. The PM's responsibility is orchestration and quality assurance, not direct execution.
 
+## 🔴 ABSOLUTE PROHIBITIONS 🔴
+
+**PM must NEVER:**
+1. Read source code files (`.py`, `.js`, `.ts`, `.tsx`, etc.) - DELEGATE to Research
+2. Use Read tool more than ONCE per session - DELEGATE to Research
+3. Investigate, debug, or analyze code directly - DELEGATE to Research
+4. Use Edit/Write tools on any file - DELEGATE to Engineer
+5. Run verification commands (curl, lsof) - DELEGATE to local-ops
+
+**Violation of any prohibition = Circuit Breaker triggered**
+
 ### Why Delegation Matters
 
 The PM delegates all work to specialized agents for three key reasons:
@@ -315,22 +326,31 @@ TodoWrite:
 
 ### Read Tool Usage (Strict Hierarchy)
 
-**DEFAULT**: Zero reads - delegate to Research instead.
+**ABSOLUTE PROHIBITION**: PM must NEVER read source code files directly.
+
+**Source code extensions** (ALWAYS delegate to Research):
+`.py`, `.js`, `.ts`, `.tsx`, `.jsx`, `.go`, `.rs`, `.java`, `.rb`, `.php`, `.swift`, `.kt`, `.c`, `.cpp`, `.h`
 
 **SINGLE EXCEPTION**: ONE config/settings file for delegation context only.
+- Allowed: `package.json`, `pyproject.toml`, `settings.json`, `.env.example`
+- NOT allowed: Any file with source code extensions above
+
+**Pre-Flight Check (MANDATORY before ANY Read call)**:
+1. Is this a source code file? → STOP, delegate to Research
+2. Have I already used Read once this session? → STOP, delegate to Research
+3. Does my task contain investigation keywords? → STOP, delegate to Research
+
+**Investigation Keywords** (trigger delegation, not Read):
+- check, look, see, find, search, analyze, investigate, debug
+- understand, explore, examine, review, inspect, trace
+- "what does", "how does", "why does", "where is"
 
 **Rules**:
 - ✅ Allowed: ONE file (`package.json`, `pyproject.toml`, `settings.json`, `.env.example`)
-- ❌ Forbidden: Source code (`.py`, `.js`, `.ts`, `.tsx`, `.go`, `.rs`)
-- ❌ Forbidden: Multiple files OR investigation keywords ("check", "analyze", "debug", "investigate")
+- ❌ NEVER: Source code (`.py`, `.js`, `.ts`, `.tsx`, `.go`, `.rs`)
+- ❌ NEVER: Multiple files OR investigation keywords ("check", "analyze", "debug", "investigate")
 - **Rationale**: Reading leads to investigating. PM must delegate, not do.
 
-**Before Using Read, Check**:
-1. Investigation keywords present? → Delegate to Research (zero reads)
-2. Source code file? → Delegate to Research
-3. Already used Read once? → Violation - delegate to Research
-4. Purpose is delegation context (not understanding)? → ONE Read allowed
-
 ## Agent Deployment Architecture
 
 ### Cache Structure
@@ -1306,6 +1326,13 @@ PM: *Delegates to Research*         # ✅ CORRECT: If vector search insufficient
 **Action**: BLOCK - Must delegate to Research instead
 **Enforcement**: Violation #1 = Warning, #2 = Session flagged, #3 = Non-compliant
 
+**Proactive Self-Check (PM must ask before EVERY Read call)**:
+1. "Is this file a source code file?" → If yes, DELEGATE
+2. "Have I already used Read this session?" → If yes, DELEGATE
+3. "Am I investigating/debugging?" → If yes, DELEGATE
+
+If ANY answer is YES → Do NOT use Read, delegate to Research instead.
+
 **Allowed Exception:**
 - ONE config file read (package.json, pyproject.toml, settings.json, .env.example)
 - Purpose: Delegation context ONLY (not investigation)

claude_mpm/cli/__init__.py

@@ -91,7 +91,11 @@ def main(argv: Optional[list] = None):
         )
 
         try:
-            run_background_services()
+            # Check for --force-sync flag or environment variable
+            force_sync = getattr(args, "force_sync", False) or os.environ.get(
+                "CLAUDE_MPM_FORCE_SYNC", "0"
+            ) in ("1", "true", "True", "yes")
+            run_background_services(force_sync=force_sync)
             launch_progress.finish(message="Ready")
 
             # Inform user about Claude Code initialization delay (3-5 seconds)

claude_mpm/cli/commands/agents.py

@@ -1379,7 +1379,7 @@ class AgentsCommand(AgentCommand):
                 return CommandResult.error_result("agent_id is required")
 
             import os
-            import subprocess
+            import subprocess  # nosec B404
 
             from ...services.agents.local_template_manager import (
                 LocalAgentTemplateManager,
@@ -1415,7 +1415,7 @@ class AgentsCommand(AgentCommand):
 
             # Use system editor
             editor = getattr(args, "editor", None) or os.environ.get("EDITOR", "nano")
-            subprocess.run([editor, str(template_file)], check=True)
+            subprocess.run([editor, str(template_file)], check=True)  # nosec B603
             return CommandResult.success_result(
                 f"Agent '{agent_id}' edited successfully"
             )
@@ -1519,8 +1519,6 @@ class AgentsCommand(AgentCommand):
             console.print("For a better experience with integrated configuration:")
             console.print("  • Agent management")
             console.print("  • Skills management")
-            console.print("  • Template editing")
-            console.print("  • Behavior configuration")
             console.print("  • Startup settings\n")
 
             console.print("Please use: [bold green]claude-mpm config[/bold green]\n")

claude_mpm/cli/commands/agents_reconcile.py

@@ -0,0 +1,197 @@
+"""
+Agent/Skill Reconciliation CLI Command
+
+Shows the reconciliation view between configured and deployed agents/skills,
+and performs reconciliation (deploy missing, remove unneeded).
+
+Usage:
+    claude-mpm agents reconcile [--dry-run] [--show-only]
+    claude-mpm skills reconcile [--dry-run] [--show-only]
+"""
+
+from pathlib import Path
+
+from rich.console import Console
+from rich.table import Table
+
+from ...core.unified_config import UnifiedConfig
+from ...services.agents.deployment.deployment_reconciler import (
+    DeploymentReconciler,
+    ReconciliationState,
+)
+from ..shared import BaseCommand, CommandResult
+
+
+class AgentsReconcileCommand(BaseCommand):
+    """CLI command for agent reconciliation."""
+
+    def __init__(self):
+        super().__init__("agents-reconcile")
+        self.console = Console()
+
+    def run(self, args) -> CommandResult:
+        """Execute reconciliation."""
+        # Load config
+        config = UnifiedConfig()
+        reconciler = DeploymentReconciler(config)
+
+        # Get project path
+        project_path = Path(getattr(args, "project_path", "."))
+
+        # Show current state
+        if getattr(args, "show_only", False) or getattr(args, "dry_run", False):
+            return self._show_reconciliation_view(reconciler, project_path)
+
+        # Perform reconciliation
+        return self._reconcile_agents(reconciler, project_path)
+
+    def _show_reconciliation_view(
+        self, reconciler: DeploymentReconciler, project_path: Path
+    ) -> CommandResult:
+        """Show reconciliation view without making changes."""
+        view = reconciler.get_reconciliation_view(project_path)
+        agent_state = view["agents"]
+        skill_state = view["skills"]
+
+        # Display agents table
+        self.console.print(
+            "\n[bold blue]═══ Agent Reconciliation View ═══[/bold blue]\n"
+        )
+        agent_table = self._build_reconciliation_table(agent_state, "Agent")
+        self.console.print(agent_table)
+
+        # Display skills table
+        self.console.print(
+            "\n[bold blue]═══ Skill Reconciliation View ═══[/bold blue]\n"
+        )
+        skill_table = self._build_reconciliation_table(skill_state, "Skill")
+        self.console.print(skill_table)
+
+        # Show summary
+        self._show_summary(agent_state, skill_state)
+
+        return CommandResult.success_result("Reconciliation view displayed")
+
+    def _build_reconciliation_table(
+        self, state: ReconciliationState, item_type: str
+    ) -> Table:
+        """Build Rich table for reconciliation state."""
+        table = Table(title=f"{item_type} Deployment Status")
+
+        table.add_column(f"{item_type}", style="cyan", no_wrap=True)
+        table.add_column("Configured", style="green")
+        table.add_column("Deployed", style="yellow")
+        table.add_column("Action", style="magenta")
+
+        # All items to consider
+        all_items = state.configured | state.deployed | state.cached
+
+        for item_id in sorted(all_items):
+            configured = "✓" if item_id in state.configured else "✗"
+            deployed = "✓" if item_id in state.deployed else "✗"
+
+            # Determine action
+            if item_id in state.to_deploy:
+                if item_id in state.cached:
+                    action = "Will deploy"
+                else:
+                    action = "[red]Missing in cache![/red]"
+            elif item_id in state.to_remove:
+                action = "Will remove"
+            elif item_id in state.unchanged:
+                action = "-"
+            elif item_id in state.cached and item_id not in state.configured:
+                action = "[dim]Available (not configured)[/dim]"
+            else:
+                action = "-"
+
+            table.add_row(item_id, configured, deployed, action)
+
+        return table
+
+    def _show_summary(
+        self, agent_state: ReconciliationState, skill_state: ReconciliationState
+    ) -> None:
+        """Show reconciliation summary."""
+        self.console.print("\n[bold]Summary:[/bold]")
+
+        # Agents
+        self.console.print("\nAgents:")
+        self.console.print(f"  Configured: {len(agent_state.configured)}")
+        self.console.print(f"  Deployed: {len(agent_state.deployed)}")
+        self.console.print(f"  To deploy: {len(agent_state.to_deploy)}")
+        self.console.print(f"  To remove: {len(agent_state.to_remove)}")
+        self.console.print(f"  Unchanged: {len(agent_state.unchanged)}")
+
+        # Skills
+        self.console.print("\nSkills:")
+        self.console.print(f"  Configured: {len(skill_state.configured)}")
+        self.console.print(f"  Deployed: {len(skill_state.deployed)}")
+        self.console.print(f"  To deploy: {len(skill_state.to_deploy)}")
+        self.console.print(f"  To remove: {len(skill_state.to_remove)}")
+        self.console.print(f"  Unchanged: {len(skill_state.unchanged)}")
+
+        # Show next steps
+        if agent_state.to_deploy or skill_state.to_deploy:
+            self.console.print(
+                "\n[yellow]Run without --show-only to perform deployment[/yellow]"
+            )
+
+    def _reconcile_agents(
+        self, reconciler: DeploymentReconciler, project_path: Path
+    ) -> CommandResult:
+        """Perform agent and skill reconciliation."""
+        # Show current state first
+        self._show_reconciliation_view(reconciler, project_path)
+
+        self.console.print("\n[bold blue]Performing reconciliation...[/bold blue]\n")
+
+        # Reconcile agents
+        self.console.print("[cyan]Reconciling agents...[/cyan]")
+        agent_result = reconciler.reconcile_agents(project_path)
+
+        if agent_result.deployed:
+            self.console.print(
+                f"  [green]✓ Deployed: {', '.join(agent_result.deployed)}[/green]"
+            )
+        if agent_result.removed:
+            self.console.print(
+                f"  [yellow]✓ Removed: {', '.join(agent_result.removed)}[/yellow]"
+            )
+        if agent_result.errors:
+            for error in agent_result.errors:
+                self.console.print(f"  [red]✗ {error}[/red]")
+
+        # Reconcile skills
+        self.console.print("\n[cyan]Reconciling skills...[/cyan]")
+        skill_result = reconciler.reconcile_skills(project_path)
+
+        if skill_result.deployed:
+            self.console.print(
+                f"  [green]✓ Deployed: {', '.join(skill_result.deployed)}[/green]"
+            )
+        if skill_result.removed:
+            self.console.print(
+                f"  [yellow]✓ Removed: {', '.join(skill_result.removed)}[/yellow]"
+            )
+        if skill_result.errors:
+            for error in skill_result.errors:
+                self.console.print(f"  [red]✗ {error}[/red]")
+
+        # Final summary
+        total_errors = len(agent_result.errors) + len(skill_result.errors)
+        if total_errors == 0:
+            self.console.print("\n[bold green]✓ Reconciliation complete![/bold green]")
+            return CommandResult.success_result("Reconciliation successful")
+        self.console.print(
+            f"\n[bold yellow]⚠ Reconciliation complete with {total_errors} errors[/bold yellow]"
+        )
+        return CommandResult.error_result(f"Reconciliation had {total_errors} errors")
+
+
+class SkillsReconcileCommand(AgentsReconcileCommand):
+    """CLI command for skill reconciliation (alias to agents reconcile)."""
+
+    def __init__(self):
+        BaseCommand.__init__(self, "skills-reconcile")
+        self.console = Console()