workspace: record _fullscreen_page_size() lambda-inline refactor

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

Author: 2 people

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

@@ -1,9 +1,20 @@
 # `_console_emitter.py` coverage
 
-Valid at: 52e0272
+Valid at: 59b0e34
 
 ## Recent changes
 
+- 59b0e34 — inlined `self._fullscreen_page_size()` into the space/b
+  action lambdas in `_handle_fullscreen_key`.  The `page` local was
+  computed unconditionally in the non-exit branch but only consumed by
+  the page-down (" ") and page-up ("b") lambdas — j/k/g/G/[/] now skip
+  the call entirely.  Space/b still compute it exactly once per keypress,
+  now at action-invocation time (under the same `_console_lock`), so
+  behavior is unchanged.  `_fullscreen_page_size()` is a pure read of
+  `self._console.size.height` in a try/except, so deferring evaluation
+  has no observable effect — the dict build and action invocation happen
+  back-to-back inside the lock.  Same Phase 4 shape as 134078d / ef176bf
+  / b19625e.
 - 52e0272 — inlined the `msg = raw.get("message", {})` local in
   `_IterationPanel._apply_assistant`.  The alias was read exactly once on
   the next line as `msg.get("usage")`.  Collapsing to

workspace/ralphs/improve-codebase/iterations.md

@@ -2,6 +2,8 @@
 
 One line per iteration: `<sha> <summary>`.
 
+59b0e34 refactor: inline `_fullscreen_page_size()` into the space/b lambdas in `_handle_fullscreen_key` — the `page` local was computed unconditionally in the non-exit branch but only consumed by the page-down (" ") and page-up ("b") action lambdas.  j/k/g/G/[/] now skip the call entirely; space/b still compute it exactly once per keypress, now at action-invocation time under the same lock.  Same Phase 4 "narrow the scope of a one-branch local" shape as 134078d / ef176bf / b19625e.
+
 52e0272 refactor: inline `msg` alias in `_apply_assistant` — the `msg = raw.get("message", {})` local was read exactly once on the next line as `msg.get("usage")`.  Collapsing into `raw.get("message", {}).get("usage")` matches the chained-get style already used by `_iter_content_blocks` (which independently does `raw.get("message", {}).get("content", [])`) and the inline-alias pattern from 497c028 / fc5e1cb.
 
 b19625e refactor: drop `new_offset` alias in `_FullscreenPeek.scroll_down` — the local was assigned to `self._offset` and then re-read only as `new_offset == 0`, which is identical to `self._offset == 0` after the assignment.  Sibling `scroll_up` needs its local because it compares old vs new before the assignment; `scroll_down` has no such comparison, so the alias was dead.  Same shape as ef176bf and 134078d.