partforge 0.143.0 → 0.143.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "partforge",
-  "version": "0.143.0",
+  "version": "0.143.1",
   "description": "Turn a declarative part definition into a parametric-CAD web app (three.js + Manifold/Replicad). Requires a Vite-based consumer.",
   "type": "module",
   "license": "MIT",

package/src/framework/geometry/probe.js

@@ -51,6 +51,29 @@ function makeProbe(onCall) {
     toSTL: () => new ArrayBuffer(0),
     toIndexedMesh: () => ({ positions: new Float32Array(9), indices: new Uint32Array(3) }),
   };
+  // Solid's optional mesh-topology queries return numbers/booleans, not a handle.
+  Object.assign(solidQueries, { genus: () => 0, isEmpty: () => false });
+  // Shape2D ops that return DATA rather than another Shape2D. Without an entry here an
+  // op falls through to the chaining branch and hands back the Shape2D handle itself,
+  // so ordinary array/number use of the result (`.corners().map(…)`, `.area() * 2`)
+  // was recorded as an unknown Shape2D op and failed lint as `unknown-solid-op`.
+  // `regions()` returns live Shape2Ds, so its dummy is an array holding the handle.
+  // The dummy region is a unit square in the contour IR (closed: last point = first).
+  const square = [[0, 0], [1, 0], [1, 1], [0, 1], [0, 0]];
+  const shape2dQueries = {
+    ...solidQueries,
+    isEmpty: () => false,
+    area: () => 1,
+    boundingBox: () => ({ min: [0, 0], max: [1, 1], center: [0.5, 0.5], size: [1, 1] }),
+    toRegions: () => [{ outer: square.map((p) => [...p]), holes: [] }],
+    simple: () => ({ outer: square.map((p) => [...p]), holes: [] }),
+    toContours: () => [{ outer: { start: [0, 0], segments: square.slice(1).map((p) => ({ to: [...p] })) }, holes: [] }],
+    corners: () => [0, 1, 2, 3].map((i) => ({
+      index: i, position: i, point: [...square[i]], interiorAngleDeg: 90, convex: true, segTypes: ["line", "line"],
+    })),
+    contains: () => false,
+    regions: () => [shape2d],
+  };
   const kernelQueries = {
     toSTEP: () => Promise.resolve(new ArrayBuffer(0)),
     cleanup: () => {},
@@ -82,10 +105,8 @@ function makeProbe(onCall) {
     },
   });
 
-  // Shape2D handles reuse solidQueries: boundingBox/volume/… dummies chain the same
-  // either way, and a 3-component bbox is as good a dummy as a 2-component one.
   const proxy = opProxy(solidQueries, "solid", () => proxy);   // a solid handle: every op chains back to itself
-  const shape2d = opProxy(solidQueries, "shape2d",
+  const shape2d = opProxy(shape2dQueries, "shape2d",
     (key) => (SOLID_YIELDING_SHAPE2D_OPS.has(key) ? proxy : shape2d));
   const kernel = opProxy(kernelQueries, "kernel",             // factory ops (cylinder/box/prism/…) return a solid
     (key) => (SHAPE2D_YIELDING_KERNEL_OPS.has(key) ? shape2d : proxy));

package/src/framework/imports.js

@@ -10,11 +10,21 @@ import { parse3MF } from "./geometry/threemf-parse.js";
 
 const EXT = { step: "step", stp: "step", stl: "stl", "3mf": "3mf" };
 
+// Font files declared as an import. Without this a TTF fell through to the STL
+// fallback below and failed as "binary STL truncated: header says 3848966777
+// triangles" — an error about a mesh the author never meant (cloud feedback #148).
+const FONT_EXT = new Set(["ttf", "otf", "woff", "woff2"]);
+const FONT_MAGIC = ["\x00\x01\x00\x00", "OTTO", "true", "ttcf", "wOFF", "wOF2"];
+
 export function detectFormat(source, bytes) {
   const path = source instanceof URL ? source.pathname : typeof source === "string" ? source.split("?")[0] : null;
   const ext = path?.match(/\.([A-Za-z0-9]+)$/)?.[1]?.toLowerCase();
-  if (ext && EXT[ext]) return EXT[ext];
   const u8 = bytes instanceof ArrayBuffer ? new Uint8Array(bytes) : bytes;
+  const magic = u8 && u8.length >= 4 ? String.fromCharCode(...u8.slice(0, 4)) : "";
+  if ((ext && FONT_EXT.has(ext)) || FONT_MAGIC.includes(magic)) {
+    throw new Error(`a font file${path ? ` ("${path}")` : ""} is declared under \`imports\` — declare it under \`fonts\` and use it with text2d`);
+  }
+  if (ext && EXT[ext]) return EXT[ext];
   if (u8 && u8.length > 0) {
     const head = String.fromCharCode(...u8.slice(0, 64));
     if (head.startsWith("ISO-10303-21")) return "step";