claude-mpm 5.6.36__py3-none-any.whl → 5.6.37__py3-none-any.whl

claude_mpm/VERSION

@@ -1 +1 @@
-5.6.36
+5.6.37

claude_mpm/cli/startup.py

@@ -13,6 +13,44 @@ import sys
 from pathlib import Path
 
 
+def cleanup_user_level_hooks() -> None:
+    """Remove stale user-level hooks directory.
+
+    WHY: claude-mpm previously deployed hooks to ~/.claude/hooks/claude-mpm/
+    (user-level). This is now deprecated in favor of project-level hooks
+    configured in .claude/settings.local.json. Stale user-level hooks can
+    cause conflicts and confusion.
+
+    DESIGN DECISION: Runs early in startup, before project hook sync.
+    Non-blocking - failures are logged at debug level but don't prevent startup.
+    """
+    import shutil
+
+    user_hooks_dir = Path.home() / ".claude" / "hooks" / "claude-mpm"
+
+    if not user_hooks_dir.exists():
+        return
+
+    try:
+        from ..core.logger import get_logger
+
+        logger = get_logger("startup")
+        logger.debug(f"Removing stale user-level hooks directory: {user_hooks_dir}")
+
+        shutil.rmtree(user_hooks_dir)
+
+        logger.debug("User-level hooks cleanup complete")
+    except Exception as e:
+        # Non-critical - log but don't fail startup
+        try:
+            from ..core.logger import get_logger
+
+            logger = get_logger("startup")
+            logger.debug(f"Failed to cleanup user-level hooks (non-fatal): {e}")
+        except Exception:  # nosec B110
+            pass  # Avoid any errors in error handling
+
+
 def sync_hooks_on_startup(quiet: bool = False) -> bool:
     """Ensure hooks are up-to-date on startup.
 
@@ -20,7 +58,12 @@ def sync_hooks_on_startup(quiet: bool = False) -> bool:
     Reinstalling hooks ensures the hook format matches the current code.
 
     DESIGN DECISION: Shows brief status message on success for user awareness.
-    Failures are logged but don't prevent startup to ensure claude-mpm remains functional.
+    Failures are logged but don't prevent startup to ensure claude-mpm
+    remains functional.
+
+    Workflow:
+    1. Cleanup stale user-level hooks (~/.claude/hooks/claude-mpm/)
+    2. Reinstall project-level hooks to .claude/settings.local.json
 
     Args:
         quiet: If True, suppress all output (used internally)
@@ -28,6 +71,9 @@ def sync_hooks_on_startup(quiet: bool = False) -> bool:
     Returns:
         bool: True if hooks were synced successfully, False otherwise
     """
+    # Step 1: Cleanup stale user-level hooks first
+    cleanup_user_level_hooks()
+
     try:
         from ..hooks.claude_hooks.installer import HookInstaller
 
@@ -1386,6 +1432,28 @@ def auto_install_chrome_devtools_on_startup():
         # Continue execution - chrome-devtools installation failure shouldn't block startup
 
 
+def sync_deployment_on_startup(force_sync: bool = False) -> None:
+    """Consolidated deployment block: hooks + agents.
+
+    WHY: Groups all deployment tasks into a single logical block for clarity.
+    This ensures hooks and agents are deployed together before other services.
+
+    Order:
+    1. Hook cleanup (remove ~/.claude/hooks/claude-mpm/)
+    2. Hook reinstall (update .claude/settings.local.json)
+    3. Agent sync from remote Git sources
+
+    Args:
+        force_sync: Force download even if cache is fresh (bypasses ETag).
+    """
+    # Step 1-2: Hooks (cleanup + reinstall handled by sync_hooks_on_startup)
+    sync_hooks_on_startup()  # Shows "Syncing Claude Code hooks... ✓"
+
+    # Step 3: Agents from remote sources
+    sync_remote_agents_on_startup(force_sync=force_sync)
+    show_agent_summary()  # Display agent counts after deployment
+
+
 def run_background_services(force_sync: bool = False):
     """
     Initialize all background services on startup.
@@ -1401,19 +1469,15 @@ def run_background_services(force_sync: bool = False):
     Args:
         force_sync: Force download even if cache is fresh (bypasses ETag).
     """
-    # Sync hooks early to ensure up-to-date configuration
-    # RATIONALE: Hooks should be synced before other services to fix stale configs
-    # This is fast (<100ms) and non-blocking, so it doesn't delay startup
-    sync_hooks_on_startup()  # Shows "Syncing Claude Code hooks... ✓"
+    # Consolidated deployment block: hooks + agents
+    # RATIONALE: Hooks and agents are deployed together before other services
+    # This ensures the deployment phase is complete before configuration checks
+    sync_deployment_on_startup(force_sync=force_sync)
 
     initialize_project_registry()
     check_mcp_auto_configuration()
     verify_mcp_gateway_startup()
     check_for_updates_async()
-    sync_remote_agents_on_startup(
-        force_sync=force_sync
-    )  # Sync agents from remote sources
-    show_agent_summary()  # Display agent counts after deployment
 
     # Skills deployment order (precedence: remote > bundled)
     # 1. Deploy bundled skills first (base layer from package)

claude_mpm/hooks/claude_hooks/event_handlers.py

@@ -17,7 +17,7 @@ import subprocess  # nosec B404 - subprocess used for safe claude CLI version ch
 import uuid
 from datetime import datetime, timezone
 from pathlib import Path
-from typing import Any, Callable, Optional
+from typing import Any, Optional
 
 # Import _log helper to avoid stderr writes (which cause hook errors)
 try:
@@ -109,25 +109,6 @@ def _get_config() -> Optional[Any]:
     return _config
 
 
-# Autotodos function (for pending todos injection)
-_get_pending_todos_fn: Optional[Callable] = None
-_get_pending_todos_loaded = False
-
-
-def _get_pending_todos_func() -> Optional[Callable]:
-    """Get get_pending_todos function with lazy loading."""
-    global _get_pending_todos_fn, _get_pending_todos_loaded
-    if not _get_pending_todos_loaded:
-        try:
-            from claude_mpm.cli.commands.autotodos import get_pending_todos
-
-            _get_pending_todos_fn = get_pending_todos
-        except ImportError:
-            _get_pending_todos_fn = None
-        _get_pending_todos_loaded = True
-    return _get_pending_todos_fn
-
-
 # Delegation detector (for anti-pattern detection)
 _delegation_detector: Optional[Any] = None
 _delegation_detector_loaded = False
@@ -186,7 +167,6 @@ class EventHandlers:
         config: Optional[Any] = None,
         delegation_detector: Optional[Any] = None,
         event_log: Optional[Any] = None,
-        get_pending_todos_fn: Optional[Callable] = None,
     ):
         """Initialize with reference to the main hook handler and optional services.
 
@@ -196,7 +176,6 @@ class EventHandlers:
             config: Optional Config for autotodos configuration
             delegation_detector: Optional DelegationDetector for anti-pattern detection
             event_log: Optional EventLog for PM violation logging
-            get_pending_todos_fn: Optional function to get pending todos
         """
         self.hook_handler = hook_handler
 
@@ -205,7 +184,6 @@ class EventHandlers:
         self._config = config
         self._delegation_detector = delegation_detector
         self._event_log = event_log
-        self._get_pending_todos_fn = get_pending_todos_fn
 
     @property
     def log_manager(self) -> Optional[Any]:
@@ -235,18 +213,6 @@ class EventHandlers:
             return self._event_log
         return _get_event_log_service()
 
-    def get_pending_todos(
-        self, max_todos: int = 10, working_dir: Optional[Path] = None
-    ) -> list:
-        """Get pending todos (using injected function or lazy loaded)."""
-        fn = self._get_pending_todos_fn or _get_pending_todos_func()
-        if fn is None:
-            return []
-        try:
-            return fn(max_todos=max_todos, working_dir=working_dir)
-        except Exception:
-            return []
-
     def handle_user_prompt_fast(self, event):
         """Handle user prompt with comprehensive data capture.
 
@@ -1119,7 +1085,9 @@ class EventHandlers:
         - Provides visibility into new conversation sessions
         - Enables tracking of session lifecycle and duration
         - Useful for monitoring concurrent sessions and resource usage
-        - Auto-inject pending autotodos if enabled in config
+
+        NOTE: This handler is intentionally lightweight - only event monitoring.
+        All initialization/deployment logic runs in MPM CLI startup, not here.
         """
         session_id = event.get("session_id", "")
         working_dir = event.get("cwd", "")
@@ -1133,35 +1101,6 @@ class EventHandlers:
             "hook_event_name": "SessionStart",
         }
 
-        # Auto-inject pending autotodos if enabled
-        # Uses injected config and get_pending_todos or lazy-loaded instances
-        config = self.config
-        if config is not None:
-            try:
-                auto_inject_enabled = config.get(
-                    "autotodos.auto_inject_on_startup", True
-                )
-                max_todos = config.get("autotodos.max_todos_per_session", 10)
-
-                if auto_inject_enabled:
-                    # Pass working directory from event to avoid Path.cwd() issues
-                    working_dir_param = None
-                    if working_dir:
-                        working_dir_param = Path(working_dir)
-
-                    pending_todos = self.get_pending_todos(
-                        max_todos=max_todos, working_dir=working_dir_param
-                    )
-                    if pending_todos:
-                        session_start_data["pending_autotodos"] = pending_todos
-                        session_start_data["autotodos_count"] = len(pending_todos)
-                        _log(
-                            f"  - Auto-injected {len(pending_todos)} pending autotodos"
-                        )
-            except Exception as e:  # nosec B110
-                # Auto-injection is optional - continue if it fails
-                _log(f"  - Failed to auto-inject autotodos: {e}")
-
         # Debug logging
         _log(f"Hook handler: Processing SessionStart - session: '{session_id}'")
 

{claude_mpm-5.6.36.dist-info → claude_mpm-5.6.37.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: claude-mpm
-Version: 5.6.36
+Version: 5.6.37
 Summary: Claude Multi-Agent Project Manager - Orchestrate Claude with agent delegation and ticket tracking
 Author-email: Bob Matsuoka <bob@matsuoka.com>
 Maintainer: Claude MPM Team

{claude_mpm-5.6.36.dist-info → claude_mpm-5.6.37.dist-info}/RECORD

@@ -1,5 +1,5 @@
 claude_mpm/BUILD_NUMBER,sha256=9JfxhnDtr-8l3kCP2U5TVXSErptHoga8m7XA8zqgGOc,4
-claude_mpm/VERSION,sha256=8w2fPmwYmLDeFLO3SEO_SZLBkk7bAj4ayqWYafYZFvA,7
+claude_mpm/VERSION,sha256=6wU_GWy-ocN_lZJMsDrvXWts8AcQ_8tGO-otGNlaDwg,7
 claude_mpm/__init__.py,sha256=AGfh00BHKvLYD-UVFw7qbKtl7NMRIzRXOWw7vEuZ-h4,2214
 claude_mpm/__main__.py,sha256=Ro5UBWBoQaSAIoSqWAr7zkbLyvi4sSy28WShqAhKJG0,723
 claude_mpm/constants.py,sha256=pz3lTrZZR5HhV3eZzYtIbtBwWo7iM6pkBHP_ixxmI6Y,6827
@@ -42,7 +42,7 @@ claude_mpm/cli/chrome_devtools_installer.py,sha256=efA_ZX1iR3oaJi3222079BQw6DEG8
 claude_mpm/cli/executor.py,sha256=eerqDVszBfTm8N2FgreLr-132Jb9BVZGNrrukvb_b-I,15106
 claude_mpm/cli/helpers.py,sha256=CypEhw0tbNH6_GzVTaQdi4w7ThCWO43Ep92YbJzPR4I,3638
 claude_mpm/cli/parser.py,sha256=Vqx9n-6Xo1uNhXR4rThmgWpZXTr0nOtkgDf3oMS9b0g,5855
-claude_mpm/cli/startup.py,sha256=4nKyk0UqVSBsiF-gnk7TnYn3pebB146c79xffo75bhA,68170
+claude_mpm/cli/startup.py,sha256=yqq44doFdcrR8f1pCPOAeLG-3kzmdWHaMEmk_uAhc5E,70361
 claude_mpm/cli/startup_display.py,sha256=KI4GZ0uVoK7FXeK5jsS4JAyaKaKf5rCkY3h5tNeEY7M,17209
 claude_mpm/cli/startup_logging.py,sha256=wHokzcCA0AJPxAqFrpY5PMOBh1iOhdhgP1gY1OvD0gY,29392
 claude_mpm/cli/utils.py,sha256=5e3v-ow1gd-5nlad9OWCsL-3SRJcPjBJ9HS3zygwkiQ,8988
@@ -429,7 +429,7 @@ claude_mpm/hooks/claude_hooks/__init__.py,sha256=b4mud_g3S-3itHY_Dzpbb_SmdMEcJwt
 claude_mpm/hooks/claude_hooks/auto_pause_handler.py,sha256=Fm8d_a0KJ1GBPPQ3xmQo-xp-j8xU2TGHTDohbySu0Cg,17184
 claude_mpm/hooks/claude_hooks/connection_pool.py,sha256=vpi-XbVf61GWhh85tHBzubbOgbJly_I-5-QmsleND2M,8658
 claude_mpm/hooks/claude_hooks/correlation_manager.py,sha256=3n-RxzqE8egG4max_NcpJgL9gzrBY6Ti529LrjleI1g,2033
-claude_mpm/hooks/claude_hooks/event_handlers.py,sha256=v1hRK_iWpYB03z5ScAyffLlSda0zA0YbW--32H4pxRY,52875
+claude_mpm/hooks/claude_hooks/event_handlers.py,sha256=VJ4kY_GMZkVv8WBAII4GTfY4qxFgoKdXpMdhEbzt980,50344
 claude_mpm/hooks/claude_hooks/hook_handler.py,sha256=AH0z6D06bo02UGyEdYlzll2EcELl-gmt8D42MPqu_SU,30459
 claude_mpm/hooks/claude_hooks/hook_wrapper.sh,sha256=XYkdYtcM0nfnwYvMdyIFCasr80ry3uI5-fLYsLtDGw4,2214
 claude_mpm/hooks/claude_hooks/installer.py,sha256=SvIgxRMocQxnqNF3ZQfKH8zA-1b4lpiCA4vCI0vWOZI,38065
@@ -439,7 +439,7 @@ claude_mpm/hooks/claude_hooks/tool_analysis.py,sha256=3_o2PP9D7wEMwLriCtIBOw0cj2
 claude_mpm/hooks/claude_hooks/__pycache__/__init__.cpython-311.pyc,sha256=EGpgXqhPM0iRRZtCqHaLVQ6wDH42OH_M7Gt5GiFLyro,346
 claude_mpm/hooks/claude_hooks/__pycache__/auto_pause_handler.cpython-311.pyc,sha256=lw3g7dHPcJ258xtcmbXOk-tCqVz0JAc5PZ10LUfG4Zo,20829
 claude_mpm/hooks/claude_hooks/__pycache__/correlation_manager.cpython-311.pyc,sha256=SQX5iiP9bQZkLL-cj_2tlGH7lpAzarO0mYal7btj3tc,3521
-claude_mpm/hooks/claude_hooks/__pycache__/event_handlers.cpython-311.pyc,sha256=NSsuyYiHY1rY6PPIceasOrzJ3f_qnvJnWimj3TbHuzk,51316
+claude_mpm/hooks/claude_hooks/__pycache__/event_handlers.cpython-311.pyc,sha256=rUcbnsARIgWenNkPv8LlHJBXmxTOExqgcwtBaBfGzg4,49040
 claude_mpm/hooks/claude_hooks/__pycache__/hook_handler.cpython-311.pyc,sha256=NQtvV9vs1OMKQTZWRQrAie83y4Q61mJ-QFawC2GnsUQ,32799
 claude_mpm/hooks/claude_hooks/__pycache__/memory_integration.cpython-311.pyc,sha256=1llucgjrun0H6q8V8_BXTHtkTiYAwNGyptluhoIi7ss,11185
 claude_mpm/hooks/claude_hooks/__pycache__/response_tracking.cpython-311.pyc,sha256=E_pRoKx-mAB5gEv2_5TneMC_K10zj7FYCPwQPnPd88g,16228
@@ -1106,10 +1106,10 @@ claude_mpm/utils/subprocess_utils.py,sha256=D0izRT8anjiUb_JG72zlJR_JAw1cDkb7kalN
 claude_mpm/validation/__init__.py,sha256=YZhwE3mhit-lslvRLuwfX82xJ_k4haZeKmh4IWaVwtk,156
 claude_mpm/validation/agent_validator.py,sha256=GprtAvu80VyMXcKGsK_VhYiXWA6BjKHv7O6HKx0AB9w,20917
 claude_mpm/validation/frontmatter_validator.py,sha256=YpJlYNNYcV8u6hIOi3_jaRsDnzhbcQpjCBE6eyBKaFY,7076
-claude_mpm-5.6.36.dist-info/licenses/LICENSE,sha256=ca3y_Rk4aPrbF6f62z8Ht5MJM9OAvbGlHvEDcj9vUQ4,3867
-claude_mpm-5.6.36.dist-info/licenses/LICENSE-FAQ.md,sha256=TxfEkXVCK98RzDOer09puc7JVCP_q_bN4dHtZKHCMcM,5104
-claude_mpm-5.6.36.dist-info/METADATA,sha256=SHCuSq8oW4Jz9yPooKmGv1QjjI4stqcyuyH-XNjBC3s,15245
-claude_mpm-5.6.36.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-claude_mpm-5.6.36.dist-info/entry_points.txt,sha256=n-Uk4vwHPpuvu-g_I7-GHORzTnN_m6iyOsoLveKKD0E,228
-claude_mpm-5.6.36.dist-info/top_level.txt,sha256=1nUg3FEaBySgm8t-s54jK5zoPnu3_eY6EP6IOlekyHA,11
-claude_mpm-5.6.36.dist-info/RECORD,,
+claude_mpm-5.6.37.dist-info/licenses/LICENSE,sha256=ca3y_Rk4aPrbF6f62z8Ht5MJM9OAvbGlHvEDcj9vUQ4,3867
+claude_mpm-5.6.37.dist-info/licenses/LICENSE-FAQ.md,sha256=TxfEkXVCK98RzDOer09puc7JVCP_q_bN4dHtZKHCMcM,5104
+claude_mpm-5.6.37.dist-info/METADATA,sha256=9S9VFtM7YgENEHeksusZFEuIArHYOdUV2KTV0ut4T9E,15245
+claude_mpm-5.6.37.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+claude_mpm-5.6.37.dist-info/entry_points.txt,sha256=n-Uk4vwHPpuvu-g_I7-GHORzTnN_m6iyOsoLveKKD0E,228
+claude_mpm-5.6.37.dist-info/top_level.txt,sha256=1nUg3FEaBySgm8t-s54jK5zoPnu3_eY6EP6IOlekyHA,11
+claude_mpm-5.6.37.dist-info/RECORD,,