Unexpected saved data can stop the hub from rendering

Offen
  • Stimmt etwas nicht?
#16 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Anfängerfreundlichkeit
72/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
typescript, vite
Bereich
frontend

Rechercherichtung

Start with shared/storage.ts and trace how settings and achievements are loaded in shared/settings.ts and shared/achievements.ts; then inspect panel setup in src/hub/main.ts and src/hub/settings-panel.ts. Reproduce the issue with the provided localStorage value, then run npm run typecheck and npm run build. Done when malformed or partial saved values use defaults without throwing and the game grid renders despite a panel initialization failure.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug severity: low

Summary

Saved data is cast to its expected type without any shape check or version field. A value from an older or newer schema, or one edited by hand, can throw during hub start-up and leave an empty game grid with no message.

Where

  • shared/storage.ts:7 (JSON.parse(raw) as T)
  • shared/settings.ts:59, shared/achievements.ts:91 (spread over defaults)
  • src/hub/settings-panel.ts:54 (PALETTES[s.palette] with no fallback; shared/ui.ts:9 does guard this)
  • src/hub/main.ts:110-111 (panels are wired before the grid renders)

Steps to reproduce

  1. In devtools: localStorage.setItem('gh-arcade:settings', '{"palette":"x"}').
  2. Reload the hub: no game cards appear.

Suggested fix

Add per-key sanitisers that coerce and default each field (numbers via Number.isFinite, known palette names, object-typed sections), add a version field, and render the game grid before wiring the panels so one bad panel cannot hide the games.

Acceptance criteria

  • Malformed or partial saved values fall back to defaults without throwing
  • The game grid renders even if a panel fails to initialise

Getting started

  • Relevant files: shared/storage.ts, shared/settings.ts, shared/achievements.ts, src/hub/main.ts, src/hub/settings-panel.ts
  • Run locally: npm install, then npm run dev
  • Checks: npm run typecheck and npm run build (there is no test suite yet)
  • Related docs: README.md

Related: #1

Filed by repo-audit/full-audit at b001f5d. Severity: low. Effort: S. audit-key: quality|shared/storage.ts|loadJSON

Vorherrschende Sprache
TypeScript
Sterne
3
Forks
3
Keine gemergten PRs in 30 T.

Entwicklungsumgebung

Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus TMHSDigital/GH-Arcade

Alle Issues in TMHSDigital/GH-Arcade

Ähnliche Issues

Weitere Issues zu TypeScript

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.