Skip to content

Audit fixes: memory leaks, perf, dialog/backend consolidation#287

Merged
milanofthe merged 16 commits intomainfrom
chore/audit-fixes
May 9, 2026
Merged

Audit fixes: memory leaks, perf, dialog/backend consolidation#287
milanofthe merged 16 commits intomainfrom
chore/audit-fixes

Conversation

@milanofthe
Copy link
Copy Markdown
Member

@milanofthe milanofthe commented May 9, 2026

Summary

  • Fix 5 store-subscribe leaks (BlockPropertiesDialog recordingData, EventsPanel, NodeLibrary, ToolboxManagerDialog, +page.svelte)
  • Avoid quadratic reallocation in mergeStreamingResult (in-place append) and replace JSON.stringify change-detection in FlowCanvas with shallow-equal helpers; 12x JSON.parse(JSON.stringify(...)) -> structuredClone
  • Extract shared abstractions: DialogShell (9 dialogs migrated), createHoverDetail action (NodeLibrary + EventsPanel), AbstractBackend (Pyodide + Flask), applyWaypointUpdate helper (4 routing methods)
  • Misc: lazy-load WelcomeModal screenshots, LRU-cap inlineMath cache at 500, consolidate routing/dimension magic numbers

@milanofthe milanofthe merged commit d36d18a into main May 9, 2026
4 checks passed
@milanofthe milanofthe deleted the chore/audit-fixes branch May 9, 2026 10:47
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