@forgeax/engine-gltf 0.1.24 → 0.1.25

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forgeax/engine-gltf",
-  "version": "0.1.24",
+  "version": "0.1.25",
   "private": false,
   "type": "module",
   "license": "Apache-2.0",
@@ -38,21 +38,21 @@
     "LICENSE"
   ],
   "dependencies": {
-    "@forgeax/engine-animation": "0.1.24",
-    "@forgeax/engine-geometry": "0.1.24",
-    "@forgeax/engine-import": "0.1.24",
-    "@forgeax/engine-math": "0.1.24",
-    "@forgeax/engine-pack": "0.1.24",
-    "@forgeax/engine-scene": "0.1.24",
-    "@forgeax/engine-types": "0.1.24",
+    "@forgeax/engine-animation": "0.1.25",
+    "@forgeax/engine-geometry": "0.1.25",
+    "@forgeax/engine-import": "0.1.25",
+    "@forgeax/engine-math": "0.1.25",
+    "@forgeax/engine-pack": "0.1.25",
+    "@forgeax/engine-scene": "0.1.25",
+    "@forgeax/engine-types": "0.1.25",
     "@webgpu/types": "^0.1.71",
     "meshoptimizer": "^0.20.0"
   },
   "devDependencies": {
-    "@forgeax/engine-assets-runtime": "0.1.24",
-    "@forgeax/engine-picking": "0.1.24",
-    "@forgeax/engine-render": "0.1.24",
-    "@forgeax/engine-runtime": "0.1.24",
+    "@forgeax/engine-assets-runtime": "0.1.25",
+    "@forgeax/engine-picking": "0.1.25",
+    "@forgeax/engine-render": "0.1.25",
+    "@forgeax/engine-runtime": "0.1.25",
     "@types/node": "^20.14.0"
   },
   "forgeax": {

package/src/__tests__/bridge-material-values.unit.test.ts

@@ -4,6 +4,29 @@ import { toMaterialAsset } from '../bridge.js';
 import type { GltfMaterialIr } from '../parse-gltf.js';
 
 describe('glTF material bridge values', () => {
+  it('publishes a parent-bearing child with only inherited contract and authored values', () => {
+    const standardRootGuid = '01a065d4-47dd-789f-96b4-36b9326abc02' as unknown as AssetGuid;
+    const material = toMaterialAsset(
+      {
+        baseColorFactor: [0.5, 0.75, 0.25, 1],
+        metallicFactor: 0.5,
+        roughnessFactor: 0.7,
+      } as unknown as GltfMaterialIr,
+      { standardRootGuid },
+    );
+
+    expect(material).toEqual({
+      kind: 'material',
+      parent: standardRootGuid,
+      values: {
+        baseColor: [0.5, 0.75, 0.25, 1],
+        metallic: 0.5,
+        roughness: 0.7,
+      },
+    });
+    expect(Object.keys(material).sort()).toEqual(['kind', 'parent', 'values']);
+  });
+
   it('preserves all four base-color channels for factor and texture composition', () => {
     const factor = [0.5, 0.75, 0.25, 0.4] as const;
     const material = toMaterialAsset({

package/src/__tests__/gltf.unit.test.ts

@@ -1840,6 +1840,15 @@ function unwrapReimport(result: ReturnType<typeof reimportReuseMeta>) {
       expect(mat).toBeDefined();
       const parentRefs = mat?.refs.filter((r) => r.sourceField?.fieldName === 'parent');
       expect(parentRefs?.map((ref) => ref.guid)).toEqual(['019e2cc6-0c86-79da-aa76-b0984c86d4ff']);
+      expect(mat?.payload).toMatchObject({
+        kind: 'material',
+        parent: '019e2cc6-0c86-79da-aa76-b0984c86d4ff',
+      });
+      expect(Object.keys((mat?.payload ?? {}) as Record<string, unknown>).sort()).toEqual([
+        'kind',
+        'parent',
+        'values',
+      ]);
     });
 
     it('(e) material refs texture edges: sourceField.componentName="<material>" and fieldName is texture slot', async () => {

package/src/bridge.ts

@@ -1169,16 +1169,16 @@ export function toMaterialAsset(mat: GltfMaterialIr, ctx?: MaterialBridgeContext
     },
   };
 
+  const child = !rootContract.extended && ctx?.standardRootGuid !== undefined;
   return {
     kind: 'material',
-    colorSpace: 'linear',
-    ...(!rootContract.extended && ctx?.standardRootGuid !== undefined
+    ...(child
       ? { parent: ctx.standardRootGuid }
-      : {}),
-    passes: [pass],
-    ...(ctx?.standardRootGuid !== undefined && !rootContract.extended
-      ? {}
-      : { parameters: standardMaterialParameters(rootContract.names) }),
+      : {
+          colorSpace: 'linear' as const,
+          passes: [pass] as [typeof pass, ...(typeof pass)[]],
+          parameters: standardMaterialParameters(rootContract.names),
+        }),
     values,
   };
 }