ref(nextjs): Add more specific mechanisms to captured errors

Author: Lms24

packages/nextjs/src/common/captureRequestError.ts

@@ -38,6 +38,7 @@ export function captureRequestError(error: unknown, request: RequestInfo, errorC
     captureException(error, {
       mechanism: {
         handled: false,
+        type: 'nextjs.onRequestError',
       },
     });
 

packages/nextjs/src/common/pages-router-instrumentation/_error.ts

@@ -45,7 +45,7 @@ export async function captureUnderscoreErrorException(contextOrProps: ContextOrP
     // is what passing a string to `captureException` will wind up doing)
     captureException(err || `_error.js called with falsy error (${err})`, {
       mechanism: {
-        type: 'instrument',
+        type: 'nextjs._error',
         handled: false,
         data: {
           function: '_error.getInitialProps',

packages/nextjs/src/common/pages-router-instrumentation/wrapApiHandlerWithSentry.ts

@@ -110,7 +110,7 @@ export function wrapApiHandlerWithSentry(apiHandler: NextApiHandler, parameteriz
 
                     captureException(objectifiedErr, {
                       mechanism: {
-                        type: 'instrument',
+                        type: 'nextjs.api',
                         handled: false,
                         data: {
                           wrapped_handler: wrappingTarget.name,

packages/nextjs/src/common/pages-router-instrumentation/wrapPageComponentWithSentry.ts

@@ -48,6 +48,7 @@ export function wrapPageComponentWithSentry(pageComponent: FunctionComponent | C
             captureException(e, {
               mechanism: {
                 handled: false,
+                type: 'nextjs.page.class',
               },
             });
             throw e;
@@ -77,6 +78,7 @@ export function wrapPageComponentWithSentry(pageComponent: FunctionComponent | C
             captureException(e, {
               mechanism: {
                 handled: false,
+                type: 'nextjs.page.function',
               },
             });
             throw e;

packages/nextjs/src/common/utils/wrapperUtils.ts

@@ -25,7 +25,11 @@ export function withErrorInstrumentation<F extends (...args: any[]) => any>(
       return await origFunction.apply(this, origFunctionArguments);
     } catch (e) {
       // TODO: Extract error logic from `withSentry` in here or create a new wrapper with said logic or something like that.
-      captureException(e, { mechanism: { handled: false } });
+      captureException(e, {
+        // TODO: check if origFunction.name actually returns the correct name or minified garbage
+        // in this case, we can add another argument to this wrapper with the respective function name
+        mechanism: { handled: false, type: 'nextjs.wrapped', data: { function: origFunction.name } },
+      });
       throw e;
     }
   };
@@ -99,7 +103,7 @@ export async function callDataFetcherTraced<F extends (...args: any[]) => Promis
   try {
     return await origFunction(...origFunctionArgs);
   } catch (e) {
-    captureException(e, { mechanism: { handled: false } });
+    captureException(e, { mechanism: { handled: false, type: 'nextjs' } });
     throw e;
   }
 }

packages/nextjs/src/common/withServerActionInstrumentation.ts

@@ -130,6 +130,7 @@ async function withServerActionInstrumentationImplementation<A extends (...args:
                   captureException(error, {
                     mechanism: {
                       handled: false,
+                      type: 'nextjs.server-action',
                     },
                   });
                 }

packages/nextjs/src/common/wrapGenerationFunctionWithSentry.ts

@@ -138,6 +138,10 @@ export function wrapGenerationFunctionWithSentry<F extends (...args: any[]) => a
                     captureException(err, {
                       mechanism: {
                         handled: false,
+                        type: 'nextjs.generation-function',
+                        data: {
+                          function: generationFunctionIdentifier,
+                        },
                       },
                     });
                   }

packages/nextjs/src/common/wrapMiddlewareWithSentry.ts

@@ -102,7 +102,7 @@ export function wrapMiddlewareWithSentry<H extends EdgeRouteHandler>(
               error => {
                 captureException(error, {
                   mechanism: {
-                    type: 'instrument',
+                    type: 'nextjs.middleware',
                     handled: false,
                   },
                 });

packages/nextjs/src/common/wrapRouteHandlerWithSentry.ts

@@ -90,6 +90,7 @@ export function wrapRouteHandlerWithSentry<F extends (...args: any[]) => any>(
                   captureException(error, {
                     mechanism: {
                       handled: false,
+                      type: 'nextjs.route-handler',
                     },
                   });
                 }

packages/nextjs/src/common/wrapServerComponentWithSentry.ts

@@ -130,6 +130,7 @@ export function wrapServerComponentWithSentry<F extends (...args: any[]) => any>
                     captureException(error, {
                       mechanism: {
                         handled: false,
+                        type: 'nextjs.server-component',
                       },
                     });
                   }