Fix: re-connecting a locked wallet

src/hooks/wallets/useOnboard.ts

@@ -179,17 +179,15 @@ export const useInitOnboard = () => {
     }
 
     // Connect to the last connected wallet
-    enableWallets().then(() => {
+    enableWallets().then(async () => {
       if (onboard.state.get().wallets.length > 0) return
 
       const label = lastWalletStorage.get()
-      if (!label) return
+      const isUnlocked = label && (await isWalletUnlocked(label))
+      if (!isUnlocked) return
 
-      isWalletUnlocked(label).then((isUnlocked) => {
-        isUnlocked &&
-          connectWallet(onboard, {
-            autoSelect: { label, disableModals: false },
-          })
+      connectWallet(onboard, {
+        autoSelect: { label, disableModals: true },
       })
     })
   }, [chain, onboard])

src/utils/__tests__/wallet.test.ts

@@ -0,0 +1,53 @@
+import { isWalletUnlocked, WalletNames } from '../wallets'
+
+describe('utils/wallet', () => {
+  it('should check if MetaMask is unlocked and return false', async () => {
+    // mock window.ethereum
+    Object.defineProperty(window, 'ethereum', {
+      value: {
+        isMetaMask: true,
+        selectedAddress: '0x123',
+        isConnected: () => true,
+        _metamask: {
+          isUnlocked: () => Promise.resolve(false),
+        },
+      },
+      writable: true,
+    })
+    const result = await isWalletUnlocked(WalletNames.METAMASK)
+    expect(result).toBe(false)
+  })
+
+  it('should check if MetaMask is unlocked and return true', async () => {
+    // mock window.ethereum
+    Object.defineProperty(window, 'ethereum', {
+      value: {
+        isMetaMask: true,
+        selectedAddress: '0x123',
+        isConnected: () => true,
+        _metamask: {
+          isUnlocked: () => Promise.resolve(true),
+        },
+      },
+      writable: true,
+    })
+    const result = await isWalletUnlocked(WalletNames.METAMASK)
+    expect(result).toBe(true)
+  })
+
+  it('should check if WalletConnect is unlocked', async () => {
+    window.localStorage.setItem('walletconnect', 'true')
+
+    expect(await isWalletUnlocked(WalletNames.WALLET_CONNECT)).toBe(true)
+    expect(await isWalletUnlocked(WalletNames.WALLET_CONNECT_V2)).toBe(true)
+
+    window.localStorage.removeItem('walletconnect')
+
+    expect(await isWalletUnlocked(WalletNames.WALLET_CONNECT)).toBe(false)
+    expect(await isWalletUnlocked(WalletNames.WALLET_CONNECT_V2)).toBe(false)
+  })
+
+  it('should return false for unhandled wallets', async () => {
+    expect(await isWalletUnlocked('PINEAPPLE')).toBe(false)
+  })
+})

src/utils/wallets.ts

@@ -32,17 +32,13 @@ export const WalletNames = {
 export const isWalletUnlocked = async (walletName: string): Promise<boolean> => {
   if (typeof window === 'undefined') return false
 
-  if (window.ethereum?.isConnected?.()) {
-    return true
-  }
-
   // Only MetaMask exposes a method to check if the wallet is unlocked
   if (walletName === WalletNames.METAMASK) {
-    return window.ethereum?._metamask?.isUnlocked?.() || false
+    return (await window.ethereum?._metamask?.isUnlocked?.()) || false
   }
 
   // Wallet connect creates a localStorage entry when connected and removes it when disconnected
-  if (walletName === WalletNames.WALLET_CONNECT) {
+  if (walletName === WalletNames.WALLET_CONNECT || walletName === WalletNames.WALLET_CONNECT_V2) {
     return window.localStorage.getItem('walletconnect') !== null
   }