ref(v7/replay): Deprecate @sentry/replay exports (#11242)

ref #11200
on v8 #11243

Author: s1gr1d

dev-packages/browser-integration-tests/suites/replay/bufferMode/test.ts

@@ -1,5 +1,5 @@
 import { expect } from '@playwright/test';
-import type { Replay } from '@sentry/replay';
+import type { InternalReplay } from '@sentry/replay';
 import type { ReplayContainer } from '@sentry/replay/build/npm/types/types';
 
 import { sentryTest } from '../../../utils/fixtures';
@@ -68,7 +68,8 @@ sentryTest(
     expect(
       await page.evaluate(() => {
         // eslint-disable-next-line deprecation/deprecation
-        const replayIntegration = (window as unknown as Window & { Replay: InstanceType<typeof Replay> }).Replay;
+        const replayIntegration = (window as unknown as Window & { Replay: InstanceType<typeof InternalReplay> })
+          .Replay;
         // @ts-expect-error private
         const replay = replayIntegration._replay;
         replayIntegration.startBuffering();
@@ -89,7 +90,7 @@ sentryTest(
       reqPromise0,
       page.evaluate(async () => {
         // eslint-disable-next-line deprecation/deprecation
-        const replayIntegration = (window as unknown as Window & { Replay: Replay }).Replay;
+        const replayIntegration = (window as unknown as Window & { Replay: InternalReplay }).Replay;
         await replayIntegration.flush();
       }),
     ]);
@@ -213,7 +214,8 @@ sentryTest(
     expect(
       await page.evaluate(() => {
         // eslint-disable-next-line deprecation/deprecation
-        const replayIntegration = (window as unknown as Window & { Replay: InstanceType<typeof Replay> }).Replay;
+        const replayIntegration = (window as unknown as Window & { Replay: InstanceType<typeof InternalReplay> })
+          .Replay;
         // @ts-expect-error private
         const replay = replayIntegration._replay;
         replayIntegration.startBuffering();
@@ -234,7 +236,7 @@ sentryTest(
       reqPromise0,
       page.evaluate(async () => {
         // eslint-disable-next-line deprecation/deprecation
-        const replayIntegration = (window as unknown as Window & { Replay: Replay }).Replay;
+        const replayIntegration = (window as unknown as Window & { Replay: InternalReplay }).Replay;
         await replayIntegration.flush({ continueRecording: false });
       }),
     ]);
@@ -329,7 +331,8 @@ sentryTest(
     expect(
       await page.evaluate(() => {
         // eslint-disable-next-line deprecation/deprecation
-        const replayIntegration = (window as unknown as Window & { Replay: InstanceType<typeof Replay> }).Replay;
+        const replayIntegration = (window as unknown as Window & { Replay: InstanceType<typeof InternalReplay> })
+          .Replay;
         const replay = replayIntegration['_replay'];
         replayIntegration.startBuffering();
         return replay.isEnabled();
@@ -348,7 +351,7 @@ sentryTest(
 
     await page.evaluate(async () => {
       // eslint-disable-next-line deprecation/deprecation
-      const replayIntegration = (window as unknown as Window & { Replay: Replay }).Replay;
+      const replayIntegration = (window as unknown as Window & { Replay: InternalReplay }).Replay;
       replayIntegration['_replay'].getOptions().errorSampleRate = 1.0;
     });
 

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

@@ -6,13 +6,13 @@ import {
   browserTracingIntegration,
   feedbackIntegration,
 } from '@sentry-internal/integration-shims';
-import { Replay, replayIntegration } from '@sentry/replay';
+import { InternalReplay, internalReplayIntegration } from '@sentry/replay';
 
 import * as Sentry from './index.bundle.base';
 
 // TODO (v8): Remove this as it was only needed for backwards compatibility
 // eslint-disable-next-line deprecation/deprecation
-Sentry.Integrations.Replay = Replay;
+Sentry.Integrations.Replay = InternalReplay;
 
 // eslint-disable-next-line deprecation/deprecation
 Sentry.Integrations.BrowserTracing = BrowserTracing;
@@ -24,8 +24,8 @@ export {
   browserTracingIntegration,
   addTracingExtensions,
   // eslint-disable-next-line deprecation/deprecation
-  Replay,
-  replayIntegration,
+  InternalReplay as Replay,
+  internalReplayIntegration as replayIntegration,
   // eslint-disable-next-line deprecation/deprecation
   Feedback,
   feedbackIntegration,

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

@@ -1,6 +1,6 @@
 import { Feedback, feedbackIntegration } from '@sentry-internal/feedback';
 import { BrowserTracing, Span, addExtensionMethods } from '@sentry-internal/tracing';
-import { Replay, replayIntegration } from '@sentry/replay';
+import { InternalReplay, internalReplayIntegration } from '@sentry/replay';
 import { bundleBrowserTracingIntegration as browserTracingIntegration } from './helpers';
 
 import * as Sentry from './index.bundle.base';
@@ -9,7 +9,7 @@ import * as Sentry from './index.bundle.base';
 // We want replay to be available under Sentry.Replay, to be consistent
 // with the NPM package version.
 // eslint-disable-next-line deprecation/deprecation
-Sentry.Integrations.Replay = Replay;
+Sentry.Integrations.Replay = InternalReplay;
 
 // eslint-disable-next-line deprecation/deprecation
 Sentry.Integrations.BrowserTracing = BrowserTracing;
@@ -21,9 +21,9 @@ export {
   // eslint-disable-next-line deprecation/deprecation
   Feedback,
   // eslint-disable-next-line deprecation/deprecation
-  Replay,
+  InternalReplay as Replay,
   feedbackIntegration,
-  replayIntegration,
+  internalReplayIntegration as replayIntegration,
   // eslint-disable-next-line deprecation/deprecation
   BrowserTracing,
   browserTracingIntegration,

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

@@ -1,6 +1,6 @@
 import { Feedback, feedbackIntegration } from '@sentry-internal/integration-shims';
 import { BrowserTracing, Span, addExtensionMethods } from '@sentry-internal/tracing';
-import { Replay, replayIntegration } from '@sentry/replay';
+import { InternalReplay, internalReplayIntegration } from '@sentry/replay';
 import { bundleBrowserTracingIntegration as browserTracingIntegration } from './helpers';
 
 import * as Sentry from './index.bundle.base';
@@ -9,7 +9,7 @@ import * as Sentry from './index.bundle.base';
 // We want replay to be available under Sentry.Replay, to be consistent
 // with the NPM package version.
 // eslint-disable-next-line deprecation/deprecation
-Sentry.Integrations.Replay = Replay;
+Sentry.Integrations.Replay = InternalReplay;
 
 // eslint-disable-next-line deprecation/deprecation
 Sentry.Integrations.BrowserTracing = BrowserTracing;
@@ -21,8 +21,8 @@ export {
   // eslint-disable-next-line deprecation/deprecation
   Feedback,
   // eslint-disable-next-line deprecation/deprecation
-  Replay,
-  replayIntegration,
+  InternalReplay as Replay,
+  internalReplayIntegration as replayIntegration,
   feedbackIntegration,
   // eslint-disable-next-line deprecation/deprecation
   BrowserTracing,

packages/browser/src/index.ts

@@ -25,20 +25,20 @@ export { INTEGRATIONS as Integrations };
 
 export {
   // eslint-disable-next-line deprecation/deprecation
-  Replay,
-  replayIntegration,
-  getReplay,
+  InternalReplay as Replay,
+  internalReplayIntegration as replayIntegration,
+  internalGetReplay as getReplay,
 } from '@sentry/replay';
 export type {
-  ReplayEventType,
-  ReplayEventWithTime,
-  ReplayBreadcrumbFrame,
-  ReplayBreadcrumbFrameEvent,
-  ReplayOptionFrameEvent,
-  ReplayFrame,
-  ReplayFrameEvent,
-  ReplaySpanFrame,
-  ReplaySpanFrameEvent,
+  InternalReplayEventType as ReplayEventType,
+  InternalReplayEventWithTime as ReplayEventWithTime,
+  InternalReplayBreadcrumbFrame as ReplayBreadcrumbFrame,
+  InternalReplayBreadcrumbFrameEvent as ReplayBreadcrumbFrameEvent,
+  InternalReplayOptionFrameEvent as ReplayOptionFrameEvent,
+  InternalReplayFrame as ReplayFrame,
+  InternalReplayFrameEvent as ReplayFrameEvent,
+  InternalReplaySpanFrame as ReplaySpanFrame,
+  InternalReplaySpanFrameEvent as ReplaySpanFrameEvent,
 } from '@sentry/replay';
 
 export {

packages/replay-canvas/src/canvas.ts

@@ -1,14 +1,14 @@
 import { CanvasManager } from '@sentry-internal/rrweb';
 import { convertIntegrationFnToClass, defineIntegration } from '@sentry/core';
-import type { CanvasManagerInterface, CanvasManagerOptions } from '@sentry/replay';
+import type { InternalCanvasManagerInterface, InternalCanvasManagerOptions } from '@sentry/replay';
 import type { Integration, IntegrationClass, IntegrationFn } from '@sentry/types';
 
 interface ReplayCanvasOptions {
   enableManualSnapshot?: boolean;
   quality: 'low' | 'medium' | 'high';
 }
 
-type GetCanvasManager = (options: CanvasManagerOptions) => CanvasManagerInterface;
+type GetCanvasManager = (options: InternalCanvasManagerOptions) => InternalCanvasManagerInterface;
 export interface ReplayCanvasIntegrationOptions {
   enableManualSnapshot?: boolean;
   recordCanvas: true;
@@ -74,7 +74,7 @@ export const _replayCanvasIntegration = ((options: Partial<ReplayCanvasOptions>
       return {
         enableManualSnapshot,
         recordCanvas: true,
-        getCanvasManager: (options: CanvasManagerOptions) => {
+        getCanvasManager: (options: InternalCanvasManagerOptions) => {
           const manager = new CanvasManager({
             ...options,
             enableManualSnapshot,

packages/replay/src/index.ts

@@ -1,25 +1,108 @@
-export {
-  // eslint-disable-next-line deprecation/deprecation
-  Replay,
-  replayIntegration,
-} from './integration';
+// eslint-disable-next-line deprecation/deprecation
+import { Replay as InternalReplay, replayIntegration as internalReplayIntegration } from './integration';
+import type {
+  CanvasManagerInterface as InternalCanvasManagerInterface,
+  CanvasManagerOptions as InternalCanvasManagerOptions,
+  ReplayBreadcrumbFrame as InternalReplayBreadcrumbFrame,
+  ReplayBreadcrumbFrameEvent as InternalReplayBreadcrumbFrameEvent,
+  ReplayConfiguration as InternalReplayConfiguration,
+  ReplayEventType as InternalReplayEventType,
+  ReplayEventWithTime as InternalReplayEventWithTime,
+  ReplayFrame as InternalReplayFrame,
+  ReplayFrameEvent as InternalReplayFrameEvent,
+  ReplayOptionFrameEvent as InternalReplayOptionFrameEvent,
+  ReplaySpanFrame as InternalReplaySpanFrame,
+  ReplaySpanFrameEvent as InternalReplaySpanFrameEvent,
+} from './types';
+import { getReplay as internalGetReplay } from './util/getReplay';
+
+/** @deprecated Use the export from `@sentry/replay` or from framework-specific SDKs like `@sentry/react` or `@sentry/vue` */
+const getReplay = internalGetReplay;
+
+/** @deprecated Use the export from `@sentry/replay` or from framework-specific SDKs like `@sentry/react` or `@sentry/vue` */
+const replayIntegration = internalReplayIntegration;
+
+/** @deprecated Use the export from `@sentry/replay` or from framework-specific SDKs like `@sentry/react` or `@sentry/vue` */
+// eslint-disable-next-line deprecation/deprecation
+class Replay extends InternalReplay {}
+
+// eslint-disable-next-line deprecation/deprecation
+export { replayIntegration, getReplay, Replay, internalReplayIntegration, internalGetReplay, InternalReplay };
+
+/** @deprecated Use the export from `@sentry/replay` or from framework-specific SDKs like `@sentry/react` or `@sentry/vue` */
+type ReplayConfiguration = InternalReplayConfiguration;
+
+/** @deprecated Use the export from `@sentry/replay` or from framework-specific SDKs like `@sentry/react` or `@sentry/vue` */
+type ReplayEventType = InternalReplayEventType;
+
+/** @deprecated Use the export from `@sentry/replay` or from framework-specific SDKs like `@sentry/react` or `@sentry/vue` */
+type ReplayEventWithTime = InternalReplayEventWithTime;
+
+/** @deprecated Use the export from `@sentry/replay` or from framework-specific SDKs like `@sentry/react` or `@sentry/vue` */
+type ReplayBreadcrumbFrame = InternalReplayBreadcrumbFrame;
+
+/** @deprecated Use the export from `@sentry/replay` or from framework-specific SDKs like `@sentry/react` or `@sentry/vue` */
+type ReplayBreadcrumbFrameEvent = InternalReplayBreadcrumbFrameEvent;
+
+/** @deprecated Use the export from `@sentry/replay` or from framework-specific SDKs like `@sentry/react` or `@sentry/vue` */
+type ReplayOptionFrameEvent = InternalReplayOptionFrameEvent;
+
+/** @deprecated Use the export from `@sentry/replay` or from framework-specific SDKs like `@sentry/react` or `@sentry/vue` */
+type ReplayFrame = InternalReplayFrame;
+
+/** @deprecated Use the export from `@sentry/replay` or from framework-specific SDKs like `@sentry/react` or `@sentry/vue` */
+type ReplayFrameEvent = InternalReplayFrameEvent;
+
+/** @deprecated Use the export from `@sentry/replay` or from framework-specific SDKs like `@sentry/react` or `@sentry/vue` */
+type ReplaySpanFrame = InternalReplaySpanFrame;
+
+/** @deprecated Use the export from `@sentry/replay` or from framework-specific SDKs like `@sentry/react` or `@sentry/vue` */
+type ReplaySpanFrameEvent = InternalReplaySpanFrameEvent;
+
+/** @deprecated Use the export from `@sentry/replay` or from framework-specific SDKs like `@sentry/react` or `@sentry/vue` */
+type CanvasManagerInterface = InternalCanvasManagerInterface;
+
+/** @deprecated Use the export from `@sentry/replay` or from framework-specific SDKs like `@sentry/react` or `@sentry/vue` */
+type CanvasManagerOptions = InternalCanvasManagerOptions;
 
 export type {
+  // eslint-disable-next-line deprecation/deprecation
+  CanvasManagerInterface,
+  // eslint-disable-next-line deprecation/deprecation
+  CanvasManagerOptions,
+  // eslint-disable-next-line deprecation/deprecation
+  ReplayBreadcrumbFrame,
+  // eslint-disable-next-line deprecation/deprecation
+  ReplayBreadcrumbFrameEvent,
+  // eslint-disable-next-line deprecation/deprecation
   ReplayConfiguration,
+  // eslint-disable-next-line deprecation/deprecation
   ReplayEventType,
+  // eslint-disable-next-line deprecation/deprecation
   ReplayEventWithTime,
-  ReplayBreadcrumbFrame,
-  ReplayBreadcrumbFrameEvent,
-  ReplayOptionFrameEvent,
+  // eslint-disable-next-line deprecation/deprecation
   ReplayFrame,
+  // eslint-disable-next-line deprecation/deprecation
   ReplayFrameEvent,
+  // eslint-disable-next-line deprecation/deprecation
+  ReplayOptionFrameEvent,
+  // eslint-disable-next-line deprecation/deprecation
   ReplaySpanFrame,
+  // eslint-disable-next-line deprecation/deprecation
   ReplaySpanFrameEvent,
-  CanvasManagerInterface,
-  CanvasManagerOptions,
-} from './types';
-
-export { getReplay } from './util/getReplay';
+  InternalCanvasManagerInterface,
+  InternalCanvasManagerOptions,
+  InternalReplayBreadcrumbFrame,
+  InternalReplayBreadcrumbFrameEvent,
+  InternalReplayConfiguration,
+  InternalReplayEventType,
+  InternalReplayEventWithTime,
+  InternalReplayFrame,
+  InternalReplayFrameEvent,
+  InternalReplayOptionFrameEvent,
+  InternalReplaySpanFrame,
+  InternalReplaySpanFrameEvent,
+};
 
 // TODO (v8): Remove deprecated types
 export * from './types/deprecated';

packages/replay/test/integration/beforeAddRecordingEvent.test.ts

@@ -2,6 +2,7 @@ import * as SentryCore from '@sentry/core';
 import type { Transport } from '@sentry/types';
 import * as SentryUtils from '@sentry/utils';
 
+// eslint-disable-next-line deprecation/deprecation
 import type { Replay } from '../../src';
 import type { ReplayContainer } from '../../src/replay';
 import { clearSession } from '../../src/session/clearSession';

packages/replay/test/integration/stop.test.ts

@@ -1,5 +1,6 @@
 import * as SentryUtils from '@sentry/utils';
 
+// eslint-disable-next-line deprecation/deprecation
 import type { Replay } from '../../src';
 import { WINDOW } from '../../src/constants';
 import type { ReplayContainer } from '../../src/replay';

packages/replay/test/mocks/mockSdk.ts

@@ -1,5 +1,6 @@
 import type { Envelope, Transport, TransportMakeRequestResponse } from '@sentry/types';
 
+// eslint-disable-next-line deprecation/deprecation
 import type { Replay as ReplayIntegration } from '../../src';
 import type { ReplayContainer } from '../../src/replay';
 import type { ReplayConfiguration } from '../../src/types';