Best score is stored in two places that can disagree
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 74/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- typescript
- Domaine
- backend
Piste de recherche
Start with shared/storage.ts and trace how getHighScore reads saved values; then compare that with shared/achievements.ts and the submitHighScore calls in shared/scenes.ts. Preserve existing best scores while making one value authoritative, and verify with npm run typecheck and npm run build.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Summary
The game-over scene writes the best score twice: under <slug>:highscore and as games[slug].best inside the trophies save. Hub cards and menus read the first; the Trophies panel reads the second; resetting trophies clears only the second.
Where
shared/storage.ts:21-30shared/achievements.ts:135-141,shared/achievements.ts:164-167shared/scenes.ts:92-93,src/hub/main.ts:58
Steps to reproduce
- Play a game, then reset trophies on the hub.
- The card still shows the old best while the Trophies panel shows none.
Suggested fix
Make getHighScore read the stats value with a one-time fallback to the legacy key, have recordPlay return whether the score is a record, then remove submitHighScore.
Acceptance criteria
- One stored best score per game; existing bests survive the change
Getting started
- Relevant files:
shared/storage.ts,shared/achievements.ts,shared/scenes.ts - Run locally:
npm install, thennpm run dev - Checks:
npm run typecheckandnpm run build(there is no test suite yet) - Related docs:
README.md
Filed by repo-audit/full-audit at b001f5d. Severity: low. Effort: S. audit-key: tech-debt|shared/storage.ts|submitHighScore
- Langage dominant
- TypeScript
- Étoiles
- 3
- Forks
- 3
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de TMHSDigital/GH-Arcade
-
enhancement good first issue severity: low
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
TMHSDigital/GH-Arcade#50 ·
-
enhancement good first issue severity: medium
Difficulté 2/5 1-3 heures Accessibilité débutants 86/100
TMHSDigital/GH-Arcade#47 ·
-
documentation good first issue severity: low
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
TMHSDigital/GH-Arcade#34 ·
-
documentation good first issue severity: low
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
TMHSDigital/GH-Arcade#33 ·
-
dependencies good first issue severity: low
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
TMHSDigital/GH-Arcade#27 ·
Toutes les issues de TMHSDigital/GH-Arcade
Issues similaires
-
automated-triage type/bug type/ci
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour
-
📦 expo-router Android contributor: external needs review sdk-58 severity: medium
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
expo/expo#53436 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Translate the READMEOuvertedocumentation good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
asasemahmed/openboard#68 ·
Les mainteneurs répondent en général sous 1 jour
-
bug good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
-
Document PowerShell shell semantics and read-only Bash deduplication in Built-in Tools referenceOuvertedocumentation
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100