Skip to content

fix(constellarium): round 3.1 — pan restored, phantom pin removed, WebGL guard - #150

Merged
vicquick merged 1 commit into
mainfrom
fix/constellarium-obsidian-round3
Aug 22, 2026
Merged

fix(constellarium): round 3.1 — pan restored, phantom pin removed, WebGL guard#150
vicquick merged 1 commit into
mainfrom
fix/constellarium-obsidian-round3

Conversation

@vicquick

Copy link
Copy Markdown
Owner

Closes the "still doesn't behave like Obsidian" thread with empirical verification: a raw-CDP behaviour suite (no playwright — playwright-python retained every exceptionThrown CDP event from a 60fps render-error storm, leaked ~3MB/s and was cgroup-OOM-killed at 4GB) runs nine asserts against the live view, all green:

assert result
hover engages after settle
hover releases on empty sky
node drag runs
background pan not hijacked after drag
pan moves the world (node reappears at old + Δ)
click pins folio
pin dismisses via ×
zoom is cursor-anchored
zoom survives without crash

Three real defects the harness surfaced:

  1. Grab radius wider than hover radiusgrabSubject used nodeRadius+4 with a 10-world-unit floor while the pixi hitArea is nodeRadius+2. An empty-sky press silently grabbed an unseen star; d3-drag claimed the gesture and the pan never reached d3-zoom. Grab now mirrors the hitArea: if it isn't hoverable, it isn't grabbable.
  2. Phantom pin after every drag — pixi pointertap fires when down+up land on the same object, and a dragged star travels with the cursor, so every drag ended pinned. Click-to-pin now lives solely in d3-drag's end handler (pointer distance < 5 px).
  3. Canvas-fallback render storm — pixi's canvas renderer has no pipe for the round-3 GL link mesh and threw on every render: frozen world, stale worldTransforms, dead hit-tests, and an error storm. Non-WebGL browsers now get a quiet notice instead of the storm.

Round-3 render core (single GPU mesh for all links — no per-frame tessellation of 58k strokes; one shared hover animation instead of 60k tween allocations) rides along with its deps: pixi.js 8.20, d3 modules, tween.js.

Dev-only test seam: window.__app exposed under import.meta.env.DEV for the behaviour suite.

🤖 Generated with Claude Code

https://claude.ai/code/session_0137xgmfhc8co2pjNDuZw6FR

@vicquick
vicquick enabled auto-merge (squash) August 22, 2026 06:52
…bGL guard

Empirically verified against a raw-CDP Playwright-free harness (playwright-
python leaked ~3MB/s of retained exceptionThrown events and got cgroup-OOM-
killed at 4GB attaching to this page): all nine behaviour asserts green —
hover engage/release, node drag, background pan moves the world, pan not
hijacked after drag, click pins, pin dismisses, zoom cursor-anchored.

Three real defects found by the harness:
- grab radius (nodeRadius+4, floor 10 world units) was wider than the pixi
  hitArea (nodeRadius+2): an empty-sky press silently grabbed an unseen
  star, so d3-drag claimed the gesture and background pan never reached
  d3-zoom. Grab now mirrors the hitArea exactly.
- pixi pointertap pinned after EVERY node drag: the dragged star travels
  with the cursor, so pixi sees down+up over the same object. Click-to-pin
  now lives solely in d3-drag's end handler (pointer distance < 5px).
- pixi's canvas fallback has no pipe for the round-3 GL link mesh and threw
  on every render — a 60fps uncaught-exception storm with frozen world and
  stale hit tests. Non-WebGL renderers now get a quiet notice instead.

Deps: pixi.js 8.20, d3-{force,selection,zoom,drag,transition}, tween.js
(round-3 render core: single GPU mesh for links, shared hover animation).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0137xgmfhc8co2pjNDuZw6FR
@vicquick
vicquick force-pushed the fix/constellarium-obsidian-round3 branch from b68e42d to 81a590e Compare August 22, 2026 06:53
@vicquick
vicquick merged commit ca322ad into main Aug 22, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant