partforge 0.67.1 → 0.67.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "partforge",
-  "version": "0.67.1",
+  "version": "0.67.2",
   "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/kernel-front.js

@@ -12,16 +12,11 @@
 //     capability (Manifold can't do toSTEP; both backends now define shape2d, so
 //     that stub is dead in practice — kept as a safety net for a future backend).
 // The per-Solid twin of this layer is addSugar() in solid-sugar.js.
-// opentype.js 2.x ships no `exports` map: bundlers take its `module` field (real
-// ESM, named `parse`), while Node ESM takes `main` (a UMD/CJS bundle, whose named
-// exports Node cannot statically detect — the namespace holds only `default`).
-// So `import * as opentype` gives a working `.parse` in the browser and `undefined`
-// under Node, which broke every headless text2d build (`opentype.parse is not a
-// function`) while the browser stayed green. Normalize both interop shapes here.
+// opentype.js's namespace shape differs between bundler and Node resolution —
+// see opentype-interop.js for the trap (it has bitten once in each direction).
 import * as opentypeNamespace from "opentype.js";
-const opentype = typeof opentypeNamespace.parse === "function"
-  ? opentypeNamespace
-  : (opentypeNamespace.default ?? opentypeNamespace);
+import { normalizeOpentype } from "./opentype-interop.js";
+const opentype = normalizeOpentype(opentypeNamespace);
 import { KernelCapabilityError } from "./errors.js";
 import { isPlainOptions, KERNEL_OP_SPECS } from "./op-options.js";
 import { textGlyphs } from "./text2d.js";

package/src/framework/geometry/opentype-interop.js

@@ -0,0 +1,13 @@
+// opentype.js 2.x ships no `exports` map, so what importing it yields splits by
+// resolver: bundlers take the `module` field (real ESM — named `parse`, NO
+// default export), while Node takes `main` (a UMD/CJS bundle whose named exports
+// Node's lexer cannot statically detect — the namespace holds ONLY `default`).
+// Reading `.default` unconditionally is therefore correct under Node and
+// `undefined` in every browser bundle; reading a named export is the reverse.
+// That asymmetry once broke headless text2d builds (kernel-front's static
+// import) and later broke every BROWSER build of a part declaring `fonts`
+// ("undefined is not an object (evaluating 'p.parse')") while headless tests
+// stayed green (jobs' dynamic import). Both call sites normalize through this
+// one function so the two interop shapes stay handled in one place.
+export const normalizeOpentype = (ns) =>
+  typeof ns?.parse === "function" ? ns : (ns?.default ?? ns);

package/src/framework/jobs.js

@@ -5,6 +5,7 @@
 import { meshTo3MF } from "./geometry/threemf.js";
 import { exportablePartNames } from "./export-select.js";
 import { resolveFonts } from "./fonts.js";
+import { normalizeOpentype } from "./geometry/opentype-interop.js";
 import { ensureImports, resolveImports } from "./imports.js";
 import { safeName } from "./safe-name.js";
 import { exportSubParts, resolveParams, buildPosed } from "./part-model.js";
@@ -100,9 +101,11 @@ export async function handle(kernel, part, msg, post, opts = {}) {
   try {
     // Preload any part-declared fonts into the kernel before building — once per
     // font name; a lazy dynamic import because this is async context (unlike the
-    // synchronous kernel-front), so it doesn't cost sync callers anything.
+    // synchronous kernel-front), so it doesn't cost sync callers anything. The
+    // namespace shape differs between bundler and Node resolution (a bare
+    // `.default` here is undefined in every browser bundle) — normalize it.
     if (part.fonts && kernel._fonts) {
-      const opentype = (await import("opentype.js")).default;
+      const opentype = normalizeOpentype(await import("opentype.js"));
       const bufs = await resolveFonts(part.fonts);
       for (const [name, buf] of bufs) if (!kernel._fonts.has(name)) kernel._fonts.set(name, opentype.parse(buf));
     }

package/src/testing/manifold.js

@@ -4,6 +4,7 @@
 import Module from "manifold-3d";
 import { createManifoldKernel } from "../framework/geometry/manifold-backend.js";
 import { resolveFonts } from "../framework/fonts.js";
+import { normalizeOpentype } from "../framework/geometry/opentype-interop.js";
 import { ensureImports } from "../framework/imports.js";
 import { nodeAssetSources } from "./assets.js";
 import { tessellateStepAssets } from "./step-mesh.js";
@@ -12,7 +13,7 @@ export async function bootManifoldKernel({ quality = "preview", fonts, imports,
   const wasm = await Module();
   wasm.setup();
   const kernel = createManifoldKernel(wasm, { quality });
-  if (fonts) { const opentype = (await import("opentype.js")).default;
+  if (fonts) { const opentype = normalizeOpentype(await import("opentype.js"));
     for (const [name, buf] of await resolveFonts(nodeAssetSources(fonts))) kernel._fonts.set(name, opentype.parse(buf)); }
   if (imports) {
     const decl = nodeAssetSources(imports);

package/src/testing/occt.js

@@ -6,6 +6,7 @@ import path from "path";
 import fs from "fs";
 import { createOcctKernel } from "../framework/geometry/occt-backend.js";
 import { resolveFonts } from "../framework/fonts.js";
+import { normalizeOpentype } from "../framework/geometry/opentype-interop.js";
 import { ensureImports } from "../framework/imports.js";
 import { nodeAssetSources } from "./assets.js";
 
@@ -18,7 +19,7 @@ export async function bootOcctKernel({ fonts, imports, importMeshes } = {}) {
   const replicad = await import("replicad");
   replicad.setOC(OC);
   const kernel = createOcctKernel(replicad);
-  if (fonts) { const opentype = (await import("opentype.js")).default;
+  if (fonts) { const opentype = normalizeOpentype(await import("opentype.js"));
     for (const [name, buf] of await resolveFonts(nodeAssetSources(fonts))) kernel._fonts.set(name, opentype.parse(buf)); }
   if (imports) await ensureImports(kernel, nodeAssetSources(imports), importMeshes ?? null);
   return kernel;