fix(core): the game loop stops silently when a library hook throws
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 72/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- typescript
- Domaine
- game-dev
Piste de recherche
Start with packages/core/src/application/nanoforge-application.ts lines 95–121, where the async loop runs library hooks and schedules the next tick; inspect the existing stop and teardown paths there. Add coverage under packages/core/test for a throwing hook and for a throwing __clear; done means the loop reports the error, stops cleanly, continues teardown across failing clears, and notifies stop observers.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Which package is this bug report for?
core
Issue description
loop is an async function scheduled with setTimeout. When a __events / __run hook rejects (for example a system throws inside ecs.runSystems), this.schedule(loop, …) is never reached and the rejection is unhandled. The game freezes with no error, no __clear, no viewport dispose and no onStop for observers such as the editor; on Bun the server process exits. The teardown branch also stops at the first __clear that throws.
Where:
Evidence:
for (const library of orderedForRun) await runHook(library, "__events");
...
for (const library of orderedForRun) await runHook(library, "__run");
...
this.schedule(loop, tickLengthMs + tickStart - Date.now());
Suggested fix: Wrap the tick body in try/catch: log the error, requestStop(), then run the normal teardown, isolating each __clear. Add a test with a throwing hook (packages/core/test has none).
Found during a code audit of main at be72076.
Code sample
No response
Versions
- Engine
mainat be72076 - pnpm 12.0.0, Node.js 26, Bun 1.3.14
Issue priority
High (immediate attention needed)
I have tested this issue on a development release
be72076
- Langage dominant
- TypeScript
- Étoiles
- 7
- Forks
- 2
- Merge moyen
- 1 j 17 h
- PR mergées (30 j)
- 17
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
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 NanoForge-dev/Engine
-
dependencies enhancement good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
NanoForge-dev/Engine#513 ·
Les mainteneurs répondent en général sous 1 jour
-
bug chore tests
Difficulté 2/5 1-3 heures Accessibilité débutants 77/100
NanoForge-dev/Engine#511 ·
Les mainteneurs répondent en général sous 1 jour
-
docs(network): TSDoc says ICE_SERVERS defaults to an empty list, the code uses public STUN serversOuvertedocumentation good first issue modules:network
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
NanoForge-dev/Engine#509 ·
Les mainteneurs répondent en général sous 1 jour
-
documentation modules:ecs packages:asset packages:core
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
NanoForge-dev/Engine#508 ·
Les mainteneurs répondent en général sous 1 jour
-
bug good first issue modules:music modules:sound
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
NanoForge-dev/Engine#505 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de NanoForge-dev/Engine
Issues similaires
-
Mend: dependency security vulnerability
Difficulté 1/5 1-3 heures Accessibilité débutants 65/100
fdm-monster/fdm-monster#5563 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
be-BOP-io-SA/be-BOP#2849 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 79/100
Les mainteneurs répondent en général sous 1 jour
-
1️⃣ phase/under-investigation flaky/confirmed flaky/needs-decision type/bug
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
growilabs/growi#12246 · 3 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
🐛 The Rulekeeper - An index run that gives up on the lock ends SucceededPeut-être pris @github-actions l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
SSWConsulting/SSW.Rules#2809 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour