update analytics error tracking to add metadata

complexdatacollective · Jan 25, 2024 · c5c7a86 · c5c7a86
1 parent e3d1456
commit c5c7a86
Show file tree

Hide file tree

Showing 8 changed files with 31 additions and 37 deletions.
diff --git a/app/(dashboard)/dashboard/interviews/_actions/export.ts b/app/(dashboard)/dashboard/interviews/_actions/export.ts
@@ -130,11 +130,10 @@ export const exportSessions = async (
     const e = ensureError(error);
     await trackEvent({
       type: 'Error',
-      error: {
-        message: e.name,
-        details: e.message,
+      error: e,
+      metadata: {
+        details: 'Server-side Error',
         path: '/(dashboard)/dashboard/interviews/_actions/export.ts',
-        stacktrace: e.stack ?? '',
       },
     });
 

diff --git a/app/(interview)/onboard/[protocolId]/route.ts b/app/(interview)/onboard/[protocolId]/route.ts
@@ -14,11 +14,10 @@ const handler = async (
   if (!protocolId || protocolId === 'undefined') {
     void trackEvent({
       type: 'Error',
-      error: {
+      error: new Error('No protocol ID provided.'),
+      metadata: {
         details: 'No protocol ID provided',
-        message: 'No protocol ID provided',
         path: '/onboard/[protocolId]/route.ts',
-        stacktrace: '',
       },
     });
 
@@ -48,11 +47,10 @@ const handler = async (
   if (error) {
     void trackEvent({
       type: 'Error',
-      error: {
-        details: error,
-        message: 'Failed to create interview',
+      error: new Error(error),
+      metadata: {
+        details: 'Failed to create interview',
         path: '/onboard/[protocolId]/route.ts',
-        stacktrace: '',
       },
     });
 

diff --git a/app/api/trpc/[trpc]/route.ts b/app/api/trpc/[trpc]/route.ts
@@ -15,11 +15,10 @@ const handler = (req: Request) =>
 
       void trackEvent({
         type: 'Error',
-        error: {
-          message: error.message,
+        error: new Error(error.message),
+        metadata: {
           details: type,
           path: path ?? 'unknown',
-          stacktrace: 'unknown',
         },
       });
 

diff --git a/components/Error.tsx b/components/Error.tsx
@@ -19,10 +19,9 @@ export default function Error({
       try {
         await trackEvent({
           type: 'Error',
-          error: {
-            message: error.message,
+          error,
+          metadata: {
             details: heading ?? '',
-            stacktrace: error.stack ?? '',
             path: window.location.pathname,
           },
         });

diff --git a/components/ProtocolImport/JobReducer.ts b/components/ProtocolImport/JobReducer.ts
@@ -124,11 +124,10 @@ export function jobReducer(state: ImportJob[], action: Action) {
 
       void trackEvent({
         type: 'Error',
-        error: {
+        error: rawError,
+        metadata: {
           details: 'Protocol import error',
-          message: rawError.message,
-          stacktrace: rawError.stack ?? '',
-          path: '',
+          path: 'components/ProtocolImport/JobReducer.ts',
         },
       });
 

diff --git a/package.json b/package.json
@@ -19,7 +19,7 @@
     "seed": "tsx prisma/seed.ts"
   },
   "dependencies": {
-    "@codaco/analytics": "^2.1.1",
+    "@codaco/analytics": "link:..\\error-analytics-microservice\\packages\\analytics",
     "@codaco/protocol-validation": "3.0.0-alpha.4",
     "@codaco/shared-consts": "^0.0.2",
     "@headlessui/react": "^1.7.17",

diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
diff --git a/server/routers/interview.ts b/server/routers/interview.ts
@@ -102,11 +102,10 @@ export const interviewRouter = router({
 
         void trackEvent({
           type: 'Error',
-          error: {
-            message: e.name,
+          error: e,
+          metadata: {
             details: e.message,
             path: '/routers/interview.ts',
-            stacktrace: e.stack ?? '',
           },
         });