chore(lint): prefer 'unknown' to 'any', fix lint warnings

.eslintrc.js

@@ -23,6 +23,9 @@ module.exports = {
     'types/**',
     'scripts/*.js',
   ],
+  rules: {
+    '@typescript-eslint/no-explicit-any': 'error',
+  },
   reportUnusedDisableDirectives: true,
   overrides: [
     {
@@ -36,6 +39,9 @@ module.exports = {
       parserOptions: {
         project: ['tsconfig.test.json'],
       },
+      rules: {
+        '@typescript-eslint/no-explicit-any': 'off',
+      },
     },
     {
       files: ['scripts/**/*.ts'],

dev-packages/.eslintrc.js

@@ -0,0 +1,7 @@
+module.exports = {
+  extends: ['../.eslintrc.js'],
+  rules: {
+    // tests often have just cause to do evil
+    '@typescript-eslint/no-explicit-any': 'off',
+  },
+};

dev-packages/browser-integration-tests/.eslintrc.js

@@ -4,7 +4,7 @@ module.exports = {
     node: true,
   },
   // todo: remove regexp plugin from here once we add it to base.js eslint config for the whole project
-  extends: ['../../.eslintrc.js', 'plugin:regexp/recommended'],
+  extends: ['../.eslintrc.js', 'plugin:regexp/recommended'],
   plugins: ['regexp'],
   ignorePatterns: [
     'suites/**/subject.js',

dev-packages/bundler-tests/.eslintrc.js

@@ -1,5 +1,5 @@
 module.exports = {
-  extends: ['../../.eslintrc.js'],
+  extends: ['../.eslintrc.js'],
   parserOptions: {
     sourceType: 'module',
   },

dev-packages/clear-cache-gh-action/.eslintrc.cjs

@@ -1,6 +1,6 @@
 module.exports = {
   // todo: remove regexp plugin from here once we add it to base.js eslint config for the whole project
-  extends: ['../../.eslintrc.js', 'plugin:regexp/recommended'],
+  extends: ['../.eslintrc.js', 'plugin:regexp/recommended'],
   plugins: ['regexp'],
   parserOptions: {
     sourceType: 'module',

dev-packages/cloudflare-integration-tests/.eslintrc.js

@@ -3,7 +3,7 @@ module.exports = {
     node: true,
   },
   // todo: remove regexp plugin from here once we add it to base.js eslint config for the whole project
-  extends: ['../../.eslintrc.js', 'plugin:regexp/recommended'],
+  extends: ['../.eslintrc.js', 'plugin:regexp/recommended'],
   plugins: ['regexp'],
   overrides: [
     {

dev-packages/cloudflare-integration-tests/suites/tracing/google-genai/mocks.ts

@@ -3,7 +3,6 @@ import type { GoogleGenAIChat, GoogleGenAIClient, GoogleGenAIResponse } from '@s
 export class MockGoogleGenAI implements GoogleGenAIClient {
   public models: {
     generateContent: (...args: unknown[]) => Promise<GoogleGenAIResponse>;
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     generateContentStream: (...args: unknown[]) => Promise<AsyncGenerator<GoogleGenAIResponse, any, unknown>>;
   };
   public chats: {

dev-packages/e2e-tests/.eslintrc.js

@@ -3,7 +3,7 @@ module.exports = {
     node: true,
   },
   // todo: remove regexp plugin from here once we add it to base.js eslint config for the whole project
-  extends: ['../../.eslintrc.js', 'plugin:regexp/recommended'],
+  extends: ['../.eslintrc.js', 'plugin:regexp/recommended'],
   plugins: ['regexp'],
   ignorePatterns: ['test-applications/**', 'tmp/**'],
   parserOptions: {

dev-packages/external-contributor-gh-action/.eslintrc.cjs

@@ -1,6 +1,6 @@
 module.exports = {
   // todo: remove regexp plugin from here once we add it to base.js eslint config for the whole project
-  extends: ['../../.eslintrc.js', 'plugin:regexp/recommended'],
+  extends: ['../.eslintrc.js', 'plugin:regexp/recommended'],
   plugins: ['regexp'],
   parserOptions: {
     sourceType: 'module',

dev-packages/node-core-integration-tests/.eslintrc.js

@@ -3,7 +3,7 @@ module.exports = {
     node: true,
   },
   // todo: remove regexp plugin from here once we add it to base.js eslint config for the whole project
-  extends: ['../../.eslintrc.js', 'plugin:regexp/recommended'],
+  extends: ['../.eslintrc.js', 'plugin:regexp/recommended'],
   plugins: ['regexp'],
   overrides: [
     {

dev-packages/node-integration-tests/.eslintrc.js

@@ -3,7 +3,7 @@ module.exports = {
     node: true,
   },
   // todo: remove regexp plugin from here once we add it to base.js eslint config for the whole project
-  extends: ['../../.eslintrc.js', 'plugin:regexp/recommended'],
+  extends: ['../.eslintrc.js', 'plugin:regexp/recommended'],
   plugins: ['regexp'],
   overrides: [
     {

dev-packages/node-overhead-gh-action/.eslintrc.cjs

@@ -3,7 +3,7 @@ module.exports = {
     node: true,
   },
   // todo: remove regexp plugin from here once we add it to base.js eslint config for the whole project
-  extends: ['../../.eslintrc.js', 'plugin:regexp/recommended'],
+  extends: ['../.eslintrc.js', 'plugin:regexp/recommended'],
   plugins: ['regexp'],
   overrides: [
     {

dev-packages/rollup-utils/.eslintrc.cjs

@@ -1,6 +1,6 @@
 module.exports = {
   // todo: remove regexp plugin from here once we add it to base.js eslint config for the whole project
-  extends: ['../../.eslintrc.js', 'plugin:regexp/recommended'],
+  extends: ['../.eslintrc.js', 'plugin:regexp/recommended'],
   plugins: ['regexp'],
   ignorePatterns: ['otelLoaderTemplate.js.tmpl'],
   sourceType: 'module',

dev-packages/size-limit-gh-action/.eslintrc.cjs

@@ -1,6 +1,6 @@
 module.exports = {
   // todo: remove regexp plugin from here once we add it to base.js eslint config for the whole project
-  extends: ['../../.eslintrc.js', 'plugin:regexp/recommended'],
+  extends: ['../.eslintrc.js', 'plugin:regexp/recommended'],
   plugins: ['regexp'],
   parserOptions: {
     sourceType: 'module',

dev-packages/test-utils/.eslintrc.js

@@ -3,6 +3,6 @@ module.exports = {
     node: true,
   },
   // todo: remove regexp plugin from here once we add it to base.js eslint config for the whole project
-  extends: ['../../.eslintrc.js', 'plugin:regexp/recommended'],
+  extends: ['../.eslintrc.js', 'plugin:regexp/recommended'],
   plugins: ['regexp'],
 };

packages/aws-serverless/src/integration/instrumentation-aws-lambda/types.ts

@@ -15,6 +15,7 @@
  * limitations under the License.
  */
 
+/* eslint-disable @typescript-eslint/no-explicit-any */
 import type { Context as OtelContext, Span } from '@opentelemetry/api';
 import type { InstrumentationConfig } from '@opentelemetry/instrumentation';
 import type { Context } from 'aws-lambda';

packages/cloudflare/src/durableobject.ts

@@ -28,7 +28,8 @@ type MethodWrapperOptions = {
 };
 
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
-type OriginalMethod = (...args: any[]) => any;
+type UncheckedMethod = (...args: any[]) => any;
+type OriginalMethod = UncheckedMethod;
 
 function wrapMethodWithSentry<T extends OriginalMethod>(
   wrapperOptions: MethodWrapperOptions,
@@ -269,8 +270,7 @@ export function instrumentDurableObjectWithSentry<
           // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access, @typescript-eslint/no-explicit-any
           (obj as any)[method] = wrapMethodWithSentry(
             { options, context, spanName: method, spanOp: 'rpc' },
-            // eslint-disable-next-line @typescript-eslint/no-explicit-any
-            value as (...args: any[]) => any,
+            value as UncheckedMethod,
           );
         }
       }
@@ -332,7 +332,7 @@ function instrumentPrototype<T extends NewableFunction>(target: T, methodsToInst
     }
 
     // Create a wrapper that gets context/options from the instance at runtime
-    const wrappedMethod = function (this: any, ...args: any[]): unknown {
+    const wrappedMethod = function (this: unknown, ...args: unknown[]): unknown {
       const thisWithSentry = this as {
         __SENTRY_CONTEXT__: DurableObjectState;
         __SENTRY_OPTIONS__: CloudflareOptions;
@@ -342,7 +342,7 @@ function instrumentPrototype<T extends NewableFunction>(target: T, methodsToInst
 
       if (!instanceOptions) {
         // Fallback to original method if no Sentry data found
-        return (originalMethod as (...args: any[]) => any).apply(this, args);
+        return (originalMethod as UncheckedMethod).apply(this, args);
       }
 
       // Use the existing wrapper but with instance-specific context/options
@@ -353,12 +353,12 @@ function instrumentPrototype<T extends NewableFunction>(target: T, methodsToInst
           spanName: methodName,
           spanOp: 'rpc',
         },
-        originalMethod as (...args: any[]) => any,
+        originalMethod as UncheckedMethod,
         undefined,
         true, // noMark = true since we'll mark the prototype method
       );
 
-      return (wrapper as (...args: any[]) => any).apply(this, args);
+      return wrapper.apply(this, args);
     };
 
     markAsInstrumented(wrappedMethod);

packages/core/src/integrations/supabase.ts

@@ -1,6 +1,7 @@
 // Based on Kamil Ogórek's work on:
 // https://github.com/supabase-community/sentry-integration-js
 
+/* eslint-disable @typescript-eslint/no-explicit-any */
 /* eslint-disable max-lines */
 import { addBreadcrumb } from '../breadcrumbs';
 import { DEBUG_BUILD } from '../debug-build';

packages/core/src/types-hoist/breadcrumb.ts

@@ -52,6 +52,7 @@ export interface Breadcrumb {
    *
    * @summary Arbitrary data associated with this breadcrumb.
    */
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   data?: { [key: string]: any };
 
   /**
@@ -70,6 +71,7 @@ export interface Breadcrumb {
 
 /** JSDoc */
 export interface BreadcrumbHint {
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   [key: string]: any;
 }
 
@@ -90,6 +92,7 @@ export interface XhrBreadcrumbData {
 }
 
 export interface FetchBreadcrumbHint {
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   input: any[];
   data?: unknown;
   response?: unknown;

packages/core/src/types-hoist/context.ts

@@ -99,6 +99,7 @@ export interface ResponseContext extends Record<string, unknown> {
 }
 
 export interface TraceContext extends Record<string, unknown> {
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   data?: { [key: string]: any };
   op?: string;
   parent_span_id?: string;

packages/core/src/types-hoist/error.ts

@@ -2,5 +2,7 @@
  * Just an Error object with arbitrary attributes attached to it.
  */
 export interface ExtendedError extends Error {
+  // TODO: fix in v11, convert any to unknown
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   [key: string]: any;
 }

packages/core/src/types-hoist/event.ts

@@ -83,6 +83,7 @@ export interface EventHint {
   syntheticException?: Error | null;
   originalException?: unknown;
   attachments?: Attachment[];
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   data?: any;
   integrations?: string[];
 }

packages/core/src/types-hoist/instrument.ts

@@ -47,6 +47,7 @@ interface SentryFetchData {
 }
 
 export interface HandlerDataFetch {
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   args: any[];
   fetchData: SentryFetchData; // This data is among other things dumped directly onto the fetch breadcrumb data
   startTimestamp: number;
@@ -74,6 +75,7 @@ export interface HandlerDataDom {
 
 export interface HandlerDataConsole {
   level: ConsoleLevel;
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   args: any[];
 }
 

packages/core/src/types-hoist/integration.ts

@@ -47,4 +47,5 @@ export interface Integration {
  * An integration in function form.
  * This is expected to return an integration.
  */
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
 export type IntegrationFn<IntegrationType = Integration> = (...rest: any[]) => IntegrationType;

packages/core/src/types-hoist/misc.ts

@@ -4,6 +4,8 @@ import type { QueryParams } from './request';
  * Data extracted from an incoming request to a node server
  */
 export interface ExtractedNodeRequestData {
+  // TODO: fix in v11, convert any to unknown
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   [key: string]: any;
 
   /** Specific headers from the request */

packages/core/src/types-hoist/polymorphics.ts

@@ -50,12 +50,14 @@ type NextjsRequest = NodeRequest & {
     [key: string]: string;
   };
   query?: {
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     [key: string]: any;
   };
 };
 
 type ExpressRequest = NodeRequest & {
   baseUrl?: string;
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   body?: string | { [key: string]: any };
   host?: string;
   hostname?: string;
@@ -70,9 +72,11 @@ type ExpressRequest = NodeRequest & {
     ];
   };
   query?: {
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     [key: string]: any;
   };
   user?: {
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     [key: string]: any;
   };
   _reconstructedRoute?: string;

packages/core/src/types-hoist/samplingcontext.ts

@@ -6,6 +6,8 @@ import type { SpanAttributes } from './span';
  * Context data passed by the user when starting a transaction, to be used by the tracesSampler method.
  */
 export interface CustomSamplingContext {
+  // TODO: fix in v11, convert any to unknown
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   [key: string]: any;
 }
 

packages/core/src/types-hoist/stackframe.ts

@@ -13,7 +13,11 @@ export interface StackFrame {
   in_app?: boolean;
   instruction_addr?: string;
   addr_mode?: string;
+  // TODO: fix in v11, convert any to unknown
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   vars?: { [key: string]: any };
   debug_id?: string;
+  // TODO: fix in v11, convert any to unknown
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   module_metadata?: any;
 }

packages/core/src/types-hoist/user.ts

@@ -2,6 +2,8 @@
  * An interface describing a user of an application or a handled request.
  */
 export interface User {
+  // TODO: fix in v11, convert any to unknown
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   [key: string]: any;
   id?: string | number;
   ip_address?: string | null;

packages/core/src/utils/aggregate-errors.ts

@@ -57,14 +57,14 @@ function aggregateExceptionsFromError(
   // Recursively call this function in order to walk down a chain of errors
   if (isInstanceOf(error[key], Error)) {
     applyExceptionGroupFieldsForParentException(exception, exceptionId);
-    const newException = exceptionFromErrorImplementation(parser, error[key]);
+    const newException = exceptionFromErrorImplementation(parser, error[key] as Error);
     const newExceptionId = newExceptions.length;
     applyExceptionGroupFieldsForChildException(newException, key, newExceptionId, exceptionId);
     newExceptions = aggregateExceptionsFromError(
       exceptionFromErrorImplementation,
       parser,
       limit,
-      error[key],
+      error[key] as ExtendedError,
       key,
       [newException, ...newExceptions],
       newException,
@@ -78,14 +78,14 @@ function aggregateExceptionsFromError(
     error.errors.forEach((childError, i) => {
       if (isInstanceOf(childError, Error)) {
         applyExceptionGroupFieldsForParentException(exception, exceptionId);
-        const newException = exceptionFromErrorImplementation(parser, childError);
+        const newException = exceptionFromErrorImplementation(parser, childError as Error);
         const newExceptionId = newExceptions.length;
         applyExceptionGroupFieldsForChildException(newException, `errors[${i}]`, newExceptionId, exceptionId);
         newExceptions = aggregateExceptionsFromError(
           exceptionFromErrorImplementation,
           parser,
           limit,
-          childError,
+          childError as ExtendedError,
           key,
           [newException, ...newExceptions],
           newException,

packages/core/src/utils/is.ts

@@ -180,7 +180,9 @@ export function isSyntheticEvent(wat: unknown): boolean {
  * @param base A constructor to be used in a check.
  * @returns A boolean representing the result.
  */
-export function isInstanceOf(wat: any, base: any): boolean {
+// TODO: fix in v11, convert any to unknown
+// export function isInstanceOf<T>(wat: unknown, base: { new (...args: any[]): T }): wat is T {
+export function isInstanceOf<T>(wat: any, base: any): wat is T {
   try {
     return wat instanceof base;
   } catch {