adjust

Author: baseballyama

.changeset/crazy-peas-laugh.md

@@ -2,4 +2,4 @@
 'eslint-plugin-svelte': minor
 ---
 
-feat: add `ignorePropertyPatterns` property and rename `ignorePatterns` to `ignoreTypePatterns` in `no-unused-props` rule. The `ignorePatterns` option existed only for a few days and is removed by this PR. Technically, this is a breaking change, but we’ll handle it as a minor release since very few users are likely affected.
+feat: add `ignorePropertyPatterns` property and rename `ignorePatterns` to `ignoreTypePatterns` in `no-unused-props` rule. The `ignorePatterns` option existed only for a few hours and is removed by this PR. Technically, this is a breaking change, but we’ll handle it as a minor release since very few users are likely affected.

docs/rules/no-unused-props.md

@@ -211,7 +211,7 @@ Examples:
   /* eslint svelte/no-unused-props: ["error", { "ignorePropertyPatterns": ["/^_/"] }] */
   // Ignore properties with names matching the pattern
   interface Props {
-    _internal: string; // This prop won't be reported even if unused
+    _internal: string;
     value: number;
   }
   let { value }: Props = $props();

packages/eslint-plugin-svelte/src/rules/no-unused-props.ts

@@ -76,38 +76,28 @@ export default createRule('no-unused-props', {
 		// From v3.3.0, it was replaced with `ignorePropertyPatterns` and `ignoreTypePatterns`.
 		if (options.ignorePatterns != null && !isDeprecationWarningShown) {
 			console.warn(
-				'eslint-plugin-svelte: The `ignorePatterns` option in the `no-unused-props` rule has been deprecated. Please use `ignorePropertyPatterns` and `ignoreTypePatterns` instead.'
+				'eslint-plugin-svelte: The `ignorePatterns` option in the `no-unused-props` rule has been removed. Please use `ignorePropertyPatterns` or/and `ignoreTypePatterns` instead.'
 			);
 			isDeprecationWarningShown = true;
 		}
 
 		const checkImportedTypes = options.checkImportedTypes ?? false;
-		const ignorePatterns = (options.ignorePatterns ?? []).map((p: string | RegExp) => {
+
+		const ignoreTypePatterns = (options.ignoreTypePatterns ?? []).map((p: string | RegExp) => {
 			if (typeof p === 'string') {
 				return toRegExp(p);
 			}
 			return p;
 		});
 
-		const ignoreTypePatterns = [
-			...ignorePatterns,
-			...(options.ignoreTypePatterns ?? []).map((p: string | RegExp) => {
+		const ignorePropertyPatterns = (options.ignorePropertyPatterns ?? []).map(
+			(p: string | RegExp) => {
 				if (typeof p === 'string') {
 					return toRegExp(p);
 				}
 				return p;
-			})
-		];
-
-		const ignorePropertyPatterns = [
-			...ignorePatterns,
-			...(options.ignorePropertyPatterns ?? []).map((p: string | RegExp) => {
-				if (typeof p === 'string') {
-					return toRegExp(p);
-				}
-				return p;
-			})
-		];
+			}
+		);
 
 		function shouldIgnoreProperty(name: string): boolean {
 			return ignorePropertyPatterns.some((pattern: RegExp) => pattern.test(name));