partforge 0.58.0 → 0.58.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "partforge",
-  "version": "0.58.0",
+  "version": "0.58.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/mount.js

@@ -346,6 +346,14 @@ export function mount(part, { createWorker, elements = {}, onBuild, onPick, onDo
     if (onPick) {
       picker = attachPicker(viewer, {
         part, getContext,
+        // Measure mode claims canvas clicks for pinning dimensions, so while it
+        // is on a click must not ALSO select-and-flash (hosts turn picks into
+        // chat chips — one click was doing both). Same idea as the hover
+        // suppression above, but pull-based: checked per click, nothing to
+        // resync on mode changes. The ?pick/?pickserver harnesses below are
+        // deliberately not guarded — one is armed by an explicit dev toggle,
+        // the other per agent request.
+        suppressed: () => measureMode.isEnabled(),
         onPick: (selection) => onPick({
           selection,
           label: selection.feature?.label ?? part.parts[selection.subPart]?.label ?? selection.subPart,

package/src/framework/selection/pick.js

@@ -6,13 +6,19 @@ import { createDragTracker } from "./drag-tracker.js";
 
 export { worldToSubPartLocal };
 
-export function attachPicker(viewer, { part, getContext, onPick }) {
+// `suppressed` is an optional pull-based guard checked per click, for a caller
+// whose suppression condition lives elsewhere (mount passes measure mode's
+// isEnabled): while it returns true a click neither raycasts, flashes, nor
+// picks — no resync bookkeeping the way an event-driven setActive would need.
+export function attachPicker(viewer, { part, getContext, onPick, suppressed }) {
   let active = false;
   const drag = createDragTracker();
 
   function onClick(ev) {
+    // consumeClick() first, unconditionally — the drag tracker is stateful and
+    // a suppressed click must still clear its just-dragged flag.
     const wasDragged = drag.consumeClick();
-    if (!active || wasDragged) return;
+    if (!active || wasDragged || suppressed?.()) return;
     const hit = raycastViewer(viewer, ev.clientX, ev.clientY);
     if (!hit) return;
     const selection = resolveSelection(part, getContext(), hit);