fix(core): the game loop stops silently when a library hook throws

Aberta
  • Algo errado?
#478 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
3/5
Tempo estimado
1-2 dias
Facilidade para iniciantes
72/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
typescript
Domínio
game-dev

Direção de pesquisa

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.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

bug packages:core
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 main at 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

Linguagem predominante
TypeScript
Estrelas
7
Forks
2
Merge médio
1d 17h
PRs com merge (30d)
17

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de NanoForge-dev/Engine

Todas as issues de NanoForge-dev/Engine

Issues semelhantes

Mais issues de TypeScript

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.