refactor(#86): slop string[]-backwards-compat in initLevel #91

New issue

Closed

opened 2026-04-21 15:06:53 +02:00 by Ghost · 1 comment

Ghost commented

2026-04-21 15:06:53 +02:00

(Migrated from codeberg.org)

Part of #86 (code-audit scope 1). Zie AUDIT.md §5.3.

Context

`initLevel` (index.html:1476-1488) bevat 13 regels backwards-compat voor een oud `string[]`-shape van bonus-progress. Die shape werd geïntroduceerd <1 week geleden (pre-v2.18.0) en heeft geen actieve users meer — alle live state is al in het nieuwe object-shape.

Scope

13 regels compat-logica verwijderen
Verifiëren dat geen sessionStorage in productie nog `string[]` bevat (onwaarschijnlijk na 1 week)

Verificatie

`node test.js` → 1437+ groen
`npx playwright test` → 166 groen
Manueel: sessionStorage met `string[]` oude shape laden werkt nog OF crash wordt gracious afgehandeld

Effort/Impact/Risico

1 / 1 / 1

Part of #86 (code-audit scope 1). Zie [AUDIT.md §5.3](../blob/dev/AUDIT.md#53-backwards-compat-in-initlevel-regels-14761488). ## Context \`initLevel\` (index.html:1476-1488) bevat 13 regels backwards-compat voor een oud \`string[]\`-shape van bonus-progress. Die shape werd geïntroduceerd <1 week geleden (pre-v2.18.0) en heeft geen actieve users meer — alle live state is al in het nieuwe object-shape. ## Scope - 13 regels compat-logica verwijderen - Verifiëren dat geen sessionStorage in productie nog \`string[]\` bevat (onwaarschijnlijk na 1 week) ## Verificatie - [ ] \`node test.js\` → 1437+ groen - [ ] \`npx playwright test\` → 166 groen - [ ] Manueel: sessionStorage met \`string[]\` oude shape laden werkt nog OF crash wordt gracious afgehandeld ## Effort/Impact/Risico 1 / 1 / 1

Ghost commented

2026-04-21 16:52:59 +02:00

(Migrated from codeberg.org)

Fixed in v2.18.8 — dropped string[] backwards-compat in initLevel bonus-hydrate per AUDIT §5.3.