@viclafouch/eslint-config-viclafouch 4.20.0 → 4.21.1-beta.0

package/index.mjs

@@ -1,3 +1,5 @@
+import eslintPluginUnicorn from 'eslint-plugin-unicorn'
+import globals from 'globals'
 import bestPracticesConfig from './rules/best-practices.mjs'
 import errorConfig from './rules/errors.mjs'
 import es6Config from './rules/es6.mjs'
@@ -16,6 +18,14 @@ export default [
       reportUnusedDisableDirectives: 'error'
     }
   },
+  {
+    languageOptions: {
+      globals: globals.builtin
+    },
+    plugins: {
+      unicorn: eslintPluginUnicorn
+    }
+  },
   bestPracticesConfig,
   nodeConfig,
   errorConfig,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@viclafouch/eslint-config-viclafouch",
-  "version": "4.20.0",
+  "version": "4.21.1-beta.0",
   "description": "ESLint and Prettier Config from Victor de la Fouchardiere",
   "main": "index.js",
   "keywords": [
@@ -32,6 +32,7 @@
   "dependencies": {
     "@babel/core": "^7.26.9",
     "@babel/eslint-parser": "^7.26.8",
+    "@eslint/js": "^9.21.0",
     "@next/eslint-plugin-next": "^15.2.1",
     "@total-typescript/ts-reset": "^0.6.1",
     "app-root-path": "^3.1.0",
@@ -48,11 +49,10 @@
     "get-tsconfig": "^4.10.0",
     "globals": "^16.0.0",
     "prettier": ">= 3",
-    "typescript-eslint": "^8.26.0",
-    "@eslint/js": "^9.21.0",
     "prettier-plugin-curly": "^0.3.1",
-    "eslint": ">= 9",
-    "typescript": ">= 5"
+    "typescript": ">= 5",
+    "typescript-eslint": "^8.26.0",
+    "eslint": ">= 9"
   },
   "scripts": {
     "lint": "eslint .",
@@ -63,7 +63,8 @@
     "publish:beta": "npm publish --tag beta"
   },
   "devDependencies": {
-    "eslint": "^9.17.0",
+    "eslint": "^9.24.0",
+    "eslint-plugin-unicorn": "^58.0.0",
     "typescript": "^5.7.2"
   }
 }

package/reset.d.ts

@@ -1,6 +1 @@
 export * from '@total-typescript/ts-reset'
-declare global {
-  // https://twitter.com/erikras/status/1673694889974833152
-  // eslint-disable-next-line @typescript-eslint/ban-types
-  type StrictOmit<T, K extends keyof T> = Omit<T, K>
-}

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
+    'unicorn/prefer-node-protocol': 'error'
   }
 }

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'