eslint-plugin-svelte 3.15.2 → 3.16.0

package/README.md

@@ -32,14 +32,33 @@
 
 `eslint-plugin-svelte` is the official [ESLint](https://eslint.org/) plugin for [Svelte](https://svelte.dev/).\
 It leverages the AST generated by [svelte-eslint-parser](https://github.com/sveltejs/svelte-eslint-parser) to provide custom linting for Svelte.\
-Note that `eslint-plugin-svelte` and `svelte-eslint-parser` cannot be used alongside [eslint-plugin-svelte3](https://github.com/sveltejs/eslint-plugin-svelte3).
+
+> [!NOTE]
+>
+> `eslint-plugin-svelte` and `svelte-eslint-parser` cannot be used alongside [eslint-plugin-svelte3](https://github.com/sveltejs/eslint-plugin-svelte3).
 
 <!--USAGE_SECTION_START-->
 <!--USAGE_GUIDE_START-->
 
 ## Installation
 
-```bash
+### CLI
+
+The recommended way to get started is to use the CLI.
+
+```sh
+# new project
+npx sv create
+
+# existing project
+npx sv add eslint
+```
+
+See the [CLI docs](https://svelte.dev/docs/cli/eslint) for more details.
+
+### Manual Setup
+
+```sh
 npm install --save-dev svelte eslint eslint-plugin-svelte globals
 ```
 
@@ -52,7 +71,7 @@ npm install --save-dev svelte eslint eslint-plugin-svelte globals
 
 ## Usage
 
-Use the `eslint.config.js` file to configure rules. For more details, see the [ESLint documentation](https://eslint.org/docs/latest/use/configure/configuration-files-new).
+Use `eslint.config.js` to configure rules. See [ESLint documentation](https://eslint.org/docs/latest/use/configure/configuration-files-new) for more details.
 
 ### Configuration
 
@@ -60,20 +79,22 @@ Use the `eslint.config.js` file to configure rules. For more details, see the [E
 
 ```js
 // eslint.config.js
+import svelteConfig from './svelte.config.js';
+import { defineConfig } from 'eslint/config';
+import globals from 'globals';
 import js from '@eslint/js';
 import svelte from 'eslint-plugin-svelte';
-import globals from 'globals';
-import svelteConfig from './svelte.config.js';
 
-/** @type {import('eslint').Linter.Config[]} */
-export default [
+export default defineConfig([
+  // ...
   js.configs.recommended,
-  ...svelte.configs.recommended,
+  svelte.configs.recommended,
   {
     languageOptions: {
       globals: {
         ...globals.browser,
-        ...globals.node // Add this if you are using SvelteKit in non-SPA mode
+        // for Sveltekit in non-SPA mode
+        ...globals.node
       }
     }
   },
@@ -81,13 +102,17 @@ export default [
     files: ['**/*.svelte', '**/*.svelte.js'],
     languageOptions: {
       parserOptions: {
-        // We recommend importing and specifying svelte.config.js.
-        // By doing so, some rules in eslint-plugin-svelte will automatically read the configuration and adjust their behavior accordingly.
-        // While certain Svelte settings may be statically loaded from svelte.config.js even if you don’t specify it,
-        // explicitly specifying it ensures better compatibility and functionality.
+        // explicitly importing allows for better compatibilty and functionality with rules and other tooling that depend on the config file.
         //
-        // If non-serializable properties are included, running ESLint with the --cache flag will fail.
-        // In that case, please remove the non-serializable properties. (e.g. `svelteConfig: { ...svelteConfig, kit: { ...svelteConfig.kit, typescript: undefined }}`)
+        // Note: `eslint --cache` will fail with non-serializable properties.
+        // In those cases, please remove the non-serializable properties.
+        // svelteConfig: {
+        //   ...svelteConfig,
+        //   kit: {
+        //     ...svelteConfig.kit,
+        //     typescript: undefined
+        //   }
+        // }
         svelteConfig
       }
     }
@@ -98,7 +123,7 @@ export default [
       // 'svelte/rule-name': 'error'
     }
   }
-];
+]);
 ```
 
 #### TypeScript project
@@ -109,23 +134,26 @@ npm install --save-dev typescript-eslint
 
 ```js
 // eslint.config.js
-import js from '@eslint/js';
-import svelte from 'eslint-plugin-svelte';
+import svelteConfig from './svelte.config.js';
+import { defineConfig } from 'eslint/config';
 import globals from 'globals';
+import js from '@eslint/js';
 import ts from 'typescript-eslint';
-import svelteConfig from './svelte.config.js';
+import svelte from 'eslint-plugin-svelte';
 
-export default ts.config(
+export default defineConfig(
   js.configs.recommended,
-  ...ts.configs.recommended,
-  ...svelte.configs.recommended,
+  ts.configs.recommended,
+  svelte.configs.recommended,
   {
     languageOptions: {
       globals: {
         ...globals.browser,
+        // for Sveltekit in non-SPA mode
         ...globals.node
       }
     }
+    // ...
   },
   {
     files: ['**/*.svelte', '**/*.svelte.ts', '**/*.svelte.js'],
@@ -133,22 +161,28 @@ export default ts.config(
     languageOptions: {
       parserOptions: {
         projectService: true,
-        extraFileExtensions: ['.svelte'], // Add support for additional file extensions, such as .svelte
-        parser: ts.parser,
+        // Enable typescript parsing for `.svelte` files.
+        extraFileExtensions: ['.svelte'],
+
         // Specify a parser for each language, if needed:
         // parser: {
         //   ts: ts.parser,
-        //   js: espree,    // Use espree for .js files (add: import espree from 'espree')
         //   typescript: ts.parser
+        //   js: espree,            // add `import espree from 'espree'`
         // },
+        parser: ts.parser,
 
-        // We recommend importing and specifying svelte.config.js.
-        // By doing so, some rules in eslint-plugin-svelte will automatically read the configuration and adjust their behavior accordingly.
-        // While certain Svelte settings may be statically loaded from svelte.config.js even if you don’t specify it,
-        // explicitly specifying it ensures better compatibility and functionality.
+        // explicitly importing allows for better compatibilty and functionality with rules and other tooling that depend on the config file.
         //
-        // If non-serializable properties are included, running ESLint with the --cache flag will fail.
-        // In that case, please remove the non-serializable properties. (e.g. `svelteConfig: { ...svelteConfig, kit: { ...svelteConfig.kit, typescript: undefined }}`)
+        // Note: `eslint --cache` will fail with non-serializable properties.
+        // In those cases, please remove the non-serializable properties.
+        // svelteConfig: {
+        //   ...svelteConfig,
+        //   kit: {
+        //     ...svelteConfig.kit,
+        //     typescript: undefined
+        //   }
+        // }
         svelteConfig
       }
     }
@@ -171,10 +205,10 @@ export default ts.config(
 
 This plugin provides the following configurations:
 
-- **`eslintPluginSvelte.configs.base`** ... Enables correct Svelte parsing.
-- **`eslintPluginSvelte.configs.recommended`** ... Includes `base` configuration, plus rules to prevent errors or unintended behavior.
-- **`eslintPluginSvelte.configs.prettier`** ... Disables rules that may conflict with [Prettier](https://prettier.io/). You still need to configure Prettier to work with Svelte manually, for example, by using [prettier-plugin-svelte](https://github.com/sveltejs/prettier-plugin-svelte).
-- **`eslintPluginSvelte.configs.all`** ... Includes all available rules. **Note:** This configuration is not recommended for production use, as it changes with every minor and major version of `eslint-plugin-svelte`. Use at your own risk.
+- **`svelte.configs.base`** - **Required** for Svelte parsing. Does not include any rules. Ideal for building a custom configurations.
+- **`svelte.configs.recommended`** - Extends `base` and includes rules for best practices.
+- **`svelte.configs.prettier`** - Extends `base` and disables rules that may conflict with [Prettier](https://prettier.io/). Prettier still needs to be configured to work with Svelte, for example, by using [prettier-plugin-svelte](https://github.com/sveltejs/prettier-plugin-svelte).
+- **`svelte.configs.all`** - **Not Recommended** - Extends `base` and includes all rules. Subject to change with every major and minor release. Use at your own risk.
 
 For more details, see [the rule list](https://sveltejs.github.io/eslint-plugin-svelte/rules/) to explore the rules provided by this plugin.
 
@@ -184,7 +218,7 @@ You can customize the behavior of this plugin using specific settings.
 
 ```js
 // eslint.config.js
-export default [
+export default defineConfig([
   // ...
   {
     settings: {
@@ -195,9 +229,10 @@ export default [
           '@typescript-eslint/no-unsafe-assignment',
           '@typescript-eslint/no-unsafe-member-access'
         ],
+
         // Specifies options for Svelte compilation.
         // This affects rules that rely on Svelte compilation,
-        // such as svelte/valid-compile and svelte/no-unused-svelte-ignore.
+        // such as `svelte/valid-compile` and `svelte/no-unused-svelte-ignore`.
         // Note that this setting does not impact ESLint’s custom parser.
         compileOptions: {
           // Specifies options related to PostCSS. You can disable the PostCSS processing by setting it to false.
@@ -206,6 +241,7 @@ export default [
             configFilePath: './path/to/my/postcss.config.js'
           }
         },
+
         // Even if settings.svelte.kit is not specified, the rules will attempt to load information from svelte.config.js.
         // However, if the default behavior does not work as expected, you should specify settings.svelte.kit explicitly.
         // If you are using SvelteKit with a non-default configuration, you need to set the following options.
@@ -220,7 +256,7 @@ export default [
     }
   }
   // ...
-];
+]);
 ```
 
 ## Editor Integrations
@@ -229,18 +265,12 @@ export default [
 Install [dbaeumer.vscode-eslint](https://marketplace.visualstudio.com/items?itemName=dbaeumer.vscode-eslint).\
 Configure `.svelte` files in `.vscode/settings.json`:
 
-```json
-{
-  "eslint.validate": ["javascript", "javascriptreact", "svelte"]
-}
-```
-
 <!--USAGE_GUIDE_END-->
 <!--USAGE_SECTION_END-->
 
 ## Migration Guide
 
-If you’re migrating from `eslint-plugin-svelte` v1 or [`@ota-meshi/eslint-plugin-svelte`](https://www.npmjs.com/package/@ota-meshi/eslint-plugin-svelte), see the [migration guide](https://sveltejs.github.io/eslint-plugin-svelte/migration/).
+If you’re migrating from `eslint-plugin-svelte@1` or [`@ota-meshi/eslint-plugin-svelte`](https://www.npmjs.com/package/@ota-meshi/eslint-plugin-svelte), see the [migration guide](https://sveltejs.github.io/eslint-plugin-svelte/migration/).
 
 ## Versioning Policy
 
@@ -341,6 +371,7 @@ These rules relate to style guidelines, and are therefore quite subjective:
 | [svelte/html-self-closing](https://sveltejs.github.io/eslint-plugin-svelte/rules/html-self-closing/) | enforce self-closing style | :wrench: |
 | [svelte/indent](https://sveltejs.github.io/eslint-plugin-svelte/rules/indent/) | enforce consistent indentation | :wrench: |
 | [svelte/max-attributes-per-line](https://sveltejs.github.io/eslint-plugin-svelte/rules/max-attributes-per-line/) | enforce the maximum number of attributes per line | :wrench: |
+| [svelte/max-lines-per-block](https://sveltejs.github.io/eslint-plugin-svelte/rules/max-lines-per-block/) | enforce maximum number of lines in svelte component blocks |  |
 | [svelte/mustache-spacing](https://sveltejs.github.io/eslint-plugin-svelte/rules/mustache-spacing/) | enforce unified spacing in mustache | :wrench: |
 | [svelte/no-extra-reactive-curlies](https://sveltejs.github.io/eslint-plugin-svelte/rules/no-extra-reactive-curlies/) | disallow wrapping single reactive statements in curly braces | :bulb: |
 | [svelte/no-restricted-html-elements](https://sveltejs.github.io/eslint-plugin-svelte/rules/no-restricted-html-elements/) | disallow specific HTML elements |  |
@@ -369,7 +400,7 @@ These rules relate to SvelteKit and its best Practices.
 | Rule ID | Description |    |
 |:--------|:------------|:---|
 | [svelte/no-export-load-in-svelte-module-in-kit-pages](https://sveltejs.github.io/eslint-plugin-svelte/rules/no-export-load-in-svelte-module-in-kit-pages/) | disallow exporting load functions in `*.svelte` module in SvelteKit page components. | :star: |
-| [svelte/no-navigation-without-resolve](https://sveltejs.github.io/eslint-plugin-svelte/rules/no-navigation-without-resolve/) | disallow using navigation (links, goto, pushState, replaceState) without a resolve() | :star: |
+| [svelte/no-navigation-without-resolve](https://sveltejs.github.io/eslint-plugin-svelte/rules/no-navigation-without-resolve/) | disallow internal navigation (links, `goto()`, `pushState()`, `replaceState()`) without a `resolve()` | :star: |
 | [svelte/valid-prop-names-in-kit-pages](https://sveltejs.github.io/eslint-plugin-svelte/rules/valid-prop-names-in-kit-pages/) | disallow props other than data or errors in SvelteKit page components. | :star: |
 
 ## Experimental

package/lib/main.d.ts

@@ -14,7 +14,7 @@ export declare const configs: {
 export declare const rules: Record<string, Rule.RuleModule>;
 export declare const meta: {
     name: "eslint-plugin-svelte";
-    version: "3.15.2";
+    version: "3.16.0";
 };
 export declare const processors: {
     '.svelte': typeof processor;

package/lib/meta.d.ts

@@ -1,2 +1,2 @@
 export declare const name: "eslint-plugin-svelte";
-export declare const version: "3.15.2";
+export declare const version: "3.16.0";

package/lib/meta.js

@@ -2,4 +2,4 @@
 // This file has been automatically generated,
 // in order to update its content execute "pnpm run update"
 export const name = 'eslint-plugin-svelte';
-export const version = '3.15.2';
+export const version = '3.16.0';

package/lib/rule-types.d.ts

@@ -85,6 +85,11 @@ export interface RuleOptions {
      * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/max-attributes-per-line/
      */
     'svelte/max-attributes-per-line'?: Linter.RuleEntry<SvelteMaxAttributesPerLine>;
+    /**
+     * enforce maximum number of lines in svelte component blocks
+     * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/max-lines-per-block/
+     */
+    'svelte/max-lines-per-block'?: Linter.RuleEntry<SvelteMaxLinesPerBlock>;
     /**
      * enforce unified spacing in mustache
      * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/mustache-spacing/
@@ -184,7 +189,7 @@ export interface RuleOptions {
      */
     'svelte/no-navigation-without-base'?: Linter.RuleEntry<SvelteNoNavigationWithoutBase>;
     /**
-     * disallow using navigation (links, goto, pushState, replaceState) without a resolve()
+     * disallow internal navigation (links, `goto()`, `pushState()`, `replaceState()`) without a `resolve()`
      * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/no-navigation-without-resolve/
      */
     'svelte/no-navigation-without-resolve'?: Linter.RuleEntry<SvelteNoNavigationWithoutResolve>;
@@ -497,6 +502,15 @@ type SvelteMaxAttributesPerLine = [] | [
         singleline?: number;
     }
 ];
+type SvelteMaxLinesPerBlock = [] | [
+    {
+        script?: number;
+        template?: number;
+        style?: number;
+        skipBlankLines?: boolean;
+        skipComments?: boolean;
+    }
+];
 type SvelteMustacheSpacing = [] | [
     {
         textExpressions?: ("never" | "always");

package/lib/rules/max-lines-per-block.d.ts

@@ -0,0 +1,2 @@
+declare const _default: import("../types.js").RuleModule;
+export default _default;

package/lib/rules/max-lines-per-block.js

@@ -0,0 +1,232 @@
+import { Input } from 'postcss';
+import tokenize from 'postcss/lib/tokenize';
+import { createRule } from '../utils/index.js';
+/** Check if a comment occupies the entire source line (matching ESLint core max-lines behavior). */
+function isFullLineComment(line, lineNumber, loc) {
+    return ((loc.start.line < lineNumber || !line.slice(0, loc.start.column).trim()) &&
+        (loc.end.line > lineNumber || !line.slice(loc.end.column).trim()));
+}
+/** Collect line numbers where AST comments occupy the full line. */
+function collectAstCommentLines(comments, sourceLines, startLine, endLine) {
+    const lines = new Set();
+    for (const comment of comments) {
+        if (comment.loc.end.line < startLine || comment.loc.start.line > endLine)
+            continue;
+        for (let i = Math.max(comment.loc.start.line, startLine); i <= Math.min(comment.loc.end.line, endLine); i++) {
+            if (isFullLineComment(sourceLines[i - 1], i, comment.loc)) {
+                lines.add(i);
+            }
+        }
+    }
+    return lines;
+}
+/** Collect line numbers where CSS comments occupy the full line, using postcss tokenizer. */
+function collectCssCommentLines(sourceLines, startLine, endLine) {
+    const result = new Set();
+    const cssText = sourceLines.slice(startLine - 1, endLine).join('\n');
+    if (!cssText.trim())
+        return result;
+    try {
+        const input = new Input(cssText);
+        const tk = tokenize(input);
+        const commentLines = new Set();
+        const codeLines = new Set();
+        let token;
+        while ((token = tk.nextToken())) {
+            if (token[2] == null)
+                continue;
+            const startPos = input.fromOffset(token[2]);
+            if (!startPos)
+                continue;
+            const tokenLine = startPos.line + startLine - 1;
+            if (token[0] === 'comment') {
+                const endPos = token[3] != null ? input.fromOffset(token[3]) : null;
+                const commentEndLine = endPos ? endPos.line + startLine - 1 : tokenLine;
+                for (let i = tokenLine; i <= commentEndLine; i++) {
+                    commentLines.add(i);
+                }
+            }
+            else {
+                codeLines.add(tokenLine);
+            }
+        }
+        for (const line of commentLines) {
+            if (!codeLines.has(line))
+                result.add(line);
+        }
+    }
+    catch {
+        // Malformed CSS — don't skip any lines
+    }
+    return result;
+}
+/** Count inner content lines, skipping blanks and/or comment lines. */
+function countLines(sourceLines, startLine, endLine, skipBlankLines, commentLines) {
+    if (endLine - startLine <= 1)
+        return 0;
+    let count = 0;
+    for (let i = startLine + 1; i < endLine; i++) {
+        if (skipBlankLines && sourceLines[i - 1].trim().length === 0)
+            continue;
+        if (commentLines.has(i))
+            continue;
+        count++;
+    }
+    return count;
+}
+function isSvelteOptions(node) {
+    return node.name.type === 'SvelteName' && node.name.name === 'svelte:options';
+}
+export default createRule('max-lines-per-block', {
+    meta: {
+        docs: {
+            description: 'enforce maximum number of lines in svelte component blocks',
+            category: 'Stylistic Issues',
+            recommended: false,
+            conflictWithPrettier: false
+        },
+        schema: [
+            {
+                type: 'object',
+                properties: {
+                    script: {
+                        type: 'integer',
+                        minimum: 1
+                    },
+                    template: {
+                        type: 'integer',
+                        minimum: 1
+                    },
+                    style: {
+                        type: 'integer',
+                        minimum: 1
+                    },
+                    skipBlankLines: {
+                        type: 'boolean'
+                    },
+                    skipComments: {
+                        type: 'boolean'
+                    }
+                },
+                additionalProperties: false
+            }
+        ],
+        messages: {
+            tooManyLines: '{{block}} block has too many lines ({{lineCount}}). Maximum allowed is {{max}}.'
+        },
+        type: 'suggestion'
+    },
+    create(context) {
+        const options = context.options[0] ?? {};
+        const scriptMax = options.script;
+        const templateMax = options.template;
+        const styleMax = options.style;
+        const skipBlankLines = options.skipBlankLines ?? false;
+        const skipComments = options.skipComments ?? false;
+        const sourceCode = context.sourceCode;
+        const htmlCommentNodes = [];
+        const emptySet = new Set();
+        return {
+            SvelteHTMLComment(node) {
+                htmlCommentNodes.push(node);
+            },
+            SvelteScriptElement(node) {
+                if (scriptMax == null)
+                    return;
+                const commentLines = skipComments
+                    ? collectAstCommentLines(sourceCode.getAllComments(), sourceCode.lines, node.loc.start.line + 1, node.loc.end.line - 1)
+                    : emptySet;
+                const lineCount = countLines(sourceCode.lines, node.loc.start.line, node.loc.end.line, skipBlankLines, commentLines);
+                if (lineCount > scriptMax) {
+                    context.report({
+                        node,
+                        messageId: 'tooManyLines',
+                        data: {
+                            block: '<script>',
+                            lineCount: String(lineCount),
+                            max: String(scriptMax)
+                        }
+                    });
+                }
+            },
+            SvelteStyleElement(node) {
+                if (styleMax == null)
+                    return;
+                const commentLines = skipComments
+                    ? collectCssCommentLines(sourceCode.lines, node.loc.start.line + 1, node.loc.end.line - 1)
+                    : emptySet;
+                const lineCount = countLines(sourceCode.lines, node.loc.start.line, node.loc.end.line, skipBlankLines, commentLines);
+                if (lineCount > styleMax) {
+                    context.report({
+                        node,
+                        messageId: 'tooManyLines',
+                        data: {
+                            block: '<style>',
+                            lineCount: String(lineCount),
+                            max: String(styleMax)
+                        }
+                    });
+                }
+            },
+            'Program:exit'(program) {
+                if (templateMax == null)
+                    return;
+                const totalLines = sourceCode.lines.length;
+                // Exclude lines occupied by <script>, <style>, and <svelte:options>
+                const excludedLines = new Set();
+                for (const child of program.body) {
+                    if (child.type === 'SvelteScriptElement' ||
+                        child.type === 'SvelteStyleElement' ||
+                        (child.type === 'SvelteElement' && isSvelteOptions(child))) {
+                        for (let i = child.loc.start.line; i <= child.loc.end.line; i++) {
+                            excludedLines.add(i);
+                        }
+                    }
+                }
+                // Collect full-line comment lines for template region
+                const commentLines = new Set();
+                if (skipComments) {
+                    const allComments = [
+                        ...htmlCommentNodes,
+                        ...sourceCode.getAllComments()
+                    ];
+                    for (const comment of allComments) {
+                        for (let i = comment.loc.start.line; i <= comment.loc.end.line; i++) {
+                            if (excludedLines.has(i))
+                                continue;
+                            if (isFullLineComment(sourceCode.lines[i - 1], i, comment.loc)) {
+                                commentLines.add(i);
+                            }
+                        }
+                    }
+                }
+                let templateLineCount = 0;
+                for (let i = 1; i <= totalLines; i++) {
+                    if (excludedLines.has(i))
+                        continue;
+                    if (skipBlankLines && sourceCode.lines[i - 1].trim().length === 0)
+                        continue;
+                    if (commentLines.has(i))
+                        continue;
+                    templateLineCount++;
+                }
+                if (templateLineCount > templateMax) {
+                    const firstTemplateNode = program.body.find((child) => child.type !== 'SvelteScriptElement' &&
+                        child.type !== 'SvelteStyleElement' &&
+                        !(child.type === 'SvelteElement' && isSvelteOptions(child)));
+                    if (firstTemplateNode) {
+                        context.report({
+                            node: firstTemplateNode,
+                            messageId: 'tooManyLines',
+                            data: {
+                                block: 'template',
+                                lineCount: String(templateLineCount),
+                                max: String(templateMax)
+                            }
+                        });
+                    }
+                }
+            }
+        };
+    }
+});

package/lib/rules/no-navigation-without-resolve.js

@@ -5,7 +5,7 @@ import { findVariable } from '../utils/ast-utils.js';
 export default createRule('no-navigation-without-resolve', {
     meta: {
         docs: {
-            description: 'disallow using navigation (links, goto, pushState, replaceState) without a resolve()',
+            description: 'disallow internal navigation (links, `goto()`, `pushState()`, `replaceState()`) without a `resolve()`',
             category: 'SvelteKit',
             recommended: true
         },

package/lib/shared/svelte-compile-warns/ignore-comment.js

@@ -1,4 +1,6 @@
 const SVELTE_IGNORE_PATTERN = /^\s*svelte-ignore\s+/;
+/** Matches parenthetical notes in svelte-ignore comments, e.g. "(because of reasons)" */
+const PARENTHETICAL_NOTE_PATTERN = /\([^)]*\)/gu;
 /**
  * Map of legacy code -> new code
  * See https://github.com/sveltejs/svelte/blob/c9202a889612df3c2fcb369096a5573668be99d6/packages/svelte/src/compiler/utils/extract_svelte_ignore.js#L6
@@ -70,11 +72,14 @@ export function getSvelteIgnoreItems(context) {
 function extractSvelteIgnore(startIndex, token, codeList, ignoreStart) {
     const start = startIndex + ignoreStart;
     const results = [];
+    // Replace parenthetical notes (e.g., "(because of reasons)") with spaces of the same length
+    // to preserve character positions while preventing note words from being treated as rule names.
+    const processedCodeList = codeList.replace(PARENTHETICAL_NOTE_PATTERN, (match) => ' '.repeat(match.length));
     const separatorPattern = /\s*[\s,]\s*/g;
-    const separators = codeList.matchAll(separatorPattern);
+    const separators = processedCodeList.matchAll(separatorPattern);
     let lastSeparatorEnd = 0;
     for (const separator of separators) {
-        const code = codeList.slice(lastSeparatorEnd, separator.index);
+        const code = processedCodeList.slice(lastSeparatorEnd, separator.index);
         if (code) {
             results.push({
                 code,
@@ -86,7 +91,7 @@ function extractSvelteIgnore(startIndex, token, codeList, ignoreStart) {
         lastSeparatorEnd = separator.index + separator[0].length;
     }
     if (results.length === 0) {
-        const code = codeList;
+        const code = processedCodeList;
         results.push({
             code,
             codeForV5: V5_REPLACEMENTS[code] || code.replace(/-/gu, '_'),
@@ -98,5 +103,5 @@ function extractSvelteIgnore(startIndex, token, codeList, ignoreStart) {
 }
 /** Checks whether given comment has missing code svelte-ignore */
 function hasMissingCodeIgnore(codeList) {
-    return !codeList.trim();
+    return !codeList.replace(PARENTHETICAL_NOTE_PATTERN, '').trim();
 }

package/lib/utils/rules.js

@@ -14,6 +14,7 @@ import htmlSelfClosing from '../rules/html-self-closing.js';
 import indent from '../rules/indent.js';
 import infiniteReactiveLoop from '../rules/infinite-reactive-loop.js';
 import maxAttributesPerLine from '../rules/max-attributes-per-line.js';
+import maxLinesPerBlock from '../rules/max-lines-per-block.js';
 import mustacheSpacing from '../rules/mustache-spacing.js';
 import noAddEventListener from '../rules/no-add-event-listener.js';
 import noAtDebugTags from '../rules/no-at-debug-tags.js';
@@ -94,6 +95,7 @@ export const rules = [
     indent,
     infiniteReactiveLoop,
     maxAttributesPerLine,
+    maxLinesPerBlock,
     mustacheSpacing,
     noAddEventListener,
     noAtDebugTags,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-svelte",
-  "version": "3.15.2",
+  "version": "3.16.0",
   "description": "ESLint plugin for Svelte using AST",
   "repository": {
     "type": "git",