@wistia/oxlint-config 0.4.4 → 0.4.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wistia/oxlint-config",
-  "version": "0.4.4",
+  "version": "0.4.6",
   "description": "Wistia's Oxlint configurations",
   "packageManager": "yarn@4.14.1",
   "type": "module",

package/rules/react.mjs

@@ -246,7 +246,8 @@ export const reactRules = {
 
     // Verify the list of the dependencies for Hooks like useEffect and similar
     // https://eslint-react.xyz/docs/rules/exhaustive-deps
-    '@eslint-react/exhaustive-deps': 'error',
+    // redundant: covered by native react/exhaustive-deps
+    '@eslint-react/exhaustive-deps': 'off',
 
     // Validates against mutating props, state, and other immutable values
     // https://eslint-react.xyz/docs/rules/immutability
@@ -254,7 +255,8 @@ export const reactRules = {
 
     // Prevent accidental JS comments from being injected into JSX as text
     // https://eslint-react.xyz/docs/rules/jsx-no-comment-textnodes
-    '@eslint-react/jsx-no-comment-textnodes': 'error',
+    // redundant: covered by native react/jsx-no-comment-textnodes
+    '@eslint-react/jsx-no-comment-textnodes': 'off',
 
     // Prevents unintentional '$' sign before expression in JSX
     // https://eslint-react.xyz/docs/rules/jsx-no-leaked-dollar
@@ -266,7 +268,8 @@ export const reactRules = {
 
     // Prevent passing of children as props
     // https://eslint-react.xyz/docs/rules/jsx-no-children-prop
-    '@eslint-react/jsx-no-children-prop': 'error',
+    // redundant: covered by native react/no-children-prop
+    '@eslint-react/jsx-no-children-prop': 'off',
 
     // Prevent children prop used alongside children in JSX
     // https://eslint-react.xyz/docs/rules/jsx-no-children-prop-with-children
@@ -282,7 +285,8 @@ export const reactRules = {
 
     // Disallow namespace in JSX
     // https://eslint-react.xyz/docs/rules/jsx-no-namespace
-    '@eslint-react/jsx-no-namespace': 'error',
+    // redundant: covered by native react/no-namespace
+    '@eslint-react/jsx-no-namespace': 'off',
 
     // Prevent using this.state within a this.setState
     // https://eslint-react.xyz/docs/rules/no-access-state-in-setstate
@@ -290,7 +294,8 @@ export const reactRules = {
 
     // Prevent usage of Array index in keys
     // https://eslint-react.xyz/docs/rules/no-array-index-key
-    '@eslint-react/no-array-index-key': 'error',
+    // redundant: covered by native react/no-array-index-key
+    '@eslint-react/no-array-index-key': 'off',
 
     // Disallow usage of Children.count
     // https://eslint-react.xyz/docs/rules/no-children-count
@@ -318,7 +323,8 @@ export const reactRules = {
 
     // Disallow usage of cloneElement
     // https://eslint-react.xyz/docs/rules/no-clone-element
-    '@eslint-react/no-clone-element': 'error',
+    // redundant: covered by native react/no-clone-element
+    '@eslint-react/no-clone-element': 'off',
 
     // Disallow usage of componentWillMount
     // https://eslint-react.xyz/docs/rules/no-component-will-mount
@@ -342,7 +348,8 @@ export const reactRules = {
 
     // Prevent direct mutation of this.state
     // https://eslint-react.xyz/docs/rules/no-direct-mutation-state
-    '@eslint-react/no-direct-mutation-state': 'error',
+    // redundant: covered by native react/no-direct-mutation-state
+    '@eslint-react/no-direct-mutation-state': 'off',
 
     // Disallow duplicate keys in JSX arrays
     // https://eslint-react.xyz/docs/rules/no-duplicate-key
@@ -382,7 +389,8 @@ export const reactRules = {
 
     // Enforce that every JSX element in a list has a key prop
     // https://eslint-react.xyz/docs/rules/no-missing-key
-    '@eslint-react/no-missing-key': 'error',
+    // redundant: covered by native react/jsx-key
+    '@eslint-react/no-missing-key': 'off',
 
     // Disallow misusing captureOwnerStack
     // https://eslint-react.xyz/docs/rules/no-misused-capture-owner-stack
@@ -398,11 +406,13 @@ export const reactRules = {
 
     // Prevent usage of shouldComponentUpdate when extending React.PureComponent
     // https://eslint-react.xyz/docs/rules/no-redundant-should-component-update
-    '@eslint-react/no-redundant-should-component-update': 'error',
+    // redundant: covered by native react/no-redundant-should-component-update
+    '@eslint-react/no-redundant-should-component-update': 'off',
 
     // Prevent usage of setState in componentDidMount
     // https://eslint-react.xyz/docs/rules/no-set-state-in-component-did-mount
-    '@eslint-react/no-set-state-in-component-did-mount': 'error',
+    // redundant: covered by native react/no-did-mount-set-state
+    '@eslint-react/no-set-state-in-component-did-mount': 'off',
 
     // Prevent usage of setState in componentDidUpdate
     // https://eslint-react.xyz/docs/rules/no-set-state-in-component-did-update
@@ -410,7 +420,8 @@ export const reactRules = {
 
     // Prevent usage of setState in componentWillUpdate
     // https://eslint-react.xyz/docs/rules/no-set-state-in-component-will-update
-    '@eslint-react/no-set-state-in-component-will-update': 'error',
+    // redundant: covered by native react/no-will-update-set-state
+    '@eslint-react/no-set-state-in-component-will-update': 'off',
 
     // Disallow unnecessary useCallback hooks
     // https://eslint-react.xyz/docs/rules/no-unnecessary-use-callback
@@ -438,7 +449,8 @@ export const reactRules = {
 
     // Prevent non-stable values used as context values
     // https://eslint-react.xyz/docs/rules/no-unstable-context-value
-    '@eslint-react/no-unstable-context-value': 'error',
+    // redundant: covered 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
@@ -480,7 +492,8 @@ export const reactRules = {
 
     // Enforce Rules of Hooks
     // https://eslint-react.xyz/docs/rules/rules-of-hooks
-    '@eslint-react/rules-of-hooks': 'error',
+    // redundant: covered by native react/rules-of-hooks
+    '@eslint-react/rules-of-hooks': 'off',
 
     // Validates against calling setState synchronously in an effect
     // https://eslint-react.xyz/docs/rules/set-state-in-effect
@@ -506,15 +519,18 @@ export const reactRules = {
 
     // Warn on usage of dangerouslySetInnerHTML
     // https://eslint-react.xyz/docs/rules/dom-no-dangerously-set-innerhtml
-    '@eslint-react/dom-no-dangerously-set-innerhtml': 'error',
+    // redundant: covered 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
-    '@eslint-react/dom-no-dangerously-set-innerhtml-with-children': 'error',
+    // redundant: covered 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
-    '@eslint-react/dom-no-find-dom-node': 'error',
+    // redundant: covered by native react/no-find-dom-node
+    '@eslint-react/dom-no-find-dom-node': 'off',
 
     // Disallow usage of flushSync
     // https://eslint-react.xyz/docs/rules/dom-no-flush-sync
@@ -526,11 +542,13 @@ export const reactRules = {
 
     // Enforce that buttons have an explicit type attribute
     // https://eslint-react.xyz/docs/rules/dom-no-missing-button-type
-    '@eslint-react/dom-no-missing-button-type': 'error',
+    // redundant: covered 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
-    '@eslint-react/dom-no-missing-iframe-sandbox': 'error',
+    // redundant: covered by native react/iframe-missing-sandbox
+    '@eslint-react/dom-no-missing-iframe-sandbox': 'off',
 
     // Disallow usage of ReactDOM.render (use createRoot instead)
     // https://eslint-react.xyz/docs/rules/dom-no-render
@@ -538,19 +556,23 @@ export const reactRules = {
 
     // Disallow using ReactDOM.render return value
     // https://eslint-react.xyz/docs/rules/dom-no-render-return-value
-    '@eslint-react/dom-no-render-return-value': 'error',
+    // redundant: covered 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
-    '@eslint-react/dom-no-script-url': 'error',
+    // redundant: covered 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
-    '@eslint-react/dom-no-string-style-prop': 'error',
+    // redundant: covered 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
-    '@eslint-react/dom-no-unknown-property': 'error',
+    // redundant: covered by native react/no-unknown-property
+    '@eslint-react/dom-no-unknown-property': 'off',
 
     // Enforce safe iframe sandbox attribute values
     // https://eslint-react.xyz/docs/rules/dom-no-unsafe-iframe-sandbox
@@ -558,7 +580,8 @@ export const reactRules = {
 
     // Disallow target="_blank" without rel="noreferrer"
     // https://eslint-react.xyz/docs/rules/dom-no-unsafe-target-blank
-    '@eslint-react/dom-no-unsafe-target-blank': 'error',
+    // redundant: covered by native react/jsx-no-target-blank
+    '@eslint-react/dom-no-unsafe-target-blank': 'off',
 
     // Disallow usage of deprecated useFormState (use useActionState instead)
     // https://eslint-react.xyz/docs/rules/dom-no-use-form-state
@@ -566,7 +589,8 @@ export const reactRules = {
 
     // Prevent void DOM elements from receiving children
     // https://eslint-react.xyz/docs/rules/dom-no-void-elements-with-children
-    '@eslint-react/dom-no-void-elements-with-children': 'error',
+    // redundant: covered by native react/void-dom-elements-no-children
+    '@eslint-react/dom-no-void-elements-with-children': 'off',
 
     // Enforce importing React DOM via a namespace import
     // https://eslint-react.xyz/docs/rules/dom-prefer-namespace-import

package/rules/typescript.mjs

@@ -2,25 +2,6 @@ export const typescriptRules = {
   plugins: ['typescript'],
   rules: {
     //Disable base eslint rules superseded by typescript equivalents --
-    'eslint/class-methods-use-this': 'off',
-    'eslint/default-param-last': 'off',
-    'eslint/init-declarations': 'off',
-    'eslint/max-params': 'off',
-    'eslint/no-array-constructor': 'off',
-    'eslint/no-dupe-class-members': 'off',
-    'eslint/no-empty-function': 'off',
-    'eslint/no-loop-func': 'off',
-    'eslint/no-magic-numbers': 'off',
-    'eslint/no-redeclare': 'off',
-    'eslint/no-restricted-imports': 'off',
-    'eslint/no-shadow': 'off',
-    'eslint/no-throw-literal': 'off',
-    'eslint/no-unused-expressions': 'off',
-    'eslint/no-unused-private-class-members': 'off',
-    'eslint/no-unused-vars': 'off',
-    'eslint/no-use-before-define': 'off',
-    'eslint/no-useless-constructor': 'off',
-    'eslint/prefer-destructuring': 'off',
     'eslint/require-await': 'off',
 
     // recommended to be disabled for TypeScript projects