@pascal-app/core 0.8.0 → 0.9.1

package/dist/store/use-scene-elevator-migration.test.js

@@ -0,0 +1,109 @@
+import { beforeEach, describe, expect, test } from 'bun:test';
+import useScene from './use-scene';
+describe('scene elevator migrations', () => {
+    beforeEach(() => {
+        useScene.setState({
+            nodes: {},
+            rootNodeIds: [],
+            dirtyNodes: new Set(),
+            collections: {},
+        });
+        useScene.temporal.getState().clear();
+    });
+    test('normalizes legacy level-parented elevators into building-scoped nodes', () => {
+        useScene.getState().setScene({
+            site_test: {
+                object: 'node',
+                id: 'site_test',
+                type: 'site',
+                parentId: null,
+                visible: true,
+                metadata: {},
+                children: ['building_test'],
+            },
+            building_test: {
+                object: 'node',
+                id: 'building_test',
+                type: 'building',
+                parentId: 'site_test',
+                visible: true,
+                metadata: {},
+                children: ['level_test'],
+            },
+            level_test: {
+                object: 'node',
+                id: 'level_test',
+                type: 'level',
+                parentId: 'building_test',
+                visible: true,
+                metadata: {},
+                children: ['elevator_test'],
+                level: 0,
+            },
+            elevator_test: {
+                object: 'node',
+                id: 'elevator_test',
+                type: 'elevator',
+                parentId: 'level_test',
+                visible: true,
+                metadata: {},
+            },
+        }, ['site_test']);
+        const nodes = useScene.getState().nodes;
+        const elevator = nodes.elevator_test;
+        const level = nodes.level_test;
+        const building = nodes.building_test;
+        expect(elevator.parentId).toBe('building_test');
+        expect(elevator.position).toEqual([0, 0, 0]);
+        expect(elevator.rotation).toBe(0);
+        expect(level.children).not.toContain('elevator_test');
+        expect(building.children).toContain('elevator_test');
+    });
+    test('migrates level-parented elevators when the level parentId is missing', () => {
+        useScene.getState().setScene({
+            site_test: {
+                object: 'node',
+                id: 'site_test',
+                type: 'site',
+                parentId: null,
+                visible: true,
+                metadata: {},
+                children: ['building_test'],
+            },
+            building_test: {
+                object: 'node',
+                id: 'building_test',
+                type: 'building',
+                parentId: 'site_test',
+                visible: true,
+                metadata: {},
+                children: ['level_test'],
+            },
+            level_test: {
+                object: 'node',
+                id: 'level_test',
+                type: 'level',
+                parentId: null,
+                visible: true,
+                metadata: {},
+                children: ['elevator_test'],
+                level: 0,
+            },
+            elevator_test: {
+                object: 'node',
+                id: 'elevator_test',
+                type: 'elevator',
+                parentId: 'level_test',
+                visible: true,
+                metadata: {},
+            },
+        }, ['site_test']);
+        const nodes = useScene.getState().nodes;
+        const elevator = nodes.elevator_test;
+        const level = nodes.level_test;
+        const building = nodes.building_test;
+        expect(elevator.parentId).toBe('building_test');
+        expect(level.children).not.toContain('elevator_test');
+        expect(building.children).toContain('elevator_test');
+    });
+});

package/dist/store/use-scene.d.ts

@@ -20,6 +20,17 @@ export type SceneState = {
         node: AnyNode;
         parentId?: AnyNodeId;
     }[]) => void;
+    applyNodeChanges: (changes: {
+        create?: {
+            node: AnyNode;
+            parentId?: AnyNodeId;
+        }[];
+        update?: {
+            id: AnyNodeId;
+            data: Partial<AnyNode>;
+        }[];
+        delete?: AnyNodeId[];
+    }) => void;
     updateNode: (id: AnyNodeId, data: Partial<AnyNode>) => void;
     updateNodes: (updates: {
         id: AnyNodeId;

package/dist/store/use-scene.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"use-scene.d.ts","sourceRoot":"","sources":["../../src/store/use-scene.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,OAAO,CAAA;AAE1C,OAAO,EAAU,KAAK,QAAQ,EAAE,KAAK,aAAa,EAAE,MAAM,SAAS,CAAA;AAEnE,OAAO,KAAK,EAAE,UAAU,EAAE,YAAY,EAAE,MAAM,uBAAuB,CAAA;AAMrE,OAAO,KAAK,EAAE,OAAO,EAAE,SAAS,EAAE,MAAM,iBAAiB,CAAA;AAgZzD,MAAM,MAAM,UAAU,GAAG;IAEvB,KAAK,EAAE,MAAM,CAAC,SAAS,EAAE,OAAO,CAAC,CAAA;IAGjC,WAAW,EAAE,SAAS,EAAE,CAAA;IAGxB,UAAU,EAAE,GAAG,CAAC,SAAS,CAAC,CAAA;IAG1B,WAAW,EAAE,MAAM,CAAC,YAAY,EAAE,UAAU,CAAC,CAAA;IAG7C,QAAQ,EAAE,OAAO,CAAA;IACjB,WAAW,EAAE,CAAC,QAAQ,EAAE,OAAO,KAAK,IAAI,CAAA;IAGxC,SAAS,EAAE,MAAM,IAAI,CAAA;IACrB,UAAU,EAAE,MAAM,IAAI,CAAA;IACtB,WAAW,EAAE,MAAM,IAAI,CAAA;IACvB,QAAQ,EAAE,CAAC,KAAK,EAAE,MAAM,CAAC,SAAS,EAAE,OAAO,CAAC,EAAE,WAAW,EAAE,SAAS,EAAE,KAAK,IAAI,CAAA;IAE/E,SAAS,EAAE,CAAC,EAAE,EAAE,SAAS,KAAK,IAAI,CAAA;IAClC,UAAU,EAAE,CAAC,EAAE,EAAE,SAAS,KAAK,IAAI,CAAA;IAEnC,UAAU,EAAE,CAAC,IAAI,EAAE,OAAO,EAAE,QAAQ,CAAC,EAAE,SAAS,KAAK,IAAI,CAAA;IACzD,WAAW,EAAE,CAAC,GAAG,EAAE;QAAE,IAAI,EAAE,OAAO,CAAC;QAAC,QAAQ,CAAC,EAAE,SAAS,CAAA;KAAE,EAAE,KAAK,IAAI,CAAA;IAErE,UAAU,EAAE,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,OAAO,CAAC,OAAO,CAAC,KAAK,IAAI,CAAA;IAC3D,WAAW,EAAE,CAAC,OAAO,EAAE;QAAE,EAAE,EAAE,SAAS,CAAC;QAAC,IAAI,EAAE,OAAO,CAAC,OAAO,CAAC,CAAA;KAAE,EAAE,KAAK,IAAI,CAAA;IAE3E,UAAU,EAAE,CAAC,EAAE,EAAE,SAAS,KAAK,IAAI,CAAA;IACnC,WAAW,EAAE,CAAC,GAAG,EAAE,SAAS,EAAE,KAAK,IAAI,CAAA;IAGvC,gBAAgB,EAAE,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,SAAS,EAAE,KAAK,YAAY,CAAA;IACvE,gBAAgB,EAAE,CAAC,EAAE,EAAE,YAAY,KAAK,IAAI,CAAA;IAC5C,gBAAgB,EAAE,CAAC,EAAE,EAAE,YAAY,EAAE,IAAI,EAAE,OAAO,CAAC,IAAI,CAAC,UAAU,EAAE,IAAI,CAAC,CAAC,KAAK,IAAI,CAAA;IACnF,eAAe,EAAE,CAAC,EAAE,EAAE,YAAY,EAAE,MAAM,EAAE,SAAS,KAAK,IAAI,CAAA;IAC9D,oBAAoB,EAAE,CAAC,EAAE,EAAE,YAAY,EAAE,MAAM,EAAE,SAAS,KAAK,IAAI,CAAA;CACpE,CAAA;AAID,KAAK,aAAa,GAAG,aAAa,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,GAAG;IACzD,QAAQ,EAAE,QAAQ,CAAC,aAAa,CAAC,IAAI,CAAC,UAAU,EAAE,OAAO,GAAG,aAAa,GAAG,aAAa,CAAC,CAAC,CAAC,CAAA;CAC7F,CAAA;AAED,QAAA,MAAM,QAAQ,EAAE,aA+Of,CAAA;AAED,eAAe,QAAQ,CAAA;AAOvB,wBAAgB,iBAAiB,SAMhC"}
+{"version":3,"file":"use-scene.d.ts","sourceRoot":"","sources":["../../src/store/use-scene.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,OAAO,CAAA;AAE1C,OAAO,EAAU,KAAK,QAAQ,EAAE,KAAK,aAAa,EAAE,MAAM,SAAS,CAAA;AAEnE,OAAO,KAAK,EAAE,UAAU,EAAE,YAAY,EAAE,MAAM,uBAAuB,CAAA;AAcrE,OAAO,KAAK,EAAE,OAAO,EAAE,SAAS,EAAE,MAAM,iBAAiB,CAAA;AAqmBzD,MAAM,MAAM,UAAU,GAAG;IAEvB,KAAK,EAAE,MAAM,CAAC,SAAS,EAAE,OAAO,CAAC,CAAA;IAGjC,WAAW,EAAE,SAAS,EAAE,CAAA;IAGxB,UAAU,EAAE,GAAG,CAAC,SAAS,CAAC,CAAA;IAG1B,WAAW,EAAE,MAAM,CAAC,YAAY,EAAE,UAAU,CAAC,CAAA;IAG7C,QAAQ,EAAE,OAAO,CAAA;IACjB,WAAW,EAAE,CAAC,QAAQ,EAAE,OAAO,KAAK,IAAI,CAAA;IAGxC,SAAS,EAAE,MAAM,IAAI,CAAA;IACrB,UAAU,EAAE,MAAM,IAAI,CAAA;IACtB,WAAW,EAAE,MAAM,IAAI,CAAA;IACvB,QAAQ,EAAE,CAAC,KAAK,EAAE,MAAM,CAAC,SAAS,EAAE,OAAO,CAAC,EAAE,WAAW,EAAE,SAAS,EAAE,KAAK,IAAI,CAAA;IAE/E,SAAS,EAAE,CAAC,EAAE,EAAE,SAAS,KAAK,IAAI,CAAA;IAClC,UAAU,EAAE,CAAC,EAAE,EAAE,SAAS,KAAK,IAAI,CAAA;IAEnC,UAAU,EAAE,CAAC,IAAI,EAAE,OAAO,EAAE,QAAQ,CAAC,EAAE,SAAS,KAAK,IAAI,CAAA;IACzD,WAAW,EAAE,CAAC,GAAG,EAAE;QAAE,IAAI,EAAE,OAAO,CAAC;QAAC,QAAQ,CAAC,EAAE,SAAS,CAAA;KAAE,EAAE,KAAK,IAAI,CAAA;IACrE,gBAAgB,EAAE,CAAC,OAAO,EAAE;QAC1B,MAAM,CAAC,EAAE;YAAE,IAAI,EAAE,OAAO,CAAC;YAAC,QAAQ,CAAC,EAAE,SAAS,CAAA;SAAE,EAAE,CAAA;QAClD,MAAM,CAAC,EAAE;YAAE,EAAE,EAAE,SAAS,CAAC;YAAC,IAAI,EAAE,OAAO,CAAC,OAAO,CAAC,CAAA;SAAE,EAAE,CAAA;QACpD,MAAM,CAAC,EAAE,SAAS,EAAE,CAAA;KACrB,KAAK,IAAI,CAAA;IAEV,UAAU,EAAE,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,OAAO,CAAC,OAAO,CAAC,KAAK,IAAI,CAAA;IAC3D,WAAW,EAAE,CAAC,OAAO,EAAE;QAAE,EAAE,EAAE,SAAS,CAAC;QAAC,IAAI,EAAE,OAAO,CAAC,OAAO,CAAC,CAAA;KAAE,EAAE,KAAK,IAAI,CAAA;IAE3E,UAAU,EAAE,CAAC,EAAE,EAAE,SAAS,KAAK,IAAI,CAAA;IACnC,WAAW,EAAE,CAAC,GAAG,EAAE,SAAS,EAAE,KAAK,IAAI,CAAA;IAGvC,gBAAgB,EAAE,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,SAAS,EAAE,KAAK,YAAY,CAAA;IACvE,gBAAgB,EAAE,CAAC,EAAE,EAAE,YAAY,KAAK,IAAI,CAAA;IAC5C,gBAAgB,EAAE,CAAC,EAAE,EAAE,YAAY,EAAE,IAAI,EAAE,OAAO,CAAC,IAAI,CAAC,UAAU,EAAE,IAAI,CAAC,CAAC,KAAK,IAAI,CAAA;IACnF,eAAe,EAAE,CAAC,EAAE,EAAE,YAAY,EAAE,MAAM,EAAE,SAAS,KAAK,IAAI,CAAA;IAC9D,oBAAoB,EAAE,CAAC,EAAE,EAAE,YAAY,EAAE,MAAM,EAAE,SAAS,KAAK,IAAI,CAAA;CACpE,CAAA;AAID,KAAK,aAAa,GAAG,aAAa,CAAC,QAAQ,CAAC,UAAU,CAAC,CAAC,GAAG;IACzD,QAAQ,EAAE,QAAQ,CAAC,aAAa,CAAC,IAAI,CAAC,UAAU,EAAE,OAAO,GAAG,aAAa,GAAG,aAAa,CAAC,CAAC,CAAC,CAAA;CAC7F,CAAA;AAED,QAAA,MAAM,QAAQ,EAAE,aAgPf,CAAA;AAED,eAAe,QAAQ,CAAA;AAOvB,wBAAgB,iBAAiB,SAMhC"}

package/dist/store/use-scene.js

@@ -3,7 +3,11 @@ import { temporal } from 'zundo';
 import { create } from 'zustand';
 import { BuildingNode } from '../schema';
 import { generateCollectionId } from '../schema/collections';
+import { DoorNode as DoorNodeSchema } from '../schema/nodes/door';
+import { ElevatorNode as ElevatorNodeSchema } from '../schema/nodes/elevator';
 import { LevelNode } from '../schema/nodes/level';
+import { getPitchFromActiveRoofHeight, } from '../schema/nodes/roof-segment';
+import { ShelfNode as ShelfNodeSchema } from '../schema/nodes/shelf';
 import { SiteNode } from '../schema/nodes/site';
 import { StairNode as StairNodeSchema } from '../schema/nodes/stair';
 import { StairSegmentNode as StairSegmentNodeSchema } from '../schema/nodes/stair-segment';
@@ -12,6 +16,10 @@ import { resetSceneHistoryPauseDepth } from './history-control';
 function getFiniteNumber(value, fallback) {
     return typeof value === 'number' && Number.isFinite(value) ? value : fallback;
 }
+function getFiniteNumberInRange(value, fallback, min, max) {
+    const finite = getFiniteNumber(value, fallback);
+    return Math.min(Math.max(finite, min), max);
+}
 function getBoolean(value, fallback) {
     return typeof value === 'boolean' ? value : fallback;
 }
@@ -19,7 +27,7 @@ function getEnumValue(value, allowed, fallback) {
     return typeof value === 'string' && allowed.includes(value) ? value : fallback;
 }
 function getNullableString(value) {
-    return typeof value === 'string' ? value : null;
+    return typeof value === 'string' && value.length > 0 ? value : null;
 }
 function getStringArray(value) {
     return Array.isArray(value)
@@ -81,6 +89,99 @@ function normalizeStairSegmentNode(node) {
     const parsed = StairSegmentNodeSchema.safeParse(sanitized);
     return parsed.success ? parsed.data : null;
 }
+function normalizeDoorNode(node) {
+    const parsed = DoorNodeSchema.safeParse(node);
+    return parsed.success ? { ...node, ...parsed.data } : null;
+}
+function normalizeShelfNode(node) {
+    const sanitized = {
+        ...node,
+        children: getStringArray(node.children),
+        position: getVector3(node.position, [0, 0, 0]),
+        rotation: getVector3(node.rotation, [0, 0, 0]),
+        width: getFiniteNumberInRange(node.width, 1.2, 0.3, 3.0),
+        depth: getFiniteNumberInRange(node.depth, 0.3, 0.1, 1.0),
+        thickness: getFiniteNumberInRange(node.thickness, 0.04, 0.01, 0.1),
+        height: getFiniteNumberInRange(node.height, 0.9, 0.05, 2.5),
+        rows: Math.round(getFiniteNumberInRange(node.rows, 1, 1, 8)),
+        columns: Math.round(getFiniteNumberInRange(node.columns, 1, 1, 6)),
+        style: getEnumValue(node.style, ['wall-shelf', 'bookshelf', 'open-rack', 'cubby'], 'wall-shelf'),
+        withBack: getBoolean(node.withBack, false),
+        withSides: getBoolean(node.withSides, true),
+        withBottom: getBoolean(node.withBottom, false),
+        bracketStyle: getEnumValue(node.bracketStyle, ['minimal', 'industrial', 'hidden'], 'minimal'),
+    };
+    const parsed = ShelfNodeSchema.safeParse(sanitized);
+    return parsed.success ? parsed.data : null;
+}
+function normalizeElevatorNode(node) {
+    const sanitized = {
+        ...node,
+        position: getVector3(node.position, [0, 0, 0]),
+        rotation: getFiniteNumber(node.rotation, 0),
+        width: getFiniteNumber(node.width, 1.84),
+        depth: getFiniteNumber(node.depth, 1.84),
+        shaftWidth: node.shaftWidth === undefined ? undefined : getFiniteNumber(node.shaftWidth, 1.84),
+        shaftDepth: node.shaftDepth === undefined ? undefined : getFiniteNumber(node.shaftDepth, 1.84),
+        shaftWallThickness: getFiniteNumber(node.shaftWallThickness, 0.09),
+        cabHeight: getFiniteNumber(node.cabHeight, 2.35),
+        doorWidth: getFiniteNumber(node.doorWidth, 0.95),
+        doorHeight: getFiniteNumber(node.doorHeight, 2.1),
+        fromLevelId: getNullableString(node.fromLevelId),
+        toLevelId: getNullableString(node.toLevelId),
+        servedLevelIds: node.servedLevelIds === undefined ? undefined : getStringArray(node.servedLevelIds),
+        disabledLevelIds: getStringArray(node.disabledLevelIds),
+        serviceOnlyLevelIds: getStringArray(node.serviceOnlyLevelIds),
+        defaultLevelId: getNullableString(node.defaultLevelId),
+        speed: getFiniteNumber(node.speed, 2.2),
+        doorDurationMs: getFiniteNumber(node.doorDurationMs, 900),
+        dwellMs: getFiniteNumber(node.dwellMs, 1400),
+    };
+    const parsed = ElevatorNodeSchema.safeParse(sanitized);
+    return parsed.success ? parsed.data : null;
+}
+function findBuildingIdForLevel(levelId, nodes) {
+    const level = nodes[levelId];
+    const directBuildingId = typeof level?.parentId === 'string' ? level.parentId : null;
+    if (directBuildingId && nodes[directBuildingId]?.type === 'building') {
+        return directBuildingId;
+    }
+    for (const [candidateId, candidate] of Object.entries(nodes)) {
+        if (candidate?.type !== 'building')
+            continue;
+        if (getStringArray(candidate.children).includes(levelId)) {
+            return candidateId;
+        }
+    }
+    return null;
+}
+function migrateElevatorParent(id, node, nodes) {
+    const parentId = typeof node.parentId === 'string' ? node.parentId : null;
+    if (!parentId)
+        return node;
+    const parent = parentId ? nodes[parentId] : null;
+    if (parent?.type !== 'level')
+        return node;
+    const buildingId = findBuildingIdForLevel(parentId, nodes);
+    if (!buildingId)
+        return node;
+    const building = buildingId ? nodes[buildingId] : null;
+    if (building?.type !== 'building')
+        return node;
+    nodes[parentId] = {
+        ...parent,
+        children: getStringArray(parent.children).filter((childId) => childId !== id),
+    };
+    const buildingChildren = getStringArray(building.children);
+    nodes[buildingId] = {
+        ...building,
+        children: buildingChildren.includes(id) ? buildingChildren : [...buildingChildren, id],
+    };
+    return {
+        ...node,
+        parentId: buildingId,
+    };
+}
 function migrateWallSurfaceMaterials(node) {
     const hasInterior = node.interiorMaterial !== undefined || typeof node.interiorMaterialPreset === 'string';
     const hasExterior = node.exteriorMaterial !== undefined || typeof node.exteriorMaterialPreset === 'string';
@@ -244,6 +345,9 @@ function migrateNodes(nodes) {
             const oldRoof = node;
             const suffix = id.includes('_') ? id.split('_')[1] : Math.random().toString(36).slice(2);
             const segmentId = `rseg_${suffix}`;
+            const segWidth = oldRoof.length ?? 8;
+            const segDepth = (oldRoof.leftWidth ?? 2.2) + (oldRoof.rightWidth ?? 2.2);
+            const legacyRoofHeight = oldRoof.height ?? 2.5;
             const segment = {
                 object: 'node',
                 id: segmentId,
@@ -254,10 +358,18 @@ function migrateNodes(nodes) {
                 position: [0, 0, 0],
                 rotation: 0,
                 roofType: 'gable',
-                width: oldRoof.length ?? 8,
-                depth: (oldRoof.leftWidth ?? 2.2) + (oldRoof.rightWidth ?? 2.2),
-                wallHeight: 0,
-                roofHeight: oldRoof.height ?? 2.5,
+                width: segWidth,
+                depth: segDepth,
+                // Schema default (0.5), NOT 0: a zero-height wall builds a flat,
+                // degenerate CSG brush → "Coplanar clip not handled" + NaN geometry, so
+                // the migrated legacy roof never renders. New roofs use 0.5 too.
+                wallHeight: 0.5,
+                pitch: getPitchFromActiveRoofHeight({
+                    roofType: 'gable',
+                    width: segWidth,
+                    depth: segDepth,
+                    roofHeight: legacyRoofHeight,
+                }),
                 wallThickness: 0.1,
                 deckThickness: 0.1,
                 overhang: 0.3,
@@ -269,6 +381,35 @@ function migrateNodes(nodes) {
                 children: [segmentId],
             };
         }
+        // 2b. roof-segment: guarantee a valid positive pitch (degrees).
+        // Saved scenes wrote `roofHeight` in metres; the schema now stores `pitch`
+        // in degrees. Convert the legacy field when present, and — crucially — fall
+        // back to the schema default for any segment that carries no usable pitch or
+        // roofHeight (older/partial saves). Without this, the slope-frame guard
+        // resolves a missing pitch to a FLAT frame, so the roof renders as a slab.
+        // The migration result is cast, not zod-parsed, so the schema default never
+        // applies on its own — this branch is the only place it lands.
+        if (node.type === 'roof-segment') {
+            const currentPitch = node.pitch;
+            const hasValidPitch = typeof currentPitch === 'number' && currentPitch > 0;
+            if (!hasValidPitch) {
+                const { roofHeight, ...rest } = node;
+                const width = typeof node.width === 'number' ? node.width : 8;
+                const depth = typeof node.depth === 'number' ? node.depth : 6;
+                const roofType = (typeof node.roofType === 'string' ? node.roofType : 'gable');
+                const derived = typeof roofHeight === 'number' && roofHeight > 0
+                    ? getPitchFromActiveRoofHeight({ roofType, width, depth, roofHeight })
+                    : 0;
+                // 40° matches the RoofSegmentNode schema default.
+                patchedNodes[id] = { ...rest, pitch: derived > 0 ? derived : 40 };
+            }
+        }
+        if (node.type === 'door') {
+            const normalized = normalizeDoorNode(node);
+            if (normalized) {
+                patchedNodes[id] = normalized;
+            }
+        }
         if (node.type === 'stair') {
             const normalized = normalizeStairNode(migrateStairSurfaceMaterials(node));
             if (normalized) {
@@ -284,9 +425,57 @@ function migrateNodes(nodes) {
         if (node.type === 'wall') {
             patchedNodes[id] = migrateWallSurfaceMaterials(patchedNodes[id]);
         }
+        if (node.type === 'shelf') {
+            const normalized = normalizeShelfNode(node);
+            if (normalized) {
+                patchedNodes[id] = normalized;
+            }
+        }
+        if (node.type === 'elevator') {
+            const parentMigrated = migrateElevatorParent(id, node, patchedNodes);
+            const normalized = normalizeElevatorNode(parentMigrated);
+            if (normalized) {
+                patchedNodes[id] = normalized;
+            }
+        }
+        // Roof-segment hosting was added in this migration cycle (the same
+        // pattern as shelf above). Older segments saved before the schema
+        // gained `children` need the field initialised so
+        // `createNode(chimney, segmentId)` finds an array to append to —
+        // without this every "Add Element" click on the roof panel results
+        // in an orphaned accessory (parented in scene state but never
+        // appended to `seg.children`, so the renderer's recursive
+        // `<NodeRenderer>` mount never sees it).
+        if (node.type === 'roof-segment' && !Array.isArray(node.children)) {
+            patchedNodes[id] = { ...node, children: [] };
+        }
         if (node.type === 'roof') {
             patchedNodes[id] = migrateRoofSurfaceMaterials(patchedNodes[id]);
         }
+        // Legacy: site.children used to hold nested BuildingNode / ItemNode
+        // objects (see the SiteNode schema before the children-as-ids fix).
+        // Flatten any leftover nested children into ids, and absorb the
+        // embedded nodes into the flat map so the rest of the loader can
+        // treat the site like every other parent.
+        if (node.type === 'site' && Array.isArray(node.children)) {
+            let needsFlatten = false;
+            const flattened = [];
+            for (const child of node.children) {
+                if (typeof child === 'string') {
+                    flattened.push(child);
+                }
+                else if (child && typeof child === 'object' && typeof child.id === 'string') {
+                    needsFlatten = true;
+                    flattened.push(child.id);
+                    if (!patchedNodes[child.id]) {
+                        patchedNodes[child.id] = { ...child, parentId: id };
+                    }
+                }
+            }
+            if (needsFlatten) {
+                patchedNodes[id] = { ...node, children: flattened };
+            }
+        }
     }
     return patchedNodes;
 }
@@ -417,7 +606,7 @@ const useScene = create()(temporal((set, get) => ({
             children: [level0.id],
         });
         const site = SiteNode.parse({
-            children: [building],
+            children: [building.id],
         });
         // Define all nodes flat
         const nodes = {
@@ -437,6 +626,7 @@ const useScene = create()(temporal((set, get) => ({
     },
     createNodes: (ops) => nodeActions.createNodesAction(set, get, ops),
     createNode: (node, parentId) => nodeActions.createNodesAction(set, get, [{ node, parentId }]),
+    applyNodeChanges: (changes) => nodeActions.applyNodeChangesAction(set, get, changes),
     updateNodes: (updates) => nodeActions.updateNodesAction(set, get, updates),
     updateNode: (id, data) => nodeActions.updateNodesAction(set, get, [{ id, data }]),
     // --- DELETE ---

package/dist/systems/elevator/elevator-dispatch.d.ts

@@ -0,0 +1,12 @@
+import type { AnyNode, AnyNodeId } from '../../schema';
+import type { ElevatorInteractiveState } from '../../store/use-interactive';
+type ElevatorRuntimeMap = Record<AnyNodeId, ElevatorInteractiveState>;
+type ResolveElevatorDispatchTargetArgs = {
+    elevators: ElevatorRuntimeMap;
+    levelId: AnyNodeId;
+    nodes: Record<string, AnyNode>;
+    requestedElevatorId: AnyNodeId;
+};
+export declare function resolveElevatorDispatchTarget({ elevators, levelId, nodes, requestedElevatorId, }: ResolveElevatorDispatchTargetArgs): AnyNodeId;
+export {};
+//# sourceMappingURL=elevator-dispatch.d.ts.map

package/dist/systems/elevator/elevator-dispatch.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"elevator-dispatch.d.ts","sourceRoot":"","sources":["../../../src/systems/elevator/elevator-dispatch.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,OAAO,EAAE,SAAS,EAAgB,MAAM,cAAc,CAAA;AACpE,OAAO,KAAK,EAAE,wBAAwB,EAAE,MAAM,6BAA6B,CAAA;AAG3E,KAAK,kBAAkB,GAAG,MAAM,CAAC,SAAS,EAAE,wBAAwB,CAAC,CAAA;AAErE,KAAK,iCAAiC,GAAG;IACvC,SAAS,EAAE,kBAAkB,CAAA;IAC7B,OAAO,EAAE,SAAS,CAAA;IAClB,KAAK,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAA;IAC9B,mBAAmB,EAAE,SAAS,CAAA;CAC/B,CAAA;AAsCD,wBAAgB,6BAA6B,CAAC,EAC5C,SAAS,EACT,OAAO,EACP,KAAK,EACL,mBAAmB,GACpB,EAAE,iCAAiC,GAAG,SAAS,CAmC/C"}

package/dist/systems/elevator/elevator-dispatch.js

@@ -0,0 +1,57 @@
+import { resolveElevatorServiceLevels } from './elevator-service';
+function getRuntimeLevelId(runtime, elevator) {
+    return runtime?.currentLevelId ?? elevator.defaultLevelId ?? elevator.fromLevelId ?? null;
+}
+function scoreDispatchCandidate({ elevator, elevators, levelId, nodes, }) {
+    if ((elevator.disabledLevelIds ?? []).includes(levelId))
+        return null;
+    if ((elevator.serviceOnlyLevelIds ?? []).includes(levelId))
+        return null;
+    const serviceLevels = resolveElevatorServiceLevels(elevator, nodes);
+    const targetIndex = serviceLevels.findIndex((level) => level.id === levelId);
+    if (targetIndex < 0)
+        return null;
+    const runtime = elevators[elevator.id];
+    const runtimeLevelId = getRuntimeLevelId(runtime, elevator);
+    const currentIndex = runtimeLevelId
+        ? serviceLevels.findIndex((level) => level.id === runtimeLevelId)
+        : -1;
+    const resolvedCurrentIndex = currentIndex >= 0 ? currentIndex : 0;
+    const distance = Math.abs(targetIndex - resolvedCurrentIndex);
+    const queuePenalty = (runtime?.queue.length ?? 0) * 2 + (runtime?.targetLevelId ? 1 : 0);
+    const motionPenalty = runtime?.phase === 'moving' || runtime?.phase === 'closing' ? 0.5 : 0;
+    const openPenalty = runtime?.phase === 'open' || runtime?.phase === 'opening' ? 0.2 : 0;
+    return distance + queuePenalty + motionPenalty + openPenalty;
+}
+export function resolveElevatorDispatchTarget({ elevators, levelId, nodes, requestedElevatorId, }) {
+    const requestedElevator = nodes[requestedElevatorId];
+    if (!(requestedElevator?.type === 'elevator' && requestedElevator.parentId)) {
+        return requestedElevatorId;
+    }
+    const building = nodes[requestedElevator.parentId];
+    if (building?.type !== 'building') {
+        return requestedElevatorId;
+    }
+    let bestElevatorId = null;
+    let bestScore = Number.POSITIVE_INFINITY;
+    for (const childId of building.children) {
+        const candidate = nodes[childId];
+        if (!(candidate?.type === 'elevator' && candidate.visible !== false))
+            continue;
+        const score = scoreDispatchCandidate({
+            elevator: candidate,
+            elevators,
+            levelId,
+            nodes,
+        });
+        if (score === null)
+            continue;
+        const tieBreaker = candidate.id === requestedElevatorId ? -0.01 : 0;
+        const finalScore = score + tieBreaker;
+        if (finalScore < bestScore) {
+            bestScore = finalScore;
+            bestElevatorId = candidate.id;
+        }
+    }
+    return bestElevatorId ?? requestedElevatorId;
+}

package/dist/systems/elevator/elevator-geometry.d.ts

@@ -0,0 +1,15 @@
+import type { ElevatorDoorPanelStyle, ElevatorDoorStyle, ElevatorNode, ElevatorShaftStyle } from '../../schema';
+export type ElevatorDoorSide = 'left' | 'right';
+export declare function getResolvedElevatorDoorStyle(doorStyle: ElevatorNode['doorStyle'] | undefined): ElevatorDoorStyle;
+export declare function getResolvedElevatorDoorPanelStyle(doorPanelStyle: ElevatorNode['doorPanelStyle'] | undefined): ElevatorDoorPanelStyle;
+export declare function getResolvedElevatorShaftStyle(shaftStyle: ElevatorNode['shaftStyle'] | undefined): ElevatorShaftStyle;
+export declare function getElevatorDoorLeafSides(doorStyle: ElevatorNode['doorStyle'] | undefined): ElevatorDoorSide[];
+export declare function getElevatorDoorLeafX(side: ElevatorDoorSide, openingWidth: number, doorOpen: number, doorStyle: ElevatorNode['doorStyle'] | undefined): number;
+export declare function getElevatorDoorLeafWidth(openingWidth: number, doorStyle: ElevatorNode['doorStyle'] | undefined): number;
+export declare function getElevatorCabWidth(node: ElevatorNode): number;
+export declare function getElevatorCabDepth(node: ElevatorNode): number;
+export declare function getElevatorShaftWallThickness(node: ElevatorNode): number;
+export declare function getElevatorShaftWidth(node: ElevatorNode, cabWidth?: number): number;
+export declare function getElevatorShaftDepth(node: ElevatorNode, cabDepth?: number): number;
+export declare function getElevatorCabCenterZ(node: ElevatorNode): number;
+//# sourceMappingURL=elevator-geometry.d.ts.map

package/dist/systems/elevator/elevator-geometry.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"elevator-geometry.d.ts","sourceRoot":"","sources":["../../../src/systems/elevator/elevator-geometry.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,sBAAsB,EACtB,iBAAiB,EACjB,YAAY,EACZ,kBAAkB,EACnB,MAAM,cAAc,CAAA;AAErB,MAAM,MAAM,gBAAgB,GAAG,MAAM,GAAG,OAAO,CAAA;AAI/C,wBAAgB,4BAA4B,CAC1C,SAAS,EAAE,YAAY,CAAC,WAAW,CAAC,GAAG,SAAS,GAC/C,iBAAiB,CAEnB;AAED,wBAAgB,iCAAiC,CAC/C,cAAc,EAAE,YAAY,CAAC,gBAAgB,CAAC,GAAG,SAAS,GACzD,sBAAsB,CAExB;AAED,wBAAgB,6BAA6B,CAC3C,UAAU,EAAE,YAAY,CAAC,YAAY,CAAC,GAAG,SAAS,GACjD,kBAAkB,CAEpB;AAED,wBAAgB,wBAAwB,CACtC,SAAS,EAAE,YAAY,CAAC,WAAW,CAAC,GAAG,SAAS,GAC/C,gBAAgB,EAAE,CAKpB;AAED,wBAAgB,oBAAoB,CAClC,IAAI,EAAE,gBAAgB,EACtB,YAAY,EAAE,MAAM,EACpB,QAAQ,EAAE,MAAM,EAChB,SAAS,EAAE,YAAY,CAAC,WAAW,CAAC,GAAG,SAAS,UAUjD;AAED,wBAAgB,wBAAwB,CACtC,YAAY,EAAE,MAAM,EACpB,SAAS,EAAE,YAAY,CAAC,WAAW,CAAC,GAAG,SAAS,UAKjD;AAED,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,YAAY,UAErD;AAED,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,YAAY,UAErD;AAED,wBAAgB,6BAA6B,CAAC,IAAI,EAAE,YAAY,UAE/D;AAED,wBAAgB,qBAAqB,CAAC,IAAI,EAAE,YAAY,EAAE,QAAQ,SAA4B,UAE7F;AAED,wBAAgB,qBAAqB,CAAC,IAAI,EAAE,YAAY,EAAE,QAAQ,SAA4B,UAE7F;AAED,wBAAgB,qBAAqB,CAAC,IAAI,EAAE,YAAY,UAEvD"}

package/dist/systems/elevator/elevator-geometry.js

@@ -0,0 +1,50 @@
+const DEFAULT_ELEVATOR_SHAFT_WALL_THICKNESS = 0.09;
+export function getResolvedElevatorDoorStyle(doorStyle) {
+    return doorStyle ?? 'center-opening';
+}
+export function getResolvedElevatorDoorPanelStyle(doorPanelStyle) {
+    return doorPanelStyle ?? 'glass-frame';
+}
+export function getResolvedElevatorShaftStyle(shaftStyle) {
+    return shaftStyle ?? 'solid';
+}
+export function getElevatorDoorLeafSides(doorStyle) {
+    const resolvedDoorStyle = getResolvedElevatorDoorStyle(doorStyle);
+    if (resolvedDoorStyle === 'single-left')
+        return ['left'];
+    if (resolvedDoorStyle === 'single-right')
+        return ['right'];
+    return ['left', 'right'];
+}
+export function getElevatorDoorLeafX(side, openingWidth, doorOpen, doorStyle) {
+    const resolvedDoorStyle = getResolvedElevatorDoorStyle(doorStyle);
+    if (resolvedDoorStyle === 'center-opening') {
+        const direction = side === 'left' ? -1 : 1;
+        return direction * (openingWidth / 4 + doorOpen * openingWidth * 0.34);
+    }
+    const direction = resolvedDoorStyle === 'single-left' ? -1 : 1;
+    return direction * doorOpen * openingWidth * 0.68;
+}
+export function getElevatorDoorLeafWidth(openingWidth, doorStyle) {
+    return getResolvedElevatorDoorStyle(doorStyle) === 'center-opening'
+        ? Math.max(openingWidth / 2 - 0.018, 0.12)
+        : Math.max(openingWidth - 0.018, 0.18);
+}
+export function getElevatorCabWidth(node) {
+    return Math.max(node.width, 0.8);
+}
+export function getElevatorCabDepth(node) {
+    return Math.max(node.depth, 0.8);
+}
+export function getElevatorShaftWallThickness(node) {
+    return Math.max(node.shaftWallThickness ?? DEFAULT_ELEVATOR_SHAFT_WALL_THICKNESS, 0.04);
+}
+export function getElevatorShaftWidth(node, cabWidth = getElevatorCabWidth(node)) {
+    return Math.max(node.shaftWidth ?? cabWidth, cabWidth, 0.8);
+}
+export function getElevatorShaftDepth(node, cabDepth = getElevatorCabDepth(node)) {
+    return Math.max(node.shaftDepth ?? cabDepth, cabDepth, 0.8);
+}
+export function getElevatorCabCenterZ(node) {
+    return -getElevatorShaftDepth(node) / 2 + getElevatorCabDepth(node) / 2;
+}

package/dist/systems/elevator/elevator-opening-sync.d.ts

@@ -0,0 +1,6 @@
+import type { AnyNode, AnyNodeId, CeilingNode, SlabNode } from '../../schema';
+export declare function syncAutoElevatorOpenings(nodes: Record<string, AnyNode>): {
+    id: AnyNodeId;
+    data: Partial<SlabNode | CeilingNode>;
+}[];
+//# sourceMappingURL=elevator-opening-sync.d.ts.map

package/dist/systems/elevator/elevator-opening-sync.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"elevator-opening-sync.d.ts","sourceRoot":"","sources":["../../../src/systems/elevator/elevator-opening-sync.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EACV,OAAO,EACP,SAAS,EACT,WAAW,EAEX,QAAQ,EAET,MAAM,cAAc,CAAA;AA8JrB,wBAAgB,wBAAwB,CAAC,KAAK,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC;QAQ1C,SAAS;UAAQ,OAAO,CAAC,QAAQ,GAAG,WAAW,CAAC;IAmG5E"}