fix(ecs): scene unload kills entities that were already killed and recycled

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#480 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
76/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
typescript
Domaine
game-dev

Piste de recherche

Commencez par modules/ecs/src/scene/ecs-scene.ts, en particulier _track et la boucle de déchargement aux lignes 77–110. Suivez la façon dont les entités suivies sont enregistrées et dont killEntity met à jour le registre ; l’issue suggère de retirer les entités tuées de _entities et byId. Le travail est terminé lorsque le déchargement après la mort d’une entité et la réutilisation de son id ne tue pas la nouvelle entité, ne détruit pas deux fois les anciens composants et n’ajoute pas de doublon d’id mort.

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

Description

bug modules:ecs
Which package is this bug report for?

ecs

Issue description

The tracked registry overrides spawnEntity, addComponent and addSystem, but not killEntity. If gameplay kills a scene entity and its id is reused by a later spawn, unloading the scene kills the new, unrelated owner of that id and calls destroy() on components that were already destroyed. It also pushes a duplicate dead id (see the double-kill issue).

Where:

Evidence:

for (const { entity, components } of this._entities.splice(0)) {
  ...destroy.call(component);
  registry.killEntity(entity);
}

Suggested fix: Override killEntity in _track so that it drops the entity from _entities / byId.

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

Langage dominant
TypeScript
Étoiles
7
Forks
2
Merge moyen
1 j 17 h
PR mergées (30 j)
17

Préparer son environnement

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 NanoForge-dev/Engine

Toutes les issues de NanoForge-dev/Engine

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.