Merge pull request #1713 from sushi-labs/sentry

sentry: add breadcrumbs for swap execution and set context
sushi-labs · Oct 1, 2024 · 78f46a8 · 78f46a8
2 parents 3604efd + 9b8dc03
commit 78f46a8
Show file tree

Hide file tree

Showing 2 changed files with 46 additions and 5 deletions.
diff --git a/apps/web/src/ui/swap/simple/derivedstate-simple-swap-provider.tsx b/apps/web/src/ui/swap/simple/derivedstate-simple-swap-provider.tsx
@@ -411,7 +411,7 @@ const useSimpleSwapTrade = () => {
     [slippagePercent, tokenTax],
   )
 
-  const apiTrade = useApiTrade({
+  const trade = useApiTrade({
     chainId,
     fromToken: token0,
     toToken: token1,
@@ -430,7 +430,7 @@ const useSimpleSwapTrade = () => {
     setTokenTax(undefined)
   }, [token0, token1, setTokenTax])
 
-  return apiTrade
+  return trade
 }
 
 export {

diff --git a/apps/web/src/ui/swap/simple/simple-swap-trade-review-dialog.tsx b/apps/web/src/ui/swap/simple/simple-swap-trade-review-dialog.tsx
@@ -1,5 +1,6 @@
 'use client'
 
+import * as Sentry from '@sentry/nextjs'
 import { createErrorToast, createToast } from '@sushiswap/notifications'
 import {
   BrowserEvent,
@@ -42,7 +43,7 @@ import { useApproved } from 'src/lib/wagmi/systems/Checker/Provider'
 import { Chain, ChainId } from 'sushi/chain'
 import { Native } from 'sushi/currency'
 import { shortenAddress } from 'sushi/format'
-import { ZERO } from 'sushi/math'
+import { Percent, ZERO } from 'sushi/math'
 import {
   SendTransactionReturnType,
   UserRejectedRequestError,
@@ -85,6 +86,23 @@ export const SimpleSwapTradeReviewDialog: FC<{
 
   const refetchBalances = useBalanceWeb3Refetch()
 
+  useEffect(() => {
+    if (!trade) return
+    Sentry.setContext('swap-context', {
+      chainId,
+      amountIn: trade?.amountIn?.toSignificant(6),
+      amountOut: trade?.amountOut?.toSignificant(6),
+      minAmountOut: trade?.minAmountOut?.toSignificant(6),
+      fromToken: trade?.route?.fromToken,
+      toToken: trade?.route?.toToken,
+      priceImpact: trade?.priceImpact?.toPercentageString(),
+      tokenTax:
+        trade?.tokenTax instanceof Percent
+          ? trade.tokenTax.toPercentageString()
+          : trade?.tokenTax,
+    })
+  }, [trade, chainId])
+
   const isWrap =
     token0?.isNative &&
     token1?.wrapped.address === Native.onChain(chainId).wrapped.address
@@ -133,7 +151,10 @@ export const SimpleSwapTradeReviewDialog: FC<{
 
       try {
         const ts = new Date().getTime()
-        const receiptPromise = client.waitForTransactionReceipt({ hash })
+        const receiptPromise = client.waitForTransactionReceipt({
+          hash,
+          retryCount: 30,
+        })
 
         sendAnalyticsEvent(SwapEventName.SWAP_SIGNED, {
           ...trace,
@@ -253,15 +274,35 @@ export const SimpleSwapTradeReviewDialog: FC<{
 
     return async (confirm: () => void) => {
       try {
+        Sentry.addBreadcrumb({
+          category: 'swap',
+          message: 'Swap execution in progress',
+          level: 'info',
+        })
         await sendTransactionAsync(simulation)
+        // Add breadcrumb for successful swap
+        Sentry.addBreadcrumb({
+          category: 'swap',
+          message: 'Swap completed successfully',
+          level: 'info',
+        })
         confirm()
-      } catch {}
+      } catch (e) {
+        Sentry.addBreadcrumb({
+          category: 'swap',
+          message: 'Swap failed',
+          level: 'error',
+        })
+        Sentry.captureException(e)
+        throw e
+      }
     }
   }, [simulation, sendTransactionAsync])
 
   const { status } = useWaitForTransactionReceipt({
     chainId: chainId,
     hash: data,
+    retryCount: 30,
   })
 
   return (