refactor: remove provider from wallet info

Author: kernelwhisperer

libs/wallet/package.json

@@ -41,7 +41,6 @@
     "@cowprotocol/iframe-transport": "workspace:*",
     "@cowprotocol/types": "workspace:*",
     "@cowprotocol/ui": "workspace:*",
-    "@cowprotocol/wallet-provider": "workspace:*",
     "@ethereumjs/util": "10.1.0",
     "@metamask/jazzicon": "2.0.0",
     "@metamask/sdk": "0.31.4",

libs/wallet/src/api/types.ts

@@ -1,8 +1,6 @@
 import { SupportedChainId } from '@cowprotocol/cow-sdk'
-import { WidgetEthereumProvider } from '@cowprotocol/iframe-transport'
 import type { SafeInfoResponse } from '@safe-global/api-kit'
 
-import { EIP1193Provider, PublicClient } from 'viem'
 import { Connector as WagmiConnector } from 'wagmi'
 import { injected, walletConnect, coinbaseWallet, safe } from 'wagmi/connectors'
 
@@ -20,8 +18,6 @@ export interface WalletInfo {
   account?: string
   active?: boolean
   connector?: WagmiConnector
-  provider?: EIP1193Provider | WidgetEthereumProvider | PublicClient
-  isConnectionRestoring?: boolean
 }
 
 export interface WalletDetails {

libs/wallet/src/wagmi/updater.ts

@@ -5,7 +5,6 @@ import { getCurrentChainIdFromUrl, getRawCurrentChainIdFromUrl, logSafeApi } fro
 import { getSafeInfo, normalizeSafeError, SAFE_RATE_LIMIT_MSG } from '@cowprotocol/core'
 import { SupportedChainId } from '@cowprotocol/cow-sdk'
 import { AccountType } from '@cowprotocol/types'
-import { useWalletProvider } from '@cowprotocol/wallet-provider'
 import type { SafeInfoResponse } from '@safe-global/api-kit'
 import type { SafeInfoExtended } from '@safe-global/safe-apps-sdk'
 
@@ -46,8 +45,7 @@ function useBrowserUrlKey(): string {
 function useWalletInfo(): WalletInfo {
   // TODO: Replace urlKey with locationNetworkAtom, which will also trigger the useMemo below less often.
   const urlKey = useBrowserUrlKey()
-  const { address, chainId, isConnected, status, connector } = useAccountState()
-  const isConnectionRestoring = status === 'reconnecting'
+  const { address, chainId, isConnected, connector } = useAccountState()
   const isChainIdUnsupported = !!chainId && !(chainId in SupportedChainId)
   const [lastStableChainId, setLastStableChainId] = useState<SupportedChainId | undefined>(undefined)
   const [lastResolvedChainId, setLastResolvedChainId] = useState<SupportedChainId>(() => getCurrentChainIdFromUrl())
@@ -83,19 +81,8 @@ function useWalletInfo(): WalletInfo {
       active: isConnected,
       account: address,
       connector,
-      isConnectionRestoring,
     }
-  }, [
-    address,
-    chainId,
-    isConnected,
-    connector,
-    isChainIdUnsupported,
-    isConnectionRestoring,
-    lastStableChainId,
-    lastResolvedChainId,
-    urlKey,
-  ])
+  }, [address, chainId, isConnected, connector, isChainIdUnsupported, lastStableChainId, lastResolvedChainId, urlKey])
 
   useEffect(() => {
     setLastResolvedChainId(walletInfo.chainId)
@@ -140,7 +127,7 @@ let shortSafeInfoInterval: ReturnType<typeof setInterval> | null = null
 let longSafeInfoInterval: ReturnType<typeof setInterval> | null = null
 
 export function WalletUpdater(): null {
-  const { chainId, active, account, connector, isConnectionRestoring } = useWalletInfo()
+  const { chainId, active, account, connector } = useWalletInfo()
 
   const walletDetails = useWalletDetails(account)
   const gnosisSafeInfo = useSafeInfo()
@@ -149,11 +136,9 @@ export function WalletUpdater(): null {
   const setWalletDetails = useSetAtom(walletDetailsAtom)
   const setGnosisSafeInfo = useSetAtom(gnosisSafeInfoAtom)
 
-  const provider = useWalletProvider()
-
   useEffect(() => {
-    setWalletInfo({ chainId, active, account, connector, isConnectionRestoring, provider })
-  }, [chainId, active, account, connector, isConnectionRestoring, provider, setWalletInfo])
+    setWalletInfo({ chainId, active, account, connector })
+  }, [chainId, active, account, connector, setWalletInfo])
 
   useEffect(() => {
     const walletType = getWalletType({ gnosisSafeInfo, isSmartContractWallet: walletDetails.isSmartContractWallet })