From 13ae16051d8002fdfec04d060d2b6ee075a573e1 Mon Sep 17 00:00:00 2001 From: funman300 Date: Thu, 7 May 2026 22:07:36 -0700 Subject: [PATCH] =?UTF-8?q?docs(handoff):=20cross-link=20skipped=20items?= =?UTF-8?q?=20+=20flag=20the=20=E2=96=8C=20replay.tsx=20deviation?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Three small clarity touch-ups to SESSION_HANDOFF.md so a future-session reader doesn't have to reconstruct intent from git log alone: - The c84d9f4 narrative listed "header text treatment" as still open; 6204db8 closed it the same session. Added a parenthetical pointer. - The cacb19c "Skipped" sub-section now cross-links each item to its follow-up status in the punch list (scanline + cursor pulse → still open; "RUSTY SOLITAIRE" wordmark → closed, the in-engine wordmark stays "Solitaire Quest"). - 6204db8 adopted "▌ replay" instead of the mockup's literal "▌replay.tsx" — the .tsx was a Stitch/React prototyping leak. Documented the deviation alongside the existing RUSTY SOLITAIRE precedent so the in-engine string isn't second-guessed later. Pure docs; no code changes, no test impact. --- SESSION_HANDOFF.md | 21 +++++++++++++++++---- 1 file changed, 17 insertions(+), 4 deletions(-) diff --git a/SESSION_HANDOFF.md b/SESSION_HANDOFF.md index 3802c68..4f83946 100644 --- a/SESSION_HANDOFF.md +++ b/SESSION_HANDOFF.md @@ -88,10 +88,12 @@ adding more is one component-attach, not three new query types. **Skipped, with rationale captured in the commit:** - Scanline overlay (needs a tiled-pattern asset or custom shader). + *Open in "Visual-identity follow-ups" below.* - Pulsing cursor on the "ready_" line (would fight the global - fade timeline). + fade timeline). *Open in "Visual-identity follow-ups" below.* - "RUSTY SOLITAIRE" wordmark from the mockup (the actual product - is "Solitaire Quest"; the mockup leaked the repo name). + is "Solitaire Quest"; the mockup leaked the repo name). *Closed + — the in-engine wordmark stays "Solitaire Quest".* ### `c84d9f4` `feat(engine): scrub fill bar + per-frame updater for replay overlay` @@ -110,8 +112,9 @@ asserting `Node.width` on the unique scrub-fill entity. Same change-detection guard as the text updaters, so an idle replay leaves the node untouched. -Header text treatment, move-log scroll, MOVE chip, and WIN MOVE -callout from the same mockup are still open — separate commits. +Header text treatment (closed by `6204db8` immediately below), +move-log scroll, MOVE chip, and WIN MOVE callout from the same +mockup are still open — separate commits. ### `6204db8` `feat(engine): port replay banner label to ▌ cursor-block treatment` @@ -124,6 +127,16 @@ resemblance between the two top-level overlay surfaces. Pure text-content change; no behavioural shift, no new components, no new systems. +**Mockup deviation (intentional):** the source mockup string in +`docs/ui-mockups/replay-overlay-mobile.html` is `▌replay.tsx`. The +`.tsx` is a prototyping leak — Stitch renders in React, so the +mockup author reached for a familiar filename — and was dropped +for the in-engine version since the codebase is Rust. The `▌` + +lowercase pattern is what reads as a Terminal-output-line; the +extension is incidental. (Same shape as the "RUSTY SOLITAIRE" +wordmark deviation noted under `cacb19c` — the mockup leaked the +repo name; the actual product is "Solitaire Quest".) + ## What shipped in v0.20.0 (frozen at `41a009a`) ### Terminal visual-identity port