@helix3/helix-cli 0.1.14-helix3.207 → 0.1.14-helix3.208

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@helix3/helix-cli",
-  "version": "0.1.14-helix3.207",
+  "version": "0.1.14-helix3.208",
   "description": "helix — the HELIX creator CLI: scaffold, validate, and publish Instant Worlds",
   "main": "dist/lib.js",
   "types": "dist/lib.d.ts",

package/plugins/beamng-vehicle/bridge-plugin.json

@@ -2,7 +2,7 @@
   "schemaVersion": "helix.bridge-plugin/1",
   "id": "hypersonic.beamng-vehicle",
   "name": "BeamNG Vehicle Importer",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "trust": "local-untrusted",
   "description": "Assess, convert, texture, lower, and acceptance-test a BeamNG vehicle plus its exact configuration before HELIX packaging.",
   "permissions": [

package/plugins/beamng-vehicle/lib/material-semantics.mjs

@@ -162,7 +162,7 @@ const MAX_SEMANTIC_CLONE_CHARS = 128 * 1024 * 1024;
  * before downstream material deduplication. Texture indices are deliberately
  * untouched, so authored atlases remain one shared binary resource.
  */
-export function stampVehicleMaterialSemantics(json, familyVersions, familyDefaults) {
+export function stampVehicleMaterialSemantics(json, familyVersions, familyDefaults, authoredBaseColorFactors) {
   // The prior two-argument injector API did not carry a Palette closure. Keep
   // it byte-semantically compatible rather than emitting a contradictory
   // family ID with a version belonging to a different family.
@@ -240,7 +240,19 @@ export function stampVehicleMaterialSemantics(json, familyVersions, familyDefaul
       if (defaults && Number.isFinite(defaults.roughness) && Number.isFinite(defaults.metallic)) {
         material.pbrMetallicRoughness ??= {};
         const alpha = material.pbrMetallicRoughness.baseColorFactor?.[3] ?? 1;
-        material.pbrMetallicRoughness.baseColorFactor = [1, 1, 1, alpha];
+        // A family default is a FINISH — how rough and how metallic the surface is. It carries no colour,
+        // so there is nothing here that could stand in for one. Forcing [1,1,1] used to look harmless
+        // because the fixtures it was written against had no authored albedo at all; run it over a real
+        // converted car and every panel, tyre, grille and lens comes out white. That is exactly what
+        // shipped: all four Subaru renditions rendered as a white car while the Blender geometry they
+        // were built from still held the WRX blue.
+        // So restore the colour the SOURCE authored. Dropping the tint this injector wrote from a
+        // binding override is still correct on a family change (`bindingFor` strips it from the binding
+        // too) — that case is the one where the snapshot is absent or already neutral.
+        const authored = authoredBaseColorFactors?.[sourceIndex];
+        material.pbrMetallicRoughness.baseColorFactor = Array.isArray(authored) && authored.length >= 3
+          ? [authored[0], authored[1], authored[2], alpha]
+          : [1, 1, 1, alpha];
         material.pbrMetallicRoughness.roughnessFactor = defaults.roughness;
         material.pbrMetallicRoughness.metallicFactor = defaults.metallic;
       }

package/plugins/beamng-vehicle/lib/vehicle-qa.mjs

@@ -135,6 +135,15 @@ function applyMaterialBindings(json, rows, familyVersions, familyDefaults) {
     }
   }
   const byName = new Map((rows ?? []).map((row) => [row.name, row]));
+  // SNAPSHOT THE AUTHORED ALBEDO BEFORE ANY TINT IS WRITTEN OVER IT.
+  // A converted BeamNG body carries its colour in `baseColorFactor`, not in a texture: WRX blue, black
+  // tyre and grille, red indicator lenses, silver mirrors. Semantic stamping below re-tints a material
+  // when it moves to a different Palette family, and it is entitled to drop a tint IT wrote from a
+  // binding override — never the colour the conversion authored. Taking the snapshot here, before the
+  // tint loop, is what makes those two cases distinguishable one function later.
+  const authoredBaseColorFactors = (json.materials ?? []).map(
+    (material) => material?.pbrMetallicRoughness?.baseColorFactor,
+  );
   for (const material of json.materials ?? []) {
     if (!material || typeof material !== 'object' || Array.isArray(material)) {
       throw new Error('converter output GLB materials must contain objects');
@@ -161,7 +170,7 @@ function applyMaterialBindings(json, rows, familyVersions, familyDefaults) {
       ];
     }
   }
-  stampVehicleMaterialSemantics(json, familyVersions, familyDefaults);
+  stampVehicleMaterialSemantics(json, familyVersions, familyDefaults, authoredBaseColorFactors);
 }
 
 function serializeGlbJson(json) {