@forgeax/engine-scene 0.1.34 → 0.1.35

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forgeax/engine-scene",
-  "version": "0.1.34",
+  "version": "0.1.35",
   "private": false,
   "type": "module",
   "license": "Apache-2.0",
@@ -22,10 +22,10 @@
     "LICENSE"
   ],
   "dependencies": {
-    "@forgeax/engine-ecs": "0.1.34",
-    "@forgeax/engine-math": "0.1.34",
-    "@forgeax/engine-plugin": "0.1.34",
-    "@forgeax/engine-types": "0.1.34"
+    "@forgeax/engine-ecs": "0.1.35",
+    "@forgeax/engine-math": "0.1.35",
+    "@forgeax/engine-plugin": "0.1.35",
+    "@forgeax/engine-types": "0.1.35"
   },
   "forgeax": {
     "metrics": {

package/src/__tests__/sparse-global-observation.contract.test.ts

@@ -0,0 +1,30 @@
+import { type EntityHandle, World } from '@forgeax/engine-ecs';
+import { expect, it, vi } from 'vitest';
+import { GlobalTransform, propagateTransforms, Transform } from '../index';
+
+const INTERNAL: unique symbol = Symbol.for(
+  'forgeax.ecs.worldInternal',
+) as unknown as typeof INTERNAL;
+
+it('observes sparse derived transforms without walking the complete entity table', () => {
+  const world = new World();
+  const movers: EntityHandle[] = [];
+  for (let index = 0; index < 50_000; index++) {
+    const entity = world.spawn({ component: Transform, data: { pos: [index, 0, 0] } }).unwrap();
+    if (index < 100) movers.push(entity);
+  }
+  propagateTransforms(world).unwrap();
+  for (const entity of movers) world.set(entity, Transform, { pos: [7, 0, 0] }).unwrap();
+  const internal = (world as unknown as { [INTERNAL]: { getGraph(): unknown } })[INTERNAL];
+  const graphReads = vi.spyOn(internal, 'getGraph');
+  try {
+    propagateTransforms(world).unwrap();
+    // A row walk reads the graph per entity. Sparse observation must stay
+    // proportional to changed ranges and table blocks, not the 50k population.
+    expect(graphReads.mock.calls.length).toBeLessThan(5_000);
+    for (const entity of movers)
+      expect(world.get(entity, GlobalTransform).unwrap().world[12]).toBe(7);
+  } finally {
+    graphReads.mockRestore();
+  }
+});

package/src/systems/propagate-transforms.ts

@@ -1264,12 +1264,12 @@ export function propagateTransforms(world: World): Result<void, SceneError> {
   const pairs = validateTransformPairs(world, scratch);
   if (!pairs.ok) return pairs;
   let globalEdited = false;
-  for (const _row of scratch.dirtyGlobals ?? []) globalEdited = true;
+  for (const _span of scratch.dirtyGlobals?.spans().unwrap() ?? []) globalEdited = true;
   const flat = propagateFlat(world, scratch);
   if (!flat.ok) return flat;
   const hierarchy = propagateHierarchy(world, scratch, globalEdited);
   // Consume our own derived publications, while preserving later external edits.
-  for (const _row of scratch.dirtyGlobals ?? []) {
+  for (const _span of scratch.dirtyGlobals?.spans().unwrap() ?? []) {
     /* observation only */
   }
   return hierarchy;