@exodus/error-tracking 1.1.2 → 1.2.0

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.
 
+## [1.2.0](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/error-tracking@1.1.3...@exodus/error-tracking@1.2.0) (2024-08-08)
+
+### Features
+
+- **error-tracking:** add remote error tracking api node ([#8169](https://github.com/ExodusMovement/exodus-hydra/issues/8169)) ([5a2d702](https://github.com/ExodusMovement/exodus-hydra/commit/5a2d702126d81bfb934c2598b732e62d23370f03))
+
+## [1.1.3](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/error-tracking@1.1.2...@exodus/error-tracking@1.1.3) (2024-07-25)
+
+### Bug Fixes
+
+- migrate definitions ([#8030](https://github.com/ExodusMovement/exodus-hydra/issues/8030)) ([d2dfde5](https://github.com/ExodusMovement/exodus-hydra/commit/d2dfde55dfa843eb52842f64b3aac3a6f9a59069))
+
 ## [1.1.2](https://github.com/ExodusMovement/exodus-hydra/compare/@exodus/error-tracking@1.1.1...@exodus/error-tracking@1.1.2) (2024-07-18)
 
 **Note:** Version bump only for package @exodus/error-tracking

package/api/index.js

@@ -1,22 +1,38 @@
 import { createErrorTracking } from '../module/create-error-tracking.js'
 
-// Implementation Note: It would have made more sense to add the `errorTracking` module as a dependency instead of
-// just its factory function. However, the issue is that adding errorTracking as a dependency to this API node
-// will cause the error-tracking-preprocessor to change the track function definition, which is not what we require here.
-
-const errorTrackingApiDefinition = {
+export const errorTrackingApiDefinition = {
   id: 'errorTrackingApi',
   type: 'api',
-  factory: ({ errorsAtom, config }) => {
+  factory: ({ errorsAtom, config, logger, remoteErrorTracking }) => {
     const { track } = createErrorTracking({ errorsAtom, config })
 
+    let trackRemote
+    if (remoteErrorTracking) {
+      trackRemote = async ({ error }) => {
+        try {
+          await remoteErrorTracking.captureError({
+            error,
+          })
+        } catch (err) {
+          logger.error('failed to remote track error', err)
+        }
+      }
+    } else {
+      trackRemote = async ({ error }) => {
+        logger.debug('remote error tracking is disabled', error)
+      }
+    }
+
     return {
       errors: {
         track,
+        trackRemote,
       },
     }
   },
-  dependencies: ['config', 'errorsAtom'],
+  dependencies: ['config', 'errorsAtom', 'logger', 'remoteErrorTracking?'],
 }
 
-export default errorTrackingApiDefinition
+// Implementation Note: It would have made more sense to add the `errorTracking` module as a dependency instead of
+// just its factory function. However, the issue is that adding errorTracking as a dependency to this API node
+// will cause the error-tracking-preprocessor to change the track function definition, which is not what we require here.

package/atoms/index.js

@@ -7,7 +7,6 @@ import { createInMemoryAtom } from '@exodus/atoms'
 */
 const errorsAtomDefinition = {
   id: 'errorsAtom',
-  private: true,
   type: 'atom',
   factory: () => createInMemoryAtom({ defaultValue: { errors: [] } }),
   dependencies: [],

package/index.js

@@ -1,6 +1,7 @@
 import { isEmpty } from './is-empty.js'
 
-import errorTrackingApiDefinition from './api/index.js'
+import { errorTrackingApiDefinition } from './api/index.js'
+
 import errorTrackingAtomDefinition from './atoms/index.js'
 import errorTrackingReportDefinition from './report/index.js'
 import { errorTrackingDefinition, remoteErrorTrackingDefinition } from './module/index.js'

package/module/error-tracking.js

@@ -7,4 +7,5 @@ export const errorTrackingDefinition = {
   type: 'module',
   factory: createErrorTracking,
   dependencies: ['config', 'errorsAtom'],
+  public: true,
 }

package/module/remote-error-tracking.js

@@ -5,7 +5,6 @@ import createFetchival from '@exodus/fetch/create-fetchival'
 export const remoteErrorTrackingDefinition = {
   id: 'remoteErrorTracking',
   type: 'module',
-  private: true,
   factory: ({ config, fetch, defaultStackTraceParser }) => {
     const fetchival = createFetchival({ fetch })
     return createSentryClient({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/error-tracking",
-  "version": "1.1.2",
+  "version": "1.2.0",
   "type": "module",
   "description": "A simple error tracking package to let any feature collect errors and create the report",
   "author": "Exodus Movement, Inc.",
@@ -38,5 +38,5 @@
     "@exodus/sentry-client": "^3.2.0",
     "@exodus/typeforce": "^1.19.0"
   },
-  "gitHead": "a76a22c96445bcaa05e18de0e0c323cf9dd04cba"
+  "gitHead": "795fabca74101cee3be6c9a0581b327e9cc3293f"
 }

package/report/index.js

@@ -8,6 +8,7 @@ const errorTrackingReportDefinition = {
     },
   }),
   dependencies: ['errorsAtom'],
+  public: true,
 }
 
 export default errorTrackingReportDefinition