editor: fix roof and placement previews

[sudhir9297]

What does this PR do?

Improves roof, dormer, lean-to, and window placement so cursor rays, ghosts, and committed items resolve to the exact pointer location, including centered freestanding lean-to placement. Transient placement previews now use live overrides during pointer movement, snapping follows the active mode, and shared helpers stay within the correct architecture boundaries.

How to test

1. Run bun run check and bun run check-types.
2. Run the focused tests with bun test packages/nodes/src/lean-to-extension packages/nodes/src/dormer packages/nodes/src/window packages/editor/src/components/tools/shared/pointer-support-cap.test.ts.
3. Run bun dev, place and move a freestanding lean-to, dormer window, wall window, and roof-face window; confirm the ghost and final item stay centered under the cursor and follow the same raycast position.
4. Confirm the lean-to footprint remains centered after rotation and movement, and that grid/line/off snapping modes behave consistently.

The package builds for core, nodes, and viewer pass. The full app production build is currently blocked by the unchanged react-scan dependency's webpack export mismatch; Turbopack also hits an environment process-permission error.

Screenshots / screen recording

Not included — visual verification should be performed with the interactive placement steps above.

Checklist

• I've tested this locally with bun dev
• My code follows the existing code style (run bun check to verify)
• I've updated relevant documentation (if applicable)
• This PR targets the main branch

---

Note

High Risk
Touches roof schema, scene migration, CSG/dormer geometry, and placement raycasting—areas that affect saved scenes and structural editing correctness.

Overview
Adds conical as a roof segment type with mesh/surface/print support, optional RoofNode.support for level vs roof-surface mounting, and resolveConicalRoofPlacement (auto / ground / roof). The roof tool draws circular footprints, cycles placement with P, and tints ghosts when roof-only placement fails.

Dormers move from inline parametric windows to WindowNode children on wall faces (with scene load migration and sidebar tree expansion). CSG and cuts use hosted window geometry; shedHighSide controls shed pitch direction.

Lean-to schema grows richer hosting (hostKind, slab edges, conical base), omittedPostSlots when managed posts are deleted, and handle connectionSnap (Alt bypass) for structural edge joining. Roof overlap logic treats mounted conicals so hosts clip towers in plan.

Editor / UX: orthographic pointer rays no longer drift off-cursor; item placement rotation stays aligned on parent-hosted surfaces; registry tools get isCameraDragging and a wider RegistryToolProvider; read-only scenes hide mutating chrome; Delete mode removes floorplan picks; Build tab roof-feature tiles only highlight the matching accessory (not the plain roof tool). Keyboard R/T defer to lean-to and item placement tools.

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

[cursor]

Cursor Bugbot has reviewed your changes using high effort and found 2 potential issues.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

^{Reviewed by Cursor Bugbot for commit 1f790d3. Configure here.}

[cursor]

Delete mode blocked by move

High Severity

Delete handling was moved into emitCanvasNodeSelection, but the existing click-to-move early return still runs first. In delete mode, clicking an already-selected movable node picks it up for move and returns before delete can run, so that click never removes the node.

 

^{Reviewed by Cursor Bugbot for commit 1f790d3. Configure here.}

[cursor]

Lean-to move double rotation

Medium Severity

isToolOwnedRotation now stands down for lean-to placement, and for moving doors, windows, and items, but not for a moving lean-to-extension. The registry move tool still handles R/T, so a lean-to move with an active selection can apply both the global and move-tool rotations.

Additional Locations (1)

• packages/editor/src/hooks/use-keyboard.test.ts#L50-L62

 

^{Reviewed by Cursor Bugbot for commit 1f790d3. Configure here.}