@wistia/oxlint-config 1.0.4 → 1.1.0

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "@wistia/oxlint-config",
-  "version": "1.0.4",
+  "version": "1.1.0",
   "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/base.mjs

@@ -684,8 +684,8 @@ export const baseRules = {
 
     // Require following curly brace conventions
     // https://oxc.rs/docs/guide/usage/linter/rules/eslint/curly.html
-    // Decision: stylistic choice best left to formatter
-    'eslint/curly': 'off',
+    // Decision: enforce curly braces for all control statements because even the `multiline` option can allow subtle bugs
+    'eslint/curly': ['error', 'all'],
 
     // Require function names to match the name of the variable or property to which they are assigned
     // https://oxc.rs/docs/guide/usage/linter/rules/eslint/func-name-matching.html

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',