@viclafouch/eslint-config-viclafouch 4.17.1-beta.3 → 4.17.1-beta.4

package/index.mjs

@@ -13,7 +13,7 @@ export default [
   {
     files: ['**/*.{js,mjs,cjs,jsx,ts,tsx}'],
     linterOptions: {
-      reportUnusedDisableDirectives: "error"
+      reportUnusedDisableDirectives: 'error'
     }
   },
   bestPracticesConfig,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@viclafouch/eslint-config-viclafouch",
-  "version": "4.17.1-beta.3",
+  "version": "4.17.1-beta.4",
   "description": "ESLint and Prettier Config from Victor de la Fouchardiere",
   "main": "index.js",
   "keywords": [
@@ -26,14 +26,13 @@
   "homepage": "https://github.com/viclafouch/eslint-config-viclafouch#readme",
   "peerDependencies": {
     "eslint": ">= 9",
-    "prettier": "^3.4.2",
-    "typescript": "^5.7.2"
+    "prettier": ">= 3",
+    "typescript": ">= 5"
   },
   "dependencies": {
     "@babel/core": "^7.26.0",
     "@babel/eslint-parser": "^7.25.9",
     "@next/eslint-plugin-next": "^15.1.0",
-    "@rushstack/eslint-patch": "^1.10.4",
     "@total-typescript/ts-reset": "^0.6.1",
     "@typescript-eslint/eslint-plugin": "^8.18.0",
     "@typescript-eslint/parser": "^8.18.0",

package/rules/best-practices.mjs

@@ -12,7 +12,7 @@ export default {
     }
   },
   rules: {
-    // enforces getter/setter pairs in objects
+    // Enforces getter/setter pairs in objects
     // https://eslint.org/docs/rules/accessor-pairs
     'accessor-pairs': 'off',
 
@@ -20,19 +20,34 @@ export default {
     // https://eslint.org/docs/rules/curly
     curly: ['error', 'all'],
 
-    // enforces return statements in callbacks of array's methods
+    // Enforces return statements in callbacks of array's methods
     // https://eslint.org/docs/rules/array-callback-return
-    'array-callback-return': ['error', { allowImplicit: true }],
+    'array-callback-return': [
+      'error',
+      { allowImplicit: true, checkForEach: true }
+    ],
+
+    // Enforce for loop update clause moving the counter in the right direction
+    // https://eslint.org/docs/latest/rules/for-direction
+    'for-direction': 'error',
+
+    // Enforce return statements in getters
+    // https://eslint.org/docs/latest/rules/getter-return
+    'getter-return': 'error',
+
+    // Disallow returning values from setters
+    // https://eslint.org/docs/latest/rules/no-setter-return
+    'no-setter-return': 'error',
 
-    // treat var statements as if they were block scoped
+    // Rreat var statements as if they were block scoped
     // https://eslint.org/docs/rules/block-scoped-var
     'block-scoped-var': 'error',
 
-    // specify the maximum cyclomatic complexity allowed in a program
+    // Specify the maximum cyclomatic complexity allowed in a program
     // https://eslint.org/docs/rules/complexity
-    complexity: ['off', 20],
+    complexity: ['error', 20],
 
-    // enforce that class methods use "this"
+    // Enforce that class methods use "this"
     // https://eslint.org/docs/rules/class-methods-use-this
     'class-methods-use-this': [
       'error',
@@ -41,13 +56,25 @@ export default {
       }
     ],
 
-    // require return statements to either always or never specify values
+    // Require return statements to either always or never specify values
     // https://eslint.org/docs/rules/consistent-return
     'consistent-return': 'error',
 
-    // require default case in switch statements
+    // Disallow comparing against -0
+    // https://eslint.org/docs/latest/rules/no-compare-neg-zero
+    'no-compare-neg-zero': 'error',
+
+    // Disallow sparse arrays
+    // https://eslint.org/docs/latest/rules/no-sparse-arrays
+    'no-sparse-arrays': 'error',
+
+    // Disallow expressions where the operation doesn't affect the value
+    // https://eslint.org/docs/latest/rules/no-constant-binary-expression
+    'no-constant-binary-expression': 'error',
+
+    // Require default case in switch statements
     // https://eslint.org/docs/rules/default-case
-    'default-case': ['error', { commentPattern: '^no default$' }],
+    'default-case': ['error'],
 
     // Enforce default clauses in switch statements to be last
     // https://eslint.org/docs/rules/default-case-last
@@ -56,23 +83,27 @@ export default {
     // https://eslint.org/docs/rules/default-param-last
     'default-param-last': 'error',
 
-    // encourages use of dot notation whenever possible
+    // Encourages use of dot notation whenever possible
     // https://eslint.org/docs/rules/dot-notation
     'dot-notation': ['error', { allowKeywords: true }],
 
-    // enforces consistent newlines before or after dots
+    // Require the use of === and !==
+    // https://eslint.org/docs/latest/rules/eqeqeq
+    eqeqeq: ['error', 'always'],
+
+    // Enforces consistent newlines before or after dots
     // https://eslint.org/docs/rules/dot-location
     'dot-location': ['error', 'property'],
 
-    // make sure for-in loops have an if statement
+    // Make sure for-in loops have an if statement
     // https://eslint.org/docs/rules/guard-for-in
     'guard-for-in': 'error',
 
-    // disallow the use of alert, confirm, and prompt
+    // Disallow the use of alert, confirm, and prompt
     // https://eslint.org/docs/rules/no-alert
     'no-alert': 'error',
 
-    // disallow lexical declarations in case/default clauses
+    // Disallow lexical declarations in case/default clauses
     // https://eslint.org/docs/rules/no-case-declarations
     'no-case-declarations': 'error',
 
@@ -80,11 +111,11 @@ export default {
     // https://eslint.org/docs/rules/no-constructor-return
     'no-constructor-return': 'error',
 
-    // disallow else after a return in an if
+    // Disallow else after a return in an if
     // https://eslint.org/docs/rules/no-else-return
     'no-else-return': ['error', { allowElseIf: false }],
 
-    // disallow empty functions, except for standalone funcs/arrows
+    // Disallow empty functions, except for standalone funcs/arrows
     // https://eslint.org/docs/rules/no-empty-function
     'no-empty-function': [
       'error',
@@ -93,63 +124,75 @@ export default {
       }
     ],
 
-    // disallow empty destructuring patterns
+    // Disallow empty destructuring patterns
     // https://eslint.org/docs/rules/no-empty-pattern
     'no-empty-pattern': 'error',
 
-    // disallow comparisons to null without a type-checking operator
+    // Disallow comparisons to null without a type-checking operator
     // https://eslint.org/docs/rules/no-eq-null
     'no-eq-null': 'off',
 
-    // disallow adding to native types
+    // Disallow adding to native types
     // https://eslint.org/docs/rules/no-extend-native
     'no-extend-native': 'error',
 
-    // disallow Unnecessary Labels
+    // Disallow Unnecessary Labels
     // https://eslint.org/docs/rules/no-extra-label
     'no-extra-label': 'error',
 
-    // disallow fallthrough of case statements
+    // Disallow fallthrough of case statements
     // https://eslint.org/docs/rules/no-fallthrough
     'no-fallthrough': 'error',
 
-    // disallow the use of leading or trailing decimal points in numeric literals
+    // Disallow the use of leading or trailing decimal points in numeric literals
     // https://eslint.org/docs/rules/no-floating-decimal
     'no-floating-decimal': 'error',
 
-    // disallow reassignments of native objects or read-only globals
+    // Disallow reassignments of native objects or read-only globals
     // https://eslint.org/docs/rules/no-global-assign
     'no-global-assign': ['error', { exceptions: [] }],
 
-    // deprecated in favor of no-global-assign
+    // Deprecated in favor of no-global-assign
     // https://eslint.org/docs/rules/no-native-reassign
     'no-native-reassign': 'off',
 
-    // disallow implicit type conversions
+    // Disallow implicit type conversions
     // https://eslint.org/docs/rules/no-implicit-coercion
     'no-implicit-coercion': 'error',
 
-    // disallow var and named functions in global scope
+    // Disallow reassigning function declarations
+    // https://eslint.org/docs/latest/rules/no-func-assign
+    'no-func-assign': 'error',
+
+    // Disallow assigning to imported bindings
+    // https://eslint.org/docs/latest/rules/no-import-assign
+    'no-import-assign': 'error',
+
+    // Disallow var and named functions in global scope
     // https://eslint.org/docs/rules/no-implicit-globals
     'no-implicit-globals': 'off',
 
-    // disallow this keywords outside of classes or class-like objects
+    // Disallow this keywords outside of classes or class-like objects
     // https://eslint.org/docs/rules/no-invalid-this
     'no-invalid-this': 'off',
 
-    // disallow usage of __iterator__ property
+    // Disallow variable or function declarations in nested blocks
+    // https://eslint.org/docs/latest/rules/no-inner-declarations
+    'no-inner-declarations': ['error', 'both'],
+
+    // Disallow usage of __iterator__ property
     // https://eslint.org/docs/rules/no-iterator
     'no-iterator': 'error',
 
-    // disallow use of labels for anything other than loops and switches
+    // Disallow use of labels for anything other than loops and switches
     // https://eslint.org/docs/rules/no-labels
     'no-labels': ['error', { allowLoop: false, allowSwitch: false }],
 
-    // disallow unnecessary nested blocks
+    // Disallow unnecessary nested blocks
     // https://eslint.org/docs/rules/no-lone-blocks
     'no-lone-blocks': 'error',
 
-    // disallow use of multiple spaces
+    // Disallow use of multiple spaces
     // https://eslint.org/docs/rules/no-multi-spaces
     'no-multi-spaces': [
       'error',
@@ -158,30 +201,30 @@ export default {
       }
     ],
 
-    // disallow use of multiline strings
+    // Disallow use of multiline strings
     // https://eslint.org/docs/rules/no-multi-str
     'no-multi-str': '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
     // https://eslint.org/docs/rules/no-new
     'no-new': 'error',
 
-    // disallows creating new instances of String, Number, and Boolean
+    // Disallows creating new instances of String, Number, and Boolean
     // https://eslint.org/docs/rules/no-new-wrappers
     'no-new-wrappers': 'error',
 
-    // disallow use of (old style) octal literals
+    // Disallow use of (old style) octal literals
     // https://eslint.org/docs/rules/no-octal
     'no-octal': 'error',
 
-    // disallow use of octal escape sequences in string literals, such as
-    // var foo = 'Copyright \251';
+    // Disallow use of octal escape sequences in string literals, such as
+    // Var foo = 'Copyright \251';
     // https://eslint.org/docs/rules/no-octal-escape
     'no-octal-escape': 'error',
 
-    // disallow reassignment of function parameters
-    // disallow parameter object manipulation except for specific exclusions
-    // rule: https://eslint.org/docs/rules/no-param-reassign.html
+    // Disallow reassignment of function parameters
+    // Disallow parameter object manipulation except for specific exclusions
+    // Rule: https://eslint.org/docs/rules/no-param-reassign.html
     'no-param-reassign': [
       'error',
       {
@@ -193,11 +236,11 @@ export default {
       }
     ],
 
-    // disallow declaring the same variable more than once
+    // Disallow declaring the same variable more than once
     // https://eslint.org/docs/rules/no-redeclare
     'no-redeclare': 'error',
 
-    // disallow certain object properties
+    // Disallow certain object properties
     // https://eslint.org/docs/rules/no-restricted-properties
     'no-restricted-properties': [
       'error',
@@ -243,11 +286,15 @@ export default {
       }
     ],
 
-    // disallow use of assignment in return statement
+    // Disallow use of assignment in return statement
     // https://eslint.org/docs/rules/no-return-assign
     'no-return-assign': ['error', 'always'],
 
-    // disallow self assignment
+    // Disallow ternary operators when simpler alternatives exist
+    // https://eslint.org/docs/rules/no-unneeded-ternary
+    'no-unneeded-ternary': 'error',
+
+    // Disallow assignments where both sides are exactly the same
     // https://eslint.org/docs/rules/no-self-assign
     'no-self-assign': [
       'error',
@@ -256,19 +303,35 @@ export default {
       }
     ],
 
-    // disallow comparisons where both sides are exactly the same
+    // Disallow comparisons where both sides are exactly the same
     // https://eslint.org/docs/rules/no-self-compare
     'no-self-compare': 'error',
 
-    // restrict what can be thrown as an exception
+    // Restrict what can be thrown as an exception
     // https://eslint.org/docs/rules/no-throw-literal
     'no-throw-literal': 'error',
 
-    // disallow unmodified conditions of loops
+    // Disallow unmodified conditions of loops
     // https://eslint.org/docs/rules/no-unmodified-loop-condition
-    'no-unmodified-loop-condition': 'off',
+    'no-unmodified-loop-condition': 'error',
+
+    // Disallow negating the left operand of relational operators
+    // https://eslint.org/docs/latest/rules/no-unsafe-negation
+    'no-unsafe-negation': 'error',
+
+    // Enforce comparing typeof expressions against valid strings
+    // https://eslint.org/docs/latest/rules/valid-typeof
+    'valid-typeof': 'error',
+
+    // Require calls to isNaN() when checking for NaN
+    // https://eslint.org/docs/latest/rules/use-isnan
+    'use-isnan': 'error',
+
+    // Disallow unreachable code after return, throw, continue, and break statements
+    // https://eslint.org/docs/latest/rules/no-unreachable
+    'no-unreachable': 'error',
 
-    // disallow usage of expressions in statement position
+    // Disallow usage of expressions in statement position
     // https://eslint.org/docs/rules/no-unused-expressions
     'no-unused-expressions': [
       'error',
@@ -279,11 +342,11 @@ export default {
       }
     ],
 
-    // disallow unused labels
+    // Disallow unused labels
     // https://eslint.org/docs/rules/no-unused-labels
     'no-unused-labels': 'error',
 
-    // disallow unnecessary .call() and .apply()
+    // Disallow unnecessary .call() and .apply()
     // https://eslint.org/docs/rules/no-useless-call
     'no-useless-call': 'off',
 
@@ -291,19 +354,19 @@ export default {
     // https://eslint.org/docs/rules/no-useless-catch
     'no-useless-catch': 'error',
 
-    // disallow useless string concatenation
+    // Disallow useless string concatenation
     // https://eslint.org/docs/rules/no-useless-concat
     'no-useless-concat': 'error',
 
-    // disallow unnecessary string escaping
+    // Disallow unnecessary string escaping
     // https://eslint.org/docs/rules/no-useless-escape
     'no-useless-escape': 'error',
 
-    // disallow redundant return; keywords
+    // Disallow redundant return; keywords
     // https://eslint.org/docs/rules/no-useless-return
     'no-useless-return': 'error',
 
-    // require using Error objects as Promise rejection reasons
+    // Require using Error objects as Promise rejection reasons
     // https://eslint.org/docs/rules/prefer-promise-reject-errors
     'prefer-promise-reject-errors': ['error', { allowEmptyReject: true }],
 
@@ -319,7 +382,7 @@ export default {
       }
     ],
 
-    // require use of the second argument for parseInt()
+    // Require use of the second argument for parseInt()
     // https://eslint.org/docs/rules/radix
     radix: 'error',
 

package/rules/errors.mjs

@@ -9,32 +9,38 @@ export default {
     // https://eslint.org/docs/rules/no-await-in-loop
     'no-await-in-loop': 'error',
 
-    // disallow assignment in conditional expressions
+    // Disallow assignment in conditional expressions
+    // https://eslint.org/docs/latest/rules/no-cond-assign
     'no-cond-assign': ['error', 'always'],
 
-    // disallow use of console
+    // Disallow use of console
     'no-console': 'warn',
 
-    // disallow use of constant expressions in conditions
-    'no-constant-condition': 'warn',
+    // Disallow use of constant expressions in conditions
+    // https://eslint.org/docs/latest/rules/no-constant-condition
+    'no-constant-condition': 'error',
 
-    // disallow use of debugger
+    // Disallow use of debugger
+    // https://eslint.org/docs/latest/rules/no-debugger
     'no-debugger': 'error',
 
     // Disallow duplicate conditions in if-else-if chains
     // https://eslint.org/docs/rules/no-dupe-else-if
     'no-dupe-else-if': 'error',
 
-    // disallow a duplicate case label.
+    // Disallow duplicate case labels
+    // https://eslint.org/docs/latest/rules/no-duplicate-case
     'no-duplicate-case': 'error',
 
-    // disallow duplicate keys when creating object literals
+    // Disallow duplicate keys when creating object literals
+    // https://eslint.org/docs/latest/rules/no-dupe-keys
     'no-dupe-keys': 'error',
 
-    // disallow assigning to the exception in a catch block
+    // Disallow reassigning exceptions in catch clauses
+    // https://eslint.org/docs/latest/rules/no-ex-assign
     'no-ex-assign': 'error',
 
-    // disallow unnecessary semicolons
+    // Disallow unnecessary semicolons
     'no-extra-semi': 'error'
   }
 }

package/rules/es6.mjs

@@ -19,26 +19,30 @@ export default {
     sourceType: 'module'
   },
   rules: {
-    // Always arrow functions
+    // Require braces around arrow function bodies
     // https://eslint.org/docs/rules/arrow-body-style
     'arrow-body-style': ['error', 'always'],
 
-    // require parens in arrow function arguments
+    // Require parens in arrow function arguments
     // https://eslint.org/docs/rules/arrow-parens
     'arrow-parens': ['error', 'always'],
 
-    // require space before/after arrow function's arrow
+    // Require space before/after arrow function's arrow
     // https://eslint.org/docs/rules/arrow-spacing
     'arrow-spacing': ['error', { before: true, after: true }],
 
-    // verify super() callings in constructors
+    // Verify super() callings in constructors
     'constructor-super': 'error',
 
-    // disallow modifying variables of class declarations
+    // Disallow modifying variables of class declarations
     // https://eslint.org/docs/rules/no-class-assign
     'no-class-assign': 'error',
 
-    // disallow arrow functions where they could be confused with comparisons
+    // Disallow duplicate class members
+    // https://eslint.org/docs/latest/rules/no-dupe-class-members
+    'no-dupe-class-members': 'error',
+
+    // Disallow arrow functions where they could be confused with comparisons
     // https://eslint.org/docs/rules/no-confusing-arrow
     'no-confusing-arrow': [
       'error',
@@ -47,39 +51,60 @@ export default {
       }
     ],
 
-    // disallow modifying variables that are declared using const
+    // Disallow modifying variables that are declared using const
+    // https://eslint.org/docs/latest/rules/no-const-assign
     'no-const-assign': 'error',
 
-    // disallow importing from the same path more than once
+    // Disallow importing from the same path more than once
     // https://eslint.org/docs/rules/no-duplicate-imports
-    // replaced by https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-duplicates.md
+    // Replaced by https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-duplicates.md
     'no-duplicate-imports': 'off',
 
+    // Disallow new operators with global non-constructor functions
+    // https://eslint.org/docs/latest/rules/no-new-native-nonconstructor
+    'no-new-native-nonconstructor': 'error',
+
+    // Disallow returning values from Promise executor functions
+    // https://eslint.org/docs/latest/rules/no-promise-executor-return
+    'no-promise-executor-return': 'error',
+
+    // Disallow invalid regular expression strings in RegExp constructors
+    // https://eslint.org/docs/latest/rules/no-invalid-regexp
+    'no-invalid-regexp': 'error',
+
     // Disallow specified names in exports
     // https://eslint.org/docs/rules/no-restricted-exports
     'no-restricted-exports': [
       'error',
       {
         restrictedNamedExports: [
-          'default', // use `export default` to provide a default export
-          'then' // this will cause tons of confusion when your module is dynamically `import()`ed, and will break in most node ESM versions
+          'default', // Use `export default` to provide a default export
+          'then' // This will cause tons of confusion when your module is dynamically `import()`ed, and will break in most node ESM versions
         ]
       }
     ],
 
-    // disallow to use this/super before super() calling in constructors.
+    // Disallow this/super before calling super() in constructors
     // https://eslint.org/docs/rules/no-this-before-super
     'no-this-before-super': 'error',
 
-    // disallow useless computed property keys
+    // Disallow use of optional chaining in contexts where the undefined value is not allowed
+    // https://eslint.org/docs/latest/rules/no-unsafe-optional-chaining
+    'no-unsafe-optional-chaining': 'error',
+
+    // Disallow control flow statements in finally blocks
+    // https://eslint.org/docs/latest/rules/no-unsafe-finally
+    'no-unsafe-finally': 'error',
+
+    // Disallow useless computed property keys
     // https://eslint.org/docs/rules/no-useless-computed-key
     'no-useless-computed-key': 'error',
 
-    // disallow unnecessary constructor
+    // Disallow unnecessary constructor
     // https://eslint.org/docs/rules/no-useless-constructor
     'no-useless-constructor': 'error',
 
-    // disallow renaming import, export, and destructured assignments to the same name
+    // Disallow renaming import, export, and destructured assignments to the same name
     // https://eslint.org/docs/rules/no-useless-rename
     'no-useless-rename': [
       'error',
@@ -90,10 +115,10 @@ export default {
       }
     ],
 
-    // 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
+    // Require method and property shorthand syntax for object literals
     // https://eslint.org/docs/rules/object-shorthand
     'object-shorthand': [
       'error',
@@ -104,7 +129,7 @@ export default {
       }
     ],
 
-    // suggest using arrow functions as callbacks
+    // Suggest using arrow functions as callbacks
     'prefer-arrow-callback': [
       'error',
       {
@@ -113,7 +138,7 @@ export default {
       }
     ],
 
-    // suggest using of const declaration for variables that are never modified after declared
+    // Suggest using of const declaration for variables that are never modified after declared
     'prefer-const': [
       'error',
       {
@@ -141,31 +166,31 @@ export default {
       }
     ],
 
-    // disallow parseInt() in favor of binary, octal, and hexadecimal literals
+    // Disallow parseInt() in favor of binary, octal, and hexadecimal literals
     // https://eslint.org/docs/rules/prefer-numeric-literals
     'prefer-numeric-literals': 'error',
 
-    // suggest using Reflect methods where applicable
+    // Suggest using Reflect methods where applicable
     // https://eslint.org/docs/rules/prefer-reflect
     'prefer-reflect': 'off',
 
-    // use rest parameters instead of arguments
+    // Use rest parameters instead of arguments
     // https://eslint.org/docs/rules/prefer-rest-params
     'prefer-rest-params': 'error',
 
-    // suggest using the spread syntax instead of .apply()
+    // Suggest using the spread syntax instead of .apply()
     // https://eslint.org/docs/rules/prefer-spread
     'prefer-spread': 'error',
 
-    // suggest using template literals instead of string concatenation
+    // Suggest using template literals instead of string concatenation
     // https://eslint.org/docs/rules/prefer-template
     'prefer-template': 'error',
 
-    // enforce spacing between object rest-spread
+    // Enforce spacing between object rest-spread
     // https://eslint.org/docs/rules/rest-spread-spacing
     'rest-spread-spacing': ['error', 'never'],
 
-    // enforce usage of spacing in template strings
+    // Enforce usage of spacing in template strings
     // https://eslint.org/docs/rules/template-curly-spacing
     'template-curly-spacing': 'error',
 
@@ -175,9 +200,17 @@ export default {
 
     // Disallow async functions which have no await expression
     // https://eslint.org/docs/latest/rules/require-await
-    'require-await': 2,
+    'require-await': 'error',
 
     // Prefer to use async/await for Promises
-    'promise/prefer-await-to-then': 2
+    'promise/prefer-await-to-then': 'error',
+
+    // Disallow using an async function as a Promise executor
+    // https://eslint.org/docs/latest/rules/no-async-promise-executor
+    'no-async-promise-executor': 'error',
+
+    // Disallow template literal placeholder syntax in regular strings
+    // https://eslint.org/docs/latest/rules/no-template-curly-in-string
+    'no-template-curly-in-string': 'error'
   }
 }

package/rules/imports.mjs

@@ -24,7 +24,7 @@ if (tsConfig && tsConfig.config.compilerOptions.paths) {
  * @type {import("eslint").Linter.Config}
  */
 export default {
-  name : 'imports',
+  name: 'imports',
   files: ['**/*.{js,mjs,cjs,jsx}'],
   languageOptions: {
     sourceType: 'module'
@@ -44,29 +44,29 @@ export default {
         groups: [
           [
             // Anything that starts with react
-            // e.g: import { useState } from 'react'
-            // e.g: import { useFela } from 'react-fela'
+            // E.g: import { useState } from 'react'
+            // E.g: import { useFela } from 'react-fela'
             '^react',
             // Anything that starts with next
-            // e.g: import { useRouter } from 'next/router'
+            // E.g: import { useRouter } from 'next/router'
             '^(next|@remix)',
             // Anything that starts with a letter
-            // e.g: import Downshift from 'downshift'
+            // E.g: import Downshift from 'downshift'
             '^[a-z]',
             // Anything that starts with an alias (see jsconfig.json)
-            // e.g: import ListDropdown from '@shared/components/ListDropdown'
+            // E.g: import ListDropdown from '@shared/components/ListDropdown'
             `^(${pathsNames.join('|')})(/.*|$)`,
             // Anything that starts with @
-            // e.g: import { yupResolver } from '@hookform/resolvers/yup'
+            // E.g: import { yupResolver } from '@hookform/resolvers/yup'
             '^@',
             // Anything that starts with a dot
-            // e.g: import { matchIsDate } from './utils/date
+            // E.g: import { matchIsDate } from './utils/date
             '^\\.',
             // Side effect imports from lib
-            // e.g: import 'react-toastify/dist/ReactToastify.css'
+            // E.g: import 'react-toastify/dist/ReactToastify.css'
             '^\\u0000',
             // Side effect import that starts with a dot
-            // e.g: import './setup-config'
+            // E.g: import './setup-config'
             '^\\u0000\\.'
           ]
         ]

package/rules/node.mjs

@@ -13,7 +13,7 @@ export default {
     }
   },
   rules: {
-    // require all requires be top-level
+    // Require all requires be top-level
     // https://eslint.org/docs/rules/global-require
     'global-require': 'error'
   }

package/rules/react-hooks.mjs

@@ -6,7 +6,7 @@ import globals from 'globals'
  * @type {import("eslint").Linter.Config}
  */
 export default {
-  name : 'react-hooks',
+  name: 'react-hooks',
   plugins: {
     'react-hooks': pluginReactHooks
   },
@@ -26,7 +26,7 @@ export default {
     'react-hooks/exhaustive-deps': 'error',
 
     // Prefer to have a convention for naming states
-    // e.g: [thing, setThing]
+    // E.g: [thing, setThing]
     // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/hook-use-state.md
     'react/hook-use-state': 'error'
   }

package/rules/react.mjs

@@ -38,7 +38,7 @@ export default [
       // Forbid the use of Proptypes
       'react/prop-types': 'off',
 
-      // defaultProps is deprecated
+      // DefaultProps is deprecated
       'react/require-default-props': [
         'error',
         {
@@ -196,15 +196,15 @@ export default [
       // https://github.com/jsx-eslint/eslint-plugin-react/blob/ac102885765be5ff37847a871f239c6703e1c7cc/docs/rules/jsx-no-target-blank.md
       'react/jsx-no-target-blank': ['error', { enforceDynamicLinks: 'always' }],
 
-      // only .jsx files may have JSX
+      // Only .jsx files may have JSX
       // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/jsx-filename-extension.md
       'react/jsx-filename-extension': ['error', { extensions: ['.jsx'] }],
 
-      // prevent accidental JS comments from being injected into JSX as text
+      // Prevent accidental JS comments from being injected into JSX as text
       // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/jsx-no-comment-textnodes.md
       'react/jsx-no-comment-textnodes': 'error',
 
-      // disallow using React.render/ReactDOM.render's return value
+      // Disallow using React.render/ReactDOM.render's return value
       // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/no-render-return-value.md
       'react/no-render-return-value': 'error',
 
@@ -331,12 +331,12 @@ export default [
       ],
 
       // This rule is turned off with the new JSX transform
-      // since `eslint-plugin-react` is used.
+      // Since `eslint-plugin-react` is used.
       // Ref: https://reactjs.org/blog/2020/09/22/introducing-the-new-jsx-transform.html#how-to-upgrade-to-the-new-jsx-transform
       'react/jsx-uses-react': 'off',
 
       // This rule is turned off with the new JSX transform
-      // since `eslint-plugin-react` is used.
+      // Since `eslint-plugin-react` is used.
       // Ref: https://reactjs.org/blog/2020/09/22/introducing-the-new-jsx-transform.html#how-to-upgrade-to-the-new-jsx-transform
       'react/react-in-jsx-scope': 'off',
 

package/rules/style.mjs

@@ -5,11 +5,40 @@ export default {
   name: 'style',
   files: ['**/*.{js,mjs,cjs,jsx}'],
   rules: {
+    // Enforce camelcase naming convention
+    // https://eslint.org/docs/latest/rules/camelcase
+    camelcase: 'error',
+
+    // Enforce a maximum number of lines per file
+    // https://eslint.org/docs/rules/max-lines
+    'max-lines': ['error', 350],
+
+    // Enforce or disallow capitalization of the first letter of a comment
+    // https://eslint.org/docs/rules/capitalized-comments
+    'capitalized-comments': ['error', 'always'],
+
+    // "red" === color is a "Yoda" condition, prefer color === "red" instead
+    // https://eslint.org/docs/latest/rules/yoda
+    yoda: 'error',
+
+    // Disallow the unary operators ++ and --
+    // https://eslint.org/docs/latest/rules/no-plusplus
+    'no-plusplus': 'error',
+
+    // Disallow inline comments after code
+    // https://eslint.org/docs/latest/rules/no-inline-comments
+    'no-inline-comments': 'error',
+
+    // Require or disallow an empty line between class members
+    // https://eslint.org/docs/latest/rules/lines-between-class-members
     'lines-between-class-members': ['error', 'always'],
+
+    // Require or disallow padding lines between statements
+    // https://eslint.org/docs/rules/padding-line-between-statements
     'padding-line-between-statements': [
       2,
       // We always want a blank line before a `return` statement or a multi-line
-      // block.
+      // Block.
       {
         blankLine: 'always',
         prev: '*',

package/rules/typescript.mjs

@@ -89,9 +89,6 @@ export default tseslint.config(
       '@typescript-eslint/no-useless-constructor':
         baseES6Rules['no-useless-constructor'],
 
-      // Replace @viclafouch/eslint 'require-await' rule with '@typescript-eslint' version
-      // https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/require-await.md
-      'require-await': 'off',
       '@typescript-eslint/require-await': baseES6Rules['require-await'],
 
       // Replace @viclafouch/eslint 'no-return-await' rule with '@typescript-eslint' version
@@ -152,13 +149,13 @@ export default tseslint.config(
       // '@typescript-eslint/ban-types': [
       //   'error',
       //   {
-      //     types: {
+      //     Types: {
       //       // Omit is not strict enought
       //       Omit: {
       //         // https://twitter.com/erikras/status/1673694889974833152
-      //         message:
+      //         Message:
       //           'Use StrictOmit instead by using reset.d.ts from @viclafouch/eslint-config-viclafouch/reset.d. See https://github.com/viclafouch/eslint-config-viclafouch#better-typing',
-      //         fixWith: 'StrictOmit'
+      //         FixWith: 'StrictOmit'
       //       }
       //     }
       //   }
@@ -176,23 +173,32 @@ export default tseslint.config(
       // Some of the rules also fail in TypeScript files, for example: https://github.com/typescript-eslint/typescript-eslint/issues/662#issuecomment-507081586
       // Rules are inspired by: https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/src/configs/eslint-recommended.ts
       'constructor-super': 'off',
-      'getter-return': 'off',
       'no-const-assign': 'off',
+      // https://eslint.org/docs/latest/rules/no-dupe-args#handled_by_typescript
       'no-dupe-args': 'off',
+      // https://eslint.org/docs/latest/rules/no-dupe-class-members#handled_by_typescript
       'no-dupe-class-members': 'off',
+      // https://eslint.org/docs/latest/rules/no-dupe-keys#handled_by_typescript
       'no-dupe-keys': 'off',
+      // https://eslint.org/docs/latest/rules/no-func-assign#handled_by_typescript
       'no-func-assign': 'off',
+      // https://eslint.org/docs/latest/rules/no-import-assign#handled_by_typescript
       'no-import-assign': 'off',
+      // https://eslint.org/docs/latest/rules/no-new-native-nonconstructor#handled_by_typescript
+      'no-new-native-nonconstructor': 'off',
       'no-new-symbol': 'off',
       'no-obj-calls': 'off',
       'no-redeclare': 'off',
+      // https://eslint.org/docs/latest/rules/no-setter-return#handled_by_typescript
       'no-setter-return': 'off',
+      // https://eslint.org/docs/latest/rules/no-this-before-super#handled_by_typescript
       'no-this-before-super': 'off',
-      // https://github.com/typescript-eslint/typescript-eslint/blob/1cf9243/docs/getting-started/linting/FAQ.md#i-get-errors-from-the-no-undef-rule-about-global-variables-not-being-defined-even-though-there-are-no-typescript-errors
+      // https://eslint.org/docs/latest/rules/no-undef#handled_by_typescript
       'no-undef': 'off',
+      // https://eslint.org/docs/latest/rules/no-unreachable#handled_by_typescript
       'no-unreachable': 'off',
+      // https://eslint.org/docs/latest/rules/no-unsafe-negation#handled_by_typescript
       'no-unsafe-negation': 'off',
-      'valid-typeof': 'off',
       // The following rules are enabled in @viclafouch/eslint config, but are recommended to be disabled within TypeScript projects
       // See: https://github.com/typescript-eslint/typescript-eslint/blob/13583e65f5973da2a7ae8384493c5e00014db51b/docs/linting/TROUBLESHOOTING.md#eslint-plugin-import
       'import/named': 'off',

package/rules/variables.mjs

@@ -5,20 +5,21 @@ export default {
   name: 'variables',
   files: ['**/*.{js,mjs,cjs,jsx}'],
   rules: {
-    // enforce or disallow variable initializations at definition
+    // Enforce or disallow variable initializations at definition
     'init-declarations': 'off',
 
-    // disallow the catch clause parameter name being the same as a variable in the outer scope
+    // Disallow the catch clause parameter name being the same as a variable in the outer scope
     'no-catch-shadow': 'off',
 
-    // disallow deletion of variables
+    // Disallow deletion of variables
+    // https://eslint.org/docs/latest/rules/no-delete-var
     'no-delete-var': 'error',
 
-    // disallow labels that share a name with a variable
+    // Disallow labels that share a name with a variable
     // https://eslint.org/docs/rules/no-label-var
     'no-label-var': 'error',
 
-    // disallow declaration of variables already declared in the outer scope
+    // Disallow declaration of variables already declared in the outer scope
     'no-shadow': 'error',
 
     // For code readability, prevent creating unclear naming
@@ -35,13 +36,13 @@ export default {
     // Ref: https://eslint.org/docs/rules/object-shorthand
     'object-shorthand': [2, 'properties'],
 
-    // disallow shadowing of names such as arguments
+    // Disallow shadowing of names such as arguments
     'no-shadow-restricted-names': 'error',
 
-    // disallow use of undeclared variables unless mentioned in a /*global */ block
-    'no-undef': 'error',
+    // Disallow use of undeclared variables unless mentioned in a /*global */ block
+    'no-undef': ['error', { typeof: true }],
 
-    // disallow use of undefined when initializing variables
+    // Disallow use of undefined when initializing variables
     'no-undef-init': 'error',
 
     // Most common naming that is not always understandable
@@ -62,7 +63,7 @@ export default {
       'str'
     ],
 
-    // disallow declaration of variables that are not used in the code
+    // Disallow declaration of variables that are not used in the code
     'no-unused-vars': [
       'error',
       {
@@ -74,7 +75,7 @@ export default {
       }
     ],
 
-    // disallow use of variables before they are defined
+    // Disallow use of variables before they are defined
     'no-use-before-define': [
       'error',
       { functions: true, classes: true, variables: true }