@kitschpatrol/eslint-config 5.3.0 → 5.4.0

package/dist/index.js

@@ -763,7 +763,7 @@ var jsonPrettierRules = {
 
 // src/presets/json-package.ts
 var jsonPackageRecommendedRules = {
-  // Begin expansion 'eslint-plugin-package-json/configs/recommended' 'rules'
+  // Begin expansion 'eslint-plugin-package-json' 'configs.recommended.rules'
   "json-package/no-empty-fields": "error",
   "json-package/order-properties": "error",
   "json-package/require-name": "error",
@@ -1295,7 +1295,8 @@ var prettierRules = {
 // src/presets/react.ts
 var reactRecommendedTypeCheckedRules = {
   // Begin expansion '@eslint-react/eslint-plugin' 'recommended-type-checked'
-  "react/ensure-forward-ref-using-ref": "warn",
+  "react/jsx-no-duplicate-props": "off",
+  "react/jsx-uses-vars": "off",
   "react/no-access-state-in-setstate": "error",
   "react/no-array-index-key": "warn",
   "react/no-children-count": "warn",
@@ -1312,12 +1313,11 @@ var reactRecommendedTypeCheckedRules = {
   "react/no-create-ref": "error",
   "react/no-default-props": "error",
   "react/no-direct-mutation-state": "error",
-  "react/no-duplicate-jsx-props": "off",
   "react/no-duplicate-key": "warn",
   "react/no-forward-ref": "warn",
   "react/no-implicit-key": "warn",
   "react/no-missing-key": "error",
-  "react/no-nested-components": "error",
+  "react/no-nested-component-definitions": "error",
   "react/no-prop-types": "error",
   "react/no-redundant-should-component-update": "error",
   "react/no-set-state-in-component-did-mount": "warn",
@@ -1332,28 +1332,30 @@ var reactRecommendedTypeCheckedRules = {
   "react/no-unused-class-component-members": "warn",
   "react/no-unused-state": "warn",
   "react/no-use-context": "warn",
-  "react/use-jsx-vars": "off",
+  "react/no-useless-forward-ref": "warn",
+  "react/no-useless-fragment": "warn",
   "react-dom/no-dangerously-set-innerhtml": "warn",
   "react-dom/no-dangerously-set-innerhtml-with-children": "error",
   "react-dom/no-find-dom-node": "error",
+  "react-dom/no-flush-sync": "error",
   "react-dom/no-missing-button-type": "warn",
   "react-dom/no-missing-iframe-sandbox": "warn",
   "react-dom/no-namespace": "error",
+  "react-dom/no-render": "error",
   "react-dom/no-render-return-value": "error",
   "react-dom/no-script-url": "warn",
+  "react-dom/no-unknown-property": "off",
   "react-dom/no-unsafe-iframe-sandbox": "warn",
   "react-dom/no-unsafe-target-blank": "warn",
-  "react-dom/no-void-elements-with-children": "warn",
+  "react-dom/no-use-form-state": "error",
+  "react-dom/no-void-elements-with-children": "error",
   "react-web-api/no-leaked-event-listener": "warn",
   "react-web-api/no-leaked-interval": "warn",
   "react-web-api/no-leaked-resize-observer": "warn",
   "react-web-api/no-leaked-timeout": "warn",
   "react-hooks-extra/no-direct-set-state-in-use-effect": "warn",
-  "react-hooks-extra/no-useless-custom-hooks": "warn",
+  "react-hooks-extra/no-unnecessary-use-prefix": "warn",
   "react-hooks-extra/prefer-use-state-lazy-initialization": "warn",
-  "react-naming-convention/context-name": "warn",
-  "react-naming-convention/use-state": "warn",
-  "react-dom/no-unknown-property": "off",
   "react/no-leaked-conditional-rendering": "warn"
   // End expansion
 };
@@ -1464,6 +1466,8 @@ var svelteRecommendedRules = {
   "svelte/no-store-async": "error",
   "svelte/no-svelte-internal": "error",
   "svelte/no-unknown-style-directive-property": "error",
+  "svelte/no-unnecessary-state-wrap": "error",
+  "svelte/no-unused-props": "error",
   "svelte/no-unused-svelte-ignore": "error",
   "svelte/no-useless-children-snippet": "error",
   "svelte/no-useless-mustaches": "error",
@@ -1953,7 +1957,14 @@ var xoTypescriptRules = {
   ],
   "ts/consistent-type-definitions": ["error", "type"],
   "ts/consistent-type-exports": ["error", { fixMixedExportsWithInlineTypeSpecifier: true }],
-  "ts/consistent-type-imports": ["error", { fixStyle: "inline-type-imports" }],
+  "ts/consistent-type-imports": [
+    "error",
+    {
+      disallowTypeAnnotations: false,
+      fixStyle: "separate-type-imports",
+      prefer: "type-imports"
+    }
+  ],
   "func-call-spacing": "off",
   // "@stylistic/func-call-spacing":["error","never"],
   indent: "off",
@@ -2259,7 +2270,7 @@ async function combine(...configs) {
 }
 async function interopDefault(m) {
   const resolved = await m;
-  return resolved.default || resolved;
+  return resolved.default ?? resolved;
 }
 function isInEditorEnv() {
   if (process2.env.CI) return false;
@@ -2403,13 +2414,16 @@ var sharedScriptConfig = {
       }
     ],
     "eslint-comments/disable-enable-pair": ["error", { allowWholeFile: true }],
+    "import/consistent-type-specifier-style": ["error", "prefer-top-level"],
     // 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
+    // Cope with Astro virtual modules.
+    // Astro uses `astro:...` and `@astrojs/...`.
+    // Starlight uses `virtual:...`.
     // https://github.com/hannoeru/vite-plugin-pages/issues/41#issuecomment-1371880072
-    "import/no-unresolved": ["error", { ignore: ["^astro:", "^@astrojs"] }],
+    "import/no-unresolved": ["error", { ignore: ["^astro:", "^@astrojs", "^virtual:"] }],
     "import/order": "off",
     // Conflicts with perfectionist/sort-imports (but never enabled)
     // Knip workaround to ignore unused exported class members:
@@ -2579,6 +2593,8 @@ var sharedScriptConfig = {
     ],
     "ts/sort-type-constituents": "off",
     // Conflicts with perfectionist/sort-intersection-types
+    "unicorn/import-style": "off",
+    // Conflicts with import/consistent-type-specifier-style prefer-top-level
     "unicorn/prevent-abbreviations": [
       "error",
       {
@@ -3303,7 +3319,7 @@ async function json(options = {}) {
       name: "kp/json/rules-tsconfig",
       rules: {
         "json/no-comments": "off",
-        "json/sort-keys": [
+        "jsonc/sort-keys": [
           "error",
           {
             order: ["extends", "compilerOptions", "references", "files", "include", "exclude"],
@@ -3331,6 +3347,7 @@ async function json(options = {}) {
               "useDefineForClassFields",
               "emitDecoratorMetadata",
               "experimentalDecorators",
+              "libReplacement",
               /* Modules */
               "baseUrl",
               "rootDir",
@@ -3405,6 +3422,7 @@ async function json(options = {}) {
               "isolatedModules",
               "preserveSymlinks",
               "verbatimModuleSyntax",
+              "erasableSyntaxOnly",
               /* Completeness */
               "skipDefaultLibCheck",
               "skipLibCheck"
@@ -3823,7 +3841,7 @@ import globals3 from "globals";
 var _pluginTest;
 async function test(options = {}) {
   const { isInEditor = false, overrides = {} } = options;
-  _pluginTest ||= {
+  _pluginTest ??= {
     ...pluginTest,
     // eslint-disable-next-line ts/no-unsafe-assignment
     rules: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kitschpatrol/eslint-config",
-  "version": "5.3.0",
+  "version": "5.4.0",
   "description": "ESLint configuration for @kitschpatrol/shared-config.",
   "keywords": [
     "shared-config",
@@ -42,35 +42,35 @@
   ],
   "dependencies": {
     "@eslint-community/eslint-plugin-eslint-comments": "^4.4.1",
-    "@eslint-react/eslint-plugin": "^1.30.1",
-    "@html-eslint/eslint-plugin": "^0.35.0",
-    "@html-eslint/parser": "^0.35.0",
+    "@eslint-react/eslint-plugin": "^1.38.2",
+    "@html-eslint/eslint-plugin": "^0.37.0",
+    "@html-eslint/parser": "^0.37.0",
     "@pinojs/json-colorizer": "^4.0.0",
-    "@typescript-eslint/eslint-plugin": "^8.26.0",
-    "@typescript-eslint/parser": "^8.26.0",
-    "@vitest/eslint-plugin": "^1.1.36",
-    "astro-eslint-parser": "^1.2.1",
-    "eslint": "^9.21.0",
+    "@typescript-eslint/eslint-plugin": "^8.28.0",
+    "@typescript-eslint/parser": "^8.28.0",
+    "@vitest/eslint-plugin": "^1.1.38",
+    "astro-eslint-parser": "^1.2.2",
+    "eslint": "^9.23.0",
     "eslint-config-flat-gitignore": "^2.1.0",
     "eslint-flat-config-utils": "^2.0.1",
-    "eslint-import-resolver-typescript": "^3.8.3",
-    "eslint-mdx": "^3.1.5",
+    "eslint-import-resolver-typescript": "^3.10.0",
+    "eslint-mdx": "^3.3.1",
     "eslint-plugin-astro": "^1.3.1",
     "eslint-plugin-depend": "^0.12.0",
     "eslint-plugin-html": "^8.1.2",
-    "eslint-plugin-import-x": "^4.6.1",
-    "eslint-plugin-jsdoc": "^50.6.3",
-    "eslint-plugin-jsonc": "^2.19.1",
+    "eslint-plugin-import-x": "^4.9.3",
+    "eslint-plugin-jsdoc": "^50.6.9",
+    "eslint-plugin-jsonc": "^2.20.0",
     "eslint-plugin-jsx-a11y": "^6.10.2",
-    "eslint-plugin-mdx": "^3.1.5",
-    "eslint-plugin-n": "^17.16.1",
+    "eslint-plugin-mdx": "^3.3.1",
+    "eslint-plugin-n": "^17.17.0",
     "eslint-plugin-no-only-tests": "^3.3.0",
-    "eslint-plugin-package-json": "^0.26.2",
-    "eslint-plugin-perfectionist": "^4.9.0",
+    "eslint-plugin-package-json": "^0.29.0",
+    "eslint-plugin-perfectionist": "^4.10.1",
     "eslint-plugin-regexp": "^2.7.0",
-    "eslint-plugin-svelte": "^3.0.2",
+    "eslint-plugin-svelte": "^3.3.3",
     "eslint-plugin-toml": "^0.12.0",
-    "eslint-plugin-unicorn": "^57.0.0",
+    "eslint-plugin-unicorn": "^58.0.0",
     "eslint-plugin-yml": "^1.17.0",
     "execa": "^9.5.2",
     "find-workspaces": "^0.3.1",
@@ -79,8 +79,8 @@
     "jsonc-eslint-parser": "^2.4.0",
     "local-pkg": "^1.1.1",
     "prettier": "^3.5.3",
-    "sort-package-json": "^2.15.1",
-    "svelte-eslint-parser": "^1.0.0",
+    "sort-package-json": "^3.0.0",
+    "svelte-eslint-parser": "^1.1.1",
     "toml-eslint-parser": "^0.10.0",
     "yaml-eslint-parser": "^1.3.0"
   },
@@ -88,11 +88,11 @@
     "@types/eslint-config-prettier": "^6.11.3",
     "@types/eslint-plugin-jsx-a11y": "^6.10.0",
     "dot-prop": "^9.0.0",
-    "eslint-config-prettier": "^10.0.2",
+    "eslint-config-prettier": "^10.1.1",
     "eslint-config-xo-typescript": "^7.0.0",
-    "eslint-typegen": "^2.0.0",
+    "eslint-typegen": "^2.1.0",
     "globby": "^14.1.0",
-    "svelte": "^5.22.1",
+    "svelte": "^5.25.3",
     "tsup": "^8.4.0"
   },
   "engines": {

package/readme.md

@@ -187,8 +187,6 @@ 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.

package/init/package.json

@@ -1,7 +0,0 @@
-{
-	"pnpm": {
-		"overrides": {
-			"eslint-plugin-markdown": "^5.1.0"
-		}
-	}
-}