[SDK] Fix: Only request specified wallet connect chains (#4832)

Falls in line with the WC spec and appeases some picky wallets

<!-- start pr-codex -->

---

## PR-Codex overview
This PR focuses on fixing wallet connection issues in the `thirdweb` package, particularly for wallets with limited chain support. It improves handling of optional chains and modifies how chains are requested.

### Detailed summary
- Updated handling of `optionalChains` to default to `[1]` if empty.
- Changed `chains` to be `undefined` if `chainToRequest` is not provided.
- Modified `setRequestedChainsIds` function to accept `chains` as `undefined`.
- Simplified logic for `chainsToRequest` to use `undefined` when no chains are available.

> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`

<!-- end pr-codex -->

Author: gregfromstl

.changeset/chilly-colts-beam.md

@@ -0,0 +1,5 @@
+---
+"thirdweb": patch
+---
+
+Fixes WC connections with wallets that have limited chain support

packages/thirdweb/src/wallets/wallet-connect/controller.ts

@@ -131,11 +131,7 @@ export async function connectWC(
         ? { pairingTopic: wcOptions?.pairingTopic }
         : {}),
       optionalChains: chainsToRequest,
-      chains: chainToRequest
-        ? [chainToRequest.id]
-        : chainsToRequest.length > 0
-          ? [chainsToRequest[0]]
-          : [1],
+      chains: chainToRequest ? [chainToRequest.id] : undefined,
       rpcMap: rpcMap,
     });
   }
@@ -265,12 +261,8 @@ async function initProvider(
     projectId: wcOptions?.projectId || DEFAULT_PROJECT_ID,
     optionalMethods: OPTIONAL_METHODS,
     optionalEvents: OPTIONAL_EVENTS,
-    optionalChains: chainsToRequest,
-    chains: chainToRequest
-      ? [chainToRequest.id]
-      : chainsToRequest.length > 0
-        ? [chainsToRequest[0]]
-        : [1],
+    optionalChains: chainsToRequest || [1],
+    chains: chainToRequest ? [chainToRequest.id] : undefined,
     metadata: {
       name: wcOptions?.appMetadata?.name || getDefaultAppMetadata().name,
       description:
@@ -509,7 +501,10 @@ async function switchChainWC(
  * Set the requested chains to the storage.
  * @internal
  */
-function setRequestedChainsIds(chains: number[], storage: AsyncStorage) {
+function setRequestedChainsIds(
+  chains: number[] | undefined,
+  storage: AsyncStorage,
+) {
   storage?.setItem(storageKeys.requestedChains, JSON.stringify(chains));
 }
 
@@ -550,13 +545,9 @@ function getChainsToRequest(options: {
     });
   }
 
-  const optionalChainIds = optionalChains.map((c) => c.id) || [];
-
-  const chainsToRequest: ArrayOneOrMore<number> = options.chain
-    ? [options.chain.id, ...optionalChainIds]
-    : optionalChainIds.length > 0
-      ? (optionalChainIds as ArrayOneOrMore<number>)
-      : [1];
+  const chainsToRequest: ArrayOneOrMore<number> | undefined = options.chain
+    ? [options.chain.id]
+    : undefined;
 
   if (!options.chain && optionalChains.length === 0) {
     rpcMap[1] = getCachedChain(1).rpc;