@exodus/error-tracking 3.1.0 → 3.1.2

package/CHANGELOG.md

@@ -3,6 +3,18 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [3.1.2](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/error-tracking@3.1.1...@exodus/error-tracking@3.1.2) (2025-09-09)
+
+### Bug Fixes
+
+- fix: respect ab-testing variant for sentry [2] (#13783)
+
+## [3.1.1](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/error-tracking@3.1.0...@exodus/error-tracking@3.1.1) (2025-08-25)
+
+### Bug Fixes
+
+- fix: pass through experiment id, make naming less confusing (#13685)
+
 ## [3.1.0](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/error-tracking@3.0.1...@exodus/error-tracking@3.1.0) (2025-08-25)
 
 ### Features

package/index.js

@@ -10,6 +10,7 @@ const defaultConfig = {
   maxErrorsCount: 100,
   sentryConfig: undefined,
   remoteErrorTrackingABExperimentId: 'sentry',
+  remoteErrorTrackingFundedWalletsABExperimentId: 'sentry-funded',
   trackWalletsCreatedAfter: new Date('2025-07-21'),
   trackFundedWallets: false,
 }
@@ -18,6 +19,7 @@ const configSchema = {
   maxErrorsCount: (value) => typeof value === 'number' && value > 0 && value <= 9999,
   sentryConfig: '?Object',
   remoteErrorTrackingABExperimentId: '?String',
+  remoteErrorTrackingFundedWalletsABExperimentId: '?String',
   trackWalletsCreatedAfter: '?Date',
   trackFundedWallets: '?Boolean',
 }
@@ -29,6 +31,7 @@ const errorTracking = (config = Object.create(null)) => {
     maxErrorsCount,
     sentryConfig,
     remoteErrorTrackingABExperimentId,
+    remoteErrorTrackingFundedWalletsABExperimentId,
     trackWalletsCreatedAfter,
     trackFundedWallets,
   } = typeforce.parse(configSchema, config, true)
@@ -63,6 +66,7 @@ const errorTracking = (config = Object.create(null)) => {
         definition: errorTrackingPluginDefinition,
         config: {
           remoteErrorTrackingABExperimentId,
+          remoteErrorTrackingFundedWalletsABExperimentId,
           trackWalletsCreatedAfter,
           trackFundedWallets,
         },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/error-tracking",
-  "version": "3.1.0",
+  "version": "3.1.2",
   "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": "8dad45b4cc01b73df0194fbfa20d411a77198baa"
+  "gitHead": "e9f71532ea6dbf4d7c9c6b40a755f221a5fd1de2"
 }

package/plugin/index.js

@@ -1,6 +1,6 @@
 import { combine, compute } from '@exodus/atoms'
 
-import whyIsRemoteTrackingDisabled from './why-is-remote-tracking-disabled.js'
+import { whyIsRemoteTrackingDisabled } from './why-is-remote-tracking-disabled.js'
 
 function errorTrackingPlugin({
   abTestingAtom,
@@ -42,6 +42,7 @@ function errorTrackingPlugin({
           trackFundedWallets,
           earliestTxDate,
           buildMetadata: await getBuildMetadata(),
+          logger,
         })
 
         if (reasonDisabled) {

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

@@ -1,4 +1,65 @@
-export default function whyIsRemoteTrackingDisabled({
+import typeforce from '@exodus/typeforce'
+
+export class ErrorTrackingAbExperiment {
+  #experimentId
+  #abExperimentSchema = typeforce.maybe({
+    enabled: 'Boolean',
+  })
+
+  #abVariantSchema = typeforce.maybe({
+    value: 'String',
+    type: 'String',
+  })
+
+  #enabled
+  #variant
+
+  constructor({ experimentId, abTesting, logger }) {
+    if (!logger) {
+      throw new Error('logger is required')
+    }
+
+    this.#experimentId = experimentId
+
+    if (experimentId) {
+      try {
+        const experimentParsed = typeforce.parse(
+          this.#abExperimentSchema,
+          abTesting?.experiments?.[experimentId],
+          true
+        )
+
+        const variantParsed = typeforce.parse(
+          this.#abVariantSchema,
+          abTesting?.variants?.[experimentId],
+          true
+        )
+
+        this.#enabled = experimentParsed?.enabled
+        this.#variant = variantParsed?.value
+      } catch {
+        // Attract attention to the issue and keep 'enabled' and 'variant' undefined to ensure remote tracking is disabled.
+        logger.debug(
+          `ErrorTrackingAbExperiment: failed to parse abTesting for experimentId: ${experimentId}`
+        )
+      }
+    }
+  }
+
+  get hasExperimentId() {
+    return !!this.#experimentId
+  }
+
+  get enabled() {
+    return this.#enabled
+  }
+
+  get variant() {
+    return this.#variant
+  }
+}
+
+export function whyIsRemoteTrackingDisabled({
   remoteErrorTrackingABExperimentId,
   remoteErrorTrackingFundedWalletsABExperimentId,
   abTesting,
@@ -7,15 +68,22 @@ export default function whyIsRemoteTrackingDisabled({
   trackFundedWallets,
   earliestTxDate,
   buildMetadata,
+  logger,
 }) {
   if (buildMetadata.dev) return 'dev-mode'
 
-  if (!remoteErrorTrackingABExperimentId) return 'missing-ab-experiment-id'
-  if (!abTesting.experiments?.[remoteErrorTrackingABExperimentId]?.enabled) {
+  const unfundedExperiment = new ErrorTrackingAbExperiment({
+    experimentId: remoteErrorTrackingABExperimentId,
+    abTesting,
+    logger,
+  })
+
+  if (!unfundedExperiment.hasExperimentId) return 'missing-ab-experiment-id'
+  if (!unfundedExperiment.enabled) {
     return 'ab-experiment-disabled'
   }
 
-  if (abTesting.variants?.[remoteErrorTrackingABExperimentId] !== 'enabled') {
+  if (unfundedExperiment.variant !== 'enabled') {
     return 'ab-experiment-variant-disabled'
   }
 
@@ -35,15 +103,21 @@ export default function whyIsRemoteTrackingDisabled({
   }
 
   // FUNDED WALLETS: separate AB experiment
-  if (!remoteErrorTrackingFundedWalletsABExperimentId) {
+  const fundedExperiment = new ErrorTrackingAbExperiment({
+    experimentId: remoteErrorTrackingFundedWalletsABExperimentId,
+    abTesting,
+    logger,
+  })
+
+  if (!fundedExperiment.hasExperimentId) {
     return 'missing-funded-wallets-ab-experiment-id'
   }
 
-  if (!abTesting.experiments?.[remoteErrorTrackingFundedWalletsABExperimentId]?.enabled) {
+  if (!fundedExperiment.enabled) {
     return 'funded-wallets-ab-experiment-disabled'
   }
 
-  if (abTesting.variants?.[remoteErrorTrackingFundedWalletsABExperimentId] !== 'enabled') {
+  if (fundedExperiment.variant !== 'enabled') {
     return 'funded-wallets-ab-experiment-variant-disabled'
   }
 }