chore: Log swap error (#8664)

pancakeswap · Dec 25, 2023 · 7bf73e6 · 7bf73e6
1 parent b69b10b
commit 7bf73e6
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 6 deletions.
diff --git a/apps/web/src/utils/log.ts b/apps/web/src/utils/log.ts
@@ -18,8 +18,8 @@ export const logSwap = ({
 }: {
   input: Currency
   output: Currency
-  inputAmount: string
-  outputAmount: string
+  inputAmount?: string
+  outputAmount?: string
   chainId: number
   account: `0x${string}`
   hash: `0x${string}`

diff --git a/apps/web/src/views/Swap/V3Swap/hooks/useSendSwapTransaction.ts b/apps/web/src/views/Swap/V3Swap/hooks/useSendSwapTransaction.ts
@@ -1,11 +1,10 @@
+import { ChainId } from '@pancakeswap/chains'
 import { useTranslation } from '@pancakeswap/localization'
 import { TradeType } from '@pancakeswap/sdk'
-import { ChainId } from '@pancakeswap/chains'
 import { SmartRouter, SmartRouterTrade } from '@pancakeswap/smart-router/evm'
 import { formatAmount } from '@pancakeswap/utils/formatFractions'
 import truncateHash from '@pancakeswap/utils/truncateHash'
 import { useUserSlippage } from '@pancakeswap/utils/user'
-import { SendTransactionResult } from 'wagmi/actions'
 import { INITIAL_ALLOWED_SLIPPAGE } from 'config/constants'
 import { useMemo } from 'react'
 import { useSwapState } from 'state/swap/hooks'
@@ -18,7 +17,9 @@ import { transactionErrorToUserReadableMessage } from 'utils/transactionErrorToU
 import { viemClients } from 'utils/viem'
 import { Address, Hex, hexToBigInt } from 'viem'
 import { useSendTransaction } from 'wagmi'
+import { SendTransactionResult } from 'wagmi/actions'
 
+import { logger } from 'utils/datadog'
 import { isZero } from '../utils/isZero'
 
 interface SwapCall {
@@ -51,7 +52,7 @@ export class TransactionRejectedError extends Error {}
 export default function useSendSwapTransaction(
   account?: Address,
   chainId?: number,
-  trade?: SmartRouterTrade<TradeType>, // trade to execute, required
+  trade?: SmartRouterTrade<TradeType> | null, // trade to execute, required
   swapCalls: SwapCall[] | WallchainSwapCall[] = [],
 ): { callback: null | (() => Promise<SendTransactionResult>) } {
   const { t } = useTranslation()
@@ -212,7 +213,18 @@ export default function useSendSwapTransaction(
               throw new TransactionRejectedError(t('Transaction rejected'))
             } else {
               // otherwise, the error was unexpected and we need to convey that
-              console.error(`Swap failed`, error, call.address, call.calldata, call.value)
+              logger.error(
+                'Swap failed',
+                {
+                  chainId,
+                  input: trade.inputAmount.currency,
+                  output: trade.outputAmount.currency,
+                  address: call.address,
+                  calldata: call.calldata,
+                  value: call.value,
+                },
+                error,
+              )
 
               throw new Error(`Swap failed: ${transactionErrorToUserReadableMessage(error, t)}`)
             }