@will-stone/eslint-config 7.2.1 → 8.0.1

package/dist/index.cjs

@@ -16,6 +16,7 @@ function astro() {
           parser: parserTypescript2.default
         }
       },
+      name: "will-stone/astro",
       plugins: {
         // @ts-expect-error -- does not confirm to type
         astro: _eslintpluginastro2.default
@@ -95,6 +96,7 @@ function base() {
         // as "warn".
         reportUnusedDisableDirectives: true
       },
+      name: "will-stone/base",
       rules: {
         /**
          * Built-in rules
@@ -374,7 +376,8 @@ var GLOB_EXCLUDE = [
 function ignores() {
   return [
     {
-      ignores: GLOB_EXCLUDE
+      ignores: GLOB_EXCLUDE,
+      name: "will-stone/ignores"
     }
   ];
 }
@@ -385,6 +388,7 @@ var _eslintpluginsimpleimportsort = require('eslint-plugin-simple-import-sort');
 function imports() {
   return [
     {
+      name: "will-stone/imports",
       plugins: {
         "import": pluginImport,
         "simple-import-sort": _eslintpluginsimpleimportsort2.default
@@ -451,10 +455,12 @@ function jest() {
       files: ["**/__mocks__/**/*", "**/*.{spec,test}.{js,cjs,mjs,jsx,ts,tsx}"],
       languageOptions: {
         globals: _globals2.default.jest
-      }
+      },
+      name: "will-stone/jest/globals"
     },
     {
       files: ["**/*.{spec,test}.{js,cjs,mjs,jsx,ts,tsx}"],
+      name: "will-stone/jest/base",
       plugins: {
         jest: _eslintpluginjest2.default
       },
@@ -478,7 +484,6 @@ function jest() {
         "jest/no-focused-tests": "error",
         "jest/no-hooks": "off",
         "jest/no-identical-title": "error",
-        "jest/no-if": "error",
         "jest/no-interpolation-in-snapshots": "error",
         "jest/no-jasmine-globals": "warn",
         "jest/no-large-snapshots": "off",
@@ -517,6 +522,7 @@ function jest() {
     },
     {
       files: ["**/*.{spec,test}.{ts,tsx}"],
+      name: "will-stone/jest/typescript",
       rules: {
         "jest/no-untyped-mock-factory": "warn"
       }
@@ -533,6 +539,7 @@ function node() {
       languageOptions: {
         globals: _globals2.default.node
       },
+      name: "will-stone/node",
       plugins: {
         n: _eslintpluginn2.default
       },
@@ -540,6 +547,9 @@ function node() {
         "n/callback-return": "error",
         "n/global-require": "error",
         "n/handle-callback-err": "error",
+        // Not sure I need this...?
+        // TODO check others' configs to see how they use this.
+        "n/hashbang": "off",
         "n/no-mixed-requires": "error",
         "n/no-new-require": "error",
         "n/no-path-concat": "error",
@@ -549,9 +559,6 @@ function node() {
         "n/no-restricted-require": "off",
         // I like synchronous functions
         "n/no-sync": "off",
-        // Not sure I need this...?
-        // TODO check others' configs to see how they use this.
-        "n/shebang": "off",
         // Turn these off until I work out if I need them
         "n/exports-style": "off",
         "n/file-extension-in-import": "off",
@@ -576,6 +583,7 @@ function node() {
         "n/prefer-global/text-encoder": "off",
         "n/prefer-global/url": "off",
         "n/prefer-global/url-search-params": "off",
+        "n/prefer-node-protocol": "warn",
         "n/prefer-promises/dns": "off",
         "n/prefer-promises/fs": "off",
         "n/process-exit-as-throw": "off"
@@ -584,21 +592,6 @@ function node() {
   ];
 }
 
-// src/configs/prettier.ts
-var _eslintpluginprettier = require('eslint-plugin-prettier'); var _eslintpluginprettier2 = _interopRequireDefault(_eslintpluginprettier);
-function prettier() {
-  return [
-    {
-      plugins: {
-        prettier: _eslintpluginprettier2.default
-      },
-      rules: {
-        "prettier/prettier": "warn"
-      }
-    }
-  ];
-}
-
 // src/configs/react.ts
 var _eslintpluginjsxa11y = require('eslint-plugin-jsx-a11y'); var _eslintpluginjsxa11y2 = _interopRequireDefault(_eslintpluginjsxa11y);
 var _eslintpluginreact = require('eslint-plugin-react'); var _eslintpluginreact2 = _interopRequireDefault(_eslintpluginreact);
@@ -616,6 +609,7 @@ function react() {
           }
         }
       },
+      name: "will-stone/react",
       plugins: {
         "jsx-a11y": _eslintpluginjsxa11y2.default,
         "react": _eslintpluginreact2.default,
@@ -898,32 +892,13 @@ function react() {
   ];
 }
 
-// src/configs/switch-case.ts
-var _eslintpluginswitchcase = require('eslint-plugin-switch-case'); var _eslintpluginswitchcase2 = _interopRequireDefault(_eslintpluginswitchcase);
-function switchCase() {
-  return [
-    {
-      plugins: {
-        "switch-case": _eslintpluginswitchcase2.default
-      },
-      rules: {
-        "switch-case/newline-between-switch-case": [
-          "warn",
-          "always",
-          { fallthrough: "never" }
-        ],
-        "switch-case/no-case-curly": "off"
-      }
-    }
-  ];
-}
-
 // src/configs/tailwind.ts
 var _eslintplugintailwindcss = require('eslint-plugin-tailwindcss'); var _eslintplugintailwindcss2 = _interopRequireDefault(_eslintplugintailwindcss);
 function tailwind() {
   return [
     {
       files: ["**/*.{jsx,tsx,astro}"],
+      name: "will-stone/tailwind",
       plugins: { tailwindcss: _eslintplugintailwindcss2.default },
       rules: {
         "tailwindcss/classnames-order": "warn",
@@ -955,6 +930,7 @@ function typescript(options) {
           } : {}
         }
       },
+      name: "will-stone/typescript",
       plugins: {
         // @ts-expect-error -- does not confirm to type
         "@typescript-eslint": _eslintplugin2.default
@@ -1180,6 +1156,7 @@ var _eslintpluginunicorn = require('eslint-plugin-unicorn'); var _eslintpluginun
 function unicorn() {
   return [
     {
+      name: "will-stone/unicorn",
       plugins: {
         unicorn: _eslintpluginunicorn2.default
       },
@@ -1205,12 +1182,14 @@ function unicorn() {
         "unicorn/import-style": "off",
         "unicorn/new-for-builtins": "warn",
         "unicorn/no-abusive-eslint-disable": "error",
+        "unicorn/no-anonymous-default-export": "error",
         "unicorn/no-array-callback-reference": "off",
         "unicorn/no-array-for-each": "warn",
         "unicorn/no-array-method-this-argument": "warn",
         "unicorn/no-array-push-push": "warn",
         "unicorn/no-array-reduce": "error",
         "unicorn/no-await-expression-member": "warn",
+        "unicorn/no-await-in-promise-methods": "error",
         "unicorn/no-console-spaces": "warn",
         "unicorn/no-document-cookie": "error",
         // TODO turn back on when it can exclude files. As this warns about the
@@ -1237,6 +1216,7 @@ function unicorn() {
         "unicorn/no-null": "off",
         "unicorn/no-object-as-default-parameter": "error",
         "unicorn/no-process-exit": "error",
+        "unicorn/no-single-promise-in-promise-methods": "warn",
         "unicorn/no-static-only-class": "error",
         "unicorn/no-thenable": "error",
         "unicorn/no-this-assignment": "error",
@@ -1331,7 +1311,6 @@ var _glob = require('glob');
 function checkEnvironment() {
   const isGitIgnore = _fs.existsSync.call(void 0, ".gitignore");
   let isNodeEngine = false;
-  let isPrettier = false;
   let isTailwind = false;
   let testingFramework = null;
   const allPackageJsonPaths = _glob.globSync.call(void 0, "**/package.json", {
@@ -1349,13 +1328,8 @@ function checkEnvironment() {
     if (!isNodeEngine && Boolean(_optionalChain([packageJson, 'access', _6 => _6.engines, 'optionalAccess', _7 => _7.node]))) {
       isNodeEngine = true;
     }
-    if (!isPrettier && Boolean(
-      _optionalChain([packageJson, 'access', _8 => _8.dependencies, 'optionalAccess', _9 => _9.prettier]) || _optionalChain([packageJson, 'access', _10 => _10.devDependencies, 'optionalAccess', _11 => _11.prettier])
-    )) {
-      isPrettier = true;
-    }
     if (!isTailwind && Boolean(
-      _optionalChain([packageJson, 'access', _12 => _12.dependencies, 'optionalAccess', _13 => _13.tailwindcss]) || _optionalChain([packageJson, 'access', _14 => _14.devDependencies, 'optionalAccess', _15 => _15.tailwindcss])
+      _optionalChain([packageJson, 'access', _8 => _8.dependencies, 'optionalAccess', _9 => _9.tailwindcss]) || _optionalChain([packageJson, 'access', _10 => _10.devDependencies, 'optionalAccess', _11 => _11.tailwindcss])
     )) {
       isTailwind = true;
     }
@@ -1364,7 +1338,6 @@ function checkEnvironment() {
   return {
     isGitIgnore,
     isNode,
-    isPrettier,
     isTailwind,
     testingFramework
   };
@@ -1372,7 +1345,7 @@ function checkEnvironment() {
 
 // src/factory.ts
 function factory(options) {
-  const { isGitIgnore, isNode, isPrettier, isTailwind, testingFramework } = checkEnvironment();
+  const { isGitIgnore, isNode, isTailwind, testingFramework } = checkEnvironment();
   const configs = [];
   if (isGitIgnore) {
     configs.push([_eslintconfigflatgitignore2.default.call(void 0, )]);
@@ -1381,7 +1354,6 @@ function factory(options) {
     ignores(),
     base(),
     imports(),
-    switchCase(),
     unicorn(),
     typescript(options),
     astro(),
@@ -1396,9 +1368,6 @@ function factory(options) {
   if (testingFramework === "jest") {
     configs.push(jest());
   }
-  if (isPrettier) {
-    configs.push(prettier());
-  }
   const merged = configs.flat();
   return merged;
 }

package/dist/index.js

@@ -16,6 +16,7 @@ function astro() {
           parser: parserTypescript
         }
       },
+      name: "will-stone/astro",
       plugins: {
         // @ts-expect-error -- does not confirm to type
         astro: pluginAstro
@@ -95,6 +96,7 @@ function base() {
         // as "warn".
         reportUnusedDisableDirectives: true
       },
+      name: "will-stone/base",
       rules: {
         /**
          * Built-in rules
@@ -374,7 +376,8 @@ var GLOB_EXCLUDE = [
 function ignores() {
   return [
     {
-      ignores: GLOB_EXCLUDE
+      ignores: GLOB_EXCLUDE,
+      name: "will-stone/ignores"
     }
   ];
 }
@@ -385,6 +388,7 @@ import pluginSimpleImport from "eslint-plugin-simple-import-sort";
 function imports() {
   return [
     {
+      name: "will-stone/imports",
       plugins: {
         "import": pluginImport,
         "simple-import-sort": pluginSimpleImport
@@ -451,10 +455,12 @@ function jest() {
       files: ["**/__mocks__/**/*", "**/*.{spec,test}.{js,cjs,mjs,jsx,ts,tsx}"],
       languageOptions: {
         globals: globals2.jest
-      }
+      },
+      name: "will-stone/jest/globals"
     },
     {
       files: ["**/*.{spec,test}.{js,cjs,mjs,jsx,ts,tsx}"],
+      name: "will-stone/jest/base",
       plugins: {
         jest: pluginJest
       },
@@ -478,7 +484,6 @@ function jest() {
         "jest/no-focused-tests": "error",
         "jest/no-hooks": "off",
         "jest/no-identical-title": "error",
-        "jest/no-if": "error",
         "jest/no-interpolation-in-snapshots": "error",
         "jest/no-jasmine-globals": "warn",
         "jest/no-large-snapshots": "off",
@@ -517,6 +522,7 @@ function jest() {
     },
     {
       files: ["**/*.{spec,test}.{ts,tsx}"],
+      name: "will-stone/jest/typescript",
       rules: {
         "jest/no-untyped-mock-factory": "warn"
       }
@@ -533,6 +539,7 @@ function node() {
       languageOptions: {
         globals: globals3.node
       },
+      name: "will-stone/node",
       plugins: {
         n: pluginNode
       },
@@ -540,6 +547,9 @@ function node() {
         "n/callback-return": "error",
         "n/global-require": "error",
         "n/handle-callback-err": "error",
+        // Not sure I need this...?
+        // TODO check others' configs to see how they use this.
+        "n/hashbang": "off",
         "n/no-mixed-requires": "error",
         "n/no-new-require": "error",
         "n/no-path-concat": "error",
@@ -549,9 +559,6 @@ function node() {
         "n/no-restricted-require": "off",
         // I like synchronous functions
         "n/no-sync": "off",
-        // Not sure I need this...?
-        // TODO check others' configs to see how they use this.
-        "n/shebang": "off",
         // Turn these off until I work out if I need them
         "n/exports-style": "off",
         "n/file-extension-in-import": "off",
@@ -576,6 +583,7 @@ function node() {
         "n/prefer-global/text-encoder": "off",
         "n/prefer-global/url": "off",
         "n/prefer-global/url-search-params": "off",
+        "n/prefer-node-protocol": "warn",
         "n/prefer-promises/dns": "off",
         "n/prefer-promises/fs": "off",
         "n/process-exit-as-throw": "off"
@@ -584,21 +592,6 @@ function node() {
   ];
 }
 
-// src/configs/prettier.ts
-import pluginPrettier from "eslint-plugin-prettier";
-function prettier() {
-  return [
-    {
-      plugins: {
-        prettier: pluginPrettier
-      },
-      rules: {
-        "prettier/prettier": "warn"
-      }
-    }
-  ];
-}
-
 // src/configs/react.ts
 import pluginJsxA11y from "eslint-plugin-jsx-a11y";
 import pluginReact from "eslint-plugin-react";
@@ -616,6 +609,7 @@ function react() {
           }
         }
       },
+      name: "will-stone/react",
       plugins: {
         "jsx-a11y": pluginJsxA11y,
         "react": pluginReact,
@@ -898,32 +892,13 @@ function react() {
   ];
 }
 
-// src/configs/switch-case.ts
-import pluginSwitchCase from "eslint-plugin-switch-case";
-function switchCase() {
-  return [
-    {
-      plugins: {
-        "switch-case": pluginSwitchCase
-      },
-      rules: {
-        "switch-case/newline-between-switch-case": [
-          "warn",
-          "always",
-          { fallthrough: "never" }
-        ],
-        "switch-case/no-case-curly": "off"
-      }
-    }
-  ];
-}
-
 // src/configs/tailwind.ts
 import pluginTailwind from "eslint-plugin-tailwindcss";
 function tailwind() {
   return [
     {
       files: ["**/*.{jsx,tsx,astro}"],
+      name: "will-stone/tailwind",
       plugins: { tailwindcss: pluginTailwind },
       rules: {
         "tailwindcss/classnames-order": "warn",
@@ -955,6 +930,7 @@ function typescript(options) {
           } : {}
         }
       },
+      name: "will-stone/typescript",
       plugins: {
         // @ts-expect-error -- does not confirm to type
         "@typescript-eslint": pluginTypescript
@@ -1180,6 +1156,7 @@ import pluginUnicorn from "eslint-plugin-unicorn";
 function unicorn() {
   return [
     {
+      name: "will-stone/unicorn",
       plugins: {
         unicorn: pluginUnicorn
       },
@@ -1205,12 +1182,14 @@ function unicorn() {
         "unicorn/import-style": "off",
         "unicorn/new-for-builtins": "warn",
         "unicorn/no-abusive-eslint-disable": "error",
+        "unicorn/no-anonymous-default-export": "error",
         "unicorn/no-array-callback-reference": "off",
         "unicorn/no-array-for-each": "warn",
         "unicorn/no-array-method-this-argument": "warn",
         "unicorn/no-array-push-push": "warn",
         "unicorn/no-array-reduce": "error",
         "unicorn/no-await-expression-member": "warn",
+        "unicorn/no-await-in-promise-methods": "error",
         "unicorn/no-console-spaces": "warn",
         "unicorn/no-document-cookie": "error",
         // TODO turn back on when it can exclude files. As this warns about the
@@ -1237,6 +1216,7 @@ function unicorn() {
         "unicorn/no-null": "off",
         "unicorn/no-object-as-default-parameter": "error",
         "unicorn/no-process-exit": "error",
+        "unicorn/no-single-promise-in-promise-methods": "warn",
         "unicorn/no-static-only-class": "error",
         "unicorn/no-thenable": "error",
         "unicorn/no-this-assignment": "error",
@@ -1331,7 +1311,6 @@ import { globSync } from "glob";
 function checkEnvironment() {
   const isGitIgnore = existsSync(".gitignore");
   let isNodeEngine = false;
-  let isPrettier = false;
   let isTailwind = false;
   let testingFramework = null;
   const allPackageJsonPaths = globSync("**/package.json", {
@@ -1349,11 +1328,6 @@ function checkEnvironment() {
     if (!isNodeEngine && Boolean(packageJson.engines?.node)) {
       isNodeEngine = true;
     }
-    if (!isPrettier && Boolean(
-      packageJson.dependencies?.prettier || packageJson.devDependencies?.prettier
-    )) {
-      isPrettier = true;
-    }
     if (!isTailwind && Boolean(
       packageJson.dependencies?.tailwindcss || packageJson.devDependencies?.tailwindcss
     )) {
@@ -1364,7 +1338,6 @@ function checkEnvironment() {
   return {
     isGitIgnore,
     isNode,
-    isPrettier,
     isTailwind,
     testingFramework
   };
@@ -1372,7 +1345,7 @@ function checkEnvironment() {
 
 // src/factory.ts
 function factory(options) {
-  const { isGitIgnore, isNode, isPrettier, isTailwind, testingFramework } = checkEnvironment();
+  const { isGitIgnore, isNode, isTailwind, testingFramework } = checkEnvironment();
   const configs = [];
   if (isGitIgnore) {
     configs.push([gitignore()]);
@@ -1381,7 +1354,6 @@ function factory(options) {
     ignores(),
     base(),
     imports(),
-    switchCase(),
     unicorn(),
     typescript(options),
     astro(),
@@ -1396,9 +1368,6 @@ function factory(options) {
   if (testingFramework === "jest") {
     configs.push(jest());
   }
-  if (isPrettier) {
-    configs.push(prettier());
-  }
   const merged = configs.flat();
   return merged;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@will-stone/eslint-config",
-  "version": "7.2.1",
+  "version": "8.0.1",
   "description": "Will Stone's ESLint config",
   "keywords": [
     "eslint-config"
@@ -28,7 +28,7 @@
     "build": "tsup src/index.ts --format esm,cjs --clean --dts --cjsInterop --splitting",
     "dev": "tsup src/index.ts --format esm,cjs --watch & config-inspector",
     "lint": "npm run stub && eslint .",
-    "prepare": "husky install",
+    "prepare": "husky",
     "prepublishOnly": "npm run build",
     "stub": "tsup src/index.ts --format esm",
     "typecheck": "tsc --noEmit"
@@ -43,42 +43,39 @@
   },
   "prettier": "@will-stone/prettier-config",
   "dependencies": {
-    "@typescript-eslint/eslint-plugin": "^7.4.0",
-    "@typescript-eslint/parser": "^7.4.0",
-    "astro-eslint-parser": "^0.16.3",
+    "@typescript-eslint/eslint-plugin": "^7.6.0",
+    "@typescript-eslint/parser": "^7.6.0",
+    "astro-eslint-parser": "^0.17.0",
     "confusing-browser-globals": "^1.0.11",
-    "eslint-config-flat-gitignore": "^0.1.3",
-    "eslint-plugin-astro": "^0.33.1",
+    "eslint-config-flat-gitignore": "^0.1.5",
+    "eslint-plugin-astro": "^0.34.0",
     "eslint-plugin-i": "^2.29.1",
-    "eslint-plugin-jest": "^27.9.0",
+    "eslint-plugin-jest": "^28.2.0",
     "eslint-plugin-jsx-a11y": "^6.8.0",
-    "eslint-plugin-n": "^16.6.2",
-    "eslint-plugin-prettier": "^5.1.3",
+    "eslint-plugin-n": "^17.0.0",
     "eslint-plugin-react": "^7.34.1",
     "eslint-plugin-react-hooks": "^4.6.0",
     "eslint-plugin-simple-import-sort": "^12.0.0",
-    "eslint-plugin-switch-case": "^1.1.2",
     "eslint-plugin-tailwindcss": "^3.15.1",
-    "eslint-plugin-unicorn": "^51.0.1",
-    "glob": "^10.3.10",
+    "eslint-plugin-unicorn": "^52.0.0",
+    "glob": "^10.3.12",
     "globals": "^15.0.0",
     "globby": "^14.0.1",
     "load-json-file": "^7.0.1"
   },
   "devDependencies": {
     "@changesets/cli": "^2.27.1",
-    "@eslint/config-inspector": "^0.1.0",
+    "@eslint/config-inspector": "^0.4.1",
     "@types/confusing-browser-globals": "^1.0.3",
-    "@types/eslint": "^8.56.6",
-    "@types/node": "^20.11.30",
+    "@types/eslint": "^8.56.7",
+    "@types/node": "^20.12.5",
     "@will-stone/prettier-config": "^7.0.3",
-    "eslint": "^8.57.0",
     "husky": "^9.0.11",
     "lint-staged": "^15.2.2",
     "prettier": "^3.2.5",
     "tsup": "^8.0.2",
-    "type-fest": "^4.14.0",
-    "typescript": "^5.4.3"
+    "type-fest": "^4.15.0",
+    "typescript": "^5.4.4"
   },
   "peerDependencies": {
     "eslint": ">=8.57.0"