@bldrs-ai/conway 1.528.1490 → 1.530.1503

package/compiled/examples/browser-bundled.cjs

@@ -30,7 +30,7 @@ var import_node_process = require("node:process");
 var readline = __toESM(require("node:readline"), 1);
 
 // compiled/src/version/version.js
-var versionString = "Conway v1.528.1490";
+var versionString = "Conway v1.530.1503";
 
 // compiled/dependencies/conway-geom/interface/conway_geometry.js
 var wasmType = "";
@@ -2626,10 +2626,12 @@ var StepParser = class extends StepHeaderParser {
    * buffer; the callback may rebase the buffer's window in place. Must be
    * synchronous on this driver — a returned Promise is a caller bug.
    * @param onRecordIndexed Called as each top-level record is indexed, with
-   * its localID, expressID and typeID (0 for external-mapping records) — the
-   * seam for incremental semantic consumers (type index, roots registry,
-   * names skeleton). Must be synchronous and cheap; expensive work belongs on
-   * a demand queue, not the parse path.
+   * its localID, expressID, typeID (0 for external-mapping records) and the
+   * record's bytes as an un-materialised `(buffer, byteOffset, byteLength)`
+   * window view — the seam for incremental semantic consumers (type index,
+   * roots registry, names skeleton). See {@link RecordEventHandler} for the
+   * window-lifetime rule. Must be synchronous and cheap; expensive work
+   * belongs on a demand queue, not the parse path.
    * @param onProgress Optional byte-cursor progress callback.
    * @return {BlockParseResult} The parsing result, including the index and result enum.
    */
@@ -2913,11 +2915,6 @@ var StepParser = class extends StepHeaderParser {
         if (!charws(SEMICOLON)) {
           return syntaxError();
         }
-        {
-          const recordLen = input2.address - startElement2;
-          onRecordIndexed?.(topLevelCount, expressID, 0, input2.buffer.subarray(input2.cursor - recordLen, input2.cursor));
-        }
-        ++topLevelCount;
         pushEntry({
           address: startElement2,
           length: input2.address - startElement2,
@@ -2927,6 +2924,11 @@ var StepParser = class extends StepHeaderParser {
           inlineEntities: inlineElements,
           multiMapping: multiElements
         });
+        {
+          const recordLen = input2.address - startElement2;
+          onRecordIndexed?.(topLevelCount, expressID, 0, input2.buffer, input2.cursor - recordLen, recordLen);
+        }
+        ++topLevelCount;
         multiElements = void 0;
         continue;
       }
@@ -3006,11 +3008,6 @@ var StepParser = class extends StepHeaderParser {
       if (!charws(SEMICOLON)) {
         return syntaxError();
       }
-      {
-        const recordLen = input2.address - startElement;
-        onRecordIndexed?.(topLevelCount, expressID, foundItem, input2.buffer.subarray(input2.cursor - recordLen, input2.cursor));
-      }
-      ++topLevelCount;
       pushEntry({
         address: startElement,
         length: input2.address - startElement,
@@ -3018,6 +3015,11 @@ var StepParser = class extends StepHeaderParser {
         expressID,
         inlineEntities: inlineElements
       });
+      {
+        const recordLen = input2.address - startElement;
+        onRecordIndexed?.(topLevelCount, expressID, foundItem, input2.buffer, input2.cursor - recordLen, recordLen);
+      }
+      ++topLevelCount;
     }
     return parseResult2(ParseResult.INCOMPLETE);
   }
@@ -5812,9 +5814,17 @@ var StepModelBase = class _StepModelBase {
    * extract reads `IfcIndexedPolygonalFace.CoordIndex` as integers —
    * following those faces pinned ~78k records per PSB product. If
    * omitted, `!descend` children use this span path (test helper).
+   * @param claimed Optional out-set of the IDs *this call* claimed and
+   * awaited residency for — a strict subset of `seen`. When `seen` is
+   * shared across concurrent walks (a batch of products sharing mapped
+   * geometry), an ID another walk claimed is skipped here and is NOT
+   * resident yet from this call's point of view: that walk's pin holds
+   * the chunk once it lands, but its read is still in flight. Anything
+   * that must synchronously read a record's bytes after this returns
+   * has to iterate `claimed`, never `seen` (conway#526).
    * @return {Promise<Set<number>>} The local IDs that were visited.
    */
-  async ensureResidentClosureByLocalID(localID, extraLocalIDs, seen = /* @__PURE__ */ new Set(), descend, leafSpans, spanLeaf) {
+  async ensureResidentClosureByLocalID(localID, extraLocalIDs, seen = /* @__PURE__ */ new Set(), descend, leafSpans, spanLeaf, claimed) {
     if (!this.isSourceExternal) {
       return seen;
     }
@@ -5839,6 +5849,7 @@ var StepModelBase = class _StepModelBase {
         seen.add(id);
         this.pinByLocalID(id);
         wave.push(id);
+        claimed?.add(id);
       }
       frontier.length = 0;
       if (wave.length === 0) {
@@ -72154,13 +72165,11 @@ async function buildIndexStreamingAsync(source, parser211, pool3, onRecordIndexe
     };
   }
 }
-function buildColumnarIndexStreaming(source, parser211, pool3, onRecordIndexed) {
-  const sink = new ColumnarIndexSink();
+function buildColumnarIndexStreaming(source, parser211, pool3, onRecordIndexed, sink = new ColumnarIndexSink()) {
   const { header, result, stats } = buildIndexStreaming(source, parser211, pool3, onRecordIndexed, sink);
   return { header, columns: sink.finalize(), result, stats };
 }
-async function buildColumnarIndexStreamingAsync(source, parser211, pool3, onRecordIndexed, onProgress, yieldIntervalMs) {
-  const sink = new ColumnarIndexSink();
+async function buildColumnarIndexStreamingAsync(source, parser211, pool3, onRecordIndexed, onProgress, yieldIntervalMs, sink = new ColumnarIndexSink()) {
   const { header, result, stats } = await buildIndexStreamingAsync(source, parser211, pool3, onRecordIndexed, sink, onProgress, yieldIntervalMs);
   return { header, columns: sink.finalize(), result, stats };
 }

package/compiled/examples/cli-bundled.cjs

@@ -14965,7 +14965,7 @@ ${t5.join("\n")}` : "";
 var import_process = require("process");
 
 // compiled/src/version/version.js
-var versionString = "Conway v1.528.1490";
+var versionString = "Conway v1.530.1503";
 
 // compiled/dependencies/conway-geom/interface/conway_geometry.js
 function pThreadsAllowed() {
@@ -18462,10 +18462,12 @@ var StepParser = class extends StepHeaderParser {
    * buffer; the callback may rebase the buffer's window in place. Must be
    * synchronous on this driver — a returned Promise is a caller bug.
    * @param onRecordIndexed Called as each top-level record is indexed, with
-   * its localID, expressID and typeID (0 for external-mapping records) — the
-   * seam for incremental semantic consumers (type index, roots registry,
-   * names skeleton). Must be synchronous and cheap; expensive work belongs on
-   * a demand queue, not the parse path.
+   * its localID, expressID, typeID (0 for external-mapping records) and the
+   * record's bytes as an un-materialised `(buffer, byteOffset, byteLength)`
+   * window view — the seam for incremental semantic consumers (type index,
+   * roots registry, names skeleton). See {@link RecordEventHandler} for the
+   * window-lifetime rule. Must be synchronous and cheap; expensive work
+   * belongs on a demand queue, not the parse path.
    * @param onProgress Optional byte-cursor progress callback.
    * @return {BlockParseResult} The parsing result, including the index and result enum.
    */
@@ -18749,11 +18751,6 @@ var StepParser = class extends StepHeaderParser {
         if (!charws(SEMICOLON)) {
           return syntaxError();
         }
-        {
-          const recordLen = input.address - startElement2;
-          onRecordIndexed?.(topLevelCount, expressID, 0, input.buffer.subarray(input.cursor - recordLen, input.cursor));
-        }
-        ++topLevelCount;
         pushEntry({
           address: startElement2,
           length: input.address - startElement2,
@@ -18763,6 +18760,11 @@ var StepParser = class extends StepHeaderParser {
           inlineEntities: inlineElements,
           multiMapping: multiElements
         });
+        {
+          const recordLen = input.address - startElement2;
+          onRecordIndexed?.(topLevelCount, expressID, 0, input.buffer, input.cursor - recordLen, recordLen);
+        }
+        ++topLevelCount;
         multiElements = void 0;
         continue;
       }
@@ -18842,11 +18844,6 @@ var StepParser = class extends StepHeaderParser {
       if (!charws(SEMICOLON)) {
         return syntaxError();
       }
-      {
-        const recordLen = input.address - startElement;
-        onRecordIndexed?.(topLevelCount, expressID, foundItem, input.buffer.subarray(input.cursor - recordLen, input.cursor));
-      }
-      ++topLevelCount;
       pushEntry({
         address: startElement,
         length: input.address - startElement,
@@ -18854,6 +18851,11 @@ var StepParser = class extends StepHeaderParser {
         expressID,
         inlineEntities: inlineElements
       });
+      {
+        const recordLen = input.address - startElement;
+        onRecordIndexed?.(topLevelCount, expressID, foundItem, input.buffer, input.cursor - recordLen, recordLen);
+      }
+      ++topLevelCount;
     }
     return parseResult(ParseResult.INCOMPLETE);
   }
@@ -21695,9 +21697,17 @@ var StepModelBase = class _StepModelBase {
    * extract reads `IfcIndexedPolygonalFace.CoordIndex` as integers —
    * following those faces pinned ~78k records per PSB product. If
    * omitted, `!descend` children use this span path (test helper).
+   * @param claimed Optional out-set of the IDs *this call* claimed and
+   * awaited residency for — a strict subset of `seen`. When `seen` is
+   * shared across concurrent walks (a batch of products sharing mapped
+   * geometry), an ID another walk claimed is skipped here and is NOT
+   * resident yet from this call's point of view: that walk's pin holds
+   * the chunk once it lands, but its read is still in flight. Anything
+   * that must synchronously read a record's bytes after this returns
+   * has to iterate `claimed`, never `seen` (conway#526).
    * @return {Promise<Set<number>>} The local IDs that were visited.
    */
-  async ensureResidentClosureByLocalID(localID, extraLocalIDs, seen = /* @__PURE__ */ new Set(), descend, leafSpans, spanLeaf) {
+  async ensureResidentClosureByLocalID(localID, extraLocalIDs, seen = /* @__PURE__ */ new Set(), descend, leafSpans, spanLeaf, claimed) {
     if (!this.isSourceExternal) {
       return seen;
     }
@@ -21722,6 +21732,7 @@ var StepModelBase = class _StepModelBase {
         seen.add(id);
         this.pinByLocalID(id);
         wave.push(id);
+        claimed?.add(id);
       }
       frontier.length = 0;
       if (wave.length === 0) {
@@ -88122,13 +88133,11 @@ async function buildIndexStreamingAsync(source, parser210, pool3, onRecordIndexe
     };
   }
 }
-function buildColumnarIndexStreaming(source, parser210, pool3, onRecordIndexed) {
-  const sink = new ColumnarIndexSink();
+function buildColumnarIndexStreaming(source, parser210, pool3, onRecordIndexed, sink = new ColumnarIndexSink()) {
   const { header, result, stats } = buildIndexStreaming(source, parser210, pool3, onRecordIndexed, sink);
   return { header, columns: sink.finalize(), result, stats };
 }
-async function buildColumnarIndexStreamingAsync(source, parser210, pool3, onRecordIndexed, onProgress, yieldIntervalMs) {
-  const sink = new ColumnarIndexSink();
+async function buildColumnarIndexStreamingAsync(source, parser210, pool3, onRecordIndexed, onProgress, yieldIntervalMs, sink = new ColumnarIndexSink()) {
   const { header, result, stats } = await buildIndexStreamingAsync(source, parser210, pool3, onRecordIndexed, sink, onProgress, yieldIntervalMs);
   return { header, columns: sink.finalize(), result, stats };
 }
@@ -93260,6 +93269,24 @@ var IfcGeometryExtraction = class _IfcGeometryExtraction {
     if (!this.model.isSourceExternal) {
       return seen ?? /* @__PURE__ */ new Set();
     }
+    try {
+      return await this.ensureResidentForProductExtractUnguarded_(localID, seen, leafSpans);
+    } catch (error) {
+      logger_default.error(`Error paging product localID ${localID} for extract: ${error instanceof Error ? error.message : String(error)}`);
+      return seen ?? /* @__PURE__ */ new Set();
+    }
+  }
+  /**
+   * {@link ensureResidentForProductExtract} without the per-product
+   * fault containment — split out so the guard cannot accidentally
+   * swallow a failure raised by its own bookkeeping.
+   *
+   * @param localID The product's local ID.
+   * @param seen Optional set to extend.
+   * @param leafSpans Coalesced Faces[] pin ranges — caller unpins.
+   * @return {Promise<Set<number>>} Pinned local IDs.
+   */
+  async ensureResidentForProductExtractUnguarded_(localID, seen, leafSpans) {
     const extra = [];
     const voids = this.productToVoidGeometryMap.get(localID);
     if (voids !== void 0) {
@@ -93277,7 +93304,8 @@ var IfcGeometryExtraction = class _IfcGeometryExtraction {
       const typeID = this.model.typeIDOf(id);
       return !isFace(id) && typeID !== entity_types_ifc_gen_default.IFCPOLYGONALFACESET && typeID !== entity_types_ifc_gen_default.IFCTRIANGULATEDFACESET;
     };
-    const visited = await this.model.ensureResidentClosureByLocalID(localID, extra, seen, descend, leafSpans, isFace);
+    const claimed = /* @__PURE__ */ new Set();
+    const visited = await this.model.ensureResidentClosureByLocalID(localID, extra, seen, descend, leafSpans, isFace, claimed);
     const styleExtra = [];
     for (const visitedID of visited) {
       const styled = this.materials.styledItemMap.get(visitedID);
@@ -93290,9 +93318,9 @@ var IfcGeometryExtraction = class _IfcGeometryExtraction {
       }
     }
     if (styleExtra.length > 0) {
-      await this.model.ensureResidentClosureByLocalID(localID, styleExtra, visited, descend, leafSpans, isFace);
+      await this.model.ensureResidentClosureByLocalID(localID, styleExtra, visited, descend, leafSpans, isFace, claimed);
     }
-    await this.ensureResidentVisitedFacesetPayloads_(visited, leafSpans);
+    await this.ensureResidentVisitedFacesetPayloads_(claimed, visited, leafSpans);
     return visited;
   }
   /**
@@ -93300,16 +93328,35 @@ var IfcGeometryExtraction = class _IfcGeometryExtraction {
    * or more tight address spans. The generic closure visits the
    * faceset but does not scan Faces[].
    *
-   * @param visited Faceset local IDs already pinned (and the set to
-   * extend with Coordinates).
+   * **Iterates `claimed`, not `visited`, and re-ensures before the
+   * scan** — both halves of conway#526. `referencedExpressIDs` is a
+   * SYNCHRONOUS acquire of the faceset's own bytes, and when a batch
+   * prefetches products concurrently through one shared `visited` set
+   * (`extractGeometryBatchAsync`'s Promise.all over 8 products), the
+   * closure walk skips any ID a sibling product already claimed. That
+   * sibling has pinned the range but its read may still be in flight,
+   * so scanning the shared set threw
+   * `StepBufferNotResidentError: STEP source range [...] is not
+   * resident` out of the prefetch and killed the whole load — a pin
+   * reserves a chunk against eviction, it does not make it resident.
+   * Iterating this call's own claims fixes that (and drops the
+   * quadratic re-scan of every sibling's facesets); the `await` below
+   * makes the guarantee local rather than inherited, since a resident
+   * chunk costs only a microtask there.
+   *
+   * @param claimed Faceset local IDs this call claimed and awaited.
+   * @param visited The (possibly shared) visited set — read for
+   * Coordinates de-duplication and extended with the ones pinned here,
+   * so the caller's unpin covers them.
    * @param leafSpans Coalesced Faces[] pin ranges — caller unpins.
    */
-  async ensureResidentVisitedFacesetPayloads_(visited, leafSpans) {
-    for (const localID of visited) {
+  async ensureResidentVisitedFacesetPayloads_(claimed, visited, leafSpans) {
+    for (const localID of claimed) {
       const typeID = this.model.typeIDOf(localID);
       if (typeID !== entity_types_ifc_gen_default.IFCPOLYGONALFACESET && typeID !== entity_types_ifc_gen_default.IFCTRIANGULATEDFACESET) {
         continue;
       }
+      await this.model.ensureResidentByLocalID(localID);
       const refs = this.model.referencedExpressIDs(localID);
       if (refs.length === 0) {
         continue;

package/compiled/examples/cli-step-bundled.cjs

@@ -15943,7 +15943,7 @@ var ParsingBuffer = class {
 };
 
 // compiled/src/version/version.js
-var versionString = "Conway v1.528.1490";
+var versionString = "Conway v1.530.1503";
 
 // compiled/dependencies/conway-geom/interface/conway_geometry.js
 function pThreadsAllowed() {
@@ -18365,10 +18365,12 @@ var StepParser = class extends StepHeaderParser {
    * buffer; the callback may rebase the buffer's window in place. Must be
    * synchronous on this driver — a returned Promise is a caller bug.
    * @param onRecordIndexed Called as each top-level record is indexed, with
-   * its localID, expressID and typeID (0 for external-mapping records) — the
-   * seam for incremental semantic consumers (type index, roots registry,
-   * names skeleton). Must be synchronous and cheap; expensive work belongs on
-   * a demand queue, not the parse path.
+   * its localID, expressID, typeID (0 for external-mapping records) and the
+   * record's bytes as an un-materialised `(buffer, byteOffset, byteLength)`
+   * window view — the seam for incremental semantic consumers (type index,
+   * roots registry, names skeleton). See {@link RecordEventHandler} for the
+   * window-lifetime rule. Must be synchronous and cheap; expensive work
+   * belongs on a demand queue, not the parse path.
    * @param onProgress Optional byte-cursor progress callback.
    * @return {BlockParseResult} The parsing result, including the index and result enum.
    */
@@ -18652,11 +18654,6 @@ var StepParser = class extends StepHeaderParser {
         if (!charws(SEMICOLON)) {
           return syntaxError();
         }
-        {
-          const recordLen = input.address - startElement2;
-          onRecordIndexed?.(topLevelCount, expressID, 0, input.buffer.subarray(input.cursor - recordLen, input.cursor));
-        }
-        ++topLevelCount;
         pushEntry({
           address: startElement2,
           length: input.address - startElement2,
@@ -18666,6 +18663,11 @@ var StepParser = class extends StepHeaderParser {
           inlineEntities: inlineElements,
           multiMapping: multiElements
         });
+        {
+          const recordLen = input.address - startElement2;
+          onRecordIndexed?.(topLevelCount, expressID, 0, input.buffer, input.cursor - recordLen, recordLen);
+        }
+        ++topLevelCount;
         multiElements = void 0;
         continue;
       }
@@ -18745,11 +18747,6 @@ var StepParser = class extends StepHeaderParser {
       if (!charws(SEMICOLON)) {
         return syntaxError();
       }
-      {
-        const recordLen = input.address - startElement;
-        onRecordIndexed?.(topLevelCount, expressID, foundItem, input.buffer.subarray(input.cursor - recordLen, input.cursor));
-      }
-      ++topLevelCount;
       pushEntry({
         address: startElement,
         length: input.address - startElement,
@@ -18757,6 +18754,11 @@ var StepParser = class extends StepHeaderParser {
         expressID,
         inlineEntities: inlineElements
       });
+      {
+        const recordLen = input.address - startElement;
+        onRecordIndexed?.(topLevelCount, expressID, foundItem, input.buffer, input.cursor - recordLen, recordLen);
+      }
+      ++topLevelCount;
     }
     return parseResult(ParseResult.INCOMPLETE);
   }
@@ -21434,9 +21436,17 @@ var StepModelBase = class _StepModelBase {
    * extract reads `IfcIndexedPolygonalFace.CoordIndex` as integers —
    * following those faces pinned ~78k records per PSB product. If
    * omitted, `!descend` children use this span path (test helper).
+   * @param claimed Optional out-set of the IDs *this call* claimed and
+   * awaited residency for — a strict subset of `seen`. When `seen` is
+   * shared across concurrent walks (a batch of products sharing mapped
+   * geometry), an ID another walk claimed is skipped here and is NOT
+   * resident yet from this call's point of view: that walk's pin holds
+   * the chunk once it lands, but its read is still in flight. Anything
+   * that must synchronously read a record's bytes after this returns
+   * has to iterate `claimed`, never `seen` (conway#526).
    * @return {Promise<Set<number>>} The local IDs that were visited.
    */
-  async ensureResidentClosureByLocalID(localID, extraLocalIDs, seen = /* @__PURE__ */ new Set(), descend, leafSpans, spanLeaf) {
+  async ensureResidentClosureByLocalID(localID, extraLocalIDs, seen = /* @__PURE__ */ new Set(), descend, leafSpans, spanLeaf, claimed) {
     if (!this.isSourceExternal) {
       return seen;
     }
@@ -21461,6 +21471,7 @@ var StepModelBase = class _StepModelBase {
         seen.add(id);
         this.pinByLocalID(id);
         wave.push(id);
+        claimed?.add(id);
       }
       frontier.length = 0;
       if (wave.length === 0) {

package/compiled/examples/validator-bundled.cjs

@@ -944,7 +944,7 @@ var EntityTypesIfcCount = 909;
 var entity_types_ifc_gen_default = EntityTypesIfc;
 
 // compiled/src/version/version.js
-var versionString = "Conway v1.528.1490";
+var versionString = "Conway v1.530.1503";
 
 // compiled/dependencies/conway-geom/interface/conway_geometry.js
 var wasmType = "";
@@ -3540,10 +3540,12 @@ var StepParser = class extends StepHeaderParser {
    * buffer; the callback may rebase the buffer's window in place. Must be
    * synchronous on this driver — a returned Promise is a caller bug.
    * @param onRecordIndexed Called as each top-level record is indexed, with
-   * its localID, expressID and typeID (0 for external-mapping records) — the
-   * seam for incremental semantic consumers (type index, roots registry,
-   * names skeleton). Must be synchronous and cheap; expensive work belongs on
-   * a demand queue, not the parse path.
+   * its localID, expressID, typeID (0 for external-mapping records) and the
+   * record's bytes as an un-materialised `(buffer, byteOffset, byteLength)`
+   * window view — the seam for incremental semantic consumers (type index,
+   * roots registry, names skeleton). See {@link RecordEventHandler} for the
+   * window-lifetime rule. Must be synchronous and cheap; expensive work
+   * belongs on a demand queue, not the parse path.
    * @param onProgress Optional byte-cursor progress callback.
    * @return {BlockParseResult} The parsing result, including the index and result enum.
    */
@@ -3827,11 +3829,6 @@ var StepParser = class extends StepHeaderParser {
         if (!charws(SEMICOLON)) {
           return syntaxError();
         }
-        {
-          const recordLen = input.address - startElement2;
-          onRecordIndexed?.(topLevelCount, expressID, 0, input.buffer.subarray(input.cursor - recordLen, input.cursor));
-        }
-        ++topLevelCount;
         pushEntry({
           address: startElement2,
           length: input.address - startElement2,
@@ -3841,6 +3838,11 @@ var StepParser = class extends StepHeaderParser {
           inlineEntities: inlineElements,
           multiMapping: multiElements
         });
+        {
+          const recordLen = input.address - startElement2;
+          onRecordIndexed?.(topLevelCount, expressID, 0, input.buffer, input.cursor - recordLen, recordLen);
+        }
+        ++topLevelCount;
         multiElements = void 0;
         continue;
       }
@@ -3920,11 +3922,6 @@ var StepParser = class extends StepHeaderParser {
       if (!charws(SEMICOLON)) {
         return syntaxError();
       }
-      {
-        const recordLen = input.address - startElement;
-        onRecordIndexed?.(topLevelCount, expressID, foundItem, input.buffer.subarray(input.cursor - recordLen, input.cursor));
-      }
-      ++topLevelCount;
       pushEntry({
         address: startElement,
         length: input.address - startElement,
@@ -3932,6 +3929,11 @@ var StepParser = class extends StepHeaderParser {
         expressID,
         inlineEntities: inlineElements
       });
+      {
+        const recordLen = input.address - startElement;
+        onRecordIndexed?.(topLevelCount, expressID, foundItem, input.buffer, input.cursor - recordLen, recordLen);
+      }
+      ++topLevelCount;
     }
     return parseResult2(ParseResult.INCOMPLETE);
   }
@@ -5810,9 +5812,17 @@ var StepModelBase = class _StepModelBase {
    * extract reads `IfcIndexedPolygonalFace.CoordIndex` as integers —
    * following those faces pinned ~78k records per PSB product. If
    * omitted, `!descend` children use this span path (test helper).
+   * @param claimed Optional out-set of the IDs *this call* claimed and
+   * awaited residency for — a strict subset of `seen`. When `seen` is
+   * shared across concurrent walks (a batch of products sharing mapped
+   * geometry), an ID another walk claimed is skipped here and is NOT
+   * resident yet from this call's point of view: that walk's pin holds
+   * the chunk once it lands, but its read is still in flight. Anything
+   * that must synchronously read a record's bytes after this returns
+   * has to iterate `claimed`, never `seen` (conway#526).
    * @return {Promise<Set<number>>} The local IDs that were visited.
    */
-  async ensureResidentClosureByLocalID(localID, extraLocalIDs, seen = /* @__PURE__ */ new Set(), descend, leafSpans, spanLeaf) {
+  async ensureResidentClosureByLocalID(localID, extraLocalIDs, seen = /* @__PURE__ */ new Set(), descend, leafSpans, spanLeaf, claimed) {
     if (!this.isSourceExternal) {
       return seen;
     }
@@ -5837,6 +5847,7 @@ var StepModelBase = class _StepModelBase {
         seen.add(id);
         this.pinByLocalID(id);
         wave.push(id);
+        claimed?.add(id);
       }
       frontier.length = 0;
       if (wave.length === 0) {
@@ -72152,13 +72163,11 @@ async function buildIndexStreamingAsync(source, parser211, pool3, onRecordIndexe
     };
   }
 }
-function buildColumnarIndexStreaming(source, parser211, pool3, onRecordIndexed) {
-  const sink = new ColumnarIndexSink();
+function buildColumnarIndexStreaming(source, parser211, pool3, onRecordIndexed, sink = new ColumnarIndexSink()) {
   const { header, result, stats } = buildIndexStreaming(source, parser211, pool3, onRecordIndexed, sink);
   return { header, columns: sink.finalize(), result, stats };
 }
-async function buildColumnarIndexStreamingAsync(source, parser211, pool3, onRecordIndexed, onProgress, yieldIntervalMs) {
-  const sink = new ColumnarIndexSink();
+async function buildColumnarIndexStreamingAsync(source, parser211, pool3, onRecordIndexed, onProgress, yieldIntervalMs, sink = new ColumnarIndexSink()) {
   const { header, result, stats } = await buildIndexStreamingAsync(source, parser211, pool3, onRecordIndexed, sink, onProgress, yieldIntervalMs);
   return { header, columns: sink.finalize(), result, stats };
 }