partforge 0.125.0 → 0.126.0

package/README.md

@@ -108,6 +108,7 @@ const runtime = mount(part, {
   },
   onBuild: ({ status, ms, error }) => {},   // per accepted build: "success" | "error"
   onPick: ({ selection, label, prompt, token }) => {}, // programmatic click-to-select
+  pickHint: "Click to edit",                // hover-tooltip nudge while onPick is set (null = none)
 });
 await runtime.ready;   // first successful build (rejects on a first-build error)
 runtime.setHostPane("rail");  // narrow layout only: show just the controls
@@ -221,7 +222,10 @@ full contract.
 back to the sub-part label/name) for compact UI, `prompt` is the LLM-ready
 sentence, `token` the compact form, `selection` the raw object. When `onPick` is
 set, the `?pick` / `?pickserver` URL modes are ignored (one click listener ever
-live); hover labels stay always-on.
+live); hover labels stay always-on, and gain a `pickHint` line under the name
+("Click to edit" by default; pass your own wording, or `null` for none). Hover
+never tints the surface under the pointer — a per-face highlight read as "only
+this spot is clickable".
 
 ## Authoring guide
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "partforge",
-  "version": "0.125.0",
+  "version": "0.126.0",
   "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/app.css

@@ -1039,6 +1039,10 @@ button.action:focus-visible, .adv-toggle:focus-visible, .sec-title:focus-visible
 .pf-hover-tip .pf-hover-sub { color: var(--pf-muted); font-size: 11px; margin-left: 7px; }
 #pf-hover-tip .pf-hover-sub:empty,
 .pf-hover-tip .pf-hover-sub:empty { display: none; }
+#pf-hover-tip .pf-hover-hint,
+.pf-hover-tip .pf-hover-hint { color: var(--pf-muted); font-size: 10.5px; margin-top: 1px; }
+#pf-hover-tip .pf-hover-hint:empty,
+.pf-hover-tip .pf-hover-hint:empty { display: none; }
 
 /* request-a-pick: agent prompt banner, floated top-centre well below the part tabs,
    laid out like a chat message (avatar + text). Slides/fades in when shown; themed. */

package/src/framework/mount.js

@@ -419,7 +419,7 @@ function createCleanupStack() {
 // Every `elements` entry defaults to the legacy global-ID lookup (below), resolved
 // exactly once here — submodules take element refs and never query the document.
 // `container`/`controls` remain as deprecated aliases for elements.viewer/.controls.
-export function mount(part, { createWorker, elements = {}, onBuild, onPick, onDownload, onViewChange, onParamsCommit, onAnnotationSend,
+export function mount(part, { createWorker, elements = {}, onBuild, onPick, pickHint = "Click to edit", onDownload, onViewChange, onParamsCommit, onAnnotationSend,
                               fontCatalog,
                               imageCatalog,
                               onAssetUpload,
@@ -497,7 +497,10 @@ export function mount(part, { createWorker, elements = {}, onBuild, onPick, onDo
       onRailLayout: () => railChrome.layoutChanged?.(),
     });
     cleanup.defer(() => paneTabs.detach());
-    const hover = attachHoverLabels(viewer, { part, tooltip }); // always-on hover inspection (no-op on touch-only devices)
+    // Always-on hover inspection (no-op on touch-only devices). With onPick the
+    // host acts on a click, so the tooltip nudges toward it with `pickHint`
+    // (a host passes its own wording, or null for none).
+    const hover = attachHoverLabels(viewer, { part, tooltip, hint: onPick ? pickHint : null });
     cleanup.defer(() => hover.detach());
     const ui = createStatusUi({ ...els.status, exports: [els.exports.stl, els.exports.step, els.exports.threeMf] });
     cleanup.defer(() => ui.setStatus(""));

package/src/framework/selection/hover.js

@@ -1,15 +1,19 @@
 // Always-on hover inspection: a cursor-following tooltip naming the feature +
-// sub-part under the pointer, and an overlay mesh highlighting the feature's
-// surface. Feature names come from Solid.label() in the part's build, carried
-// per-triangle in the mesh payload (geometry.userData.featureIds/features).
+// sub-part under the pointer. Feature names come from Solid.label() in the
+// part's build, carried per-triangle in the mesh payload
+// (geometry.userData.featureIds/features).
+//
+// There is deliberately NO surface highlight: tinting just the feature under
+// the pointer read as "only this face is clickable", and people stopped
+// clicking elsewhere on the part. `hint` (e.g. "Click to edit") adds a line
+// under the name when a click does something — mount passes it with onPick.
 import { createTooltipPresenter } from "../tooltip.js";
 import { raycastViewer } from "./raycast.js";
-import { createFeatureHighlight } from "./feature-highlight.js";
 import { runCleanupSteps } from "../teardown.js";
 
 export function attachHoverLabels(
   viewer,
-  { part, schedule = (cb) => requestAnimationFrame(cb), tooltip } = {},
+  { part, schedule = (cb) => requestAnimationFrame(cb), tooltip, hint = null } = {},
 ) {
   // Hover is a mouse idiom — skip entirely on touch-only devices. The stub
   // still answers the WHOLE interface: mount.js calls setSuppressed from the
@@ -25,8 +29,6 @@ export function attachHoverLabels(
   let presentationToken;
   let hasPresented = false;
 
-  const highlight = createFeatureHighlight(viewer);
-
   const subLabel = (name) => part.parts[name]?.label ?? name;
 
   function hide() {
@@ -35,7 +37,6 @@ export function attachHoverLabels(
       tooltipPresenter.hide(presentationToken);
       presentationToken = undefined;
     }
-    highlight.clear();
   }
 
   function show(hit, x, y) {
@@ -45,7 +46,7 @@ export function attachHoverLabels(
     } else {
       content = { title: subLabel(hit.subPart), subtitle: "" };
     }
-    highlight.show(hit);
+    if (hint) content.hint = hint;
     if (hasPresented) {
       hasPresented = false;
       tooltipPresenter.hide(presentationToken);
@@ -119,7 +120,6 @@ export function attachHoverLabels(
         () => viewer.domElement.removeEventListener("pointerup", onUp),
         () => viewer.domElement.removeEventListener("pointerleave", onLeave),
         hide,
-        () => highlight.dispose(),
         () => { if (ownsTooltip) tooltipPresenter.dispose(); },
       ], "feature hover cleanup failed");
     },

package/src/framework/tooltip.js

@@ -14,7 +14,10 @@ export function createTooltipPresenter({ id = "pf-hover-tip" } = {}) {
   const title = document.createElement("b");
   const subtitle = document.createElement("span");
   subtitle.className = "pf-hover-sub";
-  element.append(title, subtitle);
+  // A call-to-action line under the name ("Click to edit"); empty = hidden.
+  const hint = document.createElement("div");
+  hint.className = "pf-hover-hint";
+  element.append(title, subtitle, hint);
   document.body.appendChild(element);
   let disposed = false;
   const claims = [];
@@ -22,6 +25,7 @@ export function createTooltipPresenter({ id = "pf-hover-tip" } = {}) {
   function setContent(content) {
     title.textContent = content.title;
     subtitle.textContent = content.subtitle ?? "";
+    hint.textContent = content.hint ?? "";
   }
 
   function measureVisible(content, anchored) {