@wistia/oxlint-config 1.11.2 → 1.12.0

package/CHANGELOG.md

@@ -1,5 +1,26 @@
 # @wistia/oxlint-config
 
+## 1.12.0
+
+### Minor Changes
+
+- [#179](https://github.com/wistia/oxlint-config/pull/179) [`2cb8282`](https://github.com/wistia/oxlint-config/commit/2cb8282f1660a8c2b211ef86d7cbb9249500a1ec) Thanks [@okize](https://github.com/okize)! - feat: configure the React Compiler rule split from oxlint v1.79.0
+
+  - Bump `oxlint` peer minimum to `>= 1.79.0` and `oxfmt` peer minimum to `>= 0.64.0`
+  - oxlint v1.79.0 removes `react/react-compiler` (previously `error`) and splits it into 22 per-category React Compiler rules. All 22 are now configured:
+    - `error` — the upstream `recommended` preset: `react/error-boundaries`, `react/globals`, `react/immutability`, `react/incompatible-library`, `react/preserve-manual-memoization`, `react/purity`, `react/refs`, `react/set-state-in-effect`, `react/set-state-in-render`, `react/static-components`, `react/unsupported-syntax`, `react/use-memo`, `react/void-use-memo`
+    - `off` — duplicate existing native coverage: `react/hooks` (handled by `react/rules-of-hooks`), `react/exhaustive-effect-dependencies` and `react/memo-dependencies` (handled by `react/exhaustive-deps`)
+    - `off` — report compiler status rather than code defects: `react/invariant`, `react/todo`, `react/syntax`
+    - `off` — upstream keeps these off; revisit if the React Compiler transform is adopted: `react/no-deriving-state-in-effects`, `react/rule-suppression`, `react/capitalized-calls`
+  - Ten `@eslint-react` jsPlugin rules superseded by the new native rules flip from `error` to `off`: `error-boundaries`, `globals`, `immutability`, `purity`, `refs`, `set-state-in-effect`, `set-state-in-render`, `static-components`, `unsupported-syntax`, `use-memo`
+  - Behavior changes in v1.79.0 to rules already set to `error`, no config change required:
+    - `eslint/no-irregular-whitespace` now checks comments by default
+    - `eslint/no-redeclare` now runs in ES modules, skipping only the globals check
+    - `import/no-named-as-default` now reports local export conflicts
+    - `import/no-named-default` now flags type-only named defaults
+    - `react/rules-of-hooks` now reports hooks inside `try` blocks
+    - `react/exhaustive-deps` now handles destructured object dependencies
+
 ## 1.11.2
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wistia/oxlint-config",
-  "version": "1.11.2",
+  "version": "1.12.0",
   "description": "Wistia's Oxlint configurations",
   "keywords": [
     "linter",
@@ -95,16 +95,16 @@
     "@changesets/changelog-github": "^1.0.0",
     "@changesets/cli": "^3.0.0",
     "eslint": "^10.8.0",
-    "oxfmt": "^0.63.0",
-    "oxlint": "^1.78.0",
+    "oxfmt": "^0.64.0",
+    "oxlint": "^1.79.0",
     "oxlint-tsgolint": "^7.0.2001",
     "storybook": "^10.5.5",
     "vitest": "^4.1.10"
   },
   "peerDependencies": {
     "eslint": ">= 10.8.0",
-    "oxfmt": ">= 0.63.0",
-    "oxlint": ">= 1.78.0",
+    "oxfmt": ">= 0.64.0",
+    "oxlint": ">= 1.79.0",
     "oxlint-tsgolint": ">= 7.0.2001"
   },
   "peerDependenciesMeta": {

package/rules/react.mjs

@@ -279,9 +279,104 @@ 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',
+    // --- React Compiler rules ---
+
+    // Validate usage of Error Boundaries instead of try/catch for child errors
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/error-boundaries.html
+    'react/error-boundaries': 'error',
+
+    // Restrict reading or writing mutable globals during render
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/globals.html
+    'react/globals': 'error',
+
+    // Disallow mutating props, state, and other immutable values
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/immutability.html
+    'react/immutability': 'error',
+
+    // Flag library APIs known to be incompatible with memoization
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/incompatible-library.html
+    'react/incompatible-library': 'error',
+
+    // Ensure the compiler's inferred dependencies preserve existing manual memoization
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/preserve-manual-memoization.html
+    'react/preserve-manual-memoization': 'error',
+
+    // Validate that components and hooks are pure
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/purity.html
+    'react/purity': 'error',
+
+    // Validate that refs are not read or written during render
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/refs.html
+    'react/refs': 'error',
+
+    // Disallow calling setState synchronously in an effect
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/set-state-in-effect.html
+    'react/set-state-in-effect': 'error',
+
+    // Disallow setting state during render
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/set-state-in-render.html
+    'react/set-state-in-render': 'error',
+
+    // Enforce static component definitions
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/static-components.html
+    'react/static-components': 'error',
+
+    // Disallow syntax that React does not support
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/unsupported-syntax.html
+    'react/unsupported-syntax': 'error',
+
+    // Validate that useMemo callbacks return a value
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/use-memo.html
+    'react/use-memo': 'error',
+
+    // Validate that useMemo results are actually used
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/void-use-memo.html
+    'react/void-use-memo': 'error',
+
+    // Compiler-powered Rules of Hooks validation
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/hooks.html
+    // Decision: handled by native react/rules-of-hooks
+    'react/hooks': 'off',
+
+    // Validate effect dependency arrays are exhaustive without extraneous values
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/exhaustive-effect-dependencies.html
+    // Decision: handled by native react/exhaustive-deps
+    'react/exhaustive-effect-dependencies': 'off',
+
+    // Validate useMemo/useCallback dependency lists
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/memo-dependencies.html
+    // Decision: handled by native react/exhaustive-deps
+    'react/memo-dependencies': 'off',
+
+    // Report internal React Compiler invariant violations
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/invariant.html
+    // Decision: reports bugs in the compiler itself, not code defects
+    'react/invariant': 'off',
+
+    // Report code the React Compiler cannot analyze yet
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/todo.html
+    // Decision: reports compiler bail-outs, not code defects
+    'react/todo': 'off',
+
+    // Report invalid JavaScript the compiler encountered (e.g. const reassignment)
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/syntax.html
+    // Decision: invalid JS is already reported by base eslint rules like no-const-assign
+    'react/syntax': 'off',
+
+    // Disallow deriving values from state inside an effect and storing them back into state
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/no-deriving-state-in-effects.html
+    // Decision: upstream preset keeps this off; enable if we adopt the React Compiler
+    'react/no-deriving-state-in-effects': 'off',
+
+    // Report suppressions of React rules inside components (compiler skips those functions)
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/rule-suppression.html
+    // Decision: upstream preset keeps this off; enable if we adopt the React Compiler
+    'react/rule-suppression': 'off',
+
+    // Disallow calling capitalized functions directly during render
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/capitalized-calls.html
+    // Decision: upstream preset keeps this off; enable if we adopt the React Compiler
+    'react/capitalized-calls': 'off',
 
     // Disallow string literals in JSX
     // https://oxc.rs/docs/guide/usage/linter/rules/react/jsx-no-literals.html
@@ -292,11 +387,13 @@ export const reactRules = {
 
     // Validates usage of Error Boundaries instead of try/catch for child errors
     // https://eslint-react.xyz/docs/rules/error-boundaries
-    '@eslint-react/error-boundaries': 'error',
+    // Decision: handled by native react/error-boundaries
+    '@eslint-react/error-boundaries': 'off',
 
     // Restricts usage of global variables in React components
     // https://eslint-react.xyz/docs/rules/globals
-    '@eslint-react/globals': 'error',
+    // Decision: handled by native react/globals
+    '@eslint-react/globals': 'off',
 
     // Verify the list of the dependencies for Hooks like useEffect and similar
     // https://eslint-react.xyz/docs/rules/exhaustive-deps
@@ -305,7 +402,8 @@ export const reactRules = {
 
     // Validates against mutating props, state, and other immutable values
     // https://eslint-react.xyz/docs/rules/immutability
-    '@eslint-react/immutability': 'error',
+    // Decision: handled by native react/immutability
+    '@eslint-react/immutability': 'off',
 
     // Prevent accidental JS comments from being injected into JSX as text
     // https://eslint-react.xyz/docs/rules/jsx-no-comment-textnodes
@@ -520,11 +618,13 @@ export const reactRules = {
 
     // Validates that components/hooks are pure
     // https://eslint-react.xyz/docs/rules/purity
-    '@eslint-react/purity': 'error',
+    // Decision: handled by native react/purity
+    '@eslint-react/purity': 'off',
 
     // Validates correct usage of refs, not reading/writing during render
     // https://eslint-react.xyz/docs/rules/refs
-    '@eslint-react/refs': 'error',
+    // Decision: handled by native react/refs
+    '@eslint-react/refs': 'off',
 
     // Enforce Rules of Hooks
     // https://eslint-react.xyz/docs/rules/rules-of-hooks
@@ -533,23 +633,28 @@ export const reactRules = {
 
     // Validates against calling setState synchronously in an effect
     // https://eslint-react.xyz/docs/rules/set-state-in-effect
-    '@eslint-react/set-state-in-effect': 'error',
+    // Decision: handled by native react/set-state-in-effect
+    '@eslint-react/set-state-in-effect': 'off',
 
     // Validates against setting state during render
     // https://eslint-react.xyz/docs/rules/set-state-in-render
-    '@eslint-react/set-state-in-render': 'error',
+    // Decision: handled by native react/set-state-in-render
+    '@eslint-react/set-state-in-render': 'off',
 
     // Enforces static component definitions
     // https://eslint-react.xyz/docs/rules/static-components
-    '@eslint-react/static-components': 'error',
+    // Decision: handled by native react/static-components
+    '@eslint-react/static-components': 'off',
 
     // Validates against syntax that React does not support
     // https://eslint-react.xyz/docs/rules/unsupported-syntax
-    '@eslint-react/unsupported-syntax': 'error',
+    // Decision: handled by native react/unsupported-syntax
+    '@eslint-react/unsupported-syntax': 'off',
 
     // Validates usage of useMemo with a return value
     // https://eslint-react.xyz/docs/rules/use-memo
-    '@eslint-react/use-memo': 'error',
+    // Decision: handled by native react/use-memo
+    '@eslint-react/use-memo': 'off',
 
     // Ensure destructuring and symmetric naming of useState hook value and setter
     // https://eslint-react.xyz/docs/rules/use-state