fix: useConnect with chainId error bubbling (#3384)

* fix: useConnect with chainId error bubbling

* chore: changeset

* chore: remove log

.changeset/rich-forks-enjoy.md

@@ -0,0 +1,8 @@
+---
+"@wagmi/connectors": patch
+"@wagmi/core": patch
+"create-wagmi": patch
+"wagmi": patch
+---
+
+Fixed connectors not bubbling error when connecting with `chainId` and subsequent user rejection.

packages/connectors/src/coinbaseWallet.ts

@@ -71,9 +71,10 @@ export function coinbaseWallet(parameters: CoinbaseWalletParameters) {
         // Switch to chain if provided
         let currentChainId = await this.getChainId()
         if (chainId && currentChainId !== chainId) {
-          const chain = await this.switchChain!({ chainId }).catch(() => ({
-            id: currentChainId,
-          }))
+          const chain = await this.switchChain!({ chainId }).catch((error) => {
+            if (error.code === UserRejectedRequestError.code) throw error
+            return { id: currentChainId }
+          })
           currentChainId = chain?.id ?? currentChainId
         }
 

packages/connectors/src/metaMask.ts

@@ -114,9 +114,10 @@ export function metaMask(parameters: MetaMaskParameters = {}) {
         // Switch to chain if provided
         let currentChainId = await this.getChainId()
         if (chainId && currentChainId !== chainId) {
-          const chain = await this.switchChain?.({ chainId }).catch(() => ({
-            id: currentChainId,
-          }))
+          const chain = await this.switchChain!({ chainId }).catch((error) => {
+            if (error.code === UserRejectedRequestError.code) throw error
+            return { id: currentChainId }
+          })
           currentChainId = chain?.id ?? currentChainId
         }
 

packages/core/src/connectors/injected.ts

@@ -185,9 +185,10 @@ export function injected(parameters: InjectedParameters = {}) {
         // Switch to chain if provided
         let currentChainId = await this.getChainId()
         if (chainId && currentChainId !== chainId) {
-          const chain = await this.switchChain!({ chainId }).catch(() => ({
-            id: currentChainId,
-          }))
+          const chain = await this.switchChain!({ chainId }).catch((error) => {
+            if (error.code === UserRejectedRequestError.code) throw error
+            return { id: currentChainId }
+          })
           currentChainId = chain?.id ?? currentChainId
         }
 

playgrounds/vite-react/src/wagmi.ts

@@ -1,7 +1,7 @@
 import { del, get, set } from 'idb-keyval'
 import { http, createConfig } from 'wagmi'
 import { celo, mainnet, optimism, sepolia } from 'wagmi/chains'
-import { coinbaseWallet, metaMask, safe, walletConnect } from 'wagmi/connectors'
+import { coinbaseWallet, walletConnect } from 'wagmi/connectors'
 
 // biome-ignore lint/correctness/noUnusedVariables: <explanation>
 const indexedDBStorage = {
@@ -23,8 +23,6 @@ export const config = createConfig({
       projectId: import.meta.env.VITE_WC_PROJECT_ID,
     }),
     coinbaseWallet({ appName: 'Vite React Playground', darkMode: true }),
-    safe({ debug: true, shimDisconnect: true }),
-    metaMask(),
   ],
   transports: {
     [mainnet.id]: http(