src/timmyctl/cli.py

@@ -95,24 +95,8 @@ def _get_config_dir() -> Path:
     return DEFAULT_CONFIG_DIR
 
 
-@app.command()
-def daily_run(
-    dry_run: bool = typer.Option(
-        False, "--dry-run", "-n", help="Show what would run without executing"
-    ),
-    verbose: bool = typer.Option(False, "--verbose", "-v", help="Show detailed output"),
-):
-    """Run the Daily Run orchestration (agenda + summary).
-
-    Executes the daily run workflow including:
-    - Loop Guard checks
-    - Cycle Retrospective
-    - Triage scoring (if scheduled)
-    - Loop introspection (if scheduled)
-    """
-    console.print("[bold green]Timmy Daily Run[/bold green]")
-    console.print()
-
+def _load_daily_run_config() -> dict[str, Any]:
+    """Load and validate the daily run configuration."""
     config_path = _get_config_dir() / "daily_run.json"
     config = _load_json_config(config_path)
 
@@ -120,14 +104,11 @@ def daily_run(
         console.print("[yellow]No daily run configuration found.[/yellow]")
         raise typer.Exit(1)
 
-    schedules = config.get("schedules", {})
-    triggers = config.get("triggers", {})
+    return config
 
-    if verbose:
-        console.print(f"[dim]Config loaded from: {config_path}[/dim]")
-        console.print()
 
-    # Show the daily run schedule
+def _display_schedules_table(schedules: dict[str, Any]) -> None:
+    """Display the daily run schedules in a table."""
     table = Table(title="Daily Run Schedules")
     table.add_column("Schedule", style="cyan")
     table.add_column("Description", style="green")
@@ -144,7 +125,9 @@ def daily_run(
     console.print(table)
     console.print()
 
-    # Show triggers
+
+def _display_triggers_table(triggers: dict[str, Any]) -> None:
+    """Display the triggers in a table."""
     trigger_table = Table(title="Triggers")
     trigger_table.add_column("Trigger", style="cyan")
     trigger_table.add_column("Description", style="green")
@@ -161,25 +144,19 @@ def daily_run(
     console.print(trigger_table)
     console.print()
 
-    if dry_run:
-        console.print("[yellow]Dry run mode — no actions executed.[/yellow]")
-    else:
-        console.print("[green]Executing daily run automations...[/green]")
-        auto_config_path = _get_config_dir() / "automations.json"
-        auto_config = _load_json_config(auto_config_path)
-        all_automations = auto_config.get("automations", [])
-        enabled = [a for a in all_automations if a.get("enabled", False)]
-        if not enabled:
-            console.print("[yellow]No enabled automations found.[/yellow]")
-        for auto in enabled:
+
+def _execute_automation(auto: dict[str, Any], verbose: bool) -> None:
+    """Execute a single automation and display results."""
     cmd = auto.get("command")
     name = auto.get("name", auto.get("id", "unnamed"))
     if not cmd:
         console.print(f"[yellow]Skipping {name} — no command defined.[/yellow]")
-                continue
+        return
+
     console.print(f"[cyan]▶ Running: {name}[/cyan]")
     if verbose:
         console.print(f"[dim]  $ {cmd}[/dim]")
+
     try:
         result = subprocess.run(  # noqa: S602
             cmd,
@@ -202,6 +179,58 @@ def daily_run(
         console.print(f"[red]  ✗ {name} failed: {exc}[/red]")
 
 
+def _execute_all_automations(verbose: bool) -> None:
+    """Execute all enabled automations."""
+    console.print("[green]Executing daily run automations...[/green]")
+    auto_config_path = _get_config_dir() / "automations.json"
+    auto_config = _load_json_config(auto_config_path)
+    all_automations = auto_config.get("automations", [])
+    enabled = [a for a in all_automations if a.get("enabled", False)]
+
+    if not enabled:
+        console.print("[yellow]No enabled automations found.[/yellow]")
+        return
+
+    for auto in enabled:
+        _execute_automation(auto, verbose)
+
+
+@app.command()
+def daily_run(
+    dry_run: bool = typer.Option(
+        False, "--dry-run", "-n", help="Show what would run without executing"
+    ),
+    verbose: bool = typer.Option(False, "--verbose", "-v", help="Show detailed output"),
+):
+    """Run the Daily Run orchestration (agenda + summary).
+
+    Executes the daily run workflow including:
+    - Loop Guard checks
+    - Cycle Retrospective
+    - Triage scoring (if scheduled)
+    - Loop introspection (if scheduled)
+    """
+    console.print("[bold green]Timmy Daily Run[/bold green]")
+    console.print()
+
+    config = _load_daily_run_config()
+    schedules = config.get("schedules", {})
+    triggers = config.get("triggers", {})
+
+    if verbose:
+        config_path = _get_config_dir() / "daily_run.json"
+        console.print(f"[dim]Config loaded from: {config_path}[/dim]")
+        console.print()
+
+    _display_schedules_table(schedules)
+    _display_triggers_table(triggers)
+
+    if dry_run:
+        console.print("[yellow]Dry run mode — no actions executed.[/yellow]")
+    else:
+        _execute_all_automations(verbose)
+
+
 @app.command()
 def log_run(
     message: str = typer.Argument(..., help="Logbook entry message"),