feat(v8/integrations): Remove deprecated exports

- Remove class based exports
- Delete `Transaction` integration

Author: AbhiPrasad

packages/integrations/src/captureconsole.ts

@@ -1,12 +1,5 @@
-import {
-  captureException,
-  captureMessage,
-  convertIntegrationFnToClass,
-  defineIntegration,
-  getClient,
-  withScope,
-} from '@sentry/core';
-import type { CaptureContext, Client, Integration, IntegrationClass, IntegrationFn } from '@sentry/types';
+import { captureException, captureMessage, defineIntegration, getClient, withScope } from '@sentry/core';
+import type { CaptureContext, IntegrationFn } from '@sentry/types';
 import {
   CONSOLE_LEVELS,
   GLOBAL_OBJ,
@@ -45,19 +38,10 @@ const _captureConsoleIntegration = ((options: CaptureConsoleOptions = {}) => {
   };
 }) satisfies IntegrationFn;
 
-export const captureConsoleIntegration = defineIntegration(_captureConsoleIntegration);
-
 /**
  * Send Console API calls as Sentry Events.
- * @deprecated Use `captureConsoleIntegration()` instead.
  */
-// eslint-disable-next-line deprecation/deprecation
-export const CaptureConsole = convertIntegrationFnToClass(
-  INTEGRATION_NAME,
-  captureConsoleIntegration,
-) as IntegrationClass<Integration & { setup: (client: Client) => void }> & {
-  new (options?: { levels?: string[] }): Integration;
-};
+export const captureConsoleIntegration = defineIntegration(_captureConsoleIntegration);
 
 function consoleHandler(args: unknown[], level: string): void {
   const captureContext: CaptureContext = {

packages/integrations/src/contextlines.ts

@@ -1,5 +1,5 @@
-import { convertIntegrationFnToClass, defineIntegration } from '@sentry/core';
-import type { Event, Integration, IntegrationClass, IntegrationFn, StackFrame } from '@sentry/types';
+import { defineIntegration } from '@sentry/core';
+import type { Event, IntegrationFn, StackFrame } from '@sentry/types';
 import { GLOBAL_OBJ, addContextToFrame, stripUrlQueryAndFragment } from '@sentry/utils';
 
 const WINDOW = GLOBAL_OBJ as typeof GLOBAL_OBJ & Window;
@@ -31,8 +31,6 @@ const _contextLinesIntegration = ((options: ContextLinesOptions = {}) => {
   };
 }) satisfies IntegrationFn;
 
-export const contextLinesIntegration = defineIntegration(_contextLinesIntegration);
-
 /**
  * Collects source context lines around the lines of stackframes pointing to JS embedded in
  * the current page's HTML.
@@ -43,13 +41,8 @@ export const contextLinesIntegration = defineIntegration(_contextLinesIntegratio
  *
  * Use this integration if you have inline JS code in HTML pages that can't be accessed
  * by our backend (e.g. due to a login-protected page).
- *
- * @deprecated Use `contextLinesIntegration()` instead.
  */
-// eslint-disable-next-line deprecation/deprecation
-export const ContextLines = convertIntegrationFnToClass(INTEGRATION_NAME, contextLinesIntegration) as IntegrationClass<
-  Integration & { processEvent: (event: Event) => Event }
-> & { new (options?: { frameContextLines?: number }): Integration };
+export const contextLinesIntegration = defineIntegration(_contextLinesIntegration);
 
 /**
  * Processes an event and adds context lines.

packages/integrations/src/debug.ts

@@ -1,5 +1,5 @@
-import { convertIntegrationFnToClass, defineIntegration } from '@sentry/core';
-import type { Client, Event, EventHint, Integration, IntegrationClass, IntegrationFn } from '@sentry/types';
+import { defineIntegration } from '@sentry/core';
+import type { Event, EventHint, IntegrationFn } from '@sentry/types';
 import { consoleSandbox } from '@sentry/utils';
 
 const INTEGRATION_NAME = 'Debug';
@@ -11,6 +11,10 @@ interface DebugOptions {
   debugger?: boolean;
 }
 
+/**
+ * Integration to debug sent Sentry events.
+ * This integration should not be used in production.
+ */
 const _debugIntegration = ((options: DebugOptions = {}) => {
   const _options = {
     debugger: false,
@@ -50,19 +54,3 @@ const _debugIntegration = ((options: DebugOptions = {}) => {
 }) satisfies IntegrationFn;
 
 export const debugIntegration = defineIntegration(_debugIntegration);
-
-/**
- * Integration to debug sent Sentry events.
- * This integration should not be used in production.
- *
- * @deprecated Use `debugIntegration()` instead.
- */
-// eslint-disable-next-line deprecation/deprecation
-export const Debug = convertIntegrationFnToClass(INTEGRATION_NAME, debugIntegration) as IntegrationClass<
-  Integration & { setup: (client: Client) => void }
-> & {
-  new (options?: {
-    stringify?: boolean;
-    debugger?: boolean;
-  }): Integration;
-};

packages/integrations/src/dedupe.ts

@@ -1,5 +1,5 @@
-import { convertIntegrationFnToClass, defineIntegration } from '@sentry/core';
-import type { Event, Exception, Integration, IntegrationClass, IntegrationFn, StackFrame } from '@sentry/types';
+import { defineIntegration } from '@sentry/core';
+import type { Event, Exception, IntegrationFn, StackFrame } from '@sentry/types';
 import { logger } from '@sentry/utils';
 
 import { DEBUG_BUILD } from './debug-build';
@@ -33,16 +33,10 @@ const _dedupeIntegration = (() => {
   };
 }) satisfies IntegrationFn;
 
-export const dedupeIntegration = defineIntegration(_dedupeIntegration);
-
 /**
  * Deduplication filter.
- * @deprecated Use `dedupeIntegration()` instead.
  */
-// eslint-disable-next-line deprecation/deprecation
-export const Dedupe = convertIntegrationFnToClass(INTEGRATION_NAME, dedupeIntegration) as IntegrationClass<
-  Integration & { processEvent: (event: Event) => Event }
->;
+export const dedupeIntegration = defineIntegration(_dedupeIntegration);
 
 /** only exported for tests. */
 export function _shouldDropEvent(currentEvent: Event, previousEvent?: Event): boolean {

packages/integrations/src/extraerrordata.ts

@@ -1,13 +1,5 @@
-import { convertIntegrationFnToClass, defineIntegration } from '@sentry/core';
-import type {
-  Contexts,
-  Event,
-  EventHint,
-  ExtendedError,
-  Integration,
-  IntegrationClass,
-  IntegrationFn,
-} from '@sentry/types';
+import { defineIntegration } from '@sentry/core';
+import type { Contexts, Event, EventHint, ExtendedError, IntegrationFn } from '@sentry/types';
 import { addNonEnumerableProperty, isError, isPlainObject, logger, normalize } from '@sentry/utils';
 
 import { DEBUG_BUILD } from './debug-build';
@@ -21,19 +13,18 @@ interface ExtraErrorDataOptions {
   depth: number;
 
   /**
-   * Whether to capture error causes.
+   * Whether to capture error causes. Defaults to true.
    *
    * More information: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Error/cause
    */
   captureErrorCause: boolean;
 }
 
+/**
+ * Extract additional data for from original exceptions.
+ */
 const _extraErrorDataIntegration = ((options: Partial<ExtraErrorDataOptions> = {}) => {
-  const depth = options.depth || 3;
-
-  // TODO(v8): Flip the default for this option to true
-  const captureErrorCause = options.captureErrorCause || false;
-
+  const { depth = 3, captureErrorCause = true } = options;
   return {
     name: INTEGRATION_NAME,
     // TODO v8: Remove this
@@ -46,23 +37,6 @@ const _extraErrorDataIntegration = ((options: Partial<ExtraErrorDataOptions> = {
 
 export const extraErrorDataIntegration = defineIntegration(_extraErrorDataIntegration);
 
-/**
- * Extract additional data for from original exceptions.
- * @deprecated Use `extraErrorDataIntegration()` instead.
- */
-// eslint-disable-next-line deprecation/deprecation
-export const ExtraErrorData = convertIntegrationFnToClass(
-  INTEGRATION_NAME,
-  extraErrorDataIntegration,
-) as IntegrationClass<Integration & { processEvent: (event: Event, hint: EventHint) => Event }> & {
-  new (
-    options?: Partial<{
-      depth: number;
-      captureErrorCause: boolean;
-    }>,
-  ): Integration;
-};
-
 function _enhanceEventWithErrorData(
   event: Event,
   hint: EventHint = {},

packages/integrations/src/httpclient.ts

@@ -1,18 +1,5 @@
-import {
-  captureEvent,
-  convertIntegrationFnToClass,
-  defineIntegration,
-  getClient,
-  isSentryRequestUrl,
-} from '@sentry/core';
-import type {
-  Client,
-  Event as SentryEvent,
-  Integration,
-  IntegrationClass,
-  IntegrationFn,
-  SentryWrappedXMLHttpRequest,
-} from '@sentry/types';
+import { captureEvent, defineIntegration, getClient, isSentryRequestUrl } from '@sentry/core';
+import type { Client, Event as SentryEvent, IntegrationFn, SentryWrappedXMLHttpRequest } from '@sentry/types';
 import {
   GLOBAL_OBJ,
   SENTRY_XHR_DATA_KEY,
@@ -69,21 +56,10 @@ const _httpClientIntegration = ((options: Partial<HttpClientOptions> = {}) => {
   };
 }) satisfies IntegrationFn;
 
-export const httpClientIntegration = defineIntegration(_httpClientIntegration);
-
 /**
  * Create events for failed client side HTTP requests.
- * @deprecated Use `httpClientIntegration()` instead.
  */
-// eslint-disable-next-line deprecation/deprecation
-export const HttpClient = convertIntegrationFnToClass(INTEGRATION_NAME, httpClientIntegration) as IntegrationClass<
-  Integration & { setup: (client: Client) => void }
-> & {
-  new (options?: {
-    failedRequestStatusCodes: HttpStatusCodeRange[];
-    failedRequestTargets: HttpRequestTarget[];
-  }): Integration;
-};
+export const httpClientIntegration = defineIntegration(_httpClientIntegration);
 
 /**
  * Interceptor function for fetch requests

packages/integrations/src/index.ts

@@ -1,11 +1,9 @@
-/* eslint-disable deprecation/deprecation */
-export { CaptureConsole, captureConsoleIntegration } from './captureconsole';
-export { Debug, debugIntegration } from './debug';
-export { Dedupe, dedupeIntegration } from './dedupe';
-export { ExtraErrorData, extraErrorDataIntegration } from './extraerrordata';
-export { ReportingObserver, reportingObserverIntegration } from './reportingobserver';
-export { RewriteFrames, rewriteFramesIntegration } from './rewriteframes';
-export { SessionTiming, sessionTimingIntegration } from './sessiontiming';
-export { Transaction } from './transaction';
-export { HttpClient, httpClientIntegration } from './httpclient';
-export { ContextLines, contextLinesIntegration } from './contextlines';
+export { captureConsoleIntegration } from './captureconsole';
+export { debugIntegration } from './debug';
+export { dedupeIntegration } from './dedupe';
+export { extraErrorDataIntegration } from './extraerrordata';
+export { reportingObserverIntegration } from './reportingobserver';
+export { rewriteFramesIntegration } from './rewriteframes';
+export { sessionTimingIntegration } from './sessiontiming';
+export { httpClientIntegration } from './httpclient';
+export { contextLinesIntegration } from './contextlines';

packages/integrations/src/reportingobserver.ts

@@ -1,5 +1,5 @@
-import { captureMessage, convertIntegrationFnToClass, defineIntegration, getClient, withScope } from '@sentry/core';
-import type { Client, Integration, IntegrationClass, IntegrationFn } from '@sentry/types';
+import { captureMessage, defineIntegration, getClient, withScope } from '@sentry/core';
+import type { Client, IntegrationFn } from '@sentry/types';
 import { GLOBAL_OBJ, supportsReportingObserver } from '@sentry/utils';
 
 const WINDOW = GLOBAL_OBJ as typeof GLOBAL_OBJ & Window;
@@ -115,18 +115,7 @@ const _reportingObserverIntegration = ((options: ReportingObserverOptions = {})
   };
 }) satisfies IntegrationFn;
 
-export const reportingObserverIntegration = defineIntegration(_reportingObserverIntegration);
-
 /**
  * Reporting API integration - https://w3c.github.io/reporting/
- * @deprecated Use `reportingObserverIntegration()` instead.
  */
-// eslint-disable-next-line deprecation/deprecation
-export const ReportingObserver = convertIntegrationFnToClass(
-  INTEGRATION_NAME,
-  reportingObserverIntegration,
-) as IntegrationClass<Integration & { setup: (client: Client) => void }> & {
-  new (options?: {
-    types?: ReportTypes[];
-  }): Integration;
-};
+export const reportingObserverIntegration = defineIntegration(_reportingObserverIntegration);

packages/integrations/src/rewriteframes.ts

@@ -1,5 +1,5 @@
-import { convertIntegrationFnToClass, defineIntegration } from '@sentry/core';
-import type { Event, Integration, IntegrationClass, IntegrationFn, StackFrame, Stacktrace } from '@sentry/types';
+import { defineIntegration } from '@sentry/core';
+import type { Event, IntegrationFn, StackFrame, Stacktrace } from '@sentry/types';
 import { basename, relative } from '@sentry/utils';
 
 type StackFrameIteratee = (frame: StackFrame) => StackFrame;
@@ -85,16 +85,7 @@ const _rewriteFramesIntegration = ((options: RewriteFramesOptions = {}) => {
   };
 }) satisfies IntegrationFn;
 
-export const rewriteFramesIntegration = defineIntegration(_rewriteFramesIntegration);
-
 /**
  * Rewrite event frames paths.
- * @deprecated Use `rewriteFramesIntegration()` instead.
  */
-// eslint-disable-next-line deprecation/deprecation
-export const RewriteFrames = convertIntegrationFnToClass(
-  INTEGRATION_NAME,
-  rewriteFramesIntegration,
-) as IntegrationClass<Integration & { processEvent: (event: Event) => Event }> & {
-  new (options?: { root?: string; prefix?: string; iteratee?: StackFrameIteratee }): Integration;
-};
+export const rewriteFramesIntegration = defineIntegration(_rewriteFramesIntegration);

packages/integrations/src/sessiontiming.ts

@@ -1,5 +1,5 @@
-import { convertIntegrationFnToClass, defineIntegration } from '@sentry/core';
-import type { Event, Integration, IntegrationClass, IntegrationFn } from '@sentry/types';
+import { defineIntegration } from '@sentry/core';
+import type { IntegrationFn } from '@sentry/types';
 
 const INTEGRATION_NAME = 'SessionTiming';
 
@@ -26,14 +26,8 @@ const _sessionTimingIntegration = (() => {
   };
 }) satisfies IntegrationFn;
 
-export const sessionTimingIntegration = defineIntegration(_sessionTimingIntegration);
-
 /**
- * This function adds duration since Sentry was initialized till the time event was sent.
- * @deprecated Use `sessionTimingIntegration()` instead.
+ * This function adds duration since the sessionTimingIntegration was initialized
+ * till the time event was sent.
  */
-// eslint-disable-next-line deprecation/deprecation
-export const SessionTiming = convertIntegrationFnToClass(
-  INTEGRATION_NAME,
-  sessionTimingIntegration,
-) as IntegrationClass<Integration & { processEvent: (event: Event) => Event }>;
+export const sessionTimingIntegration = defineIntegration(_sessionTimingIntegration);