Update Modal

[MaryWylde]

No description provided.

[github-actions]

Claude finished @MaryWylde's task in 1m 30s —— View job

---

Claude's Review

Todo list

• Gather context (read diff, read changed file)
• Understand the request
• Review correctness, React best practices, SSR safety
• Review TypeScript quality
• Check project conventions (AGENTS.md)
• Check accessibility/security/styling
• Post final review summary

Summary

This PR switches the "new update" modal's dismissal storage from sessionStorage to localStorage in src/uxcore/components/NewUpdateModal/NewUpdateModalContainer.tsx (lines 29, 55), with the code comment updated to match ("until site data is cleared" instead of "for the rest of the session").

This is a correct, minimal fix. The prior commit's message (fix(uxcore): persist new-update modal dismissal across tabs and restarts) already stated the intent to persist dismissal across tabs and browser restarts, but sessionStorage doesn't actually do that — it's scoped per-tab and cleared on browser close. localStorage is the right primitive to achieve that stated goal.

Checked against review criteria:

• SSR/hydration safety: both localStorage accesses happen inside useEffect (line 29) and a click handler (handleClose, line 55) — never at module top level or render time, so no hydration risk.
• Error handling: getItem on line 29 is inside the surrounding try/catch (line 20–43); setItem on line 55 has its own try/catch for private-mode storage failures — both paths are safe if storage is unavailable.
• TypeScript: no any, types unchanged and still precise.
• Conventions: no import/order, barrel, or styling issues — diff only touches two lines of logic plus the comment. localStorage is already an established pattern elsewhere in this codebase (e.g. useGlobals.ts, AuthContext.tsx), so this is consistent.

No issues found — LGTM.