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

claude_mpm/cli/executor.py

@@ -146,6 +146,62 @@ def execute_command(command: str, args) -> int:
         # Convert CommandResult to exit code
         return result.exit_code if result else 0
 
+    # Handle hook-errors command with lazy import
+    if command == "hook-errors":
+        # Lazy import to avoid loading unless needed
+        from .commands.hook_errors import (
+            clear_errors,
+            diagnose_errors,
+            list_errors,
+            show_status,
+            show_summary,
+        )
+
+        # Get subcommand
+        subcommand = getattr(args, "hook_errors_command", "status")
+        if not subcommand:
+            subcommand = "status"
+
+        # Map subcommands to functions
+        handlers = {
+            "list": list_errors,
+            "summary": show_summary,
+            "clear": clear_errors,
+            "diagnose": diagnose_errors,
+            "status": show_status,
+        }
+
+        # Get handler and invoke
+        handler = handlers.get(subcommand)
+        if handler:
+            # Build Click context programmatically
+            import click
+
+            ctx = click.Context(command=handler)
+
+            # Prepare keyword arguments from args
+            kwargs = {}
+            if hasattr(args, "format"):
+                kwargs["format"] = args.format
+            if hasattr(args, "hook_type"):
+                kwargs["hook_type"] = args.hook_type
+            if hasattr(args, "yes"):
+                kwargs["yes"] = args.yes
+
+            try:
+                # Invoke handler with arguments
+                with ctx:
+                    handler.invoke(ctx, **kwargs)
+                return 0
+            except SystemExit as e:
+                return e.code if e.code is not None else 0
+            except Exception as e:
+                print(f"Error: {e}")
+                return 1
+        else:
+            print(f"Unknown hook-errors subcommand: {subcommand}")
+            return 1
+
     # Map stable commands to their implementations
     command_map = {
         CLICommands.RUN.value: run_session,

claude_mpm/cli/interactive/agent_wizard.py

@@ -55,7 +55,7 @@ class AgentWizard:
             model = self._get_model_choice()
 
             # Step 5: Inheritance Option
-            parent_agent, base_template = self._get_inheritance_option()
+            parent_agent, _base_template = self._get_inheritance_option()
 
             # Step 6: Capabilities Configuration
             capabilities = self._get_capabilities_configuration()
@@ -178,25 +178,25 @@ class AgentWizard:
                     if not result[0]:
                         print(f"❌ {result[1]}")
                 elif choice_num == len(templates) + 1:
-                    success, message = self.run_interactive_create()
+                    _, message = self.run_interactive_create()
                     if message:
                         # Message already has emoji from the function
                         print(f"\n{message}")
                     continue  # Return to main menu
                 elif choice_num == len(templates) + 2:
-                    success, message = self._interactive_delete_menu(templates)
+                    _, message = self._interactive_delete_menu(templates)
                     if message:
                         # Message already has emoji from the function
                         print(f"\n{message}")
                     continue  # Return to main menu
                 elif choice_num == len(templates) + 3:
-                    success, message = self._interactive_import()
+                    _, message = self._interactive_import()
                     if message:
                         # Message already has emoji from the function
                         print(f"\n{message}")
                     continue  # Return to main menu
                 elif choice_num == len(templates) + 4:
-                    success, message = self._interactive_export()
+                    _success, message = self._interactive_export()
                     if message:
                         # Message already has emoji from the function
                         print(f"\n{message}")

claude_mpm/cli/parsers/base_parser.py

@@ -541,6 +541,34 @@ def create_parser(
         from ..commands.verify import add_parser as add_verify_parser
 
         add_verify_parser(subparsers)
+
+        # Add hook-errors command for managing hook error memory
+        hook_errors_parser = subparsers.add_parser(
+            "hook-errors",
+            help="Manage hook error memory and diagnostics",
+        )
+        hook_errors_parser.add_argument(
+            "hook_errors_command",
+            nargs="?",
+            choices=["list", "summary", "clear", "diagnose", "status"],
+            help="Hook errors subcommand",
+        )
+        hook_errors_parser.add_argument(
+            "--format",
+            choices=["table", "json"],
+            default="table",
+            help="Output format for list command",
+        )
+        hook_errors_parser.add_argument(
+            "--hook-type",
+            help="Filter by specific hook type",
+        )
+        hook_errors_parser.add_argument(
+            "-y",
+            "--yes",
+            action="store_true",
+            help="Skip confirmation prompts",
+        )
     except ImportError:
         # Commands module may not be available during testing or refactoring
         pass

claude_mpm/cli/parsers/mpm_init_parser.py

@@ -265,5 +265,47 @@ def add_mpm_init_subparser(subparsers: Any) -> None:
         help="Path to project directory (default: current directory)",
     )
 
+    # Pause subcommand - Create session pause documents
+    pause_parser = subcommands.add_parser(
+        "pause",
+        help="Pause current session and save state",
+        description="Create session pause documents for later resume. Captures git context, "
+        "conversation state, todos, and working directory state.\n\n"
+        "Creates three file formats:\n"
+        "  - JSON: Machine-readable structured data\n"
+        "  - YAML: Human-readable configuration style\n"
+        "  - Markdown: Full documentation format",
+        formatter_class=argparse.RawDescriptionHelpFormatter,
+        epilog="Examples:\n"
+        "  claude-mpm mpm-init pause                          # Basic pause\n"
+        "  claude-mpm mpm-init pause -m 'End of day'         # With message\n"
+        "  claude-mpm mpm-init pause --no-commit             # Skip git commit\n"
+        "  claude-mpm mpm-init pause --export session.json   # Export copy",
+    )
+    pause_parser.add_argument(
+        "--message",
+        "-m",
+        type=str,
+        default=None,
+        help="Optional message describing pause reason or context",
+    )
+    pause_parser.add_argument(
+        "--no-commit",
+        action="store_true",
+        help="Skip git commit of session state",
+    )
+    pause_parser.add_argument(
+        "--export",
+        type=str,
+        default=None,
+        help="Export session state to custom location",
+    )
+    pause_parser.add_argument(
+        "project_path",
+        nargs="?",
+        default=".",
+        help="Path to project directory (default: current directory)",
+    )
+
     # Set the command handler
     mpm_init_parser.set_defaults(command="mpm-init")

claude_mpm/cli/parsers/skills_parser.py

@@ -134,4 +134,142 @@ def add_skills_subparser(subparsers) -> argparse.ArgumentParser:
         help="Show configuration file path",
     )
 
+    # GitHub deployment commands
+    # Deploy from GitHub command
+    deploy_github_parser = skills_subparsers.add_parser(
+        SkillsCommands.DEPLOY_FROM_GITHUB.value,
+        help="Deploy skills from GitHub to ~/.claude/skills/ for Claude Code",
+    )
+    deploy_github_parser.add_argument(
+        "--collection",
+        "-c",
+        help="Collection to deploy from (default: uses default collection)",
+    )
+    deploy_github_parser.add_argument(
+        "--toolchain",
+        nargs="+",
+        help="Filter by toolchain/language (e.g., python javascript rust)",
+    )
+    deploy_github_parser.add_argument(
+        "--categories",
+        nargs="+",
+        help="Filter by categories (e.g., testing debugging web)",
+    )
+    deploy_github_parser.add_argument(
+        "--force",
+        action="store_true",
+        help="Force redeployment of already deployed skills",
+    )
+    deploy_github_parser.add_argument(
+        "--all",
+        action="store_true",
+        help="Deploy all available skills (no filtering)",
+    )
+
+    # List available GitHub skills
+    list_available_parser = skills_subparsers.add_parser(
+        SkillsCommands.LIST_AVAILABLE.value,
+        help="List all available skills from GitHub repository",
+    )
+    list_available_parser.add_argument(
+        "--collection",
+        "-c",
+        help="Collection to list from (default: uses default collection)",
+    )
+    list_available_parser.add_argument(
+        "--verbose",
+        "-v",
+        action="store_true",
+        help="Show detailed skill information including names",
+    )
+
+    # Check deployed skills
+    skills_subparsers.add_parser(
+        SkillsCommands.CHECK_DEPLOYED.value,
+        help="Check which skills are currently deployed in ~/.claude/skills/",
+    )
+
+    # Remove skills
+    remove_parser = skills_subparsers.add_parser(
+        SkillsCommands.REMOVE.value,
+        help="Remove deployed skills from ~/.claude/skills/",
+    )
+    remove_parser.add_argument(
+        "skill_names",
+        nargs="*",
+        help="Specific skills to remove",
+    )
+    remove_parser.add_argument(
+        "--all",
+        action="store_true",
+        help="Remove all deployed skills",
+    )
+
+    # Collection management commands
+    # List collections
+    skills_subparsers.add_parser(
+        SkillsCommands.COLLECTION_LIST.value,
+        help="List all configured skill collections",
+    )
+
+    # Add collection
+    collection_add_parser = skills_subparsers.add_parser(
+        SkillsCommands.COLLECTION_ADD.value,
+        help="Add a new skill collection from GitHub",
+    )
+    collection_add_parser.add_argument(
+        "collection_name",
+        help="Name for the collection (e.g., obra-superpowers)",
+    )
+    collection_add_parser.add_argument(
+        "collection_url",
+        help="GitHub repository URL (e.g., https://github.com/obra/superpowers)",
+    )
+    collection_add_parser.add_argument(
+        "--priority",
+        type=int,
+        default=99,
+        help="Collection priority (lower = higher priority, default: 99)",
+    )
+
+    # Remove collection
+    collection_remove_parser = skills_subparsers.add_parser(
+        SkillsCommands.COLLECTION_REMOVE.value,
+        help="Remove a skill collection",
+    )
+    collection_remove_parser.add_argument(
+        "collection_name",
+        help="Name of the collection to remove",
+    )
+
+    # Enable collection
+    collection_enable_parser = skills_subparsers.add_parser(
+        SkillsCommands.COLLECTION_ENABLE.value,
+        help="Enable a disabled collection",
+    )
+    collection_enable_parser.add_argument(
+        "collection_name",
+        help="Name of the collection to enable",
+    )
+
+    # Disable collection
+    collection_disable_parser = skills_subparsers.add_parser(
+        SkillsCommands.COLLECTION_DISABLE.value,
+        help="Disable a collection without removing it",
+    )
+    collection_disable_parser.add_argument(
+        "collection_name",
+        help="Name of the collection to disable",
+    )
+
+    # Set default collection
+    collection_set_default_parser = skills_subparsers.add_parser(
+        SkillsCommands.COLLECTION_SET_DEFAULT.value,
+        help="Set the default collection for deployments",
+    )
+    collection_set_default_parser.add_argument(
+        "collection_name",
+        help="Name of the collection to set as default",
+    )
+
     return skills_parser

claude_mpm/cli/startup.py

@@ -19,23 +19,34 @@ def setup_early_environment(argv):
     WHY: Some commands need special environment handling before any logging
     or service initialization occurs.
 
+    CRITICAL: Suppress ALL logging by default until setup_mcp_server_logging()
+    configures the user's preference. This prevents early loggers (like
+    ProjectInitializer and service.* loggers) from logging at INFO level before
+    we know the user's logging preference.
+
     Args:
         argv: Command line arguments
 
     Returns:
         Processed argv list
     """
+    import logging
+
     # Disable telemetry and set cleanup flags early
     os.environ.setdefault("DISABLE_TELEMETRY", "1")
     os.environ.setdefault("CLAUDE_MPM_SKIP_CLEANUP", "0")
 
-    # EARLY CHECK: Suppress logging for configure command
+    # CRITICAL: Suppress ALL logging by default
+    # This catches all loggers (claude_mpm.*, service.*, framework_loader, etc.)
+    # This will be overridden by setup_mcp_server_logging() based on user preference
+    logging.getLogger().setLevel(logging.CRITICAL + 1)  # Root logger catches everything
+
+    # Process argv
     if argv is None:
         argv = sys.argv[1:]
-    if "configure" in argv or (len(argv) > 0 and argv[0] == "configure"):
-        import logging
 
-        logging.getLogger("claude_mpm").setLevel(logging.WARNING)
+    # EARLY CHECK: Additional suppression for configure command
+    if "configure" in argv or (len(argv) > 0 and argv[0] == "configure"):
         os.environ["CLAUDE_MPM_SKIP_CLEANUP"] = "1"
 
     return argv
@@ -160,6 +171,73 @@ def discover_and_link_runtime_skills():
         # Continue execution - skills discovery failure shouldn't block startup
 
 
+def deploy_output_style_on_startup():
+    """
+    Deploy claude-mpm output style to Claude Code on CLI startup.
+
+    WHY: Automatically deploy and activate the output style to ensure consistent,
+    professional communication without emojis and exclamation points. This ensures
+    the style is available even when using Claude Code directly (not via chat command).
+
+    DESIGN DECISION: This is non-blocking and idempotent. It uses OutputStyleManager
+    which handles version detection, file deployment, and settings activation.
+    Only works for Claude Code >= 1.0.83.
+    """
+    try:
+        from pathlib import Path
+
+        from ..core.output_style_manager import OutputStyleManager
+
+        # Create OutputStyleManager instance
+        output_style_manager = OutputStyleManager()
+
+        # Check if Claude Code supports output styles
+        if not output_style_manager.supports_output_styles():
+            # Silently skip - version too old or Claude not installed
+            return
+
+        # Check if already deployed and active
+        settings_file = Path.home() / ".claude" / "settings.json"
+        output_style_file = Path.home() / ".claude" / "output-styles" / "claude-mpm.md"
+
+        if settings_file.exists() and output_style_file.exists():
+            try:
+                import json
+
+                # Check if file has content (bug fix: was skipping empty files)
+                if output_style_file.stat().st_size == 0:
+                    # File is empty, need to redeploy with content
+                    pass  # Fall through to deployment below
+                else:
+                    # File has content, check if already active
+                    settings = json.loads(settings_file.read_text())
+                    if settings.get("activeOutputStyle") == "claude-mpm":
+                        # Already deployed and active with content
+                        return
+            except Exception:
+                pass  # Continue with deployment if we can't read settings
+
+        # Read OUTPUT_STYLE.md content
+        output_style_path = Path(__file__).parent.parent / "agents" / "OUTPUT_STYLE.md"
+
+        if not output_style_path.exists():
+            # No output style file to deploy
+            return
+
+        output_style_content = output_style_path.read_text()
+
+        # Deploy the output style (deploys file and activates it)
+        output_style_manager.deploy_output_style(output_style_content)
+
+    except Exception as e:
+        # Non-critical - log but don't fail startup
+        from ..core.logger import get_logger
+
+        logger = get_logger("cli")
+        logger.debug(f"Failed to deploy output style: {e}")
+        # Continue execution - output style deployment shouldn't block startup
+
+
 def run_background_services():
     """
     Initialize all background services on startup.
@@ -172,6 +250,7 @@ def run_background_services():
     check_for_updates_async()
     deploy_bundled_skills()
     discover_and_link_runtime_skills()
+    deploy_output_style_on_startup()
 
 
 def setup_mcp_server_logging(args):
@@ -285,12 +364,12 @@ def check_mcp_auto_configuration():
         mcp_manager = MCPConfigManager()
 
         # Fix any corrupted installations first
-        fix_success, fix_message = mcp_manager.fix_mcp_service_issues()
+        _fix_success, fix_message = mcp_manager.fix_mcp_service_issues()
         if fix_message and "Fixed:" in fix_message:
             logger.info(f"MCP service fixes applied: {fix_message}")
 
         # Ensure all services are configured correctly
-        config_success, config_message = mcp_manager.ensure_mcp_services_configured()
+        _config_success, config_message = mcp_manager.ensure_mcp_services_configured()
         if config_message and "Added MCP services" in config_message:
             logger.info(f"MCP services configured: {config_message}")
 
@@ -471,7 +550,7 @@ def check_for_updates_async():
 
     DESIGN DECISION: This is non-blocking and non-critical - failures are logged
     but don't prevent startup. Only runs for pip/pipx/npm installations, skips
-    editable/development installations.
+    editable/development installations. Respects user configuration settings.
     """
 
     def run_update_check():
@@ -480,11 +559,27 @@ def check_for_updates_async():
         try:
             import asyncio
 
+            from ..core.config import Config
             from ..core.logger import get_logger
             from ..services.self_upgrade_service import SelfUpgradeService
 
             logger = get_logger("upgrade_check")
 
+            # Load configuration
+            config = Config()
+            updates_config = config.get("updates", {})
+
+            # Check if update checking is enabled
+            if not updates_config.get("check_enabled", True):
+                logger.debug("Update checking disabled in configuration")
+                return
+
+            # Check frequency setting
+            frequency = updates_config.get("check_frequency", "daily")
+            if frequency == "never":
+                logger.debug("Update checking frequency set to 'never'")
+                return
+
             # Create new event loop for this thread
             loop = asyncio.new_event_loop()
             asyncio.set_event_loop(loop)
@@ -499,8 +594,16 @@ def check_for_updates_async():
                 logger.debug("Skipping version check for editable installation")
                 return
 
+            # Get configuration values
+            check_claude_code = updates_config.get("check_claude_code", True)
+            auto_upgrade = updates_config.get("auto_upgrade", False)
+
             # Check and prompt for upgrade if available (non-blocking)
-            loop.run_until_complete(upgrade_service.check_and_prompt_on_startup())
+            loop.run_until_complete(
+                upgrade_service.check_and_prompt_on_startup(
+                    auto_upgrade=auto_upgrade, check_claude_code=check_claude_code
+                )
+            )
 
         except Exception as e:
             # Non-critical - log but don't fail startup