@viclafouch/eslint-config-viclafouch 4.19.0 → 4.21.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@viclafouch/eslint-config-viclafouch",
-  "version": "4.19.0",
+  "version": "4.21.0",
   "description": "ESLint and Prettier Config from Victor de la Fouchardiere",
   "main": "index.js",
   "keywords": [

package/rules/node.mjs

@@ -9,12 +9,15 @@ export default {
   languageOptions: {
     globals: {
       ...globals.node,
-      ...globals.jest
+      ...globals.vitest
     }
   },
   rules: {
     // Require all requires be top-level
     // https://eslint.org/docs/rules/global-require
-    'global-require': 'error'
+    'global-require': 'error',
+
+    // Enforce usage of the `node:` prefix for builtin imports
+    'node/prefer-node-protocol': 'error'
   }
 }

package/rules/react.mjs

@@ -408,6 +408,10 @@ export default [
   {
     // Enable eslint-plugin-testing-library rules or preset only for matching files!
     files: ['**/__tests__/**/*.[jt]s?(x)', '**/?(*.)+(spec|test).[jt]s?(x)'],
-    ...testingLibrary.configs['flat/react']
+    ...testingLibrary.configs['flat/react'],
+    rules: {
+      ...testingLibrary.configs['flat/react'].rules,
+      'testing-library/render-result-naming-convention': 0
+    }
   }
 ]

package/rules/typescript.mjs

@@ -107,8 +107,14 @@ export default tseslint.config(
       '@typescript-eslint/naming-convention': [
         'error',
         {
-          selector: 'typeLike',
-          format: ['PascalCase']
+          selector: 'typeParameter',
+          format: ['PascalCase'],
+          leadingUnderscore: 'forbid',
+          trailingUnderscore: 'forbid',
+          custom: {
+            regex: '^(T|T[A-Z][A-Za-z]+)$',
+            match: true
+          }
         }
       ],
 
@@ -146,7 +152,29 @@ export default tseslint.config(
 
       // Disallow using the spread operator when it might cause unexpected behavior.
       // https://typescript-eslint.io/rules/no-misused-spread/
-      '@typescript-eslint/no-misused-spread': 'error'
+      '@typescript-eslint/no-misused-spread': 'error',
+
+      // Enforce import type { T }
+      // https://typescript-eslint.io/rules/consistent-type-imports
+      '@typescript-eslint/consistent-type-imports': [
+        'error',
+        { prefer: 'type-imports' }
+      ],
+
+      // Don't over-define types for simple things like strings
+      // https://typescript-eslint.io/rules/no-inferrable-types
+      '@typescript-eslint/no-inferrable-types': [
+        'error',
+        { ignoreParameters: true }
+      ],
+
+      // Disallow TypeScript namespaces
+      // https://typescript-eslint.io/rules/no-namespace
+      '@typescript-eslint/no-namespace': 'error',
+
+      // Disallow non-null assertions after an optional chain expression
+      // https://typescript-eslint.io/rules/no-non-null-asserted-optional-chain
+      '@typescript-eslint/no-non-null-asserted-optional-chain': 'error'
 
       // Prefer using nullish coalescing (??) over logical (||) when possible.
       // '@typescript-eslint/prefer-nullish-coalescing': 'error'