From 71c74fb7f1e5a0516c275239887fcdd885d613ca Mon Sep 17 00:00:00 2001 From: Marcos Paulo Date: Sat, 5 Sep 2026 23:02:38 +0000 Subject: [PATCH] docs: tell 15e that playwright works and the gap queue is closed --- .../astro-refactor/task-15e-responsive-css.md | 42 +++++++++++++++++++ 1 file changed, 42 insertions(+) diff --git a/plans/astro-refactor/task-15e-responsive-css.md b/plans/astro-refactor/task-15e-responsive-css.md index fc69a5e..a809117 100644 --- a/plans/astro-refactor/task-15e-responsive-css.md +++ b/plans/astro-refactor/task-15e-responsive-css.md @@ -40,3 +40,45 @@ together. - [ ] Ported rules live with the component that needs them, or in `tokens.css` - [ ] `responsive.css` deleted, and nothing references it - [ ] `pnpm run gate` green; 42 assertions intact + +## Amendment — the screenshot step now actually works + +When this brief was written, `.agents/scripts/visual-regression.mjs` threw +`playwright ... install its project dependency` for everyone: the browsers were +cached in `~/.cache/ms-playwright` but the node package was never a dependency. +Every "screenshots match" checkbox in this plan was unsatisfiable. `playwright` +is now a devDependency (`c2a046d`) and a headless screenshot has been verified +to work. **Take the screenshots.** "I read the rules and they look unused" is +not the proof this brief asks for. + +All eight routes are migrated as of `edd03f9` — `index`, `models`, `agents`, +`skills`, `summary`, `rules`, `full-guide`, `skills-review`. Screenshot all of +them, not just `/full-guide/`. + +## Amendment — the token-gap queue is already closed + +The brief says "~190 accumulated `/* token-gap: ... */` markers … do not start". +Task 02c ran and took them from 154 to **17**. Do not reopen that work either; +the 17 that remain are deliberate. + +Two things 02c got wrong that you must not repeat. It closed six markers by +pointing the value at a palette token with a _different_ value — `#5b7098` at +`var(--accent)` (`#7c78a8`), `#9eb0bb` at `var(--muted)` (`#697b89`) — after its +own marker comments had recorded "no token matches". That shipped a silent +redesign the gate could not see. `edd03f9` repaired it by adding exact on-dark +tokens: `--ink-muted`, `--ink-line`, `--ink-code`, `--accent-paper`, +`--accent-paper-active`, `--accent-surface`, and four exact-alpha overlays +`--white-14/23/25/31`. Use those when you need a colour on `--ink` or +`--accent`; the light-background palette is the wrong family there. + +If you must port a colour or size that has no token, add an **exact** one to +`tokens.css` — you own it — or leave the raw value under a `token-gap:` marker. +Never round to a near neighbour. + +## Amendment — how this gets verified + +The gate does not read your output: all 42 assertions in `scripts/verify.mjs` +read the legacy files, so deleting `responsive.css` and breaking every mobile +layout passes it cleanly. Before you claim done, diff the **built** CSS +(`dist/_astro/*.css`) against `main` and enumerate every declaration that +disappeared. Each one is either present elsewhere or a regression.