@wistia/oxlint-config 0.4.6 → 0.5.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wistia/oxlint-config",
-  "version": "0.4.6",
+  "version": "0.5.0",
   "description": "Wistia's Oxlint configurations",
   "packageManager": "yarn@4.14.1",
   "type": "module",
@@ -16,13 +16,29 @@
       "types": "./index.d.mts",
       "default": "./index.mjs"
     },
-    "./configs/*": "./configs/*",
-    "./rules/*": "./rules/*",
     "./javascript": "./configs/javascript.mjs",
     "./typescript": "./configs/typescript.mjs",
     "./react": "./configs/react.mjs",
     "./node": "./configs/node.mjs",
-    "./vitest": "./configs/vitest.mjs"
+    "./vitest": "./configs/vitest.mjs",
+    "./playwright": "./configs/playwright.mjs",
+    "./storybook": "./configs/storybook.mjs",
+    "./styled-components": "./configs/styled-components.mjs",
+    "./testing-library": "./configs/testing-library.mjs",
+    "./rules/barrel-files": "./rules/barrel-files.mjs",
+    "./rules/base": "./rules/base.mjs",
+    "./rules/import": "./rules/import.mjs",
+    "./rules/node": "./rules/node.mjs",
+    "./rules/playwright": "./rules/playwright.mjs",
+    "./rules/promise": "./rules/promise.mjs",
+    "./rules/react-a11y": "./rules/react-a11y.mjs",
+    "./rules/react": "./rules/react.mjs",
+    "./rules/storybook": "./rules/storybook.mjs",
+    "./rules/styled-components": "./rules/styled-components.mjs",
+    "./rules/testing-library": "./rules/testing-library.mjs",
+    "./rules/typescript": "./rules/typescript.mjs",
+    "./rules/unicorn": "./rules/unicorn.mjs",
+    "./rules/vitest": "./rules/vitest.mjs"
   },
   "scripts": {
     "changeset": "changeset",
@@ -45,7 +61,7 @@
     "eslint-plugin-jest-dom": "^5.5.0",
     "eslint-plugin-n": "^17.24.0",
     "eslint-plugin-no-only-tests": "^3.3.0",
-    "eslint-plugin-playwright": "^2.10.2",
+    "eslint-plugin-playwright": "^2.10.1",
     "eslint-plugin-storybook": "^10.3.5",
     "eslint-plugin-styled-components": "^0.0.0",
     "eslint-plugin-styled-components-a11y": "^2.2.1",
@@ -54,13 +70,13 @@
   },
   "devDependencies": {
     "@changesets/changelog-github": "^0.6.0",
-    "@changesets/cli": "^2.31.0",
-    "eslint": "^10.2.1",
-    "oxfmt": "^0.46.0",
-    "oxlint": "^1.61.0",
+    "@changesets/cli": "^2.30.0",
+    "eslint": "^10.2.0",
+    "oxfmt": "^0.45.0",
+    "oxlint": "^1.60.0",
     "oxlint-tsgolint": "^0.21.1",
     "storybook": "^10.3.5",
-    "vitest": "^4.1.5"
+    "vitest": "^4.1.4"
   },
   "engines": {
     "node": "^20.19.0 || ^22.13.0 || >=24 || >=26"

package/rules/base.mjs

@@ -389,14 +389,8 @@ export const baseRules = {
 
     // Disallow magic numbers
     // https://eslint.org/docs/rules/no-magic-numbers
-    'eslint/no-magic-numbers': [
-      'error',
-      {
-        ignore: [-1, 0, 1, 2, 3, 4, 5, 100, 1024, 1000, 10000],
-        ignoreArrayIndexes: true,
-        ignoreDefaultValues: true,
-      },
-    ],
+    // decision:
+    'eslint/no-magic-numbers': 'off',
 
     // Disallow use of chained assignment expressions
     // https://eslint.org/docs/rules/no-multi-assign

package/rules/typescript.mjs

@@ -1,16 +1,6 @@
 export const typescriptRules = {
   plugins: ['typescript'],
   rules: {
-    //Disable base eslint rules superseded by typescript equivalents --
-    'eslint/require-await': 'off',
-
-    // recommended to be disabled for TypeScript projects
-    // https://github.com/typescript-eslint/typescript-eslint/blob/main/docs/linting/TROUBLESHOOTING.md#i-get-errors-from-the-no-undef-rule-about-global-variables-not-being-defined-even-though-there-are-no-typescript-errors
-    'eslint/no-undef': 'off',
-
-    // Override base no-void to allow void as a statement (common in TS for fire-and-forget promises)
-    'eslint/no-void': ['error', { allowAsStatement: true }],
-
     // Override base id-length to disable checking generic type parameters.
     // ESLint's id-length never sees TS type params (they're not in the ESLint AST),
     // but oxlint has native TS support so it checks them. Setting checkGeneric to
@@ -25,6 +15,19 @@ export const typescriptRules = {
       },
     ],
 
+    // This is redundant with TypeScript's noUnusedLocals/noUnusedParameters compiler options
+    'eslint/no-unused-vars': 'off',
+
+    // Recommended to be disabled for TypeScript projects
+    // https://github.com/typescript-eslint/typescript-eslint/blob/main/docs/linting/TROUBLESHOOTING.md#i-get-errors-from-the-no-undef-rule-about-global-variables-not-being-defined-even-though-there-are-no-typescript-errors
+    'eslint/no-undef': 'off',
+
+    // Override base no-void to allow void as a statement (common in TS for fire-and-forget promises)
+    'eslint/no-void': ['error', { allowAsStatement: true }],
+
+    // Superseded by `typescript/require-await` below
+    'eslint/require-await': 'off',
+
     // Require that function overload signatures be consecutive
     // https://typescript-eslint.io/rules/adjacent-overload-signatures
     'typescript/adjacent-overload-signatures': 'error',