@pascal-app/nodes 1.0.0-beta.1 → 1.0.0-beta.3

package/dist/fence/definition.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"definition.d.ts","sourceRoot":"","sources":["../../src/fence/definition.ts"],"names":[],"mappings":"AAAA,OAAO,EAML,KAAK,cAAc,EAEpB,MAAM,kBAAkB,CAAA;AAkBzB,OAAO,EAAE,SAAS,EAAE,MAAM,UAAU,CAAA;AAyOpC;;;;;;;;;;;;GAYG;AACH,eAAO,MAAM,eAAe,EAAE,cAAc,CAAC,OAAO,SAAS,CAyH5D,CAAA"}
+{"version":3,"file":"definition.d.ts","sourceRoot":"","sources":["../../src/fence/definition.ts"],"names":[],"mappings":"AAAA,OAAO,EAKL,KAAK,cAAc,EAEpB,MAAM,kBAAkB,CAAA;AAkBzB,OAAO,EAAE,SAAS,EAAE,MAAM,UAAU,CAAA;AAgPpC;;;;;;;;;;;;GAYG;AACH,eAAO,MAAM,eAAe,EAAE,cAAc,CAAC,OAAO,SAAS,CAyH5D,CAAA"}

package/dist/fence/definition.js

@@ -4,7 +4,7 @@ import { buildFenceFloorplan } from './floorplan';
 import { fenceControlPointAffordance, fenceCurveAffordance, fenceMoveEndpointAffordance, fenceTangentAffordance, } from './floorplan-affordances';
 import { fenceFloorplanMoveTarget } from './floorplan-move';
 import { buildFenceGeometry } from './geometry';
-import { resolveFenceLiftElevation } from './lift';
+import { resolveFenceLiftElevation, resolveFenceLiftElevationForNodes } from './lift';
 import { fencePaint } from './paint';
 import { fenceParametrics } from './parametrics';
 import { FenceNode } from './schema';
@@ -16,7 +16,10 @@ const SIDE_HANDLE_MIN_HEIGHT = 0.4;
 const HEIGHT_HANDLE_OFFSET = 0.45;
 const MIN_FENCE_HEIGHT = 0.3;
 function fenceBaseElevation(n, sceneApi) {
-    return resolveFenceLiftElevation(n, (id) => sceneApi?.get(id));
+    const nodes = sceneApi?.nodes();
+    return nodes
+        ? resolveFenceLiftElevationForNodes(n, nodes)
+        : resolveFenceLiftElevation(n, () => undefined);
 }
 function fenceMidpointFrame(n) {
     const dx = n.end[0] - n.start[0];
@@ -110,7 +113,11 @@ function fenceElevationHandle(currentBase) {
         onDrag: (n, sceneApi) => publishStructuralElevationGuide(fenceElevationGuideSource(n), fenceBaseElevation(n, sceneApi), sceneApi.nodes()),
         onDragEnd: (n) => clearStructuralElevationGuide(n.id),
         apply: (initial, newBase, sceneApi) => {
-            const supportBase = resolveFenceLiftElevation({ ...initial, supportOffset: undefined }, (id) => sceneApi.get(id));
+            // The offset is measured from the support WITHOUT the current offset —
+            // including the ground, so dragging a fence's base to an absolute height
+            // on a hillside stores the delta from that hillside and the fence keeps
+            // following it.
+            const supportBase = resolveFenceLiftElevationForNodes({ ...initial, supportOffset: undefined }, sceneApi.nodes());
             const nextOffset = newBase - supportBase;
             return {
                 supportOffset: Math.abs(nextOffset) < 1e-6 ? undefined : nextOffset,

package/dist/fence/geometry.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"geometry.d.ts","sourceRoot":"","sources":["../../src/fence/geometry.ts"],"names":[],"mappings":"AAAA,OAAO,EAAkB,KAAK,eAAe,EAA0B,MAAM,kBAAkB,CAAA;AAC/F,OAAO,EAEL,KAAK,WAAW,EAKhB,KAAK,aAAa,EAGnB,MAAM,oBAAoB,CAAA;AAC3B,OAAO,EAAa,KAAK,EAAqC,MAAM,OAAO,CAAA;AAE3E,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,UAAU,CAAA;AAwFzC,wBAAgB,kBAAkB,CAChC,IAAI,EAAE,SAAS,EACf,GAAG,CAAC,EAAE,eAAe,EACrB,OAAO,GAAE,aAA0B,EACnC,QAAQ,UAAO,EACf,WAAW,GAAE,WAAoB,EACjC,UAAU,CAAC,EAAE,MAAM,GAClB,KAAK,CAgCP"}
+{"version":3,"file":"geometry.d.ts","sourceRoot":"","sources":["../../src/fence/geometry.ts"],"names":[],"mappings":"AAAA,OAAO,EAAkB,KAAK,eAAe,EAA0B,MAAM,kBAAkB,CAAA;AAC/F,OAAO,EAEL,KAAK,WAAW,EAKhB,KAAK,aAAa,EAGnB,MAAM,oBAAoB,CAAA;AAC3B,OAAO,EAAa,KAAK,EAAqC,MAAM,OAAO,CAAA;AAE3E,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,UAAU,CAAA;AAwFzC,wBAAgB,kBAAkB,CAChC,IAAI,EAAE,SAAS,EACf,GAAG,CAAC,EAAE,eAAe,EACrB,OAAO,GAAE,aAA0B,EACnC,QAAQ,UAAO,EACf,WAAW,GAAE,WAAoB,EACjC,UAAU,CAAC,EAAE,MAAM,GAClB,KAAK,CA0CP"}

package/dist/fence/geometry.js

@@ -51,10 +51,16 @@ function getLegacyFenceMaterial(node, shading) {
 export function buildFenceGeometry(node, ctx, shading = 'rendered', textures = true, colorPreset = 'clay', sceneTheme) {
     const group = new Group();
     const geometries = generateFenceSlotGeometries(node);
-    // A hosted railing (`supportSlabId`) stands on its slab's walking surface.
-    // The builder emits local-space children, so the lift lives on an inner
-    // group rather than the registered (React-transformed) root.
-    const lift = ctx ? resolveFenceLiftElevation(node, (id) => ctx.resolve(id)) : 0;
+    // A hosted railing (`supportSlabId`) stands on its slab's walking surface;
+    // an unhosted one stands on the ground, which `ctx.levelBaseAt` resolves at
+    // the fence's own start point — the anchor its plan geometry is measured
+    // from, so the resolver and the mesh cannot disagree about where the ground
+    // is under this fence. The builder emits local-space children, so the lift
+    // lives on an inner group rather than the registered (React-transformed)
+    // root.
+    const lift = ctx
+        ? resolveFenceLiftElevation(node, (id) => ctx.resolve(id), ctx.levelBaseAt?.(node.start[0], node.start[1]) ?? 0)
+        : 0;
     const meshParent = new Group();
     meshParent.position.y = lift;
     group.add(meshParent);

package/dist/fence/lift.d.ts

@@ -1,4 +1,4 @@
-import type { AnyNode } from '@pascal-app/core';
+import { type AnyNode } from '@pascal-app/core';
 import type { FenceNode } from './schema';
 /**
  * Elevation (meters above the level plane) a hosted fence stands at.
@@ -10,6 +10,29 @@ import type { FenceNode } from './schema';
  * falls back to the level floor, mirroring the read-path rules of the
  * other `supportSlabId` carriers. Pure so it is unit-testable and
  * callable from the geometry builder with `ctx.resolve`.
+ *
+ * `levelBase` is what "the level floor" means here — the sculpted ground
+ * under the fence (`levelBaseElevationAt`, reached through `ctx.levelBaseAt`
+ * in the builder and {@link resolveFenceLiftElevationForNodes} everywhere
+ * else), or 0 for a level with no terrain under it. A caller-resolved scalar
+ * rather than a terrain lookup inside, so this stays pure and the sample stays
+ * at the caller's chosen point; it substitutes for the literal `0` this used to
+ * return in every unhosted branch, and nowhere else — a host slab that still
+ * exists wins outright, so a railing on a deck pad keeps the pad's elevation
+ * instead of following the hillside around it.
+ */
+export declare function resolveFenceLiftElevation(node: Pick<FenceNode, 'supportSlabId' | 'supportOffset' | 'parentId'>, resolve: (id: string) => AnyNode | undefined, levelBase?: number): number;
+/**
+ * {@link resolveFenceLiftElevation} against a nodes record, with the level base
+ * resolved from the terrain under the fence's start point — the same anchor the
+ * geometry builder samples, so a handle, a snap guide and the rendered rail all
+ * agree about where the ground is.
+ *
+ * Exists so the callers that already hold the whole scene (handles via
+ * `sceneApi.nodes()`, the dependency tracker, the editor's elevation guides)
+ * don't each re-derive the level walk and the sample point. The builder keeps
+ * using the pure form, since `GeometryContext` deliberately exposes the ground
+ * as a closure rather than the store.
  */
-export declare function resolveFenceLiftElevation(node: Pick<FenceNode, 'supportSlabId' | 'supportOffset' | 'parentId'>, resolve: (id: string) => AnyNode | undefined): number;
+export declare function resolveFenceLiftElevationForNodes(node: Pick<FenceNode, 'id' | 'start' | 'supportSlabId' | 'supportOffset' | 'parentId'>, nodes: Record<string, AnyNode>): number;
 //# sourceMappingURL=lift.d.ts.map

package/dist/fence/lift.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"lift.d.ts","sourceRoot":"","sources":["../../src/fence/lift.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,OAAO,EAAY,MAAM,kBAAkB,CAAA;AACzD,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,UAAU,CAAA;AAEzC;;;;;;;;;;GAUG;AACH,wBAAgB,yBAAyB,CACvC,IAAI,EAAE,IAAI,CAAC,SAAS,EAAE,eAAe,GAAG,eAAe,GAAG,UAAU,CAAC,EACrE,OAAO,EAAE,CAAC,EAAE,EAAE,MAAM,KAAK,OAAO,GAAG,SAAS,GAC3C,MAAM,CAQR"}
+{"version":3,"file":"lift.d.ts","sourceRoot":"","sources":["../../src/fence/lift.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,KAAK,OAAO,EAKb,MAAM,kBAAkB,CAAA;AACzB,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,UAAU,CAAA;AAEzC;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,wBAAgB,yBAAyB,CACvC,IAAI,EAAE,IAAI,CAAC,SAAS,EAAE,eAAe,GAAG,eAAe,GAAG,UAAU,CAAC,EACrE,OAAO,EAAE,CAAC,EAAE,EAAE,MAAM,KAAK,OAAO,GAAG,SAAS,EAC5C,SAAS,SAAI,GACZ,MAAM,CAQR;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,iCAAiC,CAC/C,IAAI,EAAE,IAAI,CAAC,SAAS,EAAE,IAAI,GAAG,OAAO,GAAG,eAAe,GAAG,eAAe,GAAG,UAAU,CAAC,EACtF,KAAK,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAC7B,MAAM,CAIR"}

package/dist/fence/lift.js

@@ -1,3 +1,4 @@
+import { findLevelAncestorId, levelBaseElevationAt, } from '@pascal-app/core';
 /**
  * Elevation (meters above the level plane) a hosted fence stands at.
  *
@@ -8,16 +9,43 @@
  * falls back to the level floor, mirroring the read-path rules of the
  * other `supportSlabId` carriers. Pure so it is unit-testable and
  * callable from the geometry builder with `ctx.resolve`.
+ *
+ * `levelBase` is what "the level floor" means here — the sculpted ground
+ * under the fence (`levelBaseElevationAt`, reached through `ctx.levelBaseAt`
+ * in the builder and {@link resolveFenceLiftElevationForNodes} everywhere
+ * else), or 0 for a level with no terrain under it. A caller-resolved scalar
+ * rather than a terrain lookup inside, so this stays pure and the sample stays
+ * at the caller's chosen point; it substitutes for the literal `0` this used to
+ * return in every unhosted branch, and nowhere else — a host slab that still
+ * exists wins outright, so a railing on a deck pad keeps the pad's elevation
+ * instead of following the hillside around it.
  */
-export function resolveFenceLiftElevation(node, resolve) {
+export function resolveFenceLiftElevation(node, resolve, levelBase = 0) {
     const offset = node.supportOffset ?? 0;
     if (!node.supportSlabId)
-        return offset;
+        return levelBase + offset;
     const host = resolve(node.supportSlabId);
     if (host?.type !== 'slab')
-        return offset;
+        return levelBase + offset;
     if ((host.parentId ?? null) !== (node.parentId ?? null))
-        return offset;
+        return levelBase + offset;
     const elevation = host.elevation;
     return (Number.isFinite(elevation) ? elevation : 0) + offset;
 }
+/**
+ * {@link resolveFenceLiftElevation} against a nodes record, with the level base
+ * resolved from the terrain under the fence's start point — the same anchor the
+ * geometry builder samples, so a handle, a snap guide and the rendered rail all
+ * agree about where the ground is.
+ *
+ * Exists so the callers that already hold the whole scene (handles via
+ * `sceneApi.nodes()`, the dependency tracker, the editor's elevation guides)
+ * don't each re-derive the level walk and the sample point. The builder keeps
+ * using the pure form, since `GeometryContext` deliberately exposes the ground
+ * as a closure rather than the store.
+ */
+export function resolveFenceLiftElevationForNodes(node, nodes) {
+    const levelId = findLevelAncestorId(node.id, nodes);
+    const levelBase = levelId ? levelBaseElevationAt(nodes, levelId, node.start[0], node.start[1]) : 0;
+    return resolveFenceLiftElevation(node, (id) => nodes[id], levelBase);
+}

package/dist/fence/system.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"system.d.ts","sourceRoot":"","sources":["../../src/fence/system.tsx"],"names":[],"mappings":"AA+BA,QAAA,MAAM,YAAY,YAgBjB,CAAA;AAED,eAAe,YAAY,CAAA"}
+{"version":3,"file":"system.d.ts","sourceRoot":"","sources":["../../src/fence/system.tsx"],"names":[],"mappings":"AAiCA,QAAA,MAAM,YAAY,YAgBjB,CAAA;AAED,eAAe,YAAY,CAAA"}

package/dist/fence/system.js

@@ -1,7 +1,7 @@
 'use client';
 import { useScene } from '@pascal-app/core';
 import { useEffect } from 'react';
-import { resolveFenceLiftElevation } from './lift';
+import { resolveFenceLiftElevationForNodes } from './lift';
 /**
  * Hosted-railing dependency tracker. A fence with `supportSlabId` renders at
  * its host slab's elevation, but a store update only dirties the node that
@@ -17,9 +17,14 @@ function fenceLiftSignatures(nodes) {
         if (node.type !== 'fence')
             continue;
         const fence = node;
+        // Unhosted fences are covered too, and by a different mechanism: their lift
+        // is the ground, which only a sculpt moves, and `markTerrainSupportDependents`
+        // dirties them from the terrain change itself. Tracking them here would
+        // resolve every fence in the scene on every store update to catch an event
+        // this subscription never sees.
         if (!fence.supportSlabId)
             continue;
-        signatures.set(fence.id, resolveFenceLiftElevation(fence, (id) => nodes[id]));
+        signatures.set(fence.id, resolveFenceLiftElevationForNodes(fence, nodes));
     }
     return signatures;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pascal-app/nodes",
-  "version": "1.0.0-beta.1",
+  "version": "1.0.0-beta.3",
   "description": "Built-in node bundles for the Pascal 3D editor — one folder per kind, exported as `builtinPlugin`",
   "type": "module",
   "main": "./dist/index.js",
@@ -23,9 +23,9 @@
     "prepublishOnly": "bun run build && bun test"
   },
   "peerDependencies": {
-    "@pascal-app/core": "^1.0.0-beta.1",
-    "@pascal-app/editor": "^1.0.0-beta.1",
-    "@pascal-app/viewer": "^1.0.0-beta.1",
+    "@pascal-app/core": "^1.0.0-beta.3",
+    "@pascal-app/editor": "^1.0.0-beta.3",
+    "@pascal-app/viewer": "^1.0.0-beta.3",
     "@react-three/drei": "^10",
     "@react-three/fiber": "^9",
     "lucide-react": "^1",
@@ -34,9 +34,9 @@
     "zustand": "^5"
   },
   "devDependencies": {
-    "@pascal-app/core": "^1.0.0-beta.1",
-    "@pascal-app/editor": "^1.0.0-beta.1",
-    "@pascal-app/viewer": "^1.0.0-beta.1",
+    "@pascal-app/core": "^1.0.0-beta.3",
+    "@pascal-app/editor": "^1.0.0-beta.3",
+    "@pascal-app/viewer": "^1.0.0-beta.3",
     "@pascal/typescript-config": "*",
     "@types/bun": "^1.3.0",
     "@types/node": "^22.19.12",