fix: wrap Eip1193Bridge to fix outstanding bugs (#3355)

* fix: wrap Eip1193Bridge to fix outstanding bugs

* revert: widget height change

src/lib/components/Widget.tsx

 import { Provider as EthersProvider } from '@ethersproject/abstract-provider'
-import { Signer as EthersSigner } from '@ethersproject/abstract-signer'
 import { Provider as Eip1193Provider } from '@web3-react/types'
 import { DEFAULT_LOCALE, SupportedLocale } from 'constants/locales'
 import { Provider as AtomProvider } from 'jotai'
@@ -94,7 +93,7 @@ function Updaters() {
 export type WidgetProps = {
   theme?: Theme
   locale?: SupportedLocale
-  provider?: Eip1193Provider | EthersProvider | { provider: EthersProvider; signer: EthersSigner }
+  provider?: Eip1193Provider | EthersProvider
   jsonRpcEndpoint?: string
   width?: string | number
   dialog?: HTMLElement | null

src/lib/hooks/useEip1193Provider.ts

 import { Provider as EthersProvider } from '@ethersproject/abstract-provider'
-import { Signer as EthersSigner } from '@ethersproject/abstract-signer'
 import { VoidSigner } from '@ethersproject/abstract-signer'
 import { Eip1193Bridge as ExperimentalEip1193Bridge } from '@ethersproject/experimental'
-import { JsonRpcProvider } from '@ethersproject/providers'
+import { JsonRpcProvider, JsonRpcSigner } from '@ethersproject/providers'
 import { Provider as Eip1193Provider } from '@web3-react/types'
 import { ZERO_ADDRESS } from 'constants/misc'
 import { useMemo } from 'react'
@@ -17,32 +16,42 @@ class Eip1193Bridge extends ExperimentalEip1193Bridge {
         const result = await this.provider.getNetwork()
         return '0x' + result.chainId.toString(16)
       }
+      case 'eth_sendTransaction': {
+        if (!this.signer) break
+
+        // TODO(zzmp): JsonRpcProvider filters from/gas fields from the params.
+        const req = JsonRpcProvider.hexlifyTransaction(params?.[0], { from: true, gas: true })
+        const tx = await this.signer.sendTransaction(req)
+        return tx.hash
+      }
       default:
         return super.send(method, params)
     }
   }
 }
 
+interface EthersSigningProvider extends EthersProvider {
+  getSigner(): JsonRpcSigner
+}
+
 export default function useEip1193Provider(
-  provider?: Eip1193Provider | EthersProvider | JsonRpcProvider | { provider: EthersProvider; signer: EthersSigner }
+  provider?: Eip1193Provider | EthersSigningProvider | EthersProvider
 ): Eip1193Provider | undefined {
   return useMemo(() => {
     if (provider) {
-      if (provider instanceof EthersProvider) {
-        // A JsonRpcProvider includes its own Signer, otherwise use a VoidSigner.
+      if (EthersProvider.isProvider(provider)) {
         const signer = 'getSigner' in provider ? provider.getSigner() : null ?? voidSigner
         return new Eip1193Bridge(signer, provider)
+      } else if (EthersProvider.isProvider((provider as ExperimentalEip1193Bridge).provider)) {
+        /*
+         * Direct users to use our own wrapper to avoid any pitfalls:
+         * - Eip1193Bridge is experimental
+         * - signer is not straightforward
+         * - bugs out if chainId>8
+         */
+        throw new Error('Eip1193Bridge is experimental: pass your ethers Provider directly')
       }
-
-      if ('provider' in provider && 'signer' in provider) {
-        return new Eip1193Bridge(provider.signer, provider.provider)
     }
-
-      // See https://github.com/ethereum/EIPs/blob/master/EIPS/eip-1193.md.
-      if ('request' in provider && 'on' in provider && 'removeListener' in provider) {
     return provider
-      }
-    }
-    return
   }, [provider])
 }