@pascal-app/core 1.0.0-beta.3 → 1.0.0-beta.4

package/dist/store/use-scene.js

@@ -1080,22 +1080,24 @@ const useScene = create()(temporal((set, get) => ({
             });
             return; // Scene already loaded
         }
-        // Create hierarchy: Site → Building → Level
+        // Create hierarchy: Site → Building → Level. Parent links must be
+        // written explicitly — the schema defaults `parentId` to null, and the
+        // scene authority rejects parent/child asymmetry that the renderer
+        // happily traverses through `children`.
+        const site = SiteNode.parse({});
+        const building = BuildingNode.parse({
+            parentId: site.id,
+        });
         const level0 = LevelNode.parse({
+            parentId: building.id,
             level: 0,
             children: [],
             height: 2.5,
         });
-        const building = BuildingNode.parse({
-            children: [level0.id],
-        });
-        const site = SiteNode.parse({
-            children: [building.id],
-        });
         // Define all nodes flat
         const nodes = {
-            [site.id]: site,
-            [building.id]: building,
+            [site.id]: { ...site, children: [building.id] },
+            [building.id]: { ...building, children: [level0.id] },
             [level0.id]: level0,
         };
         // Site is the root

package/dist/utils/heal-scene-graph.d.ts

@@ -9,6 +9,11 @@ export interface HealSceneResult {
      * a different node (stale reparent leftovers), plus same-array duplicates.
      */
     strippedStaleChildRefs: number;
+    /**
+     * Ids of nodes whose null `parentId` was repaired to the one parent that
+     * still claims them via `children`.
+     */
+    repairedParentLinkNodeIds: string[];
 }
 /**
  * Returns a healed copy of a `nodes` map. Pure — does not mutate `input`.

package/dist/utils/heal-scene-graph.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"heal-scene-graph.d.ts","sourceRoot":"","sources":["../../src/utils/heal-scene-graph.ts"],"names":[],"mappings":"AAqBA,MAAM,WAAW,eAAe;IAC9B,KAAK,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAA;IAC9B,kDAAkD;IAClD,cAAc,EAAE,MAAM,EAAE,CAAA;IACxB,sFAAsF;IACtF,iBAAiB,EAAE,MAAM,CAAA;IACzB;;;OAGG;IACH,sBAAsB,EAAE,MAAM,CAAA;CAC/B;AAgBD;;;GAGG;AACH,wBAAgB,cAAc,CAAC,KAAK,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,eAAe,CA2E9E"}
+{"version":3,"file":"heal-scene-graph.d.ts","sourceRoot":"","sources":["../../src/utils/heal-scene-graph.ts"],"names":[],"mappings":"AA0BA,MAAM,WAAW,eAAe;IAC9B,KAAK,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAA;IAC9B,kDAAkD;IAClD,cAAc,EAAE,MAAM,EAAE,CAAA;IACxB,sFAAsF;IACtF,iBAAiB,EAAE,MAAM,CAAA;IACzB;;;OAGG;IACH,sBAAsB,EAAE,MAAM,CAAA;IAC9B;;;OAGG;IACH,yBAAyB,EAAE,MAAM,EAAE,CAAA;CACpC;AAgBD;;;GAGG;AACH,wBAAgB,cAAc,CAAC,KAAK,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,eAAe,CAiH9E"}

package/dist/utils/heal-scene-graph.js

@@ -13,6 +13,11 @@
 //     duplicate reference renders the child twice (duplicate React keys in the
 //     2D plan, doubled hosted geometry in 3D). Same-array duplicates are
 //     collapsed too.
+//  4. A node with a null `parentId` that exactly one parent still claims via
+//     `children` (the legacy site-child flatten and the old default-scene
+//     assembler both linked children without writing `parentId`). The editor
+//     renders the chain through `children`, but the hosted scene authority
+//     validates parent/child symmetry and rejects the whole scene.
 //
 // All are also prevented at the source now; this is the load-time safety net
 // for already-saved scenes.
@@ -102,5 +107,45 @@ export function healSceneNodes(input) {
         }
         nodes[id] = node;
     }
-    return { nodes, droppedWallIds, strippedChildRefs, strippedStaleChildRefs };
+    // Pass 3: repair null parent links. A node claimed as a child by exactly one
+    // parent must point back at it; legacy writers linked `children` without
+    // writing `parentId`. Embedded legacy site children claim by their `id` so
+    // the flattened flat-map node is repaired too.
+    const claimantsByChildId = new Map();
+    for (const [id, node] of Object.entries(nodes)) {
+        const children = node?.children;
+        if (!Array.isArray(children))
+            continue;
+        for (const child of children) {
+            const childId = typeof child === 'string'
+                ? child
+                : child && typeof child === 'object' && typeof child.id === 'string'
+                    ? child.id
+                    : null;
+            if (!childId || !(childId in nodes))
+                continue;
+            const claimants = claimantsByChildId.get(childId) ?? [];
+            claimants.push(id);
+            claimantsByChildId.set(childId, claimants);
+        }
+    }
+    const repairedParentLinkNodeIds = [];
+    for (const [id, node] of Object.entries(nodes)) {
+        if (!node || typeof node !== 'object')
+            continue;
+        if (node.parentId != null)
+            continue;
+        const claimants = claimantsByChildId.get(id);
+        if (claimants?.length !== 1 || claimants[0] === id)
+            continue;
+        nodes[id] = { ...node, parentId: claimants[0] };
+        repairedParentLinkNodeIds.push(id);
+    }
+    return {
+        nodes,
+        droppedWallIds,
+        strippedChildRefs,
+        strippedStaleChildRefs,
+        repairedParentLinkNodeIds,
+    };
 }

package/dist/utils/scene-migrations.d.ts

@@ -0,0 +1,3 @@
+export { type HealSceneResult, healSceneNodes } from './heal-scene-graph';
+export { type RetiredSceneNodeMigration, removeRetiredDrawingSheetNodes, } from './retired-scene-nodes';
+//# sourceMappingURL=scene-migrations.d.ts.map

package/dist/utils/scene-migrations.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"scene-migrations.d.ts","sourceRoot":"","sources":["../../src/utils/scene-migrations.ts"],"names":[],"mappings":"AAIA,OAAO,EAAE,KAAK,eAAe,EAAE,cAAc,EAAE,MAAM,oBAAoB,CAAA;AACzE,OAAO,EACL,KAAK,yBAAyB,EAC9B,8BAA8B,GAC/B,MAAM,uBAAuB,CAAA"}

package/dist/utils/scene-migrations.js

@@ -0,0 +1,6 @@
+// Server-safe scene migrations shared by the client loader and the hosted
+// scene authority. Everything exported here must stay pure data logic with no
+// store, React, or Three.js imports so it can run in Server Components and
+// API routes.
+export { healSceneNodes } from './heal-scene-graph';
+export { removeRetiredDrawingSheetNodes, } from './retired-scene-nodes';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pascal-app/core",
-  "version": "1.0.0-beta.3",
+  "version": "1.0.0-beta.4",
   "description": "Core library for Pascal 3D building editor",
   "type": "module",
   "main": "./dist/index.js",
@@ -17,9 +17,9 @@
       "default": "./dist/utils/clone-scene-graph.js"
     },
     "./scene-migrations": {
-      "types": "./dist/utils/retired-scene-nodes.d.ts",
-      "import": "./dist/utils/retired-scene-nodes.js",
-      "default": "./dist/utils/retired-scene-nodes.js"
+      "types": "./dist/utils/scene-migrations.d.ts",
+      "import": "./dist/utils/scene-migrations.js",
+      "default": "./dist/utils/scene-migrations.js"
     },
     "./registry": {
       "types": "./dist/registry/index.d.ts",