fix(core): fill slab/ceiling hole fields on scene load

[alxbouchard]

What

Fill holes, holeMetadata and (for slabs) autoFromWalls on slab and ceiling nodes when a scene is loaded, the same way doors and windows are already normalized in setScene.

Why

The schema declares defaults for these fields, so a third-party writer (a scanning app, an IFC hand-off) can legitimately omit them. But the slab opening sync and the public viewer read them on the raw node. A scene whose slab has no holes therefore throws:

[viewer] ErrorBoundary caught (viewer-scene): TypeError: Cannot read properties of undefined (reading 'map')

and the public viewer stays black, while the editor (which fills defaults on load) opens the same file fine.

Repro with a real export from A3 Atlas Scanner (69/69 nodes pass AnyNode.safeParse):

• drag pascal.json into a new project → editor renders it; open the project's public viewer → black screen, error above;
• same graph with holes: [] added on the slab → viewer renders: https://editor.pascal.app/viewer/project_ej9qncUcEV7RX4lo

How

normalizeSurfaceHoleFields in use-scene.ts, applied to slab and ceiling in the load migration. Only the three hole fields are touched: a full zod parse would also default thickness, which the vertical-model migration still needs to see as absent on legacy solids (three existing tests cover that).

Tests

• New use-scene-surface-holes-defaults.test.ts: a slab and a ceiling written without holes load with empty arrays; existing holes and metadata are kept as written. Fails on main, passes with this change.
• bun run test in packages/core: 2125 pass; the one failure and one module-resolution error are pre-existing on main.
• bun check: clean.

Context: this is the scanning-app hand-off discussed in #720 / #737.

---

Note

Low Risk
Narrow load-time normalization for slab/ceiling hole fields only; behavior matches schema defaults and is covered by new tests.

Overview
Scene load now backfills missing hole-related fields on slab and ceiling nodes so third-party exports that omit holes / holeMetadata (and autoFromWalls on slabs) no longer crash the public viewer with undefined.map.

migrateNodes in use-scene.ts calls new normalizeSurfaceHoleFields, which sets empty arrays and autoFromWalls: false when absent. It deliberately avoids a full Zod parse so legacy thickness stays undefined for the vertical-model migration.

New tests in use-scene-surface-holes-defaults.test.ts assert defaults on bare slabs/ceilings and that existing holes/metadata are unchanged.

^{Reviewed by Cursor Bugbot for commit 3f9e966. Bugbot is set up for automated code reviews on this repo. Configure here.}

[pascal]

I hit an error while handling your request (Model unavailable on AI Gateway free tier: Free tier users do not have access to this model. Upgrade to paid credits at https://vercel.com/d?to=%2F%5Bteam%5D%2F%7E%2Fai%3Fmodal%3Dtop-up for unrestricted…).

Please try again, rephrase, or reach out if it keeps failing.

Error id: 9c519d2f-cb49-4245-b6b3-9f54bdd65b60