@wistia/oxlint-config 0.7.4 → 0.7.5

package/rules/node.mjs

@@ -22,10 +22,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
     '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',
 
     //eslint-plugin-n rules via jsPlugins
@@ -136,27 +138,27 @@ export const nodeRules = {
 
     // Require error handling in callbacks
     // https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/handle-callback-err.md
-    // decision: handled by native node/handle-callback-err
+    // Decision: handled by native node/handle-callback-err
     'n/handle-callback-err': 'off',
 
     // Disallow the assignment to exports
     // https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-exports-assign.md
-    // decision: handled by native node/no-exports-assign
+    // Decision: handled by native node/no-exports-assign
     'n/no-exports-assign': 'off',
 
     // Disallow new operators with calls to require
     // https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-new-require.md
-    // decision: handled by native node/no-new-require
+    // Decision: handled by native node/no-new-require
     'n/no-new-require': 'off',
 
     // Disallow string concatenation with __dirname and __filename
     // https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-path-concat.md
-    // decision: handled by native node/no-path-concat
+    // Decision: handled by native node/no-path-concat
     'n/no-path-concat': 'off',
 
     // Enforce file extensions in import declarations
     // https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/file-extension-in-import.md
-    // decision: handled by native import/extensions and import-x-js config
+    // Decision: handled by native import/extensions and import-x-js config
     'n/file-extension-in-import': 'off',
 
     // Disallow string callbacks in setTimeout/setInterval
@@ -165,27 +167,27 @@ export const nodeRules = {
 
     // Disallow import declarations which import extraneous modules
     // https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-extraneous-import.md
-    // decision: handled by import-x-js/no-extraneous-dependencies
+    // Decision: handled by import-x-js/no-extraneous-dependencies
     'n/no-extraneous-import': 'off',
 
     // Disallow require() expressions which import extraneous modules
     // https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-extraneous-require.md
-    // decision: handled by import-x-js/no-extraneous-dependencies
+    // Decision: handled by import-x-js/no-extraneous-dependencies
     'n/no-extraneous-require': 'off',
 
     // Disallow import declarations which import non-existent modules
     // https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-missing-import.md
-    // decision: requires resolver configuration unavailable in oxlint jsPlugin runtime
+    // Decision: requires resolver configuration unavailable in oxlint jsPlugin runtime
     'n/no-missing-import': 'off',
 
     // Disallow require() expressions which import non-existent modules
     // https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-missing-require.md
-    // decision: requires resolver configuration unavailable in oxlint jsPlugin runtime
+    // Decision: requires resolver configuration unavailable in oxlint jsPlugin runtime
     'n/no-missing-require': 'off',
 
     // Disallow the use of process.env
     // https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-process-env.md
-    // decision: too restrictive for general use
+    // Decision: too restrictive for general use
     'n/no-process-env': 'off',
 
     // Disallow the use of process.exit()
@@ -194,7 +196,7 @@ export const nodeRules = {
 
     // Disallow synchronous methods
     // https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-sync.md
-    // decision: too restrictive for general use
+    // Decision: too restrictive for general use
     'n/no-sync': 'off',
 
     // Enforce the use of the global crypto
@@ -207,12 +209,12 @@ export const nodeRules = {
 
     // Disallow hiding core modules
     // https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/no-hide-core-modules.md
-    // decision: deprecated rule
+    // Decision: deprecated rule
     'n/no-hide-core-modules': 'off',
 
     // Require correct usage of hashbang (alias)
     // https://github.com/eslint-community/eslint-plugin-n/blob/HEAD/docs/rules/shebang.md
-    // decision: handled by n/hashbang (shebang is a deprecated alias)
+    // Decision: handled by n/hashbang (shebang is a deprecated alias)
     'n/shebang': 'off',
   },
 };

package/rules/promise.mjs

@@ -55,10 +55,12 @@ export const promiseRules = {
 
     // Prefer await to then()/catch()/finally() for reading Promise values
     // https://oxc.rs/docs/guide/usage/linter/rules/promise/prefer-await-to-then.html
+    // Decision: too opinionated for general use
     'promise/prefer-await-to-then': 'off',
 
     // Prefer async/await to the callback pattern
     // https://oxc.rs/docs/guide/usage/linter/rules/promise/prefer-await-to-callbacks.html
+    // Decision: too opinionated for general use
     'promise/prefer-await-to-callbacks': 'off',
 
     // Disallow use of non-standard Promise static methods

package/rules/react-a11y.mjs

@@ -50,6 +50,7 @@ export const reactA11yRules = {
 
     // Enforce that autocomplete attribute is correct
     // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/autocomplete-valid.html
+    // Decision: too opinionated for general use
     'jsx_a11y/autocomplete-valid': 'off',
 
     // Enforce a clickable non-interactive element has at least one keyboard event listener
@@ -117,10 +118,6 @@ export const reactA11yRules = {
     // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/interactive-supports-focus.html
     'jsx_a11y/interactive-supports-focus': 'error',
 
-    // WAI-ARIA roles should not be used to convert a non-interactive element to interactive
-    // https://oxc.rs/docs/guide/usage/linter/rules/jsx_a11y/no-noninteractive-element-to-interactive-role.html
-    'jsx_a11y/no-noninteractive-element-to-interactive-role': 'error',
-
     // Ensure interactive elements are not assigned non-interactive roles
     // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/no-noninteractive-tabindex.html
     'jsx_a11y/no-noninteractive-tabindex': [
@@ -153,7 +150,7 @@ export const reactA11yRules = {
 
     // Prefer semantic HTML elements over role attributes
     // https://oxc.rs/docs/guide/usage/linter/rules/jsx-a11y/prefer-tag-over-role.html
-    // decision: left to implementer
+    // Decision: left to implementer
     'jsx_a11y/prefer-tag-over-role': 'off',
   },
 };

package/rules/react.mjs

@@ -14,7 +14,7 @@ export const reactRules = {
 
     // Enforce curly braces or disallow unnecessary curly braces in JSX props and children
     // https://oxc.rs/docs/guide/usage/linter/rules/react/jsx-curly-brace-presence.html
-    // decision: stylistic choice best left for formatter
+    // Decision: stylistic choice best left for formatter
     'react/jsx-curly-brace-presence': 'off',
 
     // Enforce shorthand or standard form for React fragments
@@ -27,7 +27,7 @@ export const reactRules = {
 
     // Enforce a maximum depth that JSX can be nested
     // https://oxc.rs/docs/guide/usage/linter/rules/react/jsx-max-depth.html
-    // decision: not useful enough to justify the performance cost
+    // Decision: not useful enough to justify the performance cost
     'react/jsx-max-depth': 'off',
 
     // Disallow comments from being inserted as text nodes
@@ -40,8 +40,7 @@ export const reactRules = {
 
     // Disallow duplicate properties in JSX
     // https://oxc.rs/docs/guide/usage/linter/rules/react/jsx-no-duplicate-props.html
-    // decision: stylistic choice best left for formatter
-    'react/jsx-no-duplicate-props': 'off',
+    'react/jsx-no-duplicate-props': 'error',
 
     // Disallow javascript: URLs
     // https://oxc.rs/docs/guide/usage/linter/rules/react/jsx-no-script-url.html
@@ -53,28 +52,24 @@ export const reactRules = {
 
     // Disallow undeclared variables in JSX
     // https://oxc.rs/docs/guide/usage/linter/rules/react/jsx-no-undef.html
-    // decision: stylistic choice best left for formatter
+    // Decision: this rule is generally unnecessary if using TypeScript
     'react/jsx-no-undef': 'off',
 
     // Disallow unnecessary fragments
     // https://oxc.rs/docs/guide/usage/linter/rules/react/jsx-no-useless-fragment.html
-    // decision: disabled — oxlint's implementation differs from @eslint-react/no-useless-fragment
-    // and produces false positives on valid patterns
-    'react/jsx-no-useless-fragment': 'off',
+    'react/jsx-no-useless-fragment': ['error', { allowExpressions: true }],
 
     // Enforce PascalCase for user-defined JSX components
     // https://oxc.rs/docs/guide/usage/linter/rules/react/jsx-pascal-case.html
-    // decision: stylistic choice best left for formatter
-    'react/jsx-pascal-case': 'off',
+    'react/jsx-pascal-case': ['error', { allowNamespace: true }],
 
     // Disallow spreading props on multiple JSX elements (and merging objects)
     // https://oxc.rs/docs/guide/usage/linter/rules/react/jsx-props-no-spread-multi.html
-    // decision: stylistic choice best left for formatter
-    'react/jsx-props-no-spread-multi': 'off',
+    'react/jsx-props-no-spread-multi': 'error',
 
     // Disallow JSX prop spreading
     // https://oxc.rs/docs/guide/usage/linter/rules/react/jsx-props-no-spreading.html
-    // decision: this is a decision best left to the implementer
+    // Decision: this is a decision best left to the implementer
     'react/jsx-props-no-spreading': 'off',
 
     // Disallow Array.prototype.map() key from being an array index
@@ -103,12 +98,11 @@ export const reactRules = {
 
     // Disallow usage of isMounted
     // https://oxc.rs/docs/guide/usage/linter/rules/react/no-is-mounted.html
-    // decision: stylistic choice best left for formatter
-    'react/no-is-mounted': 'off',
+    'react/no-is-mounted': 'error',
 
     // Disallow defining multiple components in a single file (allow colocated stateless helpers)
     // https://oxc.rs/docs/guide/usage/linter/rules/react/no-multi-comp.html
-    // decision: stylistic choice best left for formatter
+    // Decision: this is a decision best left to the implementer
     'react/no-multi-comp': 'off',
 
     // Disallow usage of the return value of ReactDOM.render
@@ -117,23 +111,21 @@ export const reactRules = {
 
     // Disallow usage of setState
     // https://oxc.rs/docs/guide/usage/linter/rules/react/no-set-state.html
-    // decision: stylistic choice best left for formatter
+    // Decision: this is too broad, should be enabled at a project level if desired
     'react/no-set-state': 'off',
 
     // Disallow using string references
     // https://oxc.rs/docs/guide/usage/linter/rules/react/no-string-refs.html
-    // decision: stylistic choice best left for formatter
-    'react/no-string-refs': 'off',
+    'react/no-string-refs': 'error',
 
     // Disallow this from being used in stateless functional components
     // https://oxc.rs/docs/guide/usage/linter/rules/react/no-this-in-sfc.html
-    // decision: stylistic choice best left for formatter
+    // Decision: stylistic choice best left for formatter
     'react/no-this-in-sfc': 'off',
 
     // Disallow unescaped HTML entities from appearing in markup
     // https://oxc.rs/docs/guide/usage/linter/rules/react/no-unescaped-entities.html
-    // decision: stylistic choice best left for formatter
-    'react/no-unescaped-entities': 'off',
+    'react/no-unescaped-entities': 'error',
 
     // Disallow usage of unknown DOM property
     // https://oxc.rs/docs/guide/usage/linter/rules/react/no-unknown-property.html
@@ -141,8 +133,7 @@ export const reactRules = {
 
     // Disallow usage of unsafe lifecycle methods
     // https://oxc.rs/docs/guide/usage/linter/rules/react/no-unsafe.html
-    // decision: stylistic choice best left for formatter
-    'react/no-unsafe': 'off',
+    'react/no-unsafe': 'error',
 
     // Disallow usage of deprecated methods
     // https://oxc.rs/docs/guide/usage/linter/rules/react/no-redundant-should-component-update.html
@@ -162,7 +153,7 @@ export const reactRules = {
 
     // Disallow usage of React.Children APIs
     // https://oxc.rs/docs/guide/usage/linter/rules/react/no-react-children.html
-    // decision: stylistic choice best left for formatter
+    // Decision: stylistic choice best left for formatter
     'react/no-react-children': 'off',
 
     // Disallow usage of cloneElement
@@ -171,22 +162,22 @@ export const reactRules = {
 
     // Enforce ES5 or ES6 class for React Components
     // https://oxc.rs/docs/guide/usage/linter/rules/react/prefer-es6-class.html
-    // decision: stylistic choice best left for formatter
+    // Decision: stylistic choice best left for formatter
     'react/prefer-es6-class': 'off',
 
     // Enforce that components that only export from react can only export components
     // https://oxc.rs/docs/guide/usage/linter/rules/react/only-export-components.html
-    // decision: stylistic choice best left for formatter
+    // Decision: stylistic choice best left for formatter
     'react/only-export-components': 'off',
 
     // Prevent missing displayName in a React component definition
     // https://oxc.rs/docs/guide/usage/linter/rules/react/display-name.html
-    // decision: stylistic choice best left for formatter (equivalent @eslint-react/no-missing-component-display-name is "off")
+    // Decision: stylistic choice best left for formatter (equivalent @eslint-react/no-missing-component-display-name is "off")
     'react/display-name': 'off',
 
     // Enforce a specific function type for function components
     // https://oxc.rs/docs/guide/usage/linter/rules/react/forward-ref-uses-ref.html
-    // decision: stylistic choice best left for formatter
+    // Decision: stylistic choice best left for formatter
     'react/forward-ref-uses-ref': 'off',
 
     // Enforce sandbox attribute on iframe elements
@@ -195,7 +186,7 @@ export const reactRules = {
 
     // Enforce checked/onChange or readonly for checkboxes
     // https://oxc.rs/docs/guide/usage/linter/rules/react/checked-requires-onchange-or-readonly.html
-    // decision: stylistic choice best left for formatter
+    // Decision: stylistic choice best left for formatter
     'react/checked-requires-onchange-or-readonly': 'off',
 
     // Disallow void DOM elements from receiving children
@@ -204,7 +195,7 @@ export const reactRules = {
 
     // Enforce component methods order
     // https://oxc.rs/docs/guide/usage/linter/rules/react/require-render-return.html
-    // decision: stylistic choice best left for formatter
+    // Decision: stylistic choice best left for formatter
     'react/require-render-return': 'off',
 
     // Enforce Rules of Hooks
@@ -217,13 +208,12 @@ export const reactRules = {
 
     // Enforce JSX filename extension (allow .jsx and .tsx)
     // https://oxc.rs/docs/guide/usage/linter/rules/react/jsx-filename-extension.html
-    // decision: stylistic choice best left for formatter
+    // Decision: stylistic choice best left for formatter
     'react/jsx-filename-extension': 'off',
 
     // Enforce self-closing tags for components without children
     // https://oxc.rs/docs/guide/usage/linter/rules/react/self-closing-comp.html
-    // decision: stylistic choice best left for formatter
-    'react/self-closing-comp': 'off',
+    'react/self-closing-comp': 'error',
 
     // Enforce style prop value is an object
     // https://oxc.rs/docs/guide/usage/linter/rules/react/style-prop-object.html
@@ -231,39 +221,47 @@ export const reactRules = {
 
     // Enforce event handler naming conventions in JSX
     // https://oxc.rs/docs/guide/usage/linter/rules/react/jsx-handler-names.html
-    // decision: stylistic choice best left for formatter
+    // Decision: stylistic choice best left for formatter
     'react/jsx-handler-names': 'off',
 
     // Forbid certain props on components
     // https://oxc.rs/docs/guide/usage/linter/rules/react/forbid-component-props.html
+    // Decision: requires project-specific configuration
     'react/forbid-component-props': 'off',
 
     // Forbid certain props on DOM elements
     // https://oxc.rs/docs/guide/usage/linter/rules/react/forbid-dom-props.html
+    // Decision: requires project-specific configuration
     'react/forbid-dom-props': 'off',
 
     // Forbid certain elements
     // https://oxc.rs/docs/guide/usage/linter/rules/react/forbid-elements.html
+    // Decision: requires project-specific configuration
     'react/forbid-elements': 'off',
 
     // Enforce destructuring assignment of useState hook
     // https://oxc.rs/docs/guide/usage/linter/rules/react/hook-use-state.html
+    // Decision: too opinionated for general use
     'react/hook-use-state': 'off',
 
     // Disallow usage of setState in componentDidUpdate (deprecated lifecycle)
     // https://oxc.rs/docs/guide/usage/linter/rules/react/no-did-update-set-state.html
+    // Decision: deprecated lifecycle method
     'react/no-did-update-set-state': 'off',
 
     // Prefer function components over class components
     // https://oxc.rs/docs/guide/usage/linter/rules/react/prefer-function-component.html
+    // Decision: too opinionated for general use
     'react/prefer-function-component': 'off',
 
     // Disallow missing React when using JSX (not needed with modern React)
     // https://oxc.rs/docs/guide/usage/linter/rules/react/react-in-jsx-scope.html
+    // Decision: not needed with modern React JSX transform
     'react/react-in-jsx-scope': 'off',
 
     // Enforce state initialization in constructor
     // https://oxc.rs/docs/guide/usage/linter/rules/react/state-in-constructor.html
+    // Decision: too opinionated for general use
     'react/state-in-constructor': 'off',
 
     //rules via jsPlugins (@eslint-react/eslint-plugin)
@@ -278,7 +276,7 @@ export const reactRules = {
 
     // Verify the list of the dependencies for Hooks like useEffect and similar
     // https://eslint-react.xyz/docs/rules/exhaustive-deps
-    // redundant: covered by native react/exhaustive-deps
+    // Decision: handled by native react/exhaustive-deps
     '@eslint-react/exhaustive-deps': 'off',
 
     // Validates against mutating props, state, and other immutable values
@@ -287,7 +285,7 @@ export const reactRules = {
 
     // Prevent accidental JS comments from being injected into JSX as text
     // https://eslint-react.xyz/docs/rules/jsx-no-comment-textnodes
-    // redundant: covered by native react/jsx-no-comment-textnodes
+    // Decision: handled by native react/jsx-no-comment-textnodes
     '@eslint-react/jsx-no-comment-textnodes': 'off',
 
     // Prevents unintentional '$' sign before expression in JSX
@@ -300,7 +298,7 @@ export const reactRules = {
 
     // Prevent passing of children as props
     // https://eslint-react.xyz/docs/rules/jsx-no-children-prop
-    // redundant: covered by native react/no-children-prop
+    // Decision: handled by native react/no-children-prop
     '@eslint-react/jsx-no-children-prop': 'off',
 
     // Prevent children prop used alongside children in JSX
@@ -317,7 +315,7 @@ export const reactRules = {
 
     // Disallow namespace in JSX
     // https://eslint-react.xyz/docs/rules/jsx-no-namespace
-    // redundant: covered by native react/no-namespace
+    // Decision: handled by native react/no-namespace
     '@eslint-react/jsx-no-namespace': 'off',
 
     // Prevent using this.state within a this.setState
@@ -326,7 +324,7 @@ export const reactRules = {
 
     // Prevent usage of Array index in keys
     // https://eslint-react.xyz/docs/rules/no-array-index-key
-    // redundant: covered by native react/no-array-index-key
+    // Decision: handled by native react/no-array-index-key
     '@eslint-react/no-array-index-key': 'off',
 
     // Disallow usage of Children.count
@@ -355,7 +353,7 @@ export const reactRules = {
 
     // Disallow usage of cloneElement
     // https://eslint-react.xyz/docs/rules/no-clone-element
-    // redundant: covered by native react/no-clone-element
+    // Decision: handled by native react/no-clone-element
     '@eslint-react/no-clone-element': 'off',
 
     // Disallow usage of componentWillMount
@@ -380,7 +378,7 @@ export const reactRules = {
 
     // Prevent direct mutation of this.state
     // https://eslint-react.xyz/docs/rules/no-direct-mutation-state
-    // redundant: covered by native react/no-direct-mutation-state
+    // Decision: handled by native react/no-direct-mutation-state
     '@eslint-react/no-direct-mutation-state': 'off',
 
     // Disallow duplicate keys in JSX arrays
@@ -393,35 +391,37 @@ export const reactRules = {
 
     // Prevent implicitly passing the children prop
     // https://eslint-react.xyz/docs/rules/no-implicit-children
-    // disabled: requires TypeScript type-checker services unavailable in oxlint's JS plugin runtime
+    // Decision: requires TypeScript type-checker services unavailable in oxlint jsPlugin runtime
     '@eslint-react/no-implicit-children': 'off',
 
     // Prevent implicitly passing the key prop
     // https://eslint-react.xyz/docs/rules/no-implicit-key
-    // disabled: requires TypeScript type-checker services unavailable in oxlint's JS plugin runtime
+    // Decision: requires TypeScript type-checker services unavailable in oxlint jsPlugin runtime
     '@eslint-react/no-implicit-key': 'off',
 
     // Prevent implicitly passing the ref prop
     // https://eslint-react.xyz/docs/rules/no-implicit-ref
-    // disabled: requires TypeScript type-checker services unavailable in oxlint's JS plugin runtime
+    // Decision: requires TypeScript type-checker services unavailable in oxlint jsPlugin runtime
     '@eslint-react/no-implicit-ref': 'off',
 
     // Prevent problematic leaked values from being rendered
     // https://eslint-react.xyz/docs/rules/no-leaked-conditional-rendering
-    // disabled: requires TypeScript type-checker services unavailable in oxlint's JS plugin runtime
+    // Decision: requires TypeScript type-checker services unavailable in oxlint jsPlugin runtime
     '@eslint-react/no-leaked-conditional-rendering': 'off',
 
     // Enforce that components have a displayName for DevTools
     // https://eslint-react.xyz/docs/rules/no-missing-component-display-name
+    // Decision: too opinionated for general use
     '@eslint-react/no-missing-component-display-name': 'off',
 
     // Enforce that contexts have a displayName for DevTools
     // https://eslint-react.xyz/docs/rules/no-missing-context-display-name
+    // Decision: too opinionated for general use
     '@eslint-react/no-missing-context-display-name': 'off',
 
     // Enforce that every JSX element in a list has a key prop
     // https://eslint-react.xyz/docs/rules/no-missing-key
-    // redundant: covered by native react/jsx-key
+    // Decision: handled by native react/jsx-key
     '@eslint-react/no-missing-key': 'off',
 
     // Disallow misusing captureOwnerStack
@@ -438,7 +438,7 @@ export const reactRules = {
 
     // Prevent usage of setState in componentDidMount
     // https://eslint-react.xyz/docs/rules/no-set-state-in-component-did-mount
-    // redundant: covered by native react/no-did-mount-set-state
+    // Decision: handled by native react/no-did-mount-set-state
     '@eslint-react/no-set-state-in-component-did-mount': 'off',
 
     // Prevent usage of setState in componentDidUpdate
@@ -447,7 +447,7 @@ export const reactRules = {
 
     // Prevent usage of setState in componentWillUpdate
     // https://eslint-react.xyz/docs/rules/no-set-state-in-component-will-update
-    // redundant: covered by native react/no-will-update-set-state
+    // Decision: handled by native react/no-will-update-set-state
     '@eslint-react/no-set-state-in-component-will-update': 'off',
 
     // Disallow unnecessary "use" prefix on custom hooks
@@ -468,7 +468,7 @@ export const reactRules = {
 
     // Prevent non-stable values used as context values
     // https://eslint-react.xyz/docs/rules/no-unstable-context-value
-    // redundant: covered by native react/jsx-no-constructed-context-values
+    // Decision: handled by native react/jsx-no-constructed-context-values
     '@eslint-react/no-unstable-context-value': 'off',
 
     // Disallow referential-type variables as default props
@@ -481,7 +481,7 @@ export const reactRules = {
 
     // Warn about component props that are defined but never used
     // https://eslint-react.xyz/docs/rules/no-unused-props
-    // disabled: requires TypeScript type-checker services unavailable in oxlint's JS plugin runtime
+    // Decision: requires TypeScript type-checker services unavailable in oxlint jsPlugin runtime
     '@eslint-react/no-unused-props': 'off',
 
     // Prevent unused state values
@@ -502,7 +502,7 @@ export const reactRules = {
 
     // Enforce Rules of Hooks
     // https://eslint-react.xyz/docs/rules/rules-of-hooks
-    // redundant: covered by native react/rules-of-hooks
+    // Decision: handled by native react/rules-of-hooks
     '@eslint-react/rules-of-hooks': 'off',
 
     // Validates against calling setState synchronously in an effect
@@ -533,17 +533,17 @@ export const reactRules = {
 
     // Warn on usage of dangerouslySetInnerHTML
     // https://eslint-react.xyz/docs/rules/dom-no-dangerously-set-innerhtml
-    // redundant: covered by native react/no-danger
+    // Decision: handled by native react/no-danger
     '@eslint-react/dom-no-dangerously-set-innerhtml': 'off',
 
     // Prevent problem with children and dangerouslySetInnerHTML
     // https://eslint-react.xyz/docs/rules/dom-no-dangerously-set-innerhtml-with-children
-    // redundant: covered by native react/no-danger-with-children
+    // Decision: handled by native react/no-danger-with-children
     '@eslint-react/dom-no-dangerously-set-innerhtml-with-children': 'off',
 
     // Warn against using findDOMNode()
     // https://eslint-react.xyz/docs/rules/dom-no-find-dom-node
-    // redundant: covered by native react/no-find-dom-node
+    // Decision: handled by native react/no-find-dom-node
     '@eslint-react/dom-no-find-dom-node': 'off',
 
     // Disallow usage of flushSync
@@ -556,12 +556,12 @@ export const reactRules = {
 
     // Enforce that buttons have an explicit type attribute
     // https://eslint-react.xyz/docs/rules/dom-no-missing-button-type
-    // redundant: covered by native react/button-has-type
+    // Decision: handled by native react/button-has-type
     '@eslint-react/dom-no-missing-button-type': 'off',
 
     // Enforce sandbox attribute on iframe elements
     // https://eslint-react.xyz/docs/rules/dom-no-missing-iframe-sandbox
-    // redundant: covered by native react/iframe-missing-sandbox
+    // Decision: handled by native react/iframe-missing-sandbox
     '@eslint-react/dom-no-missing-iframe-sandbox': 'off',
 
     // Disallow usage of ReactDOM.render (use createRoot instead)
@@ -570,22 +570,22 @@ export const reactRules = {
 
     // Disallow using ReactDOM.render return value
     // https://eslint-react.xyz/docs/rules/dom-no-render-return-value
-    // redundant: covered by native react/no-render-return-value
+    // Decision: handled by native react/no-render-return-value
     '@eslint-react/dom-no-render-return-value': 'off',
 
     // Prevent usage of javascript: URLs
     // https://eslint-react.xyz/docs/rules/dom-no-script-url
-    // redundant: covered by native react/jsx-no-script-url
+    // Decision: handled by native react/jsx-no-script-url
     '@eslint-react/dom-no-script-url': 'off',
 
     // Require style prop value be an object
     // https://eslint-react.xyz/docs/rules/dom-no-string-style-prop
-    // redundant: covered by native react/style-prop-object
+    // Decision: handled by native react/style-prop-object
     '@eslint-react/dom-no-string-style-prop': 'off',
 
     // Prevent usage of unknown DOM property
     // https://eslint-react.xyz/docs/rules/dom-no-unknown-property
-    // redundant: covered by native react/no-unknown-property
+    // Decision: handled by native react/no-unknown-property
     '@eslint-react/dom-no-unknown-property': 'off',
 
     // Enforce safe iframe sandbox attribute values
@@ -594,7 +594,7 @@ export const reactRules = {
 
     // Disallow target="_blank" without rel="noreferrer"
     // https://eslint-react.xyz/docs/rules/dom-no-unsafe-target-blank
-    // redundant: covered by native react/jsx-no-target-blank
+    // Decision: handled by native react/jsx-no-target-blank
     '@eslint-react/dom-no-unsafe-target-blank': 'off',
 
     // Disallow usage of deprecated useFormState (use useActionState instead)
@@ -603,7 +603,7 @@ export const reactRules = {
 
     // Prevent void DOM elements from receiving children
     // https://eslint-react.xyz/docs/rules/dom-no-void-elements-with-children
-    // redundant: covered by native react/void-dom-elements-no-children
+    // Decision: handled by native react/void-dom-elements-no-children
     '@eslint-react/dom-no-void-elements-with-children': 'off',
 
     // --- RSC rules ---