@codehz/ecs 0.1.4 → 0.1.5

package/changeset.d.ts

@@ -29,8 +29,4 @@ export declare class ComponentChangeset {
      * Apply the changeset to existing components and return the final state
      */
     applyTo(existingComponents: Map<EntityId<any>, any>): Map<EntityId<any>, any>;
-    /**
-     * Get the final component types after applying changes
-     */
-    getFinalComponentTypes(existingComponents: Map<EntityId<any>, any>): EntityId<any>[];
 }

package/index.js

@@ -277,17 +277,23 @@ class Archetype {
     if (index === undefined) {
       return;
     }
-    this.entities.splice(index, 1);
-    this.entityToIndex.delete(entityId);
     const removedData = new Map;
     for (const componentType of this.componentTypes) {
       const dataArray = this.getComponentData(componentType);
-      removedData.set(componentType, dataArray[index]);
-      dataArray.splice(index, 1);
+      removedData.set(componentType, dataArray[index] === MISSING_COMPONENT ? undefined : dataArray[index]);
     }
-    for (let i = index;i < this.entities.length; i++) {
-      this.entityToIndex.set(this.entities[i], i);
+    const lastIndex = this.entities.length - 1;
+    if (index !== lastIndex) {
+      const lastEntity = this.entities[lastIndex];
+      this.entities[index] = lastEntity;
+      this.entityToIndex.set(lastEntity, index);
+      for (const componentType of this.componentTypes) {
+        const dataArray = this.getComponentData(componentType);
+        [dataArray[index], dataArray[lastIndex]] = [dataArray[lastIndex], dataArray[index]];
+      }
     }
+    this.entities.pop();
+    this.entityToIndex.delete(entityId);
     return removedData;
   }
   exists(entityId) {
@@ -439,10 +445,6 @@ class ComponentChangeset {
     }
     return finalComponents;
   }
-  getFinalComponentTypes(existingComponents) {
-    const finalComponents = this.applyTo(existingComponents);
-    return Array.from(finalComponents.keys()).sort((a, b) => a - b);
-  }
 }
 
 // src/command-buffer.ts
@@ -995,11 +997,10 @@ class World {
       }
     }
     const finalComponents = changeset.applyTo(currentComponents);
-    const finalComponentTypes = changeset.getFinalComponentTypes(currentComponents);
-    const currentComponentTypes = currentArchetype.componentTypes.sort((a, b) => a - b);
-    const needsArchetypeChange = finalComponentTypes.length !== currentComponentTypes.length || !finalComponentTypes.every((type, index) => type === currentComponentTypes[index]);
+    const currentComponentTypes = currentArchetype.componentTypes;
+    const needsArchetypeChange = finalComponents.size !== currentComponentTypes.length || !currentComponentTypes.every((type) => finalComponents.has(type));
     if (needsArchetypeChange) {
-      const newArchetype = this.ensureArchetype(finalComponentTypes);
+      const newArchetype = this.ensureArchetype(finalComponents.keys().toArray());
       currentArchetype.removeEntity(entityId);
       newArchetype.addEntity(entityId, finalComponents);
       this.entityToArchetype.set(entityId, newArchetype);
@@ -1030,7 +1031,7 @@ class World {
     return changeset;
   }
   ensureArchetype(componentTypes) {
-    const sortedTypes = [...componentTypes].sort((a, b) => a - b);
+    const sortedTypes = componentTypes.toSorted((a, b) => a - b);
     const hashKey = this.createArchetypeSignature(sortedTypes);
     return getOrCreateWithSideEffect(this.archetypeBySignature, hashKey, () => {
       const newArchetype = new Archetype(sortedTypes);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codehz/ecs",
-  "version": "0.1.4",
+  "version": "0.1.5",
   "type": "module",
   "main": "./index.js",
   "types": "./index.d.ts",