Skip to content

fix: handle localStorage errors - #639

Open
Guflly wants to merge 1 commit into
UTDNebula:developfrom
Guflly:bugfix/local-storage-errors
Open

fix: handle localStorage errors#639
Guflly wants to merge 1 commit into
UTDNebula:developfrom
Guflly:bugfix/local-storage-errors

Conversation

@Guflly

@Guflly Guflly commented Jul 31, 2026

Copy link
Copy Markdown

Overview

Closes #634.

Guard the app's localStorage reads and writes so browsers that block storage do not break persistent state or recent searches.

What Changed

  • Treat blocked reads as missing storage.
  • Keep in-memory state updates when persistence fails.
  • Return an empty recent-search list or skip the write when storage is unavailable.
  • Add regression tests for all four production storage calls.

Other Notes

npm run lint:check, npm run format:check, npm run type:check, and all 6 Jest tests pass.

@Guflly
Guflly requested review from a team and mikehquan19 as code owners July 31, 2026 10:24
@vercel

vercel Bot commented Jul 31, 2026

Copy link
Copy Markdown

Someone is attempting to deploy a commit to the Nebula Labs Team on Vercel.

A member of the Team first needs to authorize it.

@Guflly
Guflly force-pushed the bugfix/local-storage-errors branch from 954f1cf to 0719824 Compare July 31, 2026 12:54
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.

Surround localStorage gets with a try-catch

1 participant