fix: update metamask sdk and fix metadata handling for empty URL (#4456)

* fix: update metamask sdk and fix metadata handling for empty URL

* chore: tweaks

---------

Co-authored-by: Tom Meagher <tom@meagher.co>

Author: EdouardBougon

.changeset/warm-melons-joke.md

@@ -0,0 +1,5 @@
+---
+"@wagmi/connectors": patch
+---
+
+Bumped MetaMask SDK and fixed internal metadata handling.

packages/connectors/package.json

@@ -46,7 +46,7 @@
   },
   "dependencies": {
     "@coinbase/wallet-sdk": "4.2.3",
-    "@metamask/sdk": "0.31.1",
+    "@metamask/sdk": "0.31.2",
     "@safe-global/safe-apps-provider": "0.18.4",
     "@safe-global/safe-apps-sdk": "9.1.0",
     "@walletconnect/ethereum-provider": "2.17.0",

packages/connectors/src/metaMask.ts

@@ -260,11 +260,15 @@ export function metaMask(parameters: MetaMaskParameters = {}) {
           // Workaround cast since MetaMask SDK does not support `'exactOptionalPropertyTypes'`
           ...(parameters as RemoveUndefined<typeof parameters>),
           readonlyRPCMap,
-          dappMetadata:
-            parameters.dappMetadata ??
-            (typeof window !== 'undefined'
-              ? { url: window.location.origin }
-              : { name: 'wagmi', url: 'https://wagmi.sh' }),
+          dappMetadata: {
+            ...parameters.dappMetadata,
+            name: parameters.dappMetadata?.name ?? 'wagmi',
+            url:
+              parameters.dappMetadata?.url ??
+              (typeof window !== 'undefined'
+                ? window.location.origin
+                : 'https://wagmi.sh'),
+          },
           useDeeplink: parameters.useDeeplink ?? true,
         })
         const result = await sdk.init()