ripperdoc 0.2.8__py3-none-any.whl → 0.2.10__py3-none-any.whl

ripperdoc/cli/commands/agents_cmd.py

@@ -1,4 +1,7 @@
 from rich.markup import escape
+from rich import box
+from rich.panel import Panel
+from rich.table import Table
 
 from ripperdoc.core.agents import (
     AGENT_DIR_NAME,
@@ -8,14 +11,38 @@ from ripperdoc.core.agents import (
     save_agent_definition,
 )
 from ripperdoc.core.config import get_global_config
+from ripperdoc.tools.task_tool import (
+    list_agent_runs,
+    get_agent_run_snapshot,
+    cancel_agent_run,
+)
 from ripperdoc.utils.log import get_logger
 
-from typing import Any
+from typing import Any, Dict, Optional
 from .base import SlashCommand
 
 logger = get_logger()
 
 
+def _format_duration(duration_ms: float | None) -> str:
+    if duration_ms is None:
+        return "-"
+    try:
+        duration = float(duration_ms)
+    except (TypeError, ValueError):
+        return "-"
+    if duration < 1000:
+        return f"{int(duration)} ms"
+    seconds = duration / 1000.0
+    if seconds < 60:
+        return f"{seconds:.1f}s"
+    minutes, secs = divmod(int(seconds), 60)
+    if minutes < 60:
+        return f"{minutes}m {secs}s"
+    hours, mins = divmod(minutes, 60)
+    return f"{hours}h {mins}m"
+
+
 def _handle(ui: Any, trimmed_arg: str) -> bool:
     console = ui.console
     tokens = trimmed_arg.split()
@@ -39,18 +66,118 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
             "[bold]/agents delete <name> [location][/bold] — "
             "delete agent (location: user|project, default user)"
         )
+        console.print("[bold]/agents runs[/bold] — list subagent runs")
+        console.print("[bold]/agents show <id>[/bold] — show subagent run details")
+        console.print("[bold]/agents cancel <id>[/bold] — cancel a background subagent run")
         console.print(
             f"[dim]Agent files live in ~/.ripperdoc/{AGENT_DIR_NAME} "
             f"or ./.ripperdoc/{AGENT_DIR_NAME}[/dim]"
         )
         console.print(
-            "[dim]Model can be a profile name or pointer (task/main/etc). Defaults to 'task'.[/dim]"
+            "[dim]Model can be a profile name or pointer (main/quick). Defaults to 'main'.[/dim]"
         )
 
     if subcmd in ("help", "-h", "--help"):
         print_agents_usage()
         return True
 
+    if subcmd in ("runs", "run", "tasks", "status"):
+        console = ui.console
+        run_ids = list_agent_runs()
+        if not run_ids:
+            console.print(
+                Panel("No subagent runs recorded", title="Subagent runs", box=box.ROUNDED)
+            )
+            return True
+
+        table = Table(box=box.SIMPLE_HEAVY, expand=True)
+        table.add_column("ID", style="cyan", no_wrap=True)
+        table.add_column("Status", style="magenta", no_wrap=True)
+        table.add_column("Agent", style="white", no_wrap=True)
+        table.add_column("Duration", style="dim", no_wrap=True)
+        table.add_column("Background", style="dim", no_wrap=True)
+        table.add_column("Result", style="white")
+
+        for run_id in sorted(run_ids):
+            snapshot: Dict[Any, Any] = get_agent_run_snapshot(run_id) or {}
+            result_text = snapshot.get("result_text") or snapshot.get("error") or ""
+            result_preview = (
+                result_text if len(result_text) <= 80 else result_text[:77] + "..."
+            )
+            table.add_row(
+                escape(run_id),
+                escape(snapshot.get("status") or "unknown"),
+                escape(snapshot.get("agent_type") or "unknown"),
+                _format_duration(snapshot.get("duration_ms")),
+                "yes" if snapshot.get("is_background") else "no",
+                escape(result_preview),
+            )
+
+        console.print(
+            Panel(table, title="Subagent runs", box=box.ROUNDED, padding=(1, 2)),
+            markup=False,
+        )
+        console.print(
+            "[dim]Use /agents show <id> for details or /agents cancel <id> to stop a background run.[/dim]"
+        )
+        return True
+
+    if subcmd in ("show", "info", "details"):
+        if len(tokens) < 2:
+            console.print("[red]Usage: /agents show <id>[/red]")
+            return True
+        run_id = tokens[1]
+        snapshot = get_agent_run_snapshot(run_id)  # type: ignore[assignment]
+        if not snapshot:
+            console.print(f"[red]No subagent run found with id '{escape(run_id)}'.[/red]")
+            return True
+        details = Table(box=box.SIMPLE_HEAVY, show_header=False)
+        details.add_row("ID", escape(run_id))
+        details.add_row("Status", escape(snapshot.get("status") or "unknown"))
+        details.add_row("Agent", escape(snapshot.get("agent_type") or "unknown"))
+        details.add_row("Duration", _format_duration(snapshot.get("duration_ms")))
+        details.add_row(
+            "Background", "yes" if snapshot.get("is_background") else "no"
+        )
+        if snapshot.get("model_used"):
+            details.add_row("Model", escape(str(snapshot.get("model_used"))))
+        if snapshot.get("tool_use_count"):
+            details.add_row("Tool uses", str(snapshot.get("tool_use_count")))
+        if snapshot.get("missing_tools"):
+            details.add_row("Missing tools", escape(", ".join(snapshot["missing_tools"])))
+        if snapshot.get("error"):
+            details.add_row("Error", escape(str(snapshot.get("error"))))
+        console.print(
+            Panel(details, title=f"Subagent {escape(run_id)}", box=box.ROUNDED, padding=(1, 2)),
+            markup=False,
+        )
+        result_text = snapshot.get("result_text")
+        if result_text:
+            console.print(Panel(escape(result_text), title="Result", box=box.SIMPLE))
+        return True
+
+    if subcmd in ("cancel", "kill", "stop"):
+        if len(tokens) < 2:
+            console.print("[red]Usage: /agents cancel <id>[/red]")
+            return True
+        run_id = tokens[1]
+        runner = getattr(ui, "run_async", None)
+        try:
+            if callable(runner):
+                cancelled = runner(cancel_agent_run(run_id))
+            else:
+                import asyncio
+
+                cancelled = asyncio.run(cancel_agent_run(run_id))
+        except (OSError, RuntimeError, ValueError) as exc:
+            console.print(f"[red]Failed to cancel '{escape(run_id)}': {escape(str(exc))}[/red]")
+            return True
+        if cancelled:
+            console.print(f"[green]Cancelled subagent {escape(run_id)}[/green]")
+        else:
+            console.print(f"[yellow]No running subagent found for '{escape(run_id)}'.[/yellow]")
+        return True
+
     if subcmd in ("create", "add"):
         agent_name = tokens[1] if len(tokens) > 1 else console.input("Agent name: ").strip()
         if not agent_name:
@@ -84,7 +211,7 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
 
         config = get_global_config()
         pointer_map = config.model_pointers.model_dump()
-        default_model_value = model_arg or pointer_map.get("task", "task")
+        default_model_value = model_arg or pointer_map.get("main", "main")
         model_input = (
             console.input(f"Model profile or pointer [{default_model_value}]: ").strip()
             or default_model_value
@@ -116,7 +243,8 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
             print_agents_usage()
             logger.warning(
                 "[agents_cmd] Failed to create agent: %s: %s",
-                type(exc).__name__, exc,
+                type(exc).__name__,
+                exc,
                 extra={"agent": agent_name, "session_id": getattr(ui, "session_id", None)},
             )
         return True
@@ -148,7 +276,8 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
             print_agents_usage()
             logger.warning(
                 "[agents_cmd] Failed to delete agent: %s: %s",
-                type(exc).__name__, exc,
+                type(exc).__name__,
+                exc,
                 extra={"agent": agent_name, "session_id": getattr(ui, "session_id", None)},
             )
         return True
@@ -203,7 +332,7 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
 
         config = get_global_config()
         pointer_map = config.model_pointers.model_dump()
-        model_default = target_agent.model or pointer_map.get("task", "task")
+        model_default = target_agent.model or pointer_map.get("main", "main")
         model_input = (
             console.input(f"Model profile or pointer [{model_default}]: ").strip() or model_default
         )
@@ -226,7 +355,8 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
             print_agents_usage()
             logger.warning(
                 "[agents_cmd] Failed to update agent: %s: %s",
-                type(exc).__name__, exc,
+                type(exc).__name__,
+                exc,
                 extra={"agent": agent_name, "session_id": getattr(ui, "session_id", None)},
             )
         return True
@@ -242,7 +372,7 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
         console.print(f"  • {escape(agent.agent_type)} ({escape(str(location))})", markup=False)
         console.print(f"      {escape(agent.when_to_use)}", markup=False)
         console.print(f"      tools: {escape(tools_str)}", markup=False)
-        console.print(f"      model: {escape(agent.model or 'task (default)')}", markup=False)
+        console.print(f"      model: {escape(agent.model or 'main (default)')}", markup=False)
     if agents.failed_files:
         console.print("[yellow]Some agent files could not be loaded:[/yellow]")
         for path, error in agents.failed_files:

ripperdoc/cli/commands/clear_cmd.py

@@ -3,16 +3,21 @@ from .base import SlashCommand
 
 
 def _handle(ui: Any, _: str) -> bool:
+    try:
+        ui._run_session_end("clear")
+    except (AttributeError, RuntimeError, ValueError):
+        pass
     ui.conversation_messages = []
     ui.console.print("[green]✓ Conversation cleared[/green]")
+    try:
+        ui._run_session_start("clear")
+    except (AttributeError, RuntimeError, ValueError):
+        pass
     return True
 
 
 command = SlashCommand(
-    name="clear",
-    description="Clear conversation history",
-    handler=_handle,
-    aliases=("new",)
+    name="clear", description="Clear conversation history", handler=_handle, aliases=("new",)
 )
 
 

ripperdoc/cli/commands/config_cmd.py

@@ -16,7 +16,7 @@ def _handle(ui: Any, _: str) -> bool:
     ui.console.print(
         f"\n[bold]Model (main -> {escape(str(main_pointer))}):[/bold] {escape(str(model_label))}"
     )
-    ui.console.print(f"[bold]Safe Mode:[/bold] {escape(str(ui.safe_mode))}")
+    ui.console.print(f"[bold]Yolo Mode:[/bold] {escape(str(ui.yolo_mode))}")
     ui.console.print(f"[bold]Verbose:[/bold] {escape(str(ui.verbose))}")
     return True
 

ripperdoc/cli/commands/context_cmd.py

@@ -40,7 +40,7 @@ def _handle(ui: Any, _: str) -> bool:
     if not ui.query_context:
         ui.query_context = QueryContext(
             tools=ui.get_default_tools(),
-            safe_mode=ui.safe_mode,
+            yolo_mode=ui.yolo_mode,
             verbose=ui.verbose,
         )
 
@@ -126,7 +126,8 @@ def _handle(ui: Any, _: str) -> bool:
     except (OSError, RuntimeError, AttributeError, TypeError) as exc:
         logger.warning(
             "[context_cmd] Failed to summarize MCP tools: %s: %s",
-            type(exc).__name__, exc,
+            type(exc).__name__,
+            exc,
             extra={"session_id": getattr(ui, "session_id", None)},
         )
     for line in lines:

ripperdoc/cli/commands/doctor_cmd.py

@@ -125,10 +125,17 @@ def _mcp_status(
             servers = asyncio.run(_load())
         else:
             servers = runner(_load())
-    except (OSError, RuntimeError, ConnectionError, ValueError, TypeError) as exc:  # pragma: no cover - defensive
+    except (
+        OSError,
+        RuntimeError,
+        ConnectionError,
+        ValueError,
+        TypeError,
+    ) as exc:  # pragma: no cover - defensive
         logger.warning(
             "[doctor] Failed to load MCP servers: %s: %s",
-            type(exc).__name__, exc,
+            type(exc).__name__,
+            exc,
             exc_info=exc,
         )
         rows.append(_status_row("MCP", "error", f"Failed to load MCP config: {exc}"))
@@ -161,10 +168,17 @@ def _project_status(project_path: Path) -> Tuple[str, str, str]:
         return _status_row(
             "Project config", "ok", f".ripperdoc/config.json loaded for {project_path}"
         )
-    except (OSError, IOError, json.JSONDecodeError, ValueError, TypeError) as exc:  # pragma: no cover - defensive
+    except (
+        OSError,
+        IOError,
+        json.JSONDecodeError,
+        ValueError,
+        TypeError,
+    ) as exc:  # pragma: no cover - defensive
         logger.warning(
             "[doctor] Failed to load project config: %s: %s",
-            type(exc).__name__, exc,
+            type(exc).__name__,
+            exc,
             exc_info=exc,
         )
         return _status_row(

ripperdoc/cli/commands/exit_cmd.py

@@ -4,6 +4,7 @@ from .base import SlashCommand
 
 def _handle(ui: Any, _: str) -> bool:
     ui.console.print("[yellow]Goodbye![/yellow]")
+    ui._exit_reason = "prompt_input_exit"
     ui._should_exit = True
     return True
 

ripperdoc/cli/commands/hooks_cmd.py

@@ -56,15 +56,9 @@ class HookConfigTarget:
 def _print_usage(console: Any) -> None:
     """Display available subcommands."""
     console.print("[bold]/hooks[/bold] — show configured hooks")
-    console.print(
-        "[bold]/hooks add [scope][/bold] — guided creator (scope: local|project|global)"
-    )
-    console.print(
-        "[bold]/hooks edit [scope][/bold] — step-by-step edit of an existing hook"
-    )
-    console.print(
-        "[bold]/hooks delete [scope][/bold] — remove a hook entry (alias: del)"
-    )
+    console.print("[bold]/hooks add [scope][/bold] — guided creator (scope: local|project|global)")
+    console.print("[bold]/hooks edit [scope][/bold] — step-by-step edit of an existing hook")
+    console.print("[bold]/hooks delete [scope][/bold] — remove a hook entry (alias: del)")
     console.print(
         "[dim]Scopes: local=.ripperdoc/hooks.local.json (git-ignored), "
         "project=.ripperdoc/hooks.json (shared), "
@@ -96,7 +90,9 @@ def _get_targets(project_path: Path) -> List[HookConfigTarget]:
     ]
 
 
-def _select_target(console: Any, project_path: Path, scope_hint: Optional[str]) -> Optional[HookConfigTarget]:
+def _select_target(
+    console: Any, project_path: Path, scope_hint: Optional[str]
+) -> Optional[HookConfigTarget]:
     """Prompt user to choose a hooks config target."""
     targets = _get_targets(project_path)
 
@@ -122,9 +118,7 @@ def _select_target(console: Any, project_path: Path, scope_hint: Optional[str])
                 f"      [dim]{target.description}[/dim]"
             )
 
-        choice = console.input(
-            f"Location [1-{len(targets)}, default {default_idx + 1}]: "
-        ).strip()
+        choice = console.input(f"Location [1-{len(targets)}, default {default_idx + 1}]: ").strip()
 
         if not choice:
             return targets[default_idx]
@@ -151,9 +145,7 @@ def _load_hooks_json(console: Any, path: Path) -> Dict[str, List[Dict[str, Any]]
         logger.warning("[hooks_cmd] Invalid JSON in %s: %s", path, exc)
         return {}
     except (OSError, IOError, PermissionError) as exc:
-        console.print(
-            f"[red]Unable to read {escape(str(path))}: {escape(str(exc))}[/red]"
-        )
+        console.print(f"[red]Unable to read {escape(str(path))}: {escape(str(exc))}[/red]")
         logger.warning("[hooks_cmd] Failed to read %s: %s", path, exc)
         return {}
 
@@ -176,9 +168,7 @@ def _load_hooks_json(console: Any, path: Path) -> Dict[str, List[Dict[str, Any]]
             if not isinstance(hooks_list, list):
                 continue
             cleaned_hooks = [h for h in hooks_list if isinstance(h, dict)]
-            cleaned_matchers.append(
-                {"matcher": matcher.get("matcher"), "hooks": cleaned_hooks}
-            )
+            cleaned_matchers.append({"matcher": matcher.get("matcher"), "hooks": cleaned_hooks})
         if cleaned_matchers:
             hooks[event_name] = cleaned_matchers
 
@@ -229,9 +219,7 @@ def _render_hooks_overview(ui: Any, project_path: Path) -> bool:
         if target.path.exists():
             ui.console.print(f"  [green]✓[/green] {target.label}: {target.path}")
         else:
-            ui.console.print(
-                f"  [dim]○[/dim] {target.label}: {target.path} [dim](not found)[/dim]"
-            )
+            ui.console.print(f"  [dim]○[/dim] {target.label}: {target.path} [dim](not found)[/dim]")
 
     ui.console.print()
 
@@ -289,9 +277,7 @@ def _render_hooks_overview(ui: Any, project_path: Path) -> bool:
         ui.console.print(table)
         ui.console.print()
 
-    ui.console.print(
-        "[dim]Tip: Hooks run in order. /hooks add launches a guided setup.[/dim]"
-    )
+    ui.console.print("[dim]Tip: Hooks run in order. /hooks add launches a guided setup.[/dim]")
     return True
 
 
@@ -338,32 +324,28 @@ def _prompt_matcher_selection(
     if event_name not in MATCHER_EVENTS:
         if matchers:
             return matchers[0]
-        matcher = {"matcher": None, "hooks": []}
-        matchers.append(matcher)
-        return matcher
+        default_matcher: Dict[str, Any] = {"matcher": None, "hooks": []}
+        matchers.append(default_matcher)
+        return default_matcher
 
     if not matchers:
-        console.print(
-            "\nMatcher (tool name or regex). Leave empty to match all tools (*)."
-        )
+        console.print("\nMatcher (tool name or regex). Leave empty to match all tools (*).")
         pattern = console.input("Matcher: ").strip() or "*"
-        matcher = {"matcher": pattern, "hooks": []}
-        matchers.append(matcher)
-        return matcher
+        first_matcher: Dict[str, Any] = {"matcher": pattern, "hooks": []}
+        matchers.append(first_matcher)
+        return first_matcher
 
     console.print("\nSelect matcher:")
     for idx, matcher in enumerate(matchers, start=1):
         label = matcher.get("matcher") or "*"
-        hook_count = len(matcher.get("hooks", []))
+        hook_count = len(matcher.get("hooks") or [])
         console.print(f"  [{idx}] {escape(str(label))} ({hook_count} hook(s))")
     new_idx = len(matchers) + 1
     console.print(f"  [{new_idx}] New matcher pattern")
 
     default_choice = 1
     while True:
-        choice = console.input(
-            f"Matcher [1-{new_idx}, default {default_choice}]: "
-        ).strip()
+        choice = console.input(f"Matcher [1-{new_idx}, default {default_choice}]: ").strip()
         if not choice:
             choice = str(default_choice)
         if choice.isdigit():
@@ -372,9 +354,9 @@ def _prompt_matcher_selection(
                 return matchers[idx - 1]
             if idx == new_idx:
                 pattern = console.input("New matcher (blank for '*'): ").strip() or "*"
-                matcher = {"matcher": pattern, "hooks": []}
-                matchers.append(matcher)
-                return matcher
+                created_matcher: Dict[str, Any] = {"matcher": pattern, "hooks": []}
+                matchers.append(created_matcher)
+                return created_matcher
         console.print("[red]Choose a matcher number from the list.[/red]")
 
 
@@ -410,9 +392,7 @@ def _prompt_hook_details(
     )
     while True:
         hook_type = (
-            console.input(
-                f"Hook type ({type_label}) [default {default_type}]: "
-            ).strip()
+            console.input(f"Hook type ({type_label}) [default {default_type}]: ").strip()
             or default_type
         ).lower()
         if hook_type in allowed_types:
@@ -424,14 +404,10 @@ def _prompt_hook_details(
     if hook_type == "prompt":
         existing_prompt = (existing_hook or {}).get("prompt", "")
         if existing_prompt:
-            console.print(
-                f"[dim]Current prompt:[/dim] {escape(existing_prompt)}", markup=False
-            )
+            console.print(f"[dim]Current prompt:[/dim] {escape(existing_prompt)}", markup=False)
         while True:
             prompt_text = (
-                console.input(
-                    "Prompt template (use $ARGUMENTS for JSON input): "
-                ).strip()
+                console.input("Prompt template (use $ARGUMENTS for JSON input): ").strip()
                 or existing_prompt
             )
             if prompt_text:
@@ -519,9 +495,7 @@ def _handle_edit(ui: Any, tokens: List[str], project_path: Path) -> bool:
 
     hooks = _load_hooks_json(console, target.path)
     if not hooks:
-        console.print(
-            "[yellow]No hooks found in this file. Use /hooks add to create one.[/yellow]"
-        )
+        console.print("[yellow]No hooks found in this file. Use /hooks add to create one.[/yellow]")
         return True
 
     event_options = list(hooks.keys())

ripperdoc/cli/commands/models_cmd.py

@@ -35,7 +35,9 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
         console.print("[bold]/models edit <name>[/bold] — edit an existing model profile")
         console.print("[bold]/models delete <name>[/bold] — delete a model profile")
         console.print("[bold]/models use <name>[/bold] — set the main model pointer")
-        console.print("[bold]/models use <pointer> <name>[/bold] — set a specific pointer (main/task/reasoning/quick)")
+        console.print(
+            "[bold]/models use <pointer> <name>[/bold] — set a specific pointer (main/quick)"
+        )
 
     def parse_int(prompt_text: str, default_value: Optional[int]) -> Optional[int]:
         raw = console.input(prompt_text).strip()
@@ -186,7 +188,8 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
             console.print(f"[red]Failed to save model: {escape(str(exc))}[/red]")
             logger.warning(
                 "[models_cmd] Failed to save model profile: %s: %s",
-                type(exc).__name__, exc,
+                type(exc).__name__,
+                exc,
                 extra={"profile": profile_name, "session_id": getattr(ui, "session_id", None)},
             )
             return True
@@ -295,7 +298,8 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
             console.print(f"[red]Failed to update model: {escape(str(exc))}[/red]")
             logger.warning(
                 "[models_cmd] Failed to update model profile: %s: %s",
-                type(exc).__name__, exc,
+                type(exc).__name__,
+                exc,
                 extra={"profile": profile_name, "session_id": getattr(ui, "session_id", None)},
             )
             return True
@@ -319,21 +323,24 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
             print_models_usage()
             logger.warning(
                 "[models_cmd] Failed to delete model profile: %s: %s",
-                type(exc).__name__, exc,
+                type(exc).__name__,
+                exc,
                 extra={"profile": target, "session_id": getattr(ui, "session_id", None)},
             )
         return True
 
     if subcmd in ("use", "main", "set-main"):
         # Support both "/models use <profile>" and "/models use <pointer> <profile>"
-        valid_pointers = {"main", "task", "reasoning", "quick"}
+        valid_pointers = {"main", "quick"}
 
         if len(tokens) >= 3:
             # /models use <pointer> <profile>
             pointer = tokens[1].lower()
             target = tokens[2]
             if pointer not in valid_pointers:
-                console.print(f"[red]Invalid pointer '{escape(pointer)}'. Valid pointers: {', '.join(valid_pointers)}[/red]")
+                console.print(
+                    f"[red]Invalid pointer '{escape(pointer)}'. Valid pointers: {', '.join(valid_pointers)}[/red]"
+                )
                 print_models_usage()
                 return True
         elif len(tokens) >= 2:
@@ -346,9 +353,14 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
                 pointer = "main"
                 target = tokens[1]
         else:
-            pointer = console.input("Pointer (main/task/reasoning/quick) [main]: ").strip().lower() or "main"
+            pointer = (
+                console.input("Pointer (main/quick) [main]: ").strip().lower()
+                or "main"
+            )
             if pointer not in valid_pointers:
-                console.print(f"[red]Invalid pointer '{escape(pointer)}'. Valid pointers: {', '.join(valid_pointers)}[/red]")
+                console.print(
+                    f"[red]Invalid pointer '{escape(pointer)}'. Valid pointers: {', '.join(valid_pointers)}[/red]"
+                )
                 return True
             target = console.input(f"Model to use for '{pointer}': ").strip()
 
@@ -364,8 +376,13 @@ def _handle(ui: Any, trimmed_arg: str) -> bool:
             print_models_usage()
             logger.warning(
                 "[models_cmd] Failed to set model pointer: %s: %s",
-                type(exc).__name__, exc,
-                extra={"pointer": pointer, "profile": target, "session_id": getattr(ui, "session_id", None)},
+                type(exc).__name__,
+                exc,
+                extra={
+                    "pointer": pointer,
+                    "profile": target,
+                    "session_id": getattr(ui, "session_id", None),
+                },
             )
         return True