chore: migrate to oxlint (#19134)

Replaces eslint with oxlint, we had an outdated Eslint 8.x setup anyways
and we needed to either upgrade it or move to something else. Oxlint
brings so much speed gains given how slow linting is, and almost no one
is running it locally project wide because of how slow it is.

The changes can look like a lot but most of it is just:

- Comment Replacement due to rule name changes.
- Config file swapping (from `.eslintrc.js` to `.oxlintrc.json`

I downgraded a few rules, but they will be cleaned up in a follow up PR
in

## Benchmarks

### Overall

| Metric | Before (ESLint) | After (Oxlint) | Speedup |
|--------|-----------------|----------------|---------|
| CI Time | ~6 minutes | ~10 seconds | **36x**\* |

Note that:

- ~Lerna adds a considerable overhead that eats a lot of the gains we
can potentially get.~ We removed Lerna and also we will just lint the
entire project rather than go package by package.
- ~CI time is hogged by building types step~ Not relevant anymore,
typeaware mode works wonderfully after
[oxc-project/tsgolint#739](oxc-project/tsgolint#739)
was merged.

#### SDK Packages

| Package           | Files | ESLint | Oxlint | Speedup  |
| ----------------- | ----- | ------ | ------ | -------- |
| `core`            | 365   | 9.6s   | 53ms   | **181x** |
| `browser`         | 136   | 6.8s   | 55ms   | **124x** |
| `node`            | 105   | 6.1s   | 64ms   | **95x**  |
| `node-core`       | 101   | 6.2s   | 56ms   | **111x** |
| `nextjs`          | 181   | 10.9s  | 79ms   | **138x** |
| `sveltekit`       | 63    | 6.4s   | 71ms   | **90x**  |
| `opentelemetry`   | 58    | 4.3s   | 52ms   | **83x**  |
| `cloudflare`      | 43    | 3.8s   | 45ms   | **84x**  |
| `remix`           | 38    | 7.1s   | 42ms   | **169x** |
| `react`           | 39    | 6.5s   | 49ms   | **133x** |
| `feedback`        | 38    | 3.8s   | 48ms   | **79x**  |
| `replay-internal` | 152   | 5.6s   | 38ms   | **147x** |
| `vue`             | 24    | 4.0s   | 48ms   | **83x**  |
| `svelte`          | 15    | 4.0s   | 52ms   | **77x**  |
| `angular`         | 12    | 3.7s   | 37ms   | **100x** |

#### Dev Packages

| Package | Files | ESLint | Oxlint | Speedup |
| ------------------------------ | ----- | -------- | ------ | --------
|
| `browser-integration-tests` | 778 | 10.8s | 209ms | **52x** |
| `node-integration-tests` | 605 | 9.0s | 291ms | **31x** |
| `node-core-integration-tests` | 268 | 6.2s | 74ms | **84x** |
| `e2e-tests` | 10 | 2.6s | 44ms | **59x** |
| `cloudflare-integration-tests` | 27 | 2.5s | 35ms | **71x** |
| `test-utils` | 5 | 2.4s | 21ms | **114x** |
| `rollup-utils`                 | 13    | ❌ error | 22ms   | N/A      |
| `bundler-tests`                | 3     | ❌ error | 51ms   | N/A      |

---

closes #19222

Author: logaretm

.eslintrc.js

@@ -308,7 +308,7 @@ jobs:
         with:
           dependency_cache_key: ${{ needs.job_build.outputs.dependency_cache_key }}
       - name: Lint source files
-        run: yarn lint:eslint
+        run: yarn lint:oxlint
       - name: Lint for ES compatibility
         run: yarn lint:es-compatibility
 

.github/workflows/build.yml

@@ -0,0 +1,168 @@
+{
+  "$schema": "./node_modules/oxlint/configuration_schema.json",
+  "plugins": ["typescript", "import", "jsdoc", "jest", "vitest"],
+  "jsPlugins": [
+    {
+      "name": "sdk",
+      "specifier": "@sentry-internal/eslint-plugin-sdk"
+    }
+  ],
+  "categories": {},
+  "rules": {
+    // === Base rules from eslint-config-sdk/base.js ===
+    "no-console": "error",
+    "no-alert": "error",
+    "no-param-reassign": "error",
+    "prefer-template": "error",
+    "no-bitwise": "error",
+    "complexity": ["error", { "max": 33 }],
+    "no-unused-expressions": ["error", { "allowShortCircuit": true }],
+    "guard-for-in": "error",
+    "array-callback-return": ["error", { "allowImplicit": true }],
+    "quotes": ["error", "single", { "avoidEscape": true }],
+    "no-return-await": "error",
+    "max-lines": ["error", { "max": 300, "skipComments": true, "skipBlankLines": true }],
+
+    // === Import rules ===
+    "import/namespace": "off",
+    "import/no-unresolved": "off",
+
+    // === Jest/Vitest rules ===
+    "jest/no-focused-tests": "error",
+    "jest/no-disabled-tests": "error",
+
+    // === Rules turned off (not enforced in ESLint or causing false positives) ===
+    "no-control-regex": "off",
+    "jsdoc/check-tag-names": "off",
+    "jsdoc/require-yields": "off",
+    "no-useless-rename": "off",
+    "no-constant-binary-expression": "off",
+    "jest/no-conditional-expect": "off",
+    "jest/expect-expect": "off",
+    "jest/no-standalone-expect": "off",
+    "jest/require-to-throw-message": "off",
+    "jest/valid-title": "off",
+    "jest/no-export": "off",
+    "jest/valid-describe-callback": "off",
+    "vitest/hoisted-apis-on-top": "off",
+    "vitest/no-conditional-tests": "off",
+    "no-unsafe-optional-chaining": "off",
+    "no-eval": "off",
+    "no-import-assign": "off",
+
+    // === Custom SDK rules (via JS plugin) ===
+    "sdk/no-eq-empty": "error"
+  },
+  "overrides": [
+    {
+      "files": ["**/*.ts", "**/*.tsx", "**/*.d.ts"],
+      "rules": {
+        "typescript/ban-ts-comment": "error",
+        "typescript/consistent-type-imports": "error",
+        "typescript/no-unnecessary-type-assertion": "error",
+        "typescript/prefer-for-of": "error",
+        // "typescript/no-floating-promises": ["error", { "ignoreVoid": false }],
+        "typescript/no-dynamic-delete": "error",
+        // "typescript/no-unsafe-member-access": "error",
+        "typescript/unbound-method": "error",
+        "typescript/no-explicit-any": "error",
+        "typescript/no-empty-function": "off",
+
+        // === FIXME: Rules to turn back as error ===
+        "typescript/prefer-optional-chain": "warn",
+        "typescript/no-floating-promises": "warn",
+        "typescript/no-unsafe-member-access": "warn"
+      }
+    },
+    {
+      "files": ["**/*.js", "**/*.mjs", "**/*.cjs"],
+      "rules": {
+        "typescript/ban-ts-comment": "off",
+        "typescript/consistent-type-imports": "off",
+        "typescript/prefer-optional-chain": "off",
+        "typescript/no-unnecessary-type-assertion": "off",
+        "typescript/prefer-for-of": "off",
+        "typescript/no-floating-promises": "off",
+        "typescript/no-dynamic-delete": "off",
+        "typescript/no-unsafe-member-access": "off",
+        "typescript/unbound-method": "off",
+        "typescript/no-explicit-any": "off"
+      }
+    },
+    {
+      "files": [
+        "**/*.test.ts",
+        "**/*.test.tsx",
+        "**/*.test.js",
+        "**/*.test.jsx",
+        "**/test/**",
+        "**/tests/**",
+        "**/suites/**",
+        "**/loader-suites/**"
+      ],
+      "rules": {
+        "typescript/explicit-function-return-type": "off",
+        "no-unused-expressions": "off",
+        "typescript/no-unused-expressions": "off",
+        "typescript/no-unnecessary-type-assertion": "off",
+        "typescript/no-unsafe-member-access": "off",
+        "typescript/no-explicit-any": "off",
+        "typescript/no-non-null-assertion": "off",
+        "typescript/no-floating-promises": "off",
+        "typescript/unbound-method": "off",
+        "max-lines": "off",
+        "complexity": "off"
+      }
+    },
+    {
+      "files": ["*.tsx"],
+      "rules": {
+        "jsdoc/require-jsdoc": "off"
+      }
+    },
+    {
+      "files": ["*.config.js", "*.config.mjs", "*.config.ts", "vite.config.ts", ".size-limit.js"],
+      "rules": {
+        "no-console": "off",
+        "max-lines": "off"
+      }
+    },
+    {
+      "files": [
+        "**/scenarios/**",
+        "**/rollup-utils/**",
+        "**/bundle-analyzer-scenarios/**",
+        "**/bundle-analyzer-scenarios/*.cjs",
+        "**/bundle-analyzer-scenarios/*.js"
+      ],
+      "rules": {
+        "no-console": "off"
+      }
+    },
+    {
+      "files": ["**/src/**"],
+      "rules": {
+        "no-restricted-globals": ["error", "window", "document", "location", "navigator"],
+        "sdk/no-class-field-initializers": "error",
+        "sdk/no-regexp-constructor": "error"
+      }
+    }
+  ],
+  "env": {
+    "es2017": true,
+    "node": true
+  },
+  "globals": {},
+  "ignorePatterns": [
+    "coverage/**",
+    "build/**",
+    "dist/**",
+    "cjs/**",
+    "esm/**",
+    "examples/**",
+    "test/manual/**",
+    "types/**",
+    "scripts/*.js",
+    "node_modules/**"
+  ]
+}

.oxlintrc.json

@@ -20,7 +20,7 @@ Use **yarn**: `yarn install`, `yarn build:dev`, `yarn test`, `yarn lint`
 | `yarn build:dev:filter @sentry/<pkg>` | Build one package + deps      |
 | `yarn build:bundle`                   | Browser bundles only          |
 | `yarn test`                           | All unit tests                |
-| `yarn lint`                           | ESLint + Oxfmt                |
+| `yarn lint`                           | Oxlint + Oxfmt                |
 | `yarn fix`                            | Auto-fix lint + format        |
 | `yarn format`                         | Auto-fix formatting (Oxfmt)   |
 

AGENTS.md

@@ -0,0 +1,9 @@
+{
+  "$schema": "../node_modules/oxlint/configuration_schema.json",
+  "extends": ["../.oxlintrc.json"],
+  "rules": {
+    "typescript/no-explicit-any": "off",
+    "max-lines": "off",
+    "no-unused-expressions": "off"
+  }
+}

dev-packages/.eslintrc.js

@@ -0,0 +1,31 @@
+{
+  "$schema": "../../node_modules/oxlint/configuration_schema.json",
+  "extends": ["../.oxlintrc.json"],
+  "env": {
+    "browser": true,
+    "node": true
+  },
+  "ignorePatterns": [
+    "suites/**/subject.js",
+    "suites/**/dist/*",
+    "loader-suites/**/dist/*",
+    "loader-suites/**/subject.js",
+    "scripts/**",
+    "fixtures/**",
+    "tmp/**"
+  ],
+  "overrides": [
+    {
+      "files": ["**/*.ts", "**/*.tsx"],
+      "rules": {
+        "typescript/no-unsafe-member-access": "off"
+      }
+    },
+    {
+      "files": ["loader-suites/**/{subject,init}.js"],
+      "globals": {
+        "Sentry": "readonly"
+      }
+    }
+  ]
+}

dev-packages/.oxlintrc.json

@@ -10,8 +10,8 @@
   "scripts": {
     "clean": "rimraf -g suites/**/dist loader-suites/**/dist tmp",
     "install-browsers": "[[ -z \"$SKIP_PLAYWRIGHT_BROWSER_INSTALL\" ]] && npx playwright install --with-deps || echo 'Skipping browser installation'",
-    "lint": "eslint . --format stylish",
-    "fix": "eslint . --format stylish --fix",
+    "lint": "oxlint .",
+    "fix": "oxlint . --fix",
     "type-check": "tsc",
     "postinstall": "yarn install-browsers",
     "pretest": "yarn clean && yarn type-check",