wallet-connect: fix "ChainId not found"

update namespace handling to conform to:
https://docs.walletconnect.com/web3wallet/namespaces#case-no-1---recommended-default-for-all-apps

Until now we were relying only on requiredNamespace to match the
pairing request to the correct chain and account, however, for some
reason, requiredNamespace is not actually required, can be empty and
the necessary namespace details are instead defined in optionalNamespace

Author: Tomasvrba

CHANGELOG.md

@@ -9,6 +9,8 @@
 - Fix BitBoxApp crash on GrapheneOS and other phones without Google Play Services when scanning QR codes.
 - Add DMG installer for macOS
 - Use mempool.space as preferred fee estimation source for BTC
+- Fix Wallet Connect issue where account unspecified by the connecting dapp caused a UI crash
+- Fix Wallet Connect issue with required/optionalNamespace and handling all possible namespace definitions
 
 ## 4.42.0
 - Preselect backup when there's only one backup available

frontends/web/src/locales/en/app.json

@@ -1753,7 +1753,8 @@
     "connect": {
       "button": "Connect",
       "dappLabel": "Enter URI address of dapp",
-      "invalidPairingUri": "Invalid pairing uri"
+      "invalidPairingUri": "Invalid pairing uri",
+      "missingNamespace": "Connecting app request did not define necessary namespace"
     },
     "dashboard": {
       "allSessions": "All sessions",

frontends/web/src/routes/account/walletconnect/components/incoming-pairing/incoming-pairing.tsx

@@ -61,12 +61,18 @@ export const WCIncomingPairing = ({
     setPairingLoading(true);
     try {
       const { id, params } = currentProposal;
-      const { requiredNamespaces } = params;
-      const eipList = Object.values(requiredNamespaces);
+      const { requiredNamespaces, optionalNamespaces } = params;
+      const eipList = Object.keys(requiredNamespaces).length !== 0 ? Object.values(requiredNamespaces) : Object.values(optionalNamespaces);
+      if (!eipList) {
+        alertUser(`${t('walletConnect.connect.missingNamespace')}`);
+        await handleRejectPairing();
+        setPairingLoading(false);
+        return;
+      }
       const accounts = eipList.flatMap(eip => eip.chains?.map(chain => `${chain}:${receiveAddress}`) || []);
-      // For supported chains, use an intersection of supported chains and required chains
+      // For supported chains, use an intersection of supported chains and required chains, default to mainnet if no chains present
       const chains: string[] = eipList.flatMap(proposal =>
-        proposal.chains ? proposal.chains.filter(chain => Object.keys(SUPPORTED_CHAINS).includes(chain)) : []
+        proposal.chains ? proposal.chains.filter(chain => Object.keys(SUPPORTED_CHAINS).includes(chain)) : ['eip155:1']
       );
 
       // buildApprovedNamespaces is a