Review deck v2 — the approved page, answers file + submit hand-off, rig-measured boxes, rig fixes by itsdestin · Pull Request #1 · itsdestin/youcoded-dev

docs(spec): review deck v2 — the approved page (mockup G) plus the answers server, rig-measured highlights and writing rules

itsdestin

and

claude

committed

2d8cb44

docs(plan): review deck v2 — 15 tasks from spec loader to the rebuilt Phase C deck

itsdestin

and

claude

committed

b9f1e96

docs(plan): review deck v2 — review round folded in: worktree-root lookup, 3×3 dilate, rule-based layout test, coverage orders by run id, no crop command, wait fallback, Destin checkpoint before th…

itsdestin

and

claude

committed

d6775dc

feat(ui-review): deck v2 spec loader and the writing rules the builder enforces

itsdestin

and

claude

committed

10418ef

feat(ui-review): highlight-box maths — measured rect to crop %, pixel-diff bounding box

itsdestin

and

claude

committed

5f23796

feat(ui-review): built-in theme tokens for the deck, pinned to globals.css

itsdestin

and

claude

committed

f0ada65

test(ui-review): synthetic run fixture for the deck tests

itsdestin

and

claude

committed

696e94b

feat(ui-review): shot.mjs measures named elements for the deck and stamps the run id

itsdestin

and

claude

committed

94a87dd

fix(ui-review): coverage orders manifests by run id before file time — a late-finishing old shard cannot resurrect a MISSED row

itsdestin

and

claude

committed

fb77a5b

fix(ui-review): run-review probes CDP ports, stamps a run id, rebuilds only this sweep's sheets

itsdestin

and

claude

committed

66031b8

fix(ui-review): word_count keeps a curly apostrophe inside the word (lost in transcription; a pasted ’ over-counted headlines)

itsdestin

and

claude

committed

ffd8d7d

feat(ui-review): deck server — answers file on every click, submit ends the process with the summary; wait reads the file alone

itsdestin

and

claude

committed

380840d

docs(ui-review): probe-ports comment names the real fallback (/dev/tcp, not nc)

itsdestin

and

claude

committed

7dc9488

feat(ui-review): crop step — cut each crop once, resolve highlight boxes from measurements or pixel diff

itsdestin

and

claude

committed

65c93ec

fix(ui-review): diff_bbox tells "everything changed" from "nothing changed" — a 1 px border gives trim something to shrink from

itsdestin

and

claude

committed

76b4d17

fix(ui-review): serve — a live lock owned by someone else still refuses; a non-JSON POST gets a 400; WHY on shutdown-from-a-thread

itsdestin

and

claude

committed

b0f1370

feat(ui-review): deck v2 page (approved mockup G) and the builder that inlines it

itsdestin

and

claude

committed

7238037

feat(ui-review): review-cards.py v2 CLI — build, serve, wait

itsdestin

and

claude

committed

b6036dc

fix(ui-review): deck page — theme tokens outrank the page defaults (Halftone radii now apply), ready only after a real layout, first paint in the first theme, no double-counted seconds, nothing mov…

itsdestin

and

claude

committed

7f56ca3

test(ui-review): headless render check — layout follows the spec rule at three sizes, an answer round-trips to the file

itsdestin

and

claude

committed

f801058

docs(plan): review deck v2 — reference code matches what shipped (bordered trim, serve fixes, token specificity, 400 px compact case)

itsdestin

and

claude

committed

500010c

test(ui-review): the first-paint theme test now reorders the themes, so a hardcoded midnight would fail it

itsdestin

and

claude

committed

3db448b

docs(ui-review): deck v2 flow — build, serve, wait; measure is planned before the Before run; hand-off gaps closed or filed

itsdestin

and

claude

committed

d81214a

docs(plan): node --test takes the file glob, and Node 26 prints "pass N", not "# pass N"

itsdestin

and

claude

committed

c21c284

docs(ui-audit): Phase C review rebuilt as the first v2 deck (10 of 13 points — two are Before-only, one lacks a Meadow Mist capture)

itsdestin

and

claude

committed

b9e825b

docs(plan): Phase C deck is 10 steps; example copy carries the measured numbers

itsdestin

and

claude

committed

f821656

fix(ui-review): final-review fixes — answers saved on every click, a failed Submit says so, run-review never orphans its workbench, lock checked before build, Host/Origin guard, same-size guard in …

itsdestin

and

claude

committed

f3b2027

docs(plan): what shipped after the final review, pointed at the code

itsdestin

and

claude

committed

82991e1

fix(ui-review): re-serving a deck after its submit starts a fresh review — the old answers are kept aside, not loaded as "already submitted"

itsdestin

and

claude

committed

1398954

fix(ui-review): read-only label survives repaints, a dead server flips the dialog to copy/paste, localhost accepted, SIGTERM cleans the lock

itsdestin

and

claude

committed

94e3b75

docs: archive the review-deck v2 spec and plan as shipped; pointers follow them

itsdestin

and

claude

committed

bfbf2c2

Navigation Menu

Review deck v2 — the approved page, answers file + submit hand-off, rig-measured boxes, rig fixes - #1

Review deck v2 — the approved page, answers file + submit hand-off, rig-measured boxes, rig fixes#1
itsdestin merged 31 commits into
masterfrom
feat/review-deck-v2

Commits on Aug 27, 2026

docs(spec): review deck v2 — the approved page (mockup G) plus the answers server, rig-measured highlights and writing rules

docs(plan): review deck v2 — 15 tasks from spec loader to the rebuilt Phase C deck

docs(plan): review deck v2 — review round folded in: worktree-root lookup, 3×3 dilate, rule-based layout test, coverage orders by run id, no crop command, wait fallback, Destin checkpoint before th…

feat(ui-review): deck v2 spec loader and the writing rules the builder enforces

feat(ui-review): highlight-box maths — measured rect to crop %, pixel-diff bounding box

feat(ui-review): built-in theme tokens for the deck, pinned to globals.css

test(ui-review): synthetic run fixture for the deck tests

feat(ui-review): shot.mjs measures named elements for the deck and stamps the run id

fix(ui-review): coverage orders manifests by run id before file time — a late-finishing old shard cannot resurrect a MISSED row

fix(ui-review): run-review probes CDP ports, stamps a run id, rebuilds only this sweep's sheets

fix(ui-review): word_count keeps a curly apostrophe inside the word (lost in transcription; a pasted ’ over-counted headlines)

feat(ui-review): deck server — answers file on every click, submit ends the process with the summary; wait reads the file alone

docs(ui-review): probe-ports comment names the real fallback (/dev/tcp, not nc)

feat(ui-review): crop step — cut each crop once, resolve highlight boxes from measurements or pixel diff

fix(ui-review): diff_bbox tells "everything changed" from "nothing changed" — a 1 px border gives trim something to shrink from

fix(ui-review): serve — a live lock owned by someone else still refuses; a non-JSON POST gets a 400; WHY on shutdown-from-a-thread

feat(ui-review): deck v2 page (approved mockup G) and the builder that inlines it

feat(ui-review): review-cards.py v2 CLI — build, serve, wait

fix(ui-review): deck page — theme tokens outrank the page defaults (Halftone radii now apply), ready only after a real layout, first paint in the first theme, no double-counted seconds, nothing mov…

test(ui-review): headless render check — layout follows the spec rule at three sizes, an answer round-trips to the file

docs(plan): review deck v2 — reference code matches what shipped (bordered trim, serve fixes, token specificity, 400 px compact case)

test(ui-review): the first-paint theme test now reorders the themes, so a hardcoded midnight would fail it

docs(ui-review): deck v2 flow — build, serve, wait; measure is planned before the Before run; hand-off gaps closed or filed

docs(plan): node --test takes the file glob, and Node 26 prints "pass N", not "# pass N"

docs(ui-audit): Phase C review rebuilt as the first v2 deck (10 of 13 points — two are Before-only, one lacks a Meadow Mist capture)

docs(plan): Phase C deck is 10 steps; example copy carries the measured numbers

fix(ui-review): final-review fixes — answers saved on every click, a failed Submit says so, run-review never orphans its workbench, lock checked before build, Host/Origin guard, same-size guard in …

docs(plan): what shipped after the final review, pointed at the code

fix(ui-review): re-serving a deck after its submit starts a fresh review — the old answers are kept aside, not loaded as "already submitted"

fix(ui-review): read-only label survives repaints, a dead server flips the dialog to copy/paste, localhost accepted, SIGTERM cleans the lock

docs: archive the review-deck v2 spec and plan as shipped; pointers follow them