Repository navigation
every browser reload wipes all open tabs... session restore has been commented out #348
Description
The useEffect in GraphWorkspace.jsx that restores graphs from localStorage on mount is fully commented out. Comment even says "currently on hold". So every refresh = blank canvas, all your tabs gone.
The annoying part is localStorageManager is already doing its job nd it saves every graph on change. The data is there, it's just never read back on reload.
There's also a secondary bug in local-storage-manager.js: allgs is a module-level Set initialized once at import time, but getAllGraphs() reads fresh from localStorage independently. These diverge after any restart so you can end up with inconsistent state about which graphs are "known."
Needs the mount effect uncommented and rewritten properly, the in-memory Set removed in favor of a single localStorage source of truth, and some basic handling for stale/corrupt entries.
Activity
avinxshKD commentedon Feb 25, 2026
Hey @pradeeban reproduced on main
Plan is to re-enable session restore in GraphWorkspace and clean up localStorageManager so graph IDs come from a single source of truth (no in-memory/storage drift). I’ll include guards for stale/corrupt entries and duplicate IDs so reload behavior is deterministic.
I can also include a regression test for reload/restore path.
Do you have any suggestions? would love to know
pradeeban commentedon Mar 1, 2026
Sounds good.
fix: restore session tabs on reload, remove allgs Set divergence (Con…