chore: remove logs - bump version

Author: cowdan

package.json

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

src/bridging/BridgingSdk/getCrossChainOrder.ts

@@ -20,13 +20,9 @@ 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) {
@@ -41,8 +37,6 @@ 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]
@@ -58,11 +52,7 @@ 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}`)
     }
 
@@ -80,15 +70,13 @@ 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('SDK getCrossChainOrder Cannot get bridging status', e)
+      console.error('Cannot get bridging status', e)
       return state
     }
   }

src/bridging/PROVIDER_README.md

@@ -385,10 +385,11 @@ export class YourBridgeProvider implements BridgeProvider<YourBridgeQuoteResult>
   ): Promise<number> {
     // Use utility function or implement custom gas estimation
     return getGasLimitEstimationForHook(
-      this.cowShedSdk,
-      request as QuoteBridgeRequest, // cast needed due to omit
-      this.getRpcProvider(request.sellTokenChainId),
-      request.extraGas, // to add extra gas to the hook.
+      cowshed: this.cowShedSdk,
+      request: request as QuoteBridgeRequest, // cast needed due to omit
+      provider: this.getRpcProvider(request.sellTokenChainId),
+      extraGas: request.extraGas, // to add extra gas to the hook
+      extraGasProxyCreation: request.extraGasProxyCreation // to add extra gas to the hook and deploy proxy account
     )
   }
 

src/bridging/providers/bungee/BungeeApi.ts

@@ -338,7 +338,6 @@ export class BungeeApi {
       params,
       isValidBungeeEventsResponse,
     )
-    console.log('SDK getEvents response ==>', response)
     if (!response.success) {
       throw new BridgeProviderError('Bungee Events Api Error', response)
     }
@@ -502,7 +501,7 @@ function isValidBungeeEventsResponse(response: unknown): response is BungeeEvent
       'fromChainId' in e &&
       'isCowswapTrade' in e &&
       'orderId' in e &&
-      'recipient' in e &&
+      // 'recipient' in e &&
       'sender' in e &&
       'srcTxStatus' in e &&
       'destTxStatus' in e

src/bridging/providers/bungee/BungeeBridgeProvider.ts

@@ -210,10 +210,7 @@ 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 })
-    console.log('SDK getBridgingParams events ==>', events)
     const event = events?.[0]
 
     if (!event) return null

src/bridging/providers/bungee/getBridgingStatusFromEvents.ts

@@ -19,11 +19,8 @@ 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 {
@@ -41,15 +38,12 @@ export async function getBridgingStatusFromEvents(
       }
     }
 
-    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,

src/bridging/providers/utils/getGasLimitEstimationForHook.ts

@@ -22,13 +22,8 @@ export async function getGasLimitEstimationForHook({
   // Proxy is not deployed
   if (!proxyCode || proxyCode === '0x') {
     const baseGas = DEFAULT_GAS_COST_FOR_HOOK_ESTIMATION + COW_SHED_PROXY_CREATION_GAS
-    return baseGas + (extraGasProxyCreation ? extraGasProxyCreation : 0)
+    return baseGas + (extraGasProxyCreation && extraGasProxyCreation > 0 ? extraGasProxyCreation : 0)
   }
 
-  // Some bridges require extra gas to be added to the hook.
-  if (extraGas && extraGas > 0) {
-    return DEFAULT_GAS_COST_FOR_HOOK_ESTIMATION + extraGas
-  }
-
-  return DEFAULT_GAS_COST_FOR_HOOK_ESTIMATION
+  return DEFAULT_GAS_COST_FOR_HOOK_ESTIMATION + (extraGas && extraGas > 0 ? extraGas : 0)
 }

src/bridging/types.ts

@@ -209,8 +209,8 @@ 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 150,000 gas‐unit buffer to the hook
-   * and `extraGasProxyCreation` adds 360,000 gas‐unit buffer for the proxy creation
+   * For some providers, the `extraGas` parameter adds additional gas‐unit buffer to the hook
+   * and `extraGasProxyCreation` parameter adds additional gas‐unit buffer for the proxy creation
    * (see DEFAULT_EXTRA_GAS_FOR_HOOK_ESTIMATION and DEFAULT_EXTRA_GAS_PROXY_CREATION).
    */
   getGasLimitEstimationForHook(