lisa-cli 0.3.0__tar.gz → 0.3.1__tar.gz

{lisa_cli-0.3.0/src/lisa_cli.egg-info → lisa_cli-0.3.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: lisa-cli
-Version: 0.3.0
+Version: 0.3.1
 Summary: Terminal CLI for the Lisa Harness
 License-Expression: MIT
 Requires-Python: >=3.11

lisa_cli-0.3.1/pyproject.toml

@@ -0,0 +1,177 @@
+[build-system]
+requires = ["setuptools>=77"]
+build-backend = "setuptools.build_meta"
+
+[tool.setuptools.packages.find]
+where = ["src"]
+
+[project]
+name = "lisa-cli"
+version = "0.3.1"
+description = "Terminal CLI for the Lisa Harness"
+requires-python = ">=3.11"
+license = "MIT"
+license-files = ["LICENSE"]
+dependencies = [
+    "lisa-harness-runtime",
+    "lisa-harness-kernel",
+    "lisa-harness-sdk",
+    "lisa-harness-mcp-servers",
+    "lisa-pack-core",
+    "lisa-pack-introspect",
+    "lisa-pack-memory",
+    "lisa-pack-recovery",
+    "openai>=1.30.0",
+    "rich>=13.7.0",
+    "typer>=0.12.3",
+    "pyyaml>=6.0",
+    "httpx>=0.27.0",
+    "pydantic>=2.7.0",
+    "python-dotenv>=1.0.0",
+]
+
+[project.scripts]
+lisa = "hosts.cli.main:main"
+
+[project.optional-dependencies]
+dev = [
+    "pyinstaller>=6.10.0",
+    "pytest>=8.0.0",
+    "pytest-asyncio>=0.23.0",
+    "ruff>=0.8.0",
+    "types-PyYAML>=6.0",
+]
+# Interactive no-args `lisa prompt` editor: single-box input with
+# history. Without the extra the plain EOF-terminated reader is used.
+prompt = ["prompt-toolkit>=3.0"]
+
+[tool.pytest.ini_options]
+pythonpath = ["src"]
+testpaths = ["tests"]
+markers = [
+    "live: hits real external signal or provider APIs and prints a tester-visible summary",
+    "integration: multi-module in-process test; may require local services",
+    "eval: deterministic agent regression evals (no model calls, no I/O)",
+]
+
+[tool.coverage.run]
+branch = true
+relative_files = true
+source = ["src"]
+omit = [
+    "*/tests/*",
+    "*/test_*.py",
+    "*/conftest.py",
+]
+source_pkgs = ["hosts.cli"]
+
+[tool.coverage.paths]
+source = [
+    "src",
+    "*/src",
+]
+
+[tool.coverage.report]
+show_missing = true
+skip_covered = true
+fail_under = 85
+exclude_also = [
+    ".*/tests/.*",
+    ".*/test_[^/]*\\.py$",
+]
+
+[tool.ruff]
+line-length = 100
+target-version = "py311"
+extend-exclude = ["vulture_allowlist.py"]
+
+[tool.ruff.lint]
+select = [
+    "E",
+    "F",
+    "I",
+    "UP",
+    "B",
+    "SIM",
+    "PL",
+    "C90",
+    "RET",
+    "RUF",
+    "W",
+    "YTT",
+    "C4",
+    "PERF",
+    "PIE",
+    "FURB",
+    "G",
+]
+ignore = ["PLR0913", "PLC0415", "RUF001", "RUF002", "RUF003"]
+
+[tool.ruff.lint.per-file-ignores]
+"**/tests/**" = ["PLC0415", "PLR2004"]
+"**/hosts/cli/main.py" = ["E402", "PLC0415", "PLR0917"]
+"**/hosts/cli/render.py" = ["PLC0415"]
+"**/hosts/cli/entry.py" = ["PLC0415", "RUF001"]
+
+[tool.ruff.lint.mccabe]
+max-complexity = 10
+
+[tool.ruff.lint.pylint]
+max-returns = 6
+max-branches = 12
+max-statements = 50
+max-locals = 15
+
+[tool.ruff.format]
+line-ending = "lf"
+
+[tool.mypy]
+python_version = "3.11"
+files = ["src/hosts/cli"]
+mypy_path = ["$MYPY_CONFIG_FILE_DIR/src"]
+explicit_package_bases = true
+strict = true
+warn_unused_ignores = true
+warn_redundant_casts = true
+warn_return_any = true
+no_implicit_optional = true
+check_untyped_defs = true
+disallow_untyped_defs = true
+disallow_incomplete_defs = true
+exclude = ["vulture_allowlist.py", '(^|/)(tests|test_[^/]*\.py$)']
+
+[tool.vulture]
+paths = ["src/hosts/cli", "vulture_allowlist.py"]
+min_confidence = 80
+exclude = [
+    "**/tests/",
+    "**/test_*.py",
+    "**/conftest.py",
+    "**/*egg-info*/",
+    "**/__pycache__/",
+    ".venv/",
+]
+ignore_names = [
+    "main",
+    "*SignalCollector",
+    "*Collector",
+    "*ProcessingLoop",
+    "*Detector",
+    "*Registry",
+    "*Store",
+    "*Broker",
+    "*Catalog",
+    "*Spec",
+    "*Config",
+    "*Policy",
+    "*Manifest",
+    "*Error",
+    "*Exception",
+    "*Context",
+    "*Fragments",
+    "*Sections",
+    "*Evidence",
+    "*Summary",
+    "*Anomalies",
+]
+ignore_decorators = ["@property"]

{lisa_cli-0.3.0 → lisa_cli-0.3.1}/src/hosts/cli/entry.py

@@ -50,10 +50,13 @@ from __future__ import annotations
 import os
 import unicodedata
 from pathlib import Path
-from typing import Any
+from typing import TYPE_CHECKING, Any
 
 from rich.console import Console
 
+if TYPE_CHECKING:
+    from prompt_toolkit.key_binding import KeyBindings
+
 #: Unicode general categories that extend a grapheme cluster: combining
 #: marks, enclosing marks, format characters (variation selectors, ZWSP),
 #: and modifier symbols (skin-tone modifiers). ZWJ itself is ``Cf`` but is
@@ -186,7 +189,7 @@ def settings_file_path(data_dir: str | Path, profile: str) -> Path:
     """
     from runtime.config import profile_data_dir  # lazy: import cost
 
-    base = str(data_dir).strip() if str(data_dir).strip() else ""
+    base = str(data_dir).strip() or ""
     if not base and profile.strip():
         base = str(profile_data_dir(profile.strip()))
     if not base:
@@ -289,7 +292,7 @@ def prompt_history_path(data_dir: str | Path, profile: str) -> Path:
     the profile dir; empty hands the decision to the framework's default
     (``LISA_DATA_DIR`` > ``~/.lisa``).
     """
-    base = str(data_dir).strip() if str(data_dir).strip() else ""
+    base = str(data_dir).strip() or ""
     if not base and profile.strip():
         from runtime.config import profile_data_dir  # lazy: import cost
 
@@ -429,7 +432,7 @@ def _delete_first_grapheme(text: str) -> int:
     return i
 
 
-def _add_navigate_bindings(bindings: Any, *, swapped: bool = True) -> None:
+def _add_navigate_bindings(bindings: KeyBindings, *, swapped: bool = True) -> None:
     """Bind submit/newline and navigation keys.
 
     ``swapped`` (the default) puts newline on Enter and send on
@@ -476,7 +479,7 @@ def _add_navigate_bindings(bindings: Any, *, swapped: bool = True) -> None:
         event.current_buffer.history_forward()
 
 
-def _add_grapheme_delete_bindings(bindings: Any) -> None:
+def _add_grapheme_delete_bindings(bindings: KeyBindings) -> None:
     """Bind backspace (Ctrl-H) and forward Delete to cluster-aware cuts.
 
     prompt_toolkit deletes one codepoint, tearing multi-codepoint emoji
@@ -504,14 +507,17 @@ def _add_grapheme_delete_bindings(bindings: Any) -> None:
         buffer.cursor_position = pos
 
 
-def _tui_key_bindings(swapped: bool = True) -> Any:
+def _tui_key_bindings(swapped: bool = True) -> KeyBindings:
     """Build the owned key model for the TUI reader.
 
     ``swapped`` selects the send/newline layout (see
     :func:`_add_navigate_bindings`); this exists so the reader body
     stays under its complexity budget.
     """
-    bindings = _PT["KeyBindings"]()
+    pt = _PT
+    if pt is None:  # pragma: no cover - guarded by the caller's _PT check
+        raise RuntimeError("_tui_key_bindings requires the prompt extra")
+    bindings: KeyBindings = pt["KeyBindings"]()
     _add_navigate_bindings(bindings, swapped=swapped)
     _add_grapheme_delete_bindings(bindings)
     return bindings

{lisa_cli-0.3.0 → lisa_cli-0.3.1}/src/hosts/cli/main.py

@@ -19,7 +19,7 @@ import subprocess
 import sys
 from collections.abc import Callable
 from pathlib import Path
-from typing import TYPE_CHECKING, Annotated, Any, Literal, NamedTuple
+from typing import TYPE_CHECKING, Annotated, Any, Literal, NamedTuple, cast
 
 import typer
 from kernel.models import McpServerHealth
@@ -187,7 +187,7 @@ _TRUTHY = {"1", "true", "yes", "on"}
 # by ``_cli_callback``) recorded here for the prompt command to OR in when
 # lazy routing lands there (``lisa -r "fix it"``). One CLI process runs the
 # group callback at most once, before any command body.
-_GROUP_HOIST: dict[str, bool] = {}
+_GROUP_HOIST: dict[str, bool | None] = {}
 
 # Env var enabling lazy routing: an unrecognized first positional is
 # resolved by the custom group (``_LisaGroup.resolve_command``) to the
@@ -301,7 +301,7 @@ def _make_pack_command(name: str) -> Any:
             bare_prefix = prefix.lstrip("/")
             if bare_prefix and tokens[0].startswith(bare_prefix) and len(bare_prefix) > len(best):
                 best = bare_prefix
-        command = "/" + (best if best else name)
+        command = "/" + (best or name)
         for token in args:
             command += " " + token
         _run_host_command(
@@ -417,7 +417,7 @@ def _get_version() -> str:
 
         return _pkg_version("lisa-cli")
     except Exception:  # pylint: disable=broad-except
-        return "0.3.0"
+        return "0.3.1"
 
 
 _VERSION_OPTION = typer.Option(
@@ -489,8 +489,9 @@ ProfileOption = Annotated[
     typer.Option(
         "--profile",
         help=(
-            "Named data-dir profile: resolves to ~/.lisa/profiles/<name> "
-            "(overrides LISA_PROFILE; --data-dir still wins). Shares the "
+            "Named data-dir profile: resolves to <profiles-root>/<name> "
+            "(default ~/.lisa/profiles; relocatable via LISA_PROFILES_DIR). "
+            "Overrides LISA_PROFILE; --data-dir still wins. Shares the "
             "base packs venv, isolates threads/memory/models/policy."
         ),
     ),
@@ -711,7 +712,7 @@ def _initialize_runtime(
     wiring -> MCP config compilation) lives in the shared bootstrap so
     the CLI and web hosts cannot drift.
 
-    ``profile`` resolves to ``~/.lisa/profiles/<name>`` here (CLI arg >
+    ``profile`` resolves to ``<profiles-root>/<name>`` here (CLI arg >
     ``LISA_PROFILE``; an explicit ``data_dir`` still wins) because
     ``--data-dir`` and ``--profile`` are both CLI-level data-dir inputs —
     the framework's ``HostConfig.from_env`` only knows the env var.
@@ -777,7 +778,11 @@ def _render_runtime_error(console: Console, exc: Exception, *, title: str = "Err
 # ---------------------------------------------------------------------------
 
 
-@app.callback()
+# invoke_without_command lets the eager --version flag short-circuit on a bare
+# `lisa --version` (Click otherwise raises "Missing command" before the
+# callback runs). With no subcommand and no --version the callback mirrors the
+# group's no-args help + exit 2.
+@app.callback(invoke_without_command=True)
 def _cli_callback(
     ctx: typer.Context,
     version: bool = _VERSION_OPTION,
@@ -821,6 +826,9 @@ def _cli_callback(
     if version:
         console.print(f"lisa v{_get_version()}")
         raise typer.Exit()
+    if ctx.invoked_subcommand is None:
+        console.print(ctx.get_help())
+        raise typer.Exit(code=2)
 
 
 # ---------------------------------------------------------------------------
@@ -2109,8 +2117,10 @@ def _collect_interactive_or_error(
     if sys.stdin.isatty() and sys.stdout.isatty():
         from hosts.cli import entry as entry_mod
 
-        reader = _prompt_reader or entry_mod.read_prompt_message_tui
-        if reader is entry_mod.read_prompt_message_tui:
+        if _prompt_reader is not None:
+            # Custom reader seam (tests): the plain-reader contract.
+            terminal_message = _prompt_reader(console)
+        else:
             # Flag state -> reader layout (True=shipped, False=chat-box):
             #   absent (None)      -> env decides (swapped_keys=None)
             #   --no-swap-keys     -> shipped (swapped_keys=True)
@@ -2122,16 +2132,13 @@ def _collect_interactive_or_error(
                 resolved = True
             else:
                 resolved = False
-            terminal_message = reader(
+            terminal_message = entry_mod.read_prompt_message_tui(
                 console,
                 history_path=entry_mod.prompt_history_path(data_dir, profile),
                 swapped_keys=resolved,
                 data_dir=data_dir,
                 profile=profile,
             )
-        else:
-            # Custom reader seam (tests): the plain-reader contract.
-            terminal_message = reader(console)
         if terminal_message is None:
             raise typer.Exit(code=_ABORT_EXIT) from None
         if terminal_message.strip():
@@ -2226,7 +2233,9 @@ async def _resolve_pending_request_id(runtime: HostRuntime) -> str:
         logger.debug("prompt: pending-approval resolution failed: %s", exc)
         return ""
     if len(pending) == 1:
-        request_id = pending[0].invocation.request_id
+        # The service reaches the CLI as Any (duck-typed seam), so reassert
+        # the declared kernel field type for the return contract.
+        request_id = cast("str", pending[0].invocation.request_id)
         logger.info("prompt: resolved bare decision to pending %s", request_id[:8])
         return request_id
     return ""

{lisa_cli-0.3.0 → lisa_cli-0.3.1}/src/hosts/cli/render.py

@@ -423,8 +423,10 @@ def _containment_badge(containment: str) -> tuple[str, str]:
     return "✗ NOT SANDBOXED", STYLE_CONTAINMENT_NONE
 
 
-def _format_approval_expiry(seconds: object) -> str:
+def _format_approval_expiry(seconds: int | float | str | None) -> str:
     """Format a host-supplied approval TTL for compact panel display."""
+    if seconds is None:
+        return ""
     try:
         remaining = max(0, int(seconds))
     except (TypeError, ValueError):
@@ -466,7 +468,10 @@ def render_approval_panel(
     reasons = str(payload.get("containment_reasons") or "").strip()
     policy_reason = str(payload.get("policy_reason") or "").strip()
     policy_tier = str(payload.get("policy_tier") or "").strip()
-    expiry = _format_approval_expiry(payload.get("expires_in_seconds"))
+    raw_expiry = payload.get("expires_in_seconds")
+    expiry = _format_approval_expiry(
+        raw_expiry if isinstance(raw_expiry, (int, float, str)) else None
+    )
 
     body = Text()
     if _looks_like_diff(preview or ""):

{lisa_cli-0.3.0 → lisa_cli-0.3.1}/src/hosts/cli/transport.py

@@ -337,6 +337,9 @@ class PromptTransport:
         self._tool_error_count: int = 0
         self._fork_children: dict[int, dict[str, Any]] = {}
         self._fork_total: int = 0
+        # Tool call hierarchy: stack of (tool_name, tool_kind, parent_index)
+        # for rendering nested tool calls with indentation
+        self._tool_call_stack: list[tuple[str, str, int]] = []
         # Per-model per-token prices (USD) injected by the command path:
         # ``{model_id: (prompt_price, completion_price)}``. Empty = the
         # footer omits the cost estimate (manifest unreadable/unknown model).
@@ -496,6 +499,7 @@ class PromptTransport:
         self._status_reasoning_chars = 0
         self._fork_children = {}
         self._fork_total = 0
+        self._tool_call_stack = []
         self._current_mode = ""
         self._reset_model_status()
         self._active_label = ""
@@ -733,7 +737,7 @@ class PromptTransport:
         fails fast on stderr instead of printing a panel the operator can
         never answer.
         """
-        from hosts.cli.render import render_approval_panel  # noqa: PLC0415
+        from hosts.cli.render import render_approval_panel
 
         self.pending_approval = approval
         self.approval_unresolved = True
@@ -774,7 +778,11 @@ class PromptTransport:
         if exc is not None:
             body = f"{type(exc).__name__}: {exc}\n\n{message}"
         logger.error("transport error: %s", body)
-        self.console.print(Panel(body, title="Error", border_style=STYLE_BORDER_ERROR))
+        # Escape Rich markup in the error body to avoid MarkupError
+        # when the message contains paths like /home/adam/Code/lisa
+        from rich.markup import escape
+
+        self.console.print(Panel(escape(body), title="Error", border_style=STYLE_BORDER_ERROR))
 
     # ── Internals ───────────────────────────────────────────────────
 
@@ -861,7 +869,15 @@ class PromptTransport:
         """
         if label == "tool_dispatch":
             self._tool_call_count += 1
-            self._active_label = self._render_tool_dispatch(data).plain
+            # Track tool call hierarchy for nested visualization
+            parent_idx = len(self._tool_call_stack) - 1 if self._tool_call_stack else -1
+            tool_name = str(data.get("tool") or "")
+            tool_kind = str(data.get("tool_kind") or "")
+            self._tool_call_stack.append((tool_name, tool_kind, parent_idx))
+            # Update activity label with hierarchy depth indicator
+            depth = len(self._tool_call_stack) - 1
+            depth_indicator = "  " * depth + "└─ " if depth > 0 else ""
+            self._active_label = f"{depth_indicator}{self._render_tool_dispatch(data).plain}"
             self._last_tool_data = data
             self._last_tool_streaming = bool(data.get("stream_output"))
             self._last_tool_output_lines = 0
@@ -900,6 +916,9 @@ class PromptTransport:
             self._tool_ok_count += 1
         elif status == "error":
             self._tool_error_count += 1
+        # Pop from tool call stack for hierarchy tracking
+        if self._tool_call_stack:
+            self._tool_call_stack.pop()
         # The in-flight tool-output box closes with the row: a succeeded tool
         # discards its window (the ``⎿ N lines`` summary reports the shape),
         # a failed one flushes the tail to scrollback so the operator still
@@ -999,7 +1018,7 @@ class PromptTransport:
         if not isinstance(arguments, dict) or not arguments:
             return
         try:
-            import json  # noqa: PLC0415
+            import json
 
             rendered = json.dumps(arguments, indent=2, sort_keys=True, default=str)
         except (TypeError, ValueError):
@@ -1374,7 +1393,9 @@ class PromptTransport:
         if age < _STALL_AFTER_S:
             return
         style = STYLE_ROW_ERROR if age >= _STALL_ALERT_S else STYLE_ROW_TRANSITION
-        parts.append(Text(f"no data {age:.0f}s", style=style))
+        # Visual stall indicator: pulsing block + age
+        stall_glyph = "▮" if int(age) % 2 == 0 else "▯"
+        parts.append(Text(f"{stall_glyph} stalled {age:.0f}s", style=style))
 
     def _activity_spinner(self) -> str:
         """Pick the spinner for the current phase.
@@ -1658,25 +1679,9 @@ class PromptTransport:
 
         out = Text()
         if kind == "transition":
-            target = ""
-            if data.get("handoff_to"):
-                target = f"→ {data['handoff_to']}"
-            elif data.get("transition") == "fork":
-                n = data.get("fork_children") or 0
-                target = f"fork ({n} children)"
-            elif data.get("transitioned_to"):
-                target = f"→ {data['transitioned_to']}"
-            out.append("stage: tool_dispatch ", style=STYLE_TEXT_MUTED)
-            out.append(glyph, style=STYLE_ROW_TRANSITION)
-            out.append(f" {tool}", style=STYLE_TOOL_NAME)
-            if target:
-                out.append(f" {target}", style=STYLE_ROW_TRANSITION)
+            self._render_transition_dispatch(out, data, glyph=glyph, tool=tool, complete=complete)
         else:
-            out.append(glyph, style=STYLE_ROW_TOOL)
-            out.append(f" {tool}", style=STYLE_TOOL_NAME)
-            arg = self._primary_arg_preview(data)
-            if arg:
-                out.append(f" {arg}", style=STYLE_ARG_PREVIEW)
+            self._render_plain_dispatch(out, data, glyph=glyph, tool=tool)
 
         if status_glyph:
             status_style = STYLE_ROW_ERROR if status_glyph == "✗" else STYLE_ROW_SUCCESS
@@ -1690,6 +1695,52 @@ class PromptTransport:
                 out.append(summary, style=STYLE_TEXT_MUTED)
         return out
 
+    def _render_transition_dispatch(
+        self,
+        out: Text,
+        data: dict[str, Any],
+        *,
+        glyph: str,
+        tool: str,
+        complete: dict[str, Any] | None = None,
+    ) -> None:
+        """Render the transition-stage dispatch line (handoff/fork marker).
+
+        The transition target rides the *completion* event (the kernel
+        knows where it routed only when the transition landed), so read
+        it from ``complete`` falling back to the dispatch payload — the
+        same pattern the duration read uses.
+        """
+        data = {**data, **(complete or {})}
+        target = ""
+        if data.get("handoff_to"):
+            target = f"→ {data['handoff_to']}"
+        elif data.get("transition") == "fork":
+            n = data.get("fork_children") or 0
+            target = f"fork ({n} children)"
+        elif data.get("transitioned_to"):
+            target = f"→ {data['transitioned_to']}"
+        out.append("stage: tool_dispatch ", style=STYLE_TEXT_MUTED)
+        out.append(glyph, style=STYLE_ROW_TRANSITION)
+        out.append(f" {tool}", style=STYLE_TOOL_NAME)
+        if target:
+            out.append(f" {target}", style=STYLE_ROW_TRANSITION)
+
+    def _render_plain_dispatch(
+        self, out: Text, data: dict[str, Any], *, glyph: str, tool: str
+    ) -> None:
+        """Render the plain dispatch line with tool-hierarchy indentation."""
+        out.append(glyph, style=STYLE_ROW_TOOL)
+        # Add hierarchy indentation for nested tool calls
+        depth = len(self._tool_call_stack) - 1
+        if depth > 0:
+            indent = "  " * depth + "└─ "
+            out.append(indent, style=STYLE_TEXT_MUTED)
+        out.append(f" {tool}", style=STYLE_TOOL_NAME)
+        arg = self._primary_arg_preview(data)
+        if arg:
+            out.append(f" {arg}", style=STYLE_ARG_PREVIEW)
+
     def _append_aligned_status(self, row: Text, tail: str, style: str, *, suffix: str = "") -> None:
         """Append ``tail`` right-aligned in a fixed status column (tty only).
 

{lisa_cli-0.3.0 → lisa_cli-0.3.1/src/lisa_cli.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: lisa-cli
-Version: 0.3.0
+Version: 0.3.1
 Summary: Terminal CLI for the Lisa Harness
 License-Expression: MIT
 Requires-Python: >=3.11

{lisa_cli-0.3.0 → lisa_cli-0.3.1}/src/lisa_cli.egg-info/SOURCES.txt

@@ -14,6 +14,10 @@ src/lisa_cli.egg-info/entry_points.txt
 src/lisa_cli.egg-info/requires.txt
 src/lisa_cli.egg-info/top_level.txt
 tests/test_approval_panel.py
+tests/test_changed_coverage.py
+tests/test_check_coverage_gate.py
+tests/test_check_docs.py
+tests/test_direct_io_policy.py
 tests/test_group_resolution.py
 tests/test_last_command.py
 tests/test_models_command.py
@@ -31,4 +35,5 @@ tests/test_runtime_teardown.py
 tests/test_streaming.py
 tests/test_transport_stage.py
 tests/test_update_command.py
-tests/test_verify_command.py
+tests/test_verify_command.py
+tests/test_version_output.py

{lisa_cli-0.3.0 → lisa_cli-0.3.1}/tests/test_approval_panel.py

@@ -56,7 +56,7 @@ def test_render_approval_panel_includes_command_and_hints() -> None:
     assert "run a shell command" in text
     assert "ls -la" in text
     assert "approve" in text and "deny" in text
-    assert 'lisa approve abc123def456' in text
+    assert "lisa approve abc123def456" in text
     assert "execute action requires approval" in text
     assert "expires in 1m 30s" in text
     # Containment badge is shown and styled as contained.
@@ -175,4 +175,3 @@ async def test_start_thinking_disarms_pending_approval() -> None:
     transport.pending_approval = _payload()
     transport.start_thinking()
     assert transport.pending_approval is None
-