remove unused eslint-disable directives

Author: s1gr1d

dev-packages/e2e-tests/prepare.ts

@@ -1,4 +1,3 @@
-/* eslint-disable max-lines */
 /* eslint-disable no-console */
 import * as dotenv from 'dotenv';
 

dev-packages/e2e-tests/publish-packages.ts

@@ -1,4 +1,3 @@
-/* eslint-disable no-console */
 import * as childProcess from 'child_process';
 import * as path from 'path';
 import * as glob from 'glob';

dev-packages/e2e-tests/run.ts

@@ -1,4 +1,3 @@
-/* eslint-disable max-lines */
 /* eslint-disable no-console */
 import { spawn } from 'child_process';
 import { resolve } from 'path';

packages/bun/src/index.ts

@@ -125,7 +125,6 @@ export { bunServerIntegration } from './integrations/bunserver';
 const INTEGRATIONS = {
   // eslint-disable-next-line deprecation/deprecation
   ...CoreIntegrations,
-  // eslint-disable-next-line deprecation/deprecation
   ...NodeIntegrations,
   BunServer,
 };

packages/core/src/baseclient.ts

@@ -150,7 +150,7 @@ export abstract class BaseClient<O extends ClientOptions> implements Client<O> {
   /**
    * @inheritDoc
    */
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/explicit-module-boundary-types
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   public captureException(exception: any, hint?: EventHint, scope?: Scope): string | undefined {
     // ensure we haven't captured this very object before
     if (checkOrSetAlreadyCaught(exception)) {
@@ -857,7 +857,7 @@ export abstract class BaseClient<O extends ClientOptions> implements Client<O> {
   /**
    * @inheritDoc
    */
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/explicit-module-boundary-types
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   public abstract eventFromException(_exception: any, _hint?: EventHint): PromiseLike<Event>;
 
   /**

packages/core/src/integrations/inboundfilters.ts

@@ -173,7 +173,6 @@ function _getPossibleEventMessages(event: Event): string[] {
   let lastException;
   try {
     // @ts-expect-error Try catching to save bundle size
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
     lastException = event.exception.values[event.exception.values.length - 1];
   } catch (e) {
     // try catching to save bundle size checking existence of variables
@@ -198,7 +197,6 @@ function _getPossibleEventMessages(event: Event): string[] {
 function _isSentryError(event: Event): boolean {
   try {
     // @ts-expect-error can't be a sentry error if undefined
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
     return event.exception.values[0].type === 'SentryError';
   } catch (e) {
     // ignore

packages/core/src/tracing/hubextensions.ts

@@ -57,7 +57,6 @@ function _startTransaction(
 The transaction will not be sampled. Please use the ${configInstrumenter} instrumentation to start transactions.`,
       );
 
-    // eslint-disable-next-line deprecation/deprecation
     transactionContext.sampled = false;
   }
 

packages/core/src/tracing/idletransaction.ts

@@ -1,4 +1,3 @@
-/* eslint-disable max-lines */
 import type { Hub, SpanTimeInput, TransactionContext } from '@sentry/types';
 import { logger, timestampInSeconds } from '@sentry/utils';
 

packages/core/src/tracing/sampling.ts

@@ -105,7 +105,6 @@ export function sampleTransaction<T extends Transaction>(
  */
 function isValidSampleRate(rate: unknown): boolean {
   // we need to check NaN explicitly because it's of type 'number' and therefore wouldn't get caught by this typecheck
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   if (isNaN(rate) || !(typeof rate === 'number' || typeof rate === 'boolean')) {
     DEBUG_BUILD &&
       logger.warn(

packages/core/src/tracing/transaction.ts

@@ -89,7 +89,6 @@ export class Transaction extends SentrySpan implements TransactionInterface {
   }
 
   // This sadly conflicts with the getter/setter ordering :(
-  /* eslint-disable @typescript-eslint/member-ordering */
 
   /**
    * Get the metadata for this transaction.

packages/core/test/lib/api.test.ts

@@ -1,4 +1,3 @@
-/* eslint-disable deprecation/deprecation */
 import type { ClientOptions, DsnComponents } from '@sentry/types';
 import { makeDsn } from '@sentry/utils';
 

packages/core/test/lib/session.test.ts

@@ -1,5 +1,3 @@
-/* eslint-disable deprecation/deprecation */
-
 import type { SessionContext } from '@sentry/types';
 import { timestampInSeconds } from '@sentry/utils';
 

packages/core/test/lib/sessionflusher.test.ts

@@ -1,5 +1,3 @@
-/* eslint-disable deprecation/deprecation */
-
 import type { Client } from '@sentry/types';
 
 import { SessionFlusher } from '../../src/sessionflusher';

packages/core/test/lib/tracing/span.test.ts

@@ -4,7 +4,6 @@ import { TRACE_FLAG_NONE, TRACE_FLAG_SAMPLED, spanToJSON } from '../../../src/ut
 
 describe('span', () => {
   describe('name', () => {
-    /* eslint-disable deprecation/deprecation */
     it('works with name', () => {
       const span = new SentrySpan({ name: 'span name' });
       expect(spanToJSON(span).description).toEqual('span name');

packages/core/test/mocks/client.ts

@@ -49,13 +49,11 @@ export class TestClient extends BaseClient<TestClientOptions> {
     TestClient.instance = this;
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/explicit-module-boundary-types
   public eventFromException(exception: any): PromiseLike<Event> {
     const event: Event = {
       exception: {
         values: [
           {
-            /* eslint-disable @typescript-eslint/no-unsafe-member-access */
             type: exception.name,
             value: exception.message,
             /* eslint-enable @typescript-eslint/no-unsafe-member-access */
@@ -86,7 +84,6 @@ export class TestClient extends BaseClient<TestClientOptions> {
       super.sendEvent(event, hint);
       return;
     }
-    // eslint-disable-next-line @typescript-eslint/no-unused-expressions
     TestClient.sendEventCalled && TestClient.sendEventCalled(event);
   }
 

packages/feedback/test/utils/TestClient.ts

@@ -14,7 +14,6 @@ export class TestClient extends BaseClient<TestClientOptions> {
       exception: {
         values: [
           {
-            /* eslint-disable @typescript-eslint/no-unsafe-member-access */
             type: exception.name,
             value: exception.message,
             /* eslint-enable @typescript-eslint/no-unsafe-member-access */

packages/profiling-node/src/cpu_profiler.ts

@@ -34,7 +34,6 @@ export function importCppBindingsModule(): PrivateV8CpuProfilerBindings {
     return require(`${binaryPath}.node`);
   }
 
-  /* eslint-disable no-fallthrough */
   // We need the fallthrough so that in the end, we can fallback to the require dynamice require.
   // This is for cases where precompiled binaries were not provided, but may have been compiled from source.
   if (platform === 'darwin') {

packages/profiling-node/src/integration.ts

@@ -56,7 +56,6 @@ export class ProfilingIntegration implements Integration {
    */
   public setupOnce(addGlobalEventProcessor: (callback: EventProcessor) => void, getCurrentHub: () => Hub): void {
     this.getCurrentHub = getCurrentHub;
-    // eslint-disable-next-line deprecation/deprecation
     const client = this.getCurrentHub().getClient() as NodeClient;
 
     if (client && typeof client.on === 'function') {
@@ -137,7 +136,6 @@ export class ProfilingIntegration implements Integration {
 
           // Remove the profile from the transaction context before sending, relay will take care of the rest.
           if (profileContext) {
-            // eslint-disable-next-line @typescript-eslint/no-dynamic-delete
             delete profiledTransaction.contexts?.['profile'];
           }
 
@@ -199,7 +197,6 @@ export class ProfilingIntegration implements Integration {
       // and forward it to the next event processor.
       const hub = this.getCurrentHub();
 
-      // eslint-disable-next-line deprecation/deprecation
       const client = hub.getClient();
 
       if (!client) {

packages/profiling-node/src/utils.ts

@@ -26,7 +26,6 @@ import type { DebugImage } from './types';
 
 // We require the file because if we import it, it will be included in the bundle.
 // I guess tsc does not check file contents when it's imported.
-// eslint-disable-next-line
 const THREAD_ID_STRING = String(threadId);
 const THREAD_NAME = isMainThread ? 'main' : 'worker';
 const FORMAT_VERSION = '1';
@@ -454,9 +453,7 @@ export function applyDebugMetadata(resource_paths: ReadonlyArray<string>): Debug
     return [];
   }
 
-  // eslint-disable-next-line deprecation/deprecation
   const hub = Sentry.getCurrentHub();
-  // eslint-disable-next-line deprecation/deprecation
   const client = hub.getClient();
   const options = client && client.getOptions();