@forgeax/engine-ecs 0.1.21 → 0.1.23

package/src/query/derived-range-writer.ts

@@ -0,0 +1,332 @@
+import { err, ok, type Result } from '@forgeax/engine-types';
+import type { Component } from '../component';
+import * as componentOwner from '../component';
+import {
+  DerivedRangeOutOfBoundsError,
+  SharedKernelFailureError,
+  WorldPoisonedError,
+} from '../errors';
+import type { Table } from '../storage/table';
+import type { EcsError, World } from '../world';
+import { worldInternal } from '../world-internal';
+import type { MutableColumnShape, ReadonlyColumnShape } from './query';
+
+/** Package-internal whole-column storage owned by one dense query table. */
+export interface DerivedColumnBinding<R extends Component, C extends Component> {
+  readonly read: ReadonlyColumnShape<R>;
+  readonly write: MutableColumnShape<C>;
+  readonly rowCapacity: number;
+}
+
+export type DerivedRangeKernel<R extends Component, C extends Component> = (
+  binding: DerivedColumnBinding<R, C>,
+  base: number,
+  start: number,
+  count: number,
+  context: unknown,
+) => void;
+
+/** Package-internal row probe used when the probe already has the derived value in scratch storage. */
+export type DerivedRangeRowProbe<R extends Component, C extends Component> = (
+  binding: DerivedColumnBinding<R, C>,
+  row: number,
+  context: unknown,
+) => boolean;
+
+/** Package-internal commit for a value prepared by a preceding row probe. */
+export type DerivedRangeRowCommit<R extends Component, C extends Component> = (
+  binding: DerivedColumnBinding<R, C>,
+  row: number,
+  context: unknown,
+) => void;
+
+interface DerivedRangeBindingSource {
+  readonly structureEpoch: () => number;
+  readonly tables: () => readonly Table[];
+  readonly readComponents: readonly Component[];
+  readonly writeComponents: readonly Component[];
+}
+
+export interface DerivedRangeAllocationTrace {
+  subarrayCalls: number;
+  shapeAllocations: number;
+  closureAllocations: number;
+  columnWindowAllocations: number;
+  rowFacadeAllocations: number;
+  proxyAllocations: number;
+  rangeCacheAllocations: number;
+  bindingRebuilds: number;
+}
+
+let allocationTrace: DerivedRangeAllocationTrace | undefined;
+
+export function beginDerivedRangeAllocationTrace(): void {
+  allocationTrace = {
+    subarrayCalls: 0,
+    shapeAllocations: 0,
+    closureAllocations: 0,
+    columnWindowAllocations: 0,
+    rowFacadeAllocations: 0,
+    proxyAllocations: 0,
+    rangeCacheAllocations: 0,
+    bindingRebuilds: 0,
+  };
+}
+
+export function endDerivedRangeAllocationTrace(): DerivedRangeAllocationTrace {
+  const trace = allocationTrace;
+  allocationTrace = undefined;
+  return (
+    trace ?? {
+      subarrayCalls: 0,
+      shapeAllocations: 0,
+      closureAllocations: 0,
+      columnWindowAllocations: 0,
+      rowFacadeAllocations: 0,
+      proxyAllocations: 0,
+      rangeCacheAllocations: 0,
+      bindingRebuilds: 0,
+    }
+  );
+}
+
+function countAllocation(name: keyof DerivedRangeAllocationTrace): void {
+  const trace = allocationTrace;
+  if (trace !== undefined) trace[name] += 1;
+}
+
+export interface DerivedRangeWriter<R extends Component, C extends Component> {
+  readonly bindings: readonly DerivedColumnBinding<R, C>[];
+  writeRange(
+    bindingIndex: number,
+    base: number,
+    start: number,
+    count: number,
+    kernel: DerivedRangeKernel<R, C>,
+    context: unknown,
+  ): Result<void, EcsError>;
+  /**
+   * Probe and commit rows in one pass. The probe must only inspect the derived
+   * value prepared in reusable scratch storage; the commit copies that value.
+   * The writer publishes only contiguous changed runs and does not allocate in
+   * the hot loop.
+   */
+  probeAndCommitRange(
+    bindingIndex: number,
+    base: number,
+    start: number,
+    count: number,
+    probe: DerivedRangeRowProbe<R, C>,
+    commit: DerivedRangeRowCommit<R, C>,
+    context: unknown,
+  ): Result<void, EcsError>;
+}
+
+export function createDerivedRangeWriter<R extends Component, C extends Component>(
+  world: World,
+  component: C,
+  source: DerivedRangeBindingSource,
+): DerivedRangeWriter<R, C> {
+  let boundEpoch = -1;
+  let bindingTables: readonly Table[] = [];
+  let bindings: readonly DerivedColumnBinding<R, C>[] = [];
+  let runStartBuffers: readonly Int32Array[] = [];
+  let runCountBuffers: readonly Int32Array[] = [];
+
+  const rebind = (): void => {
+    countAllocation('bindingRebuilds');
+    const tables = source.tables();
+    const nextBindings: DerivedColumnBinding<R, C>[] = [];
+    const nextRunStartBuffers: Int32Array[] = [];
+    const nextRunCountBuffers: Int32Array[] = [];
+    for (const table of tables) {
+      nextBindings.push({
+        read: buildWholeColumnShape(table, source.readComponents) as ReadonlyColumnShape<R>,
+        write: buildWholeColumnShape(table, source.writeComponents) as MutableColumnShape<C>,
+        rowCapacity: table.size,
+      });
+      // A table with N rows can contain at most N changed runs. These buffers
+      // are owned by the structural rebind and reused by every hot call.
+      const runCapacity = Math.max(1, table.size);
+      nextRunStartBuffers.push(new Int32Array(runCapacity));
+      nextRunCountBuffers.push(new Int32Array(runCapacity));
+    }
+    bindingTables = tables;
+    bindings = nextBindings;
+    runStartBuffers = nextRunStartBuffers;
+    runCountBuffers = nextRunCountBuffers;
+    boundEpoch = source.structureEpoch();
+  };
+
+  rebind();
+
+  const writer: DerivedRangeWriter<R, C> = {
+    get bindings() {
+      if (boundEpoch !== source.structureEpoch()) rebind();
+      return bindings;
+    },
+    writeRange(bindingIndex, base, start, count, kernel, context) {
+      if (world.execution.health === 'poisoned') {
+        return err(new WorldPoisonedError(world.identity, world.execution.fault));
+      }
+      if (boundEpoch !== source.structureEpoch()) rebind();
+      const binding = bindings[bindingIndex];
+      const table = bindingTables[bindingIndex];
+      if (
+        binding === undefined ||
+        table === undefined ||
+        !Number.isSafeInteger(bindingIndex) ||
+        !Number.isSafeInteger(base) ||
+        !Number.isSafeInteger(start) ||
+        !Number.isSafeInteger(count) ||
+        bindingIndex < 0 ||
+        base < 0 ||
+        start < 0 ||
+        count < 0 ||
+        base + start + count > binding.rowCapacity ||
+        table.storage.get(componentOwner.componentId(component)) === undefined
+      ) {
+        return err(new DerivedRangeOutOfBoundsError(start, count, binding?.rowCapacity ?? 0));
+      }
+      if (count === 0) return ok(undefined);
+
+      const previousEpoch = world[worldInternal].getMutationEpoch();
+      let epoch: number;
+      try {
+        epoch = world[worldInternal].nextMutationEpoch();
+      } catch (cause) {
+        return err(cause as EcsError);
+      }
+      try {
+        kernel(binding, base, start, count, context);
+        world[worldInternal].publishDerivedRange(
+          table,
+          componentOwner.componentId(component),
+          base + start,
+          count,
+          epoch,
+        );
+        return ok(undefined);
+      } catch (cause) {
+        world[worldInternal].restoreMutationEpoch(previousEpoch);
+        world[worldInternal].poisonExecution({
+          code: 'shared-kernel-failed',
+          kernelName: `derived-range:${component.name}`,
+          cause,
+          partialWrite: true,
+          retryable: false,
+        });
+        return err(new SharedKernelFailureError(component.name, world.identity, cause, true));
+      }
+    },
+    probeAndCommitRange(bindingIndex, base, start, count, probe, commit, context) {
+      if (world.execution.health === 'poisoned') {
+        return err(new WorldPoisonedError(world.identity, world.execution.fault));
+      }
+      if (boundEpoch !== source.structureEpoch()) rebind();
+      const binding = bindings[bindingIndex];
+      const table = bindingTables[bindingIndex];
+      const runStarts = runStartBuffers[bindingIndex];
+      const runCounts = runCountBuffers[bindingIndex];
+      if (
+        binding === undefined ||
+        table === undefined ||
+        runStarts === undefined ||
+        runCounts === undefined ||
+        !Number.isSafeInteger(bindingIndex) ||
+        !Number.isSafeInteger(base) ||
+        !Number.isSafeInteger(start) ||
+        !Number.isSafeInteger(count) ||
+        bindingIndex < 0 ||
+        base < 0 ||
+        start < 0 ||
+        count < 0 ||
+        base + start + count > binding.rowCapacity ||
+        table.storage.get(componentOwner.componentId(component)) === undefined
+      ) {
+        return err(new DerivedRangeOutOfBoundsError(start, count, binding?.rowCapacity ?? 0));
+      }
+      if (count === 0) return ok(undefined);
+
+      const previousEpoch = world[worldInternal].getMutationEpoch();
+      let runCount = 0;
+      let epoch = 0;
+      let epochReserved = false;
+      let runStart = -1;
+      try {
+        for (let offset = 0; offset < count; offset += 1) {
+          const row = base + start + offset;
+          const changed = probe(binding, row, context);
+          if (changed) {
+            if (!epochReserved) {
+              try {
+                epoch = world[worldInternal].nextMutationEpoch();
+                epochReserved = true;
+              } catch (cause) {
+                // Nothing has been committed yet. Keep the World healthy and
+                // preserve the source error so the caller can retry unchanged.
+                return err(cause as EcsError);
+              }
+            }
+            if (runStart < 0) runStart = row;
+            commit(binding, row, context);
+          } else if (runStart >= 0) {
+            runStarts[runCount] = runStart;
+            runCounts[runCount] = row - runStart;
+            runCount += 1;
+            runStart = -1;
+          }
+        }
+        if (runStart >= 0) {
+          runStarts[runCount] = runStart;
+          runCounts[runCount] = base + start + count - runStart;
+          runCount += 1;
+        }
+
+        // Every changed run in this call shares the epoch reserved immediately
+        // before the first commit. Publication remains deferred until the
+        // complete probe/commit pass has succeeded.
+        if (runCount > 0) {
+          const componentIdentifier = componentOwner.componentId(component);
+          for (let index = 0; index < runCount; index += 1) {
+            const runStartValue = runStarts[index] ?? 0;
+            const runLength = runCounts[index] ?? 0;
+            world[worldInternal].publishDerivedRange(
+              table,
+              componentIdentifier,
+              runStartValue,
+              runLength,
+              epoch,
+            );
+          }
+        }
+        return ok(undefined);
+      } catch (cause) {
+        if (epochReserved) world[worldInternal].restoreMutationEpoch(previousEpoch);
+        world[worldInternal].poisonExecution({
+          code: 'shared-kernel-failed',
+          kernelName: `derived-range:${component.name}:probe`,
+          cause,
+          partialWrite: true,
+          retryable: false,
+        });
+        return err(new SharedKernelFailureError(component.name, world.identity, cause, true));
+      }
+    },
+  };
+  return writer;
+}
+
+function buildWholeColumnShape(
+  table: Table,
+  components: readonly Component[],
+): Record<string, unknown> {
+  countAllocation('shapeAllocations');
+  const shape: Record<string, unknown> = {};
+  for (const component of components) {
+    const fields = table.storage.get(componentOwner.componentId(component))?.fields;
+    if (fields === undefined) continue;
+    for (const [fieldName, column] of fields) shape[fieldName] = column.view;
+  }
+  return shape;
+}

package/src/query/query.ts

@@ -19,12 +19,14 @@ import {
   QuerySpanUnavailableError,
   type QuerySpanUnavailableReason,
 } from '../errors';
+import { DERIVED_WRITER } from '../internal';
 import type { Archetype, ArchetypeId } from '../storage/archetype';
 import type { ArchetypeGraph } from '../storage/archetype-graph';
 import { sparseTagIndex } from '../storage/change-detection';
 import type { FieldView, ManagedColumnReader } from '../storage/column';
 import type { Table } from '../storage/table';
 import { type WorldInternal, worldInternal } from '../world-internal';
+import { createDerivedRangeWriter, type DerivedRangeWriter } from './derived-range-writer';
 
 export interface QueryDescriptor<
   R extends readonly Component[] = readonly Component[],
@@ -91,7 +93,7 @@ export interface Query<
    * Read one known entity without iterating the query's matching tables.
    * Structural descriptor constraints are applied; temporal `changed` and
    * `added` filters are intentionally not evaluated because this is an
-   * identity-addressed read, not a journal drain.
+   * identity-addressed read, not a version scan.
    */
   at(entity: EntityHandle): QueryRow<R, W, O> | undefined;
   spans(): Result<Iterable<QuerySpan<R, W>>, QuerySpanUnavailableError>;
@@ -254,8 +256,13 @@ class QuerySpanFacade<R extends readonly Component[], W extends readonly Compone
     readonly length: number,
   ) {
     const entityColumn = table.storage.get(componentId(Entity))?.fields.get('self');
-    this.entities = (entityColumn?.view.subarray(rowStart, rowStart + length) ??
-      new Uint32Array(0)) as Readonly<Uint32Array>;
+    this.entities = (
+      entityColumn === undefined
+        ? new Uint32Array(0)
+        : rowStart === 0 && length === entityColumn.view.length
+          ? entityColumn.view
+          : entityColumn.view.subarray(rowStart, rowStart + length)
+    ) as Readonly<Uint32Array>;
   }
 
   get<C extends R[number]>(component: C): ReadonlyColumnShape<C> {
@@ -288,7 +295,7 @@ function makeManagedColumnReader(
   length: number,
   fieldType: string,
 ): ManagedColumnReader<string> {
-  const slots = view.subarray(0, length);
+  const slots = view.length === length ? view : view.subarray(0, length);
   return Object.freeze({
     length,
     get(i: number): number {
@@ -310,7 +317,8 @@ function buildColumnShape(
   for (const [fieldName, column] of fields) {
     const start = rowStart * column.arity;
     const end = start + rowCount * column.arity;
-    const view = column.view.subarray(start, end);
+    const view =
+      start === 0 && end === column.view.length ? column.view : column.view.subarray(start, end);
     const fieldType = componentSchema(component)[fieldName];
     shape[fieldName] =
       fieldType !== undefined && isManagedField(fieldType)
@@ -328,6 +336,7 @@ class ExecutableQuery<
 {
   private matchedArchetypes: ArchetypeId[] = [];
   private matchedTables: number[] = [];
+  private matchedTableObjects: Table[] = [];
   private lastGraphGeneration = -1;
   private lastObservedEpoch = 0;
   private active = false;
@@ -432,7 +441,8 @@ class ExecutableQuery<
         query.refreshMatches();
         const structureEpoch = query.world[worldInternal].getStructureEpoch();
         const upperBound = query.world[worldInternal].getMutationEpoch();
-        let index = 0;
+        let tableIndex = 0;
+        let rowIndex = 0;
         let finished = false;
         const close = (commit: boolean): void => {
           if (finished) return;
@@ -450,13 +460,44 @@ class ExecutableQuery<
                 query.world[worldInternal].getStructureEpoch(),
               );
             }
-            while (index < query.matchedTables.length) {
+            while (tableIndex < query.matchedTables.length) {
               const table =
-                query.world[worldInternal].getGraph().tables[query.matchedTables[index++] ?? -1];
-              if (table === undefined || table.size === 0) continue;
+                query.world[worldInternal].getGraph().tables[query.matchedTables[tableIndex] ?? -1];
+              if (table === undefined || table.size === 0) {
+                tableIndex += 1;
+                rowIndex = 0;
+                continue;
+              }
+              if (query.compiled.changedIds.length === 0 && query.compiled.addedIds.length === 0) {
+                tableIndex += 1;
+                rowIndex = 0;
+                return {
+                  done: false,
+                  value: new QuerySpanFacade<R, W>(query.world, table, 0, table.size),
+                };
+              }
+              while (
+                rowIndex < table.size &&
+                !query.denseChangeMatches(table, rowIndex, upperBound)
+              ) {
+                rowIndex += 1;
+              }
+              if (rowIndex >= table.size) {
+                tableIndex += 1;
+                rowIndex = 0;
+                continue;
+              }
+              const start = rowIndex;
+              rowIndex += 1;
+              while (
+                rowIndex < table.size &&
+                query.denseChangeMatches(table, rowIndex, upperBound)
+              ) {
+                rowIndex += 1;
+              }
               return {
                 done: false,
-                value: new QuerySpanFacade<R, W>(query.world, table, 0, table.size),
+                value: new QuerySpanFacade<R, W>(query.world, table, start, rowIndex - start),
               };
             }
             close(true);
@@ -471,6 +512,28 @@ class ExecutableQuery<
     });
   }
 
+  [DERIVED_WRITER]<C extends W[number]>(
+    component: C,
+  ): Result<DerivedRangeWriter<R[number], C>, QuerySpanUnavailableError> {
+    const reason = this.spanUnavailableReason();
+    if (reason !== undefined) return err(new QuerySpanUnavailableError(reason));
+    return ok(
+      createDerivedRangeWriter<R[number], C>(
+        this.world as unknown as import('../world').World,
+        component,
+        {
+          structureEpoch: () => this.world[worldInternal].getStructureEpoch(),
+          tables: () => {
+            this.refreshMatches();
+            return this.matchedTableObjects;
+          },
+          readComponents: this.compiled.descriptor.read ?? [],
+          writeComponents: this.compiled.descriptor.write ?? [],
+        },
+      ),
+    );
+  }
+
   combinations(k = 2): Iterable<readonly QueryRow<R, W, O>[]> {
     const query = this;
     return {
@@ -527,6 +590,10 @@ class ExecutableQuery<
     this.matchedTables = graph.tables
       .filter((table) => this.tableMatches(table))
       .map((table) => table.id);
+    this.matchedTableObjects = this.matchedTables.flatMap((id) => {
+      const table = graph.tables[id];
+      return table === undefined ? [] : [table];
+    });
     this.lastGraphGeneration = graph.generation;
   }
 
@@ -580,14 +647,21 @@ class ExecutableQuery<
     return true;
   }
 
+  private denseChangeMatches(table: Table, row: number, upperBound: number): boolean {
+    for (const componentId of this.compiled.changedIds) {
+      const epoch = table.storage.get(componentId)?.epochs.changed[row] ?? 0;
+      if (epoch <= this.lastObservedEpoch || epoch > upperBound) return false;
+    }
+    for (const componentId of this.compiled.addedIds) {
+      const epoch = table.storage.get(componentId)?.epochs.added[row] ?? 0;
+      if (epoch <= this.lastObservedEpoch || epoch > upperBound) return false;
+    }
+    return true;
+  }
+
   private spanUnavailableReason(): QuerySpanUnavailableReason | undefined {
     if (this.compiled.sparseRoute) return 'sparse-component';
     if ((this.compiled.descriptor.optional?.length ?? 0) > 0) return 'optional-data';
-    if (
-      (this.compiled.descriptor.changed?.length ?? 0) > 0 ||
-      (this.compiled.descriptor.added?.length ?? 0) > 0
-    )
-      return 'row-change-filter';
     return undefined;
   }
 }

package/src/relationship-index.ts

@@ -24,6 +24,14 @@ export interface RelationshipOptions {
   readonly sourceField: string;
   readonly targetName: string;
   readonly targetField: string;
+  /**
+   * Components materialized with the writable source side.  This is the
+   * relationship equivalent of `defineComponent(..., { requires })`: the
+   * dependency is resolved once at the structural boundary, never by a frame
+   * system.  It applies only to the source; the reverse target remains an
+   * engine-owned projection.
+   */
+  readonly sourceRequires?: readonly Component[];
   readonly exclusive?: boolean;
   readonly linkedSpawn?: boolean;
   /** Allow a source to point at its own holder (e.g. self-animated entities). */
@@ -75,7 +83,11 @@ export function defineRelationship<
   };
   const target = defineComponent(options.targetName, targetFields);
   const sourceFields: FieldsInput = { [options.sourceField]: { type: 'entity' } };
-  const source = defineComponent(options.sourceName, sourceFields);
+  const source = defineComponent(
+    options.sourceName,
+    sourceFields,
+    options.sourceRequires === undefined ? undefined : { requires: options.sourceRequires },
+  );
 
   roles.set(source, {
     kind: 'source',

package/src/shared-ref-store.ts

@@ -84,7 +84,7 @@ import {
   SharedRefStaleError,
 } from './errors';
 
-const SHARED_REF_MUTATION_JOURNAL_CAPACITY = 4096;
+const SHARED_REF_RELEASE_EVIDENCE_CAPACITY = 4096;
 
 export interface SharedRefMutation {
   readonly epoch: number;
@@ -98,17 +98,10 @@ export interface SharedRefReleaseEvidence {
   readonly evidence: 'released';
 }
 
-export type SharedRefMutationRead =
-  | {
-      readonly status: 'ok';
-      readonly cursor: number;
-      readonly records: readonly SharedRefMutation[];
-    }
-  | {
-      readonly status: 'overflow';
-      readonly cursor: number;
-      readonly oldestAvailable: number;
-    };
+export interface SharedRefMutationRead {
+  readonly cursor: number;
+  readonly records: readonly SharedRefMutation[];
+}
 
 // MAX_SLOT is now imported from @forgeax/engine-types (codec SSOT, D-1).
 // The local constant is removed to avoid drift (AC-15).
@@ -147,7 +140,8 @@ export class SharedRefStore {
   >();
   private nextSlot = BUILTIN_BASE;
   private mutationEpoch = 0;
-  private readonly mutationJournal: SharedRefMutation[] = [];
+  /** Latest published mutation epoch per live handle; not an event journal. */
+  private readonly mutationEpochs = new Map<number, number>();
   private readonly releaseJournal: SharedRefReleaseEvidence[] = [];
 
   /**
@@ -271,10 +265,7 @@ export class SharedRefStore {
       throw new RangeError('SharedRefStore mutation epoch exhausted');
     }
     this.mutationEpoch += 1;
-    this.mutationJournal.push({ epoch: this.mutationEpoch, handle: unwrapHandle(handle) });
-    if (this.mutationJournal.length > SHARED_REF_MUTATION_JOURNAL_CAPACITY) {
-      this.mutationJournal.shift();
-    }
+    this.mutationEpochs.set(unwrapHandle(handle), this.mutationEpoch);
     return ok(undefined);
   }
 
@@ -283,16 +274,16 @@ export class SharedRefStore {
     return this.mutationEpoch;
   }
 
-  /** Read exact changed handles after `cursor`; overflow requires consumer resync. */
+  /** Read each live handle whose latest published mutation is after `cursor`. */
   readChangesSince(cursor: number): SharedRefMutationRead {
-    const oldestAvailable = this.mutationJournal[0]?.epoch ?? this.mutationEpoch + 1;
-    if (cursor < oldestAvailable - 1) {
-      return { status: 'overflow', cursor: this.mutationEpoch, oldestAvailable };
+    const records: SharedRefMutation[] = [];
+    for (const [handle, epoch] of this.mutationEpochs) {
+      if (epoch > cursor) records.push({ epoch, handle });
     }
+    records.sort((left, right) => left.epoch - right.epoch || left.handle - right.handle);
     return {
-      status: 'ok',
       cursor: this.mutationEpoch,
-      records: this.mutationJournal.filter((record) => record.epoch > cursor),
+      records,
     };
   }
 
@@ -372,6 +363,7 @@ export class SharedRefStore {
     }
     this.refcounts.delete(raw);
     this.payloads.delete(raw);
+    this.mutationEpochs.delete(raw);
     // Gen increment + retire (AC-07): bump gen; once it would exceed MAX_GEN
     // (gen 255 is still usable; the bump to 256 triggers retire) the slot is
     // permanently retired — NOT pushed to freeSlots. This prevents handle
@@ -389,7 +381,7 @@ export class SharedRefStore {
       evidence: 'released' as const,
     });
     this.releaseJournal.push(evidence);
-    if (this.releaseJournal.length > SHARED_REF_MUTATION_JOURNAL_CAPACITY) {
+    if (this.releaseJournal.length > SHARED_REF_RELEASE_EVIDENCE_CAPACITY) {
       this.releaseJournal.shift();
     }
     return ok(evidence);

package/src/storage/change-detection.ts

@@ -2,14 +2,10 @@
 
 import type { Component } from '../component';
 import * as componentOwner from '../component';
-import { Entity } from '../entity';
 import { type EntityHandle, entityIndex } from '../entity-handle';
 
-export { WorldChangeJournal, type WorldChangeRead } from '../world-change-journal';
-
 import type { ArchetypeGraph } from './archetype-graph';
 import { getOrCreateSparseTagSet } from './archetype-graph';
-import type { Table } from './table';
 
 const INITIAL_SPARSE_CAPACITY = 64;
 
@@ -218,14 +214,3 @@ export function markComponentChanged(
   const tableRow = archetype.rows[location.archetypeRow] ?? -1;
   epochs.changed[tableRow] = epoch();
 }
-
-export function readTableEntityRange(
-  table: Table,
-  rowStart: number,
-  rowCount: number,
-): readonly EntityHandle[] {
-  const entities = table.storage.get(componentOwner.componentId(Entity))?.fields.get('self')
-    ?.view as Uint32Array | undefined;
-  if (entities === undefined) return [];
-  return entities.subarray(rowStart, rowStart + rowCount) as unknown as readonly EntityHandle[];
-}