chore: increase extra gas for hook to 200k and for proxy to 400k

Author: cowdan

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cowprotocol/cow-sdk",
-  "version": "6.3.2-RC.0",
+  "version": "6.3.2-RC.1",
   "license": "(MIT OR Apache-2.0)",
   "files": [
     "/dist"

src/bridging/PROVIDER_README.md

@@ -381,7 +381,7 @@ import { getGasLimitEstimationForHook } from '../utils/getGasLimitEstimationForH
 
 export class YourBridgeProvider implements BridgeProvider<YourBridgeQuoteResult> {
   async getGasLimitEstimationForHook(
-    request: Omit<QuoteBridgeRequest, 'amount'> & { extraGas?: number },
+    request: Omit<QuoteBridgeRequest, 'amount'> & { extraGas?: number; extraGasProxyCreation?: number },
   ): Promise<number> {
     // Use utility function or implement custom gas estimation
     return getGasLimitEstimationForHook(

src/bridging/const.ts

@@ -3,7 +3,8 @@ import { RAW_FILES_PATH } from '../common/consts/path'
 export const RAW_PROVIDERS_FILES_PATH = `${RAW_FILES_PATH}/src/bridging/providers`
 // Based on https://dashboard.tenderly.co/shoom/project/simulator/a5e29dac-d0f2-407f-9e3d-d1b916da595b
 export const DEFAULT_GAS_COST_FOR_HOOK_ESTIMATION = 240_000
-export const DEFAULT_EXTRA_GAS_FOR_HOOK_ESTIMATION = 150_000
+export const DEFAULT_EXTRA_GAS_FOR_HOOK_ESTIMATION = 200_000
 // Based on https://dashboard.tenderly.co/shoom/project/tx/0x2971aee9aa7237d24b254da8ccd4345ff77410c8829c8e825e9a02cb2cece5e6/gas-usage
 export const COW_SHED_PROXY_CREATION_GAS = 360_000
+export const DEFAULT_EXTRA_GAS_PROXY_CREATION = 400_000
 export const HOOK_DAPP_BRIDGE_PROVIDER_PREFIX = 'cow-sdk://bridging/providers'

src/bridging/providers/across/AcrossBridgeProvider.ts

@@ -145,7 +145,11 @@ export class AcrossBridgeProvider implements BridgeProvider<AcrossQuoteResult> {
   }
 
   async getGasLimitEstimationForHook(request: QuoteBridgeRequest): Promise<number> {
-    return getGasLimitEstimationForHook(this.cowShedSdk, request, this.getRpcProvider(request.sellTokenChainId))
+    return getGasLimitEstimationForHook({
+      cowShedSdk: this.cowShedSdk,
+      request,
+      provider: this.getRpcProvider(request.sellTokenChainId),
+    })
   }
 
   async getSignedHook(

src/bridging/providers/bungee/BungeeBridgeProvider.ts

@@ -13,7 +13,11 @@ import {
   GetProviderBuyTokens,
   QuoteBridgeRequest,
 } from '../../types'
-import { DEFAULT_EXTRA_GAS_FOR_HOOK_ESTIMATION, RAW_PROVIDERS_FILES_PATH } from '../../const'
+import {
+  DEFAULT_EXTRA_GAS_FOR_HOOK_ESTIMATION,
+  DEFAULT_EXTRA_GAS_PROXY_CREATION,
+  RAW_PROVIDERS_FILES_PATH,
+} from '../../const'
 import { ChainId, ChainInfo, SupportedChainId } from '../../../chains'
 import { EvmCall, TokenInfo } from '../../../common'
 import { mainnet } from '../../../chains/details/mainnet'
@@ -150,14 +154,17 @@ export class BungeeBridgeProvider implements BridgeProvider<BungeeQuoteResult> {
   }
 
   async getGasLimitEstimationForHook(request: QuoteBridgeRequest): Promise<number> {
-    const extraGas = this.isExtraGasRequired(request) ? DEFAULT_EXTRA_GAS_FOR_HOOK_ESTIMATION : undefined
+    const isExtraGasRequired = this.isExtraGasRequired(request)
+    const extraGas = isExtraGasRequired ? DEFAULT_EXTRA_GAS_FOR_HOOK_ESTIMATION : undefined
+    const extraGasProxyCreation = isExtraGasRequired ? DEFAULT_EXTRA_GAS_PROXY_CREATION : undefined
 
-    return getGasLimitEstimationForHook(
-      this.cowShedSdk,
+    return getGasLimitEstimationForHook({
+      cowShedSdk: this.cowShedSdk,
       request,
-      this.getRpcProvider(request.sellTokenChainId),
+      provider: this.getRpcProvider(request.sellTokenChainId),
       extraGas,
-    )
+      extraGasProxyCreation,
+    })
   }
 
   async getSignedHook(

src/bridging/providers/utils/getGasLimitEstimationForHook.ts

@@ -3,18 +3,26 @@ import type { JsonRpcProvider } from '@ethersproject/providers'
 import { COW_SHED_PROXY_CREATION_GAS, DEFAULT_GAS_COST_FOR_HOOK_ESTIMATION } from '../../const'
 import { CowShedSdk } from '../../../cow-shed'
 
-export async function getGasLimitEstimationForHook(
-  cowShedSdk: CowShedSdk,
-  request: QuoteBridgeRequest,
-  provider: JsonRpcProvider,
-  extraGas?: number,
-): Promise<number> {
+export async function getGasLimitEstimationForHook({
+  cowShedSdk,
+  request,
+  provider,
+  extraGas,
+  extraGasProxyCreation,
+}: {
+  cowShedSdk: CowShedSdk
+  request: QuoteBridgeRequest
+  provider: JsonRpcProvider
+  extraGas?: number
+  extraGasProxyCreation?: number
+}): Promise<number> {
   const proxyAddress = cowShedSdk.getCowShedAccount(request.sellTokenChainId, request.owner || request.account)
   const proxyCode = await provider.getCode(proxyAddress)
 
   // Proxy is not deployed
   if (!proxyCode || proxyCode === '0x') {
-    return DEFAULT_GAS_COST_FOR_HOOK_ESTIMATION + COW_SHED_PROXY_CREATION_GAS
+    const baseGas = DEFAULT_GAS_COST_FOR_HOOK_ESTIMATION + COW_SHED_PROXY_CREATION_GAS
+    return baseGas + (extraGasProxyCreation ? extraGasProxyCreation : 0)
   }
 
   // Some bridges require extra gas to be added to the hook.

src/bridging/types.ts

@@ -209,10 +209,13 @@ export interface BridgeProvider<Q extends BridgeQuoteResult> {
    * 2. The final amount could affect hook gas costs
    *
    * By estimating gas costs independently, we can resolve this dependency cycle.
-   * For some providers, the `extraGas` flag adds a 100,000 gas‐unit buffer to the hook
-   * (see DEFAULT_EXTRA_GAS_FOR_HOOK_ESTIMATION).
+   * For some providers, the `extraGas` flag adds a 150,000 gas‐unit buffer to the hook
+   * and `extraGasProxyCreation` adds 360,000 gas‐unit buffer for the proxy creation
+   * (see DEFAULT_EXTRA_GAS_FOR_HOOK_ESTIMATION and DEFAULT_EXTRA_GAS_PROXY_CREATION).
    */
-  getGasLimitEstimationForHook(request: Omit<QuoteBridgeRequest, 'amount'> & { extraGas?: number }): Promise<number>
+  getGasLimitEstimationForHook(
+    request: Omit<QuoteBridgeRequest, 'amount'> & { extraGas?: number; extraGasProxyCreation?: number },
+  ): Promise<number>
 
   /**
    * Get a pre-authorized hook for initiating a bridge.