test(ui): align compaction-seam and recap spacing expectations

Update rendering tests to match the committed compaction/recap seam
behavior: compaction commits a leading blank row plus block, and turn
recaps are framed by blank rows.

Author: elkaix

tests/ui_and_conv/test_empty_think_part_indicator.py

@@ -382,8 +382,9 @@ def test_compaction_takes_priority_over_moon():
     # Compaction starts — should show compaction, not moon
     view.dispatch_wire_message(CompactionBegin())
     agent_blocks = view.compose_agent_output()
-    # Should be the compaction block, not the moon
-    assert len(agent_blocks) == 1
+    # Should be the compaction block, not the moon. Compaction commits with a
+    # leading seam (blank row + block), so there are two blocks.
+    assert len(agent_blocks) == 2
     assert view._compaction_block is not None
 
 

tests/ui_and_conv/test_live_view_notifications.py

@@ -2,14 +2,15 @@
 
 from pythinker_core.message import ToolCall
 from pythinker_core.tooling import ToolResult, ToolReturnValue
-from rich.console import Console
+from rich.console import Console, Group
 
 from pythinker_code.tools.display import TodoDisplayBlock, TodoDisplayItem
 from pythinker_code.ui.shell.console import console as shell_console
 from pythinker_code.ui.shell.keyboard import KeyEvent
 from pythinker_code.ui.shell.visualize import _live_view as live_view_module
 from pythinker_code.ui.shell.visualize import _LiveView, _PromptLiveView
 from pythinker_code.wire.types import (
+    CompactionBegin,
     HookOutput,
     HookResolved,
     HookTriggered,
@@ -318,20 +319,32 @@ def test_prompt_live_view_keeps_non_background_task_notifications(monkeypatch):
 
 def test_live_view_prints_turn_recap_when_enabled(monkeypatch):
     printed = []
-    monkeypatch.setattr(
-        live_view_module.console, "print", lambda *args, **_kwargs: printed.extend(args)
-    )
+
+    def fake_print(*args, **_kwargs):
+        printed.append(args[0] if args else None)
+
+    monkeypatch.setattr(live_view_module.console, "print", fake_print)
 
     view = _LiveView(StatusUpdate(), show_turn_recaps=True)
     view.dispatch_wire_message(TurnBegin(user_input="implement recaps"))
-    view.dispatch_wire_message(TextPart(text="Implemented a /recap command."))
+    view.dispatch_wire_message(TextPart(text="Implemented a "))
+    view.dispatch_wire_message(TextPart(text="/recap command."))
     view.dispatch_wire_message(TurnEnd())
     view.cleanup(is_interrupt=False)
 
-    plain = "\n".join(getattr(item, "plain", str(item)) for item in printed)
+    plain = "\n".join(getattr(item, "plain", str(item)) for item in printed if item is not None)
     assert "※ recap: Implemented a /recap command." in plain
+    assert "Implemented a  /recap" not in plain
     assert "disable recaps in /settings" in plain
 
+    recap_index = next(
+        index
+        for index, item in enumerate(printed)
+        if item is not None and "※ recap:" in getattr(item, "plain", str(item))
+    )
+    assert printed[recap_index - 1] is None
+    assert printed[recap_index + 1] is None
+
 
 def test_cleanup_flushes_notifications_to_terminal_history(monkeypatch):
     view = _LiveView(StatusUpdate())
@@ -371,6 +384,15 @@ def test_cleanup_flushes_all_notifications_even_when_live_view_shows_only_latest
         assert f"Background task completed: build project {index}" in rendered
 
 
+def test_compaction_status_keeps_one_blank_row_above_live_block():
+    view = _LiveView(StatusUpdate(context_tokens=221_300))
+
+    view.dispatch_wire_message(CompactionBegin())
+
+    rendered = _render(Group(*view.compose_agent_output(include_working_indicator=False)))
+    assert rendered.startswith("\n· Compacting conversation…")
+
+
 def test_compose_inserts_gap_under_agent_output_before_nonempty_status():
     """Non-interactive compose() puts one blank row under the spinner verb
     before a non-empty status line (the under-gap), and the status line stays last."""

tests/ui_and_conv/test_plan_display_panel.py

@@ -24,14 +24,17 @@ def fake_render_worklog_card(title, body, *, subtitle=None, border_style="grey50
         return panel
 
     monkeypatch.setattr(_live_view, "render_worklog_card", fake_render_worklog_card)
-    monkeypatch.setattr(_live_view.console, "print", printed.append)
+    monkeypatch.setattr(_live_view.console, "print", lambda *args, **kwargs: printed.extend(args))
     view = _live_view._LiveView(StatusUpdate())
 
     view.display_plan(PlanDisplay(content="# Steps\n\n- Step one", file_path="plans/one.md"))
 
     assert card_call["title"] == "Plan"
     assert card_call["subtitle"] == "plans/one.md"
     assert card_call["border_style"] == tui_rich_style("border")
+    # _print_action_block emits a leading blank line (zero-arg console.print())
+    # before the panel; only positional args are captured, so printed holds the
+    # panel alone.
     assert printed == [card_call["panel"]]
 
     console = Console(record=True, width=120, color_system=None)