@kitschpatrol/eslint-config 5.2.0 → 5.3.0

package/bin/cli.js

@@ -5552,7 +5552,7 @@ var Yargs = YargsFactory(esm_default);
 var yargs_default = Yargs;
 
 // ../../package.json
-var version = "5.2.0";
+var version = "5.3.0";
 
 // ../../src/execa-utilities.ts
 function isErrorExecaError(error) {

package/dist/index.js

@@ -2399,12 +2399,14 @@ var sharedScriptConfig = {
         ignoreConsecutiveComments: true,
         ignoreInlineComments: true,
         // Forgive some additional common patterns arising from temporarily commenting out lines of code
-        ignorePattern: String.raw`if|else|await|const|let|var|import|export|pragma|ignore|prettier-ignore|webpack\w+:|c8|type-coverage:`
+        ignorePattern: String.raw`if|else|await|macOS|const|let|var|import|export|pragma|ignore|prettier-ignore|webpack\w+:|c8|type-coverage:`
       }
     ],
     "eslint-comments/disable-enable-pair": ["error", { allowWholeFile: true }],
     // Overlaps with `unicorn/no-named-default`, and gives false positives for imports from CJS modules
     "import/default": "off",
+    // IMO using the default can help with code readability / disambiguation of a function's origin
+    "import/no-named-as-default-member": "off",
     // Cope with Astro virtual modules
     // https://github.com/hannoeru/vite-plugin-pages/issues/41#issuecomment-1371880072
     "import/no-unresolved": ["error", { ignore: ["^astro:", "^@astrojs"] }],
@@ -2422,15 +2424,17 @@ var sharedScriptConfig = {
     ],
     "max-params": ["warn", { max: 8 }],
     "no-await-in-loop": "off",
-    "no-unreachable": "warn",
     // TSConfig must have allowUnreachableCode: false, this is preferable because it will flag but not spontaneously delete unreachable code
+    "no-unreachable": "warn",
     "no-warning-comments": "off",
     "node/hashbang": "off",
     "node/no-extraneous-import": "off",
-    "node/no-missing-import": "off",
     // Trouble resolving in ts
-    "node/no-process-exit": "off",
+    "node/no-missing-import": "off",
     // Duplicated in unicorn
+    "node/no-process-exit": "off",
+    // Too many false positives
+    "node/no-unpublished-import": "off",
     "node/no-unsupported-features/node-builtins": ["error", { ignores: ["fs/promises.glob"] }],
     "perfectionist/sort-imports": [
       "error",
@@ -2446,6 +2450,12 @@ var sharedScriptConfig = {
     "perfectionist/sort-modules": "off",
     "perfectionist/sort-objects": [
       "error",
+      // Note precedence sensitivity...
+      // This has to come before the `min` rules to sort
+      // strings like `{ minImageWidth: 1, minImageHeight: 1 }` correctly
+      generatePerfectionistSortConfig(["width", "height"]),
+      generatePerfectionistSortConfig(["width", "height"], "leading"),
+      generatePerfectionistSortConfig(["Width", "Height"], "trailing"),
       generatePerfectionistSortConfig(["r", "g", "b"]),
       generatePerfectionistSortConfig(["R", "G", "B"], "trailing"),
       generatePerfectionistSortConfig(["red", "green", "blue"]),
@@ -2462,12 +2472,12 @@ var sharedScriptConfig = {
       generatePerfectionistSortConfig(["Red", "Green", "Blue", "Alpha"], "trailing"),
       generatePerfectionistSortConfig(["x", "y", "z", "w"]),
       generatePerfectionistSortConfig(["X", "Y", "Z", "W"], "trailing"),
+      generatePerfectionistSortConfig(["start", "end"]),
+      generatePerfectionistSortConfig(["start", "end"], "leading"),
+      generatePerfectionistSortConfig(["Start", "End"], "trailing"),
       generatePerfectionistSortConfig(["min", "max"]),
       generatePerfectionistSortConfig(["min", "max"], "leading"),
       generatePerfectionistSortConfig(["Min", "Max"], "trailing"),
-      generatePerfectionistSortConfig(["width", "height"]),
-      generatePerfectionistSortConfig(["width", "height"], "leading"),
-      generatePerfectionistSortConfig(["Width", "Height"], "trailing"),
       { newlinesBetween: "never", order: "asc", type: "natural" }
     ],
     "sort-imports": "off",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kitschpatrol/eslint-config",
-  "version": "5.2.0",
+  "version": "5.3.0",
   "description": "ESLint configuration for @kitschpatrol/shared-config.",
   "keywords": [
     "shared-config",
@@ -42,7 +42,7 @@
   ],
   "dependencies": {
     "@eslint-community/eslint-plugin-eslint-comments": "^4.4.1",
-    "@eslint-react/eslint-plugin": "^1.30.0",
+    "@eslint-react/eslint-plugin": "^1.30.1",
     "@html-eslint/eslint-plugin": "^0.35.0",
     "@html-eslint/parser": "^0.35.0",
     "@pinojs/json-colorizer": "^4.0.0",
@@ -65,7 +65,7 @@
     "eslint-plugin-mdx": "^3.1.5",
     "eslint-plugin-n": "^17.16.1",
     "eslint-plugin-no-only-tests": "^3.3.0",
-    "eslint-plugin-package-json": "^0.26.1",
+    "eslint-plugin-package-json": "^0.26.2",
     "eslint-plugin-perfectionist": "^4.9.0",
     "eslint-plugin-regexp": "^2.7.0",
     "eslint-plugin-svelte": "^3.0.2",
@@ -92,7 +92,7 @@
     "eslint-config-xo-typescript": "^7.0.0",
     "eslint-typegen": "^2.0.0",
     "globby": "^14.1.0",
-    "svelte": "^5.21.0",
+    "svelte": "^5.22.1",
     "tsup": "^8.4.0"
   },
   "engines": {

package/readme.md

@@ -187,6 +187,8 @@ kpi-eslint print-config [file]
 
 Regrettably the `eslint-config init --location package` option is not supported due to ESLint 9's deprecation of support for putting configuration in `package.json`. See ESLint discussion thread [18131](https://github.com/eslint/eslint/discussions/18131).
 
+[`eslint-plugin-mdx`](https://www.npmjs.com/package/eslint-plugin-mdx) currently depends on an old version of [`eslint-plugin-markdown`](https://www.npmjs.com/package/eslint-plugin-markdown) which does not officially support ESLint 9. This dependency is overridden at the root of the [`@kitschpatrol/shared-config` monorepo](https://github.com/kitschpatrol/shared-config). See the [related issue #477](https://github.com/mdx-js/eslint-mdx/issues/477) in the `eslint-mdx` repo.
+
 ### Origins
 
 This config is a heavily modified variation on Anthony Fu's [@antfu/eslint-config](https://github.com/antfu/eslint-config). This package is a somewhat leaner approach intended to work with other tools wrapped behind a monolithic CLI instead of handling everything on its own. It mainly leverages the factory / type generation implementation from the original repo, which itself builds on Kevin Deng's [@sxzz/eslint-config](https://github.com/sxzz/eslint-config). See the [modification notes](./modification-notes.md) for more details on what's changed from Anthony's approach.