partforge 0.137.1 → 0.138.1

package/src/framework/lint/rules-schema.js

@@ -330,6 +330,33 @@ export const SCHEMA_RULES = [
       return out;
     },
   },
+  {
+    // The legacy section shape (`advanced`, `toggles`, `features`) gives every entry a
+    // fixed widget and never reads `type`: a `{ type: "select", options }` there used to
+    // draw only two "unrecognised field" warnings and render as a SLIDER, which then
+    // wrote numbers into a key the build compared against strings. `control:` is read,
+    // but only knows the slider family. Either way the panel shows the wrong widget, so
+    // this is an error, not a warning.
+    id: "legacy-shape-widget",
+    run: ({ part }) => {
+      const LEGACY_CONTROLS = ["slider", "number", "text", "textarea"];
+      const out = [];
+      for (const { d, path, authored, container } of collectDescriptors(part)) {
+        if (authored || container) continue;
+        const renders = path.includes(".toggles[") ? "checkbox" : (LEGACY_CONTROLS.includes(d.control) ? d.control : "slider");
+        const hint = "Move the section to the `controls` shape (`controls: [...]`), where `type` picks the widget — select, radio, checkbox and readout exist only there.";
+        if (typeof d.type === "string")
+          out.push(err("legacy-shape-widget",
+            `\`type: "${d.type}"\` is not read in the legacy section shape — this control renders as a ${renders}`,
+            hint, `${path}.type`));
+        if (typeof d.control === "string" && !LEGACY_CONTROLS.includes(d.control))
+          out.push(err("legacy-shape-widget",
+            `\`control: "${d.control}"\` is not a legacy widget (legacy \`control\` takes ${LEGACY_CONTROLS.join(", ")})`,
+            hint, `${path}.control`));
+      }
+      return out;
+    },
+  },
   {
     id: "unknown-control-type",
     run: ({ part }) => collectDescriptors(part)

package/src/framework/pose-probe.js

@@ -4,20 +4,42 @@
 // without dragging in the part-model/jobs layer (purity).
 import { probeSubPartPose } from "./pose-probe-core.js";
 import { viewSubParts, resolveParams } from "./part-model.js";
+import { fontsFor } from "./fonts.js";
+import { imagesFor } from "./images.js";
+import { vectorsFor } from "./vectors.js";
+import { h, byteAwareReplacer } from "./geometry/solid-hash.js";
+
+// The sources behind every NAMED asset a build can reach. A build says
+// `k.text2d(s, { font: "face" })`, `k.heightfield("img")` or `k.vector2d("art")`,
+// and the probe sees only the name — which stays the same whichever font,
+// image or artwork a control has picked. Without this, a font pick hashed
+// identically before and after, so the fast path re-posed the OLD letters
+// instead of rebuilding them (the Lattice Box's raised text kept its previous
+// typeface while the pocket cut from the same text updated, 2026-10-02). Folded
+// into every trusted sub-part's baseHash: coarser than tracking which sub-part
+// names which asset, and the cost of that is only a rebuild on an asset change.
+// Serialized with byteAwareReplacer, not h(): h() reads an ArrayBuffer as {},
+// so two byte-valued fonts would hash alike — the same bug one level down.
+function assetSources(part, p) {
+  return [fontsFor(part, p) ?? null, imagesFor(part, p) ?? null, vectorsFor(part, p) ?? null];
+}
 
 // Probe every subpart the view shows. Never throws; a failing/queried/weird
 // subpart yields { trusted: false } and the others still probe.
 export function probePoses(part, view, params) {
   const out = new Map();
-  let resolved;
-  try { resolved = resolveParams(part, params); }
-  catch {
+  let resolved, assets;
+  try {
+    resolved = resolveParams(part, params);
+    assets = h("assets", JSON.stringify(assetSources(part, resolved.p), byteAwareReplacer));
+  } catch {
     for (const name of viewSubParts(part, view, params)) out.set(name, { trusted: false });
     return out;
   }
   const { p, d } = resolved;
   for (const name of viewSubParts(part, view, params)) {
-    out.set(name, probeSubPartPose(part.parts[name], { view, purpose: "display", p, d }));
+    const entry = probeSubPartPose(part.parts[name], { view, purpose: "display", p, d });
+    out.set(name, entry.trusted ? { ...entry, baseHash: h("with-assets", entry.baseHash, assets) } : entry);
   }
   return out;
 }

package/src/parts/screw.js

@@ -17,7 +17,9 @@ export default {
           description: "Axial rise per turn. A coarse pitch on a small major diameter runs the root radius down toward zero, which is why Major diameter starts at 4 mm. The 0.5 mm floor is the ISO fine pitch for the smallest diameter offered here — and a floor is needed, because cost scales with turns = length / pitch." },
         { key: "length", label: "Threaded length", unit: "mm", min: 5, max: 40, step: 1,
           description: "Length of the threaded shank, excluding the head. Capped at 40 mm so the worst case reachable from these sliders — 40 mm at a 0.5 mm pitch, 80 turns — stays a couple of seconds of preview rather than minutes." },
-        { key: "lefthand", label: "Left-hand thread", control: "toggle",
+      ],
+      toggles: [
+        { key: "lefthand", label: "Left-hand thread", on: true,
           description: "Reverses the helix. Rare outside gas fittings and bicycle pedals." },
       ],
     },