@jsse/eslint-config 0.2.16 → 0.2.18

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.16";
+var version = "0.2.18";
 
 // 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.16";
+var version = "0.2.18";
 
 // 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"
     }
   ];
 };
@@ -10714,32 +10726,7 @@ function typescriptRulesTypeOblivious() {
     "no-console": [
       "warn",
       {
-        allow: [
-          "log",
-          "warn",
-          "dir",
-          "time",
-          "timeEnd",
-          "timeLog",
-          "trace",
-          "assert",
-          "clear",
-          "count",
-          "countReset",
-          "group",
-          "groupEnd",
-          "table",
-          "debug",
-          "info",
-          "dirxml",
-          "error",
-          "groupCollapsed",
-          "Console",
-          "profile",
-          "profileEnd",
-          "timeStamp",
-          "context"
-        ]
+        allow: ["warn", "error"]
       }
     ],
     "object-curly-spacing": "off",
@@ -11340,6 +11327,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) {
@@ -11535,24 +11525,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,
@@ -11604,13 +11576,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(),
@@ -11623,10 +11596,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;
 };
@@ -11944,6 +11946,7 @@ function typescriptParser(options) {
     componentExts = [],
     filesTypeAware: files = [GLOB_TS, GLOB_TSX],
     ignoresTypeAware: ignores2 = [GLOB_MARKDOWN],
+    name,
     parserOptions: parserOptionsOverride = {},
     react: react2,
     tsconfig,
@@ -11971,7 +11974,7 @@ function typescriptParser(options) {
           ...parserOptionsOverride
         }
       },
-      name: `jsse/typescript/${typeAware ? "type-aware-parser" : "parser"}`
+      name: name ?? `jsse/typescript/${typeAware ? "type-aware-parser" : "parser"}`
     };
   }
   return {
@@ -11991,7 +11994,7 @@ function typescriptParser(options) {
         ...parserOptionsOverride
       }
     },
-    name: `jsse/typescript/${typeAware ? "type-aware-parser" : "parser"}`
+    name: name ?? `jsse/typescript/${typeAware ? "type-aware-parser" : "parser"}`
   };
 }
 
@@ -12163,6 +12166,7 @@ var unicorn = async () => {
         ...unicornRecommended(),
         ...unicornOff(),
         "unicorn/better-regex": "error",
+        "unicorn/consistent-empty-array-spread": "error",
         "unicorn/custom-error-definition": "error",
         // Pass error message when throwing errors
         "unicorn/error-message": "error",
@@ -12296,7 +12300,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",
@@ -12405,15 +12409,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",
@@ -12451,7 +12446,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" })
@@ -12630,9 +12625,13 @@ async function jsse(options = {}, ...userConfigs) {
       }
     }
     for (const config of combinedConfigs) {
-      for (const rule of Object.keys(config.rules ?? {})) {
-        if (off.includes(rule) && config.rules) {
-          config.rules[rule] = "off";
+      for (const cfgRules of [config.rules, config.plugins]) {
+        if (cfgRules) {
+          for (const rule of off) {
+            if (cfgRules[rule] !== void 0 && cfgRules[rule] !== "off" && cfgRules[rule] !== 0) {
+              cfgRules[rule] = "off";
+            }
+          }
         }
       }
     }
@@ -12647,21 +12646,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)) {
@@ -12680,6 +12664,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;
 }
 
@@ -12725,6 +12724,7 @@ function jsseReact() {
   GLOB_TSCONFIG,
   GLOB_TSX,
   GLOB_YAML,
+  SLOW_RULES,
   combine,
   combineAsync,
   importJsoncLibs,