@ifc-lite/create 2.2.1 → 2.4.0

package/dist/in-store/drawing-markup.js

@@ -0,0 +1,196 @@
+/* This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at https://mozilla.org/MPL/2.0/. */
+/**
+ * Translate 2D drawing markup (issue #4153's "save into the model" option)
+ * into IFC entities via the additive `StoreEditor` overlay — modelled on
+ * `space.ts`'s `addSpaceToStore`.
+ *
+ * WRITE SIDE ONLY. This module has no UI wiring and does not import or edit
+ * `apps/viewer/src/store/slices/drawing2DSlice.ts` — a "save into model"
+ * action, and the read-side translator that recognises
+ * `DRAWING_MARKUP_OBJECTTYPE` and rehydrates `Drawing2DState`, are both
+ * future follow-ups. The param shapes below (`MarkupPoint2D`,
+ * `MeasureMarkupParams`, …) intentionally mirror `Drawing2DState`'s arrays
+ * structurally without importing `apps/viewer` — a published package
+ * (`@ifc-lite/create`) cannot depend on the app.
+ *
+ * Geometry frame: every point is treated as already resolved into the
+ * target `IfcGeometricRepresentationSubContext`'s 2D plane, in METRES —
+ * the same "drawing units (typically meters)" `Drawing2DState`'s own
+ * comments describe. Reprojecting a plan/section drawing's local 2D
+ * coordinates into that frame (accounting for the `SectionConfig` cut
+ * plane) is a UI/wiring concern for the future "save into model" action,
+ * not this translation layer.
+ *
+ * Additive by construction, same guarantee `StoreEditor.addEntity` documents:
+ * every call only APPENDS overlay entities (new `IfcAnnotation` +
+ * geometry + property/quantity sets); nothing existing is read, mutated,
+ * or renumbered.
+ */
+import { generateIfcGuid } from '@ifc-lite/encoding';
+import { toNativeLength } from './anchor.js';
+import { emitLocalPlacement, ownerHistoryRef } from './_emit-helpers.js';
+import { emitMarkupPolyline, emitMarkupPoint2DPlacement, emitMarkupSubContext, emitMarkupRepresentation, } from './drawing-markup-geometry.js';
+import { DRAWING_MARKUP_OBJECTTYPE, DRAWING_MARKUP_PSET_NAME, DRAWING_MARKUP_QSET_NAME } from './drawing-markup-tags.js';
+export { DRAWING_MARKUP_OBJECTTYPE, DRAWING_MARKUP_PSET_NAME, DRAWING_MARKUP_QSET_NAME, } from './drawing-markup-tags.js';
+/**
+ * A drawing-space coordinate may legitimately be negative or zero — the
+ * caller's own drawing origin, not this module's to constrain — so this only
+ * rejects NaN/Infinity, never a sign check. Same split `spatial-zone.ts`'s
+ * `validateZone` uses for its Footprint points, the closest analog: without
+ * it, a NaN/Infinity coordinate reaches `IfcCartesianPoint` as the literal
+ * `$` inside a mandatory-REAL attribute list — schema-invalid STEP written
+ * with no error.
+ */
+function assertFinitePoint(functionName, label, p) {
+    if (!Number.isFinite(p.x) || !Number.isFinite(p.y)) {
+        throw new Error(`${functionName}: ${label} needs a finite point`);
+    }
+}
+/**
+ * Distance/area/perimeter/extent are derived measurements: never negative,
+ * but legitimately zero (a degenerate measurement, e.g. a zero-length
+ * measure). So this rejects NaN/Infinity and negative values, not `<= 0`
+ * like `_emit-helpers.ts`'s `assertPositiveFinite` (which guards true
+ * dimensions — Width, Height — that can never be zero). Without this, a NaN
+ * silently becomes a zeroed quantity value on write (`toNativeLength(NaN)`
+ * multiplies through to `NaN`, which the STEP writer would still need to
+ * catch) rather than failing loudly here.
+ */
+function assertFiniteNonNegative(functionName, label, value) {
+    if (!Number.isFinite(value) || value < 0) {
+        throw new Error(`${functionName}: ${label} must be finite and non-negative`);
+    }
+}
+/** IfcAnnotation attribute order: GlobalId, OwnerHistory, Name, Description,
+ *  ObjectType, ObjectPlacement, Representation — same 7-slot shape across
+ *  IFC2X3/IFC4/IFC4X3 (schema-registry.ts `IfcAnnotation.allAttributes`). */
+function emitAnnotation(editor, anchor, name, objectType, placementId, productShapeId) {
+    return editor.addEntity('IfcAnnotation', [
+        generateIfcGuid(anchor.guidRandom),
+        ownerHistoryRef(anchor.ownerHistoryId),
+        name,
+        null,
+        objectType,
+        `#${placementId}`,
+        `#${productShapeId}`,
+    ]).expressId;
+}
+/** A polyline annotation for one `Measure2DResult`. Distance is preserved as
+ *  a LENGTH quantity (native length unit, matching every other in-store
+ *  builder's LENGTH convention — see `space.ts`'s `GrossPerimeter`) rather
+ *  than recomputed from the two endpoints on read-back. */
+export function addMeasureMarkupToStore(editor, anchor, contextId, params) {
+    assertFinitePoint('addMeasureMarkupToStore', 'start', params.start);
+    assertFinitePoint('addMeasureMarkupToStore', 'end', params.end);
+    assertFiniteNonNegative('addMeasureMarkupToStore', 'distance', params.distance);
+    const n = (metres) => toNativeLength(anchor, metres);
+    const placementId = emitLocalPlacement(editor, anchor.storeyPlacementId, [0, 0, 0]);
+    const polylineId = emitMarkupPolyline(editor, [[n(params.start.x), n(params.start.y)], [n(params.end.x), n(params.end.y)]], false);
+    const { shapeRepId, productShapeId } = emitMarkupRepresentation(editor, contextId, 'Curve2D', polylineId);
+    const annotationId = emitAnnotation(editor, anchor, params.Name ?? 'Measurement', DRAWING_MARKUP_OBJECTTYPE.MEASURE, placementId, productShapeId);
+    editor.addQuantitySet(annotationId, DRAWING_MARKUP_QSET_NAME, [
+        { name: 'Distance', value: n(params.distance), quantityType: 'LENGTH' },
+    ]);
+    return { annotationId, placementId, polylineId, shapeRepId, productShapeId };
+}
+/** A closed polyline annotation for one `PolygonArea2DResult`. Area/perimeter
+ *  preserved so a reader doesn't recompute a shoelace sum from re-projected
+ *  points (area is SI m², not unit-scaled — same split `space.ts` uses for
+ *  AREA vs LENGTH quantities). */
+export function addPolygonAreaMarkupToStore(editor, anchor, contextId, params) {
+    if (params.points.length < 3) {
+        throw new Error('addPolygonAreaMarkupToStore: needs at least 3 points');
+    }
+    for (const [i, point] of params.points.entries()) {
+        assertFinitePoint('addPolygonAreaMarkupToStore', `points[${i}]`, point);
+    }
+    assertFiniteNonNegative('addPolygonAreaMarkupToStore', 'area', params.area);
+    assertFiniteNonNegative('addPolygonAreaMarkupToStore', 'perimeter', params.perimeter);
+    const n = (metres) => toNativeLength(anchor, metres);
+    const placementId = emitLocalPlacement(editor, anchor.storeyPlacementId, [0, 0, 0]);
+    const polylineId = emitMarkupPolyline(editor, params.points.map((p) => [n(p.x), n(p.y)]), true);
+    const { shapeRepId, productShapeId } = emitMarkupRepresentation(editor, contextId, 'Curve2D', polylineId);
+    const annotationId = emitAnnotation(editor, anchor, params.Name ?? 'Area', DRAWING_MARKUP_OBJECTTYPE.POLYGON_AREA, placementId, productShapeId);
+    editor.addQuantitySet(annotationId, DRAWING_MARKUP_QSET_NAME, [
+        { name: 'Area', value: params.area, quantityType: 'AREA' },
+        { name: 'Perimeter', value: n(params.perimeter), quantityType: 'LENGTH' },
+    ]);
+    return { annotationId, placementId, polylineId, shapeRepId, productShapeId };
+}
+/** An `IfcTextLiteralWithExtent` annotation for one `TextAnnotation2D`. The
+ *  literal string is the derived value here — it lives in `Literal` itself,
+ *  the same field the read-side symbolic parser already consumes, so no
+ *  separate property-set copy is needed (unlike distance/area/perimeter,
+ *  which have nowhere else to live). `fontSize`/colors are display-only
+ *  presentation state, not a derived value this issue asks to preserve, and
+ *  are left for a future presentation-style follow-up. */
+export function addTextMarkupToStore(editor, anchor, contextId, params) {
+    assertFinitePoint('addTextMarkupToStore', 'position', params.position);
+    const extent = params.extent ?? { sizeX: 1, sizeY: 0.25 };
+    assertFiniteNonNegative('addTextMarkupToStore', 'extent.sizeX', extent.sizeX);
+    assertFiniteNonNegative('addTextMarkupToStore', 'extent.sizeY', extent.sizeY);
+    const n = (metres) => toNativeLength(anchor, metres);
+    const placementId = emitLocalPlacement(editor, anchor.storeyPlacementId, [0, 0, 0]);
+    const textPlacementId = emitMarkupPoint2DPlacement(editor, [n(params.position.x), n(params.position.y)]);
+    const extentId = editor.addEntity('IfcPlanarExtent', [n(extent.sizeX), n(extent.sizeY)]).expressId;
+    const textLiteralId = editor.addEntity('IfcTextLiteralWithExtent', [
+        params.text,
+        `#${textPlacementId}`,
+        '.RIGHT.',
+        `#${extentId}`,
+        params.boxAlignment ?? 'top-left',
+    ]).expressId;
+    const { shapeRepId, productShapeId } = emitMarkupRepresentation(editor, contextId, 'Annotation2D', textLiteralId);
+    const annotationId = emitAnnotation(editor, anchor, params.Name ?? 'Note', DRAWING_MARKUP_OBJECTTYPE.TEXT, placementId, productShapeId);
+    return { annotationId, placementId, textLiteralId, shapeRepId, productShapeId };
+}
+/**
+ * An `IfcAnnotationFillArea` annotation for one `CloudAnnotation2D`.
+ *
+ * Chose fill-area over a bare polyline because a revision cloud is
+ * semantically a bounded REGION (its two stored points are a rectangle's
+ * corners, not an open path), and `IfcAnnotationFillArea` is what the read
+ * side's own symbolic parser already recognises as a fill (`AnnotationFill2D`
+ * in `apps/viewer/src/lib/overlay-parse/symbolic-shapes.ts`) — a bare
+ * polyline would render as an outline only, losing the "this is a cloud
+ * region" distinction a reader needs to tell it apart from a measurement.
+ */
+export function addCloudMarkupToStore(editor, anchor, contextId, params) {
+    const [topLeft, bottomRight] = params.points;
+    assertFinitePoint('addCloudMarkupToStore', 'points[0]', topLeft);
+    assertFinitePoint('addCloudMarkupToStore', 'points[1]', bottomRight);
+    const n = (metres) => toNativeLength(anchor, metres);
+    const corners = [
+        [n(topLeft.x), n(topLeft.y)],
+        [n(bottomRight.x), n(topLeft.y)],
+        [n(bottomRight.x), n(bottomRight.y)],
+        [n(topLeft.x), n(bottomRight.y)],
+    ];
+    const placementId = emitLocalPlacement(editor, anchor.storeyPlacementId, [0, 0, 0]);
+    const polylineId = emitMarkupPolyline(editor, corners, true);
+    const fillAreaId = editor.addEntity('IfcAnnotationFillArea', [`#${polylineId}`, null]).expressId;
+    const { shapeRepId, productShapeId } = emitMarkupRepresentation(editor, contextId, 'Annotation2D', fillAreaId);
+    const annotationId = emitAnnotation(editor, anchor, params.Name ?? 'Cloud', DRAWING_MARKUP_OBJECTTYPE.CLOUD, placementId, productShapeId);
+    editor.addPropertySet(annotationId, DRAWING_MARKUP_PSET_NAME, [
+        { name: 'Label', value: params.label, type: 'TEXT' },
+    ]);
+    return { annotationId, placementId, polylineId, fillAreaId, shapeRepId, productShapeId };
+}
+/**
+ * Translate a full `Drawing2DState` markup snapshot into the store overlay:
+ * one shared `Annotation` `IfcGeometricRepresentationSubContext` (created
+ * fresh under `rootContextId`, the model's root 3D
+ * `IfcGeometricRepresentationContext`), then one `IfcAnnotation` per markup
+ * item, tagged per {@link DRAWING_MARKUP_OBJECTTYPE}.
+ */
+export function addDrawingMarkupToStore(editor, anchor, rootContextId, input, targetView = 'PLAN_VIEW') {
+    const subContextId = emitMarkupSubContext(editor, rootContextId, targetView);
+    const measures = (input.measure2DResults ?? []).map((m) => addMeasureMarkupToStore(editor, anchor, subContextId, m));
+    const polygons = (input.polygonArea2DResults ?? []).map((p) => addPolygonAreaMarkupToStore(editor, anchor, subContextId, p));
+    const texts = (input.textAnnotations2D ?? []).map((t) => addTextMarkupToStore(editor, anchor, subContextId, t));
+    const clouds = (input.cloudAnnotations2D ?? []).map((c) => addCloudMarkupToStore(editor, anchor, subContextId, c));
+    return { subContextId, measures, polygons, texts, clouds };
+}
+//# sourceMappingURL=drawing-markup.js.map

package/dist/in-store/drawing-markup.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"drawing-markup.js","sourceRoot":"","sources":["../../src/in-store/drawing-markup.ts"],"names":[],"mappings":"AAAA;;+DAE+D;AAE/D;;;;;;;;;;;;;;;;;;;;;;;;;;GA0BG;AAEH,OAAO,EAAE,eAAe,EAAqB,MAAM,oBAAoB,CAAC;AAGxE,OAAO,EAAE,cAAc,EAAE,MAAM,aAAa,CAAC;AAC7C,OAAO,EAAE,kBAAkB,EAAE,eAAe,EAAE,MAAM,oBAAoB,CAAC;AACzE,OAAO,EACL,kBAAkB,EAClB,0BAA0B,EAC1B,oBAAoB,EACpB,wBAAwB,GACzB,MAAM,8BAA8B,CAAC;AACtC,OAAO,EAAE,yBAAyB,EAAE,wBAAwB,EAAE,wBAAwB,EAAE,MAAM,0BAA0B,CAAC;AAEzH,OAAO,EACL,yBAAyB,EACzB,wBAAwB,EACxB,wBAAwB,GAEzB,MAAM,0BAA0B,CAAC;AAiGlC;;;;;;;;GAQG;AACH,SAAS,iBAAiB,CAAC,YAAoB,EAAE,KAAa,EAAE,CAAgB;IAC9E,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;QACnD,MAAM,IAAI,KAAK,CAAC,GAAG,YAAY,KAAK,KAAK,uBAAuB,CAAC,CAAC;IACpE,CAAC;AACH,CAAC;AAED;;;;;;;;;GASG;AACH,SAAS,uBAAuB,CAAC,YAAoB,EAAE,KAAa,EAAE,KAAa;IACjF,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,IAAI,KAAK,GAAG,CAAC,EAAE,CAAC;QACzC,MAAM,IAAI,KAAK,CAAC,GAAG,YAAY,KAAK,KAAK,kCAAkC,CAAC,CAAC;IAC/E,CAAC;AACH,CAAC;AAED;;6EAE6E;AAC7E,SAAS,cAAc,CACrB,MAAmB,EACnB,MAAoB,EACpB,IAAY,EACZ,UAAkB,EAClB,WAAmB,EACnB,cAAsB;IAEtB,OAAO,MAAM,CAAC,SAAS,CAAC,eAAe,EAAE;QACvC,eAAe,CAAC,MAAM,CAAC,UAAU,CAAC;QAClC,eAAe,CAAC,MAAM,CAAC,cAAc,CAAC;QACtC,IAAI;QACJ,IAAI;QACJ,UAAU;QACV,IAAI,WAAW,EAAE;QACjB,IAAI,cAAc,EAAE;KACrB,CAAC,CAAC,SAAS,CAAC;AACf,CAAC;AAED;;;2DAG2D;AAC3D,MAAM,UAAU,uBAAuB,CACrC,MAAmB,EACnB,MAAoB,EACpB,SAAiB,EACjB,MAA2B;IAE3B,iBAAiB,CAAC,yBAAyB,EAAE,OAAO,EAAE,MAAM,CAAC,KAAK,CAAC,CAAC;IACpE,iBAAiB,CAAC,yBAAyB,EAAE,KAAK,EAAE,MAAM,CAAC,GAAG,CAAC,CAAC;IAChE,uBAAuB,CAAC,yBAAyB,EAAE,UAAU,EAAE,MAAM,CAAC,QAAQ,CAAC,CAAC;IAChF,MAAM,CAAC,GAAG,CAAC,MAAc,EAAE,EAAE,CAAC,cAAc,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAC7D,MAAM,WAAW,GAAG,kBAAkB,CAAC,MAAM,EAAE,MAAM,CAAC,iBAAiB,EAAE,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC;IACpF,MAAM,UAAU,GAAG,kBAAkB,CACnC,MAAM,EACN,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,EAC5E,KAAK,CACN,CAAC;IACF,MAAM,EAAE,UAAU,EAAE,cAAc,EAAE,GAAG,wBAAwB,CAAC,MAAM,EAAE,SAAS,EAAE,SAAS,EAAE,UAAU,CAAC,CAAC;IAC1G,MAAM,YAAY,GAAG,cAAc,CACjC,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,IAAI,IAAI,aAAa,EAAE,yBAAyB,CAAC,OAAO,EAAE,WAAW,EAAE,cAAc,CAC7G,CAAC;IACF,MAAM,CAAC,cAAc,CAAC,YAAY,EAAE,wBAAwB,EAAE;QAC5D,EAAE,IAAI,EAAE,UAAU,EAAE,KAAK,EAAE,CAAC,CAAC,MAAM,CAAC,QAAQ,CAAC,EAAE,YAAY,EAAE,QAAQ,EAAE;KACxE,CAAC,CAAC;IACH,OAAO,EAAE,YAAY,EAAE,WAAW,EAAE,UAAU,EAAE,UAAU,EAAE,cAAc,EAAE,CAAC;AAC/E,CAAC;AAED;;;kCAGkC;AAClC,MAAM,UAAU,2BAA2B,CACzC,MAAmB,EACnB,MAAoB,EACpB,SAAiB,EACjB,MAA+B;IAE/B,IAAI,MAAM,CAAC,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC7B,MAAM,IAAI,KAAK,CAAC,sDAAsD,CAAC,CAAC;IAC1E,CAAC;IACD,KAAK,MAAM,CAAC,CAAC,EAAE,KAAK,CAAC,IAAI,MAAM,CAAC,MAAM,CAAC,OAAO,EAAE,EAAE,CAAC;QACjD,iBAAiB,CAAC,6BAA6B,EAAE,UAAU,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC;IAC1E,CAAC;IACD,uBAAuB,CAAC,6BAA6B,EAAE,MAAM,EAAE,MAAM,CAAC,IAAI,CAAC,CAAC;IAC5E,uBAAuB,CAAC,6BAA6B,EAAE,WAAW,EAAE,MAAM,CAAC,SAAS,CAAC,CAAC;IACtF,MAAM,CAAC,GAAG,CAAC,MAAc,EAAE,EAAE,CAAC,cAAc,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAC7D,MAAM,WAAW,GAAG,kBAAkB,CAAC,MAAM,EAAE,MAAM,CAAC,iBAAiB,EAAE,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC;IACpF,MAAM,UAAU,GAAG,kBAAkB,CAAC,MAAM,EAAE,MAAM,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,EAAoB,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,EAAE,IAAI,CAAC,CAAC;IAClH,MAAM,EAAE,UAAU,EAAE,cAAc,EAAE,GAAG,wBAAwB,CAAC,MAAM,EAAE,SAAS,EAAE,SAAS,EAAE,UAAU,CAAC,CAAC;IAC1G,MAAM,YAAY,GAAG,cAAc,CACjC,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,IAAI,IAAI,MAAM,EAAE,yBAAyB,CAAC,YAAY,EAAE,WAAW,EAAE,cAAc,CAC3G,CAAC;IACF,MAAM,CAAC,cAAc,CAAC,YAAY,EAAE,wBAAwB,EAAE;QAC5D,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,CAAC,IAAI,EAAE,YAAY,EAAE,MAAM,EAAE;QAC1D,EAAE,IAAI,EAAE,WAAW,EAAE,KAAK,EAAE,CAAC,CAAC,MAAM,CAAC,SAAS,CAAC,EAAE,YAAY,EAAE,QAAQ,EAAE;KAC1E,CAAC,CAAC;IACH,OAAO,EAAE,YAAY,EAAE,WAAW,EAAE,UAAU,EAAE,UAAU,EAAE,cAAc,EAAE,CAAC;AAC/E,CAAC;AAED;;;;;;0DAM0D;AAC1D,MAAM,UAAU,oBAAoB,CAClC,MAAmB,EACnB,MAAoB,EACpB,SAAiB,EACjB,MAAwB;IAExB,iBAAiB,CAAC,sBAAsB,EAAE,UAAU,EAAE,MAAM,CAAC,QAAQ,CAAC,CAAC;IACvE,MAAM,MAAM,GAAG,MAAM,CAAC,MAAM,IAAI,EAAE,KAAK,EAAE,CAAC,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC;IAC1D,uBAAuB,CAAC,sBAAsB,EAAE,cAAc,EAAE,MAAM,CAAC,KAAK,CAAC,CAAC;IAC9E,uBAAuB,CAAC,sBAAsB,EAAE,cAAc,EAAE,MAAM,CAAC,KAAK,CAAC,CAAC;IAC9E,MAAM,CAAC,GAAG,CAAC,MAAc,EAAE,EAAE,CAAC,cAAc,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAC7D,MAAM,WAAW,GAAG,kBAAkB,CAAC,MAAM,EAAE,MAAM,CAAC,iBAAiB,EAAE,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC;IACpF,MAAM,eAAe,GAAG,0BAA0B,CAAC,MAAM,EAAE,CAAC,CAAC,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;IACzG,MAAM,QAAQ,GAAG,MAAM,CAAC,SAAS,CAAC,iBAAiB,EAAE,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC;IACnG,MAAM,aAAa,GAAG,MAAM,CAAC,SAAS,CAAC,0BAA0B,EAAE;QACjE,MAAM,CAAC,IAAI;QACX,IAAI,eAAe,EAAE;QACrB,SAAS;QACT,IAAI,QAAQ,EAAE;QACd,MAAM,CAAC,YAAY,IAAI,UAAU;KAClC,CAAC,CAAC,SAAS,CAAC;IACb,MAAM,EAAE,UAAU,EAAE,cAAc,EAAE,GAAG,wBAAwB,CAAC,MAAM,EAAE,SAAS,EAAE,cAAc,EAAE,aAAa,CAAC,CAAC;IAClH,MAAM,YAAY,GAAG,cAAc,CACjC,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,IAAI,IAAI,MAAM,EAAE,yBAAyB,CAAC,IAAI,EAAE,WAAW,EAAE,cAAc,CACnG,CAAC;IACF,OAAO,EAAE,YAAY,EAAE,WAAW,EAAE,aAAa,EAAE,UAAU,EAAE,cAAc,EAAE,CAAC;AAClF,CAAC;AAED;;;;;;;;;;GAUG;AACH,MAAM,UAAU,qBAAqB,CACnC,MAAmB,EACnB,MAAoB,EACpB,SAAiB,EACjB,MAAyB;IAEzB,MAAM,CAAC,OAAO,EAAE,WAAW,CAAC,GAAG,MAAM,CAAC,MAAM,CAAC;IAC7C,iBAAiB,CAAC,uBAAuB,EAAE,WAAW,EAAE,OAAO,CAAC,CAAC;IACjE,iBAAiB,CAAC,uBAAuB,EAAE,WAAW,EAAE,WAAW,CAAC,CAAC;IACrE,MAAM,CAAC,GAAG,CAAC,MAAc,EAAE,EAAE,CAAC,cAAc,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAC7D,MAAM,OAAO,GAA4B;QACvC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC;QAC5B,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC;QAChC,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC;QACpC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC;KACjC,CAAC;IACF,MAAM,WAAW,GAAG,kBAAkB,CAAC,MAAM,EAAE,MAAM,CAAC,iBAAiB,EAAE,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC;IACpF,MAAM,UAAU,GAAG,kBAAkB,CAAC,MAAM,EAAE,OAAO,EAAE,IAAI,CAAC,CAAC;IAC7D,MAAM,UAAU,GAAG,MAAM,CAAC,SAAS,CAAC,uBAAuB,EAAE,CAAC,IAAI,UAAU,EAAE,EAAE,IAAI,CAAC,CAAC,CAAC,SAAS,CAAC;IACjG,MAAM,EAAE,UAAU,EAAE,cAAc,EAAE,GAAG,wBAAwB,CAAC,MAAM,EAAE,SAAS,EAAE,cAAc,EAAE,UAAU,CAAC,CAAC;IAC/G,MAAM,YAAY,GAAG,cAAc,CACjC,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,IAAI,IAAI,OAAO,EAAE,yBAAyB,CAAC,KAAK,EAAE,WAAW,EAAE,cAAc,CACrG,CAAC;IACF,MAAM,CAAC,cAAc,CAAC,YAAY,EAAE,wBAAwB,EAAE;QAC5D,EAAE,IAAI,EAAE,OAAO,EAAE,KAAK,EAAE,MAAM,CAAC,KAAK,EAAE,IAAI,EAAE,MAAM,EAAE;KACrD,CAAC,CAAC;IACH,OAAO,EAAE,YAAY,EAAE,WAAW,EAAE,UAAU,EAAE,UAAU,EAAE,UAAU,EAAE,cAAc,EAAE,CAAC;AAC3F,CAAC;AAsBD;;;;;;GAMG;AACH,MAAM,UAAU,uBAAuB,CACrC,MAAmB,EACnB,MAAoB,EACpB,aAAqB,EACrB,KAA8B,EAC9B,aAA2C,WAAW;IAEtD,MAAM,YAAY,GAAG,oBAAoB,CAAC,MAAM,EAAE,aAAa,EAAE,UAAU,CAAC,CAAC;IAC7E,MAAM,QAAQ,GAAG,CAAC,KAAK,CAAC,gBAAgB,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,uBAAuB,CAAC,MAAM,EAAE,MAAM,EAAE,YAAY,EAAE,CAAC,CAAC,CAAC,CAAC;IACrH,MAAM,QAAQ,GAAG,CAAC,KAAK,CAAC,oBAAoB,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,2BAA2B,CAAC,MAAM,EAAE,MAAM,EAAE,YAAY,EAAE,CAAC,CAAC,CAAC,CAAC;IAC7H,MAAM,KAAK,GAAG,CAAC,KAAK,CAAC,iBAAiB,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,oBAAoB,CAAC,MAAM,EAAE,MAAM,EAAE,YAAY,EAAE,CAAC,CAAC,CAAC,CAAC;IAChH,MAAM,MAAM,GAAG,CAAC,KAAK,CAAC,kBAAkB,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,qBAAqB,CAAC,MAAM,EAAE,MAAM,EAAE,YAAY,EAAE,CAAC,CAAC,CAAC,CAAC;IACnH,OAAO,EAAE,YAAY,EAAE,QAAQ,EAAE,QAAQ,EAAE,KAAK,EAAE,MAAM,EAAE,CAAC;AAC7D,CAAC"}

package/dist/in-store/extract-walls.d.ts

@@ -25,23 +25,10 @@
  * callers (and the viewer's Auto Spaces UI) can surface why a wall
  * didn't contribute to the planar graph.
  */
-import { type IfcDataStore, type IfcAttributeValue } from '@ifc-lite/parser';
+import { type IfcDataStore } from '@ifc-lite/parser';
 import type { Segment, Vec2 } from './auto-space-detect.js';
-/**
- * Optional overlay reader. If supplied, overlay walls (entities
- * created via `editor.addEntity('IfcWall', ...)` since the model was
- * parsed) are included alongside the source walls.
- */
-export interface OverlayWallReader {
-    /** Iterate every overlay-created entity. */
-    getNewEntities(): Iterable<{
-        expressId: number;
-        type: string;
-        attributes: IfcAttributeValue[];
-    }>;
-    /** Resolve a positional attribute (with mutations applied). */
-    getAttribute?(expressId: number, index: number): IfcAttributeValue | undefined;
-}
+import { type OverlayWallReader } from './placement-frame.js';
+export type { OverlayWallReader };
 export type WallSkipReason = 'no-source-bytes' | 'wall-not-parsed' | 'no-placement' | 'no-representation' | 'placement-not-resolvable' | 'no-axis-or-rect-profile' | 'zero-length-axis' | 'sloped-axis';
 export interface WallSkip {
     wallId: number;
@@ -90,6 +77,19 @@ export declare function extractWallSegmentsForStorey(store: IfcDataStore, storey
  * the new rooms that overlap an already-present space (per-space dedup), while
  * still adding rooms an empty part of the floor lacks. Keyed by storey
  * expressId; storeys with no resolvable space footprints are omitted.
+ *
+ * If the model's length-unit scale itself can't be trusted, this refuses
+ * for the WHOLE store (empty map) rather than dividing every footprint by a
+ * wrong factor: `storeyPlanFrame` — the write side of the same round-trip
+ * (`useSpaceBake.ts` folds a new room through it, then reads existing
+ * footprints back through this function to dedup against) — refuses the
+ * same way per storey. A silent `?? 1` here on a genuinely degenerate scale
+ * would have scaled every existing footprint to a value the write side had
+ * already refused to trust, so the dedup compare would run on stale
+ * coordinates with no warning that anything was wrong. An empty map means
+ * dedup is skipped, matching the existing `--force` behaviour of this same
+ * call site (`generateSpaces` in `generate-spaces-all.ts`) rather than
+ * emitting mis-scaled polygons.
  */
 export declare function existingSpaceFootprintsByStorey(store: IfcDataStore): Map<number, Vec2[][]>;
 /**

package/dist/in-store/extract-walls.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"extract-walls.d.ts","sourceRoot":"","sources":["../../src/in-store/extract-walls.ts"],"names":[],"mappings":"AAIA;;;;;;;;;;;;;;;;;;;;;;;;;;GA0BG;AAEH,OAAO,EAIL,KAAK,YAAY,EACjB,KAAK,iBAAiB,EACvB,MAAM,kBAAkB,CAAC;AAC1B,OAAO,KAAK,EAAE,OAAO,EAAE,IAAI,EAAE,MAAM,wBAAwB,CAAC;AAE5D;;;;GAIG;AACH,MAAM,WAAW,iBAAiB;IAChC,4CAA4C;IAC5C,cAAc,IAAI,QAAQ,CAAC;QAAE,SAAS,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAC;QAAC,UAAU,EAAE,iBAAiB,EAAE,CAAA;KAAE,CAAC,CAAC;IACjG,+DAA+D;IAC/D,YAAY,CAAC,CAAC,SAAS,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,GAAG,iBAAiB,GAAG,SAAS,CAAC;CAChF;AAED,MAAM,MAAM,cAAc,GACtB,iBAAiB,GACjB,iBAAiB,GACjB,cAAc,GACd,mBAAmB,GACnB,0BAA0B,GAC1B,yBAAyB,GACzB,kBAAkB,GAClB,aAAa,CAAC;AAElB,MAAM,WAAW,QAAQ;IACvB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,cAAc,CAAC;CACxB;AAED,MAAM,WAAW,oBAAoB;IACnC,QAAQ,EAAE,OAAO,EAAE,CAAC;IACpB,wDAAwD;IACxD,mBAAmB,EAAE,MAAM,EAAE,CAAC;IAC9B;;;;;OAKG;IACH,eAAe,EAAE,CAAC,MAAM,GAAG,SAAS,CAAC,EAAE,CAAC;IACxC,uDAAuD;IACvD,OAAO,EAAE,QAAQ,EAAE,CAAC;IACpB,+EAA+E;IAC/E,UAAU,EAAE,MAAM,CAAC;IACnB;;;;OAIG;IACH,eAAe,EAAE,MAAM,CAAC;CACzB;AAED,MAAM,WAAW,0BAA0B;IACzC;;;;OAIG;IACH,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB;;;;OAIG;IACH,iBAAiB,CAAC,EAAE,MAAM,EAAE,CAAC;CAC9B;AAuBD,wBAAgB,4BAA4B,CAC1C,KAAK,EAAE,YAAY,EACnB,eAAe,EAAE,MAAM,EACvB,OAAO,CAAC,EAAE,iBAAiB,EAC3B,OAAO,GAAE,0BAA+B,GACvC,oBAAoB,CA6GtB;AAgGD;;;;;;GAMG;AACH,wBAAgB,+BAA+B,CAAC,KAAK,EAAE,YAAY,GAAG,GAAG,CAAC,MAAM,EAAE,IAAI,EAAE,EAAE,CAAC,CAiC1F;AA6sBD;;;;;;;;GAQG;AACH,wBAAgB,qBAAqB,CACnC,KAAK,EAAE,YAAY,EACnB,MAAM,EAAE;IAAE,IAAI,CAAC,EAAE,MAAM,CAAA;CAAE,EACzB,QAAQ,EAAE,MAAM,GACf,MAAM,CAIR"}
+{"version":3,"file":"extract-walls.d.ts","sourceRoot":"","sources":["../../src/in-store/extract-walls.ts"],"names":[],"mappings":"AAIA;;;;;;;;;;;;;;;;;;;;;;;;;;GA0BG;AAEH,OAAO,EAGL,KAAK,YAAY,EAElB,MAAM,kBAAkB,CAAC;AAC1B,OAAO,KAAK,EAAE,OAAO,EAAE,IAAI,EAAE,MAAM,wBAAwB,CAAC;AAE5D,OAAO,EAQL,KAAK,iBAAiB,EACvB,MAAM,sBAAsB,CAAC;AAE9B,YAAY,EAAE,iBAAiB,EAAE,CAAC;AAElC,MAAM,MAAM,cAAc,GACtB,iBAAiB,GACjB,iBAAiB,GACjB,cAAc,GACd,mBAAmB,GACnB,0BAA0B,GAC1B,yBAAyB,GACzB,kBAAkB,GAClB,aAAa,CAAC;AAElB,MAAM,WAAW,QAAQ;IACvB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,cAAc,CAAC;CACxB;AAED,MAAM,WAAW,oBAAoB;IACnC,QAAQ,EAAE,OAAO,EAAE,CAAC;IACpB,wDAAwD;IACxD,mBAAmB,EAAE,MAAM,EAAE,CAAC;IAC9B;;;;;OAKG;IACH,eAAe,EAAE,CAAC,MAAM,GAAG,SAAS,CAAC,EAAE,CAAC;IACxC,uDAAuD;IACvD,OAAO,EAAE,QAAQ,EAAE,CAAC;IACpB,+EAA+E;IAC/E,UAAU,EAAE,MAAM,CAAC;IACnB;;;;OAIG;IACH,eAAe,EAAE,MAAM,CAAC;CACzB;AAED,MAAM,WAAW,0BAA0B;IACzC;;;;OAIG;IACH,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB;;;;OAIG;IACH,iBAAiB,CAAC,EAAE,MAAM,EAAE,CAAC;CAC9B;AAqBD,wBAAgB,4BAA4B,CAC1C,KAAK,EAAE,YAAY,EACnB,eAAe,EAAE,MAAM,EACvB,OAAO,CAAC,EAAE,iBAAiB,EAC3B,OAAO,GAAE,0BAA+B,GACvC,oBAAoB,CA+FtB;AAgGD;;;;;;;;;;;;;;;;;;;GAmBG;AACH,wBAAgB,+BAA+B,CAAC,KAAK,EAAE,YAAY,GAAG,GAAG,CAAC,MAAM,EAAE,IAAI,EAAE,EAAE,CAAC,CAkC1F;AA0dD;;;;;;;;GAQG;AACH,wBAAgB,qBAAqB,CACnC,KAAK,EAAE,YAAY,EACnB,MAAM,EAAE;IAAE,IAAI,CAAC,EAAE,MAAM,CAAA;CAAE,EACzB,QAAQ,EAAE,MAAM,GACf,MAAM,CAIR"}

package/dist/in-store/extract-walls.js

@@ -28,7 +28,9 @@
  * callers (and the viewer's Auto Spaces UI) can surface why a wall
  * didn't contribute to the planar graph.
  */
-import { EntityExtractor, extractLengthUnitScale, extractMaterialsOnDemand, } from '@ifc-lite/parser';
+import { EntityExtractor, extractMaterialsOnDemand, } from '@ifc-lite/parser';
+import { safeLengthUnitScale } from './length-unit-scale.js';
+import { AXIS_EPS, applyFrame, frameInStoreyFrame, numericAttr, readEntity, readVec3, storeyPlacementChain, } from './placement-frame.js';
 /**
  * Element types treated as "wall-like" dividers by default. Extends
  * the obvious walls with curtain walls (often the only divider on a
@@ -47,7 +49,6 @@ const DEFAULT_DIVIDER_TYPES = new Set([
     'ifcmember',
     'ifcrailing',
 ]);
-const AXIS_EPS = 1e-6;
 export function extractWallSegmentsForStorey(store, storeyExpressId, overlay, options = {}) {
     const segments = [];
     const contributing = [];
@@ -59,31 +60,18 @@ export function extractWallSegmentsForStorey(store, storeyExpressId, overlay, op
     // detector are always in METRES — without this a millimetre model
     // would produce coords like (31614, 23345) and the panel's
     // metre-based snap tolerance would be effectively zero.
-    let lengthUnitScale = 1.0;
-    if (store.source) {
-        try {
-            lengthUnitScale = extractLengthUnitScale(store.source, store.entityIndex);
-            if (!Number.isFinite(lengthUnitScale) || lengthUnitScale <= 0)
-                lengthUnitScale = 1.0;
-        }
-        catch (error) {
-            // Keep the metre fallback, but don't hide a THROWN failure — a wrong
-            // scale silently mis-scales every extracted segment (a millimetre model
-            // read as metres yields coords like 31614, collapsing the snap
-            // tolerance). Mirrors resolve-anchor.ts / resolve-source.ts.
-            //
-            // Deliberately NOT claiming the case is covered: extractLengthUnitScale
-            // returns 1.0 in band rather than throwing, so a missing IFCPROJECT or
-            // UnitsInContext, or a malformed unit declaration, never reaches this
-            // catch — it sees only a THROWN failure. Those paths are no longer silent
-            // (#2104: `warnUnknownUnit` in parser/src/unit-extractor.ts warns once
-            // per model). Still NOT supported: telling "unknown" from "genuinely
-            // metres" here — both arrive as 1.0, and a warning is not branchable.
-            // That needs null from a function with 7+ callers; not done, anywhere.
-            console.warn('extractWallSegmentsForStorey: failed to extract length unit scale; defaulting to metres', error);
-            lengthUnitScale = 1.0;
-        }
-    }
+    // Deliberately NOT claiming the case is covered: extractLengthUnitScale
+    // returns 1.0 in band rather than throwing, so a missing IFCPROJECT or
+    // UnitsInContext, or a malformed unit declaration, never reaches
+    // `safeLengthUnitScale`'s catch — it sees only a THROWN failure or a
+    // non-finite/non-positive return. Those paths are no longer silent
+    // (#2104: `warnUnknownUnit` in parser/src/unit-extractor.ts warns once
+    // per model). Still NOT supported: telling "unknown" from "genuinely
+    // metres" here — both arrive as 1.0, and a warning is not branchable.
+    // That needs null from a function with 7+ callers; not done, anywhere.
+    const lengthUnitScale = store.source
+        ? safeLengthUnitScale(store.source, store.entityIndex, 'extractWallSegmentsForStorey') ?? 1.0
+        : 1.0;
     log(`length unit scale = ${lengthUnitScale} (raw → metres)`);
     if (!store.source) {
         log('no source bytes on data store — extraction cannot run');
@@ -245,13 +233,28 @@ function collectDividerIdsOnStorey(store, extractor, storeyId, dividerTypes, log
  * the new rooms that overlap an already-present space (per-space dedup), while
  * still adding rooms an empty part of the floor lacks. Keyed by storey
  * expressId; storeys with no resolvable space footprints are omitted.
+ *
+ * If the model's length-unit scale itself can't be trusted, this refuses
+ * for the WHOLE store (empty map) rather than dividing every footprint by a
+ * wrong factor: `storeyPlanFrame` — the write side of the same round-trip
+ * (`useSpaceBake.ts` folds a new room through it, then reads existing
+ * footprints back through this function to dedup against) — refuses the
+ * same way per storey. A silent `?? 1` here on a genuinely degenerate scale
+ * would have scaled every existing footprint to a value the write side had
+ * already refused to trust, so the dedup compare would run on stale
+ * coordinates with no warning that anything was wrong. An empty map means
+ * dedup is skipped, matching the existing `--force` behaviour of this same
+ * call site (`generateSpaces` in `generate-spaces-all.ts`) rather than
+ * emitting mis-scaled polygons.
  */
 export function existingSpaceFootprintsByStorey(store) {
     const out = new Map();
     if (!store.source)
         return out;
     const extractor = new EntityExtractor(store.source);
-    const scale = extractLengthUnitScale(store.source, store.entityIndex) ?? 1;
+    const scale = safeLengthUnitScale(store.source, store.entityIndex, 'existingSpaceFootprintsByStorey');
+    if (scale === null)
+        return out;
     const aggregated = buildRelatingChildrenIndex(store, extractor, 'IFCRELAGGREGATES', 4, 5);
     const contained = buildRelatingChildrenIndex(store, extractor, 'IFCRELCONTAINEDINSPATIALSTRUCTURE', 5, 4);
     for (const st of store.getEntitiesByType('IfcBuildingStorey')) {
@@ -660,229 +663,6 @@ function finaliseSegment(start, end, wallId, log, source) {
     log(`wall #${wallId}: axis (${start[0].toFixed(3)},${start[1].toFixed(3)})→(${end[0].toFixed(3)},${end[1].toFixed(3)}) len=${len.toFixed(3)} source=${source}`);
     return { segment: { a: start, b: end } };
 }
-/**
- * Walk IfcLocalPlacement → IfcAxis2Placement3D → CartesianPoint and read the
- * ground-plane origin + RefDirection *of this one placement*, i.e. expressed
- * in its own `PlacementRelTo` parent's frame. Returns null when any link is
- * missing.
- */
-function readOwnPlacementFrame(store, extractor, overlay, placementId) {
-    const placement = readEntity(store, extractor, overlay, placementId);
-    if (!placement)
-        return null;
-    const axisPlacementId = numericAttr(placement.attributes[1]);
-    if (axisPlacementId === null)
-        return null;
-    const axisPlacement = readEntity(store, extractor, overlay, axisPlacementId);
-    if (!axisPlacement)
-        return null;
-    const locationId = numericAttr(axisPlacement.attributes[0]);
-    const refDirId = numericAttr(axisPlacement.attributes[2]);
-    if (locationId === null)
-        return null;
-    const locationEnt = readEntity(store, extractor, overlay, locationId);
-    if (!locationEnt)
-        return null;
-    const origin = readVec3(locationEnt.attributes[0]);
-    if (!origin)
-        return null;
-    let axisX = [1, 0];
-    if (refDirId !== null) {
-        const refDir = readEntity(store, extractor, overlay, refDirId);
-        if (refDir) {
-            const dir = readVec3(refDir.attributes[0]);
-            if (dir) {
-                const len = Math.hypot(dir[0], dir[1]);
-                if (len > AXIS_EPS)
-                    axisX = [dir[0] / len, dir[1] / len];
-            }
-        }
-    }
-    return { origin: [origin[0], origin[1]], axisX };
-}
-/**
- * The storey's OWN placement chain: its `ObjectPlacement` and every ancestor
- * reachable from it via `IfcLocalPlacement.PlacementRelTo`, mapped to the
- * number of hops from the storey's own placement (`0` for that placement
- * itself). Composition stops when it reaches one of these, and the hop count
- * says how much of the storey's own chain still separates the stopping point
- * from the storey frame — see `frameInStoreyFrame`.
- *
- * Returns `null` — NOT an empty map — when the storey has no resolvable
- * placement, which `IfcProduct.ObjectPlacement` being OPTIONAL makes a
- * well-formed possibility. An empty map is not "no storey frame", it is
- * "a storey frame the walk can never reach": the walk would find nothing to
- * stop on and compose every hop up to the world root, labelling world
- * coordinates storey-local. `frameInStoreyFrame` treats `null` as "compose
- * nothing" instead — see there.
- */
-function storeyPlacementChain(store, extractor, overlay, storeyId) {
-    const chain = new Map();
-    const storey = readEntity(store, extractor, overlay, storeyId);
-    if (!storey)
-        return null;
-    let id = numericAttr(storey.attributes[5]); // ObjectPlacement
-    if (id === null)
-        return null;
-    while (id !== null && !chain.has(id)) {
-        chain.set(id, chain.size);
-        const placement = readEntity(store, extractor, overlay, id);
-        if (!placement)
-            break;
-        id = numericAttr(placement.attributes[0]); // PlacementRelTo
-    }
-    return chain;
-}
-/**
- * The storey's frame expressed in the frame of the chain entry `hops` steps
- * above its own placement: compose the storey's first `hops` placements, the
- * innermost one first. `hops === 0` is the storey's own placement, whose frame
- * relative to itself is the identity.
- *
- * Returns `null` when any of those placements has no readable frame — the
- * caller must then leave the element where it is rather than move it by a
- * partial chain.
- */
-function storeyFrameAboveBy(store, extractor, overlay, storeyChain, hops) {
-    let frame = { origin: [0, 0], axisX: [1, 0] };
-    let composed = 0;
-    for (const id of storeyChain.keys()) {
-        if (composed >= hops)
-            break;
-        const own = readOwnPlacementFrame(store, extractor, overlay, id);
-        if (!own)
-            return null;
-        frame = composeFrames(own, frame);
-        composed += 1;
-    }
-    return composed === hops ? frame : null;
-}
-/**
- * The inverse of a planar rigid frame: `invertFrame(f)` maps a point expressed
- * in `f`'s parent frame back into `f`'s own frame, so
- * `composeFrames(invertFrame(f), g)` re-expresses `g` — a sibling of `f` in
- * that parent frame — relative to `f`.
- *
- * With `axisX = (c, s)` the frame applies `p ↦ R·p + origin`, `R = [[c,−s],
- * [s,c]]`. The inverse applies `p ↦ Rᵀ·(p − origin)`, and `Rᵀ` is the rotation
- * whose `axisX` is `(c, −s)`, which is why only the Y component flips.
- */
-function invertFrame(frame) {
-    const c = frame.axisX[0];
-    const s = frame.axisX[1];
-    const [ox, oy] = frame.origin;
-    return { origin: [-(c * ox + s * oy), -(-s * ox + c * oy)], axisX: [c, -s] };
-}
-/**
- * Compose `inner` (expressed in `outer`'s frame) with `outer`, giving the
- * frame of `inner` expressed in whatever frame `outer` is expressed in.
- */
-function composeFrames(outer, inner) {
-    const ax = outer.axisX[0];
-    const ay = outer.axisX[1];
-    // Perpendicular = rotate axisX 90° CCW around +Z — same convention as
-    // `applyFrame`, which this must agree with exactly.
-    const px = -ay;
-    const py = ax;
-    return {
-        origin: applyFrame(outer, inner.origin),
-        // Directions rotate but do not translate.
-        axisX: [ax * inner.axisX[0] + px * inner.axisX[1], ay * inner.axisX[0] + py * inner.axisX[1]],
-    };
-}
-/**
- * Frame of `placementId` expressed in the STOREY's frame: this placement's
- * own frame composed with every intermediate `IfcLocalPlacement.
- * PlacementRelTo` hop, stopping *before* the storey's own placement.
- *
- * An element's `ObjectPlacement` is not always one hop from its storey. The
- * IFC-standard grouping pattern — an `IfcElementAssembly` for a curtain
- * wall, precast panel run or railing system, all of them in
- * `DEFAULT_DIVIDER_TYPES` — inserts an intermediate `IfcLocalPlacement`
- * between the member and the storey. Reading only the member's own
- * `RelativePlacement` silently drops that hop's translation and rotation, so
- * the member is extracted at the wrong position relative to walls placed
- * directly under the storey: corners that should meet come out disconnected
- * and the enclosed room is never detected.
- *
- * Composition deliberately STOPS at the storey rather than continuing to the
- * root. Storey-local is the frame the write side uses: `generateSpacesFromWalls`
- * hands these segments to `addSpaceToStore`, which authors the new `IfcSpace`
- * with `anchor.storeyPlacementId` as its `PlacementRelTo` (see `space.ts` and
- * `resolve-anchor.ts`). Composing to the root would bake the storey's own
- * offset into the coordinates and the storey placement would then apply it a
- * second time, putting the space a whole site-offset away from its room.
- *
- * A `null` `storeyChain` (storey with no `ObjectPlacement`) means there is no
- * storey frame to compose towards, so no hop is composed at all.
- */
-function frameInStoreyFrame(store, extractor, overlay, placementId, storeyChain) {
-    let frame = readOwnPlacementFrame(store, extractor, overlay, placementId);
-    if (!frame)
-        return null;
-    // No storey placement at all (`ObjectPlacement` is OPTIONAL): there is no
-    // storey frame to compose towards and nothing for the walk to stop on, so
-    // walking parents would run to the world root and return world coordinates
-    // where every caller expects storey-local ones. Compose nothing — the
-    // element's own frame, which is what this returned before the chain walk
-    // existed — and leave the element where its own placement puts it rather
-    // than silently moving it by the site and building offsets.
-    if (!storeyChain)
-        return frame;
-    // Visited-set termination: exact for cyclic/self-referential
-    // `PlacementRelTo` in malformed IFC, and needs no arbitrary depth bound.
-    const visited = new Set([placementId]);
-    let currentId = placementId;
-    for (;;) {
-        const placement = readEntity(store, extractor, overlay, currentId);
-        const relToId = placement ? numericAttr(placement.attributes[0]) : null;
-        // The walk joins the storey's own chain (or runs out of parents at the
-        // root, where the storey's chain also ends). `frame` is now expressed in
-        // that shared ancestor's frame, and the storey sits `hops` placements
-        // below it — `hops === 0` when the ancestor IS the storey's placement,
-        // the common shape, where the frame is already storey-local.
-        if (relToId === null || storeyChain.has(relToId)) {
-            const hops = relToId === null ? storeyChain.size : storeyChain.get(relToId);
-            if (hops === 0)
-                return frame;
-            // #3003: the wall joined the chain ABOVE the storey, so the storey's own
-            // hops are in `frame` and every caller reads the result as storey-local.
-            // Divide them out — the INVERSE of the storey's frame in the shared
-            // ancestor, applied to the wall's frame in that same ancestor. Applying
-            // the storey frame itself instead would double the offset rather than
-            // remove it, so the two directions land on different coordinates and the
-            // #3003 fixture pins which one this is.
-            const storeyFrame = storeyFrameAboveBy(store, extractor, overlay, storeyChain, hops);
-            return storeyFrame ? composeFrames(invertFrame(storeyFrame), frame) : frame;
-        }
-        // A cycle in a malformed `PlacementRelTo`: nothing above is trustworthy,
-        // so stop with what has been composed so far.
-        if (visited.has(relToId))
-            return frame;
-        const parent = readOwnPlacementFrame(store, extractor, overlay, relToId);
-        if (!parent)
-            return frame;
-        frame = composeFrames(parent, frame);
-        visited.add(relToId);
-        currentId = relToId;
-    }
-}
-/**
- * Apply a placement frame to a storey-local 2D point. The point's X is
- * along the wall's local axis; Y is perpendicular (perpendicular to
- * the wall direction in the ground plane).
- */
-function applyFrame(frame, local) {
-    const ax = frame.axisX[0];
-    const ay = frame.axisX[1];
-    // Perpendicular = rotate axisX 90° CCW around +Z.
-    const px = -ay;
-    const py = ax;
-    return [
-        frame.origin[0] + ax * local[0] + px * local[1],
-        frame.origin[1] + ay * local[0] + py * local[1],
-    ];
-}
 /**
  * Walk the wall's representations, looking for the standard `Axis`
  * representation. Returns the first two vertices of the first
@@ -1003,34 +783,6 @@ export function resolveEntityTypeName(store, entity, entityId) {
     const name = (fromTable && fromTable !== 'Unknown' ? fromTable : entity.type) ?? '';
     return name.toLowerCase();
 }
-function readEntity(store, extractor, overlay, expressId) {
-    const ref = store.entityIndex.byId.get(expressId);
-    if (ref && ref.byteLength > 0 && ref.byteOffset >= 0) {
-        return extractor.extractEntity(ref);
-    }
-    // Overlay-only entity: fall back to the overlay reader.
-    if (overlay) {
-        for (const ent of overlay.getNewEntities()) {
-            if (ent.expressId === expressId) {
-                return { type: ent.type, attributes: ent.attributes };
-            }
-        }
-    }
-    return null;
-}
-function numericAttr(v) {
-    if (typeof v === 'number')
-        return v;
-    if (typeof v === 'string') {
-        if (v.startsWith('#')) {
-            const n = Number(v.slice(1));
-            return Number.isFinite(n) ? n : null;
-        }
-        const n = Number(v);
-        return Number.isFinite(n) ? n : null;
-    }
-    return null;
-}
 function stringAttr(v) {
     if (typeof v === 'string') {
         // Strip STEP single quotes if present (parser sometimes returns them, sometimes not).
@@ -1040,14 +792,4 @@ function stringAttr(v) {
     }
     return null;
 }
-function readVec3(v) {
-    if (!Array.isArray(v) || v.length < 2)
-        return null;
-    const x = numericAttr(v[0]);
-    const y = numericAttr(v[1]);
-    const z = v.length >= 3 ? numericAttr(v[2]) : 0;
-    if (x === null || y === null || z === null)
-        return null;
-    return [x, y, z];
-}
 //# sourceMappingURL=extract-walls.js.map