Stop demo annotations from burying the nodes underneath them

[zntznt]

The bug

Notes and charts paint above nodes and win the hit test, so an annotation laid over a node makes it genuinely unreachable: it cannot be clicked, selected or inspected, and the properties panel opens the annotation instead.

Business Cycle laid its CIRCULAR FLOW note across the dashboard row. The note box ran 140..440 x 130..280 and the gdp register sits at (300, 200), so gdp was completely underneath it.

It was not one node

Checking every demo for the same shape found five buried nodes across three demos:

demo	node	buried by
Business Cycle	gdp	note
Business Cycle	rate	note
Business Cycle	Investment	chart
F2P	Rarity Roll	chart
Civ	prodYield	chart

The fix

The two Business Cycle captions now sit above their group rather than inside it, and the three charts move clear of the rows they crossed. Nothing else about the layouts changes.

There was no room to slide the notes along the dashboard row: the widest gap between registers there is 169px against a 300px note, so above the group is where they go. That puts them at negative y, which fitView handles (verified: finite scale and pan).

Scope

The reported finding was Business Cycle, and all three of its buried nodes are fixed here. F2P and Civ are also fixed. The regression net for this is a sweep across every demo, and a sweep that has to whitelist two known-broken demos is not much of a net; they are the identical defect and one coordinate each. Happy to split them out if that is preferred.

Files

• js/app-demos.js — two note positions, three chart positions
• test/smoke.js — regression sweep

Demo layout data and a test. No change to model.js or engine.js, no serialized fields, no behaviour outside the demos.

Testing

• node test/run.js → 235 passed, 0 failed (unchanged)
• npm run smoke → SMOKE PASSED, no console or page errors
• The new smoke test sweeps all 11 demos and requires a hit test at each node's own position to return that node. Without this change it fails naming all five:

buried demo nodes: {"demos":11,"bad":["_demoF2P:Rarity Roll->chart","_demoCiv:prodYield->chart",
 "_demoBusinessCycle:gdp->note","_demoBusinessCycle:rate->note","_demoBusinessCycle:Investment->chart"]}

That sweep is how the F2P and Civ cases were found; the original finding named only Business Cycle.

---

Generated by Claude Code