claude-mpm 5.6.35__tar.gz → 5.6.37__tar.gz

{claude_mpm-5.6.35/src/claude_mpm.egg-info → claude_mpm-5.6.37}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: claude-mpm
-Version: 5.6.35
+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.35 → claude_mpm-5.6.37}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "claude-mpm"
-version = "5.6.35"
+version = "5.6.37"
 description = "Claude Multi-Agent Project Manager - Orchestrate Claude with agent delegation and ticket tracking"
 readme = "README.md"
 requires-python = ">=3.11"
@@ -189,7 +189,7 @@ ignore-imports = false
 
 [tool.commitizen]
 name = "cz_conventional_commits"
-version = "5.6.35"
+version = "5.6.37"
 version_files = [ "VERSION", "src/claude_mpm/VERSION", "package.json:version",]
 tag_format = "v$version"
 update_changelog_on_bump = true

claude_mpm-5.6.37/src/claude_mpm/VERSION

@@ -0,0 +1 @@
+5.6.37

{claude_mpm-5.6.35 → claude_mpm-5.6.37}/src/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-5.6.35 → claude_mpm-5.6.37}/src/claude_mpm/core/hook_manager.py

@@ -186,8 +186,9 @@ class HookManager:
             env["CLAUDE_MPM_HOOK_DEBUG"] = "true"
 
             # Execute with timeout in background thread
+            # Run as module to ensure proper package context for relative imports
             result = subprocess.run(  # nosec B603 B607
-                ["python", str(self.hook_handler_path)],
+                ["python", "-m", "claude_mpm.hooks.claude_hooks.hook_handler"],
                 input=event_json,
                 text=True,
                 capture_output=True,

{claude_mpm-5.6.35 → claude_mpm-5.6.37}/src/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.35 → claude_mpm-5.6.37/src/claude_mpm.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: claude-mpm
-Version: 5.6.35
+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.35 → claude_mpm-5.6.37}/src/claude_mpm.egg-info/SOURCES.txt

@@ -447,22 +447,13 @@ src/claude_mpm/hooks/claude_hooks/memory_integration.py
 src/claude_mpm/hooks/claude_hooks/response_tracking.py
 src/claude_mpm/hooks/claude_hooks/tool_analysis.py
 src/claude_mpm/hooks/claude_hooks/__pycache__/__init__.cpython-311.pyc
-src/claude_mpm/hooks/claude_hooks/__pycache__/__init__.cpython-314.pyc
 src/claude_mpm/hooks/claude_hooks/__pycache__/auto_pause_handler.cpython-311.pyc
-src/claude_mpm/hooks/claude_hooks/__pycache__/auto_pause_handler.cpython-314.pyc
 src/claude_mpm/hooks/claude_hooks/__pycache__/correlation_manager.cpython-311.pyc
-src/claude_mpm/hooks/claude_hooks/__pycache__/correlation_manager.cpython-314.pyc
 src/claude_mpm/hooks/claude_hooks/__pycache__/event_handlers.cpython-311.pyc
-src/claude_mpm/hooks/claude_hooks/__pycache__/event_handlers.cpython-314.pyc
 src/claude_mpm/hooks/claude_hooks/__pycache__/hook_handler.cpython-311.pyc
-src/claude_mpm/hooks/claude_hooks/__pycache__/hook_handler.cpython-314.pyc
-src/claude_mpm/hooks/claude_hooks/__pycache__/installer.cpython-314.pyc
 src/claude_mpm/hooks/claude_hooks/__pycache__/memory_integration.cpython-311.pyc
-src/claude_mpm/hooks/claude_hooks/__pycache__/memory_integration.cpython-314.pyc
 src/claude_mpm/hooks/claude_hooks/__pycache__/response_tracking.cpython-311.pyc
-src/claude_mpm/hooks/claude_hooks/__pycache__/response_tracking.cpython-314.pyc
 src/claude_mpm/hooks/claude_hooks/__pycache__/tool_analysis.cpython-311.pyc
-src/claude_mpm/hooks/claude_hooks/__pycache__/tool_analysis.cpython-314.pyc
 src/claude_mpm/hooks/claude_hooks/services/__init__.py
 src/claude_mpm/hooks/claude_hooks/services/connection_manager.py
 src/claude_mpm/hooks/claude_hooks/services/connection_manager_http.py
@@ -472,19 +463,12 @@ src/claude_mpm/hooks/claude_hooks/services/protocols.py
 src/claude_mpm/hooks/claude_hooks/services/state_manager.py
 src/claude_mpm/hooks/claude_hooks/services/subagent_processor.py
 src/claude_mpm/hooks/claude_hooks/services/__pycache__/__init__.cpython-311.pyc
-src/claude_mpm/hooks/claude_hooks/services/__pycache__/__init__.cpython-314.pyc
 src/claude_mpm/hooks/claude_hooks/services/__pycache__/connection_manager_http.cpython-311.pyc
-src/claude_mpm/hooks/claude_hooks/services/__pycache__/connection_manager_http.cpython-314.pyc
 src/claude_mpm/hooks/claude_hooks/services/__pycache__/container.cpython-311.pyc
-src/claude_mpm/hooks/claude_hooks/services/__pycache__/container.cpython-314.pyc
 src/claude_mpm/hooks/claude_hooks/services/__pycache__/duplicate_detector.cpython-311.pyc
-src/claude_mpm/hooks/claude_hooks/services/__pycache__/duplicate_detector.cpython-314.pyc
 src/claude_mpm/hooks/claude_hooks/services/__pycache__/protocols.cpython-311.pyc
-src/claude_mpm/hooks/claude_hooks/services/__pycache__/protocols.cpython-314.pyc
 src/claude_mpm/hooks/claude_hooks/services/__pycache__/state_manager.cpython-311.pyc
-src/claude_mpm/hooks/claude_hooks/services/__pycache__/state_manager.cpython-314.pyc
 src/claude_mpm/hooks/claude_hooks/services/__pycache__/subagent_processor.cpython-311.pyc
-src/claude_mpm/hooks/claude_hooks/services/__pycache__/subagent_processor.cpython-314.pyc
 src/claude_mpm/hooks/failure_learning/__init__.py
 src/claude_mpm/hooks/failure_learning/failure_detection_hook.py
 src/claude_mpm/hooks/failure_learning/fix_detection_hook.py

claude_mpm-5.6.35/src/claude_mpm/VERSION

@@ -1 +0,0 @@
-5.6.35