ref: Make setupOnce optional in integrations

Author: mydea

packages/browser/src/integrations/breadcrumbs.ts

@@ -70,8 +70,6 @@ const _breadcrumbsIntegration = ((options: Partial<BreadcrumbsOptions> = {}) =>
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     setup(client) {
       if (_options.console) {
         addConsoleInstrumentationHandler(_getConsoleBreadcrumbHandler(client));

packages/browser/src/integrations/dedupe.ts

@@ -11,8 +11,6 @@ const _dedupeIntegration = (() => {
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     processEvent(currentEvent) {
       // We want to ignore any non-error type events, e.g. transactions or replays
       // These should never be deduped, and also not be compared against as _previousEvent.

packages/browser/src/integrations/httpcontext.ts

@@ -8,8 +8,6 @@ const INTEGRATION_NAME = 'HttpContext';
 const _httpContextIntegration = (() => {
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     preprocessEvent(event) {
       // if none of the information we want exists, don't bother
       if (!WINDOW.navigator && !WINDOW.location && !WINDOW.document) {

packages/browser/src/integrations/linkederrors.ts

@@ -19,8 +19,6 @@ const _linkedErrorsIntegration = ((options: LinkedErrorsOptions = {}) => {
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     preprocessEvent(event, hint, client) {
       const options = client.getOptions();
 

packages/browser/src/profiling/integration.ts

@@ -22,7 +22,6 @@ const _browserProfilingIntegration = (() => {
   return {
     name: INTEGRATION_NAME,
     // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     setup(client) {
       const scope = getCurrentScope();
 

packages/core/src/integration.ts

@@ -129,7 +129,7 @@ export function setupIntegration(client: Client, integration: Integration, integ
   integrationIndex[integration.name] = integration;
 
   // `setupOnce` is only called the first time
-  if (installedIntegrations.indexOf(integration.name) === -1) {
+  if (installedIntegrations.indexOf(integration.name) === -1 && typeof integration.setupOnce === 'function') {
     // eslint-disable-next-line deprecation/deprecation
     integration.setupOnce(addGlobalEventProcessor, getCurrentHub);
     installedIntegrations.push(integration.name);

packages/core/src/integrations/inboundfilters.ts

@@ -33,8 +33,6 @@ const INTEGRATION_NAME = 'InboundFilters';
 const _inboundFiltersIntegration = ((options: Partial<InboundFiltersOptions> = {}) => {
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     processEvent(event, _hint, client) {
       const clientOptions = client.getOptions();
       const mergedOptions = _mergeOptions(options, clientOptions);

packages/core/src/integrations/linkederrors.ts

@@ -18,8 +18,6 @@ const _linkedErrorsIntegration = ((options: LinkedErrorsOptions = {}) => {
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     preprocessEvent(event, hint, client) {
       const options = client.getOptions();
 

packages/core/src/integrations/metadata.ts

@@ -9,8 +9,6 @@ const INTEGRATION_NAME = 'ModuleMetadata';
 const _moduleMetadataIntegration = (() => {
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     setup(client) {
       // We need to strip metadata from stack frames before sending them to Sentry since these are client side only.
       client.on('beforeEnvelope', envelope => {

packages/core/src/integrations/requestdata.ts

@@ -75,8 +75,6 @@ const _requestDataIntegration = ((options: RequestDataIntegrationOptions = {}) =
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     processEvent(event, _hint, client) {
       // Note: In the long run, most of the logic here should probably move into the request data utility functions. For
       // the moment it lives here, though, until https://github.com/getsentry/sentry-javascript/issues/5718 is addressed.

packages/deno/src/integrations/context.ts

@@ -55,8 +55,6 @@ async function addDenoRuntimeContext(event: Event): Promise<Event> {
 const _denoContextIntegration = (() => {
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     processEvent(event) {
       return addDenoRuntimeContext(event);
     },

packages/deno/src/integrations/contextlines.ts

@@ -52,8 +52,6 @@ const _contextLinesIntegration = ((options: ContextLinesOptions = {}) => {
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     processEvent(event) {
       return addSourceContext(event, contextLines);
     },

packages/deno/src/integrations/globalhandlers.ts

@@ -31,8 +31,6 @@ const _globalHandlersIntegration = ((options?: GlobalHandlersIntegrations) => {
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     setup(client) {
       if (_options.error) {
         installGlobalErrorHandler(client);

packages/deno/src/integrations/normalizepaths.ts

@@ -70,8 +70,6 @@ const _normalizePathsIntegration = (() => {
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     processEvent(event) {
       // This error.stack hopefully contains paths that traverse the app cwd
       const error = new Error();

packages/integrations/src/captureconsole.ts

@@ -20,8 +20,6 @@ const _captureConsoleIntegration = ((options: CaptureConsoleOptions = {}) => {
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     setup(client) {
       if (!('console' in GLOBAL_OBJ)) {
         return;

packages/integrations/src/contextlines.ts

@@ -23,8 +23,6 @@ const _contextLinesIntegration = ((options: ContextLinesOptions = {}) => {
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     processEvent(event) {
       return addSourceContext(event, contextLines);
     },

packages/integrations/src/debug.ts

@@ -24,8 +24,6 @@ const _debugIntegration = ((options: DebugOptions = {}) => {
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     setup(client) {
       client.on('beforeSendEvent', (event: Event, hint?: EventHint) => {
         if (_options.debugger) {

packages/integrations/src/dedupe.ts

@@ -11,8 +11,6 @@ const _dedupeIntegration = (() => {
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     processEvent(currentEvent) {
       // We want to ignore any non-error type events, e.g. transactions or replays
       // These should never be deduped, and also not be compared against as _previousEvent.

packages/integrations/src/extraerrordata.ts

@@ -27,8 +27,6 @@ const _extraErrorDataIntegration = ((options: Partial<ExtraErrorDataOptions> = {
   const { depth = 3, captureErrorCause = true } = options;
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     processEvent(event, hint) {
       return _enhanceEventWithErrorData(event, hint, depth, captureErrorCause);
     },

packages/integrations/src/httpclient.ts

@@ -47,8 +47,6 @@ const _httpClientIntegration = ((options: Partial<HttpClientOptions> = {}) => {
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     setup(client): void {
       _wrapFetch(client, _options);
       _wrapXHR(client, _options);

packages/integrations/src/rewriteframes.ts

@@ -71,8 +71,6 @@ const _rewriteFramesIntegration = ((options: RewriteFramesOptions = {}) => {
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     processEvent(originalEvent) {
       let processedEvent = originalEvent;
 

packages/integrations/src/sessiontiming.ts

@@ -8,8 +8,6 @@ const _sessionTimingIntegration = (() => {
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     processEvent(event) {
       const now = Date.now();
 

packages/node/src/integrations/anr/index.ts

@@ -55,8 +55,6 @@ const INTEGRATION_NAME = 'Anr';
 const _anrIntegration = ((options: Partial<AnrIntegrationOptions> = {}) => {
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     setup(client: NodeClient) {
       if (NODE_VERSION.major < 16 || (NODE_VERSION.major === 16 && NODE_VERSION.minor < 17)) {
         throw new Error('ANR detection requires Node 16.17.0 or later');

packages/node/src/integrations/console.ts

@@ -8,8 +8,6 @@ const INTEGRATION_NAME = 'Console';
 const _consoleIntegration = (() => {
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     setup(client) {
       addConsoleInstrumentationHandler(({ args, level }) => {
         if (getClient() !== client) {

packages/node/src/integrations/context.ts

@@ -102,8 +102,6 @@ const _nodeContextIntegration = ((options: ContextOptions = {}) => {
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     processEvent(event) {
       return addContext(event);
     },

packages/node/src/integrations/contextlines.ts

@@ -40,8 +40,6 @@ const _contextLinesIntegration = ((options: ContextLinesOptions = {}) => {
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     processEvent(event) {
       return addSourceContext(event, contextLines);
     },

packages/node/src/integrations/local-variables/local-variables-async.ts

@@ -220,8 +220,6 @@ const _localVariablesAsyncIntegration = ((options: LocalVariablesIntegrationOpti
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     setup(client: NodeClient) {
       const clientOptions = client.getOptions();
 

packages/node/src/integrations/modules.ts

@@ -79,8 +79,6 @@ function _getModules(): { [key: string]: string } {
 const _modulesIntegration = (() => {
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     processEvent(event) {
       event.modules = {
         ...event.modules,

packages/node/src/integrations/onuncaughtexception.ts

@@ -48,8 +48,6 @@ const _onUncaughtExceptionIntegration = ((options: Partial<OnUncaughtExceptionOp
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     setup(client: NodeClient) {
       global.process.on('uncaughtException', makeErrorHandler(client, _options));
     },

packages/node/src/integrations/onunhandledrejection.ts

@@ -21,8 +21,6 @@ const _onUnhandledRejectionIntegration = ((options: Partial<OnUnhandledRejection
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     setup(client) {
       global.process.on('unhandledRejection', makeUnhandledPromiseHandler(client, { mode }));
     },

packages/node/src/integrations/spotlight.ts

@@ -21,8 +21,6 @@ const _spotlightIntegration = ((options: Partial<SpotlightConnectionOptions> = {
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     setup(client) {
       if (typeof process === 'object' && process.env && process.env.NODE_ENV !== 'development') {
         logger.warn("[Spotlight] It seems you're not in dev mode. Do you really want to have Spotlight enabled?");

packages/replay-canvas/src/canvas.ts

@@ -66,8 +66,6 @@ export const _replayCanvasIntegration = ((options: Partial<ReplayCanvasOptions>
 
   return {
     name: INTEGRATION_NAME,
-    // eslint-disable-next-line @typescript-eslint/no-empty-function
-    setupOnce() {},
     getOptions(): ReplayCanvasIntegrationOptions {
       const { quality, enableManualSnapshot } = _canvasOptions;
 

packages/types/src/integration.ts

@@ -24,10 +24,8 @@ export interface IntegrationFnResult {
   /**
    * This hook is only called once, even if multiple clients are created.
    * It does not receives any arguments, and should only use for e.g. global monkey patching and similar things.
-   *
-   * NOTE: In v8, this will become optional.
    */
-  setupOnce(): void;
+  setupOnce?(): void;
 
   /**
    * Set up an integration for the given client.
@@ -74,10 +72,8 @@ export interface Integration {
   /**
    * This hook is only called once, even if multiple clients are created.
    * It does not receives any arguments, and should only use for e.g. global monkey patching and similar things.
-   *
-   * NOTE: In v8, this will become optional, and not receive any arguments anymore.
    */
-  setupOnce(addGlobalEventProcessor: (callback: EventProcessor) => void, getCurrentHub: () => Hub): void;
+  setupOnce?(addGlobalEventProcessor: (callback: EventProcessor) => void, getCurrentHub: () => Hub): void;
 
   /**
    * Set up an integration for the given client.

packages/vercel-edge/src/integrations/wintercg-fetch.ts

@@ -87,8 +87,6 @@ const _winterCGFetch = ((options: Partial<Options> = {}) => {
 
   return {
     name: INTEGRATION_NAME,
-    // TODO v8: Remove this again
-    // eslint-disable-next-line @typescript-eslint/no-empty-function
     setupOnce() {
       addFetchInstrumentationHandler(handlerData => {
         const client = getClient();

packages/vercel-edge/test/wintercg-fetch.test.ts

@@ -43,7 +43,7 @@ describe('WinterCGFetch instrumentation', () => {
     addFetchInstrumentationHandlerSpy.mockImplementationOnce(() => undefined);
 
     const integration = winterCGFetchIntegration();
-    integration.setupOnce();
+    integration.setupOnce!();
     integration.setup!(client);
 
     const [fetchInstrumentationHandlerCallback] = addFetchInstrumentationHandlerSpy.mock.calls[0];
@@ -77,7 +77,7 @@ describe('WinterCGFetch instrumentation', () => {
     addFetchInstrumentationHandlerSpy.mockImplementationOnce(() => undefined);
 
     const integration = winterCGFetchIntegration();
-    integration.setupOnce();
+    integration.setupOnce!();
     // integration.setup!(client) is not called!
 
     const [fetchInstrumentationHandlerCallback] = addFetchInstrumentationHandlerSpy.mock.calls[0];
@@ -97,7 +97,7 @@ describe('WinterCGFetch instrumentation', () => {
     addFetchInstrumentationHandlerSpy.mockImplementationOnce(() => undefined);
 
     const integration = winterCGFetchIntegration();
-    integration.setupOnce();
+    integration.setupOnce!();
     integration.setup!(client);
 
     const [fetchInstrumentationHandlerCallback] = addFetchInstrumentationHandlerSpy.mock.calls[0];
@@ -121,7 +121,7 @@ describe('WinterCGFetch instrumentation', () => {
         return url === 'http://only-acceptable-url.com/';
       },
     });
-    integration.setupOnce();
+    integration.setupOnce!();
     integration.setup!(client);
 
     const [fetchInstrumentationHandlerCallback] = addFetchInstrumentationHandlerSpy.mock.calls[0];
@@ -145,7 +145,7 @@ describe('WinterCGFetch instrumentation', () => {
     addFetchInstrumentationHandlerSpy.mockImplementationOnce(() => undefined);
 
     const integration = winterCGFetchIntegration();
-    integration.setupOnce();
+    integration.setupOnce!();
     integration.setup!(client);
 
     const [fetchInstrumentationHandlerCallback] = addFetchInstrumentationHandlerSpy.mock.calls[0];
@@ -182,7 +182,7 @@ describe('WinterCGFetch instrumentation', () => {
     addFetchInstrumentationHandlerSpy.mockImplementationOnce(() => undefined);
 
     const integration = winterCGFetchIntegration({ breadcrumbs: false });
-    integration.setupOnce();
+    integration.setupOnce!();
     integration.setup!(client);
 
     const [fetchInstrumentationHandlerCallback] = addFetchInstrumentationHandlerSpy.mock.calls[0];

packages/vue/src/integration.ts

@@ -25,7 +25,6 @@ const _vueIntegration = ((integrationOptions: Partial<VueOptions> = {}) => {
   return {
     name: INTEGRATION_NAME,
     // TODO v8: Remove this
-    setupOnce() {}, // eslint-disable-line @typescript-eslint/no-empty-function
     setup(client) {
       _setupIntegration(client, integrationOptions);
     },