@toolpath/viewer 0.2.0 → 0.3.1

package/README.md

@@ -22,7 +22,7 @@ import { EnginePart } from '@toolpath/viewer/engine'
 </Suspense>
 ```
 
-`EnginePart` takes a `PartReportResponse` exactly as `@toolpath/api` returns it and validates it:
+`EnginePart` takes a `PartResponse` exactly as `@toolpath/api` returns it and validates it:
 a malformed report throws `PartReportFormatError` carrying every problem it found, and one from a
 kernel older than `0.3.0` — before `regions[]` and `featureTag` existed — throws
 `UnsupportedKernelVersionError`. It fetches `meshGlbUrl`, falls back to `meshStlUrl`, and refuses a

package/dist/{chunk-7NQBV7EQ.js → chunk-RICG6UVH.js}

@@ -47,107 +47,6 @@ function trim(value) {
   return Number.parseFloat(value.toFixed(3)).toString();
 }
 
-// src/model/surfaces.ts
-var CONTINUES_WITHIN = Math.cos(1 * Math.PI / 180);
-var MERGEABLE = "Plane";
-var worked = /* @__PURE__ */ new WeakMap();
-function visualSurfaces(geometry, regions) {
-  const already = worked.get(geometry);
-  if (already && already.regions === regions) return already.of;
-  const found = computeSurfaces(geometry, regions);
-  worked.set(geometry, { regions, of: found });
-  return found;
-}
-function computeSurfaces(geometry, regions) {
-  const position = geometry.getAttribute("position");
-  const surfaces = /* @__PURE__ */ new Map();
-  for (const region of regions) surfaces.set(region.idx, region.idx);
-  if (!position || geometry.index) return surfaces;
-  const triangleCount = Math.floor(position.count / 3);
-  const regionOf = new Int32Array(triangleCount).fill(-1);
-  const kindOf = /* @__PURE__ */ new Map();
-  for (const region of regions) {
-    kindOf.set(region.idx, region.shapeKind);
-    const end = Math.min(region.triangles.end, triangleCount);
-    for (let triangle = region.triangles.start; triangle < end; triangle += 1) {
-      regionOf[triangle] = region.idx;
-    }
-  }
-  const normals = facetNormals(geometry, triangleCount);
-  const parent = new Map(surfaces);
-  const find = (idx) => {
-    let root = idx;
-    while (parent.get(root) !== root) root = parent.get(root) ?? root;
-    let walk = idx;
-    while (parent.get(walk) !== root) {
-      const next = parent.get(walk) ?? root;
-      parent.set(walk, root);
-      walk = next;
-    }
-    return root;
-  };
-  const union = (a, b) => {
-    const [rootA, rootB] = [find(a), find(b)];
-    if (rootA !== rootB) parent.set(rootB, rootA);
-  };
-  const seen = /* @__PURE__ */ new Map();
-  for (let triangle = 0; triangle < triangleCount; triangle += 1) {
-    for (let corner = 0; corner < 3; corner += 1) {
-      const a = triangle * 3 + corner;
-      const b = triangle * 3 + (corner + 1) % 3;
-      const id = edgeKey(position, a, b);
-      const met = seen.get(id);
-      if (met === void 0) {
-        seen.set(id, triangle);
-        continue;
-      }
-      const here = regionOf[triangle] ?? -1;
-      const there = regionOf[met] ?? -1;
-      if (here === -1 || there === -1 || here === there) continue;
-      if (kindOf.get(here) !== kindOf.get(there)) continue;
-      const facing = normals[triangle * 3] * normals[met * 3] + normals[triangle * 3 + 1] * normals[met * 3 + 1] + normals[triangle * 3 + 2] * normals[met * 3 + 2];
-      if (kindOf.get(here) !== MERGEABLE) continue;
-      if (facing >= CONTINUES_WITHIN) union(here, there);
-    }
-  }
-  for (const region of regions) surfaces.set(region.idx, find(region.idx));
-  return surfaces;
-}
-function facetNormals(geometry, triangleCount) {
-  const position = geometry.getAttribute("position");
-  const normals = new Float32Array(triangleCount * 3);
-  for (let triangle = 0; triangle < triangleCount; triangle += 1) {
-    const at = triangle * 3;
-    const ax = position.getX(at);
-    const ay = position.getY(at);
-    const az = position.getZ(at);
-    const bx = position.getX(at + 1) - ax;
-    const by = position.getY(at + 1) - ay;
-    const bz = position.getZ(at + 1) - az;
-    const cx = position.getX(at + 2) - ax;
-    const cy = position.getY(at + 2) - ay;
-    const cz = position.getZ(at + 2) - az;
-    const nx = by * cz - bz * cy;
-    const ny = bz * cx - bx * cz;
-    const nz = bx * cy - by * cx;
-    const length = Math.hypot(nx, ny, nz) || 1;
-    normals[at] = nx / length;
-    normals[at + 1] = ny / length;
-    normals[at + 2] = nz / length;
-  }
-  return normals;
-}
-function edgeKey(position, a, b) {
-  const ax = position.getX(a);
-  const ay = position.getY(a);
-  const az = position.getZ(a);
-  const bx = position.getX(b);
-  const by = position.getY(b);
-  const bz = position.getZ(b);
-  const first = ax < bx || ax === bx && (ay < by || ay === by && az <= bz);
-  return first ? `${ax},${ay},${az}|${bx},${by},${bz}` : `${bx},${by},${bz}|${ax},${ay},${az}`;
-}
-
 // src/render/theme.ts
 var HIGHLIGHT_COLORS = {
   default: 16753434,
@@ -262,31 +161,6 @@ function applyHighlightLayers(part, layers, theme) {
   if (layers.hoverRegion != null && !selectedRegions(part, layers).has(layers.hoverRegion)) {
     part.paintRegion(layers.hoverRegion, theme.hover, HOVER_WEIGHT);
   }
-  spreadAcrossSurfaces(part);
-}
-function spreadAcrossSurfaces(part) {
-  const surfaces = visualSurfaces(part.mesh.geometry, part.model.regions);
-  const claims = /* @__PURE__ */ new Map();
-  for (const region of part.model.regions) {
-    const paint = part.regionPaint(region.idx);
-    if (!paint || paint.weight === 0) continue;
-    const surface = surfaces.get(region.idx) ?? region.idx;
-    const claim = claims.get(surface);
-    if (!claim) {
-      claims.set(surface, { color: paint.color, weight: paint.weight });
-      continue;
-    }
-    if (claim === "contested") continue;
-    if (claim.color !== paint.color || claim.weight !== paint.weight) {
-      claims.set(surface, "contested");
-    }
-  }
-  for (const region of part.model.regions) {
-    const paint = part.regionPaint(region.idx);
-    if (paint && paint.weight > 0) continue;
-    const claim = claims.get(surfaces.get(region.idx) ?? region.idx);
-    if (claim && claim !== "contested") part.paintRegion(region.idx, claim.color, claim.weight);
-  }
 }
 
 // src/render/section.ts
@@ -432,13 +306,20 @@ import {
 
 // src/render/edges.ts
 import { BufferGeometry as Buffer, Float32BufferAttribute } from "three";
+
+// src/model/surfaces.ts
+function visualSurfaces(regions) {
+  return new Map(regions.map((region) => [region.idx, region.splitOrigin]));
+}
+
+// src/render/edges.ts
 function regionEdgesGeometry(geometry, model) {
   const position = geometry.getAttribute("position");
   const edges = new Buffer();
   edges.setAttribute("position", new Float32BufferAttribute([], 3));
   if (!position || geometry.index) return edges;
   const triangleCount = Math.floor(position.count / 3);
-  const surfaces = visualSurfaces(geometry, model.regions);
+  const surfaces = visualSurfaces(model.regions);
   const regionOf = new Int32Array(triangleCount).fill(-1);
   for (const region of model.regions) {
     const end = Math.min(region.triangles.end, triangleCount);
@@ -1942,7 +1823,7 @@ function smoothRegionNormals(geometry, regions) {
   if (!position || geometry.index) return;
   const vertexCount = position.count;
   const triangleCount = Math.floor(vertexCount / 3);
-  const surfaces = visualSurfaces(geometry, regions);
+  const surfaces = visualSurfaces(regions);
   const regionOf = new Int32Array(triangleCount).fill(-1);
   for (const region of regions) {
     const end = Math.min(region.triangles.end, triangleCount);
@@ -2304,15 +2185,16 @@ function readRegions(value, issues) {
       continue;
     }
     const idx = raw["idx"];
+    const splitOrigin = raw["splitOrigin"];
     const start = raw["triangleStart"];
     const end = raw["triangleEnd"];
     const area = raw["area"];
     const shapeKind = raw["shapeKind"];
-    if (!isNonNegativeInteger(idx) || !isNonNegativeInteger(start) || !isNonNegativeInteger(end) || !isFiniteNumber(area) || !isString(shapeKind)) {
+    if (!isNonNegativeInteger(idx) || !isNonNegativeInteger(splitOrigin) || !isNonNegativeInteger(start) || !isNonNegativeInteger(end) || !isFiniteNumber(area) || !isString(shapeKind)) {
       issues.push(`regions[${i}] does not match the Region schema`);
       continue;
     }
-    regions.push({ idx, shapeKind, area, triangles: { start, end } });
+    regions.push({ idx, splitOrigin, shapeKind, area, triangles: { start, end } });
   }
   return regions;
 }
@@ -2446,8 +2328,6 @@ export {
   directionIndexOf,
   groupByDirection,
   directionLabel,
-  CONTINUES_WITHIN,
-  visualSurfaces,
   HIGHLIGHT_COLORS,
   DIRECTION_COLORS,
   DEFAULT_THEME,
@@ -2476,6 +2356,7 @@ export {
   movedFar,
   trackTaps,
   useTapGuard,
+  visualSurfaces,
   regionEdgesGeometry,
   REGION_ATTRIBUTE,
   buildRegionTexels,

package/dist/engine/index.d.ts

@@ -1,5 +1,5 @@
-import { P as PartModel, a as PartMeshRefs } from '../normalize-B0HBvzGu.js';
-export { E as EnginePart, b as EnginePartProps, M as MIN_KERNEL_VERSION, c as assertSupportedKernelVersion, n as normalizePartReport, s as smoothRegionNormals } from '../normalize-B0HBvzGu.js';
+import { P as PartModel, a as PartMeshRefs } from '../normalize-Bzzwwxt4.js';
+export { E as EnginePart, b as EnginePartProps, M as MIN_KERNEL_VERSION, c as assertSupportedKernelVersion, n as normalizePartReport, s as smoothRegionNormals } from '../normalize-Bzzwwxt4.js';
 import { BufferGeometry } from 'three';
 import 'react';
 

package/dist/engine/index.js

@@ -12,7 +12,7 @@ import {
   parsePartGeometry,
   partMeshAssets,
   smoothRegionNormals
-} from "../chunk-7NQBV7EQ.js";
+} from "../chunk-RICG6UVH.js";
 export {
   EnginePart,
   MIN_KERNEL_VERSION,

package/dist/index.d.ts

@@ -1,5 +1,5 @@
-import { V as Vec3, d as ViewerTheme, P as PartModel, e as PartModelRegion, f as ViewerCamera, g as ViewerView, h as Projection, T as TriangleRange, F as FeatureTag, R as RegionIndex, i as PartModelFeature, j as FeatureType } from './normalize-B0HBvzGu.js';
-export { B as BuildPickInput, C as CAD_CAMERA_UP, k as CANDIDATE_WEIGHT, D as DEFAULT_FIT_MARGIN, l as DEFAULT_THEME, m as DIRECTION_COLORS, o as EXCLUDE_FROM_FRAME, E as EnginePart, p as FeatureHighlight, H as HANDLE_PIXELS, q as HIGHLIGHT_COLORS, r as HIGHLIGHT_WEIGHT, t as HOVER_WEIGHT, u as HighlightLayers, K as KnownFeatureType, N as NO_MODIFIERS, v as PERSPECTIVE_FOV, w as PICKED_SURFACE_LABEL, x as PartMesh, y as PartMeshProps, a as PartMeshRefs, z as PartObject, A as PartPick, G as PickModifiers, I as REGION_ATTRIBUTE, J as RegionHighlight, L as RegionPaint, S as SECTION_RENDER_ORDER, O as SceneBounds, Q as SectionAnchor, U as SectionBounds, W as SectionOptions, X as SectionPlacement, Y as SectionState, Z as SectionView, _ as ViewportSize, $ as applyHighlightLayers, a0 as applyProjection, a1 as aspectRatio, a2 as boundsFromBox, a3 as buildPick, a4 as buildRegionAttribute, a5 as buildRegionTexels, a6 as cadViewDirections, a7 as contentBounds, a8 as createPart, a9 as currentViewDirection, aa as defaultBounds, ab as directionColor, ac as dragPlane, ad as fitDistance, ae as focusForPick, n as normalizePartReport, af as orthographicHalfHeight, ag as perspectiveFitDistance, ah as pickedStartDepth, ai as resolveSectionPlane, aj as resolveTheme, ak as screenLength, al as sectionBounds, am as sectionConstant, an as sectionDepth, ao as sectionDepthConstant, ap as sectionDepthRange, aq as sectionFromPick, ar as sectionOffset, as as sectionPlane, s as smoothRegionNormals, at as startPosition, au as themesEqual, av as viewDirection } from './normalize-B0HBvzGu.js';
+import { V as Vec3, d as ViewerTheme, P as PartModel, e as PartModelRegion, f as ViewerCamera, g as ViewerView, h as Projection, T as TriangleRange, F as FeatureTag, R as RegionIndex, i as PartModelFeature, j as FeatureType } from './normalize-Bzzwwxt4.js';
+export { B as BuildPickInput, C as CAD_CAMERA_UP, k as CANDIDATE_WEIGHT, D as DEFAULT_FIT_MARGIN, l as DEFAULT_THEME, m as DIRECTION_COLORS, o as EXCLUDE_FROM_FRAME, E as EnginePart, p as FeatureHighlight, H as HANDLE_PIXELS, q as HIGHLIGHT_COLORS, r as HIGHLIGHT_WEIGHT, t as HOVER_WEIGHT, u as HighlightLayers, K as KnownFeatureType, N as NO_MODIFIERS, v as PERSPECTIVE_FOV, w as PICKED_SURFACE_LABEL, x as PartMesh, y as PartMeshProps, a as PartMeshRefs, z as PartObject, A as PartPick, G as PickModifiers, I as REGION_ATTRIBUTE, J as RegionHighlight, L as RegionPaint, S as SECTION_RENDER_ORDER, O as SceneBounds, Q as SectionAnchor, U as SectionBounds, W as SectionOptions, X as SectionPlacement, Y as SectionState, Z as SectionView, _ as ViewportSize, $ as applyHighlightLayers, a0 as applyProjection, a1 as aspectRatio, a2 as boundsFromBox, a3 as buildPick, a4 as buildRegionAttribute, a5 as buildRegionTexels, a6 as cadViewDirections, a7 as contentBounds, a8 as createPart, a9 as currentViewDirection, aa as defaultBounds, ab as directionColor, ac as dragPlane, ad as fitDistance, ae as focusForPick, n as normalizePartReport, af as orthographicHalfHeight, ag as perspectiveFitDistance, ah as pickedStartDepth, ai as resolveSectionPlane, aj as resolveTheme, ak as screenLength, al as sectionBounds, am as sectionConstant, an as sectionDepth, ao as sectionDepthConstant, ap as sectionDepthRange, aq as sectionFromPick, ar as sectionOffset, as as sectionPlane, s as smoothRegionNormals, at as startPosition, au as themesEqual, av as viewDirection } from './normalize-Bzzwwxt4.js';
 import * as react from 'react';
 import { RefObject, ReactNode, CSSProperties } from 'react';
 import { BufferGeometry, CanvasTexture, Vector3, Box3 } from 'three';
@@ -283,59 +283,27 @@ declare function gridGeometry(spec: GridSpec): BufferGeometry;
  * With one qualification. The Engine splits a surface where that makes a better
  * machining plan, and those splits are boundaries between regions without being
  * edges of the part: a floor cut in two to be reached from two directions is
- * still one flat floor. So the walk is over *visual surfaces* — regions grouped
- * where they continue each other — and a split leaves no line. See
- * `visualSurfaces`; nothing about picking or features goes through it.
+ * still one face to look at. So the walk is over *visual surfaces* — regions
+ * grouped by the kernel's exact `splitOrigin` lineage — and a split leaves no
+ * line. Nothing about picking or features goes through that grouping.
  *
  * The mesh must be non-indexed, which `parsePartGeometry` guarantees.
  */
 declare function regionEdgesGeometry(geometry: BufferGeometry, model: Pick<PartModel, 'regions' | 'regionIndex'>): BufferGeometry;
 
+/** Region index → the visual surface it belongs to. */
+type SurfaceOf = ReadonlyMap<number, number>;
 /**
- * Which regions are one surface to look at.
- *
- * The Engine splits a surface where that makes a better machining plan — a
- * floor cut in two so each half can be reached from a different direction, say.
- * Those splits are real and features depend on them, but they are not edges of
- * the part: a plane divided in two is still flat, and drawing the join or
- * shading across it makes a clean model look faceted and creased where nothing
- * creases.
- *
- * So this groups regions that continue each other, and the grouping is used for
- * the two things that are about how the part *looks* — its edges and its
- * shading. Nothing about picking, highlighting or features goes through here:
- * a split is still two regions to click on and still two regions a feature can
- * own, which is the whole reason the Engine made it.
- *
- * Two regions continue each other when they meet along an edge, are both
- * **planes**, and are flat to within a degree of one another — which for a
- * plane means they are the same plane.
- *
- * Planes only, and that is a limit of what the report says rather than caution.
- * A region carries an `idx`, a `shapeKind`, an area and a triangle range: there
- * is nothing in it that says which analytic surface a region was cut from. On a
- * flat face that does not matter, because two coplanar planes meeting along an
- * edge *are* one plane and no part has an edge there. On a curved one it
- * matters entirely: a fillet running tangentially into a shaft and a fillet
- * split down the middle look identical from the facets alone, and guessing
- * between them either rubs out a line the part has or leaves one it does not.
- * A line the part has is the worse of the two to lose, so curved boundaries are
- * all drawn.
- *
- * The exact version of this wants the Engine to say which surface a region came
- * from. Until it does, this is the half that can be proved.
+ * The visual surface of each post-split region.
+ *
+ * The kernel divides B-rep faces to make machining relationships unambiguous.
+ * Those divisions remain meaningful to feature extraction and picking, but
+ * they are not physical edges. `splitOrigin` is the kernel's exact lineage
+ * relation: equal values mean the regions were one face before that analysis
+ * split. It is therefore the single grouping used for rendering, rather than
+ * attempting to infer continuations from mesh normals or surface kinds.
  */
-/**
- * How far two planes may disagree across a shared edge and still be one plane.
- *
- * A degree, which is a rounding error rather than a judgement: a split is
- * exactly coplanar, and anything a part actually turns through is a chamfer at
- * fifteen degrees or more.
- */
-declare const CONTINUES_WITHIN: number;
-/** Region index → the surface it belongs to, by region `idx`. */
-type SurfaceOf = ReadonlyMap<number, number>;
-declare function visualSurfaces(geometry: BufferGeometry, regions: readonly PartModelRegion[]): SurfaceOf;
+declare function visualSurfaces(regions: readonly PartModelRegion[]): SurfaceOf;
 
 /**
  * Mouse and trackpad presets.
@@ -634,4 +602,4 @@ declare function bestOwner(model: PartModel, owners: readonly FeatureTag[], cont
  */
 declare function cycleOwner(owners: readonly FeatureTag[], current: FeatureTag | null): FeatureTag | null;
 
-export { type ArrowPlacement, Axes, type AxesProps, CHAMFER, CONTINUES_WITHIN, CadCameraControls, type CadCameraControlsProps, type ControlScheme, type CubeZone, DirectionArrows, type DirectionArrowsProps, type DirectionGroup, ExtendedCameraControls, type ExtendedCameraControlsOptions, FEATURE_TYPE_RANKS, FeatureTag, FeatureType, Grid, type GridProps, type GridSpec, type NamedDirection, PartModel, PartModelFeature, PartModelRegion, PartReportFormatError, Projection, type RankingContext, RegionIndex, type SurfaceOf, TAP_SLOP, type TapPoint, type TapTracker, TriangleRange, UnsupportedKernelVersionError, VIEW_NAMES, VIEW_SIGNS, Vec3, ViewCube, type ViewCubeProps, type ViewKind, type ViewName, Viewer, ViewerCamera, type ViewerControls, type ViewerHandle, type ViewerProps, ViewerTheme, ViewerView, arrowPlacement, bestOwner, buildRegionIndex, cubeOutlineGeometry, cubeZones, cycleOwner, directionIndexOf, directionLabel, featureTypeRank, gridGeometry, gridSpec, groupByDirection, labelGeometry, labelTexture, movedFar, panelGeometry, rankOwners, regionEdgesGeometry, regionNormals, sameDirection, trackTaps, useContentBox, useTapGuard, useViewerControls, viewKind, viewUp, viewVector, visualSurfaces };
+export { type ArrowPlacement, Axes, type AxesProps, CHAMFER, CadCameraControls, type CadCameraControlsProps, type ControlScheme, type CubeZone, DirectionArrows, type DirectionArrowsProps, type DirectionGroup, ExtendedCameraControls, type ExtendedCameraControlsOptions, FEATURE_TYPE_RANKS, FeatureTag, FeatureType, Grid, type GridProps, type GridSpec, type NamedDirection, PartModel, PartModelFeature, PartModelRegion, PartReportFormatError, Projection, type RankingContext, RegionIndex, type SurfaceOf, TAP_SLOP, type TapPoint, type TapTracker, TriangleRange, UnsupportedKernelVersionError, VIEW_NAMES, VIEW_SIGNS, Vec3, ViewCube, type ViewCubeProps, type ViewKind, type ViewName, Viewer, ViewerCamera, type ViewerControls, type ViewerHandle, type ViewerProps, ViewerTheme, ViewerView, arrowPlacement, bestOwner, buildRegionIndex, cubeOutlineGeometry, cubeZones, cycleOwner, directionIndexOf, directionLabel, featureTypeRank, gridGeometry, gridSpec, groupByDirection, labelGeometry, labelTexture, movedFar, panelGeometry, rankOwners, regionEdgesGeometry, regionNormals, sameDirection, trackTaps, useContentBox, useTapGuard, useViewerControls, viewKind, viewUp, viewVector, visualSurfaces };

package/dist/index.js

@@ -2,7 +2,6 @@ import {
   CAD_CAMERA_UP,
   CANDIDATE_WEIGHT,
   CONE_AXIS,
-  CONTINUES_WITHIN,
   CadCameraControls,
   DEFAULT_FIT_MARGIN,
   DEFAULT_THEME,
@@ -81,7 +80,7 @@ import {
   useViewerControls,
   viewDirection,
   visualSurfaces
-} from "./chunk-7NQBV7EQ.js";
+} from "./chunk-RICG6UVH.js";
 
 // src/primitives.tsx
 import { GizmoHelper } from "@react-three/drei";
@@ -581,7 +580,6 @@ export {
   CAD_CAMERA_UP,
   CANDIDATE_WEIGHT,
   CHAMFER,
-  CONTINUES_WITHIN,
   CadCameraControls,
   DEFAULT_FIT_MARGIN,
   DEFAULT_THEME,

package/dist/{normalize-B0HBvzGu.d.ts → normalize-Bzzwwxt4.d.ts}

@@ -5,7 +5,7 @@ import { Group, Mesh, LineSegments, Box3, Plane, BufferGeometry, Camera, Vector3
  * The normalized part — the only thing the renderer consumes.
  *
  * `engine/` produces this; nothing under `model/` or `render/` sees a
- * `PartReportResponse`. The renderer therefore survives an API change, and the
+ * `PartResponse`. The renderer therefore survives an API change, and the
  * viewer can be driven from a local file with no API at all, which is also how
  * it gets tested.
  *
@@ -84,6 +84,16 @@ interface RegionIndex {
     rangeForRegion(region: number): TriangleRange | null;
 }
 interface PartModelRegion {
+    /**
+     * Opaque, report-local identity of the B-rep face this region came from before
+     * analysis split it. Equal values are one visual surface; it is not an index
+     * or a kernel surface identifier.
+     */
+    readonly splitOrigin: number;
+    /**
+     * Post-split region identity. Feature ownership, picking, and triangle
+     * ranges deliberately continue to use this rather than `splitOrigin`.
+     */
     readonly idx: number;
     readonly shapeKind: ShapeKind;
     /**
@@ -246,7 +256,7 @@ interface PartObject {
     paintRegion(region: number, color: number, weight: number): void;
     /** What a region is painted with now. `null` for a region it does not have. */
     regionPaint(region: number): RegionPaint | null;
-    /** Paints every region a feature owns. */
+    /** Paints every region the feature explicitly owns. */
     paintFeature(tag: FeatureTag, color: number, weight: number): void;
     clearPaint(): void;
     /** A feature's bounds in part space, for framing. `null` if it has none. */
@@ -265,8 +275,10 @@ interface RegionPaint {
     /** 0 for untouched, 1 for fully painted. */
     readonly weight: number;
 }
-/** The part of a `PartModel` the buffer builders need. */
-type RegionTable = Pick<PartModel, 'regions'>;
+/** The part of a region table the buffer builders need. */
+type RegionTable = {
+    readonly regions: readonly Pick<PartModel['regions'][number], 'idx' | 'triangles'>[];
+};
 /**
  * Maps a region's `idx` to its column in the state texture.
  *
@@ -821,10 +833,9 @@ declare const EnginePart: ({ report, ...props }: EnginePartProps) => react.JSX.E
  * triangle its own normal, which is honest but leaves a bore looking like a
  * fifty-sided nut, because that is exactly what its triangles are.
  *
- * Neither is necessary here, because the report says which triangles belong to
- * one analytic surface. Averaging *within* a region and never *across* one
- * gives a bore that shades like a bore and an edge that stays an edge: the
- * distinction a mesh cannot express is one the region table can.
+ * The report says which post-split regions came from one original B-rep face.
+ * Averaging within that exact visual surface and never across one gives a bore
+ * that shades like a bore and an edge that stays an edge.
  *
  * Two vertices are the same point if their coordinates match exactly. That is
  * safe rather than optimistic — `toNonIndexed` copies each shared vertex from

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@toolpath/viewer",
-  "version": "0.2.0",
+  "version": "0.3.1",
   "description": "React Three Fiber viewer for Toolpath Engine part reports",
   "license": "MIT",
   "repository": {