@pascal-app/nodes 1.0.7 → 1.1.0

package/dist/duct-segment/profile.d.ts

@@ -0,0 +1,42 @@
+import { Vector3 } from 'three';
+export declare const INCHES_TO_METERS = 0.0254;
+/**
+ * Area-equivalent round diameter (inches) for a rect cross-section —
+ * what a rect trunk advertises on its ports so round fittings / branches
+ * mate at a sensible size.
+ */
+export declare function equivalentDiameterIn(widthIn: number, heightIn: number): number;
+/**
+ * Area-equivalent round diameter (inches) for a flat-oval cross-section:
+ * a rectangle of (width − height) × height plus the two semicircular caps.
+ */
+export declare function ovalEquivalentDiameterIn(widthIn: number, heightIn: number): number;
+/** The diameter (inches) a duct segment presents at its ports. */
+export declare function ductPortDiameterIn(node: {
+    shape?: 'round' | 'rect' | 'oval';
+    diameter: number;
+    width?: number;
+    height?: number;
+}): number;
+/**
+ * Cross-section axes for a rect run along `dir`, rolled `roll` radians
+ * about the run direction. At roll 0: width is the horizontal axis
+ * (UP × dir) and height the vertical one — vertical runs, where that
+ * cross product degenerates, fall back to world X/Z. `roll` rotates the
+ * pair in the plane perpendicular to `dir`, letting a riser carry the
+ * orientation of the run it turned off instead of the bare fallback.
+ */
+export declare function rectSectionAxes(dir: Vector3, roll?: number): {
+    width: Vector3;
+    height: Vector3;
+};
+/**
+ * Roll (radians) that keeps a rect cross-section continuous across an
+ * elbow: the dimension lying along the joint's hinge — the bend-plane
+ * normal `portDir × newDir`, perpendicular to both legs — must stay on
+ * the same physical face on the new run as on the source run. Returns 0
+ * for an in-plane (degenerate-normal) joint, so horizontal turns keep
+ * the natural width-horizontal orientation.
+ */
+export declare function rollToContinueAcrossElbow(sourceDir: Vector3, sourceRoll: number, portDir: Vector3, newDir: Vector3): number;
+//# sourceMappingURL=profile.d.ts.map

package/dist/duct-segment/profile.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"profile.d.ts","sourceRoot":"","sources":["../../src/duct-segment/profile.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,OAAO,EAAE,MAAM,OAAO,CAAA;AAE/B,eAAO,MAAM,gBAAgB,SAAS,CAAA;AAGtC;;;;GAIG;AACH,wBAAgB,oBAAoB,CAAC,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,MAAM,CAE9E;AAED;;;GAGG;AACH,wBAAgB,wBAAwB,CAAC,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,MAAM,CAKlF;AAED,kEAAkE;AAClE,wBAAgB,kBAAkB,CAAC,IAAI,EAAE;IACvC,KAAK,CAAC,EAAE,OAAO,GAAG,MAAM,GAAG,MAAM,CAAA;IACjC,QAAQ,EAAE,MAAM,CAAA;IAChB,KAAK,CAAC,EAAE,MAAM,CAAA;IACd,MAAM,CAAC,EAAE,MAAM,CAAA;CAChB,GAAG,MAAM,CAQT;AAED;;;;;;;GAOG;AACH,wBAAgB,eAAe,CAAC,GAAG,EAAE,OAAO,EAAE,IAAI,SAAI,GAAG;IAAE,KAAK,EAAE,OAAO,CAAC;IAAC,MAAM,EAAE,OAAO,CAAA;CAAE,CAW3F;AAED;;;;;;;GAOG;AACH,wBAAgB,yBAAyB,CACvC,SAAS,EAAE,OAAO,EAClB,UAAU,EAAE,MAAM,EAClB,OAAO,EAAE,OAAO,EAChB,MAAM,EAAE,OAAO,GACd,MAAM,CAeR"}

package/dist/duct-segment/profile.js

@@ -0,0 +1,78 @@
+import { Vector3 } from 'three';
+export const INCHES_TO_METERS = 0.0254;
+const UP = new Vector3(0, 1, 0);
+/**
+ * Area-equivalent round diameter (inches) for a rect cross-section —
+ * what a rect trunk advertises on its ports so round fittings / branches
+ * mate at a sensible size.
+ */
+export function equivalentDiameterIn(widthIn, heightIn) {
+    return 2 * Math.sqrt((widthIn * heightIn) / Math.PI);
+}
+/**
+ * Area-equivalent round diameter (inches) for a flat-oval cross-section:
+ * a rectangle of (width − height) × height plus the two semicircular caps.
+ */
+export function ovalEquivalentDiameterIn(widthIn, heightIn) {
+    const minor = Math.min(widthIn, heightIn);
+    const major = Math.max(widthIn, heightIn);
+    const area = (major - minor) * minor + Math.PI * (minor / 2) ** 2;
+    return 2 * Math.sqrt(area / Math.PI);
+}
+/** The diameter (inches) a duct segment presents at its ports. */
+export function ductPortDiameterIn(node) {
+    if (node.shape === 'rect' && node.width && node.height) {
+        return equivalentDiameterIn(node.width, node.height);
+    }
+    if (node.shape === 'oval' && node.width && node.height) {
+        return ovalEquivalentDiameterIn(node.width, node.height);
+    }
+    return node.diameter;
+}
+/**
+ * Cross-section axes for a rect run along `dir`, rolled `roll` radians
+ * about the run direction. At roll 0: width is the horizontal axis
+ * (UP × dir) and height the vertical one — vertical runs, where that
+ * cross product degenerates, fall back to world X/Z. `roll` rotates the
+ * pair in the plane perpendicular to `dir`, letting a riser carry the
+ * orientation of the run it turned off instead of the bare fallback.
+ */
+export function rectSectionAxes(dir, roll = 0) {
+    const d = dir.clone().normalize();
+    const xBase = new Vector3().crossVectors(UP, d);
+    if (xBase.lengthSq() < 1e-8)
+        xBase.set(1, 0, 0);
+    xBase.normalize();
+    const zBase = new Vector3().crossVectors(xBase, d);
+    const c = Math.cos(roll);
+    const s = Math.sin(roll);
+    const width = xBase.clone().multiplyScalar(c).addScaledVector(zBase, s);
+    const height = xBase.clone().multiplyScalar(-s).addScaledVector(zBase, c);
+    return { width, height };
+}
+/**
+ * Roll (radians) that keeps a rect cross-section continuous across an
+ * elbow: the dimension lying along the joint's hinge — the bend-plane
+ * normal `portDir × newDir`, perpendicular to both legs — must stay on
+ * the same physical face on the new run as on the source run. Returns 0
+ * for an in-plane (degenerate-normal) joint, so horizontal turns keep
+ * the natural width-horizontal orientation.
+ */
+export function rollToContinueAcrossElbow(sourceDir, sourceRoll, portDir, newDir) {
+    const n = new Vector3().crossVectors(portDir, newDir);
+    if (n.lengthSq() < 1e-8)
+        return 0;
+    n.normalize();
+    const src = rectSectionAxes(sourceDir, sourceRoll);
+    const carriesWidth = Math.abs(src.width.dot(n)) >= Math.abs(src.height.dot(n));
+    const d = newDir.clone().normalize();
+    const xBase = new Vector3().crossVectors(UP, d);
+    if (xBase.lengthSq() < 1e-8)
+        xBase.set(1, 0, 0);
+    xBase.normalize();
+    const zBase = new Vector3().crossVectors(xBase, d);
+    // Place the hinge-aligned face on the same axis the source carries it.
+    return carriesWidth
+        ? Math.atan2(n.dot(zBase), n.dot(xBase))
+        : Math.atan2(-n.dot(xBase), n.dot(zBase));
+}

package/dist/duct-segment/selection.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"selection.d.ts","sourceRoot":"","sources":["../../src/duct-segment/selection.tsx"],"names":[],"mappings":"AAgJA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA6BG;AACH,QAAA,MAAM,8BAA8B,0CAuCnC,CAAA;eA68Cc,8BAA8B"}
+{"version":3,"file":"selection.d.ts","sourceRoot":"","sources":["../../src/duct-segment/selection.tsx"],"names":[],"mappings":"AAqJA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA6BG;AACH,QAAA,MAAM,8BAA8B,0CAuCnC,CAAA;eA4gDc,8BAA8B"}

package/dist/duct-segment/selection.js

@@ -1,7 +1,7 @@
 'use client';
 import { Fragment as _Fragment, jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
-import { analyzePortConnectivity, DuctSegmentNode, nodeRegistry, pauseSceneHistory, resolveConnectivityUpdates, resumeSceneHistory, sceneRegistry, useLiveNodeOverrides, useScene, } from '@pascal-app/core';
-import { clearPlacementSurface, DimensionPill, isAngleSnapActive, isGridSnapActive, swallowNextClick, triggerSFX, useEditor, } from '@pascal-app/editor';
+import { analyzePortConnectivity, DuctSegmentNode, findLevelAncestorId, nodeRegistry, pauseSceneHistory, resolveConnectivityUpdates, resumeSceneHistory, sceneRegistry, useLiveNodeOverrides, useScene, } from '@pascal-app/core';
+import { AXIS_MOVE_DRAG_LABEL, clearPlacementSurface, DimensionPill, isAngleSnapActive, isGridSnapActive, PATH_POINT_DRAG_LABEL, ROTATE_HANDLE_DRAG_LABEL, swallowNextClick, triggerSFX, useEditor, } from '@pascal-app/editor';
 import { collectScenePorts, findNearestPortXZ } from '@pascal-app/editor/distribution';
 import { useViewer } from '@pascal-app/viewer';
 import { Html } from '@react-three/drei';
@@ -11,6 +11,7 @@ import { Euler, Matrix4, Plane, Quaternion, Raycaster, Vector2, Vector3, } from
 import { newAutoOffsetGroupId, readAutoOffsetTag, translateAutoOffsetBase, withAutoOffsetTag, withoutAutoOffsetTag, } from '../shared/auto-offset-tag.js';
 import { planRunEndCapFollowUpdates } from '../shared/automatic-run-end-cap.js';
 import { detectFittingEndpoint, planFittingEndpointReaim, } from '../shared/fitting-endpoint-reaim.js';
+import { useHandleDragScope } from '../shared/handle-drag-scope.js';
 import { DuctSegmentGhost, FittingGhost } from '../shared/mep-ghost.js';
 import { DUCT_PORT_SYSTEMS } from '../shared/port-systems.js';
 import { planRunTranslationOffsets } from '../shared/run-translation-offset.js';
@@ -18,7 +19,7 @@ import { ContinuePlusHandle, HandleCube, MoveChevron, RotateArc } from '../share
 import { planVerticalOffsets } from '../shared/vertical-offset.js';
 import { refreshWallRunAttachment } from '../shared/wall-run-move.js';
 import { activateDuctContinuation, ductContinuationHandlePlan, } from './continuation.js';
-import { INCHES_TO_METERS } from './geometry.js';
+import { INCHES_TO_METERS } from './profile.js';
 /** Port-snap radius for dragged run endpoints (meters, XZ). */
 const PORT_SNAP_RADIUS_M = 0.4;
 // In-world arrow handle layout (meters) — the arrows stand off the run body so
@@ -146,6 +147,7 @@ const DuctPointHandles = ({ duct, target }) => {
     });
     const unit = useViewer((s) => s.unit);
     const [draggingIndex, setDraggingIndex] = useState(null);
+    const claimHandleDrag = useHandleDragScope();
     const [rolling, setRolling] = useState(false);
     const [runMoving, setRunMoving] = useState(false);
     // Auto-routed offset preview shown while the run-center cube's ±Y arrows
@@ -307,9 +309,12 @@ const DuctPointHandles = ({ duct, target }) => {
             ]);
         }
         const path = drag.initialPath.map((p, i) => (i === drag.index ? next : p));
+        const followers = detached ? [] : connectivityUpdatesForPath(drag.connectivity, path);
+        if (!followers)
+            return null;
         const updates = [
             { id: duct.id, data: { path, wallAttachment: attachmentFor(path) } },
-            ...(detached ? [] : connectivityUpdatesForPath(drag.connectivity, path)),
+            ...followers,
         ];
         return detached ? updates : withEndCapFollow(path, updates);
     };
@@ -322,12 +327,13 @@ const DuctPointHandles = ({ duct, target }) => {
     };
     // Follow-updates for fittings / ducts mated to this run's endpoints, given
     // the run's live path. Endpoints whose position didn't change resolve to a
-    // zero delta, so only the dragged endpoint's partner actually moves.
+    // zero delta, so only the dragged endpoint's partner actually moves. Null
+    // when the mated joints cannot follow the path.
     const connectivityUpdatesForPath = (connectivity, path) => {
         if (!connectivity)
             return [];
         const preview = { ...duct, path };
-        return resolveConnectivityUpdates(connectivity, preview).filter((u) => useScene.getState().nodes[u.id]);
+        return (resolveConnectivityUpdates(connectivity, preview)?.filter((u) => useScene.getState().nodes[u.id]) ?? null);
     };
     // A corner arrow drag: the point is locked to one axis (X / Z / Y). All the
     // rich behaviour — port-snap, elbow re-aim, connectivity follow, single-undo
@@ -342,10 +348,20 @@ const DuctPointHandles = ({ duct, target }) => {
         const livePreviewIds = new Set();
         const publishLivePreview = (updates) => {
             const scene = useScene.getState();
+            const overrides = useLiveNodeOverrides.getState();
             const entries = updates
                 .filter((update) => scene.nodes[update.id])
                 .map((update) => [update.id, update.data]);
-            useLiveNodeOverrides.getState().setMany(entries);
+            const previewed = new Set(entries.map(([id]) => id));
+            for (const id of livePreviewIds) {
+                if (previewed.has(id))
+                    continue;
+                overrides.clear(id);
+                if (scene.nodes[id])
+                    scene.markDirty(id);
+                livePreviewIds.delete(id);
+            }
+            overrides.setMany(entries);
             for (const [id] of entries) {
                 livePreviewIds.add(id);
                 scene.markDirty(id);
@@ -363,6 +379,10 @@ const DuctPointHandles = ({ duct, target }) => {
         useViewer.getState().setInputDragging(true);
         document.body.style.cursor = kind.axis === 'y' ? 'ns-resize' : 'grabbing';
         setDraggingIndex(index);
+        const releaseScope = claimHandleDrag(duct.id, PATH_POINT_DRAG_LABEL, () => {
+            swallowNextClick();
+            finish(false);
+        });
         const isEndpoint = index === 0 || index === initialPath.length - 1;
         // Swing pivot: the across-run (side) and up / down arrows DON'T stretch the
         // run — they sweep the grabbed point around its neighbour at a fixed radius
@@ -452,6 +472,7 @@ const DuctPointHandles = ({ duct, target }) => {
             if (isEndpoint && (detached || !drag.fittingEndpoint)) {
                 const port = findNearestPortXZ([next[0], next[1], next[2]], collectScenePorts({
                     excludeNodeId: duct.id,
+                    levelId: (findLevelAncestorId(duct.id, useScene.getState().nodes) ?? undefined),
                     systems: DUCT_PORT_SYSTEMS,
                 }), PORT_SNAP_RADIUS_M);
                 if (port)
@@ -471,16 +492,10 @@ const DuctPointHandles = ({ duct, target }) => {
                 triggerSFX('sfx:grid-snap');
             publishLivePreview(batch);
         };
-        const onUp = () => {
+        const finish = (commit) => {
             const drag = dragRef.current;
             if (!drag)
                 return;
-            // Swallow the trailing synthetic click so it doesn't reach the
-            // background-click deselect handler — `cleanup()` drops `inputDragging`
-            // synchronously here, so without this the click that fires after
-            // pointerup would land with the drag gate already down and clear the
-            // selection. Mirrors `useHandleDrag`'s onUp.
-            swallowNextClick();
             drag.cleanup();
             dragRef.current = null;
             setDraggingIndex(null);
@@ -491,7 +506,7 @@ const DuctPointHandles = ({ duct, target }) => {
             // detached — just the duct path).
             const detached = drag.detached;
             const moved = drag.current.some((v, axis) => v !== drag.initialPath[drag.index][axis]);
-            const finalBatch = buildDragBatch(drag, drag.current, detached);
+            const finalBatch = commit ? buildDragBatch(drag, drag.current, detached) : null;
             resumeSceneHistory(useScene);
             if (moved && finalBatch) {
                 // A manual corner edit invalidates any stored auto-offset base (the
@@ -509,10 +524,23 @@ const DuctPointHandles = ({ duct, target }) => {
                 useScene.getState().updateNodes(finalWithTagStrip);
             }
         };
+        const onUp = () => {
+            if (!dragRef.current)
+                return;
+            // Swallow the trailing synthetic click so it doesn't reach the
+            // background-click deselect handler — `cleanup()` drops `inputDragging`
+            // synchronously here, so without this the click that fires after
+            // pointerup would land with the drag gate already down and clear the
+            // selection. Mirrors `useHandleDrag`'s onUp.
+            swallowNextClick();
+            finish(true);
+        };
+        const onCancel = () => finish(false);
         const cleanup = () => {
             window.removeEventListener('pointermove', onMove);
             window.removeEventListener('pointerup', onUp);
-            window.removeEventListener('pointercancel', onUp);
+            window.removeEventListener('pointercancel', onCancel);
+            releaseScope();
             useViewer.getState().setInputDragging(false);
             document.body.style.cursor = '';
         };
@@ -530,7 +558,7 @@ const DuctPointHandles = ({ duct, target }) => {
         };
         window.addEventListener('pointermove', onMove);
         window.addEventListener('pointerup', onUp);
-        window.addEventListener('pointercancel', onUp);
+        window.addEventListener('pointercancel', onCancel);
     };
     // Roll: spin the rect / oval cross-section about the run (length) axis. The
     // cursor's bearing in the plane perpendicular to the run direction (taken in
@@ -572,6 +600,10 @@ const DuctPointHandles = ({ duct, target }) => {
         useViewer.getState().setInputDragging(true);
         document.body.style.cursor = 'grabbing';
         setRolling(true);
+        const releaseScope = claimHandleDrag(duct.id, ROTATE_HANDLE_DRAG_LABEL, () => {
+            swallowNextClick();
+            finish(false);
+        });
         let current = startRoll;
         let lastStep = Number.NaN;
         const onMove = (event) => {
@@ -580,15 +612,16 @@ const DuctPointHandles = ({ duct, target }) => {
             const b = bearing(event.clientX, event.clientY);
             if (b === null)
                 return;
-            // Snap the roll to 45° steps only in the active angle mode.
+            // Snap the roll to 45° steps only in the active angle mode; held Alt rolls freely.
             const raw = b - startBearing;
-            const delta = isAngleSnapActive() ? Math.round(raw / ROLL_STEP_RAD) * ROLL_STEP_RAD : raw;
+            const snapped = !event.altKey && isAngleSnapActive();
+            const delta = snapped ? Math.round(raw / ROLL_STEP_RAD) * ROLL_STEP_RAD : raw;
             const next = startRoll + delta;
             if (next === current)
                 return;
             current = next;
             // Tick the rotate SFX each time a fresh snapped step is crossed.
-            if (isAngleSnapActive()) {
+            if (snapped) {
                 const step = Math.round(raw / ROLL_STEP_RAD);
                 if (step !== lastStep) {
                     lastStep = step;
@@ -598,11 +631,11 @@ const DuctPointHandles = ({ duct, target }) => {
             useLiveNodeOverrides.getState().set(duct.id, { roll: next });
             useScene.getState().markDirty(duct.id);
         };
-        const onUp = () => {
-            swallowNextClick();
+        const finish = (commit) => {
             window.removeEventListener('pointermove', onMove);
             window.removeEventListener('pointerup', onUp);
-            window.removeEventListener('pointercancel', onUp);
+            window.removeEventListener('pointercancel', onCancel);
+            releaseScope();
             useViewer.getState().setInputDragging(false);
             clearPlacementSurface();
             document.body.style.cursor = '';
@@ -610,16 +643,21 @@ const DuctPointHandles = ({ duct, target }) => {
             useLiveNodeOverrides.getState().clear(duct.id);
             useScene.getState().markDirty(duct.id);
             resumeSceneHistory(useScene);
-            if (current !== startRoll) {
+            if (commit && current !== startRoll) {
                 useScene.getState().updateNode(duct.id, {
                     roll: current,
                     ...(readAutoOffsetTag(duct) ? { metadata: withoutAutoOffsetTag(duct.metadata) } : {}),
                 });
             }
         };
+        const onUp = () => {
+            swallowNextClick();
+            finish(true);
+        };
+        const onCancel = () => finish(false);
         window.addEventListener('pointermove', onMove);
         window.addEventListener('pointerup', onUp);
-        window.addEventListener('pointercancel', onUp);
+        window.addEventListener('pointercancel', onCancel);
     };
     // Move the WHOLE run, locked to one direction: the vertical world axis, or a
     // run-relative horizontal line (node-local XZ unit dir) so the along-/across-
@@ -755,10 +793,14 @@ const DuctPointHandles = ({ duct, target }) => {
         useViewer.getState().setInputDragging(true);
         document.body.style.cursor = kind.axis === 'y' ? 'ns-resize' : 'grabbing';
         setRunMoving(true);
+        const releaseScope = claimHandleDrag(duct.id, AXIS_MOVE_DRAG_LABEL, () => {
+            swallowNextClick();
+            finish(false);
+        });
         let delta = 0;
         // Last frame's offset outcome (vertical move only): a valid plan to mint on
         // release, an invalid marker (preview-only, snap back), or null for a plain
-        // translate. `onUp` reads it to decide create vs. translate vs. nothing.
+        // translate. `finish` reads it to decide create vs. translate vs. nothing.
         let offsetResult = null;
         // Shift the LOGICAL L: vertical adds the carried `baseDy` plus this drag's
         // delta (so a re-drag accumulates from the L, not the lifted Z); horizontal
@@ -774,10 +816,10 @@ const DuctPointHandles = ({ duct, target }) => {
             }
             return next;
         });
-        const batchFor = (path) => [
-            { id: duct.id, data: { path } },
-            ...connectivityUpdatesForPath(connectivity, path),
-        ];
+        const batchFor = (path) => {
+            const followers = connectivityUpdatesForPath(connectivity, path);
+            return followers && [{ id: duct.id, data: { path } }, ...followers];
+        };
         // Patches restoring every connected partner to its post-rewind (logical-L)
         // pose — runs by path, fittings by position (+ a re-aimed elbow's rotation /
         // angle). Used to FREEZE partners on an invalid frame.
@@ -878,30 +920,37 @@ const DuctPointHandles = ({ duct, target }) => {
             const s = sample(event.clientX, event.clientY);
             if (s === null)
                 return;
-            const step = isGridSnapActive() ? useEditor.getState().gridSnapStep : 0;
+            const step = !event.altKey && isGridSnapActive() ? useEditor.getState().gridSnapStep : 0;
             const next = snap(s - startSample, step);
             if (next === delta)
                 return;
-            delta = next;
-            if (step > 0)
-                triggerSFX('sfx:grid-snap');
             const dyEff = baseDy + next;
             // Vertical move: connected ends stay welded to their (stationary)
             // partner via an auto-routed elbow → riser → elbow offset, planned from
             // the LOGICAL L by the accumulated lift. The planner returns valid (offset
             // fits), invalid (connected but no clean offset at this height), or null
             // (no connections, or dyEff 0 → back at the L → plain translate).
-            offsetResult =
-                isVertical && dyEff !== 0
-                    ? planVerticalOffsets({
-                        duct: logicalDuct,
-                        dy: dyEff,
-                        profile,
-                        connections: connectivity?.connections ?? [],
-                        scenePorts,
-                        nodesById,
-                    })
-                    : null;
+            const result = isVertical && dyEff !== 0
+                ? planVerticalOffsets({
+                    duct: logicalDuct,
+                    dy: dyEff,
+                    profile,
+                    connections: connectivity?.connections ?? [],
+                    scenePorts,
+                    nodesById,
+                })
+                : null;
+            const followUpdates = result?.status === 'invalid'
+                ? []
+                : connectivityUpdatesForPath(connectivity, result?.status === 'valid' ? result.plan.followPath : shiftedPath(next));
+            // Joints that cannot follow hold the last shown frame, which is what
+            // release commits.
+            if (!followUpdates)
+                return;
+            delta = next;
+            offsetResult = result;
+            if (step > 0)
+                triggerSFX('sfx:grid-snap');
             if (offsetResult?.status === 'valid') {
                 applyLogicalBasePreview();
                 // Lift the run to the offset path; trim run-offset partners back one
@@ -913,7 +962,6 @@ const DuctPointHandles = ({ duct, target }) => {
                 const scene = useScene.getState();
                 const deletePreview = (plan.delete ?? []).filter((id) => scene.nodes[id]);
                 setPreviewHidden([...mintedIds, ...deletePreview]);
-                const followUpdates = connectivityUpdatesForPath(connectivity, plan.followPath);
                 const updates = [
                     { id: duct.id, data: { path: plan.ductPath } },
                     ...plan.updates.filter((u) => scene.nodes[u.id]),
@@ -941,15 +989,17 @@ const DuctPointHandles = ({ duct, target }) => {
             else {
                 applyLogicalBasePreview();
                 setVerticalGhost(null);
-                const updates = batchFor(shiftedPath(next));
-                publishLivePreview(updates);
+                publishLivePreview([
+                    { id: duct.id, data: { path: shiftedPath(next) } },
+                    ...followUpdates,
+                ]);
             }
         };
-        const onUp = () => {
-            swallowNextClick();
+        const finish = (commit) => {
             window.removeEventListener('pointermove', onMove);
             window.removeEventListener('pointerup', onUp);
-            window.removeEventListener('pointercancel', onUp);
+            window.removeEventListener('pointercancel', onCancel);
+            releaseScope();
             useViewer.getState().setInputDragging(false);
             document.body.style.cursor = '';
             setRunMoving(false);
@@ -957,7 +1007,7 @@ const DuctPointHandles = ({ duct, target }) => {
             clearLivePreview();
             setPreviewHidden([]);
             resumeSceneHistory(useScene);
-            if (delta === 0)
+            if (!commit || delta === 0)
                 return;
             const dyEff = baseDy + delta;
             const result = offsetResult;
@@ -983,6 +1033,8 @@ const DuctPointHandles = ({ duct, target }) => {
                     base,
                 };
                 const followUpdates = connectivityUpdatesForPath(connectivity, plan.followPath);
+                if (!followUpdates)
+                    return;
                 const updates = [
                     {
                         id: duct.id,
@@ -1068,6 +1120,8 @@ const DuctPointHandles = ({ duct, target }) => {
                     return;
                 }
                 const updates = batchFor(finalPath);
+                if (!updates)
+                    return;
                 if (isVertical && existingTag) {
                     updates[0] = {
                         ...updates[0],
@@ -1095,9 +1149,14 @@ const DuctPointHandles = ({ duct, target }) => {
                 ensureSceneObjectsVisible(updates.map((update) => update.id));
             }
         };
+        const onUp = () => {
+            swallowNextClick();
+            finish(true);
+        };
+        const onCancel = () => finish(false);
         window.addEventListener('pointermove', onMove);
         window.addEventListener('pointerup', onUp);
-        window.addEventListener('pointercancel', onUp);
+        window.addEventListener('pointercancel', onCancel);
     };
     const cornerArrows = useMemo(() => getCornerArrows(displayDuct), [displayDuct]);
     const rollGizmo = useMemo(() => (displayDuct.shape === 'round' ? null : runAxisAndCenter(displayDuct)), [displayDuct]);

package/dist/duct-segment/tool.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"tool.d.ts","sourceRoot":"","sources":["../../src/duct-segment/tool.tsx"],"names":[],"mappings":"AAEA,OAAO,EAAE,KAAK,OAAO,EAAE,KAAK,eAAe,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAA;AAQtF,OAAO,EAOL,KAAK,UAAU,EACf,KAAK,gBAAgB,EACrB,KAAK,SAAS,EAGf,MAAM,iCAAiC,CAAA;AA+JxC,qEAAqE;AACrE,KAAK,YAAY,GAAG;IAClB,KAAK,EAAE,OAAO,GAAG,MAAM,GAAG,MAAM,CAAA;IAChC,QAAQ,EAAE,MAAM,CAAA;IAChB,KAAK,EAAE,MAAM,CAAA;IACb,MAAM,EAAE,MAAM,CAAA;CACf,CAAA;AAED,wBAAgB,qBAAqB,CAAC,OAAO,EAAE,YAAY,EAAE,IAAI,UAAQ,GAAG,MAAM,CAUjF;AAgDD;;;;;6BAK6B;AAC7B,KAAK,YAAY,GAAG;IAClB,iBAAiB,EAAE,MAAM,GAAG,IAAI,CAAA;IAChC,QAAQ,EAAE,eAAe,EAAE,CAAA;IAC3B,KAAK,EAAE,eAAe,EAAE,CAAA;IACxB,KAAK,EAAE,eAAe,EAAE,CAAA;IACxB,OAAO,EAAE;QAAE,EAAE,EAAE,OAAO,CAAC,IAAI,CAAC,CAAC;QAAC,IAAI,EAAE,OAAO,CAAC,OAAO,CAAC,CAAA;KAAE,EAAE,CAAA;IACxD,MAAM,EAAE,OAAO,CAAC,IAAI,CAAC,EAAE,CAAA;CACxB,CAAA;AA4CD;;;;;;;;GAQG;AACH,wBAAgB,YAAY,CAC1B,KAAK,EAAE,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,EAC/B,GAAG,EAAE,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,EAC7B,SAAS,EAAE,SAAS,GAAG,IAAI,EAC3B,SAAS,EAAE,UAAU,GAAG,IAAI,EAC5B,OAAO,EAAE,SAAS,GAAG,IAAI,EACzB,OAAO,EAAE,UAAU,GAAG,IAAI,EAC1B,OAAO,EAAE,YAAY,EACrB,KAAK,EAAE,QAAQ,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,EACxC,OAAO,CAAC,EAAE,gBAAgB,GAAG,IAAI,EACjC,WAAW,UAAQ,EACnB,YAAY,GAAE,OAAO,CAAC,eAAe,CAAM,EAC3C,WAAW,GAAE,QAAQ,GAAG,QAAmB,GAC1C,YAAY,GAAG,IAAI,CAwLrB;AAED,QAAA,MAAM,eAAe,0CAuRpB,CAAA;eAEc,eAAe"}
+{"version":3,"file":"tool.d.ts","sourceRoot":"","sources":["../../src/duct-segment/tool.tsx"],"names":[],"mappings":"AAEA,OAAO,EAAE,KAAK,OAAO,EAAE,KAAK,eAAe,EAAE,eAAe,EAAgB,MAAM,kBAAkB,CAAA;AAQpG,OAAO,EAOL,KAAK,UAAU,EACf,KAAK,gBAAgB,EACrB,KAAK,SAAS,EAGf,MAAM,iCAAiC,CAAA;AAgKxC,qEAAqE;AACrE,KAAK,YAAY,GAAG;IAClB,KAAK,EAAE,OAAO,GAAG,MAAM,GAAG,MAAM,CAAA;IAChC,QAAQ,EAAE,MAAM,CAAA;IAChB,KAAK,EAAE,MAAM,CAAA;IACb,MAAM,EAAE,MAAM,CAAA;CACf,CAAA;AAED,wBAAgB,qBAAqB,CAAC,OAAO,EAAE,YAAY,EAAE,IAAI,UAAQ,GAAG,MAAM,CAUjF;AAgDD;;;;;6BAK6B;AAC7B,KAAK,YAAY,GAAG;IAClB,iBAAiB,EAAE,MAAM,GAAG,IAAI,CAAA;IAChC,QAAQ,EAAE,eAAe,EAAE,CAAA;IAC3B,KAAK,EAAE,eAAe,EAAE,CAAA;IACxB,KAAK,EAAE,eAAe,EAAE,CAAA;IACxB,OAAO,EAAE;QAAE,EAAE,EAAE,OAAO,CAAC,IAAI,CAAC,CAAC;QAAC,IAAI,EAAE,OAAO,CAAC,OAAO,CAAC,CAAA;KAAE,EAAE,CAAA;IACxD,MAAM,EAAE,OAAO,CAAC,IAAI,CAAC,EAAE,CAAA;CACxB,CAAA;AA4CD;;;;;;;;GAQG;AACH,wBAAgB,YAAY,CAC1B,KAAK,EAAE,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,EAC/B,GAAG,EAAE,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,EAC7B,SAAS,EAAE,SAAS,GAAG,IAAI,EAC3B,SAAS,EAAE,UAAU,GAAG,IAAI,EAC5B,OAAO,EAAE,SAAS,GAAG,IAAI,EACzB,OAAO,EAAE,UAAU,GAAG,IAAI,EAC1B,OAAO,EAAE,YAAY,EACrB,KAAK,EAAE,QAAQ,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,EACxC,OAAO,CAAC,EAAE,gBAAgB,GAAG,IAAI,EACjC,WAAW,UAAQ,EACnB,YAAY,GAAE,OAAO,CAAC,eAAe,CAAM,EAC3C,WAAW,GAAE,QAAQ,GAAG,QAAmB,GAC1C,YAAY,GAAG,IAAI,CAsNrB;AAED,QAAA,MAAM,eAAe,0CA6VpB,CAAA;eAEc,eAAe"}