Skip to content

Board: plain-words wool letters until numbers unlock - #15

Merged
deadp merged 1 commit into
complete-gamefrom
queue/q369
Oct 22, 2026
Merged

deadp merged 1 commit into
complete-gamefrom
queue/q369

Conversation

@deadp

@deadp deadp commented Oct 22, 2026

Copy link
Copy Markdown
Owner

Wool-order letters showed 'finer than N µm' before the numbers unlock. New util helper letterText swaps in 'fine fleece' until then, applied in board.ts and report.ts; post-unlock text unchanged.
ui.test.ts is untouched and passes (45/45), as does all of src/ui; tsc --noEmit is clean.
The full-package vitest run timed out here, so the complete npm test was not confirmed.

Queued from the dashboard (item 369). The request:

TIERED shape=B builder=lead review=glm reason="One known failing UI test with a clear done-when on an approved repo; tests exist and CI now gates it; Joshua asked for cheap. Escalates to Sonnet by itself if it cannot fix it"
Goal: make npm test fully green by fixing the board panel, not the test. The failing test is packages/game/src/ui/ui.test.ts, "every panel renders in every fixture", fixture "midAct1/board: µm before numbers unlock". It fails because the board panel shows a µm (fineness) value before the player has unlocked "numbers". Rule to keep: before the numbers unlock, no panel text may show a µm value; after it, the board shows it as today.

Area: the board panel in packages/game/src/ui/ (find it from the test and the fixture name; likely board.ts or the panel the fixture renders) and where it reads the numbers-unlock state. Find which text leaks the µm value, gate it on the same unlock flag other panels use for numbers, and show the plain-words version (a word, not a number) before it, as other panels do before the unlock.

Done when: npm ci && npx tsc -b && npm test all pass (CI runs exactly these three), the test file is unchanged (do not edit, skip or loosen ui.test.ts), and no other panel's output changes. Add nothing else.

Do not touch: ui.test.ts, game values (core/config.ts), save format, world3d/, scripts/, workflows, dependency files, AGENTS.md/CLAUDE.md. Finish with a one-line RESULT summary.

Opened by the queue runner on LXC 113. Tiered build: never auto-merged. Tachikoma checks the diff, CI and these notes, then merges (ask mode).

🤖 Generated with Claude Code

@deadp
deadp merged commit 9745e15 into complete-game Oct 22, 2026
2 checks passed
@deadp
deadp deleted the queue/q369 branch October 22, 2026 06:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant