feat(core): Deprecate addTracingExtensions (#11579)

Instead, users can use `registerSpanErrorInstrumentation` which is now
does much less things than before.

I opted to not fully remove this yet because I guess the chance that
somebody may have been using this is somewhat high, and we can easily
continue to have it with no bundle size hit for those who don't need it.
We also couldn't really deprecate this in v7 so I think it's OK to leave
this in for now.

Author: mydea

MIGRATION.md

@@ -65,7 +65,7 @@ We've removed the following packages:
 `@sentry/tracing` has been removed and will no longer be published. See
 [below](./MIGRATION.md/#3-removal-of-deprecated-apis) for more details.
 
-For Browser SDKs you can import `BrowserTracing` from the SDK directly:
+For Browser SDKs you can import `browserTracingIntegration` from the SDK directly:
 
 ```js
 // v7
@@ -86,12 +86,13 @@ import * as Sentry from '@sentry/browser';
 Sentry.init({
   dsn: '__DSN__',
   tracesSampleRate: 1.0,
-  integrations: [new Sentry.BrowserTracing()],
+  integrations: [Sentry.browserTracingIntegration()],
 });
 ```
 
-If you were importing `@sentry/tracing` for the side effect, you can now use `Sentry.addTracingExtensions()` to add the
-tracing extensions to the SDK. `addTracingExtensions` replaces the `addExtensionMethods` method from `@sentry/tracing`.
+If you don't want to use `browserTracingIntegration` but still manually start spans, you can now use
+`Sentry.registerSpanErrorInstrumentation()` to setup handlers for span instrumentation.
+`registerSpanErrorInstrumentation` replaces the `addExtensionMethods` method from `@sentry/tracing`.
 
 ```js
 // v7
@@ -108,7 +109,7 @@ Sentry.init({
 // v8
 import * as Sentry from '@sentry/browser';
 
-Sentry.addTracingExtensions();
+Sentry.registerSpanErrorInstrumentation();
 
 Sentry.init({
   dsn: '__DSN__',

dev-packages/browser-integration-tests/suites/public-api/startSpan/init.js

@@ -2,8 +2,6 @@ import * as Sentry from '@sentry/browser';
 
 window.Sentry = Sentry;
 
-Sentry.addTracingExtensions();
-
 Sentry.init({
   dsn: 'https://[email protected]/1337',
   tracesSampleRate: 1.0,

packages/browser-utils/test/browser/utils.test.ts

@@ -1,18 +1,9 @@
-import {
-  SentrySpan,
-  addTracingExtensions,
-  getCurrentScope,
-  getIsolationScope,
-  setCurrentClient,
-  spanToJSON,
-} from '@sentry/core';
+import { SentrySpan, getCurrentScope, getIsolationScope, setCurrentClient, spanToJSON } from '@sentry/core';
 import { startAndEndSpan } from '../../src/metrics/utils';
 import { TestClient, getDefaultClientOptions } from '../utils/TestClient';
 
 describe('startAndEndSpan()', () => {
   beforeEach(() => {
-    addTracingExtensions();
-
     getCurrentScope().clear();
     getIsolationScope().clear();
 

packages/browser/src/index.bundle.feedback.ts

@@ -1,9 +1,4 @@
-// This is exported so the loader does not fail when switching off Replay/Tracing
-import {
-  addTracingExtensionsShim,
-  browserTracingIntegrationShim,
-  replayIntegrationShim,
-} from '@sentry-internal/integration-shims';
+import { browserTracingIntegrationShim, replayIntegrationShim } from '@sentry-internal/integration-shims';
 
 export * from './index.bundle.base';
 
@@ -14,8 +9,4 @@ export {
   getFeedback,
 } from '@sentry-internal/feedback';
 
-export {
-  browserTracingIntegrationShim as browserTracingIntegration,
-  addTracingExtensionsShim as addTracingExtensions,
-  replayIntegrationShim as replayIntegration,
-};
+export { browserTracingIntegrationShim as browserTracingIntegration, replayIntegrationShim as replayIntegration };

packages/browser/src/index.bundle.replay.ts

@@ -1,6 +1,4 @@
-// This is exported so the loader does not fail when switching off Replay/Tracing
 import {
-  addTracingExtensionsShim,
   browserTracingIntegrationShim,
   feedbackIntegrationShim,
   feedbackModalIntegrationShim,
@@ -13,7 +11,6 @@ export { replayIntegration } from '@sentry-internal/replay';
 
 export {
   browserTracingIntegrationShim as browserTracingIntegration,
-  addTracingExtensionsShim as addTracingExtensions,
   feedbackIntegrationShim as feedbackIntegration,
   feedbackModalIntegrationShim as feedbackModalIntegration,
   feedbackScreenshotIntegrationShim as feedbackScreenshotIntegration,

packages/browser/src/index.bundle.tracing.replay.feedback.ts

@@ -1,7 +1,6 @@
-import { addTracingExtensions } from '@sentry/core';
+import { registerSpanErrorInstrumentation } from '@sentry/core';
 
-// We are patching the global object with our hub extension methods
-addTracingExtensions();
+registerSpanErrorInstrumentation();
 
 export * from './index.bundle.base';
 
@@ -14,7 +13,6 @@ export {
   withActiveSpan,
   getSpanDescendants,
   setMeasurement,
-  addTracingExtensions,
 } from '@sentry/core';
 
 export {

packages/browser/src/index.bundle.tracing.replay.ts

@@ -3,10 +3,9 @@ import {
   feedbackModalIntegrationShim,
   feedbackScreenshotIntegrationShim,
 } from '@sentry-internal/integration-shims';
-import { addTracingExtensions } from '@sentry/core';
+import { registerSpanErrorInstrumentation } from '@sentry/core';
 
-// We are patching the global object with our hub extension methods
-addTracingExtensions();
+registerSpanErrorInstrumentation();
 
 export * from './index.bundle.base';
 
@@ -19,7 +18,6 @@ export {
   withActiveSpan,
   getSpanDescendants,
   setMeasurement,
-  addTracingExtensions,
 } from '@sentry/core';
 
 export {

packages/browser/src/index.bundle.tracing.ts

@@ -1,14 +1,12 @@
-// This is exported so the loader does not fail when switching off Replay
 import {
   feedbackIntegrationShim,
   feedbackModalIntegrationShim,
   feedbackScreenshotIntegrationShim,
   replayIntegrationShim,
 } from '@sentry-internal/integration-shims';
-import { addTracingExtensions } from '@sentry/core';
+import { registerSpanErrorInstrumentation } from '@sentry/core';
 
-// We are patching the global object with our hub extension methods
-addTracingExtensions();
+registerSpanErrorInstrumentation();
 
 export * from './index.bundle.base';
 
@@ -21,7 +19,6 @@ export {
   withActiveSpan,
   getSpanDescendants,
   setMeasurement,
-  addTracingExtensions,
 } from '@sentry/core';
 
 export {

packages/browser/src/index.bundle.ts

@@ -1,6 +1,4 @@
-// This is exported so the loader does not fail when switching off Replay/Tracing
 import {
-  addTracingExtensionsShim,
   browserTracingIntegrationShim,
   feedbackIntegrationShim,
   feedbackModalIntegrationShim,
@@ -11,7 +9,6 @@ import {
 export * from './index.bundle.base';
 
 export {
-  addTracingExtensionsShim as addTracingExtensions,
   browserTracingIntegrationShim as browserTracingIntegration,
   feedbackIntegrationShim as feedbackIntegration,
   feedbackModalIntegrationShim as feedbackModalIntegration,

packages/browser/src/index.ts

@@ -49,7 +49,9 @@ export {
 } from './tracing/browserTracingIntegration';
 export type { RequestInstrumentationOptions } from './tracing/request';
 export {
+  // eslint-disable-next-line deprecation/deprecation
   addTracingExtensions,
+  registerSpanErrorInstrumentation,
   getActiveSpan,
   getRootSpan,
   startSpan,