@plumile/eslint-config-typescript 0.1.200 → 0.1.205

package/README.md

@@ -15,8 +15,8 @@ The package provides:
 - a `createConfig` helper for custom `tsconfig` locations
 - a Relay-specific preset in `relay.js`
 
-It is intended for projects that want Kronex's TypeScript, React, accessibility
-and code-quality defaults without copying rule definitions manually.
+It is intended for projects that want Kronex's TypeScript, React hooks and
+code-quality defaults without copying rule definitions manually.
 
 ## Installation
 
@@ -31,9 +31,7 @@ npm install --save-dev \
   @stylistic/eslint-plugin \
   @typescript-eslint/eslint-plugin \
   eslint-plugin-jsdoc \
-  eslint-plugin-jsx-a11y \
   eslint-plugin-n \
-  eslint-plugin-react \
   eslint-plugin-react-hooks \
   eslint-plugin-sonarjs
 ```
@@ -100,7 +98,6 @@ export default [...plumileTs, ...relayPreset];
 
 - TypeScript parsing and project-aware linting
 - React hooks rules
-- accessibility checks
 - jsdoc and general code-quality rules
 - stylistic rules aligned with the repository expectations
 

package/index.js

@@ -80,14 +80,11 @@ export function createConfig(options = {}) {
       },
       /* Rules applicable to all files (https://eslint.org/docs/rules) */
       rules: {
-        /* Enforce best practices rules */
-        'accessor-pairs': 'off', // enforces getter/setter pairs in objects
+        // enforces return statements in callbacks of array's methods
+        'array-callback-return': ['error', { allowImplicit: true }],
 
-        'array-callback-return': ['error', { allowImplicit: true }], // enforces return statements in callbacks of array's methods
-
-        'block-scoped-var': 'error', // treat var statements as if they were block scoped
-
-        complexity: ['off', 20], // specify the maximum cyclomatic complexity allowed in a program
+        // treat var statements as if they were block scoped
+        'block-scoped-var': 'error',
 
         // enforce that class methods use "this"
         'class-methods-use-this': [
@@ -101,58 +98,64 @@ export function createConfig(options = {}) {
         'consistent-return': 'error',
 
         // specify curly brace conventions for all control statements
-        curly: ['error', 'multi-line'], // multiline
+        // multiline
+        curly: ['error', 'multi-line'],
 
         // require default case in switch statements
         'default-case': ['error', { commentPattern: '^no default$' }],
 
-        'default-case-last': 'error', // Enforce default clauses in switch statements to be last
+        // Enforce default clauses in switch statements to be last
+        'default-case-last': 'error',
 
         'default-param-last': 'error',
 
-        'dot-notation': ['error', { allowKeywords: true }], // encourages use of dot notation whenever possible
+        // encourages use of dot notation whenever possible
+        'dot-notation': ['error', { allowKeywords: true }],
 
-        'dot-location': ['error', 'property'], // enforces consistent newlines before or after dots
+        // enforces consistent newlines before or after dots
+        'dot-location': ['error', 'property'],
 
-        eqeqeq: ['error', 'always', { null: 'ignore' }], // require the use of === and !==
+        // require the use of === and !==
+        eqeqeq: ['error', 'always', { null: 'ignore' }],
 
         // Require grouped accessor pairs in object literals and classes
         'grouped-accessor-pairs': 'error',
 
-        // make sure for-in loops have an if statement
-        'guard-for-in': 0,
-
-        'max-classes-per-file': ['error', 1], // enforce a maximum number of classes per file
+        // enforce a maximum number of classes per file
+        'max-classes-per-file': ['error', 1],
 
-        'no-alert': 2, // disallow the use of alert, confirm, and prompt
+        // disallow the use of alert, confirm, and prompt
+        'no-alert': 2,
 
-        'no-caller': 'error', // disallow use of arguments.caller or arguments.callee
+        // disallow use of arguments.caller or arguments.callee
+        'no-caller': 'error',
 
-        'no-case-declarations': 'error', // disallow lexical declarations in case/default clauses
+        // disallow lexical declarations in case/default clauses
+        'no-case-declarations': 'error',
 
-        'no-constructor-return': 'error', // Disallow returning value in constructor
-
-        // disallow division operators explicitly at beginning of regular expression
-        'no-div-regex': 'off',
+        // Disallow returning value in constructor
+        'no-constructor-return': 'error',
 
         // disallow else after a return in an if
         'no-else-return': ['error', { allowElseIf: false }],
 
-        'no-empty-function': 0, // disallow empty functions,- handle by typescript-eslint
-
-        'no-empty-pattern': 'error', // disallow empty destructuring patterns
-
-        'no-eq-null': 'off', // disallow comparisons to null without a type-checking operator
+        // disallow empty destructuring patterns
+        'no-empty-pattern': 'error',
 
-        'no-eval': 'error', // disallow use of eval()
+        // disallow use of eval()
+        'no-eval': 'error',
 
-        'no-extend-native': 'error', // disallow adding to native types
+        // disallow adding to native types
+        'no-extend-native': 'error',
 
-        'no-extra-bind': 'error', // disallow unnecessary function binding
+        // disallow unnecessary function binding
+        'no-extra-bind': 'error',
 
-        'no-extra-label': 'error', // disallow Unnecessary Labels
+        // disallow Unnecessary Labels
+        'no-extra-label': 'error',
 
-        'no-fallthrough': 'error', // disallow fallthrough of case statements
+        // disallow fallthrough of case statements
+        'no-fallthrough': 'error',
 
         // disallow the use of leading or trailing decimal points in numeric literals
         'no-floating-decimal': 'error',
@@ -160,32 +163,17 @@ export function createConfig(options = {}) {
         // disallow reassignments of native objects or read-only globals
         'no-global-assign': ['error', { exceptions: [] }],
 
-        // disallow implicit type conversions
-        'no-implicit-coercion': [
-          'off',
-          {
-            boolean: false,
-            number: true,
-            string: true,
-            allow: [],
-          },
-        ],
-
-        'no-implicit-globals': 'off', // disallow var and named functions in global scope
-
-        'no-implied-eval': 0, // disallow use of eval()-like methods
-
-        // disallow this keywords outside of classes or class-like objects - checked by TS compiler
-        'no-invalid-this': 'off',
-
-        'no-iterator': 'error', // disallow usage of __iterator__ property
+        // disallow usage of __iterator__ property
+        'no-iterator': 'error',
 
         // disallow use of labels for anything other than loops and switches
         'no-labels': ['error', { allowLoop: false, allowSwitch: false }],
 
-        'no-lone-blocks': 'error', // disallow unnecessary nested blocks
+        // disallow unnecessary nested blocks
+        'no-lone-blocks': 'error',
 
-        'no-loop-func': 'error', // disallow creation of functions within loops
+        // disallow creation of functions within loops
+        'no-loop-func': 'error',
 
         // disallow use of multiple spaces
         'no-multi-spaces': [
@@ -195,11 +183,14 @@ export function createConfig(options = {}) {
           },
         ],
 
-        'no-multi-str': 'error', // disallow use of multiline strings
+        // disallow use of multiline strings
+        'no-multi-str': 'error',
 
-        'no-new': 'error', // disallow use of new operator when not part of the assignment or comparison
+        // disallow use of new operator when not part of the assignment or comparison
+        'no-new': 'error',
 
-        'no-new-func': 'error', // disallow use of new operator for Function object
+        // disallow use of new operator for Function object
+        'no-new-func': 'error',
 
         // disallows creating new instances of String, Number, and Boolean
         'no-new-wrappers': 'error',
@@ -207,7 +198,8 @@ export function createConfig(options = {}) {
         // Disallow \8 and \9 escape sequences in string literals
         'no-nonoctal-decimal-escape': 'error',
 
-        'no-octal': 'error', // disallow use of (old style) octal literals
+        // disallow use of (old style) octal literals
+        'no-octal': 'error',
 
         // disallow use of octal escape sequences in string literals, such as
         // var foo = 'Copyright \251';
@@ -235,9 +227,11 @@ export function createConfig(options = {}) {
           },
         ],
 
-        'no-proto': 'error', // disallow usage of __proto__ property
+        // disallow usage of __proto__ property
+        'no-proto': 'error',
 
-        'no-redeclare': 'error', // disallow declaring the same variable more than once
+        // disallow declaring the same variable more than once
+        'no-redeclare': 'error',
 
         // disallow certain object properties
         'no-restricted-properties': [
@@ -295,9 +289,11 @@ export function createConfig(options = {}) {
         // disallow use of assignment in return statement
         'no-return-assign': ['error', 'always'],
 
-        'no-return-await': 'error', // disallow redundant `return await`
+        // disallow redundant `return await`
+        'no-return-await': 'error',
 
-        'no-script-url': 'error', // disallow use of `javascript:` urls.
+        // disallow use of `javascript:` urls.
+        'no-script-url': 'error',
 
         // disallow self assignment
         'no-self-assign': [
@@ -307,51 +303,36 @@ export function createConfig(options = {}) {
           },
         ],
 
-        'no-self-compare': 'error', // disallow comparisons where both sides are exactly the same
-
-        'no-sequences': 'error', // disallow use of comma operator
-
-        // restrict what can be thrown as an exception - handle by typescript-eslint
-        'no-throw-literal': 'off', // handle by only-throw-error in typescript-eslint
-
-        // disallow unmodified conditions of loops
-        'no-unmodified-loop-condition': 'off',
-
-        // disallow usage of expressions in statement position - handle by typescript-eslint
-        'no-unused-expressions': 0,
-
-        'no-unused-labels': 'error', // disallow unused labels
+        // disallow comparisons where both sides are exactly the same
+        'no-self-compare': 'error',
 
-        'no-useless-call': 'off', // disallow unnecessary .call() and .apply()
+        // disallow use of comma operator
+        'no-sequences': 'error',
 
-        'no-useless-catch': 'error', // Disallow unnecessary catch clauses
+        // disallow unused labels
+        'no-unused-labels': 'error',
 
-        'no-useless-concat': 'error', // disallow useless string concatenation
+        // Disallow unnecessary catch clauses
+        'no-useless-catch': 'error',
 
-        'no-useless-escape': 'error', // disallow unnecessary string escaping
+        // disallow useless string concatenation
+        'no-useless-concat': 'error',
 
-        'no-useless-return': 'error', // disallow redundant return; keywords
+        // disallow unnecessary string escaping
+        'no-useless-escape': 'error',
 
-        'no-void': 'error', // disallow use of void operator
+        // disallow redundant return; keywords
+        'no-useless-return': 'error',
 
-        // disallow usage of configurable warning terms in comments: e.g. todo
-        'no-warning-comments': [
-          'off',
-          { terms: ['todo', 'fixme', 'xxx'], location: 'start' },
-        ],
+        // disallow use of void operator
+        'no-void': 'error',
 
-        'no-with': 'error', // disallow use of the with statement
+        // disallow use of the with statement
+        'no-with': 'error',
 
         // require using Error objects as Promise rejection reasons
         'prefer-promise-reject-errors': ['error', { allowEmptyReject: true }],
 
-        // Suggest using named capture group in regular expression
-        'prefer-named-capture-group': 'off',
-
-        // Prefer Object.hasOwn() over Object.prototype.hasOwnProperty.call()
-        // TODO: semver-major: enable thus rule, once eslint v8.5.0 is required
-        'prefer-object-has-own': 'off',
-
         // Disallow use of the RegExp constructor in favor of regular expression literals
         'prefer-regex-literals': [
           'error',
@@ -360,54 +341,64 @@ export function createConfig(options = {}) {
           },
         ],
 
-        radix: 'error', // require use of the second argument for parseInt()
+        // require use of the second argument for parseInt()
+        radix: 'error',
 
-        // require `await` in `async function` (note: this is a horrible rule that should never be used)
-        'require-await': 'off',
-
-        // Enforce the use of u flag on RegExp
-        'require-unicode-regexp': 'off',
-
-        'vars-on-top': 'error', // requires to declare all vars on top of their containing scope
+        // requires to declare all vars on top of their containing scope
+        'vars-on-top': 'error',
 
         // require immediate function invocation to be wrapped in parentheses
         'wrap-iife': ['error', 'outside', { functionPrototypeMethods: false }],
 
-        yoda: 'error', // require or disallow Yoda conditions
+        // require or disallow Yoda conditions
+        yoda: 'error',
 
         /* Enforce Errors rules */
 
-        'for-direction': 'error', // Enforce “for” loop update clause moving the counter in the right direction
-
-        'getter-return': ['error', { allowImplicit: true }], // Enforces that a return statement is present in property getters
+        // Enforce “for” loop update clause moving the counter in the right direction
+        'for-direction': 'error',
 
-        'no-async-promise-executor': 'error', // disallow using an async function as a Promise executor
+        // Enforces that a return statement is present in property getters
+        'getter-return': ['error', { allowImplicit: true }],
 
-        'no-await-in-loop': 0, // Disallow await inside of loops
+        // disallow using an async function as a Promise executor
+        'no-async-promise-executor': 'error',
 
-        'no-compare-neg-zero': 'error', // Disallow comparisons to negative zero
+        // Disallow comparisons to negative zero
+        'no-compare-neg-zero': 'error',
 
-        'no-cond-assign': ['error', 'always'], // disallow assignment in conditional expressions
+        // disallow assignment in conditional expressions
+        'no-cond-assign': ['error', 'always'],
 
-        'no-console': 'warn', // disallow use of console
+        // disallow use of console
+        'no-console': 'warn',
 
-        'no-constant-binary-expression': 2, // Disallows expressions where the operation doesn't affect the value
+        // Disallows expressions where the operation doesn't affect the value
+        'no-constant-binary-expression': 2,
 
-        'no-constant-condition': 'warn', // disallow use of constant expressions in conditions
+        // disallow use of constant expressions in conditions
+        'no-constant-condition': 'warn',
 
-        'no-control-regex': 'error', // disallow control characters in regular expressions
+        // disallow control characters in regular expressions
+        'no-control-regex': 'error',
 
-        'no-debugger': 'error', // disallow use of debugger
+        // disallow use of debugger
+        'no-debugger': 'error',
 
-        'no-dupe-args': 'error', // disallow duplicate arguments in functions
+        // disallow duplicate arguments in functions
+        'no-dupe-args': 'error',
 
-        'no-dupe-else-if': 'error', // Disallow duplicate conditions in if-else-if chains
+        // Disallow duplicate conditions in if-else-if chains
+        'no-dupe-else-if': 'error',
 
-        'no-dupe-keys': 'error', // disallow duplicate keys when creating object literals
+        // disallow duplicate keys when creating object literals
+        'no-dupe-keys': 'error',
 
-        'no-duplicate-case': 'error', // disallow a duplicate case label.
+        // disallow a duplicate case label.
+        'no-duplicate-case': 'error',
 
-        'no-empty': 'error', // disallow empty statements
+        // disallow empty statements
+        'no-empty': 'error',
 
         // disallow the use of empty character classes in regular expressions
         'no-empty-character-class': 'error',
@@ -418,17 +409,23 @@ export function createConfig(options = {}) {
         // disallow double-negation boolean casts in a boolean context
         'no-extra-boolean-cast': 'error',
 
-        'no-func-assign': 'error', // disallow overwriting functions written as function declarations
+        // disallow overwriting functions written as function declarations
+        'no-func-assign': 'error',
 
-        'no-import-assign': 'error', // Disallow assigning to imported bindings
+        // Disallow assigning to imported bindings
+        'no-import-assign': 'error',
 
-        'no-inner-declarations': 'error', // disallow function or variable declarations in nested blocks
+        // disallow function or variable declarations in nested blocks
+        'no-inner-declarations': 'error',
 
-        'no-invalid-regexp': 'error', // disallow invalid regular expression strings in the RegExp constructor
+        // disallow invalid regular expression strings in the RegExp constructor
+        'no-invalid-regexp': 'error',
 
-        'no-irregular-whitespace': 'error', // disallow irregular whitespace outside of strings and comments
+        // disallow irregular whitespace outside of strings and comments
+        'no-irregular-whitespace': 'error',
 
-        'no-loss-of-precision': 'error', // Disallow Number Literals That Lose Precision
+        // Disallow Number Literals That Lose Precision
+        'no-loss-of-precision': 'error',
 
         // Disallow characters which are made with multiple code points in character class syntax
         'no-misleading-character-class': 'error',
@@ -442,11 +439,14 @@ export function createConfig(options = {}) {
         // disallow use of Object.prototypes builtins directly
         'no-prototype-builtins': 'error',
 
-        'no-regex-spaces': 'error', // disallow multiple spaces in a regular expression literal
+        // disallow multiple spaces in a regular expression literal
+        'no-regex-spaces': 'error',
 
-        'no-setter-return': 'error', // Disallow returning values from setters
+        // Disallow returning values from setters
+        'no-setter-return': 'error',
 
-        'no-sparse-arrays': 'error', // disallow sparse arrays
+        // disallow sparse arrays
+        'no-sparse-arrays': 'error',
 
         // Disallow template literal placeholder syntax in regular strings
         'no-template-curly-in-string': 'error',
@@ -466,7 +466,8 @@ export function createConfig(options = {}) {
           },
         ],
 
-        'no-unsafe-finally': 'error', // disallow return/throw/break/continue inside finally blocks
+        // disallow return/throw/break/continue inside finally blocks
+        'no-unsafe-finally': 'error',
 
         // disallow negating the left operand of relational operators
         'no-unsafe-negation': 'error',
@@ -477,15 +478,14 @@ export function createConfig(options = {}) {
           { disallowArithmeticOperators: true },
         ],
 
-        'no-unused-private-class-members': 1, // Disallow Unused Private Class Members
+        // Disallow Unused Private Class Members
+        'no-unused-private-class-members': 1,
 
-        'no-useless-backreference': 'error', // Disallow useless backreferences in regular expressions
+        // Disallow useless backreferences in regular expressions
+        'no-useless-backreference': 'error',
 
-        // Disallow assignments that can lead to race conditions due to usage of await or yield
-        // note: not enabled because it is very buggy
-        'require-atomic-updates': 0,
-
-        'use-isnan': 'error', // disallow comparisons with the value NaN
+        // disallow comparisons with the value NaN
+        'use-isnan': 'error',
 
         // ensure that the results of typeof are compared against a valid string
         'valid-typeof': ['error', { requireStringLiterals: true }],
@@ -494,17 +494,20 @@ export function createConfig(options = {}) {
         // enforces no braces where they can be omitted
         'arrow-body-style': ['error', 'always'],
 
-        'arrow-parens': ['error', 'always'], // require parens in arrow function arguments
+        // require parens in arrow function arguments
+        'arrow-parens': ['error', 'always'],
 
         // require space before/after arrow function's arrow
         'arrow-spacing': ['error', { before: true, after: true }],
 
-        'constructor-super': 'error', // verify super() callings in constructors
+        // verify super() callings in constructors
+        'constructor-super': 'error',
 
         // enforce the spacing around the * in generator functions
         'generator-star-spacing': ['error', { before: false, after: true }],
 
-        'no-class-assign': 'error', // disallow modifying variables of class declarations
+        // disallow modifying variables of class declarations
+        'no-class-assign': 'error',
 
         // disallow arrow functions where they could be confused with comparisons
         'no-confusing-arrow': [
@@ -514,13 +517,14 @@ export function createConfig(options = {}) {
           },
         ],
 
-        'no-const-assign': 'error', // disallow modifying variables that are declared using const
-
-        'no-dupe-class-members': 0, // disallow duplicate class members - handle by typescript-eslint
+        // disallow modifying variables that are declared using const
+        'no-const-assign': 'error',
 
-        'no-duplicate-imports': 2, // disallow importing from the same path more than once
+        // disallow importing from the same path more than once
+        'no-duplicate-imports': 2,
 
-        'no-new-symbol': 'error', // disallow symbol constructor
+        // disallow symbol constructor
+        'no-new-symbol': 'error',
 
         // Disallow specified names in exports
         'no-restricted-exports': [
@@ -533,20 +537,11 @@ export function createConfig(options = {}) {
           },
         ],
 
-        // disallow specific imports
-        'no-restricted-imports': [
-          'off',
-          {
-            paths: [],
-            patterns: [],
-          },
-        ],
-
-        'no-this-before-super': 'error', // disallow to use this/super before super() calling in constructors.
+        // disallow to use this/super before super() calling in constructors.
+        'no-this-before-super': 'error',
 
-        'no-useless-computed-key': 'error', // disallow useless computed property keys
-
-        'no-useless-constructor': 0, // disallow unnecessary constructor - handle by typescript-eslint
+        // disallow useless computed property keys
+        'no-useless-computed-key': 'error',
 
         // disallow renaming import, export, and destructured assignments to the same name
         'no-useless-rename': [
@@ -558,7 +553,8 @@ export function createConfig(options = {}) {
           },
         ],
 
-        'no-var': 'error', // require let or const instead of var
+        // require let or const instead of var
+        'no-var': 'error',
 
         // require method and property shorthand syntax for object literals
         'object-shorthand': [
@@ -606,68 +602,54 @@ export function createConfig(options = {}) {
           },
         ],
 
-        'prefer-numeric-literals': 'error', // disallow parseInt() in favor of binary, octal, and hexadecimal literals
-
-        'prefer-reflect': 'off', // suggest using Reflect methods where applicable
-
-        'prefer-rest-params': 'error', // use rest parameters instead of arguments
-
-        'prefer-spread': 'error', // suggest using the spread syntax instead of .apply()
-
-        'prefer-template': 'error', // suggest using template literals instead of string concatenation
-
-        'require-yield': 'error', // disallow generator functions that do not have yield
-
-        'rest-spread-spacing': ['error', 'never'], // enforce spacing between object rest-spread
-
-        // import sorting
-        'sort-imports': [
-          'off',
-          {
-            ignoreCase: false,
-            ignoreDeclarationSort: false,
-            ignoreMemberSort: false,
-            memberSyntaxSortOrder: ['none', 'all', 'multiple', 'single'],
-          },
-        ],
-
-        'symbol-description': 'error', // require a Symbol description
+        // disallow parseInt() in favor of binary, octal, and hexadecimal literals
+        'prefer-numeric-literals': 'error',
 
-        'template-curly-spacing': 'error', // enforce usage of spacing in template strings
+        // use rest parameters instead of arguments
+        'prefer-rest-params': 'error',
 
-        'yield-star-spacing': ['error', 'after'], // enforce spacing around the * in yield* expressions
+        // suggest using the spread syntax instead of .apply()
+        'prefer-spread': 'error',
 
-        /* Enforce node rules://eslint - https://eslint.org/docs/latest/rules/ */
+        // suggest using template literals instead of string concatenation
+        'prefer-template': 'error',
 
-        'callback-return': 'off', // enforce return after a callback
+        // disallow generator functions that do not have yield
+        'require-yield': 'error',
 
-        'global-require': 'error', // require all requires be top-level
+        // enforce spacing between object rest-spread
+        'rest-spread-spacing': ['error', 'never'],
 
-        'handle-callback-err': 'off', // enforces error handling in callbacks (node environment)
+        // require a Symbol description
+        'symbol-description': 'error',
 
-        'no-buffer-constructor': 'error', // disallow use of the Buffer() constructor
+        // enforce usage of spacing in template strings
+        'template-curly-spacing': 'error',
 
-        'no-mixed-requires': ['off', false], // disallow mixing regular variable and require declarations
+        // enforce spacing around the * in yield* expressions
+        'yield-star-spacing': ['error', 'after'],
 
-        'no-new-require': 'error', // disallow use of new operator with the require function
+        // require all requires be top-level
+        'global-require': 'error',
 
-        'no-path-concat': 'error', // disallow string concatenation with __dirname and __filename
+        // disallow use of the Buffer() constructor
+        'no-buffer-constructor': 'error',
 
-        'no-process-env': 'off', // disallow use of process.env
+        // disallow use of new operator with the require function
+        'no-new-require': 'error',
 
-        'no-process-exit': 'off', // disallow process.exit()
-
-        'no-restricted-modules': 'off', // restrict usage of specified node modules
-
-        'no-sync': 'off', // disallow use of synchronous methods (off by default)
+        // disallow string concatenation with __dirname and __filename
+        'no-path-concat': 'error',
 
         /* Enforce strict rules */
 
-        strict: ['error', 'never'], // babel inserts `'use strict';` for us
+        // babel inserts `'use strict';` for us
+        strict: ['error', 'never'],
 
         /* Enforce style rules */
 
-        'no-ternary': ['error'], // disallow the use of ternary operators
+        // disallow the use of ternary operators
+        'no-ternary': ['error'],
 
         // disallow trailing whitespace at the end of lines
         'no-trailing-spaces': [
@@ -678,17 +660,6 @@ export function createConfig(options = {}) {
           },
         ],
 
-        // disallow dangling underscores in identifiers
-        'no-underscore-dangle': [
-          0,
-          {
-            allow: [],
-            allowAfterThis: false,
-            allowAfterSuper: false,
-            enforceInMethodNames: true,
-          },
-        ],
-
         // disallow the use of Boolean literals in conditional expressions
         // also, prefer `a || b` over `a ? a : b`
         'no-unneeded-ternary': ['error', { defaultAssignment: false }],
@@ -703,34 +674,8 @@ export function createConfig(options = {}) {
           { overrides: {} },
         ],
 
-        '@stylistic-eslint/object-curly-spacing': ['error', 'always'], // require padding inside curly braces
-
-        // enforce line breaks between braces
-        'object-curly-newline': [
-          0,
-          {
-            ObjectExpression: {
-              minProperties: 4,
-              multiline: true,
-              consistent: true,
-            },
-            ObjectPattern: {
-              minProperties: 4,
-              multiline: true,
-              consistent: true,
-            },
-            ImportDeclaration: {
-              minProperties: 4,
-              multiline: true,
-              consistent: true,
-            },
-            ExportDeclaration: {
-              minProperties: 4,
-              multiline: true,
-              consistent: true,
-            },
-          },
-        ],
+        // require padding inside curly braces
+        '@stylistic-eslint/object-curly-spacing': ['error', 'always'],
 
         // enforce "same line" or "multiple line" on object properties.
         'object-property-newline': [
@@ -740,15 +685,15 @@ export function createConfig(options = {}) {
           },
         ],
 
-        'one-var': ['error', 'never'], // allow just one var statement per function
+        // allow just one var statement per function
+        'one-var': ['error', 'never'],
 
-        'one-var-declaration-per-line': ['error', 'always'], // require a newline around variable declaration
+        // require a newline around variable declaration
+        'one-var-declaration-per-line': ['error', 'always'],
 
         // require assignment operator shorthand where possible or prohibit it entirely
         'operator-assignment': ['error', 'always'],
 
-        '@stylistic-eslint/operator-linebreak': 0,
-
         // disallow padding within blocks
         '@stylistic-eslint/padded-blocks': [
           'error',
@@ -762,13 +707,11 @@ export function createConfig(options = {}) {
           },
         ],
 
-        // Require or disallow padding lines between statements
-        '@stylistic-eslint/padding-line-between-statements': 'off',
-
         // Disallow the use of Math.pow in favor of the ** operator
         'prefer-exponentiation-operator': 'error',
 
-        'prefer-object-spread': 'error', // Prefer use of an object spread over Object.assign
+        // Prefer use of an object spread over Object.assign
+        'prefer-object-spread': 'error',
 
         // require quotes around object literal property names
         '@stylistic-eslint/quote-props': [
@@ -791,12 +734,8 @@ export function createConfig(options = {}) {
           { before: false, after: true },
         ],
 
-        '@stylistic-eslint/semi-style': ['error', 'last'], // Enforce location of semicolons
-
-        // requires object keys to be sorted
-        'sort-keys': ['off', 'asc', { caseSensitive: false, natural: true }],
-
-        'sort-vars': 'off', // sort variables within the same declaration block
+        // Enforce location of semicolons
+        '@stylistic-eslint/semi-style': ['error', 'last'],
 
         // require or disallow space before blocks
         '@stylistic-eslint/space-before-blocks': 'error',
@@ -814,7 +753,8 @@ export function createConfig(options = {}) {
         // require or disallow spaces inside parentheses
         '@stylistic-eslint/space-in-parens': ['error', 'never'],
 
-        '@stylistic-eslint/space-infix-ops': 'error', // require spaces around operators
+        // require spaces around operators
+        '@stylistic-eslint/space-infix-ops': 'error',
 
         // Require or disallow spaces before/after unary operators
         '@stylistic-eslint/space-unary-ops': [
@@ -852,16 +792,12 @@ export function createConfig(options = {}) {
         // Require or disallow spacing between template tags and their literals
         '@stylistic-eslint/template-tag-spacing': ['error', 'never'],
 
-        'unicode-bom': ['error', 'never'], // require or disallow the Unicode Byte Order Mark
+        // require or disallow the Unicode Byte Order Mark
+        'unicode-bom': ['error', 'never'],
 
-        '@stylistic-eslint/wrap-regex': 'off', // require regex literals to be wrapped in parentheses
-
-        /* Enforce variables rules */
-        'init-declarations': 'off',
-        'no-array-constructor': 'off', // disallow use of the Array constructor - handle by typescript-eslint
-        'no-catch-shadow': 'off',
         'no-delete-var': 'error',
         'no-label-var': 'error',
+
         'no-restricted-globals': [
           'error',
           {
@@ -876,22 +812,14 @@ export function createConfig(options = {}) {
           },
         ].concat(confusingBrowserGlobals),
 
-        'no-shadow': 0, // disallow declaration of variables already declared in the outer scope
-
-        'no-shadow-restricted-names': 'error', // disallow shadowing of names such as arguments
-
-        'no-undef': 'error', // disallow use of undeclared variables unless mentioned in a /*global */ block
-
-        'no-undef-init': 'error', // disallow use of undefined when initializing variables
-
-        // Alternative with no-undef-init , no-shadow-restricted-names, no-global-assign activated
-        'no-undefined': 'off', // disallow use of undefined variable
+        // disallow shadowing of names such as arguments
+        'no-shadow-restricted-names': 'error',
 
-        // disallow declaration of variables that are not used in the code - handle by typescript-eslint
-        'no-unused-vars': 'off',
+        // disallow use of undeclared variables unless mentioned in a /*global */ block
+        'no-undef': 'error',
 
-        // disallow use of variables before they are defined - handle by typescript-eslint
-        'no-use-before-define': 'off',
+        // disallow use of undefined when initializing variables
+        'no-undef-init': 'error',
 
         'no-restricted-syntax': [
           'error',
@@ -935,6 +863,7 @@ export function createConfig(options = {}) {
             ignoreStrings: true,
           },
         ],
+
         '@typescript-eslint/naming-convention': [
           'error',
           {
@@ -1115,32 +1044,14 @@ export function createConfig(options = {}) {
             allowRuleToRunWithoutStrictNullChecksIKnowWhatIAmDoing: false,
           },
         ],
-        '@stylistic-eslint/no-extra-parens': [
-          'off',
-          'all',
-          {
-            conditionalAssign: true,
-            nestedBinaryExpressions: false,
-            returnAssign: false,
-            ignoreJSX: 'all', // delegate to eslint-plugin-react
-            enforceForArrowConditionals: false,
-          },
-        ],
+
         '@typescript-eslint/no-empty-function': [
           'error',
           {
             allow: ['arrowFunctions', 'functions', 'methods'],
           },
         ],
-        '@stylistic-eslint/no-magic-numbers': [
-          'off',
-          {
-            ignore: [],
-            ignoreArrayIndexes: true,
-            enforceConst: true,
-            detectObjects: false,
-          },
-        ],
+
         '@typescript-eslint/no-unused-expressions': [
           'error',
           {
@@ -1149,18 +1060,20 @@ export function createConfig(options = {}) {
             allowTaggedTemplates: false,
           },
         ],
+
         '@typescript-eslint/no-unused-vars': [
           'error',
           { vars: 'all', args: 'after-used', ignoreRestSiblings: true },
         ],
+
         '@typescript-eslint/no-use-before-define': [
           'error',
           { functions: true, classes: true, variables: true },
         ],
 
-        '@stylistic-eslint/adjacent-overload-signatures': 'off',
         '@typescript-eslint/array-type': ['error'],
         '@typescript-eslint/await-thenable': ['error'],
+
         '@typescript-eslint/ban-ts-comment': [
           'error',
           {
@@ -1168,32 +1081,29 @@ export function createConfig(options = {}) {
             minimumDescriptionLength: 3,
           },
         ],
+
         '@typescript-eslint/ban-tslint-comment': ['error'],
         '@typescript-eslint/no-restricted-types': 2,
         '@typescript-eslint/no-empty-object-type': 2,
         '@typescript-eslint/no-unsafe-function-type': 2,
         '@typescript-eslint/no-wrapper-object-types': 2,
-        '@stylistic-eslint/brace-style': 'off',
-        '@stylistic-eslint/class-literal-property-style': 'off',
+
         '@stylistic-eslint/comma-spacing': [
           'error',
           { before: false, after: true },
         ],
+
         '@typescript-eslint/consistent-generic-constructors': ['error'],
+
         '@typescript-eslint/consistent-indexed-object-style': [
           'error',
           'record',
         ],
+
         '@typescript-eslint/consistent-type-assertions': ['error'],
-        '@typescript-eslint/consistent-type-definitions': 'off',
-        '@typescript-eslint/consistent-type-exports': 'off',
-        '@typescript-eslint/consistent-type-imports': 'off',
-        // Activate explicit-module-boundary-type as an alternative
-        '@typescript-eslint/explicit-function-return-type': 'off',
         '@typescript-eslint/explicit-member-accessibility': ['error'],
         '@typescript-eslint/explicit-module-boundary-types': 2,
         '@/func-call-spacing': ['error', 'never'],
-        '@stylistic-eslint/indent': 0,
         '@stylistic-eslint/member-delimiter-style': ['error'],
         '@typescript-eslint/member-ordering': ['error'],
         '@typescript-eslint/method-signature-style': ['error'],
@@ -1202,7 +1112,6 @@ export function createConfig(options = {}) {
         '@typescript-eslint/no-confusing-non-null-assertion': ['error'],
         '@typescript-eslint/no-confusing-void-expression': ['error'],
         '@typescript-eslint/no-duplicate-enum-values': ['error'],
-        '@typescript-eslint/no-dupe-class-members': 0, // automatically checked by the TypeScript compiler
         '@typescript-eslint/no-dynamic-delete': ['error'],
         '@typescript-eslint/no-empty-interface': ['error'],
         '@typescript-eslint/no-explicit-any': [2],
@@ -1221,7 +1130,6 @@ export function createConfig(options = {}) {
         '@typescript-eslint/no-non-null-asserted-optional-chain': ['error'],
         '@typescript-eslint/no-non-null-assertion': ['error'],
         '@typescript-eslint/no-redundant-type-constituents': 'error',
-        '@typescript-eslint/no-require-imports': 'off',
         '@typescript-eslint/no-this-alias': ['error'],
         '@typescript-eslint/only-throw-error': 'error',
         '@typescript-eslint/no-unnecessary-boolean-literal-compare': ['error'],
@@ -1230,16 +1138,13 @@ export function createConfig(options = {}) {
         '@typescript-eslint/no-unnecessary-type-arguments': ['error'],
         '@typescript-eslint/no-unnecessary-type-assertion': ['error'],
         '@typescript-eslint/no-unnecessary-type-constraint': ['error'],
-        '@typescript-eslint/no-unsafe-argument': 'off',
-        '@typescript-eslint/no-unsafe-assignment': 0,
-        '@typescript-eslint/no-unsafe-call': ['warn'],
-        '@typescript-eslint/no-unsafe-member-access': [1],
-        '@typescript-eslint/no-unsafe-return': ['warn'],
-        '@typescript-eslint/no-useless-empty-export': 'off',
+        '@typescript-eslint/no-unsafe-argument': 'error',
+        '@typescript-eslint/no-unsafe-assignment': 'error',
+        '@typescript-eslint/no-unsafe-call': ['error'],
+        '@typescript-eslint/no-unsafe-member-access': ['error'],
+        '@typescript-eslint/no-unsafe-return': ['error'],
         '@typescript-eslint/no-useless-constructor': 'error',
-        '@typescript-eslint/no-var-requires': 'off',
         '@typescript-eslint/non-nullable-type-assertion-style': ['error'],
-        '@typescript-eslint/parameter-properties': 'off',
         '@typescript-eslint/prefer-as-const': ['error'],
         '@typescript-eslint/prefer-enum-initializers': ['error'],
         '@typescript-eslint/prefer-for-of': ['error'],
@@ -1249,8 +1154,6 @@ export function createConfig(options = {}) {
         '@typescript-eslint/prefer-namespace-keyword': ['error'],
         '@typescript-eslint/prefer-nullish-coalescing': ['error'],
         '@typescript-eslint/prefer-optional-chain': ['error'],
-        '@typescript-eslint/prefer-readonly-parameter-types': 'off', // ['error'],
-        '@typescript-eslint/prefer-readonly': 'off',
         '@typescript-eslint/prefer-reduce-type-parameter': ['error'],
         '@typescript-eslint/prefer-regexp-exec': ['error'],
         '@typescript-eslint/prefer-return-this-type': ['error'],
@@ -1262,55 +1165,12 @@ export function createConfig(options = {}) {
         '@typescript-eslint/switch-exhaustiveness-check': ['error'],
         '@typescript-eslint/triple-slash-reference': ['error'],
         '@stylistic-eslint/type-annotation-spacing': ['error'],
-        '@typescript-eslint/typedef': 'off',
         '@typescript-eslint/unbound-method': ['error'],
         '@typescript-eslint/unified-signatures': ['error'],
-
-        'brace-style': 'off',
         'func-style': ['error', 'declaration'],
-
         'no-continue': 1,
-
-        'jsdoc/require-param-description': 0,
-        'jsdoc/require-param-type': 0,
-        'jsdoc/require-param': 0,
-        'jsdoc/require-returns-type': 0,
-        'jsdoc/require-returns': 0,
-        'jsdoc/require-yields': 0,
-
-        camelcase: 'off',
-
-        indent: 'off',
-
         'sonarjs/no-duplicate-string': 'error',
-        // Lot of new rules, we disable them for now
-        'sonarjs/concise-regex': 0,
-        'sonarjs/default-param-last': 0,
-        'sonarjs/deprecation': 0,
-        'sonarjs/duplicates-in-character-class': 0,
-        'sonarjs/function-return-type': 0,
-        'sonarjs/hashing': 0,
-        'sonarjs/no-base-to-string': 0,
-        'sonarjs/no-commented-code': 0,
-        'sonarjs/no-dead-store': 0,
         'sonarjs/no-duplicate-string': 1,
-        'sonarjs/no-empty-function': 0,
-        'sonarjs/no-hardcoded-credentials': 0,
-        'sonarjs/no-ignored-exceptions': 0,
-        'sonarjs/no-incomplete-assertions': 0,
-        'sonarjs/no-misused-promises': 0,
-        'sonarjs/no-nested-functions': 0,
-        'sonarjs/no-redundant-optional': 0,
-        'sonarjs/no-redundant-type-constituents': 0,
-        'sonarjs/no-useless-intersection': 0,
-        'sonarjs/prefer-for-of': 0,
-        'sonarjs/pseudo-random': 0,
-        'sonarjs/redundant-type-aliases': 0,
-        'sonarjs/single-character-alternation': 0,
-        'sonarjs/slow-regex': 0,
-        'sonarjs/sonar-no-control-regex': 0,
-        'sonarjs/sonar-no-unused-vars': 0,
-        'sonarjs/todo-tag': 0,
       },
     },
     {

package/package.json

@@ -1,7 +1,7 @@
 {
   "author": "Olivier Hardy <olivier@plumile.com>",
   "name": "@plumile/eslint-config-typescript",
-  "version": "0.1.200",
+  "version": "0.1.205",
   "description": "Shared ESLint flat config tuned for TypeScript-forward React projects",
   "type": "module",
   "license": "MIT",
@@ -31,29 +31,25 @@
     "npm": ">=8.0.0"
   },
   "dependencies": {
-    "@typescript-eslint/parser": "8.66.0"
+    "@typescript-eslint/parser": "8.67.0"
   },
   "devDependencies": {
     "@stylistic/eslint-plugin": "5.10.0",
     "eslint": "10.1.0",
-    "eslint-plugin-jsdoc": "64.0.1",
-    "eslint-plugin-jsx-a11y": "6.10.2",
+    "eslint-plugin-jsdoc": "64.1.0",
     "eslint-plugin-n": "18.3.0",
-    "eslint-plugin-react": "7.37.5",
     "eslint-plugin-react-hooks": "7.1.1",
-    "eslint-plugin-relay": "2.0.0",
+    "eslint-plugin-relay": "2.1.0",
     "eslint-plugin-sonarjs": "4.2.0"
   },
   "peerDependencies": {
     "@stylistic/eslint-plugin": "5.10.0",
-    "@typescript-eslint/eslint-plugin": ">=8.66.0",
+    "@typescript-eslint/eslint-plugin": ">=8.67.0",
     "eslint": "10.x.x",
-    "eslint-plugin-jsdoc": ">=64.0.1",
-    "eslint-plugin-jsx-a11y": ">=6.10.2",
+    "eslint-plugin-jsdoc": ">=64.1.0",
     "eslint-plugin-n": "18.3.0",
-    "eslint-plugin-react": ">=7.37.5",
     "eslint-plugin-react-hooks": "7.1.1",
-    "eslint-plugin-relay": ">=2.0.0",
+    "eslint-plugin-relay": ">=2.1.0",
     "eslint-plugin-sonarjs": ">=4.2.0"
   },
   "files": [