@luxass/eslint-config 4.12.0 → 4.13.0

package/dist/index.cjs

@@ -102,7 +102,7 @@ __export(src_exports, {
 });
 module.exports = __toCommonJS(src_exports);
 
-// node_modules/.pnpm/tsup@8.2.4_jiti@1.21.6_postcss@8.4.44_tsx@4.19.0_typescript@5.6.2_yaml@2.5.0/node_modules/tsup/assets/cjs_shims.js
+// node_modules/.pnpm/tsup@8.3.5_jiti@2.4.0_postcss@8.4.47_tsx@4.19.2_typescript@5.7.2_yaml@2.6.0/node_modules/tsup/assets/cjs_shims.js
 var getImportMetaUrl = () => typeof document === "undefined" ? new URL(`file:${__filename}`).href : document.currentScript && document.currentScript.src || new URL("main.js", document.baseURI).href;
 var importMetaUrl = /* @__PURE__ */ getImportMetaUrl();
 
@@ -345,6 +345,7 @@ async function astro(options = {}) {
       },
       processor: "astro/client-side-ts",
       rules: {
+        // use recommended rules
         "astro/missing-client-only-directive-value": "error",
         "astro/no-conflict-set-directives": "error",
         "astro/no-deprecated-astro-canonicalurl": "error",
@@ -353,12 +354,11 @@ async function astro(options = {}) {
         "astro/no-deprecated-getentrybyslug": "error",
         "astro/no-set-html-directive": "off",
         "astro/no-unused-define-vars-in-style": "error",
-        "astro/semi": "error",
+        "astro/semi": "off",
         "astro/valid-compile": "error",
         ...stylistic2 ? {
           "style/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"
         } : {},
@@ -384,9 +384,8 @@ async function comments() {
         "eslint-comments/no-duplicate-disable": "error",
         // https://github.com/eslint-community/eslint-plugin-eslint-comments/blob/main/docs/rules/no-unlimited-disable.md
         "eslint-comments/no-unlimited-disable": "error",
-        // Deprecated in favor of official reportUnusedDisableDirectives
-        // https://github.com/eslint-community/eslint-plugin-eslint-comments/issues/133
-        "eslint-comments/no-unused-enable": "off"
+        // https://github.com/eslint-community/eslint-plugin-eslint-comments/blob/main/docs/rules/no-unused-enable.md
+        "eslint-comments/no-unused-enable": "error"
       }
     }
   ];
@@ -396,17 +395,19 @@ async function comments() {
 async function disables() {
   return [
     {
-      files: [`scripts/${GLOB_SRC}`],
+      files: [`**/scripts/${GLOB_SRC}`],
       name: "luxass/disables/scripts",
       rules: {
+        "antfu/no-top-level-await": "off",
         "no-console": "off",
         "ts/explicit-function-return-type": "off"
       }
     },
     {
-      files: [`cli/${GLOB_SRC}`, `cli.${GLOB_SRC_EXT}`],
+      files: [`**/cli/${GLOB_SRC}`, `**/cli.${GLOB_SRC_EXT}`],
       name: "luxass/disables/cli",
       rules: {
+        "antfu/no-top-level-await": "off",
         "no-console": "off"
       }
     },
@@ -429,17 +430,27 @@ async function disables() {
       }
     },
     {
-      files: ["**/*.{test,spec}.([tj])s?(x)"],
-      name: "luxass/disables/test",
+      files: ["**/*.js", "**/*.cjs"],
+      name: "luxass/disables/cjs",
       rules: {
-        "no-unused-expressions": "off"
+        "ts/no-require-imports": "off"
       }
     },
     {
-      files: ["**/*.js", "**/*.cjs"],
-      name: "luxass/disables/cjs",
+      name: "luxass/disables/github-actions",
+      files: ["**/.github/workflows/*.{yml,yaml}"],
       rules: {
-        "ts/no-require-imports": "off"
+        // GitHub Actions supports empty values to enable features
+        "yaml/no-empty-mapping-value": "off"
+      }
+    },
+    {
+      files: [`**/*.config.${GLOB_SRC_EXT}`, `**/*.config.*.${GLOB_SRC_EXT}`],
+      name: "luxass/disables/config-files",
+      rules: {
+        "antfu/no-top-level-await": "off",
+        "no-console": "off",
+        "ts/explicit-function-return-type": "off"
       }
     }
   ];
@@ -495,6 +506,16 @@ async function stylistic(options = {}) {
 }
 
 // src/configs/formatters.ts
+function mergePrettierOptions(options, overrides = {}) {
+  return {
+    ...options,
+    ...overrides,
+    plugins: [
+      ...overrides.plugins || [],
+      ...options.plugins || []
+    ]
+  };
+}
 async function formatters(options = {}, stylistic2 = {}) {
   if (options === true) {
     options = {
@@ -557,10 +578,9 @@ async function formatters(options = {}, stylistic2 = {}) {
         rules: {
           "format/prettier": [
             "error",
-            {
-              ...prettierOptions,
+            mergePrettierOptions(prettierOptions, {
               parser: "css"
-            }
+            })
           ]
         }
       },
@@ -573,10 +593,9 @@ async function formatters(options = {}, stylistic2 = {}) {
         rules: {
           "format/prettier": [
             "error",
-            {
-              ...prettierOptions,
+            mergePrettierOptions(prettierOptions, {
               parser: "scss"
-            }
+            })
           ]
         }
       },
@@ -589,10 +608,9 @@ async function formatters(options = {}, stylistic2 = {}) {
         rules: {
           "format/prettier": [
             "error",
-            {
-              ...prettierOptions,
+            mergePrettierOptions(prettierOptions, {
               parser: "less"
-            }
+            })
           ]
         }
       }
@@ -608,10 +626,9 @@ async function formatters(options = {}, stylistic2 = {}) {
       rules: {
         "format/prettier": [
           "error",
-          {
-            ...prettierOptions,
+          mergePrettierOptions(prettierOptions, {
             parser: "html"
-          }
+          })
         ]
       }
     });
@@ -627,11 +644,10 @@ async function formatters(options = {}, stylistic2 = {}) {
       rules: {
         [`format/${formater}`]: [
           "error",
-          formater === "prettier" ? {
-            ...prettierOptions,
+          formater === "prettier" ? mergePrettierOptions(prettierOptions, {
             embeddedLanguageFormatting: "off",
             parser: "markdown"
-          } : {
+          }) : {
             ...dprintOptions,
             language: "markdown"
           }
@@ -649,13 +665,12 @@ async function formatters(options = {}, stylistic2 = {}) {
       rules: {
         "format/prettier": [
           "error",
-          {
-            ...prettierOptions,
+          mergePrettierOptions(prettierOptions, {
             parser: "astro",
             plugins: [
               "prettier-plugin-astro"
             ]
-          }
+          })
         ]
       }
     });
@@ -670,10 +685,9 @@ async function formatters(options = {}, stylistic2 = {}) {
       rules: {
         "format/prettier": [
           "error",
-          {
-            ...prettierOptions,
+          mergePrettierOptions(prettierOptions, {
             parser: "graphql"
-          }
+          })
         ]
       }
     });
@@ -772,6 +786,7 @@ async function javascript(options = {}) {
           "error",
           { enforceForClassMembers: true, setWithoutGet: true }
         ],
+        "antfu/no-top-level-await": "error",
         "array-callback-return": "error",
         "block-scoped-var": "error",
         "constructor-super": "error",
@@ -1168,6 +1183,7 @@ async function markdown(options = {}) {
         }
       },
       rules: {
+        "antfu/no-top-level-await": "off",
         "import/newline-after-import": "off",
         "no-alert": "off",
         "no-console": "off",
@@ -1182,6 +1198,7 @@ async function markdown(options = {}) {
         "style/comma-dangle": "off",
         "style/eol-last": "off",
         "ts/consistent-type-imports": "off",
+        "ts/explicit-function-return-type": "off",
         "ts/no-namespace": "off",
         "ts/no-redeclare": "off",
         "ts/no-require-imports": "off",
@@ -1234,12 +1251,12 @@ async function perfectionist() {
         "perfectionist/sort-exports": ["error", { order: "asc", type: "natural" }],
         "perfectionist/sort-imports": ["error", {
           groups: [
+            "type",
+            ["parent-type", "sibling-type", "index-type"],
             "builtin",
             "external",
-            "type",
             ["internal", "internal-type"],
             ["parent", "sibling", "index"],
-            ["parent-type", "sibling-type", "index-type"],
             "side-effect",
             "object",
             "unknown"
@@ -1266,34 +1283,41 @@ var RemixPackages = [
   "@remix-run/serve",
   "@remix-run/dev"
 ];
+var NextJsPackages = [
+  "next"
+];
 async function react(options = {}) {
   const {
     files = [GLOB_JS, GLOB_JSX, GLOB_TS, GLOB_TSX],
-    overrides = {}
+    overrides = {},
+    filesTypeAware = [GLOB_TS, GLOB_TSX],
+    ignoresTypeAware = [
+      `${GLOB_MARKDOWN}/**`,
+      GLOB_ASTRO_TS
+    ],
+    tsconfigPath
   } = options;
   await ensure([
     "@eslint-react/eslint-plugin",
     "eslint-plugin-react-hooks",
     "eslint-plugin-react-refresh"
   ]);
-  const tsconfigPath = options?.tsconfigPath ? toArray(options.tsconfigPath) : void 0;
   const isTypeAware = !!tsconfigPath;
+  const typeAwareRules = {
+    "react/no-leaked-conditional-rendering": "warn"
+  };
   const [
     pluginReact,
     pluginReactHooks,
-    pluginReactRefresh,
-    parserTs
+    pluginReactRefresh
   ] = await Promise.all([
     interop(import("@eslint-react/eslint-plugin")),
     interop(import("eslint-plugin-react-hooks")),
-    interop(import("eslint-plugin-react-refresh")),
-    interop(import("@typescript-eslint/parser"))
+    interop(import("eslint-plugin-react-refresh"))
   ]);
-  const isAllowConstantExport = ReactRefreshAllowConstantExportPackages.some(
-    (i) => (0, import_local_pkg2.isPackageExists)(i)
-  );
+  const isAllowConstantExport = ReactRefreshAllowConstantExportPackages.some((i) => (0, import_local_pkg2.isPackageExists)(i));
   const isUsingRemix = RemixPackages.some((i) => (0, import_local_pkg2.isPackageExists)(i));
-  const isUsingNext = (0, import_local_pkg2.isPackageExists)("next");
+  const isUsingNext = NextJsPackages.some((i) => (0, import_local_pkg2.isPackageExists)(i));
   const plugins = pluginReact.configs.all.plugins;
   return [
     {
@@ -1311,12 +1335,10 @@ async function react(options = {}) {
       name: "luxass/react/rules",
       files,
       languageOptions: {
-        parser: parserTs,
         parserOptions: {
           ecmaFeatures: {
             jsx: true
-          },
-          ...isTypeAware ? { project: tsconfigPath } : {}
+          }
         },
         sourceType: "module"
       },
@@ -1343,6 +1365,13 @@ async function react(options = {}) {
             allowConstantExport: isAllowConstantExport,
             allowExportNames: [
               ...isUsingNext ? [
+                "dynamic",
+                "dynamicParams",
+                "revalidate",
+                "fetchCache",
+                "runtime",
+                "preferredRegion",
+                "maxDuration",
                 "config",
                 "runtime",
                 "generateStaticParams",
@@ -1398,13 +1427,18 @@ async function react(options = {}) {
         "react/prefer-destructuring-assignment": "warn",
         "react/prefer-shorthand-boolean": "warn",
         "react/prefer-shorthand-fragment": "warn",
-        ...isTypeAware ? {
-          "react/no-leaked-conditional-rendering": "warn"
-        } : {},
         // overrides
         ...overrides
       }
-    }
+    },
+    ...isTypeAware ? [{
+      files: filesTypeAware,
+      ignores: ignoresTypeAware,
+      name: "antfu/react/type-aware-rules",
+      rules: {
+        ...typeAwareRules
+      }
+    }] : []
   ];
 }
 
@@ -1743,7 +1777,6 @@ async function test(options = {}) {
       name: "luxass/test/rules",
       files,
       rules: {
-        "node/prefer-global/process": "off",
         "test/consistent-test-it": [
           "error",
           { fn: "it", withinDescribe: "it" }
@@ -1753,7 +1786,13 @@ async function test(options = {}) {
         "test/no-focused-tests": isInEditor ? "off" : ["error", { fixable: true }],
         "test/prefer-hooks-in-order": "error",
         "test/prefer-lowercase-title": "error",
-        "ts/explicit-function-return-type": "off",
+        // Disables
+        ...{
+          "antfu/no-top-level-await": "off",
+          "no-unused-expressions": "off",
+          "node/prefer-global/process": "off",
+          "ts/explicit-function-return-type": "off"
+        },
         ...overrides
       }
     }
@@ -1827,6 +1866,7 @@ async function typescript(options = {}) {
     exts = [],
     overrides = {},
     parserOptions = {},
+    overridesTypeAware = {},
     type = "app"
   } = options ?? {};
   const files = options.files ?? [
@@ -1951,8 +1991,13 @@ async function typescript(options = {}) {
         "ts/no-invalid-this": "error",
         "ts/no-invalid-void-type": "off",
         "ts/no-non-null-assertion": "off",
-        "ts/no-redeclare": "error",
+        "ts/no-redeclare": ["error", { builtinGlobals: false }],
         "ts/no-require-imports": "error",
+        "ts/no-unused-expressions": ["error", {
+          allowShortCircuit: true,
+          allowTaggedTemplates: true,
+          allowTernary: true
+        }],
         "ts/no-unused-vars": "off",
         "ts/no-use-before-define": [
           "error",
@@ -1976,7 +2021,10 @@ async function typescript(options = {}) {
       files: filesTypeAware,
       ignores: ignoresTypeAware,
       name: "luxass/typescript/rules-type-aware",
-      rules: typeAwareRules
+      rules: {
+        ...typeAwareRules,
+        ...overridesTypeAware
+      }
     }] : []
   ];
 }
@@ -2160,7 +2208,9 @@ async function vue(options = {}) {
         ...pluginVue.configs["vue3-essential"].rules,
         ...pluginVue.configs["vue3-strongly-recommended"].rules,
         ...pluginVue.configs["vue3-recommended"].rules,
+        "antfu/no-top-level-await": "off",
         "node/prefer-global/process": "off",
+        "ts/explicit-function-return-type": "off",
         "vue/block-order": [
           "error",
           {
@@ -2321,10 +2371,7 @@ async function yaml(options = {}) {
           "yaml/indent": ["error", indent === "tab" ? 2 : indent],
           "yaml/key-spacing": "error",
           "yaml/no-tab-indent": "error",
-          "yaml/quotes": [
-            "error",
-            { avoidEscape: false, prefer: quotes }
-          ],
+          "yaml/quotes": ["error", { avoidEscape: true, prefer: quotes === "backtick" ? "single" : quotes }],
           "yaml/spaced-comment": "error"
         } : {},
         ...overrides