Bug: disconnect() calls a blanket localStorage.clear()/sessionStorage.clear(), wiping unrelated app state like the theme preference

[Jaydbrown]

Location

contexts/WalletContext.tsx:397-398 (disconnect)

Description

disconnect() calls localStorage.clear() and sessionStorage.clear() unconditionally, not scoped to the wallet-specific storage key. Confirmed via next-themes (a real package.json dependency, used in components/ThemeToggle.tsx) that this also wipes the user's persisted theme preference — an unrelated feature — every time they disconnect their wallet.

Why it matters

Disconnecting a wallet is a routine, frequent action; resetting the user's theme (and any other unrelated localStorage/sessionStorage state any future feature might add) every single time is a surprising, unintended side effect with a growing blast radius as more features start using storage.

Suggested fix

Remove only the specific wallet-session storage key(s) instead of calling a blanket clear().

Acceptance criteria

• disconnect() only removes wallet-specific storage entries
• Theme preference (and other unrelated storage) survives a disconnect
• Test covering that unrelated storage keys are preserved across disconnect

[paud1615-tech]

Hi, i'd like to work on this issue because am very conversant with solving problems like this, and i can submit the pr in less than 48hr pls can u assign it to me

[grantfox-oss]

🦊 GrantFox — @paud1615-tech has been assigned to this issue as part of the Official Campaign | FWC26 campaign!

Next steps:

1. Open a Pull Request referencing this issue (e.g., Closes #237)
2. Your PR will be reviewed by the conduit-protocol maintainers

Good luck! Track your progress on GrantFox.