fix: #4118

.changeset/serious-pots-rescue.md

@@ -0,0 +1,5 @@
+---
+"@wagmi/connectors": patch
+---
+
+Fixed `metaMask` connector switch chain issue.

packages/connectors/package.json

@@ -46,7 +46,7 @@
   },
   "dependencies": {
     "@coinbase/wallet-sdk": "4.0.4",
-    "@metamask/sdk": "0.26.4",
+    "@metamask/sdk": "0.26.5",
     "@safe-global/safe-apps-provider": "0.18.3",
     "@safe-global/safe-apps-sdk": "9.1.0",
     "@walletconnect/ethereum-provider": "2.13.0",

packages/connectors/src/metaMask.ts

@@ -218,10 +218,21 @@ export function metaMask(parameters: MetaMaskParameters = {}) {
 
       try {
         await Promise.all([
-          provider.request({
-            method: 'wallet_switchEthereumChain',
-            params: [{ chainId: numberToHex(chainId) }],
-          }),
+          provider
+            .request({
+              method: 'wallet_switchEthereumChain',
+              params: [{ chainId: numberToHex(chainId) }],
+            })
+            // During `'wallet_switchEthereumChain'`, MetaMask makes a `'net_version'` RPC call to the target chain.
+            // If this request fails, MetaMask does not emit the `'chainChanged'` event, but will still switch the chain.
+            // To counter this behavior, we request and emit the current chain ID to confirm the chain switch either via
+            // this callback or an externally emitted `'chainChanged'` event.
+            // https://github.com/MetaMask/metamask-extension/issues/24247
+            .then(async () => {
+              const currentChainId = await this.getChainId()
+              if (currentChainId === chainId)
+                config.emitter.emit('change', { chainId })
+            }),
           new Promise<void>((resolve) =>
             config.emitter.once('change', ({ chainId: currentChainId }) => {
               if (currentChainId === chainId) resolve()