orion-ecs / keen-eye / 20213035333

65%

Build Type

push

github

Committed by tyevco

Commit Message

fix: Use proper archetype migration in SetComponent to preserve entity IDs

The previous implementation despawned the old entity and spawned a new one
when adding components via SetComponent, which changed the entity ID.
This broke the ECS contract that entity IDs should be stable across
archetype migrations.

Changes:
- Add AddComponentBoxed method to ArchetypeManager for proper migration
- Simplify SetComponent to use AddComponentBoxed instead of despawn/spawn
- Remove unnecessary entity map workaround from DeltaRestorer
- Update test to verify entity ID stability

The entity ID is now preserved during archetype migration, matching the
behavior of the generic Add<T> method.

Coverage Stats

1818 of 1963 branches covered (92.61%)

Branch coverage included in aggregate %.

11 of 14 new or added lines in 3 files covered. (78.57%)

17 existing lines in 4 files now uncovered.

9991 of 10871 relevant lines covered (91.91%)

1.46 hits per line