fix: zksync gas estimation from safe account

package.json

@@ -31,8 +31,8 @@
     "test:coverage": "jest --coverage"
   },
   "dependencies": {
-    "@aave/contract-helpers": "1.32.1",
-    "@aave/math-utils": "1.32.1",
+    "@aave/contract-helpers": "1.32.2-51a91f59a8ffb3609b3e1d811454d42b056e09c0.0+0e8fe1e",
+    "@aave/math-utils": "1.32.2-51a91f59a8ffb3609b3e1d811454d42b056e09c0.0+0e8fe1e",
     "@bgd-labs/aave-address-book": "4.10.0",
     "@emotion/cache": "11.10.3",
     "@emotion/react": "11.10.4",
@@ -151,5 +151,6 @@
     "budget": null,
     "budgetPercentIncreaseRed": 20,
     "showDetails": true
-  }
+  },
+  "packageManager": "[email protected]+sha512.a6b2f7906b721bba3d67d4aff083df04dad64c399707841b7acf00f6b133b7ac24255f2652fa22ae3534329dc6180534e98d17432037ff6fd140556e2bb3137e"
 }

src/hooks/useIsContractAddress.ts

@@ -9,7 +9,7 @@ export const useIsContractAddress = (address: string, chainId?: number) => {
   return useQuery({
     queryFn: () => provider.getCode(address),
     queryKey: ['isContractAddress', address],
-    enabled: true,
+    enabled: address !== '',
     staleTime: Infinity,
     select: (data) => data !== '0x',
   });

src/libs/web3-data-provider/Web3Provider.tsx

@@ -3,11 +3,13 @@ import { SignatureLike } from '@ethersproject/bytes';
 import { JsonRpcProvider, TransactionResponse } from '@ethersproject/providers';
 import { BigNumber, PopulatedTransaction, utils } from 'ethers';
 import React, { ReactElement, useEffect, useState } from 'react';
+import { useIsContractAddress } from 'src/hooks/useIsContractAddress';
 import { useRootStore } from 'src/store/root';
 import { wagmiConfig } from 'src/ui-config/wagmiConfig';
 import { hexToAscii } from 'src/utils/utils';
 import { UserRejectedRequestError } from 'viem';
 import { useAccount, useConnect, useSwitchChain, useWatchAsset } from 'wagmi';
+import { useShallow } from 'zustand/shallow';
 
 import { Web3Context } from '../hooks/useWeb3Context';
 import { getEthersProvider } from './adapters/EthersAdapter';
@@ -47,7 +49,9 @@ export const Web3ContextProvider: React.FC<{ children: ReactElement }> = ({ chil
 
   const [readOnlyModeAddress, setReadOnlyModeAddress] = useState<string | undefined>();
   const [switchNetworkError, setSwitchNetworkError] = useState<Error>();
-  const setAccount = useRootStore((store) => store.setAccount);
+  const [setAccount, setConnectedAccountIsContract] = useRootStore(
+    useShallow((store) => [store.setAccount, store.setConnectedAccountIsContract])
+  );
 
   const account = address;
   const readOnlyMode = utils.isAddress(readOnlyModeAddress || '');
@@ -179,6 +183,21 @@ export const Web3ContextProvider: React.FC<{ children: ReactElement }> = ({ chil
     }
   }, [readOnlyModeAddress, setAccount]);
 
+  console.log('account', account);
+  const { data: isContractAddress } = useIsContractAddress(account || '', chainId);
+
+  useEffect(() => {
+    if (!account) {
+      setConnectedAccountIsContract(false);
+      return;
+    }
+
+    console.log('isContractAddress', isContractAddress);
+    if (isContractAddress) {
+      setConnectedAccountIsContract(true);
+    }
+  }, [isContractAddress, setConnectedAccountIsContract, account]);
+
   return (
     <Web3Context.Provider
       value={{

src/store/poolSlice.ts

@@ -3,6 +3,7 @@ import {
   ApproveDelegationType,
   ApproveType,
   BaseDebtToken,
+  ChainId,
   DebtSwitchAdapterService,
   ERC20_2612Service,
   ERC20Service,
@@ -701,7 +702,11 @@ export const createPoolSlice: StateCreator<
     poolComputed: {
       get minRemainingBaseTokenBalance() {
         if (!get()) return '0.001';
-        const { currentNetworkConfig, currentChainId } = { ...get() };
+
+        const { currentNetworkConfig, currentChainId, connectedAccountIsContract } = { ...get() };
+
+        if (connectedAccountIsContract) return '0';
+
         const chainId = currentNetworkConfig.underlyingChainId || currentChainId;
         const min = minBaseTokenRemainingByNetwork[chainId];
         return min || '0.001';
@@ -766,7 +771,21 @@ export const createPoolSlice: StateCreator<
       return JSON.stringify(typeData);
     },
     estimateGasLimit: async (tx: PopulatedTransaction, chainId?: number) => {
-      const provider = get().jsonRpcProvider(chainId);
+      const { currentChainId, connectedAccountIsContract, jsonRpcProvider } = get();
+
+      const effectiveChainId = chainId ?? currentChainId;
+
+      /**
+       *  Trying to estimate gas on zkSync when connected with a smart contract address
+       *  will fail. In that case, we'll just return the default value for all transactions.
+       *
+       *  See here for more details: https://github.com/zkSync-Community-Hub/zksync-developers/discussions/144
+       */
+      if (effectiveChainId === ChainId.zksync && connectedAccountIsContract) {
+        return tx;
+      }
+
+      const provider = jsonRpcProvider(chainId);
       const defaultGasLimit: BigNumber = tx.gasLimit ? tx.gasLimit : BigNumber.from('0');
       delete tx.gasLimit;
       let estimatedGas = await provider.estimateGas(tx);

src/store/walletSlice.ts

@@ -17,6 +17,8 @@ export interface WalletSlice {
   walletApprovalMethodPreference: ApprovalMethod;
   setWalletApprovalMethodPreference: (method: ApprovalMethod) => void;
   refreshWalletApprovalMethod: () => void;
+  connectedAccountIsContract: boolean;
+  setConnectedAccountIsContract: (isContract: boolean) => void;
 }
 
 const getWalletPreferences = () => {
@@ -73,4 +75,8 @@ export const createWalletSlice: StateCreator<
       }));
     }
   },
+  connectedAccountIsContract: false,
+  setConnectedAccountIsContract(isContract) {
+    set({ connectedAccountIsContract: isContract });
+  },
 });

yarn.lock

@@ -2,17 +2,17 @@
 # yarn lockfile v1
 
 
-"@aave/[email protected].1":
-  version "1.32.1"
-  resolved "https://registry.yarnpkg.com/@aave/contract-helpers/-/contract-helpers-1.32.1.tgz#ad3e216118f282f450b7ab160bd270930ccf5fa4"
-  integrity sha512-iYkopRnzbfnW7Pxa/qNmqVJVSCWKqF14FNK9SKpmLAkndH8P0UZqSkIdcnvW/ekNx6cyGIc506gKc00j5OnegQ==
+"@aave/[email protected].2-51a91f59a8ffb3609b3e1d811454d42b056e09c0.0+0e8fe1e":
+  version "1.32.2-51a91f59a8ffb3609b3e1d811454d42b056e09c0.0"
+  resolved "https://registry.yarnpkg.com/@aave/contract-helpers/-/contract-helpers-1.32.2-51a91f59a8ffb3609b3e1d811454d42b056e09c0.0.tgz#763822a3e0f7bd0df87684c87135f7401b287db5"
+  integrity sha512-82MysayYNdHv4XCTqGZJTPTaf59cc0MN95b4CYMrDRnwnQHdpXPf59kZcVVfgK04X9j96uaXWXlA4IpPwcCvuA==
   dependencies:
     isomorphic-unfetch "^3.1.0"
 
-"@aave/[email protected].1":
-  version "1.32.1"
-  resolved "https://registry.yarnpkg.com/@aave/math-utils/-/math-utils-1.32.1.tgz#9cd4bb9343f2fabb078b6ca61b5845c7044fe643"
-  integrity sha512-uLjwGNEOilWWguW7CX6Dwsk9uP4aT9HmOCzXpHqDrPbR2oGSLV8BlvHWx/8wzY9Cdt5Jr4ofP0vm/Xs4lrgDNg==
+"@aave/[email protected].2-51a91f59a8ffb3609b3e1d811454d42b056e09c0.0+0e8fe1e":
+  version "1.32.2-51a91f59a8ffb3609b3e1d811454d42b056e09c0.0"
+  resolved "https://registry.yarnpkg.com/@aave/math-utils/-/math-utils-1.32.2-51a91f59a8ffb3609b3e1d811454d42b056e09c0.0.tgz#e1f2bd05df263def0278eb2ceb4c356c69e7d7fe"
+  integrity sha512-ZIZCAvmbtxdiBuStCgZEUuEki+CAxkTc6ivpKQByV5/jaIKWLpy3si7ksw7/UKfL5i3nxmi2wUiCKMqcam6Ymw==
 
 "@adobe/css-tools@^4.0.1":
   version "4.4.1"