ref(browser): Use new addInstrumentationHandler

Author: AbhiPrasad

packages/browser/src/integrations/breadcrumbs.ts

@@ -84,34 +84,19 @@ export class Breadcrumbs implements Integration {
    */
   public setupOnce(): void {
     if (this._options.console) {
-      addInstrumentationHandler({
-        callback: _consoleBreadcrumb,
-        type: 'console',
-      });
+      addInstrumentationHandler('console', _consoleBreadcrumb);
     }
     if (this._options.dom) {
-      addInstrumentationHandler({
-        callback: _domBreadcrumb(this._options.dom),
-        type: 'dom',
-      });
+      addInstrumentationHandler('dom', _domBreadcrumb(this._options.dom));
     }
     if (this._options.xhr) {
-      addInstrumentationHandler({
-        callback: _xhrBreadcrumb,
-        type: 'xhr',
-      });
+      addInstrumentationHandler('xhr', _xhrBreadcrumb);
     }
     if (this._options.fetch) {
-      addInstrumentationHandler({
-        callback: _fetchBreadcrumb,
-        type: 'fetch',
-      });
+      addInstrumentationHandler('fetch', _fetchBreadcrumb);
     }
     if (this._options.history) {
-      addInstrumentationHandler({
-        callback: _historyBreadcrumb,
-        type: 'history',
-      });
+      addInstrumentationHandler('history', _historyBreadcrumb);
     }
   }
 }

packages/browser/src/integrations/globalhandlers.ts

@@ -74,9 +74,10 @@ export class GlobalHandlers implements Integration {
 
 /** JSDoc */
 function _installGlobalOnErrorHandler(): void {
-  addInstrumentationHandler({
+  addInstrumentationHandler(
+    'error',
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    callback: (data: { msg: any; url: any; line: any; column: any; error: any }) => {
+    (data: { msg: any; url: any; line: any; column: any; error: any }) => {
       const [hub, attachStacktrace] = getHubAndAttachStacktrace();
       if (!hub.getIntegration(GlobalHandlers)) {
         return;
@@ -101,15 +102,15 @@ function _installGlobalOnErrorHandler(): void {
 
       addMechanismAndCapture(hub, error, event, 'onerror');
     },
-    type: 'error',
-  });
+  );
 }
 
 /** JSDoc */
 function _installGlobalOnUnhandledRejectionHandler(): void {
-  addInstrumentationHandler({
+  addInstrumentationHandler(
+    'unhandledrejection',
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    callback: (e: any) => {
+    (e: any) => {
       const [hub, attachStacktrace] = getHubAndAttachStacktrace();
       if (!hub.getIntegration(GlobalHandlers)) {
         return;
@@ -151,8 +152,7 @@ function _installGlobalOnUnhandledRejectionHandler(): void {
       addMechanismAndCapture(hub, error, event, 'onunhandledrejection');
       return;
     },
-    type: 'unhandledrejection',
-  });
+  );
 }
 
 /**

packages/browser/src/sdk.ts

@@ -226,15 +226,12 @@ function startSessionTracking(): void {
   hub.captureSession();
 
   // We want to create a session for every navigation as well
-  addInstrumentationHandler({
-    callback: ({ from, to }) => {
-      // Don't create an additional session for the initial route or if the location did not change
-      if (from === undefined || from === to) {
-        return;
-      }
-      hub.startSession({ ignoreDuration: true });
-      hub.captureSession();
-    },
-    type: 'history',
+  addInstrumentationHandler('history', ({ from, to }) => {
+    // Don't create an additional session for the initial route or if the location did not change
+    if (from === undefined || from === to) {
+      return;
+    }
+    hub.startSession({ ignoreDuration: true });
+    hub.captureSession();
   });
 }