@ifc-lite/wasm 6.0.0 → 6.1.0

package/package.json

@@ -5,7 +5,7 @@
     "IFC-Lite Contributors"
   ],
   "description": "WebAssembly bindings for IFC-Lite",
-  "version": "6.0.0",
+  "version": "6.1.0",
   "license": "MPL-2.0",
   "repository": {
     "type": "git",

package/pkg/ifc-lite.d.ts

@@ -891,7 +891,10 @@ export class MeshCollection {
      * worker reads each mesh exactly once, so moving avoids the full vertex-
      * data clone `get` pays — one fewer copy of positions/normals/indices/uvs/
      * texture per mesh (the JS getters still do the single Rust→JS copy). Calling
-     * it twice for the same index yields the second call an empty mesh.
+     * it twice for the same index yields the second call a DEFAULT mesh:
+     * `expressId` 0 and every buffer empty, rather than the metadata-bearing
+     * husk the hand-written copy used to leave. The method is read-once by
+     * contract and the wasm-contract test pins this.
      */
     takeMesh(index: number): MeshDataJs | undefined;
     /**
@@ -1015,6 +1018,27 @@ export class MeshCollection {
 
 /**
  * Individual mesh data with express ID and color (matches MeshData interface)
+ *
+ * `Clone` is derived so the three sites that used to enumerate all 21 fields
+ * by hand -- `get`, `takeMesh` and `MeshCollection`'s own `Clone` -- reduce to
+ * `.cloned()`, `mem::take` and `.clone()`. Adding a field to #3199 meant
+ * editing three literals in lockstep; the allowlist row for this file records
+ * exactly that cost.
+ *
+ * To be precise about what this does and does not buy, because the obvious
+ * claim is wrong: an exhaustive struct literal that OMITS a field is a compile
+ * error, so those literals were never silently lossy. What they were is three
+ * places to edit for one field, and `..Default::default()` is the shortcut a
+ * hurried author reaches for when the compiler complains -- which WOULD be
+ * silently lossy. `Default` is what makes `takeMesh` a one-liner, so the rule
+ * is: TWO literals remain, `new` and `Default` below (both spelled
+ * `Self { .. }`, which is why a grep for `MeshDataJs {` finds neither). Both
+ * must stay exhaustive AND must agree on every field `new` does not take as an
+ * argument. The compiler catches an omitted field in either; it cannot catch
+ * the two DISAGREEING, which is the failure the pair exists to prevent, so
+ * `default_agrees_with_new_on_the_fields_new_does_not_take` covers that.
+ * Never spread `Default` into `new` -- `new` is the only place a field's
+ * initial value is decided, so a field defaulted there is inert everywhere.
  */
 export class MeshDataJs {
     private constructor();
@@ -1034,6 +1058,11 @@ export class MeshDataJs {
      * type geometry (hidden in Model mode, shown in Types mode).
      */
     readonly geometryClass: number;
+    /**
+     * Source `IfcRepresentationItem`, or `undefined` where identity is merged
+     * away. Never set alongside `materialId` (#3199).
+     */
+    readonly geometryItemId: number | undefined;
     /**
      * True when this mesh carries a surface texture (#961).
      */
@@ -1058,6 +1087,10 @@ export class MeshDataJs {
      * `local_bounds` above).
      */
     readonly localToWorld: Float64Array | undefined;
+    /**
+     * `IfcMaterial` layer sliced, or `undefined` (#3199).
+     */
+    readonly materialId: number | undefined;
     /**
      * Get normals as Float32Array (copy to JS)
      */
@@ -1976,11 +2009,13 @@ export interface InitOutput {
     readonly meshdatajs_color: (a: number, b: number) => void;
     readonly meshdatajs_expressId: (a: number) => number;
     readonly meshdatajs_geometryClass: (a: number) => number;
+    readonly meshdatajs_geometryItemId: (a: number) => number;
     readonly meshdatajs_hasTexture: (a: number) => number;
     readonly meshdatajs_ifcType: (a: number, b: number) => void;
     readonly meshdatajs_indices: (a: number) => number;
     readonly meshdatajs_localBounds: (a: number, b: number) => void;
     readonly meshdatajs_localToWorld: (a: number, b: number) => void;
+    readonly meshdatajs_materialId: (a: number) => number;
     readonly meshdatajs_normals: (a: number) => number;
     readonly meshdatajs_origin: (a: number) => number;
     readonly meshdatajs_positions: (a: number) => number;

package/pkg/ifc-lite.js

@@ -2417,7 +2417,10 @@ export class MeshCollection {
      * worker reads each mesh exactly once, so moving avoids the full vertex-
      * data clone `get` pays — one fewer copy of positions/normals/indices/uvs/
      * texture per mesh (the JS getters still do the single Rust→JS copy). Calling
-     * it twice for the same index yields the second call an empty mesh.
+     * it twice for the same index yields the second call a DEFAULT mesh:
+     * `expressId` 0 and every buffer empty, rather than the metadata-bearing
+     * husk the hand-written copy used to leave. The method is read-once by
+     * contract and the wasm-contract test pins this.
      * @param {number} index
      * @returns {MeshDataJs | undefined}
      */
@@ -2446,6 +2449,27 @@ if (Symbol.dispose) MeshCollection.prototype[Symbol.dispose] = MeshCollection.pr
 
 /**
  * Individual mesh data with express ID and color (matches MeshData interface)
+ *
+ * `Clone` is derived so the three sites that used to enumerate all 21 fields
+ * by hand -- `get`, `takeMesh` and `MeshCollection`'s own `Clone` -- reduce to
+ * `.cloned()`, `mem::take` and `.clone()`. Adding a field to #3199 meant
+ * editing three literals in lockstep; the allowlist row for this file records
+ * exactly that cost.
+ *
+ * To be precise about what this does and does not buy, because the obvious
+ * claim is wrong: an exhaustive struct literal that OMITS a field is a compile
+ * error, so those literals were never silently lossy. What they were is three
+ * places to edit for one field, and `..Default::default()` is the shortcut a
+ * hurried author reaches for when the compiler complains -- which WOULD be
+ * silently lossy. `Default` is what makes `takeMesh` a one-liner, so the rule
+ * is: TWO literals remain, `new` and `Default` below (both spelled
+ * `Self { .. }`, which is why a grep for `MeshDataJs {` finds neither). Both
+ * must stay exhaustive AND must agree on every field `new` does not take as an
+ * argument. The compiler catches an omitted field in either; it cannot catch
+ * the two DISAGREEING, which is the failure the pair exists to prevent, so
+ * `default_agrees_with_new_on_the_fields_new_does_not_take` covers that.
+ * Never spread `Default` into `new` -- `new` is the only place a field's
+ * initial value is decided, so a field defaulted there is inert everywhere.
  */
 export class MeshDataJs {
     static __wrap(ptr) {
@@ -2499,6 +2523,15 @@ export class MeshDataJs {
         const ret = wasm.meshdatajs_geometryClass(this.__wbg_ptr);
         return ret;
     }
+    /**
+     * Source `IfcRepresentationItem`, or `undefined` where identity is merged
+     * away. Never set alongside `materialId` (#3199).
+     * @returns {number | undefined}
+     */
+    get geometryItemId() {
+        const ret = wasm.meshdatajs_geometryItemId(this.__wbg_ptr);
+        return ret === Number.MAX_SAFE_INTEGER ? undefined : ret;
+    }
     /**
      * True when this mesh carries a surface texture (#961).
      * @returns {boolean}
@@ -2579,6 +2612,14 @@ export class MeshDataJs {
             wasm.__wbindgen_add_to_stack_pointer(16);
         }
     }
+    /**
+     * `IfcMaterial` layer sliced, or `undefined` (#3199).
+     * @returns {number | undefined}
+     */
+    get materialId() {
+        const ret = wasm.meshdatajs_materialId(this.__wbg_ptr);
+        return ret === Number.MAX_SAFE_INTEGER ? undefined : ret;
+    }
     /**
      * Get normals as Float32Array (copy to JS)
      * @returns {Float32Array}