adjust type to match trace origin spec

Lms24 · Lms24 · commit f5bf5d1e444f · 2025-08-01T15:28:24.000+02:00
diff --git a/dev-packages/browser-integration-tests/suites/public-api/instrumentation/eventListener/event-target/test.ts b/dev-packages/browser-integration-tests/suites/public-api/instrumentation/eventListener/event-target/test.ts
@@ -13,10 +13,9 @@ sentryTest('should capture target name in mechanism data', async ({ getLocalTest
     type: 'Error',
     value: 'event_listener_error',
     mechanism: {
-      type: 'browserApiErrors',
+      type: 'auto.browser.browserapierrors.addEventListener',
       handled: false,
       data: {
-        function: 'addEventListener',
         handler: 'functionListener',
         target: 'EventTarget',
       },
diff --git a/dev-packages/browser-integration-tests/suites/public-api/instrumentation/eventListener/named-function/test.ts b/dev-packages/browser-integration-tests/suites/public-api/instrumentation/eventListener/named-function/test.ts
@@ -13,10 +13,9 @@ sentryTest('should capture built-in handlers fn name in mechanism data', async (
     type: 'Error',
     value: 'event_listener_error',
     mechanism: {
-      type: 'browserApiErrors',
+      type: 'auto.browser.browserapierrors.addEventListener',
       handled: false,
       data: {
-        function: 'addEventListener',
         handler: 'clickHandler',
         target: 'EventTarget',
       },
diff --git a/dev-packages/browser-integration-tests/suites/public-api/instrumentation/eventListener/remove/test.ts b/dev-packages/browser-integration-tests/suites/public-api/instrumentation/eventListener/remove/test.ts
@@ -13,7 +13,7 @@ sentryTest('should transparently remove event listeners from wrapped functions',
     type: 'Error',
     value: 'foo',
     mechanism: {
-      type: 'browserApiErrors',
+      type: 'auto.browser.browserapierrors.addEventListener',
       handled: false,
     },
     stacktrace: {
diff --git a/dev-packages/browser-integration-tests/suites/public-api/instrumentation/eventListener/thrown-error/test.ts b/dev-packages/browser-integration-tests/suites/public-api/instrumentation/eventListener/thrown-error/test.ts
@@ -15,10 +15,9 @@ sentryTest(
       type: 'Error',
       value: 'event_listener_error',
       mechanism: {
-        type: 'browserApiErrors',
+        type: 'auto.browser.browserapierrors.addEventListener',
         handled: false,
         data: {
-          function: 'addEventListener',
           handler: '<anonymous>',
           target: 'EventTarget',
         },
diff --git a/dev-packages/browser-integration-tests/suites/public-api/instrumentation/requestAnimationFrame/thrown-errors/test.ts b/dev-packages/browser-integration-tests/suites/public-api/instrumentation/requestAnimationFrame/thrown-errors/test.ts
@@ -13,7 +13,7 @@ sentryTest('should capture exceptions inside callback', async ({ getLocalTestUrl
     type: 'Error',
     value: 'requestAnimationFrame_error',
     mechanism: {
-      type: 'browserApiErrors',
+      type: 'auto.browser.browserapierrors.requestAnimationFrame',
       handled: false,
     },
     stacktrace: {
diff --git a/dev-packages/browser-integration-tests/suites/public-api/instrumentation/setInterval/test.ts b/dev-packages/browser-integration-tests/suites/public-api/instrumentation/setInterval/test.ts
@@ -13,7 +13,7 @@ sentryTest('Instrumentation should capture errors in setInterval', async ({ getL
     type: 'Error',
     value: 'setInterval_error',
     mechanism: {
-      type: 'browserApiErrors',
+      type: 'auto.browser.browserapierrors.setInterval',
       handled: false,
     },
     stacktrace: {
diff --git a/dev-packages/browser-integration-tests/suites/public-api/instrumentation/setTimeout/test.ts b/dev-packages/browser-integration-tests/suites/public-api/instrumentation/setTimeout/test.ts
@@ -13,11 +13,8 @@ sentryTest('Instrumentation should capture errors in setTimeout', async ({ getLo
     type: 'Error',
     value: 'setTimeout_error',
     mechanism: {
-      type: 'browserApiErrors',
+      type: 'auto.browser.browserapierrors.setTimeout',
       handled: false,
-      data: {
-        function: 'setTimeout',
-      },
     },
     stacktrace: {
       frames: expect.any(Array),
diff --git a/dev-packages/browser-integration-tests/suites/public-api/instrumentation/setTimeoutFrozen/test.ts b/dev-packages/browser-integration-tests/suites/public-api/instrumentation/setTimeoutFrozen/test.ts
@@ -25,7 +25,7 @@ sentryTest(
       type: 'Error',
       value: 'setTimeout_error',
       mechanism: {
-        type: 'browserApiErrors',
+        type: 'auto.browser.browserapierrors.setTimeout',
         handled: false,
       },
       stacktrace: {
diff --git a/dev-packages/browser-integration-tests/suites/public-api/instrumentation/xhr/thrown-error/test.ts b/dev-packages/browser-integration-tests/suites/public-api/instrumentation/xhr/thrown-error/test.ts
@@ -15,11 +15,8 @@ sentryTest(
       type: 'Error',
       value: 'xhr_error',
       mechanism: {
-        type: 'browserApiErrors',
+        type: 'auto.browser.browserapierrors.xhr.onreadystatechange',
         handled: false,
-        data: {
-          function: 'onreadystatechange',
-        },
       },
       stacktrace: {
         frames: expect.any(Array),
diff --git a/dev-packages/e2e-tests/test-applications/ember-classic/tests/errors.test.ts b/dev-packages/e2e-tests/test-applications/ember-classic/tests/errors.test.ts
@@ -19,7 +19,7 @@ test('sends an error', async ({ page }) => {
           type: 'TypeError',
           value: 'this.nonExistentFunction is not a function',
           mechanism: {
-            type: 'browserApiErrors',
+            type: 'auto.browser.browserapierrors.addEventListener',
             handled: false,
           },
         },
@@ -55,7 +55,7 @@ test('assigns the correct transaction value after a navigation', async ({ page }
           type: 'TypeError',
           value: 'this.nonExistentFunction is not a function',
           mechanism: {
-            type: 'browserApiErrors',
+            type: 'auto.browser.browserapierrors.addEventListener',
             handled: false,
           },
         },
diff --git a/dev-packages/e2e-tests/test-applications/svelte-5/tests/errors.test.ts b/dev-packages/e2e-tests/test-applications/svelte-5/tests/errors.test.ts
@@ -19,7 +19,7 @@ test('sends an error', async ({ page }) => {
           type: 'Error',
           value: 'Error thrown from Svelte 5 E2E test app',
           mechanism: {
-            type: 'browserApiErrors',
+            type: 'auto.browser.browserapierrors.addEventListener',
             handled: false,
           },
         },
diff --git a/packages/browser/src/integrations/browserapierrors.ts b/packages/browser/src/integrations/browserapierrors.ts
@@ -108,9 +108,8 @@ function _wrapTimeFunction(original: () => void): () => number {
     const originalCallback = args[0];
     args[0] = wrap(originalCallback, {
       mechanism: {
-        data: { function: getFunctionName(original) },
         handled: false,
-        type: 'browserApiErrors',
+        type: `auto.browser.browserapierrors.${getFunctionName(original)}`,
       },
     });
     return original.apply(this, args);
@@ -123,11 +122,10 @@ function _wrapRAF(original: () => void): (callback: () => void) => unknown {
       wrap(callback, {
         mechanism: {
           data: {
-            function: 'requestAnimationFrame',
             handler: getFunctionName(original),
           },
           handled: false,
-          type: 'browserApiErrors',
+          type: 'auto.browser.browserapierrors.requestAnimationFrame',
         },
       }),
     ]);
@@ -146,11 +144,10 @@ function _wrapXHR(originalSend: () => void): () => void {
           const wrapOptions = {
             mechanism: {
               data: {
-                function: prop,
                 handler: getFunctionName(original),
               },
               handled: false,
-              type: 'browserApiErrors',
+              type: `auto.browser.browserapierrors.xhr.${prop}`,
             },
           };
 
@@ -194,12 +191,11 @@ function _wrapEventTarget(target: string, integrationOptions: BrowserApiErrorsOp
           fn.handleEvent = wrap(fn.handleEvent, {
             mechanism: {
               data: {
-                function: 'handleEvent',
                 handler: getFunctionName(fn),
                 target,
               },
               handled: false,
-              type: 'browserApiErrors',
+              type: 'auto.browser.browserapierrors.handleEvent',
             },
           });
         }
@@ -216,12 +212,11 @@ function _wrapEventTarget(target: string, integrationOptions: BrowserApiErrorsOp
         wrap(fn, {
           mechanism: {
             data: {
-              function: 'addEventListener',
               handler: getFunctionName(fn),
               target,
             },
             handled: false,
-            type: 'browserApiErrors',
+            type: 'auto.browser.browserapierrors.addEventListener',
           },
         }),
         options,
diff --git a/packages/remix/test/integration/test/client/click-error.test.ts b/packages/remix/test/integration/test/client/click-error.test.ts
@@ -20,7 +20,7 @@ test('should report a manually captured message on click with the correct stackt
       type: 'Error',
       value: 'ClickError',
       stacktrace: { frames: expect.any(Array) },
-      mechanism: { type: 'browserApiErrors', handled: false },
+      mechanism: { type: 'auto.browser.browserapierrors.addEventListener', handled: false },
     },
   ]);
 
