fix: v5 connectors not appearing on solana (#2979)

packages/base/adapters/solana/web3js/client.ts

@@ -715,7 +715,11 @@ export class SolanaWeb3JsClient implements ChainAdapter<SolStoreUtilState, CaipN
       type: provider.type,
       imageUrl: provider.icon,
       name: provider.name,
-      provider,
+      /**
+       * When the provider is different from 'AUTH', we don't need to pass it to the connector.
+       * This avoids issues with the valtio proxy and non-serializable state and follows same logic from other clients.
+       */
+      provider: provider.type === 'AUTH' ? provider : undefined,
       chain: CommonConstantsUtil.CHAIN.SOLANA,
       ...provider.auth
     }))

packages/core/src/controllers/ConnectorController.ts

@@ -32,7 +32,29 @@ export const ConnectorController = {
 
   setConnectors(connectors: ConnectorControllerState['connectors']) {
     connectors.forEach(this.syncIfAuthConnector)
-    state.unMergedConnectors = [...state.unMergedConnectors, ...connectors]
+    state.unMergedConnectors = [...state.unMergedConnectors, ...connectors].filter(connector => {
+      /**
+       * This is a fix for non-serializable objects that may prevent all the connectors in the list from being displayed
+       * Check more about this issue on https://valtio.dev/docs/api/basic/proxy#Gotchas
+       */
+      try {
+        const canProxyConnector = Boolean(proxy(connector))
+
+        if (!canProxyConnector) {
+          throw new Error('Connector is not available')
+        }
+
+        return true
+      } catch (error) {
+        // eslint-disable-next-line no-console
+        console.error('ConnectorController.setConnectors: Not possible to add connector', {
+          connector,
+          error
+        })
+
+        return false
+      }
+    })
     state.connectors = this.mergeMultiChainConnectors(state.unMergedConnectors)
   },