chore: increase gas for Gnosis bridge hook to 150k

Author: cowdan

package.json

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

src/bridging/BridgingSdk/getCrossChainOrder.ts

@@ -20,9 +20,13 @@ interface GetCrossChainOrderParams {
 export async function getCrossChainOrder(params: GetCrossChainOrderParams): Promise<CrossChainOrder | null> {
   const { chainId, orderId, orderBookApi, providers, env } = params
 
+  console.log('SDK getCrossChainOrder params ==>', params)
+
   const chainContext = { chainId, env }
   const order = await orderBookApi.getOrder(orderId, chainContext)
 
+  console.log('SDKgetCrossChainOrder order ==>', order)
+
   // Find the provider by name (note that I could just have use this.provider, but just wanted to leave it ready in case we implement multiple providers)
   const provider = order.fullAppData && findBridgeProviderFromHook(order.fullAppData, providers)
   if (!provider) {
@@ -37,6 +41,8 @@ export async function getCrossChainOrder(params: GetCrossChainOrderParams): Prom
   // Check if there are any trades for this order
   const trades = await orderBookApi.getTrades({ orderUid: order.uid }, chainContext)
 
+  console.log('SDK getCrossChainOrder trades ==>', trades)
+
   if (trades.length > 0) {
     // Bridging already initiated
     const firstTrade = trades[0]
@@ -52,7 +58,11 @@ export async function getCrossChainOrder(params: GetCrossChainOrderParams): Prom
     const { params: bridgingParams, status: statusResult } =
       (await provider.getBridgingParams(chainId, orderId, tradeTxHash)) || {}
 
+    console.log('SDK getCrossChainOrder bridgingParams ==>', bridgingParams)
+    console.log('SDK getCrossChainOrder statusResult ==>', statusResult)
+
     if (!bridgingParams || !statusResult) {
+      console.log('SDK getCrossChainOrder bridgingParams || !statusResult ==>', bridgingParams || !statusResult)
       throw new BridgeOrderParsingError(`Bridging params cannot be derived from transaction: ${tradeTxHash}`)
     }
 
@@ -70,13 +80,15 @@ export async function getCrossChainOrder(params: GetCrossChainOrderParams): Prom
     try {
       const explorerUrl = provider.getExplorerUrl(bridgingParams.bridgingId)
 
+      console.log('SDK getCrossChainOrder explorerUrl ==>', explorerUrl)
+
       return {
         ...state,
         statusResult,
         explorerUrl,
       }
     } catch (e) {
-      console.error('Cannot get bridging status', e)
+      console.error('SDK getCrossChainOrder Cannot get bridging status', e)
       return state
     }
   }

src/bridging/const.ts

@@ -3,7 +3,7 @@ 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 = 100_000
+export const DEFAULT_EXTRA_GAS_FOR_HOOK_ESTIMATION = 150_000
 // Based on https://dashboard.tenderly.co/shoom/project/tx/0x2971aee9aa7237d24b254da8ccd4345ff77410c8829c8e825e9a02cb2cece5e6/gas-usage
 export const COW_SHED_PROXY_CREATION_GAS = 360_000
 export const HOOK_DAPP_BRIDGE_PROVIDER_PREFIX = 'cow-sdk://bridging/providers'

src/bridging/providers/bungee/BungeeApi.ts

@@ -338,6 +338,7 @@ export class BungeeApi {
       params,
       isValidBungeeEventsResponse,
     )
+    console.log('SDK getEvents response ==>', response)
     if (!response.success) {
       throw new BridgeProviderError('Bungee Events Api Error', response)
     }

src/bridging/providers/bungee/BungeeBridgeProvider.ts

@@ -203,8 +203,11 @@ export class BungeeBridgeProvider implements BridgeProvider<BungeeQuoteResult> {
     orderId: string,
     _txHash: string,
   ): Promise<{ params: BridgingDepositParams; status: BridgeStatusResult } | null> {
+    console.log('SDK getBridgingParams orderId ==>', orderId)
+    console.log('SDK getBridgingParams txHash ==>', _txHash)
     const events = await this.api.getEvents({ orderId })
-    const event = events[0]
+    console.log('SDK getBridgingParams events ==>', events)
+    const event = events?.[0]
 
     if (!event) return null
 

src/bridging/providers/bungee/getBridgingStatusFromEvents.ts

@@ -19,8 +19,11 @@ export async function getBridgingStatusFromEvents(
     return { status: BridgeStatus.IN_PROGRESS }
   }
 
+  console.log('SDK getBridgingStatusFromEvents event ==>', event)
   // if srcTxStatus = completed & destTxStatus = pending,
   if (event.srcTxStatus === BungeeEventStatus.COMPLETED && event.destTxStatus === BungeeEventStatus.PENDING) {
+    console.log('SDK getBridgingStatusFromEvents event.COMPLETED ==>', event.bridgeName)
+
     // if bridgeName = across,
     if (event.bridgeName === BungeeBridgeName.ACROSS) {
       try {
@@ -37,12 +40,16 @@ export async function getBridgingStatusFromEvents(
         console.error('BungeeBridgeProvider get across status error', e)
       }
     }
+
+    console.log('SDK getBridgingStatusFromEvents is not across ==>', event.bridgeName)
     // if not across or across API fails, waiting for dest tx, return in_progress
     return { status: BridgeStatus.IN_PROGRESS, depositTxHash: event.srcTransactionHash }
   }
 
   // if srcTxStatus = completed & destTxStatus = completed, return executed
   if (event.srcTxStatus === BungeeEventStatus.COMPLETED && event.destTxStatus === BungeeEventStatus.COMPLETED) {
+    console.log('SDK getBridgingStatusFromEvents is completed ==>', event.bridgeName)
+
     return {
       status: BridgeStatus.EXECUTED,
       depositTxHash: event.srcTransactionHash,