crackerjack 0.31.10__py3-none-any.whl → 0.31.12__py3-none-any.whl

crackerjack/cli/handlers.py

@@ -9,6 +9,10 @@ from .options import Options
 
 
 def setup_ai_agent_env(ai_agent: bool, verbose: bool = False) -> None:
+    # Set debug environment variable if verbose is enabled
+    if verbose:
+        os.environ["CRACKERJACK_DEBUG"] = "1"
+
     if ai_agent:
         os.environ["AI_AGENT"] = "1"
         os.environ["AI_AGENT_DEBUG"] = "1"
@@ -19,7 +23,7 @@ def setup_ai_agent_env(ai_agent: bool, verbose: bool = False) -> None:
         if verbose:
             console = Console()
             console.print(
-                "[bold cyan]🐛 AI Agent Debug Mode Configuration: [/bold cyan]",
+                "[bold cyan]🐛 AI Agent Debug Mode Configuration: [/ bold cyan]",
             )
             console.print(f" • AI Agent: {'✅ Enabled' if ai_agent else '❌ Disabled'}")
             console.print(
@@ -34,7 +38,6 @@ def setup_ai_agent_env(ai_agent: bool, verbose: bool = False) -> None:
 def handle_mcp_server(websocket_port: int | None = None) -> None:
     from crackerjack.mcp.server import main as start_mcp_main
 
-    # Always pass current working directory as project path
     project_path = str(Path.cwd())
 
     if websocket_port:
@@ -47,45 +50,45 @@ def handle_monitor_mode(dev_mode: bool = False) -> None:
     from crackerjack.mcp.progress_monitor import run_progress_monitor
 
     console = Console()
-    console.print("[bold cyan]🌟 Starting Multi-Project Progress Monitor[/bold cyan]")
+    console.print("[bold cyan]🌟 Starting Multi-Project Progress Monitor[/ bold cyan]")
     console.print(
-        "[bold yellow]🐕 With integrated Service Watchdog and WebSocket polling[/bold yellow]",
+        "[bold yellow]🐕 With integrated Service Watchdog and WebSocket polling[/ bold yellow]",
     )
 
     try:
         asyncio.run(run_progress_monitor(dev_mode=dev_mode))
     except KeyboardInterrupt:
-        console.print("\n[yellow]🛑 Monitor stopped[/yellow]")
+        console.print("\n[yellow]🛑 Monitor stopped[/ yellow]")
 
 
 def handle_enhanced_monitor_mode(dev_mode: bool = False) -> None:
     from crackerjack.mcp.enhanced_progress_monitor import run_enhanced_progress_monitor
 
     console = Console()
-    console.print("[bold magenta]✨ Starting Enhanced Progress Monitor[/bold magenta]")
+    console.print("[bold magenta]✨ Starting Enhanced Progress Monitor[/ bold magenta]")
     console.print(
-        "[bold cyan]📊 With advanced MetricCard widgets and modern web UI patterns[/bold cyan]",
+        "[bold cyan]📊 With advanced MetricCard widgets and modern web UI patterns[/ bold cyan]",
     )
 
     try:
         asyncio.run(run_enhanced_progress_monitor(dev_mode=dev_mode))
     except KeyboardInterrupt:
-        console.print("\n[yellow]🛑 Enhanced Monitor stopped[/yellow]")
+        console.print("\n[yellow]🛑 Enhanced Monitor stopped[/ yellow]")
 
 
 def handle_dashboard_mode(dev_mode: bool = False) -> None:
     from crackerjack.mcp.dashboard import run_dashboard
 
     console = Console()
-    console.print("[bold green]🎯 Starting Comprehensive Dashboard[/bold green]")
+    console.print("[bold green]🎯 Starting Comprehensive Dashboard[/ bold green]")
     console.print(
-        "[bold cyan]📈 With system metrics, job tracking, and performance monitoring[/bold cyan]",
+        "[bold cyan]📈 With system metrics, job tracking, and performance monitoring[/ bold cyan]",
     )
 
     try:
         run_dashboard()
     except KeyboardInterrupt:
-        console.print("\n[yellow]🛑 Dashboard stopped[/yellow]")
+        console.print("\n[yellow]🛑 Dashboard stopped[/ yellow]")
 
 
 def handle_watchdog_mode() -> None:
@@ -95,7 +98,7 @@ def handle_watchdog_mode() -> None:
     try:
         asyncio.run(start_watchdog())
     except KeyboardInterrupt:
-        console.print("\n[yellow]🛑 Watchdog stopped[/yellow]")
+        console.print("\n[yellow]🛑 Watchdog stopped[/ yellow]")
 
 
 def handle_start_websocket_server(port: int = 8675) -> None:
@@ -120,14 +123,14 @@ def handle_stop_mcp_server() -> None:
     from crackerjack.services.server_manager import list_server_status, stop_all_servers
 
     console = Console()
-    console.print("[bold red]🛑 Stopping MCP Servers[/bold red]")
+    console.print("[bold red]🛑 Stopping MCP Servers[/ bold red]")
 
     list_server_status(console)
 
     if stop_all_servers(console):
-        console.print("\n[bold green]✅ All servers stopped successfully[/bold green]")
+        console.print("\n[bold green]✅ All servers stopped successfully[/ bold green]")
     else:
-        console.print("\n[bold red]❌ Some servers failed to stop[/bold red]")
+        console.print("\n[bold red]❌ Some servers failed to stop[/ bold red]")
         raise SystemExit(1)
 
 
@@ -136,9 +139,9 @@ def handle_restart_mcp_server(websocket_port: int | None = None) -> None:
 
     console = Console()
     if restart_mcp_server(websocket_port, console):
-        console.print("\n[bold green]✅ MCP server restart completed[/bold green]")
+        console.print("\n[bold green]✅ MCP server restart completed[/ bold green]")
     else:
-        console.print("\n[bold red]❌ MCP server restart failed[/bold red]")
+        console.print("\n[bold red]❌ MCP server restart failed[/ bold red]")
         raise SystemExit(1)
 
 
@@ -161,6 +164,11 @@ def handle_standard_mode(
 
     console = Console()
 
+    # Configure global lock manager from CLI options
+    from crackerjack.executors.hook_lock_manager import hook_lock_manager
+
+    hook_lock_manager.configure_from_options(options)
+
     if orchestrated:
         handle_orchestrated_mode(options, job_id)
     else:
@@ -175,15 +183,20 @@ def handle_standard_mode(
             orchestrator = AsyncWorkflowOrchestrator(
                 console=console,
                 pkg_path=pkg_path,
+                dry_run=getattr(options, "dry_run", False),
                 web_job_id=job_id,
+                verbose=options.verbose,
+                debug=getattr(options, "debug", False),
             )
             success = asyncio.run(orchestrator.run_complete_workflow_async(options))
         else:
             orchestrator = WorkflowOrchestrator(
                 console=console,
                 pkg_path=pkg_path,
+                dry_run=getattr(options, "dry_run", False),
                 web_job_id=job_id,
                 verbose=options.verbose,
+                debug=getattr(options, "debug", False),
             )
             success = asyncio.run(orchestrator.run_complete_workflow(options))
 
@@ -195,7 +208,12 @@ def handle_orchestrated_mode(options: Options, job_id: str | None = None) -> Non
     from rich.console import Console
 
     console = Console()
-    console.print("[bold bright_blue]🚀 ORCHESTRATED MODE ENABLED[/bold bright_blue]")
+    console.print("[bold bright_blue]🚀 ORCHESTRATED MODE ENABLED[/ bold bright_blue]")
+
+    # Configure global lock manager from CLI options
+    from crackerjack.executors.hook_lock_manager import hook_lock_manager
+
+    hook_lock_manager.configure_from_options(options)
 
     try:
         from crackerjack.core.session_coordinator import SessionCoordinator
@@ -209,8 +227,8 @@ def handle_orchestrated_mode(options: Options, job_id: str | None = None) -> Non
             ProgressLevel,
         )
     except ImportError as e:
-        console.print(f"[red]Orchestrated mode not available: {e}[/red]")
-        console.print("[yellow]Falling back to standard mode[/yellow]")
+        console.print(f"[red]Orchestrated mode not available: {e}[/ red]")
+        console.print("[yellow]Falling back to standard mode[/ yellow]")
         handle_standard_mode(options, False, job_id)
         return
 
@@ -218,7 +236,7 @@ def handle_orchestrated_mode(options: Options, job_id: str | None = None) -> Non
         strategy = ExecutionStrategy(options.orchestration_strategy)
     except ValueError:
         console.print(
-            f"[red]Invalid orchestration strategy: {options.orchestration_strategy}[/red]",
+            f"[red]Invalid orchestration strategy: {options.orchestration_strategy}[/ red]",
         )
         strategy = ExecutionStrategy.ADAPTIVE
 
@@ -226,14 +244,14 @@ def handle_orchestrated_mode(options: Options, job_id: str | None = None) -> Non
         progress = ProgressLevel(options.orchestration_progress)
     except ValueError:
         console.print(
-            f"[red]Invalid progress level: {options.orchestration_progress}[/red]",
+            f"[red]Invalid progress level: {options.orchestration_progress}[/ red]",
         )
         progress = ProgressLevel.GRANULAR
 
     try:
         ai_mode = AICoordinationMode(options.orchestration_ai_mode)
     except ValueError:
-        console.print(f"[red]Invalid AI mode: {options.orchestration_ai_mode}[/red]")
+        console.print(f"[red]Invalid AI mode: {options.orchestration_ai_mode}[/ red]")
         ai_mode = AICoordinationMode.SINGLE_AGENT
 
     config = OrchestrationConfig(
@@ -242,9 +260,9 @@ def handle_orchestrated_mode(options: Options, job_id: str | None = None) -> Non
         ai_coordination_mode=ai_mode,
     )
 
-    console.print(f"[cyan]Execution Strategy: [/cyan] {strategy.value}")
-    console.print(f"[cyan]Progress Level: [/cyan] {progress.value}")
-    console.print(f"[cyan]AI Coordination: [/cyan] {ai_mode.value}")
+    console.print(f"[cyan]Execution Strategy: [/ cyan] {strategy.value}")
+    console.print(f"[cyan]Progress Level: [/ cyan] {progress.value}")
+    console.print(f"[cyan]AI Coordination: [/ cyan] {ai_mode.value}")
 
     pkg_path = Path.cwd()
     session = SessionCoordinator(console, pkg_path, web_job_id=job_id)
@@ -254,14 +272,14 @@ def handle_orchestrated_mode(options: Options, job_id: str | None = None) -> Non
         success = asyncio.run(orchestrator.execute_orchestrated_workflow(options))
         if success:
             console.print(
-                "\n[bold green]🎉 ORCHESTRATED WORKFLOW COMPLETED SUCCESSFULLY![/bold green]",
+                "\n[bold green]🎉 ORCHESTRATED WORKFLOW COMPLETED SUCCESSFULLY ![/ bold green]",
             )
         else:
-            console.print("\n[bold red]❌ ORCHESTRATED WORKFLOW FAILED[/bold red]")
+            console.print("\n[bold red]❌ ORCHESTRATED WORKFLOW FAILED[/ bold red]")
             sys.exit(1)
     except KeyboardInterrupt:
-        console.print("\n[yellow]🛑 Orchestrated workflow interrupted[/yellow]")
+        console.print("\n[yellow]🛑 Orchestrated workflow interrupted[/ yellow]")
         sys.exit(130)
     except Exception as e:
-        console.print(f"\n[red]💥 Orchestrated workflow error: {e}[/red]")
+        console.print(f"\n[red]💥 Orchestrated workflow error: {e}[/ red]")
         sys.exit(1)

crackerjack/cli/interactive.py

@@ -115,7 +115,7 @@ class InteractiveWorkflowManager:
             deps = ["cleaning"] if options.clean else []
             self.add_task(
                 "hooks",
-                "Run pre - commit hooks (fast + comprehensive)",
+                "Run pre-commit hooks (fast + comprehensive)",
                 "run_hooks_phase",
                 dependencies=deps,
             )
@@ -284,7 +284,7 @@ class InteractiveWorkflowManager:
         self.setup_workflow(options)
         if not self.tasks:
             self.console.print(
-                "[yellow]⚠️ No tasks to execute based on options[/yellow]",
+                "[yellow]⚠️ No tasks to execute based on options[/ yellow]",
             )
             return True
 
@@ -292,7 +292,7 @@ class InteractiveWorkflowManager:
         self.console.print(self.layout)
 
         if not Confirm.ask("\n🚀 Start workflow?", default=True):
-            self.console.print("[yellow]⏹️ Workflow cancelled[/yellow]")
+            self.console.print("[yellow]⏹️ Workflow cancelled[/ yellow]")
             return False
 
         return True
@@ -328,7 +328,7 @@ class InteractiveWorkflowManager:
         )
 
         if not retry:
-            self.console.print("[red]⏹️ Workflow stopped due to task failure[/red]")
+            self.console.print("[red]⏹️ Workflow stopped due to task failure[/ red]")
             return False
 
         failed_task.skip()
@@ -373,7 +373,7 @@ class InteractiveWorkflowManager:
             }
             status_text = task.status.name
             style = status_styles.get(task.status, "white")
-            duration_text = f"{task.duration: .1f}s" if task.duration else " - "
+            duration_text = f"{task.duration: .1f}s" if task.duration else "-"
             details = task.error.message if task.error else task.description
             table.add_row(
                 task.name,
@@ -385,11 +385,11 @@ class InteractiveWorkflowManager:
         self.console.print(table)
         if failed_count == 0:
             self.console.print(
-                f"\n[bold green]🎉 Workflow completed ! {success_count} / {len(self.tasks)} tasks successful[/bold green]",
+                f"\n[bold green]🎉 Workflow completed ! {success_count} / {len(self.tasks)} tasks successful[/ bold green]",
             )
         else:
             self.console.print(
-                f"\n[bold yellow]⚠️ Workflow completed with issues: {failed_count} failed, {skipped_count} skipped[/bold yellow]",
+                f"\n[bold yellow]⚠️ Workflow completed with issues: {failed_count} failed, {skipped_count} skipped[/ bold yellow]",
             )
 
 
@@ -414,7 +414,7 @@ class InteractiveCLI:
             if not success:
                 raise SystemExit(1)
         except KeyboardInterrupt:
-            self.console.print("\n[yellow]⏹️ Interactive session cancelled[/yellow]")
+            self.console.print("\n[yellow]⏹️ Interactive session cancelled[/ yellow]")
             raise SystemExit(130)
         except Exception as e:
             error = CrackerjackError(
@@ -426,10 +426,10 @@ class InteractiveCLI:
 
     def _show_welcome(self) -> None:
         welcome_panel = Panel(
-            f"[bold cyan]Welcome to Crackerjack Interactive Mode ! [/bold cyan]\n\n"
+            f"[bold cyan]Welcome to Crackerjack Interactive Mode ! [/ bold cyan]\n\n"
             f"Version: {self.pkg_version}\n"
             f"This interactive interface will guide you through the crackerjack workflow\n"
-            f"with real - time feedback and customizable options.",
+            f"with real-time feedback and customizable options.",
             title="🚀 Crackerjack Interactive",
             border_style="cyan",
         )
@@ -437,7 +437,7 @@ class InteractiveCLI:
         self.console.print()
 
     def _get_user_preferences(self, options: OptionsProtocol) -> OptionsProtocol:
-        self.console.print("[bold]🔧 Workflow Configuration[/bold]")
+        self.console.print("[bold]🔧 Workflow Configuration[/ bold]")
         self.console.print("Configure your crackerjack workflow: \n")
         updated_options = type(options)(**vars(options))
         updated_options.clean = Confirm.ask(

crackerjack/cli/options.py

@@ -23,6 +23,7 @@ class Options(BaseModel):
     publish: BumpOption | None = None
     bump: BumpOption | None = None
     verbose: bool = False
+    debug: bool = False
     clean: bool = False
     test: bool = False
     benchmark: bool = False
@@ -62,12 +63,28 @@ class Options(BaseModel):
     websocket_port: int | None = None
     dev: bool = False
     dashboard: bool = False
-    max_iterations: int = 10
+    max_iterations: int = 5
     enterprise_batch: str | None = None
     monitor_dashboard: str | None = None
     coverage_status: bool = False
     coverage_goal: float | None = None
     no_coverage_ratchet: bool = False
+    skip_config_merge: bool = False
+    disable_global_locks: bool = False
+    global_lock_timeout: int = 600
+    global_lock_cleanup: bool = True
+    global_lock_dir: str | None = None
+    quick: bool = False
+    thorough: bool = False
+
+    @property
+    def effective_max_iterations(self) -> int:
+        """Get the effective max iterations based on quick/thorough flags."""
+        if self.quick:
+            return 3  # Quick mode: 3 iterations for CI/CD
+        if self.thorough:
+            return 8  # Thorough mode: 8 iterations for complex refactoring
+        return self.max_iterations  # Default: 5 iterations
 
     @classmethod
     @field_validator("publish", "bump", mode="before")
@@ -112,6 +129,7 @@ CLI_OPTIONS = {
         help="Update pre-commit hooks configuration.",
     ),
     "verbose": typer.Option(False, "-v", "--verbose", help="Enable verbose output."),
+    "debug": typer.Option(False, "--debug", help="Enable debug output."),
     "publish": typer.Option(
         None,
         "-p",
@@ -130,7 +148,7 @@ CLI_OPTIONS = {
         False,
         "-x",
         "--clean",
-        help="Remove docstrings, line comments, and unnecessary whitespace from source code (doesn't affect test files).",
+        help="Remove docstrings, line comments, and unnecessary whitespace from source code with automatic backup protection (doesn't affect test files).",
     ),
     "test": typer.Option(False, "-t", "--test", help="Run tests."),
     "benchmark": typer.Option(
@@ -275,9 +293,9 @@ CLI_OPTIONS = {
         help="Start the comprehensive dashboard with system metrics, job tracking, and performance monitoring.",
     ),
     "max_iterations": typer.Option(
-        10,
+        5,
         "--max-iterations",
-        help="Maximum number of iterations for AI agent auto-fixing workflows (default: 10).",
+        help="Maximum number of iterations for AI agent auto-fixing workflows (default: 5).",
     ),
     "ai_debug": typer.Option(
         False,
@@ -330,6 +348,36 @@ CLI_OPTIONS = {
         "--boost-coverage/--no-boost-coverage",
         help="Automatically improve test coverage after successful workflow execution (default: True).",
     ),
+    "disable_global_locks": typer.Option(
+        False,
+        "--disable-global-locks",
+        help="Disable global locking (allow concurrent hook execution across sessions).",
+    ),
+    "global_lock_timeout": typer.Option(
+        600,
+        "--global-lock-timeout",
+        help="Global lock timeout in seconds (default: 600).",
+    ),
+    "global_lock_cleanup": typer.Option(
+        True,
+        "--cleanup-stale-locks/--no-cleanup-stale-locks",
+        help="Clean up stale global lock files before execution (default: True).",
+    ),
+    "global_lock_dir": typer.Option(
+        None,
+        "--global-lock-dir",
+        help="Custom directory for global lock files (default: ~/.crackerjack/locks).",
+    ),
+    "quick": typer.Option(
+        False,
+        "--quick",
+        help="Quick mode: Run with maximum 3 iterations (ideal for CI/CD).",
+    ),
+    "thorough": typer.Option(
+        False,
+        "--thorough",
+        help="Thorough mode: Run with maximum 8 iterations (for complex refactoring).",
+    ),
 }
 
 
@@ -339,6 +387,7 @@ def create_options(
     no_config_updates: bool,
     update_precommit: bool,
     verbose: bool,
+    debug: bool,
     publish: BumpOption | None,
     all: BumpOption | None,
     bump: BumpOption | None,
@@ -369,6 +418,12 @@ def create_options(
     coverage_goal: float | None,
     no_coverage_ratchet: bool,
     boost_coverage: bool,
+    disable_global_locks: bool,
+    global_lock_timeout: int,
+    global_lock_cleanup: bool,
+    global_lock_dir: str | None,
+    quick: bool,
+    thorough: bool,
 ) -> Options:
     return Options(
         commit=commit,
@@ -376,6 +431,7 @@ def create_options(
         no_config_updates=no_config_updates,
         update_precommit=update_precommit,
         verbose=verbose,
+        debug=debug,
         publish=publish,
         bump=bump,
         clean=clean,
@@ -406,4 +462,10 @@ def create_options(
         coverage_goal=coverage_goal,
         no_coverage_ratchet=no_coverage_ratchet,
         boost_coverage=boost_coverage,
+        disable_global_locks=disable_global_locks,
+        global_lock_timeout=global_lock_timeout,
+        global_lock_cleanup=global_lock_cleanup,
+        global_lock_dir=global_lock_dir,
+        quick=quick,
+        thorough=thorough,
     )