Skip to content

fix(web): keep ready subissue labels visible - #90

Merged
teloverge merged 1 commit into
mainfrom
agent/ready-subissue-label-visibility
Aug 8, 2026
Merged

teloverge merged 1 commit into
mainfrom
agent/ready-subissue-label-visibility

Conversation

@teloverge

@teloverge teloverge commented Aug 8, 2026

Copy link
Copy Markdown
Owner

What changed

  • Keep ready subissue labels visible when their outward geometry grazes only their own padded star obstacle.
  • Preserve collision avoidance against every other star and previously placed label.
  • Add a canvas-level regression that checks label visibility and rendered-ring clearance through 20 real camera-easing frames.
  • Declare npm 12.0.2 as the web workspace development package manager and activate it before all nine CI, bundle, and release npm ci steps.
  • Add a job-scoped workflow contract that rejects cross-job setup borrowing and enforces the nine-step inventory.
  • Record both pending changes in the Unreleased changelog.

Root cause

The outward label geometry already cleared its own rendered star and emphasis ring, but the generic collision pass compared its padded reservation with that same star's square obstacle. On diagonal spokes those conservative boxes could graze, causing the ready label to disappear.

The first CI run also showed that Node 24 runners currently provide npm 11.16.0, which correctly rejected the intentional npm 12.0.2 development-engine requirement. Each workflow now activates the declared npm version before dependency installation.

Validation

  • npm test — 28 files, 225 tests passed
  • npm run check — 0 errors and 0 warnings
  • npm run build — production build passed
  • npm toolchain contract — 9 workflow install paths verified
  • Windows, macOS, Linux packaging contracts — passed
  • Release-boundary contract — passed
  • Independent pre-merge review — no Critical or Important findings after amendments

@teloverge
teloverge marked this pull request as ready for review August 8, 2026 19:08
@teloverge
teloverge force-pushed the agent/ready-subissue-label-visibility branch from ed0d491 to 6a8fc3c Compare August 8, 2026 19:24
@teloverge
teloverge merged commit eaab24d into main Aug 8, 2026
9 checks passed
@teloverge
teloverge deleted the agent/ready-subissue-label-visibility branch August 8, 2026 19:38
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