fix(ecs): killing an entity twice makes spawnEntity return the same id twice

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#479 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
85/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
cpp
Domaine
game-dev

Piste de recherche

Lisez les deux plages liées dans modules/ecs/wasm/Registry.hpp (lignes 142–151 et 172–177), puis examinez les tests dans modules/ecs/test. Vérifiez comment les ID d’entité sont attribués et comment les entités mortes sont suivies. Ajoutez des tests pour tuer deux fois la même entité et pour rejeter un ID supérieur ou égal à _next_entity ; le résultat attendu est que des spawns répétés renvoient des ID distincts et que les morts invalides ne corrompent pas l’allocation.

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

kill_entity pushes to _dead_entities with no check. A second kill of the same entity (gameplay, then scene unload or the editor) leaves [e, e], so the next two spawn_entity() calls return the same id and two live entities share components. An id >= _next_entity is accepted too.

Where:

Evidence:

void kill_entity(Entity const &entity)
{
    _dead_entities.push_back(entity);
    for (...) remove_function(*this, entity);
}

Suggested fix: Return early when entity >= _next_entity or the entity is already dead (a std::vector<bool> alive flag makes the check O(1)). Add a double-kill test in modules/ecs/test.

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.