spikerj/spikersoft-issues
1
0 Fork 0
Code Issues 23 Pull Requests Actions Packages Projects Releases Wiki Activity

RadarHUD.rebuildUI() disposes the blip pool so contacts vanish after resize #1072

New Issue
Closed
opened 2026-08-13 13:29:09 +00:00 by spikerj · 1 comment
spikerj commented 2026-08-13 13:29:09 +00:00
Owner
Copy link
Copy source

Where: projects/spikersoft/src/app/_components/_games/space-game/radar/radar-hud.ts — rebuildUI() (called from resize() once UI already exists).

What: Constructor createBlipPool(50) adds 50 meshes to this.group. A later resize() walks every child, dispose()s Mesh geometry/material, then createUI() without re-adding pool blips. activeBlips is also not cleared, so updateBlips() reuses disposed meshes that are no longer in the scene graph.

Result: after a window resize, radar contacts stop rendering (or render with disposed geometry).

Expected: rebuild should either skip pooled blips, recreate the pool, or clear activeBlips and re-add unused pool meshes to the group.

Found by Wave 8 a2 coverage tests.

**Where:** `projects/spikersoft/src/app/_components/_games/space-game/radar/radar-hud.ts` — `rebuildUI()` (called from `resize()` once UI already exists). **What:** Constructor `createBlipPool(50)` adds 50 meshes to `this.group`. A later `resize()` walks every child, `dispose()`s Mesh geometry/material, then `createUI()` without re-adding pool blips. `activeBlips` is also not cleared, so `updateBlips()` reuses disposed meshes that are no longer in the scene graph. **Result:** after a window resize, radar contacts stop rendering (or render with disposed geometry). **Expected:** rebuild should either skip pooled blips, recreate the pool, or clear `activeBlips` and re-add unused pool meshes to the group. Found by Wave 8 a2 coverage tests.
spikerj commented 2026-08-14 03:14:32 +00:00
Author
Owner
Copy link
Copy source

Resolved in spikersoft-angular PR #819 (merged to master). RadarHUD resize keeps the blip pool so contacts do not vanish. Closing.

Resolved in spikersoft-angular [PR #819](https://git.spikersoft.com/spikerj/spikersoft-angular/pulls/819) (merged to `master`). RadarHUD resize keeps the blip pool so contacts do not vanish. Closing.
Sign in to join this conversation.
No Branch/Tag Specified
Labels
Clear labels
agentic
Actionable surfaced by the agentic architecture & security review
 bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 epic
Cross-repo epic — children live in per-repo trackers
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 sonarqube
Code quality issue surfaced by SonarQube (bug / code smell)
 wontfix
This won't be fixed
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
spikerj (Joseph Spiker)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: spikerj/spikersoft-issues#1072