chore: Enable all Rules and Cleanup Configuration

.github/workflows/build.yml

@@ -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
 

.oxlintrc.json

@@ -0,0 +1,165 @@
+{
+  "$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": true }],
+        "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",
+        "typescript/prefer-optional-chain": ["error"]
+      }
+    },
+    {
+      "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",
+        "typescript/prefer-optional-chain": "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/**"
+  ]
+}

dev-packages/.oxlintrc.json

@@ -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/browser-integration-tests/.oxlintrc.json

@@ -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/browser-integration-tests/package.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",

dev-packages/bundler-tests/.oxlintrc.json

@@ -0,0 +1,4 @@
+{
+  "$schema": "../../node_modules/oxlint/configuration_schema.json",
+  "extends": ["../.oxlintrc.json"]
+}

dev-packages/clear-cache-gh-action/.oxlintrc.json

@@ -0,0 +1,4 @@
+{
+  "$schema": "../../node_modules/oxlint/configuration_schema.json",
+  "extends": ["../.oxlintrc.json"]
+}

dev-packages/clear-cache-gh-action/package.json

@@ -10,8 +10,8 @@
   "main": "index.mjs",
   "type": "module",
   "scripts": {
-    "lint": "eslint . --format stylish",
-    "fix": "eslint . --format stylish --fix"
+    "lint": "oxlint .",
+    "fix": "oxlint . --fix"
   },
   "dependencies": {
     "@actions/core": "1.10.1",