@lenne.tech/eslint-config-ts 2.1.3 → 2.2.0

package/index.js

@@ -2,7 +2,7 @@ console.log("Linting TypeScript...");
 import tsParser from "@typescript-eslint/parser";
 import typescript from "@typescript-eslint/eslint-plugin";
 import perfectionist from "eslint-plugin-perfectionist";
-import importPlugin from "eslint-plugin-import";
+import importPlugin from "eslint-plugin-import-x";
 import unusedImportPlugin from "eslint-plugin-unused-imports";
 import yml from "eslint-plugin-yml";
 import stylistic from "@stylistic/eslint-plugin";
@@ -10,6 +10,26 @@ import markdown from "eslint-plugin-markdown";
 import jsonc from "eslint-plugin-jsonc";
 import eslintcomments from "eslint-plugin-eslint-comments";
 import noonlytests from "eslint-plugin-no-only-tests";
+import eslintConfigPrettier from 'eslint-config-prettier';
+
+/**
+ * Re-key a shipped config's rules from the `import-x/` prefix to `import/`.
+ *
+ * We moved from `eslint-plugin-import` to its maintained fork
+ * `eslint-plugin-import-x` (the original pins minimatch ^3, which is stuck on a
+ * brace-expansion line that upstream no longer patches — GHSA-mh99-v99m-4gvg —
+ * and its peer range has no eslint 10 entry at all).
+ *
+ * The fork registers its own rules under `import-x/`, but a rule's public name
+ * comes from the key it is registered under in `plugins`, not from the package.
+ * So we keep registering it as `import` and only re-key the rule objects the
+ * plugin ships. That keeps every consumer's `import/...` rule override AND every
+ * `// eslint-disable-next-line import/...` comment across all lt projects
+ * working — renaming them would silently stop suppressing, which is a far worse
+ * outcome than the dependency swap is worth.
+ */
+const asImportPrefix = (rules) =>
+  Object.fromEntries(Object.entries(rules).map(([key, value]) => [key.replace(/^import-x\//, "import/"), value]));
 
 export default [
   { ignores: ["dist/", ".yalc/", "node_modules/", ".husky/", "extras/"] },
@@ -35,8 +55,8 @@ export default [
       ...perfectionist.configs["recommended-natural"].rules,
       ...typescript.configs["eslint-recommended"].rules,
       ...typescript.configs["recommended"].rules,
-      ...importPlugin.configs["typescript"].rules,
-      ...importPlugin.configs["recommended"].rules,
+      ...asImportPrefix(importPlugin.configs["typescript"].rules),
+      ...asImportPrefix(importPlugin.configs["recommended"].rules),
       ...eslintcomments.configs["recommended"].rules,
       ...jsonc.configs["recommended-with-jsonc"].rules,
       ...yml.configs["standard"].rules,
@@ -70,6 +90,10 @@ export default [
       "@typescript-eslint/consistent-type-definitions": ["error", "interface"],
       "@typescript-eslint/prefer-ts-expect-error": "error",
       "@typescript-eslint/no-require-imports": "off",
+      "@typescript-eslint/no-unused-expressions": [
+        "warn",
+        { allowShortCircuit: true, allowTernary: true },
+      ],
       // Override JS
       "no-useless-constructor": "off",
       "no-invalid-this": "off",
@@ -118,6 +142,7 @@ export default [
         "always",
         { exceptAfterSingleLine: true },
       ],
+      "@stylistic/indent": ["error", 2],
 
       // off
       "@typescript-eslint/consistent-indexed-object-style": "off",
@@ -157,10 +182,7 @@ export default [
       "unused-imports/no-unused-vars": [
         "warn",
         {
-          vars: "all",
-          varsIgnorePattern: "^_",
-          args: "after-used",
-          argsIgnorePattern: "\\b(type|of|returns)\\b", // Arguments containing the words "type", "of", or "returns" will be ignored --> nest-server
+          caughtErrors: "none",
         },
       ],
 
@@ -291,4 +313,6 @@ export default [
       "yml/no-empty-document": "off",
     },
   },
+  // Prettier config must be last to override conflicting rules
+  eslintConfigPrettier,
 ];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lenne.tech/eslint-config-ts",
-  "version": "2.1.3",
+  "version": "2.2.0",
   "description": "ESLint config of lenne.Tech for TypeScript",
   "license": "MIT",
   "keywords": [
@@ -19,23 +19,22 @@
     "eslint": ">=9.26.0"
   },
   "dependencies": {
-    "@stylistic/eslint-plugin": "5.2.2",
-    "@typescript-eslint/eslint-plugin": "8.38.0",
-    "@typescript-eslint/parser": "8.38.0",
+    "@stylistic/eslint-plugin": "5.4.0",
+    "@typescript-eslint/eslint-plugin": "8.65.0",
+    "@typescript-eslint/parser": "8.65.0",
     "eslint-config-prettier": "10.1.8",
+    "eslint-import-resolver-node": "0.3.9",
     "eslint-plugin-eslint-comments": "3.2.0",
-    "eslint-plugin-import": "2.32.0",
-    "eslint-plugin-jsonc": "2.20.1",
+    "eslint-plugin-import-x": "4.17.1",
+    "eslint-plugin-jsonc": "2.21.0",
     "eslint-plugin-markdown": "5.1.0",
     "eslint-plugin-no-only-tests": "3.3.0",
-    "eslint-plugin-perfectionist": "4.15.0",
-    "eslint-plugin-prettier": "5.5.3",
-    "eslint-plugin-unused-imports": "4.1.4",
-    "eslint-plugin-yml": "1.18.0",
-    "functions-have-names": "1.2.3"
+    "eslint-plugin-perfectionist": "4.15.1",
+    "eslint-plugin-unused-imports": "4.2.0",
+    "eslint-plugin-yml": "1.19.0"
   },
   "devDependencies": {
-    "eslint": "9.31.0",
+    "eslint": "9.37.0",
     "watch": "1.0.2"
   }
 }