@wistia/oxlint-config 1.0.5 → 1.1.1

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "@wistia/oxlint-config",
-  "version": "1.0.5",
+  "version": "1.1.1",
   "description": "Wistia's Oxlint configurations",
-  "packageManager": "yarn@4.14.1",
+  "packageManager": "yarn@4.17.0",
   "type": "module",
   "files": [
     "configs",
@@ -51,13 +51,13 @@
     "validate": "vitest run test/validate-configs.test.mjs"
   },
   "peerDependencies": {
-    "eslint": ">= 10.4.1",
-    "oxlint": ">= 1.69.0",
+    "eslint": ">= 10.5.0",
+    "oxlint": ">= 1.70.0",
     "oxlint-tsgolint": ">= 0.23.0"
   },
   "dependencies": {
-    "@eslint-react/eslint-plugin": "^5.8.16",
-    "@vitest/eslint-plugin": "^1.6.19",
+    "@eslint-react/eslint-plugin": "^5.9.0",
+    "@vitest/eslint-plugin": "^1.6.20",
     "confusing-browser-globals": "^1.0.11",
     "eslint-plugin-barrel-files": "^3.0.1",
     "eslint-plugin-import-x": "^4.16.2",
@@ -65,7 +65,7 @@
     "eslint-plugin-n": "^18.1.0",
     "eslint-plugin-no-only-tests": "^3.4.0",
     "eslint-plugin-playwright": "^2.10.4",
-    "eslint-plugin-storybook": "^10.4.2",
+    "eslint-plugin-storybook": "^10.4.5",
     "eslint-plugin-styled-components": "^0.0.0",
     "eslint-plugin-styled-components-a11y": "^2.2.1",
     "eslint-plugin-testing-library": "^7.16.2",
@@ -74,12 +74,12 @@
   "devDependencies": {
     "@changesets/changelog-github": "^0.7.0",
     "@changesets/cli": "^2.31.0",
-    "eslint": "^10.4.1",
-    "oxfmt": "^0.54.0",
-    "oxlint": "^1.69.0",
+    "eslint": "^10.5.0",
+    "oxfmt": "^0.55.0",
+    "oxlint": "^1.70.0",
     "oxlint-tsgolint": "^0.23.0",
-    "storybook": "^10.4.2",
-    "vitest": "^4.1.8"
+    "storybook": "^10.4.5",
+    "vitest": "^4.1.9"
   },
   "engines": {
     "node": "^20.19.0 || ^22.13.0 || >=24 || >=26"

package/rules/node.mjs

@@ -4,6 +4,10 @@ export const nodeRules = {
   plugins: ['node'],
   jsPlugins: [{ name: 'n', specifier: 'eslint-plugin-n' }],
   rules: {
+    // Require return statements after callbacks
+    // https://oxc.rs/docs/guide/usage/linter/rules/node/callback-return.html
+    'node/callback-return': 'error',
+
     // Require error handling in callbacks
     // https://oxc.rs/docs/guide/usage/linter/rules/node/handle-callback-err.html
     'node/handle-callback-err': 'error',
@@ -22,13 +26,12 @@ export const nodeRules = {
 
     // Disallow the use of process.env
     // https://oxc.rs/docs/guide/usage/linter/rules/node/no-process-env.html
-    // Decision: too opinionated for general use
+    // Decision: too opinionated for general use; individual projects can enable this rule if desired
     'node/no-process-env': 'off',
 
     // Require require() to be called in the top-level module scope
     // https://oxc.rs/docs/guide/usage/linter/rules/node/global-require.html
-    // Decision: restricts valid language feature
-    'node/global-require': 'off',
+    'node/global-require': 'error',
 
     //eslint-plugin-n rules via jsPlugins
 
@@ -70,7 +73,8 @@ export const nodeRules = {
 
     // Require return statements after callbacks
     // https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/callback-return.md
-    'n/callback-return': 'error',
+    // Decision: handled by native node/callback-return
+    'n/callback-return': 'off',
 
     // Enforce either module.exports or exports
     // https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/exports-style.md
@@ -78,7 +82,8 @@ export const nodeRules = {
 
     // Require require() to be called in the top-level module scope
     // https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/global-require.md
-    'n/global-require': 'error',
+    // Decision: handled by native node/global-require
+    'n/global-require': 'off',
 
     // Disallow require calls to be mixed with regular variable declarations
     // https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-mixed-requires.md

package/rules/react.mjs

@@ -272,6 +272,15 @@ export const reactRules = {
     // Decision: too opinionated for general use
     'react/state-in-constructor': 'off',
 
+    // Surface code that violates the Rules of React the compiler relies on
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/react-compiler.html
+    'react/react-compiler': 'error',
+
+    // Disallow string literals in JSX
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/jsx-no-literals.html
+    // Decision: stylistic/i18n rule, too strict for general use
+    'react/jsx-no-literals': 'off',
+
     //rules via jsPlugins (@eslint-react/eslint-plugin)
 
     // Validates usage of Error Boundaries instead of try/catch for child errors

package/rules/unicorn.mjs

@@ -95,6 +95,10 @@ export const unicornRules = {
     // https://oxc.rs/docs/guide/usage/linter/rules/unicorn/no-array-callback-reference.html
     'unicorn/no-array-callback-reference': 'error',
 
+    // Disallow `Array#fill()` with a reference type, which shares one reference across all slots
+    // https://oxc.rs/docs/guide/usage/linter/rules/unicorn/no-array-fill-with-reference-type.html
+    'unicorn/no-array-fill-with-reference-type': 'error',
+
     // Prefer `for...of` over `Array#forEach`
     // https://oxc.rs/docs/guide/usage/linter/rules/unicorn/no-array-for-each.html
     // Decision: `Array#forEach` is a legitimate iteration pattern
@@ -395,6 +399,10 @@ export const unicornRules = {
     // Decision: EventTarget vs EventEmitter is architectural; let consumers choose
     'unicorn/prefer-event-target': 'off',
 
+    // Prefer `export ... from` over importing then exporting in two steps
+    // https://oxc.rs/docs/guide/usage/linter/rules/unicorn/prefer-export-from.html
+    'unicorn/prefer-export-from': 'error',
+
     // Prefer `globalThis` over `window` / `self` / `global`
     // https://oxc.rs/docs/guide/usage/linter/rules/unicorn/prefer-global-this.html
     // Decision: we should consider enabling this in the future but it will involve a lot of refactoring
@@ -490,6 +498,10 @@ export const unicornRules = {
     // https://oxc.rs/docs/guide/usage/linter/rules/unicorn/prefer-set-size.html
     'unicorn/prefer-set-size': 'error',
 
+    // Prefer a single call over consecutive identical method calls (e.g. repeated `Array#push`)
+    // https://oxc.rs/docs/guide/usage/linter/rules/unicorn/prefer-single-call.html
+    'unicorn/prefer-single-call': 'error',
+
     // Prefer the spread operator over `Array.from(…)` etc.
     // https://oxc.rs/docs/guide/usage/linter/rules/unicorn/prefer-spread.html
     'unicorn/prefer-spread': 'error',