eslint-plugin-svelte 3.21.0 → 3.22.0

package/README.md

@@ -297,6 +297,7 @@ These rules relate to possible syntax or logic errors in Svelte code:
 |:--------|:------------|:---|
 | [svelte/infinite-reactive-loop](https://sveltejs.github.io/eslint-plugin-svelte/rules/infinite-reactive-loop/) | Svelte runtime prevents calling the same reactive statement twice in a microtask. But between different microtask, it doesn't prevent. | :star: |
 | [svelte/no-bind-value-on-checkable-inputs](https://sveltejs.github.io/eslint-plugin-svelte/rules/no-bind-value-on-checkable-inputs/) | disallow useless `bind:value` on `<input type="checkbox">` and `<input type="radio">` | :bulb: |
+| [svelte/no-conflicting-module-names](https://sveltejs.github.io/eslint-plugin-svelte/rules/no-conflicting-module-names/) | disallow a `.svelte` component and a same-named runes module (e.g. `Foo.svelte` and `Foo.svelte.ts`) from coexisting |  |
 | [svelte/no-dom-manipulating](https://sveltejs.github.io/eslint-plugin-svelte/rules/no-dom-manipulating/) | disallow DOM manipulating | :star: |
 | [svelte/no-dupe-else-if-blocks](https://sveltejs.github.io/eslint-plugin-svelte/rules/no-dupe-else-if-blocks/) | disallow duplicate conditions in `{#if}` / `{:else if}` chains | :star: |
 | [svelte/no-dupe-on-directives](https://sveltejs.github.io/eslint-plugin-svelte/rules/no-dupe-on-directives/) | disallow duplicate `on:` directives | :star: |

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.21.0";
+    version: "3.22.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.21.0";
+export declare const version: "3.22.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.21.0';
+export const version = '3.22.0';

package/lib/rule-types.d.ts

@@ -120,6 +120,11 @@ export interface RuleOptions {
      * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/no-bind-value-on-checkable-inputs/
      */
     'svelte/no-bind-value-on-checkable-inputs'?: Linter.RuleEntry<[]>;
+    /**
+     * disallow a `.svelte` component and a same-named runes module (e.g. `Foo.svelte` and `Foo.svelte.ts`) from coexisting
+     * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/no-conflicting-module-names/
+     */
+    'svelte/no-conflicting-module-names'?: Linter.RuleEntry<[]>;
     /**
      * disallow DOM manipulating
      * @see https://sveltejs.github.io/eslint-plugin-svelte/rules/no-dom-manipulating/

package/lib/rules/no-conflicting-module-names.d.ts

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

package/lib/rules/no-conflicting-module-names.js

@@ -0,0 +1,104 @@
+import fs from 'node:fs';
+import path from 'node:path';
+import { createRule } from '../utils/index.js';
+// Extensions that plain TypeScript resolution appends to an unknown `.svelte`
+// specifier. `.d.ts` is intentionally excluded: `Foo.svelte.d.ts` is a
+// hand-written component declaration and does not shadow the component.
+const MODULE_EXTENSIONS = ['.ts', '.tsx', '.js', '.jsx', '.mts', '.cts', '.mjs', '.cjs'];
+function isFile(filePath) {
+    try {
+        return fs.statSync(filePath).isFile();
+    }
+    catch {
+        return false;
+    }
+}
+// Resolve the real on-disk casing of a path's basename. On case-insensitive
+// file systems (macOS, Windows) `fs.statSync` matches a differently-cased
+// sibling, so the constructed name may not exist with that exact casing. Read
+// the directory and use the actual entry name when one matches.
+function realBasename(filePath) {
+    const wanted = path.basename(filePath);
+    try {
+        const entries = fs.readdirSync(path.dirname(filePath));
+        return (entries.find((entry) => entry === wanted) ??
+            entries.find((entry) => entry.toLowerCase() === wanted.toLowerCase()) ??
+            wanted);
+    }
+    catch {
+        return wanted;
+    }
+}
+export default createRule('no-conflicting-module-names', {
+    meta: {
+        docs: {
+            description: 'disallow a `.svelte` component and a same-named runes module (e.g. `Foo.svelte` and `Foo.svelte.ts`) from coexisting',
+            category: 'Possible Errors',
+            recommended: false
+        },
+        schema: [],
+        messages: {
+            conflictOnComponent: 'The module `{{moduleName}}` has the same name as this component. TypeScript resolves the import `{{specifier}}` to that module, not to this component. Rename `{{moduleName}}`.',
+            conflictOnModule: 'This module has the same name as the component `{{svelteName}}`. TypeScript resolves the import `{{specifier}}` to this module, not to the component. Rename this file.'
+        },
+        type: 'problem'
+    },
+    create(context) {
+        const filename = context.physicalFilename;
+        // Skip virtual/untitled files that do not exist on disk (editors, tests
+        // with fake paths). Only real files on disk are checked.
+        if (!isFile(filename)) {
+            return {};
+        }
+        // Both sides of the collision are linted by this plugin, and a lint run
+        // may contain only one of them, so each side reports on itself.
+        if (filename.endsWith('.svelte')) {
+            return {
+                Program(node) {
+                    const svelteName = path.basename(filename);
+                    for (const ext of MODULE_EXTENSIONS) {
+                        const modulePath = `${filename}${ext}`;
+                        if (isFile(modulePath)) {
+                            context.report({
+                                node,
+                                loc: { line: 1, column: 0 },
+                                messageId: 'conflictOnComponent',
+                                data: {
+                                    moduleName: realBasename(modulePath),
+                                    specifier: `./${svelteName}`
+                                }
+                            });
+                            return;
+                        }
+                    }
+                }
+            };
+        }
+        // A module named `<name>.svelte.<ext>`. Dropping the extension gives the
+        // component path it collides with. Declaration files such as
+        // `Foo.svelte.d.ts` do not match, because dropping `.ts` leaves
+        // `Foo.svelte.d`, which is not a component name.
+        const moduleExt = MODULE_EXTENSIONS.find((ext) => filename.endsWith(ext));
+        if (moduleExt == null) {
+            return {};
+        }
+        const sveltePath = filename.slice(0, -moduleExt.length);
+        if (!sveltePath.endsWith('.svelte') || !isFile(sveltePath)) {
+            return {};
+        }
+        return {
+            Program(node) {
+                const svelteName = realBasename(sveltePath);
+                context.report({
+                    node,
+                    loc: { line: 1, column: 0 },
+                    messageId: 'conflictOnModule',
+                    data: {
+                        svelteName,
+                        specifier: `./${svelteName}`
+                    }
+                });
+            }
+        };
+    }
+});

package/lib/utils/rules.js

@@ -21,6 +21,7 @@ import noAtConstTags from '../rules/no-at-const-tags.js';
 import noAtDebugTags from '../rules/no-at-debug-tags.js';
 import noAtHtmlTags from '../rules/no-at-html-tags.js';
 import noBindValueOnCheckableInputs from '../rules/no-bind-value-on-checkable-inputs.js';
+import noConflictingModuleNames from '../rules/no-conflicting-module-names.js';
 import noDomManipulating from '../rules/no-dom-manipulating.js';
 import noDupeElseIfBlocks from '../rules/no-dupe-else-if-blocks.js';
 import noDupeOnDirectives from '../rules/no-dupe-on-directives.js';
@@ -106,6 +107,7 @@ export const rules = [
     noAtDebugTags,
     noAtHtmlTags,
     noBindValueOnCheckableInputs,
+    noConflictingModuleNames,
     noDomManipulating,
     noDupeElseIfBlocks,
     noDupeOnDirectives,

package/package.json

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