@exodus/error-tracking 3.0.0 → 3.0.1

package/CHANGELOG.md

@@ -3,6 +3,12 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [3.0.1](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/error-tracking@3.0.0...@exodus/error-tracking@3.0.1) (2025-08-21)
+
+### Bug Fixes
+
+- fix: respect ab-testing variant for sentry (#13655)
+
 ## [3.0.0](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/error-tracking@2.2.0...@exodus/error-tracking@3.0.0) (2025-07-01)
 
 ### ⚠ BREAKING CHANGES

package/module/error-tracking.js

@@ -7,19 +7,25 @@ const createErrorTracking = ({
   config,
   logger,
 }) => {
-  const track = async ({ error, context, namespace }) => {
+  const track = async ({ error, context, namespace, silent = true, timeout = 5000 }) => {
+    const onError = (err) => {
+      logger.error(err)
+      if (!silent) throw err
+    }
+
+    logger.error(error)
+
     if (namespace !== undefined && typeof namespace !== 'string') {
-      throw new Error('namespace must be a string')
+      return onError(new Error('namespace must be a string'))
     }
 
     if (!(error instanceof Error)) {
-      throw new TypeError('error must be an instance of Error')
+      return onError(new TypeError('error must be an instance of Error'))
     }
 
     // TODO: figure out what to do with `context`
-
-    // eventually kill this and only track remote
-    await errorsAtom.set(({ errors }) => {
+    // TODO: eventually kill this and only track remote
+    const localPromise = errorsAtom.set(({ errors }) => {
       return {
         // this array can be big. not sure about prefering spread operator here
         // concat function seems like a better option
@@ -30,13 +36,29 @@ const createErrorTracking = ({
       }
     })
 
-    if (remoteErrorTracking) {
-      remoteErrorTrackingEnabledAtom
-        .get()
-        .then((enabled) => enabled && remoteErrorTracking.track({ error }))
-        .catch((err) => {
-          logger.error('failed to upload error', error, err)
-        })
+    const remotePromise = remoteErrorTracking
+      ? remoteErrorTrackingEnabledAtom
+          .get()
+          .then((enabled) => enabled && remoteErrorTracking.track({ error }))
+      : Promise.resolve()
+
+    let timeoutId
+    const timeoutPromise = new Promise((resolve, reject) => {
+      timeoutId = setTimeout(() => {
+        reject(new Error('track call timed out'))
+      }, timeout)
+    })
+
+    try {
+      await Promise.race([
+        //
+        Promise.all([localPromise, remotePromise]),
+        timeoutPromise,
+      ])
+    } catch (err) {
+      onError(err)
+    } finally {
+      clearTimeout(timeoutId)
     }
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/error-tracking",
-  "version": "3.0.0",
+  "version": "3.0.1",
   "type": "module",
   "description": "A simple error tracking package to let any feature collect errors and create the report",
   "author": "Exodus Movement, Inc.",
@@ -43,5 +43,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "7425cf446dbe32c407ac46bd1c0463787cf8deb6"
+  "gitHead": "16007bf14e75a4a9b80efba3a884593f9fab15d4"
 }

package/plugin/why-is-remote-tracking-disabled.js

@@ -8,8 +8,14 @@ export default function whyIsRemoteTrackingDisabled({
   buildMetadata,
 }) {
   if (!remoteErrorTrackingABExperimentId) return 'missing-ab-experiment-id'
-  if (!abTesting.experiments?.[remoteErrorTrackingABExperimentId]?.enabled)
+  if (!abTesting.experiments?.[remoteErrorTrackingABExperimentId]?.enabled) {
     return 'ab-experiment-disabled'
+  }
+
+  if (abTesting.variants?.[remoteErrorTrackingABExperimentId] !== 'enabled') {
+    return 'ab-experiment-variant-disabled'
+  }
+
   if (
     trackWalletsCreatedAfter &&
     // the negation's a bit harder to read but is a safer check