feat(berajs): add gas limit estimation to prevent hitting gas limits …

packages/berajs/src/hooks/modules/dex/useCreatePool.ts

@@ -20,9 +20,9 @@ import { balancerPoolCreationHelperAbi } from "~/abi";
 import { ADDRESS_ZERO } from "~/config";
 import { IContractWrite } from "~/hooks/useContractWrite";
 import { Token, TokenInput } from "~/types";
+import { DEFAULT_METAMASK_GAS_LIMIT } from "~/utils";
 
 const DEFAULT_WEIGHTS_DUPLICATION_THRESHOLD = 0.005;
-const DEFAULT_POOL_CREATE_GAS_LIMIT = 7920027n; // NOTE: this is the metamask gas limit, in experiments we find we can easily use 75% of this.
 
 interface UseCreatePoolProps {
   poolCreateTokens: Token[];
@@ -56,7 +56,7 @@ const createStablePoolArgs = (
   poolSymbol: string,
   owner: string,
   amplification: number,
-  gasLimit: bigint = DEFAULT_POOL_CREATE_GAS_LIMIT,
+  gasLimit: bigint = DEFAULT_METAMASK_GAS_LIMIT,
 ) => {
   // Map and sort pool creation token addresses NOTE: we should never see BERA in this array.
   const sortedPoolCreateAddresses = poolCreateTokens
@@ -135,7 +135,7 @@ const createWeightedPoolArgs = (
   poolName: string,
   poolSymbol: string,
   owner: string,
-  gasLimit: bigint = DEFAULT_POOL_CREATE_GAS_LIMIT,
+  gasLimit: bigint = DEFAULT_METAMASK_GAS_LIMIT,
 ) => {
   // When joining weighted pools with BERA, we allow it as a joinPoolToken but never as a createPoolToken.
   const createPoolTokens: string[] = [];

packages/berajs/src/hooks/useContractWrite/useBeraContractWrite.ts

@@ -10,6 +10,7 @@ import {
   type IUseContractWriteArgs,
   type useContractWriteApi,
 } from "./types";
+import { DEFAULT_METAMASK_GAS_LIMIT } from "~/utils";
 
 const increaseByPercentage = (value: bigint, percentage: number) => {
   return value + (value * BigInt(percentage)) / BigInt(100);
@@ -45,7 +46,8 @@ const useBeraContractWrite = ({
       params,
       value = 0n,
       data,
-      gasLimit = 2000000n,
+      gasLimit = DEFAULT_METAMASK_GAS_LIMIT,
+      ...rest
     }: IContractWrite): Promise<void> => {
       dispatch({ type: ActionEnum.LOADING });
       onLoading?.();
@@ -60,18 +62,31 @@ const useBeraContractWrite = ({
             gas: gasLimit,
           });
         } else {
+          // Run simulation and gas estimation in parallel
           // TODO: figure out clean way to early detect errors and effectively show them on the UI
-          const { request } = await publicClient.simulateContract({
-            address: address,
-            abi: abi,
-            functionName: functionName,
-            args: params,
-            value: value,
-            account: account,
-          });
+          const [{ request }, wagmiPubEstimateContractGas] = await Promise.all([
+            publicClient.simulateContract({
+              address: address,
+              abi: abi,
+              functionName: functionName,
+              args: params,
+              value: value,
+              account: account,
+            }),
+            publicClient.estimateContractGas({
+              address: address,
+              abi: abi,
+              functionName: functionName,
+              args: params,
+              value: value,
+              account: account,
+            }),
+          ]);
+
           receipt = await writeContractAsync({
             ...request,
-            gas: gasLimit ?? request.gas,
+            gas:
+              increaseByPercentage(wagmiPubEstimateContractGas, 10) ?? gasLimit,
           });
         }
 

packages/berajs/src/utils/const.ts

@@ -0,0 +1 @@
+export const DEFAULT_METAMASK_GAS_LIMIT = 7920027n; // NOTE: this is the metamask gas limit, https://github.com/MetaMask/metamask-extension/blob/06cf7459b963e26d0ae60312d4f9a342d07ab89d/ui/pages/confirmations/send/send.constants.js#L8

packages/berajs/src/utils/index.ts

@@ -13,3 +13,4 @@ export * from "./formatNumber";
 export * from "./formatTimestamps";
 export * from "./tokenWrapping";
 export * from "./errorMessages";
+export * from "./const";