@wistia/oxlint-config 0.7.4 → 0.8.0

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,37 +98,42 @@ 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 object types as default props
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/no-object-type-as-default-prop.html
+    'react/no-object-type-as-default-prop': 'error',
+
+    // Disallow creating unstable components inside components
+    // https://oxc.rs/docs/guide/usage/linter/rules/react/no-unstable-nested-components.html
+    'react/no-unstable-nested-components': 'error',
+
     // Disallow usage of the return value of ReactDOM.render
     // https://oxc.rs/docs/guide/usage/linter/rules/react/no-render-return-value.html
     'react/no-render-return-value': 'error',
 
     // 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 +141,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 +161,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 +170,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 +194,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 +203,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 +216,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 +229,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 +284,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 +293,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 +306,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 +323,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 +332,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 +361,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 +386,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 +399,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
@@ -430,7 +438,8 @@ export const reactRules = {
 
     // Prevent creating unstable components inside components
     // https://eslint-react.xyz/docs/rules/no-nested-component-definitions
-    '@eslint-react/no-nested-component-definitions': 'error',
+    // Decision: handled by native react/no-unstable-nested-components
+    '@eslint-react/no-nested-component-definitions': 'off',
 
     // Prevent creating lazy components inside components
     // https://eslint-react.xyz/docs/rules/no-nested-lazy-component-declarations
@@ -438,7 +447,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 +456,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,12 +477,13 @@ 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
     // https://eslint-react.xyz/docs/rules/no-unstable-default-props
-    '@eslint-react/no-unstable-default-props': 'error',
+    // Decision: handled by native react/no-object-type-as-default-prop
+    '@eslint-react/no-unstable-default-props': 'off',
 
     // Prevent declaring unused methods of component class
     // https://eslint-react.xyz/docs/rules/no-unused-class-component-members
@@ -481,7 +491,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 +512,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 +543,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 +566,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 +580,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 +604,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 +613,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 ---

package/rules/typescript.mjs

@@ -16,16 +16,19 @@ export const typescriptRules = {
     ],
 
     // This is redundant with TypeScript's noUnusedLocals/noUnusedParameters compiler options
+    // Decision: handled by TypeScript noUnusedLocals/noUnusedParameters
     '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
+    // Decision: handled by TypeScript
     '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
+    // Decision: handled by typescript/require-await
     'eslint/require-await': 'off',
 
     // Require that function overload signatures be consecutive
@@ -70,7 +73,7 @@ export const typescriptRules = {
 
     // Enforce type definitions to consistently use either interface or type
     // https://oxc.rs/docs/guide/usage/linter/rules/typescript/consistent-type-definitions.html
-    // decision: prefer type to interface for consistency
+    // Decision: prefer type to interface for consistency
     'typescript/consistent-type-definitions': ['error', 'type'],
 
     // Enforce consistent usage of type exports
@@ -314,7 +317,7 @@ export const typescriptRules = {
 
     // Enforce non-null assertions over explicit type casts
     // https://oxc.rs/docs/guide/usage/linter/rules/typescript/non-nullable-type-assertion-style.html
-    // decision: disabled because it conflicts with `no-non-null-assertion` — this rule's autofix
+    // Decision: disabled because it conflicts with `no-non-null-assertion` — this rule's autofix
     // rewrites `value as T` to `value!`, which `no-non-null-assertion` then rejects
     'typescript/non-nullable-type-assertion-style': 'off',
 
@@ -444,34 +447,42 @@ export const typescriptRules = {
 
     // Disallow certain types (deprecated rule)
     // https://oxc.rs/docs/guide/usage/linter/rules/typescript/ban-types.html
+    // Decision: deprecated rule
     'typescript/ban-types': 'off',
 
     // Require explicit return types on functions and class methods
     // https://oxc.rs/docs/guide/usage/linter/rules/typescript/explicit-function-return-type.html
+    // Decision: too opinionated for general use
     'typescript/explicit-function-return-type': 'off',
 
     // Require explicit accessibility modifiers on class properties and methods
     // https://oxc.rs/docs/guide/usage/linter/rules/typescript/explicit-member-accessibility.html
+    // Decision: too opinionated for general use
     'typescript/explicit-member-accessibility': 'off',
 
     // Disallow empty interfaces (deprecated rule)
     // https://oxc.rs/docs/guide/usage/linter/rules/typescript/no-empty-interface.html
+    // Decision: deprecated rule
     'typescript/no-empty-interface': 'off',
 
     // Disallow require statements except in import statements (deprecated rule)
     // https://oxc.rs/docs/guide/usage/linter/rules/typescript/no-var-requires.html
+    // Decision: deprecated rule
     'typescript/no-var-requires': 'off',
 
     // Require function parameters to be typed as readonly
     // https://oxc.rs/docs/guide/usage/linter/rules/typescript/prefer-readonly-parameter-types.html
+    // Decision: too opinionated for general use
     'typescript/prefer-readonly-parameter-types': 'off',
 
     // Enforce using @ts-expect-error over @ts-ignore (deprecated rule)
     // https://oxc.rs/docs/guide/usage/linter/rules/typescript/prefer-ts-expect-error.html
+    // Decision: deprecated rule
     'typescript/prefer-ts-expect-error': 'off',
 
     // Enforce void return type for functions that don't return a value
     // https://oxc.rs/docs/guide/usage/linter/rules/typescript/strict-void-return.html
+    // Decision: too opinionated for general use
     'typescript/strict-void-return': 'off',
   },
 };