breaking: redesigned the config

Author: baseballyama

.vscode/settings.json

@@ -4,6 +4,7 @@
 	"editor.defaultFormatter": "esbenp.prettier-vscode",
 	"editor.codeActionsOnSave": {
 		"source.fixAll": "explicit",
+		"source.fixAll.eslint": "explicit",
 		"source.fixAll.stylelint": "explicit"
 	},
 	"svelte.plugin.typescript.diagnostics.enable": false,

README.md

@@ -79,7 +79,7 @@
     "@types/json-schema": "^7.0.15",
     "@types/less": "^3.0.7",
     "@types/mocha": "^10.0.10",
-    "@types/node": "^22.10.0",
+    "@types/node": "^22.10.2",
     "@types/postcss-safe-parser": "^5.0.4",
     "@types/semver": "^7.5.8",
     "@types/stylus": "^0.48.43",
@@ -91,12 +91,12 @@
     "eslint-visitor-keys": "^4.2.0",
     "espree": "^10.3.0",
     "less": "^4.2.1",
-    "mocha": "^11.0.0",
+    "mocha": "^11.0.1",
     "postcss-nested": "^7.0.2",
-    "sass": "^1.81.0",
+    "sass": "^1.83.0",
     "source-map-js": "^1.2.1",
     "stylus": "^0.64.0",
-    "svelte": "^5.2.9",
+    "svelte": "^5.12.0",
     "svelte-i18n": "^4.0.1",
     "tsx": "^4.19.2",
     "type-coverage": "^2.29.7",

docs/rules.md

@@ -9,7 +9,6 @@ const config: Linter.Config[] = [
 		name: 'svelte:recommended:rules',
 		rules: {
 			// eslint-plugin-svelte rules
-			'svelte/comment-directive': 'error',
 			'svelte/no-at-debug-tags': 'warn',
 			'svelte/no-at-html-tags': 'error',
 			'svelte/no-dupe-else-if-blocks': 'error',
@@ -21,7 +20,6 @@ const config: Linter.Config[] = [
 			'svelte/no-shorthand-style-property-overrides': 'error',
 			'svelte/no-unknown-style-directive-property': 'error',
 			'svelte/no-unused-svelte-ignore': 'error',
-			'svelte/system': 'error',
 			'svelte/valid-compile': 'error'
 		}
 	}

docs/rules/comment-directive.md

@@ -95,7 +95,7 @@ export default createRule('@typescript-eslint/no-unnecessary-condition', {
 		docs: {
 			description: 'disallow conditionals where the type is always truthy or always falsy',
 			category: 'Extension Rules',
-			recommended: false,
+			configNames: [],
 			extensionRule: {
 				plugin: '@typescript-eslint/eslint-plugin',
 				url: 'https://typescript-eslint.io/rules/no-unnecessary-condition/'

docs/rules/no-at-debug-tags.md

@@ -9,7 +9,7 @@ export default createRule('block-lang', {
 			description:
 				'disallows the use of languages other than those specified in the configuration for the lang attribute of `<script>` and `<style>` blocks.',
 			category: 'Best Practices',
-			recommended: false
+			configNames: []
 		},
 		schema: [
 			{

docs/rules/no-at-html-tags.md

@@ -18,7 +18,7 @@ export default createRule('button-has-type', {
 		docs: {
 			description: 'disallow usage of button without an explicit type attribute',
 			category: 'Best Practices',
-			recommended: false
+			configNames: []
 		},
 		schema: [
 			{

docs/rules/no-dupe-else-if-blocks.md

@@ -32,7 +32,7 @@ export default createRule('comment-directive', {
 		docs: {
 			description: 'support comment-directives in HTML template',
 			category: 'System',
-			recommended: 'base'
+			configNames: ['base']
 		},
 		schema: [
 			{

docs/rules/no-dupe-style-properties.md

@@ -8,7 +8,7 @@ export default createRule('derived-has-same-inputs-outputs', {
 		docs: {
 			description: 'derived store should use same variable names between values and callback',
 			category: 'Stylistic Issues',
-			recommended: false,
+			configNames: [],
 			conflictWithPrettier: false
 		},
 		schema: [],

docs/rules/no-dynamic-slot-name.md

@@ -8,7 +8,7 @@ export default createRule('experimental-require-slot-types', {
 		docs: {
 			description: 'require slot type declaration using the `$$Slots` interface',
 			category: 'Experimental',
-			recommended: false
+			configNames: []
 		},
 		schema: [],
 		messages: {

docs/rules/no-inner-declarations.md

@@ -10,7 +10,7 @@ export default createRule('experimental-require-strict-events', {
 		docs: {
 			description: 'require the strictEvents attribute on `<script>` tags',
 			category: 'Experimental',
-			recommended: false
+			configNames: []
 		},
 		schema: [],
 		messages: {

docs/rules/no-not-function-handler.md

@@ -7,7 +7,7 @@ export default createRule('first-attribute-linebreak', {
 		docs: {
 			description: 'enforce the location of first attribute',
 			category: 'Stylistic Issues',
-			recommended: false,
+			configNames: [],
 			conflictWithPrettier: true
 		},
 		fixable: 'whitespace',

docs/rules/no-object-in-text-mustaches.md

@@ -89,7 +89,7 @@ export default createRule('html-closing-bracket-new-line', {
 		docs: {
 			description: "Require or disallow a line break before tag's closing brackets",
 			category: 'Stylistic Issues',
-			recommended: false,
+			configNames: [],
 			conflictWithPrettier: true
 		},
 		schema: [

docs/rules/no-shorthand-style-property-overrides.md

@@ -7,7 +7,7 @@ export default createRule('html-closing-bracket-spacing', {
 			description: "require or disallow a space before tag's closing brackets",
 			category: 'Stylistic Issues',
 			conflictWithPrettier: true,
-			recommended: false
+			configNames: []
 		},
 		schema: [
 			{

docs/rules/no-unknown-style-directive-property.md

@@ -20,7 +20,7 @@ export default createRule('html-quotes', {
 		docs: {
 			description: 'enforce quotes style of HTML attributes',
 			category: 'Stylistic Issues',
-			recommended: false,
+			configNames: [],
 			conflictWithPrettier: true
 		},
 		fixable: 'code',

docs/rules/no-unused-svelte-ignore.md

@@ -18,7 +18,7 @@ export default createRule('html-self-closing', {
 		docs: {
 			description: 'enforce self-closing style',
 			category: 'Stylistic Issues',
-			recommended: false,
+			configNames: [],
 			conflictWithPrettier: true
 		},
 		type: 'layout',

docs/rules/system.md

@@ -6,7 +6,7 @@ export default createRule('indent', {
 		docs: {
 			description: 'enforce consistent indentation',
 			category: 'Stylistic Issues',
-			recommended: false,
+			configNames: [],
 			conflictWithPrettier: true
 		},
 		fixable: 'whitespace',

docs/rules/valid-compile.md

@@ -370,7 +370,7 @@ export default createRule('infinite-reactive-loop', {
 				"Svelte runtime prevents calling the same reactive statement twice in a microtask. But between different microtask, it doesn't prevent.",
 			category: 'Possible Errors',
 			// TODO Switch to recommended in the major version.
-			recommended: false
+			configNames: []
 		},
 		schema: [],
 		messages: {

packages/eslint-plugin-svelte/package.json

@@ -30,7 +30,7 @@ export default createRule('max-attributes-per-line', {
 		docs: {
 			description: 'enforce the maximum number of attributes per line',
 			category: 'Stylistic Issues',
-			recommended: false,
+			configNames: [],
 			conflictWithPrettier: true
 		},
 		fixable: 'whitespace',

packages/eslint-plugin-svelte/src/configs/flat/recommended.ts

@@ -39,7 +39,7 @@ export default createRule('mustache-spacing', {
 		docs: {
 			description: 'enforce unified spacing in mustache',
 			category: 'Stylistic Issues',
-			recommended: false,
+			configNames: [],
 			conflictWithPrettier: true
 		},
 		fixable: 'code',

packages/eslint-plugin-svelte/src/rules/@typescript-eslint/no-unnecessary-condition.ts

@@ -5,7 +5,7 @@ export default createRule('no-at-debug-tags', {
 		docs: {
 			description: 'disallow the use of `{@debug}`',
 			category: 'Best Practices',
-			recommended: true,
+			configNames: ['recommended'],
 			default: 'warn'
 		},
 		schema: [],

packages/eslint-plugin-svelte/src/rules/block-lang.ts

@@ -6,7 +6,7 @@ export default createRule('no-at-html-tags', {
 		docs: {
 			description: 'disallow use of `{@html}` to prevent XSS attack',
 			category: 'Security Vulnerability',
-			recommended: true
+			configNames: ['recommended']
 		},
 		schema: [],
 		messages: {

packages/eslint-plugin-svelte/src/rules/button-has-type.ts

@@ -10,7 +10,7 @@ export default createRule('no-deprecated-raw-special-elements', {
 			description: 'Recommends not using raw special elements in Svelte versions previous to 5.',
 			category: 'Possible Errors',
 			// TODO: Switch to recommended in the major version
-			recommended: false
+			configNames: []
 		},
 		schema: [],
 		messages: {

packages/eslint-plugin-svelte/src/rules/comment-directive.ts

@@ -35,7 +35,7 @@ export default createRule('no-dom-manipulating', {
 		docs: {
 			description: 'disallow DOM manipulating',
 			category: 'Possible Errors',
-			recommended: false
+			configNames: []
 		},
 		schema: [],
 		messages: {

packages/eslint-plugin-svelte/src/rules/derived-has-same-inputs-outputs.ts

@@ -72,7 +72,7 @@ export default createRule('no-dupe-else-if-blocks', {
 		docs: {
 			description: 'disallow duplicate conditions in `{#if}` / `{:else if}` chains',
 			category: 'Possible Errors',
-			recommended: true
+			configNames: ['recommended']
 		},
 		schema: [],
 		messages: {

packages/eslint-plugin-svelte/src/rules/experimental-require-slot-types.ts

@@ -9,7 +9,7 @@ export default createRule('no-dupe-on-directives', {
 		docs: {
 			description: 'disallow duplicate `on:` directives',
 			category: 'Possible Errors',
-			recommended: false
+			configNames: []
 		},
 		schema: [],
 		messages: {