Extract a shared endRun helper for the game-over handoff

Ouverte
  • Quelque chose ne va pas ?
#39 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
4/5
Temps estimé
3-5 jours
Accessibilité débutants
57/100
Type d'issue
Refactorisation
Clarté
Clairement spécifiée
Activité
Active
Stack technique
typescript
Domaine
game-dev

Piste de recherche

Start with shared/scenes.ts and shared/pause.ts, then inspect the listed GameScene.ts examples and locate the other game-over call sites. Trace how each game guards the end of a run and builds restart data; migrate the ten games to the typed helper and run npm run typecheck and npm run build. Done when all ten use the helper and both checks pass.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

severity: low tech-debt

Summary

Every game repeats the same end-of-run code: set a private "over" flag, wait, then scene.start('GameOver', { gameId, score, ... }) with an unchecked object literal.

Where

  • shared/scenes.ts:72-79 (GameOverData is exported but imported by no game)
  • For example games/astro/scenes/GameScene.ts:486-490, games/breakout/scenes/GameScene.ts:432-441, games/siege/scenes/GameScene.ts:541-547, games/stack/scenes/GameScene.ts:447-453

Details

  • The payload is untyped at all ten call sites, so a misspelled field compiles.
  • The "over" flag has five spellings (over, levelDone, alive, phase === 'over', state === 'over'), and only Siege guards against a double call.
  • Restart data is stated twice per game, once for the pause menu and once for game over.
  • Scene keys are bare strings in the shared modules and every scene constructor.

Suggested fix

  1. Add endRun(scene, data: GameOverData, delayMs) to shared/scenes.ts with a once-only guard, and export scene-key constants.
  2. Migrate one game per commit.
  3. Let PauseController and endRun read restart data from one place.

Acceptance criteria

  • All ten games end a run through one typed helper

Getting started

  • Relevant files: shared/scenes.ts, shared/pause.ts, games/*/scenes/GameScene.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

Filed by repo-audit/full-audit at b001f5d. Severity: low. Effort: M. audit-key: tech-debt|shared/scenes.ts|GameOverData

Langage dominant
TypeScript
Étoiles
3
Forks
3
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

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de TMHSDigital/GH-Arcade

Toutes les issues de TMHSDigital/GH-Arcade

Issues similaires

Plus d'issues TypeScript

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.