@luxass/eslint-config 4.5.0 → 4.7.0

package/dist/index.js

@@ -424,172 +424,6 @@ function sortTsconfig() {
   ];
 }
 
-// src/utils.ts
-import process from "node:process";
-import { isPackageExists } from "local-pkg";
-var parserPlain = {
-  meta: {
-    name: "parser-plain"
-  },
-  parseForESLint: (code) => ({
-    ast: {
-      body: [],
-      comments: [],
-      loc: { end: code.length, start: 0 },
-      range: [0, code.length],
-      tokens: [],
-      type: "Program"
-    },
-    scopeManager: null,
-    services: { isPlain: true },
-    visitorKeys: {
-      Program: []
-    }
-  })
-};
-async function combine(...configs2) {
-  const resolved = await Promise.all(configs2);
-  return resolved.flat();
-}
-function renameRules(rules, from, to) {
-  return Object.fromEntries(
-    Object.entries(rules).map(([key, value]) => {
-      if (key.startsWith(from)) {
-        return [to + key.slice(from.length), value];
-      }
-      return [key, value];
-    })
-  );
-}
-function toArray(value) {
-  return Array.isArray(value) ? value : [value];
-}
-async function interop(m) {
-  const resolved = await m;
-  return resolved.default || resolved;
-}
-async function ensure(packages) {
-  if (process.env.CI || process.stdout.isTTY === false) {
-    return;
-  }
-  ;
-  const nonExistingPackages = packages.filter((i) => i && !isPackageExists(i));
-  if (nonExistingPackages.length === 0) {
-    return;
-  }
-  const p = await import("@clack/prompts");
-  const result = await p.confirm({
-    message: `${nonExistingPackages.length === 1 ? "Package is" : "Packages are"} required for this config: ${nonExistingPackages.join(", ")}. Do you want to install them?`
-  });
-  if (result) {
-    await import("@antfu/install-pkg").then((i) => i.installPackage(nonExistingPackages, { dev: true }));
-  }
-}
-function resolveSubOptions(options, key) {
-  return typeof options[key] === "boolean" ? {} : options[key] || {};
-}
-function getOverrides(options, key) {
-  const sub = resolveSubOptions(options, key);
-  return {
-    ..."overrides" in sub ? sub.overrides : {}
-  };
-}
-
-// src/configs/svelte.ts
-async function svelte(options = {}) {
-  const {
-    files = [GLOB_SVELTE],
-    overrides = {},
-    stylistic: stylistic2 = true
-  } = options;
-  const {
-    indent = 2,
-    quotes = "single"
-  } = typeof stylistic2 === "boolean" ? {} : stylistic2;
-  await ensure([
-    "eslint-plugin-svelte"
-  ]);
-  const [
-    pluginSvelte,
-    parserSvelte
-  ] = await Promise.all([
-    interop(import("eslint-plugin-svelte")),
-    interop(import("svelte-eslint-parser"))
-  ]);
-  return [
-    {
-      name: "antfu/svelte/setup",
-      plugins: {
-        svelte: pluginSvelte
-      }
-    },
-    {
-      files,
-      languageOptions: {
-        parser: parserSvelte,
-        parserOptions: {
-          extraFileExtensions: [".svelte"],
-          parser: options.typescript ? await interop(import("@typescript-eslint/parser")) : null
-        }
-      },
-      name: "antfu/svelte/rules",
-      processor: pluginSvelte.processors[".svelte"],
-      rules: {
-        "import/no-mutable-exports": "off",
-        "no-undef": "off",
-        // incompatible with most recent (attribute-form) generic types RFC
-        "no-unused-vars": ["error", {
-          args: "none",
-          caughtErrors: "none",
-          ignoreRestSiblings: true,
-          vars: "all",
-          varsIgnorePattern: "^(\\$\\$Props$|\\$\\$Events$|\\$\\$Slots$)"
-        }],
-        "svelte/comment-directive": "error",
-        "svelte/no-at-debug-tags": "warn",
-        "svelte/no-at-html-tags": "error",
-        "svelte/no-dupe-else-if-blocks": "error",
-        "svelte/no-dupe-style-properties": "error",
-        "svelte/no-dupe-use-directives": "error",
-        "svelte/no-dynamic-slot-name": "error",
-        "svelte/no-export-load-in-svelte-module-in-kit-pages": "error",
-        "svelte/no-inner-declarations": "error",
-        "svelte/no-not-function-handler": "error",
-        "svelte/no-object-in-text-mustaches": "error",
-        "svelte/no-reactive-functions": "error",
-        "svelte/no-reactive-literals": "error",
-        "svelte/no-shorthand-style-property-overrides": "error",
-        "svelte/no-unknown-style-directive-property": "error",
-        "svelte/no-unused-svelte-ignore": "error",
-        "svelte/no-useless-mustaches": "error",
-        "svelte/require-store-callbacks-use-set-param": "error",
-        "svelte/system": "error",
-        "svelte/valid-compile": "error",
-        "svelte/valid-each-key": "error",
-        "unused-imports/no-unused-vars": [
-          "error",
-          { args: "after-used", argsIgnorePattern: "^_", vars: "all", varsIgnorePattern: "^(_|\\$\\$Props$|\\$\\$Events$|\\$\\$Slots$)" }
-        ],
-        ...stylistic2 ? {
-          "style/indent": "off",
-          // superseded by svelte/indent
-          "style/no-trailing-spaces": "off",
-          // superseded by svelte/no-trailing-spaces
-          "svelte/derived-has-same-inputs-outputs": "error",
-          "svelte/html-closing-bracket-spacing": "error",
-          "svelte/html-quotes": ["error", { prefer: quotes }],
-          "svelte/indent": ["error", { alignAttributesVertically: true, indent }],
-          "svelte/mustache-spacing": "error",
-          "svelte/no-spaces-around-equal-signs-in-attribute": "error",
-          "svelte/no-trailing-spaces": "error",
-          "svelte/spaced-html-comment": "error"
-        } : {},
-        ...overrides
-      }
-    }
-  ];
-}
-
 // src/configs/imports.ts
 import pluginImport from "eslint-plugin-import-x";
 import pluginAntfu from "eslint-plugin-antfu";
@@ -889,6 +723,77 @@ async function javascript(options = {}) {
   ];
 }
 
+// src/utils.ts
+import process from "node:process";
+import { isPackageExists } from "local-pkg";
+var parserPlain = {
+  meta: {
+    name: "parser-plain"
+  },
+  parseForESLint: (code) => ({
+    ast: {
+      body: [],
+      comments: [],
+      loc: { end: code.length, start: 0 },
+      range: [0, code.length],
+      tokens: [],
+      type: "Program"
+    },
+    scopeManager: null,
+    services: { isPlain: true },
+    visitorKeys: {
+      Program: []
+    }
+  })
+};
+async function combine(...configs2) {
+  const resolved = await Promise.all(configs2);
+  return resolved.flat();
+}
+function renameRules(rules, from, to) {
+  return Object.fromEntries(
+    Object.entries(rules).map(([key, value]) => {
+      if (key.startsWith(from)) {
+        return [to + key.slice(from.length), value];
+      }
+      return [key, value];
+    })
+  );
+}
+function toArray(value) {
+  return Array.isArray(value) ? value : [value];
+}
+async function interop(m) {
+  const resolved = await m;
+  return resolved.default || resolved;
+}
+async function ensure(packages) {
+  if (process.env.CI || process.stdout.isTTY === false) {
+    return;
+  }
+  ;
+  const nonExistingPackages = packages.filter((i) => i && !isPackageExists(i));
+  if (nonExistingPackages.length === 0) {
+    return;
+  }
+  const p = await import("@clack/prompts");
+  const result = await p.confirm({
+    message: `${nonExistingPackages.length === 1 ? "Package is" : "Packages are"} required for this config: ${nonExistingPackages.join(", ")}. Do you want to install them?`
+  });
+  if (result) {
+    await import("@antfu/install-pkg").then((i) => i.installPackage(nonExistingPackages, { dev: true }));
+  }
+}
+function resolveSubOptions(options, key) {
+  return typeof options[key] === "boolean" ? {} : options[key] || {};
+}
+function getOverrides(options, key) {
+  const sub = resolveSubOptions(options, key);
+  return {
+    ..."overrides" in sub ? sub.overrides : {}
+  };
+}
+
 // src/configs/jsdoc.ts
 async function jsdoc(options = {}) {
   const {
@@ -1104,13 +1009,13 @@ import pluginAntfu3 from "eslint-plugin-antfu";
 var StylisticConfigDefaults = {
   indent: 2,
   jsx: true,
-  quotes: "single",
-  semi: false
+  quotes: "double",
+  semi: true
 };
 async function stylistic(options = {}) {
   const {
     indent,
-    jsx,
+    jsx: jsx2,
     overrides = {},
     quotes,
     semi
@@ -1122,7 +1027,7 @@ async function stylistic(options = {}) {
   const config = pluginStylistic.configs.customize({
     flat: true,
     indent,
-    jsx,
+    jsx: jsx2,
     pluginName: "style",
     quotes,
     semi
@@ -1558,7 +1463,7 @@ async function yaml(options = {}) {
   ]);
   const {
     indent = 2,
-    quotes = "single"
+    quotes = "double"
   } = typeof stylistic2 === "boolean" ? {} : stylistic2;
   return [
     {
@@ -1852,7 +1757,6 @@ async function astro(options = {}) {
   const {
     files = [GLOB_ASTRO],
     overrides = {},
-    typescript: typescript2 = true,
     stylistic: stylistic2 = true
   } = options;
   await ensure([
@@ -1879,63 +1783,35 @@ async function astro(options = {}) {
       name: "luxass/astro/rules",
       files,
       languageOptions: {
+        globals: pluginAstro.environments.astro.globals,
         parser: parserAstro,
         parserOptions: {
           extraFileExtensions: [".astro"],
-          parser: typescript2 ? parserTs : null,
-          sourceType: "module"
-        }
+          parser: parserTs
+        },
+        sourceType: "module"
       },
+      processor: "astro/client-side-ts",
       rules: {
-        // Disallow conflicting set directives and child contents
-        // https://ota-meshi.github.io/eslint-plugin-astro/rules/no-conflict-set-directives/
+        "astro/missing-client-only-directive-value": "error",
         "astro/no-conflict-set-directives": "error",
-        // Disallow use of `set:html` directive
-        // https://ota-meshi.github.io/eslint-plugin-astro/rules/no-set-html-directive/
+        "astro/no-deprecated-astro-canonicalurl": "error",
+        "astro/no-deprecated-astro-fetchcontent": "error",
+        "astro/no-deprecated-astro-resolve": "error",
+        "astro/no-deprecated-getentrybyslug": "error",
         "astro/no-set-html-directive": "off",
+        "astro/no-unused-define-vars-in-style": "error",
+        "astro/semi": "error",
+        "astro/valid-compile": "error",
         ...stylistic2 ? {
           "style/indent": "off",
-          "style/jsx-indent": "off",
           "style/jsx-closing-tag-location": "off",
+          "style/jsx-indent": "off",
           "style/jsx-one-expression-per-line": "off",
           "style/no-multiple-empty-lines": "off"
         } : {},
         ...overrides
       }
-    },
-    {
-      name: "luxass/astro/scripts-js",
-      files: [
-        "**/*.astro/*.js",
-        "*.astro/*.js"
-      ],
-      languageOptions: {
-        globals: {
-          browser: true,
-          es2020: true
-        },
-        parserOptions: {
-          sourceType: "module"
-        }
-      }
-    },
-    {
-      name: "luxass/astro/scripts-ts",
-      files: [
-        "**/*.astro/*.ts",
-        "*.astro/*.ts"
-      ],
-      languageOptions: {
-        globals: {
-          browser: true,
-          es2020: true
-        },
-        parser: typescript2 ? parserTs : null,
-        parserOptions: {
-          project: null,
-          sourceType: "module"
-        }
-      }
     }
   ];
 }
@@ -2239,83 +2115,6 @@ async function toml(options = {}) {
   ];
 }
 
-// src/configs/solid.ts
-async function solid(options = {}) {
-  const {
-    overrides = {},
-    typescript: typescript2 = true,
-    files = [GLOB_JSX, GLOB_TSX]
-  } = options;
-  await ensure([
-    "eslint-plugin-solid"
-  ]);
-  const tsconfigPath = options?.tsconfigPath ? toArray(options.tsconfigPath) : void 0;
-  const isTypeAware = !!tsconfigPath;
-  const [
-    pluginSolid,
-    parserTs
-  ] = await Promise.all([
-    interop(import("eslint-plugin-solid")),
-    interop(import("@typescript-eslint/parser"))
-  ]);
-  return [
-    {
-      name: "luxass/solid/setup",
-      plugins: {
-        solid: pluginSolid
-      }
-    },
-    {
-      name: "luxass/solid/rules",
-      files,
-      languageOptions: {
-        parser: parserTs,
-        parserOptions: {
-          ecmaFeatures: {
-            jsx: true
-          },
-          ...isTypeAware ? { project: tsconfigPath } : {}
-        },
-        sourceType: "module"
-      },
-      rules: {
-        // reactivity
-        "solid/components-return-once": "warn",
-        "solid/event-handlers": ["error", {
-          // if true, don't warn on ambiguously named event handlers like `onclick` or `onchange`
-          ignoreCase: false,
-          // if true, warn when spreading event handlers onto JSX. Enable for Solid < v1.6.
-          warnOnSpread: false
-        }],
-        // these rules are mostly style suggestions
-        "solid/imports": "error",
-        // identifier usage is important
-        "solid/jsx-no-duplicate-props": "error",
-        "solid/jsx-no-script-url": "error",
-        "solid/jsx-no-undef": "error",
-        "solid/jsx-uses-vars": "error",
-        "solid/no-destructure": "error",
-        // security problems
-        "solid/no-innerhtml": ["error", { allowStatic: true }],
-        "solid/no-react-deps": "error",
-        "solid/no-react-specific-props": "error",
-        "solid/no-unknown-namespaces": "error",
-        "solid/prefer-for": "error",
-        "solid/reactivity": "warn",
-        "solid/self-closing-comp": "error",
-        "solid/style-prop": ["error", { styleProps: ["style", "css"] }],
-        ...typescript2 ? {
-          "solid/jsx-no-undef": ["error", { typescriptEnabled: true }],
-          // namespaces taken care of by TS
-          "solid/no-unknown-namespaces": "off"
-        } : {},
-        // overrides
-        ...overrides
-      }
-    }
-  ];
-}
-
 // src/configs/regexp.ts
 import { configs } from "eslint-plugin-regexp";
 async function regexp(options = {}) {
@@ -2342,6 +2141,23 @@ async function regexp(options = {}) {
   ];
 }
 
+// src/configs/jsx.ts
+async function jsx() {
+  return [
+    {
+      name: "luxass/jsx/setup",
+      files: [GLOB_JSX, GLOB_TSX],
+      languageOptions: {
+        parserOptions: {
+          ecmaFeatures: {
+            jsx: true
+          }
+        }
+      }
+    }
+  ];
+}
+
 // src/factory.ts
 var FLAT_CONFIG_PROPS = [
   "name",
@@ -2376,21 +2192,20 @@ function luxass(options = {}, ...userConfigs) {
   const {
     astro: enableAstro = false,
     autoRenamePlugins = true,
-    editor = !!((process3.env.VSCODE_PID || process3.env.JETBRAINS_IDE || process3.env.VIM) && !process3.env.CI),
     exts = [],
     gitignore: enableGitignore = true,
+    editor = !!((process3.env.VSCODE_PID || process3.env.JETBRAINS_IDE || process3.env.VIM) && !process3.env.CI),
+    jsx: enableJsx = true,
     react: enableReact = false,
-    tailwindcss: enableTailwindCSS = false,
-    svelte: enableSvelte = false,
-    solid: enableSolid = false,
+    regexp: enableRegexp = true,
     typescript: enableTypeScript = isPackageExists4("typescript"),
     unocss: enableUnoCSS = false,
-    regexp: enableRegexp = true,
+    tailwindcss: enableTailwindCSS = false,
     vue: enableVue = VuePackages.some((i) => isPackageExists4(i))
   } = options;
   const stylisticOptions = options.stylistic === false ? false : typeof options.stylistic === "object" ? options.stylistic : {};
   if (stylisticOptions && !("jsx" in stylisticOptions)) {
-    stylisticOptions.jsx = options.jsx ?? true;
+    stylisticOptions.jsx = enableJsx;
   }
   const configs2 = [];
   if (enableGitignore) {
@@ -2402,6 +2217,8 @@ function luxass(options = {}, ...userConfigs) {
       }
     }
   }
+  const typescriptOptions = resolveSubOptions(options, "typescript");
+  const tsconfigPath = "tsconfigPath" in typescriptOptions ? typescriptOptions.tsconfigPath : void 0;
   configs2.push(
     ignores(),
     javascript({
@@ -2421,9 +2238,12 @@ function luxass(options = {}, ...userConfigs) {
   if (enableVue) {
     exts.push("vue");
   }
+  if (enableJsx) {
+    configs2.push(jsx());
+  }
   if (enableTypeScript) {
     configs2.push(typescript({
-      ...resolveSubOptions(options, "typescript"),
+      ...typescriptOptions,
       exts,
       overrides: getOverrides(options, "typescript")
     }));
@@ -2450,28 +2270,9 @@ function luxass(options = {}, ...userConfigs) {
     configs2.push(react({
       ...resolveSubOptions(options, "react"),
       overrides: getOverrides(options, "react"),
-      tsconfigPath: getOverrides(options, "typescript").tsconfigPath
+      tsconfigPath
     }));
   }
-  if (enableSolid) {
-    configs2.push(
-      solid({
-        ...resolveSubOptions(options, "solid"),
-        overrides: getOverrides(options, "solid"),
-        typescript: !!enableTypeScript
-      })
-    );
-  }
-  if (enableSvelte) {
-    configs2.push(
-      svelte({
-        ...resolveSubOptions(options, "svelte"),
-        overrides: getOverrides(options, "svelte"),
-        stylistic: stylisticOptions,
-        typescript: !!enableTypeScript
-      })
-    );
-  }
   if (enableVue) {
     configs2.push(
       vue({
@@ -2486,8 +2287,7 @@ function luxass(options = {}, ...userConfigs) {
     configs2.push(
       astro({
         ...resolveSubOptions(options, "astro"),
-        overrides: getOverrides(options, "astro"),
-        typescript: !!enableTypeScript
+        overrides: getOverrides(options, "astro")
       })
     );
   }
@@ -2603,6 +2403,7 @@ export {
   javascript,
   jsdoc,
   jsonc,
+  jsx,
   luxass,
   markdown,
   node,
@@ -2611,11 +2412,9 @@ export {
   regexp,
   renameRules,
   resolveSubOptions,
-  solid,
   sortPackageJson,
   sortTsconfig,
   stylistic,
-  svelte,
   tailwindcss,
   test,
   toArray,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@luxass/eslint-config",
-  "version": "4.5.0",
+  "version": "4.7.0",
   "description": "ESLint config for @luxass",
   "type": "module",
   "author": {
@@ -8,7 +8,7 @@
     "email": "lucasnrgaard@gmail.com",
     "url": "https://luxass.dev"
   },
-  "packageManager": "pnpm@9.1.3",
+  "packageManager": "pnpm@9.2.0",
   "license": "MIT",
   "repository": {
     "type": "git",
@@ -56,11 +56,8 @@
     "eslint-plugin-format": ">=0.1.0",
     "eslint-plugin-react-hooks": "^4.6.0",
     "eslint-plugin-react-refresh": "^0.4.4",
-    "eslint-plugin-solid": "^0.14.0",
-    "eslint-plugin-svelte": ">=2.37.0",
     "eslint-plugin-tailwindcss": "^3.17.0",
-    "prettier-plugin-astro": "^0.14.0",
-    "svelte-eslint-parser": "^0.36.0"
+    "prettier-plugin-astro": "^0.14.0"
   },
   "peerDependenciesMeta": {
     "@unocss/eslint-plugin": {
@@ -72,9 +69,6 @@
     "eslint-plugin-astro": {
       "optional": true
     },
-    "eslint-plugin-solid": {
-      "optional": true
-    },
     "eslint-plugin-format": {
       "optional": true
     },
@@ -92,12 +86,6 @@
     },
     "eslint-plugin-tailwindcss": {
       "optional": true
-    },
-    "svelte-eslint-parser": {
-      "optional": true
-    },
-    "eslint-plugin-svelte": {
-      "optional": true
     }
   },
   "dependencies": {
@@ -105,17 +93,17 @@
     "@clack/prompts": "^0.7.0",
     "@eslint-community/eslint-plugin-eslint-comments": "^4.3.0",
     "@stylistic/eslint-plugin": "^2.1.0",
-    "@typescript-eslint/eslint-plugin": "^7.11.0",
-    "@typescript-eslint/parser": "^7.11.0",
+    "@typescript-eslint/eslint-plugin": "^7.12.0",
+    "@typescript-eslint/parser": "^7.12.0",
     "eslint-config-flat-gitignore": "^0.1.5",
     "eslint-flat-config-utils": "^0.2.5",
     "eslint-merge-processors": "^0.1.0",
     "eslint-plugin-antfu": "^2.3.3",
     "eslint-plugin-import-x": "^0.5.1",
-    "eslint-plugin-jsdoc": "^48.2.7",
+    "eslint-plugin-jsdoc": "^48.2.9",
     "eslint-plugin-jsonc": "^2.16.0",
     "eslint-plugin-markdown": "^5.0.0",
-    "eslint-plugin-n": "^17.7.0",
+    "eslint-plugin-n": "^17.8.1",
     "eslint-plugin-regexp": "^2.6.0",
     "eslint-plugin-toml": "^0.11.0",
     "eslint-plugin-unicorn": "^53.0.0",
@@ -124,7 +112,7 @@
     "eslint-plugin-vue": "^9.26.0",
     "eslint-plugin-yml": "^1.14.0",
     "eslint-processor-vue-blocks": "^0.1.2",
-    "globals": "^15.3.0",
+    "globals": "^15.4.0",
     "jsonc-eslint-parser": "^2.4.0",
     "local-pkg": "^0.5.0",
     "parse-gitignore": "^2.0.0",
@@ -133,31 +121,27 @@
     "yaml-eslint-parser": "^1.2.3"
   },
   "devDependencies": {
-    "@eslint-react/eslint-plugin": "^1.5.14",
+    "@eslint-react/eslint-plugin": "^1.5.15",
     "@eslint/config-inspector": "^0.4.10",
     "@stylistic/eslint-plugin-migrate": "^2.1.0",
     "@types/eslint": "^8.56.10",
     "@types/estree": "^1.0.5",
     "@types/node": "^20.12.7",
-    "@typescript-eslint/rule-tester": "^7.11.0",
-    "@unocss/eslint-plugin": "^0.60.0",
+    "@typescript-eslint/rule-tester": "^7.12.0",
+    "@unocss/eslint-plugin": "^0.60.4",
     "astro-eslint-parser": "^1.0.2",
     "eslint": "npm:eslint-ts-patch@9.2.0-6",
     "eslint-plugin-astro": "^1.2.0",
     "eslint-plugin-format": "^0.1.1",
     "eslint-plugin-react-hooks": "^4.6.2",
     "eslint-plugin-react-refresh": "^0.4.7",
-    "eslint-plugin-solid": "^0.14.0",
-    "eslint-plugin-svelte": "^2.39.0",
-    "eslint-plugin-tailwindcss": "^3.17.0",
+    "eslint-plugin-tailwindcss": "^3.17.3",
     "eslint-typegen": "^0.2.4",
     "lint-staged": "^15.2.5",
     "prettier-plugin-astro": "^0.14.0",
     "simple-git-hooks": "^2.11.1",
-    "svelte": "^4.2.17",
-    "svelte-eslint-parser": "^0.36.0",
-    "tsup": "^8.0.2",
-    "tsx": "^4.11.0",
+    "tsup": "^8.1.0",
+    "tsx": "^4.14.0",
     "typescript": "^5.4.5",
     "vitest": "^1.6.0",
     "vue": "^3.4.27"