partforge 0.85.0 → 0.85.2

package/docs/ERROR-PATTERNS.md

@@ -521,6 +521,18 @@ Variant literal for a curve-adjacent corner: `filletProfile: corner <i> at (<x>,
 - **Cause:** two sharp-tagged vertices (or two contour corners) in the same section are numerically coincident, so the arc between them has zero length — e.g. tagging *both* ends of a closed NACA trailing edge, whose closure coefficient already brings them to the same point (or the same point after a per-section `rotate`). This also fires for the v1 meaning of the message: a genuinely degenerate section.
 - **Fix:** tag only one of the coincident vertices as the corner (see `src/parts/propeller.js`'s `sharpTE` comment for why a second tag there would create exactly this zero-length arc). See [AUTHORING-PARTS.md](AUTHORING-PARTS.md)'s `k.loftSmooth` row.
 
+## loftsmooth-stations-out-of-range
+
+- **Symptom:** `loftSmooth: stations must be 2…1024 (or "controls")` — an explicit `stations` was rejected.
+- **Cause:** `stations` is clamped to 2…1024 (a preview/export density guard); a non-integer, non-finite, or out-of-range value throws. The default (8 per span + 1 open, 8 per section closed) caps itself at 1024, so omitting the option never trips this.
+- **Fix:** pass an integer in 2…1024, or omit `stations` for the capped default. More rings than 1024 along the spine is a density smell — the surface is already spline-smooth between stations; raise `samples` instead if the banding is around the ring. See the `loftSmooth` row in [KERNEL-CONTRACT.md](KERNEL-CONTRACT.md).
+
+## loftsmooth-samples-out-of-range
+
+- **Symptom:** `loftSmooth: samples must be 8…2048` — possibly without ever passing `samples`.
+- **Cause:** `samples` is clamped to 8…2048. The default is `max(64, largest section)`, so a control section with more than 2048 points pushes the DEFAULT out of range — the throw then names an option the caller never passed.
+- **Fix:** pass a `samples` value in 8…2048, and thin any control section above ~2048 points — `loftSmooth` interpolates a smooth outline through the points, so sections that dense defeat the sparse-sections design (pass the dense rings straight to `k.loft` instead). See the `loftSmooth` row in [KERNEL-CONTRACT.md](KERNEL-CONTRACT.md).
+
 ## duplicate-preset-name-throws
 
 - **Symptom:** `duplicate preset name across sections:` thrown from verify/measure, naming the repeated preset (e.g. `duplicate preset name across sections: "Compact"`).

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "partforge",
-  "version": "0.85.0",
+  "version": "0.85.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/loft-smooth.js

@@ -303,8 +303,8 @@ function reconcile(resolved, V) {
  *                          corners implicitly.
  * @param {{stations?: number|"controls", samples?: number, closed?: boolean}} opts
  *   stations — output ring count along the spine (default 8 per span + 1 open,
- *     8 per section closed; ≥ 2; raised to the section count when lower; every
- *     control knot is always emitted). The string "controls" skips cross-station
+ *     8 per section closed, capped at 1024; ≥ 2; raised to the section count when
+ *     lower; every control knot is always emitted). The string "controls" skips cross-station
  *     interpolation entirely and emits one ring per control section at its own z
  *     — the B-rep path, where the backend's native smooth loft (`ruled: false`)
  *     does the skinning through exact wires and only the around-ring
@@ -320,7 +320,9 @@ export function smoothLoftRings(sections, { stations, samples, closed = false }
   if (closed && stations === "controls")
     throw new Error('loftSmooth: closed:true cannot combine with stations:"controls"');
   if (closed && n < 3) throw new Error("loftSmooth: closed:true needs at least 3 control sections");
-  const S = stations ?? (closed ? n * 8 : (n - 1) * 8 + 1);
+  // The default caps at the clamp ceiling: at 129+ sections the raw formula
+  // would trip the range check on an option the caller never passed.
+  const S = stations ?? Math.min(closed ? n * 8 : (n - 1) * 8 + 1, 1024);
   const V = samples ?? Math.max(64, ...resolved.map((r) => r.pts2d.length));
   if (stations !== "controls" && !(Number.isFinite(S) && S >= 2 && S <= 1024))
     throw new Error('loftSmooth: stations must be 2…1024 (or "controls")');

package/types/kernel.d.ts

@@ -323,7 +323,8 @@ export interface ExtrudeOptions {
 }
 
 export interface LoftRing {
-  polygon?: PointsContour;
+  /** Points, a curve contour, or a single-region hole-free `Shape2D` outline. */
+  polygon?: Contour | Shape2D;
   sides?: number;
   radius?: number;
   z: number;
@@ -398,7 +399,7 @@ export interface LoftSmoothSection {
 export interface LoftSmoothOptions {
   /** Sparse control sections; vertex counts may differ between sections. */
   sections: LoftSmoothSection[];
-  /** Output ring count along the spine (default 8 per span + 1; closed: 8 per section). */
+  /** Output ring count along the spine (default 8 per span + 1; closed: 8 per section; capped at 1024). */
   stations?: number;
   /** Output vertex count around each ring (default max(64, largest section)). */
   samples?: number;