@jsse/eslint-config 0.2.15 → 0.2.17

package/README.md

@@ -4,15 +4,15 @@
 
 Eslint config
 
--   eslint flat config
--   double quotes + semi
--   react, react-hooks & react-refresh, OH MY!
--   no vue (I don't use vue (as of 2023-10-12) don't want/need the vue plugins/parsers/etc)
--   assumes you're using prettier
--   Based on:
-    -   My old, very long, and very messy (non-flat) eslint config (`.eslintrc.js`)
-    -   antfu eslint-config [eslint-config](https://github.com/antfu/eslint-config)
-    -   sxzz eslint-config [eslint-config](https://github.com/sxzz/eslint-config)
+- eslint flat config
+- double quotes + semi
+- react, react-hooks & react-refresh, OH MY!
+- no vue (I don't use vue (as of 2023-10-12) don't want/need the vue plugins/parsers/etc)
+- assumes you're using prettier
+- Based on:
+  - My old, very long, and very messy (non-flat) eslint config (`.eslintrc.js`)
+  - antfu eslint-config [eslint-config](https://github.com/antfu/eslint-config)
+  - sxzz eslint-config [eslint-config](https://github.com/sxzz/eslint-config)
 
 ## Usage
 
@@ -31,11 +31,11 @@ With [`"type": "module"`](https://nodejs.org/api/packages.html#type) in `package
 import jsse from "@jsse/eslint-config";
 
 export default jsse({
-    // options
-    // typescript: {
-    //     tsconfigPath: "tsconfig.json", // or ["tsconfig.json", "tsconfig.eslint.json", ...]
-    // },
-    // react: true
+  // options
+  // typescript: {
+  //     tsconfigPath: "tsconfig.json", // or ["tsconfig.json", "tsconfig.eslint.json", ...]
+  // },
+  // react: true
 });
 ```
 
@@ -44,11 +44,11 @@ With CJS:
 ```js
 // eslint.config.js
 module.exports = require("@jsse/eslint-config").jsse({
-    // options
-    // typescript: {
-    //     tsconfigPath: "tsconfig.json", // or ["tsconfig.json", "tsconfig.eslint.json", ...]
-    // },
-    // react: true
+  // options
+  // typescript: {
+  //     tsconfigPath: "tsconfig.json", // or ["tsconfig.json", "tsconfig.eslint.json", ...]
+  // },
+  // react: true
 });
 ```
 
@@ -60,10 +60,10 @@ For example:
 
 ```json
 {
-    "scripts": {
-        "lint": "eslint .",
-        "lint:fix": "eslint . --fix"
-    }
+  "scripts": {
+    "lint": "eslint .",
+    "lint:fix": "eslint . --fix"
+  }
 }
 ```
 
@@ -77,18 +77,18 @@ options object to the `typescript` config:
 import jsse from "@jsse/eslint-config";
 
 export default jsse({
-    typescript: {
-        tsconfigPath: "tsconfig.json", // or ["tsconfig.json", "tsconfig.eslint.json", ...]
-    },
+  typescript: {
+    tsconfigPath: "tsconfig.json", // or ["tsconfig.json", "tsconfig.eslint.json", ...]
+  },
 });
 ```
 
 ## Thanks
 
--   [antfu](https://github.com/antfu) and [sxzz](https://github.com/sxzz) for their eslint-configs which
-    this is based on
--   Younger me for being more willing to spend time configuring eslint
--   My fans for being my fans
+- [antfu](https://github.com/antfu) and [sxzz](https://github.com/sxzz) for their eslint-configs which
+  this is based on
+- Younger me for being more willing to spend time configuring eslint
+- My fans for being my fans
 
 ## FAQ
 

package/dist/cli.cjs

@@ -612,7 +612,7 @@ var CAC = class extends import_events.EventEmitter {
 var cac = (name = "") => new CAC(name);
 
 // package.json
-var version = "0.2.15";
+var version = "0.2.17";
 
 // src/cli.ts
 var cli = cac("jsselint");

package/dist/cli.js

@@ -588,7 +588,7 @@ var CAC = class extends EventEmitter {
 var cac = (name = "") => new CAC(name);
 
 // package.json
-var version = "0.2.15";
+var version = "0.2.17";
 
 // src/cli.ts
 var cli = cac("jsselint");

package/dist/index.cjs

@@ -3004,6 +3004,7 @@ __export(src_exports, {
   GLOB_TSCONFIG: () => GLOB_TSCONFIG,
   GLOB_TSX: () => GLOB_TSX,
   GLOB_YAML: () => GLOB_YAML,
+  SLOW_RULES: () => SLOW_RULES,
   combine: () => combine,
   combineAsync: () => combineAsync,
   default: () => jsse,
@@ -3040,6 +3041,16 @@ __export(src_exports, {
 });
 module.exports = __toCommonJS(src_exports);
 
+// src/const.ts
+var SLOW_RULES = [
+  "@typescript-eslint/no-misused-promises",
+  "@typescript-eslint/no-unsafe-assignment",
+  "@typescript-eslint/no-unsafe-call",
+  "@typescript-eslint/no-unsafe-member-access",
+  "@typescript-eslint/no-unsafe-return",
+  "unicorn/no-unnecessary-polyfills"
+];
+
 // src/factory.ts
 var import_node_fs3 = __toESM(require("fs"), 1);
 var import_node_process6 = __toESM(require("process"), 1);
@@ -10180,7 +10191,8 @@ var comments = async () => [
 var ignores = async () => {
   return [
     {
-      ignores: GLOB_EXCLUDE
+      ignores: GLOB_EXCLUDE,
+      name: "jsse/ignores"
     }
   ];
 };
@@ -10432,16 +10444,21 @@ var javascript = async (options) => {
         "prefer-rest-params": "error",
         "prefer-spread": "error",
         "prefer-template": "error",
-        "sort-imports": [
-          "error",
-          {
-            allowSeparatedGroups: false,
-            ignoreCase: false,
-            ignoreDeclarationSort: true,
-            ignoreMemberSort: false,
-            memberSyntaxSortOrder: ["none", "all", "multiple", "single"]
-          }
-        ],
+        "sort-imports": "off",
+        // use `perfectionist/sort-imports` instead
+        /**
+         *  previously used this before switching to `perfectionist/sort-imports`
+         * "sort-imports": [
+         *   "error",
+         *   {
+         *     allowSeparatedGroups: false,
+         *     ignoreCase: false,
+         *     ignoreDeclarationSort: true,
+         *     ignoreMemberSort: false,
+         *     memberSyntaxSortOrder: ["none", "all", "multiple", "single"],
+         *   },
+         * ],
+         */
         "symbol-description": "error",
         "unicode-bom": ["error", "never"],
         "unused-imports/no-unused-imports": isInEditor2 ? "off" : "error",
@@ -11335,6 +11352,9 @@ var prettier = async () => {
   ];
 };
 
+// src/configs/react.ts
+var import_compat = require("@eslint/compat");
+
 // src/configs/ts/typescript-language-options.ts
 var import_node_process4 = __toESM(require("process"), 1);
 function typescriptLanguageOptions(options) {
@@ -11530,24 +11550,6 @@ function reactRules() {
     "react/void-dom-elements-no-children": "error"
   };
 }
-function reactRefreshFlatConfigRules() {
-  return [
-    {
-      files: [GLOB_SRC],
-      name: "jsse/react/refresh",
-      rules: {
-        "react-refresh/only-export-components": "error"
-      }
-    },
-    {
-      files: ["**/*.stories.tsx"],
-      name: "jsse/react:refresh/stories",
-      rules: {
-        "react-refresh/only-export-components": "off"
-      }
-    }
-  ];
-}
 function reactRecomendedRules() {
   return {
     "react/display-name": 2,
@@ -11599,13 +11601,14 @@ var react = async (options) => {
         // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
         react: pluginReact,
         // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
-        "react-hooks": pluginReactHooks,
+        "react-hooks": (0, import_compat.fixupPluginRules)(pluginReactHooks),
         // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
         "react-refresh": pluginReactRefresh
       }
     },
     {
       files: [GLOB_SRC],
+      name: "jsse/react/rules",
       // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
       rules: {
         ...reactRecomendedRules(),
@@ -11618,10 +11621,39 @@ var react = async (options) => {
           version: "detect"
         }
       }
+    },
+    {
+      files: [GLOB_SRC],
+      name: "jsse/react-hooks/rules",
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
+      rules: {
+        "react-hooks/exhaustive-deps": "error",
+        "react-hooks/rules-of-hooks": "error"
+      },
+      settings: {
+        react: {
+          version: "detect"
+        }
+      }
     }
   ];
   if (reactRefresh) {
-    config.push(...reactRefreshFlatConfigRules());
+    config.push(
+      {
+        files: [GLOB_SRC],
+        name: "jsse/react/refresh",
+        rules: {
+          "react-refresh/only-export-components": "error"
+        }
+      },
+      {
+        files: ["**/*.stories.tsx"],
+        name: "jsse/react-refresh/stories",
+        rules: {
+          "react-refresh/only-export-components": "off"
+        }
+      }
+    );
   }
   return config;
 };
@@ -11939,6 +11971,7 @@ function typescriptParser(options) {
     componentExts = [],
     filesTypeAware: files = [GLOB_TS, GLOB_TSX],
     ignoresTypeAware: ignores2 = [GLOB_MARKDOWN],
+    name,
     parserOptions: parserOptionsOverride = {},
     react: react2,
     tsconfig,
@@ -11966,7 +11999,7 @@ function typescriptParser(options) {
           ...parserOptionsOverride
         }
       },
-      name: `jsse/typescript/${typeAware ? "type-aware-parser" : "parser"}`
+      name: name ?? `jsse/typescript/${typeAware ? "type-aware-parser" : "parser"}`
     };
   }
   return {
@@ -11986,7 +12019,7 @@ function typescriptParser(options) {
         ...parserOptionsOverride
       }
     },
-    name: `jsse/typescript/${typeAware ? "type-aware-parser" : "parser"}`
+    name: name ?? `jsse/typescript/${typeAware ? "type-aware-parser" : "parser"}`
   };
 }
 
@@ -12291,7 +12324,7 @@ var yml = async (options) => {
       languageOptions: {
         parser: parserYaml
       },
-      name: "antfu:yaml/rules",
+      name: "jsse/yaml/rules",
       rules: {
         "style/spaced-comment": "off",
         "yaml/block-mapping": "error",
@@ -12400,15 +12433,6 @@ var Lager = class {
 };
 var log = new Lager();
 
-// src/slow.ts
-var slowRules = [
-  "@typescript-eslint/no-misused-promises",
-  "@typescript-eslint/no-unsafe-assignment",
-  "@typescript-eslint/no-unsafe-call",
-  "@typescript-eslint/no-unsafe-member-access",
-  "@typescript-eslint/no-unsafe-return"
-];
-
 // src/factory.ts
 var flatConfigProps = [
   "files",
@@ -12446,7 +12470,7 @@ function normalizeOptions(options = {}) {
   const off = [
     .../* @__PURE__ */ new Set([
       ...options.off ?? [],
-      ...options.fast ? [...slowRules] : []
+      ...options.fast ? [...SLOW_RULES] : []
     ])
   ].sort(
     (a, b) => a.localeCompare(b, void 0, { numeric: true, sensitivity: "base" })
@@ -12642,21 +12666,6 @@ async function jsse(options = {}, ...userConfigs) {
       }
     }
   }
-  if (normalizedOptions.preReturn) {
-    log.debug("Running preReturn");
-    const preReturned = normalizedOptions.preReturn(combinedConfigs);
-    if (preReturned instanceof Promise) {
-      const res = await preReturned;
-      log.debug("preReturn finished");
-      log.debug("final config", res);
-      return res;
-    } else {
-      const res = preReturned;
-      log.debug("preReturn finished");
-      log.debug("final config", res);
-      return res;
-    }
-  }
   for (const config of combinedConfigs) {
     if (config.plugins) {
       for (const [key, value] of Object.entries(config.plugins)) {
@@ -12675,6 +12684,21 @@ async function jsse(options = {}, ...userConfigs) {
       }
     }
   }
+  if (normalizedOptions.preReturn) {
+    log.debug("Running preReturn");
+    const preReturned = normalizedOptions.preReturn(combinedConfigs);
+    if (preReturned instanceof Promise) {
+      const res = await preReturned;
+      log.debug("preReturn finished");
+      log.debug("final config", res);
+      return res;
+    } else {
+      const res = preReturned;
+      log.debug("preReturn finished");
+      log.debug("final config", res);
+      return res;
+    }
+  }
   return combinedConfigs;
 }
 
@@ -12720,6 +12744,7 @@ function jsseReact() {
   GLOB_TSCONFIG,
   GLOB_TSX,
   GLOB_YAML,
+  SLOW_RULES,
   combine,
   combineAsync,
   importJsoncLibs,