brepjs-bim 0.5.0 → 0.7.0

package/dist/brepjs-bim.cjs

@@ -16615,13 +16615,25 @@ function addOpenings(model, host, wallId, containerId, idByKeyPath) {
 		if (opening.type !== "Opening") continue;
 		const fill = opening.children[0];
 		if (fill === void 0) return (0, brepjs.err)(specError("FAMILIES_OPENING_NO_FILL", `familiesToBim: opening '${opening.keyPath}' has no fill element`));
+		const keyed = requireKeyed(opening);
+		if (!keyed.ok) return keyed;
 		const fillT = peelTranslates(opening.geometry).total;
+		const axisX = host.props["axisX"] ?? [
+			1,
+			0,
+			0
+		];
+		const delta = [
+			fillT[0] - hostT[0],
+			fillT[1] - hostT[1],
+			fillT[2] - hostT[2]
+		];
 		const added = addFill(model, fill, {
 			materialName: SPEC_DEFAULTS.materialName,
 			...stripGeometryProps(fill.props),
 			wallLocalId: wallId,
-			offsetAlongWall: fillT[0] - hostT[0],
-			offsetFromFloor: fillT[2] - hostT[2]
+			offsetAlongWall: delta[0] * axisX[0] + delta[1] * axisX[1] + delta[2] * axisX[2],
+			offsetFromFloor: delta[2]
 		}, {
 			stableKey: fill.keyPath,
 			openingStableKey: opening.keyPath
@@ -16634,6 +16646,13 @@ function addOpenings(model, host, wallId, containerId, idByKeyPath) {
 	}
 	return (0, brepjs.ok)(void 0);
 }
+/** Every element that mints an IFC identity needs an explicit key: an
+*  index-fallback path is order-dependent, which would silently break the
+*  reorder-stable GlobalId contract. */
+function requireKeyed(el) {
+	if (el.keyed) return (0, brepjs.ok)(void 0);
+	return (0, brepjs.err)(specError("FAMILIES_UNKEYED_ELEMENT", `familiesToBim: '${el.keyPath}' has no explicit key — IFC identity needs order-independent key paths (add a key to the element)`));
+}
 /**
 * Project a resolved families tree into an eager BimModel. The caller owns
 * the returned model (`using`); families stays domain-neutral — this adapter
@@ -16650,6 +16669,8 @@ function familiesToBim(root, options) {
 	const walk = (el, storeyId) => {
 		let containerId = storeyId;
 		if (el.type === "Storey") {
+			const keyed = requireKeyed(el);
+			if (!keyed.ok) return keyed;
 			const id = model.addStorey({
 				name: el.attributes["name"] ?? el.keyPath,
 				elevation: el.props["elevation"] ?? 0
@@ -16658,6 +16679,8 @@ function familiesToBim(root, options) {
 			idByKeyPath.set(el.keyPath, id);
 			containerId = id;
 		} else if (el.type === "Wall" || el.type === "Slab") {
+			const keyed = requireKeyed(el);
+			if (!keyed.ok) return keyed;
 			const parsed = el.type === "Wall" ? parseWallSpec(specInput(el)) : parseSlabSpec(specInput(el));
 			if (!parsed.ok) return parsed;
 			const added = el.type === "Wall" ? model.addWall(parsed.value, { stableKey: el.keyPath }) : model.addSlab(parsed.value, { stableKey: el.keyPath });

package/dist/brepjs-bim.js

@@ -16592,13 +16592,25 @@ function addOpenings(model, host, wallId, containerId, idByKeyPath) {
 		if (opening.type !== "Opening") continue;
 		const fill = opening.children[0];
 		if (fill === void 0) return err(specError("FAMILIES_OPENING_NO_FILL", `familiesToBim: opening '${opening.keyPath}' has no fill element`));
+		const keyed = requireKeyed(opening);
+		if (!keyed.ok) return keyed;
 		const fillT = peelTranslates(opening.geometry).total;
+		const axisX = host.props["axisX"] ?? [
+			1,
+			0,
+			0
+		];
+		const delta = [
+			fillT[0] - hostT[0],
+			fillT[1] - hostT[1],
+			fillT[2] - hostT[2]
+		];
 		const added = addFill(model, fill, {
 			materialName: SPEC_DEFAULTS.materialName,
 			...stripGeometryProps(fill.props),
 			wallLocalId: wallId,
-			offsetAlongWall: fillT[0] - hostT[0],
-			offsetFromFloor: fillT[2] - hostT[2]
+			offsetAlongWall: delta[0] * axisX[0] + delta[1] * axisX[1] + delta[2] * axisX[2],
+			offsetFromFloor: delta[2]
 		}, {
 			stableKey: fill.keyPath,
 			openingStableKey: opening.keyPath
@@ -16611,6 +16623,13 @@ function addOpenings(model, host, wallId, containerId, idByKeyPath) {
 	}
 	return ok(void 0);
 }
+/** Every element that mints an IFC identity needs an explicit key: an
+*  index-fallback path is order-dependent, which would silently break the
+*  reorder-stable GlobalId contract. */
+function requireKeyed(el) {
+	if (el.keyed) return ok(void 0);
+	return err(specError("FAMILIES_UNKEYED_ELEMENT", `familiesToBim: '${el.keyPath}' has no explicit key — IFC identity needs order-independent key paths (add a key to the element)`));
+}
 /**
 * Project a resolved families tree into an eager BimModel. The caller owns
 * the returned model (`using`); families stays domain-neutral — this adapter
@@ -16627,6 +16646,8 @@ function familiesToBim(root, options) {
 	const walk = (el, storeyId) => {
 		let containerId = storeyId;
 		if (el.type === "Storey") {
+			const keyed = requireKeyed(el);
+			if (!keyed.ok) return keyed;
 			const id = model.addStorey({
 				name: el.attributes["name"] ?? el.keyPath,
 				elevation: el.props["elevation"] ?? 0
@@ -16635,6 +16656,8 @@ function familiesToBim(root, options) {
 			idByKeyPath.set(el.keyPath, id);
 			containerId = id;
 		} else if (el.type === "Wall" || el.type === "Slab") {
+			const keyed = requireKeyed(el);
+			if (!keyed.ok) return keyed;
 			const parsed = el.type === "Wall" ? parseWallSpec(specInput(el)) : parseSlabSpec(specInput(el));
 			if (!parsed.ok) return parsed;
 			const added = el.type === "Wall" ? model.addWall(parsed.value, { stableKey: el.keyPath }) : model.addSlab(parsed.value, { stableKey: el.keyPath });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "brepjs-bim",
-  "version": "0.5.0",
+  "version": "0.7.0",
   "description": "BIM layer for brepjs — IFC4-aligned parametric building elements",
   "keywords": [
     "bim",