workspace: record visible alias inline in _build_body

Co-authored-by: Ralphify <noreply@ralphify.co>

Author: 2 people

workspace/ralphs/improve-codebase/coverage/_console_emitter.md

@@ -1,9 +1,21 @@
 # `_console_emitter.py` coverage
 
-Valid at: 3823019
+Valid at: 2fda4f0
 
 ## Recent changes
 
+- 2fda4f0 — inlined the `visible = self._scroll_lines[-_MAX_VISIBLE_SCROLL:]`
+  local in `_LivePanelBase._build_body`.  The alias was read exactly
+  once, as the iterable of the very next `for line in visible:` loop.
+  Collapsing to `for line in self._scroll_lines[-_MAX_VISIBLE_SCROLL:]:`
+  matches the inline-alias pattern from 497c028 (`agent`), fc5e1cb
+  (`total_in`), 52e0272 (`msg`), ce487d3 (`text`), and e1ad87a
+  (`binary`).  Behavior unchanged — each iteration still mutates the
+  Text in place (`no_wrap` / `overflow`) before appending to `rows`,
+  and the slice still materializes the last `_MAX_VISIBLE_SCROLL`
+  items.  No other `visible` locals remain in the class; the name is
+  reused elsewhere in the module (fullscreen viewport height, scrollbar
+  geometry) but all in unrelated scopes.
 - 3823019 — narrowed `line = escape_markup(data["line"])` scope in
   `_on_agent_output_line` past the
   `if not isinstance(target, _IterationSpinner): return` guard.  The

workspace/ralphs/improve-codebase/iterations.md

@@ -2,6 +2,8 @@
 
 One line per iteration: `<sha> <summary>`.
 
+2fda4f0 refactor: inline `visible` alias in `_LivePanelBase._build_body` — the `visible = self._scroll_lines[-_MAX_VISIBLE_SCROLL:]` local was read exactly once, as the iterable of the very next `for line in visible:` loop.  Collapsing to `for line in self._scroll_lines[-_MAX_VISIBLE_SCROLL:]:` matches the inline-alias pattern from 497c028 / fc5e1cb / 52e0272 / ce487d3 / e1ad87a.  Behavior unchanged — the slice still materializes the last `_MAX_VISIBLE_SCROLL` items, and each loop iteration still mutates the Text in-place (`no_wrap` / `overflow`) before appending to `rows`.  Pinned by the broad `test_console_emitter.py` suite (peek-visible rendering paths).
+
 e1ad87a refactor: inline `binary` alias in `_supports_stream_json` — the local was assigned to `Path(cmd[0]).stem` and then read exactly once on the next line as `binary == CLAUDE_BINARY`.  Collapsing to the chained form `Path(cmd[0]).stem == CLAUDE_BINARY` matches the already-inline sibling check in `_console_emitter.py:_is_claude_command` (`return Path(parts[0]).stem == CLAUDE_BINARY`) and the inline-alias pattern from ce487d3 / 52e0272 / 497c028 / fc5e1cb.  Same behavior — `_supports_stream_json` still short-circuits on empty `cmd` first, and the final boolean result is unchanged.  Pinned by `tests/test_agent.py::test_streaming_mode_used_for_claude` and the broader agent-selection tests.
 
 c5ce11d refactor: inline `reason = state.status.reason` alias in `run_loop`'s `RUN_STOPPED` emit.  The local was read exactly once as the `reason=` kwarg to `RunStoppedData(...)` on the next line; collapsing to `reason=state.status.reason` matches the inline-alias pattern from ce487d3 (`text`) / 52e0272 (`msg`) / 497c028 (`agent`) / fc5e1cb (`total_in`).  Safe at this call site — the immediately-preceding `if state.status == RunStatus.RUNNING: state.status = RunStatus.COMPLETED` normalizes status to a terminal value (STOPPED / COMPLETED / FAILED), so the `RunStatus.reason` property's "non-terminal raises ValueError" guard cannot fire.