feat(v8): Remove all class based integrations (#11345)

- Removes all class based integrations from `tracing-internal` and
re-organizes exports
- Removes top level `Integrations` export from browser package

resolves #8844

Author: AbhiPrasad

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

@@ -1,21 +1 @@
-import type { IntegrationFn } from '@sentry/types/src';
-
 export * from './exports';
-
-import type { Integration } from '@sentry/types';
-
-import { WINDOW } from './helpers';
-
-let windowIntegrations = {};
-
-// This block is needed to add compatibility with the integrations packages when used with a CDN
-if (WINDOW.Sentry && WINDOW.Sentry.Integrations) {
-  windowIntegrations = WINDOW.Sentry.Integrations;
-}
-
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-const INTEGRATIONS: Record<string, (new (...args: any[]) => Integration) | IntegrationFn> = {
-  ...windowIntegrations,
-};
-
-export { INTEGRATIONS as Integrations };

packages/browser/src/index.ts

@@ -1,22 +1,5 @@
 export * from './exports';
 
-import { WINDOW } from './helpers';
-
-let windowIntegrations = {};
-
-// This block is needed to add compatibility with the integrations packages when used with a CDN
-if (WINDOW.Sentry && WINDOW.Sentry.Integrations) {
-  windowIntegrations = WINDOW.Sentry.Integrations;
-}
-
-/** @deprecated Import the integration function directly, e.g. `inboundFiltersIntegration()` instead of `new Integrations.InboundFilter(). */
-const INTEGRATIONS = {
-  ...windowIntegrations,
-};
-
-// eslint-disable-next-line deprecation/deprecation
-export { INTEGRATIONS as Integrations };
-
 export { reportingObserverIntegration } from './integrations/reportingobserver';
 export { httpClientIntegration } from './integrations/httpclient';
 export { contextLinesIntegration } from './integrations/contextlines';

packages/browser/test/unit/index.bundle.feedback.test.ts

@@ -5,10 +5,6 @@ import * as TracingReplayBundle from '../../src/index.bundle.feedback';
 
 describe('index.bundle.feedback', () => {
   it('has correct exports', () => {
-    Object.keys(TracingReplayBundle.Integrations).forEach(key => {
-      expect((TracingReplayBundle.Integrations[key] as any).id).toStrictEqual(expect.any(String));
-    });
-
     expect(TracingReplayBundle.replayIntegration).toBe(replayIntegrationShim);
     expect(TracingReplayBundle.feedbackIntegration).toBe(feedbackIntegration);
   });

packages/browser/test/unit/index.bundle.replay.test.ts

@@ -5,10 +5,6 @@ import * as TracingReplayBundle from '../../src/index.bundle.replay';
 
 describe('index.bundle.replay', () => {
   it('has correct exports', () => {
-    Object.keys(TracingReplayBundle.Integrations).forEach(key => {
-      expect((TracingReplayBundle.Integrations[key] as any).id).toStrictEqual(expect.any(String));
-    });
-
     expect(TracingReplayBundle.replayIntegration).toBe(replayIntegration);
     expect(TracingReplayBundle.feedbackIntegration).toBe(feedbackIntegrationShim);
   });

packages/browser/test/unit/index.bundle.test.ts

@@ -4,10 +4,6 @@ import * as TracingBundle from '../../src/index.bundle';
 
 describe('index.bundle', () => {
   it('has correct exports', () => {
-    Object.keys(TracingBundle.Integrations).forEach(key => {
-      expect((TracingBundle.Integrations[key] as any).name).toStrictEqual(expect.any(String));
-    });
-
     expect(TracingBundle.replayIntegration).toBe(replayIntegrationShim);
     expect(TracingBundle.feedbackIntegration).toBe(feedbackIntegrationShim);
   });

packages/browser/test/unit/index.bundle.tracing.replay.feedback.test.ts

@@ -5,10 +5,6 @@ import * as TracingReplayFeedbackBundle from '../../src/index.bundle.tracing.rep
 
 describe('index.bundle.tracing.replay.feedback', () => {
   it('has correct exports', () => {
-    Object.keys(TracingReplayFeedbackBundle.Integrations).forEach(key => {
-      expect((TracingReplayFeedbackBundle.Integrations[key] as any).id).toStrictEqual(expect.any(String));
-    });
-
     expect(TracingReplayFeedbackBundle.replayIntegration).toBe(replayIntegration);
     expect(TracingReplayFeedbackBundle.browserTracingIntegration).toBe(browserTracingIntegration);
     expect(TracingReplayFeedbackBundle.feedbackIntegration).toBe(feedbackIntegration);

packages/browser/test/unit/index.bundle.tracing.replay.test.ts

@@ -6,10 +6,6 @@ import * as TracingReplayBundle from '../../src/index.bundle.tracing.replay';
 
 describe('index.bundle.tracing.replay', () => {
   it('has correct exports', () => {
-    Object.keys(TracingReplayBundle.Integrations).forEach(key => {
-      expect((TracingReplayBundle.Integrations[key] as any).id).toStrictEqual(expect.any(String));
-    });
-
     expect(TracingReplayBundle.replayIntegration).toBe(replayIntegration);
 
     expect(TracingReplayBundle.browserTracingIntegration).toBe(browserTracingIntegration);

packages/browser/test/unit/index.bundle.tracing.test.ts

@@ -5,10 +5,6 @@ import * as TracingBundle from '../../src/index.bundle.tracing';
 
 describe('index.bundle.tracing', () => {
   it('has correct exports', () => {
-    Object.keys(TracingBundle.Integrations).forEach(key => {
-      expect((TracingBundle.Integrations[key] as any).id).toStrictEqual(expect.any(String));
-    });
-
     expect(TracingBundle.replayIntegration).toBe(replayIntegrationShim);
     expect(TracingBundle.browserTracingIntegration).toBe(browserTracingIntegration);
     expect(TracingBundle.feedbackIntegration).toBe(feedbackIntegrationShim);

packages/nextjs/src/index.types.ts

@@ -23,8 +23,6 @@ export declare const getClient: typeof clientSdk.getClient;
 export declare const getRootSpan: typeof serverSdk.getRootSpan;
 export declare const continueTrace: typeof clientSdk.continueTrace;
 
-export declare const Integrations: undefined; // TODO(v8): Remove this line. Can only be done when dependencies don't export `Integrations` anymore.
-
 export declare const linkedErrorsIntegration: typeof clientSdk.linkedErrorsIntegration;
 export declare const contextLinesIntegration: typeof clientSdk.contextLinesIntegration;
 

packages/remix/src/index.types.ts

@@ -12,10 +12,6 @@ import type { RemixOptions } from './utils/remixOptions';
 /** Initializes Sentry Remix SDK */
 export declare function init(options: RemixOptions): void;
 
-// We export a merged Integrations object so that users can (at least typing-wise) use all integrations everywhere.
-// eslint-disable-next-line deprecation/deprecation
-export declare const Integrations: typeof clientSdk.Integrations;
-
 export declare const linkedErrorsIntegration: typeof clientSdk.linkedErrorsIntegration;
 export declare const contextLinesIntegration: typeof clientSdk.contextLinesIntegration;