@kitschpatrol/eslint-config 5.0.6 → 5.0.8

package/bin/cli.js

@@ -1,4 +1,6 @@
 #!/usr/bin/env node
+import { createRequire } from "module";
+const require = createRequire(import.meta.url);
 var __create = Object.create;
 var __defProp = Object.defineProperty;
 var __getOwnPropDesc = Object.getOwnPropertyDescriptor;
@@ -635,7 +637,7 @@ import { fileURLToPath as fileURLToPath3 } from "node:url";
 // ../../node_modules/.pnpm/package-up@5.0.0/node_modules/package-up/index.js
 import process4 from "node:process";
 
-// ../../node_modules/.pnpm/find-up-simple@1.0.0/node_modules/find-up-simple/index.js
+// ../../node_modules/.pnpm/find-up-simple@1.0.1/node_modules/find-up-simple/index.js
 import process3 from "node:process";
 import fsPromises from "node:fs/promises";
 import { fileURLToPath } from "node:url";
@@ -650,8 +652,9 @@ async function findUp(name, {
   let directory = path.resolve(toPath(cwd) ?? "");
   const { root } = path.parse(directory);
   stopAt = path.resolve(directory, toPath(stopAt ?? root));
-  while (directory && directory !== stopAt && directory !== root) {
-    const filePath = path.isAbsolute(name) ? name : path.join(directory, name);
+  const isAbsoluteName = path.isAbsolute(name);
+  while (directory) {
+    const filePath = isAbsoluteName ? name : path.join(directory, name);
     try {
       const stats = await fsPromises.stat(filePath);
       if (type === "file" && stats.isFile() || type === "directory" && stats.isDirectory()) {
@@ -659,6 +662,9 @@ async function findUp(name, {
       }
     } catch {
     }
+    if (directory === stopAt || directory === root) {
+      break;
+    }
     directory = path.dirname(directory);
   }
 }
@@ -670,8 +676,9 @@ function findUpSync(name, {
   let directory = path.resolve(toPath(cwd) ?? "");
   const { root } = path.parse(directory);
   stopAt = path.resolve(directory, toPath(stopAt) ?? root);
-  while (directory && directory !== stopAt && directory !== root) {
-    const filePath = path.isAbsolute(name) ? name : path.join(directory, name);
+  const isAbsoluteName = path.isAbsolute(name);
+  while (directory) {
+    const filePath = isAbsoluteName ? name : path.join(directory, name);
     try {
       const stats = fs.statSync(filePath, { throwIfNoEntry: false });
       if (type === "file" && stats?.isFile() || type === "directory" && stats?.isDirectory()) {
@@ -679,6 +686,9 @@ function findUpSync(name, {
       }
     } catch {
     }
+    if (directory === stopAt || directory === root) {
+      break;
+    }
     directory = path.dirname(directory);
   }
 }
@@ -5541,7 +5551,7 @@ var Yargs = YargsFactory(esm_default);
 var yargs_default = Yargs;
 
 // ../../package.json
-var version = "5.0.6";
+var version = "5.0.8";
 
 // ../../src/execa-utilities.ts
 function isErrorExecaError(error) {
@@ -5843,11 +5853,11 @@ async function executeCliCommand(logStream, positionalArguments, optionFlags, co
       env: {
         // Use colorful output unless NO_COLOR is set
         // eslint-disable-next-line ts/naming-convention
-        ...process.env.NO_COLOR === void 0 ? { FORCE_COLOR: "true" } : {},
-        // Quiet node for when processing *.config.ts files in Node 22
+        ...process.env.NO_COLOR === void 0 ? { FORCE_COLOR: "true" } : {}
+        // Quiet Node when processing *.config.ts files in Node 22
         // Suppress experimental type stripping warning with --no-warnings
-        // eslint-disable-next-line ts/naming-convention
-        NODE_OPTIONS: "--experimental-strip-types --disable-warning=ExperimentalWarning"
+        // TODO what's the story here on Node 20?
+        // NODE_OPTIONS: '--experimental-strip-types --disable-warning=ExperimentalWarning',
       },
       preferLocal: true,
       reject: false,

package/dist/index.d.ts

@@ -3237,6 +3237,11 @@ interface RuleOptions {
 	 * @see https://eslint-react.xyz/docs/rules/dom-no-find-dom-node
 	 */
 	'react-dom/no-find-dom-node'?: Linter.RuleEntry<[]>
+	/**
+	 * warns against using `flushSync`
+	 * @see https://eslint-react.xyz/docs/rules/dom-no-flush-sync
+	 */
+	'react-dom/no-flush-sync'?: Linter.RuleEntry<[]>
 	/**
 	 * enforce that button component have an explicit 'type' attribute
 	 * @see https://eslint-react.xyz/docs/rules/dom-no-missing-button-type
@@ -3337,6 +3342,11 @@ interface RuleOptions {
 	 * @see https://eslint-react.xyz/docs/rules/naming-convention-component-name
 	 */
 	'react-naming-convention/component-name'?: Linter.RuleEntry<ReactNamingConventionComponentName>
+	/**
+	 * enforce context name to end with `Context`.
+	 * @see https://eslint-react.xyz/docs/rules/naming-convention-context-name
+	 */
+	'react-naming-convention/context-name'?: Linter.RuleEntry<[]>
 	/**
 	 * enforce naming convention for JSX filenames
 	 * @see https://eslint-react.xyz/docs/rules/naming-convention-filename
@@ -4178,6 +4188,11 @@ interface RuleOptions {
 	 * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/comment-directive/
 	 */
 	'svelte/comment-directive'?: Linter.RuleEntry<SvelteCommentDirective>
+	/**
+	 * enforce a consistent style for CSS selectors
+	 * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/consistent-selector-style/
+	 */
+	'svelte/consistent-selector-style'?: Linter.RuleEntry<SvelteConsistentSelectorStyle>
 	/**
 	 * derived store should use same variable names between values and callback
 	 * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/derived-has-same-inputs-outputs/
@@ -4276,6 +4291,7 @@ interface RuleOptions {
 	/**
 	 * disallow dynamic slot name
 	 * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/no-dynamic-slot-name/
+	 * @deprecated
 	 */
 	'svelte/no-dynamic-slot-name'?: Linter.RuleEntry<[]>
 	/**
@@ -4291,6 +4307,7 @@ interface RuleOptions {
 	/**
 	 * disallow using goto() without the base path
 	 * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/no-goto-without-base/
+	 * @deprecated
 	 */
 	'svelte/no-goto-without-base'?: Linter.RuleEntry<[]>
 	/**
@@ -4318,6 +4335,11 @@ interface RuleOptions {
 	 * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/no-inspect/
 	 */
 	'svelte/no-inspect'?: Linter.RuleEntry<[]>
+	/**
+	 * disallow using navigation (links, goto, pushState, replaceState) without the base path
+	 * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/no-navigation-without-base/
+	 */
+	'svelte/no-navigation-without-base'?: Linter.RuleEntry<SvelteNoNavigationWithoutBase>
 	/**
 	 * disallow use of not function in event handler
 	 * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/no-not-function-handler/
@@ -4328,6 +4350,11 @@ interface RuleOptions {
 	 * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/no-object-in-text-mustaches/
 	 */
 	'svelte/no-object-in-text-mustaches'?: Linter.RuleEntry<[]>
+	/**
+	 * Checks for invalid raw HTML elements
+	 * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/no-raw-special-elements/
+	 */
+	'svelte/no-raw-special-elements'?: Linter.RuleEntry<[]>
 	/**
 	 * it's not necessary to define functions in reactive statements
 	 * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/no-reactive-functions/
@@ -4393,6 +4420,11 @@ interface RuleOptions {
 	 * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/no-unused-svelte-ignore/
 	 */
 	'svelte/no-unused-svelte-ignore'?: Linter.RuleEntry<[]>
+	/**
+	 * disallow explicit children snippet where it's not needed
+	 * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/no-useless-children-snippet/
+	 */
+	'svelte/no-useless-children-snippet'?: Linter.RuleEntry<[]>
 	/**
 	 * disallow unnecessary mustache interpolations
 	 * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/no-useless-mustaches/
@@ -4403,6 +4435,11 @@ interface RuleOptions {
 	 * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/prefer-class-directive/
 	 */
 	'svelte/prefer-class-directive'?: Linter.RuleEntry<SveltePreferClassDirective>
+	/**
+	 * Require `const` declarations for variables that are never reassigned after declared
+	 * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/prefer-const/
+	 */
+	'svelte/prefer-const'?: Linter.RuleEntry<SveltePreferConst>
 	/**
 	 * destructure values from object stores for better change tracking & fewer redraws
 	 * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/prefer-destructured-store-props/
@@ -4483,6 +4520,11 @@ interface RuleOptions {
 	 * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/valid-prop-names-in-kit-pages/
 	 */
 	'svelte/valid-prop-names-in-kit-pages'?: Linter.RuleEntry<[]>
+	/**
+	 * require valid style element parsing
+	 * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/valid-style-parse/
+	 */
+	'svelte/valid-style-parse'?: Linter.RuleEntry<[]>
 	/**
 	 * Enforce spacing around colons of switch statements
 	 * @see https://eslint.org/docs/latest/rules/switch-colon-spacing
@@ -11225,7 +11267,23 @@ type NodeNoSync =
 	| [
 			{
 				allowAtRootLevel?: boolean
-				ignores?: string[]
+				ignores?: (
+					| string
+					| {
+							from?: 'file'
+							path?: string
+							name?: string[]
+					  }
+					| {
+							from?: 'lib'
+							name?: string[]
+					  }
+					| {
+							from?: 'package'
+							package?: string
+							name?: string[]
+					  }
+				)[]
 			},
 	  ]
 // ----- node/no-unpublished-bin -----
@@ -17379,6 +17437,20 @@ type SvelteCommentDirective =
 				reportUnusedDisableDirectives?: boolean
 			},
 	  ]
+// ----- svelte/consistent-selector-style -----
+type SvelteConsistentSelectorStyle =
+	| []
+	| [
+			{
+				checkGlobal?: boolean
+
+				style?:
+					| []
+					| ['class' | 'id' | 'type']
+					| ['class' | 'id' | 'type', 'class' | 'id' | 'type']
+					| ['class' | 'id' | 'type', 'class' | 'id' | 'type', 'class' | 'id' | 'type']
+			},
+	  ]
 // ----- svelte/first-attribute-linebreak -----
 type SvelteFirstAttributeLinebreak =
 	| []
@@ -17430,7 +17502,8 @@ type SvelteHtmlSelfClosing =
 			| {
 					void?: 'never' | 'always' | 'ignore'
 					normal?: 'never' | 'always' | 'ignore'
-					foreign?: 'never' | 'always' | 'ignore'
+					svg?: 'never' | 'always' | 'ignore'
+					math?: 'never' | 'always' | 'ignore'
 					component?: 'never' | 'always' | 'ignore'
 					svelte?: 'never' | 'always' | 'ignore'
 			  }
@@ -17493,6 +17566,17 @@ type SvelteNoInnerDeclarations =
 				blockScopedFunctions?: 'allow' | 'disallow'
 			},
 	  ]
+// ----- svelte/no-navigation-without-base -----
+type SvelteNoNavigationWithoutBase =
+	| []
+	| [
+			{
+				ignoreGoto?: boolean
+				ignoreLinks?: boolean
+				ignorePushState?: boolean
+				ignoreReplaceState?: boolean
+			},
+	  ]
 // ----- svelte/no-reactive-reassign -----
 type SvelteNoReactiveReassign =
 	| []
@@ -17570,6 +17654,15 @@ type SveltePreferClassDirective =
 				prefer?: 'always' | 'empty'
 			},
 	  ]
+// ----- svelte/prefer-const -----
+type SveltePreferConst =
+	| []
+	| [
+			{
+				destructuring?: 'any' | 'all'
+				ignoreReadBeforeAssign?: boolean
+			},
+	  ]
 // ----- svelte/shorthand-attribute -----
 type SvelteShorthandAttribute =
 	| []

package/dist/index.js

@@ -1437,21 +1437,40 @@ var regexpRecommendedRules = {
 
 // src/presets/svelte.ts
 var svelteRecommendedRules = {
-  // Begin expansion 'eslint-plugin-svelte' 'recommended'
+  // Brittle since there's no negative index access in dot-prop
+  // Begin expansion 'eslint-plugin-svelte' 'flat/recommended[3]'
   "svelte/comment-directive": "error",
+  "svelte/infinite-reactive-loop": "error",
   "svelte/no-at-debug-tags": "warn",
   "svelte/no-at-html-tags": "error",
+  "svelte/no-dom-manipulating": "error",
   "svelte/no-dupe-else-if-blocks": "error",
+  "svelte/no-dupe-on-directives": "error",
   "svelte/no-dupe-style-properties": "error",
-  "svelte/no-dynamic-slot-name": "error",
+  "svelte/no-dupe-use-directives": "error",
+  "svelte/no-export-load-in-svelte-module-in-kit-pages": "error",
+  "svelte/no-immutable-reactive-statements": "error",
   "svelte/no-inner-declarations": "error",
+  "svelte/no-inspect": "warn",
   "svelte/no-not-function-handler": "error",
   "svelte/no-object-in-text-mustaches": "error",
+  "svelte/no-raw-special-elements": "error",
+  "svelte/no-reactive-functions": "error",
+  "svelte/no-reactive-literals": "error",
+  "svelte/no-reactive-reassign": "error",
   "svelte/no-shorthand-style-property-overrides": "error",
+  "svelte/no-store-async": "error",
+  "svelte/no-svelte-internal": "error",
   "svelte/no-unknown-style-directive-property": "error",
   "svelte/no-unused-svelte-ignore": "error",
+  "svelte/no-useless-children-snippet": "error",
+  "svelte/no-useless-mustaches": "error",
+  "svelte/require-each-key": "error",
+  "svelte/require-event-dispatcher-types": "error",
+  "svelte/require-store-reactive-access": "error",
   "svelte/system": "error",
-  "svelte/valid-compile": "error"
+  "svelte/valid-each-key": "error",
+  "svelte/valid-prop-names-in-kit-pages": "error"
   // End expansion
 };
 
@@ -2456,6 +2475,8 @@ var sharedScriptConfig = {
         modifiers: ["const", "exported"],
         // Allow UPPER_CASE const exports
         selector: "variable"
+        // Not objects...
+        // types: ['boolean', 'string', 'number', 'array'],
       }
       // {
       // 	filter: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kitschpatrol/eslint-config",
-  "version": "5.0.6",
+  "version": "5.0.8",
   "description": "ESLint configuration for @kitschpatrol/shared-config.",
   "keywords": [
     "shared-config",
@@ -42,13 +42,13 @@
   ],
   "dependencies": {
     "@eslint-community/eslint-plugin-eslint-comments": "^4.4.1",
-    "@eslint-react/eslint-plugin": "^1.27.0",
+    "@eslint-react/eslint-plugin": "^1.29.0",
     "@html-eslint/eslint-plugin": "^0.35.0",
     "@html-eslint/parser": "^0.35.0",
     "@pinojs/json-colorizer": "^4.0.0",
     "@typescript-eslint/eslint-plugin": "^8.25.0",
     "@typescript-eslint/parser": "^8.25.0",
-    "@vitest/eslint-plugin": "^1.1.33",
+    "@vitest/eslint-plugin": "^1.1.36",
     "astro-eslint-parser": "^1.2.1",
     "eslint": "^9.21.0",
     "eslint-config-flat-gitignore": "^2.1.0",
@@ -63,12 +63,12 @@
     "eslint-plugin-jsonc": "^2.19.1",
     "eslint-plugin-jsx-a11y": "^6.10.2",
     "eslint-plugin-mdx": "^3.1.5",
-    "eslint-plugin-n": "^17.15.1",
+    "eslint-plugin-n": "^17.16.1",
     "eslint-plugin-no-only-tests": "^3.3.0",
-    "eslint-plugin-package-json": "^0.26.0",
+    "eslint-plugin-package-json": "^0.26.1",
     "eslint-plugin-perfectionist": "^4.9.0",
     "eslint-plugin-regexp": "^2.7.0",
-    "eslint-plugin-svelte": "^2.46.1",
+    "eslint-plugin-svelte": "^3.0.2",
     "eslint-plugin-toml": "^0.12.0",
     "eslint-plugin-unicorn": "^57.0.0",
     "eslint-plugin-yml": "^1.17.0",
@@ -77,26 +77,26 @@
     "fs-extra": "^11.3.0",
     "globals": "^16.0.0",
     "jsonc-eslint-parser": "^2.4.0",
-    "local-pkg": "^1.0.0",
+    "local-pkg": "^1.1.0",
     "prettier": "^3.5.2",
-    "sort-package-json": "^2.14.0",
-    "svelte-eslint-parser": "^0.43.0",
+    "sort-package-json": "^2.15.1",
+    "svelte-eslint-parser": "^1.0.0",
     "toml-eslint-parser": "^0.10.0",
-    "yaml-eslint-parser": "^1.2.3"
+    "yaml-eslint-parser": "^1.3.0"
   },
   "devDependencies": {
     "@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.1",
+    "eslint-config-prettier": "^10.0.2",
     "eslint-config-xo-typescript": "^7.0.0",
     "eslint-typegen": "^2.0.0",
-    "svelte": "^5.20.4",
+    "globby": "^14.1.0",
+    "svelte": "^5.20.5",
     "tsup": "^8.4.0"
   },
   "engines": {
-    "node": ">=22.13.1",
-    "pnpm": ">=10.0.0"
+    "node": ">=20.9.0"
   },
   "publishConfig": {
     "access": "public"