Go to Definition Misbehaves for Imported Variables in Svelte Files

[luis11011]

I'm encountering an issue with the "Go to Definition" feature in Svelte files. When I try to go to the definition of a variable that is imported from another file, instead of navigating to the variable's actual definition, the feature takes me to the point in the file where the variable is imported.

This behavior occurs consistently and disrupts my workflow, as I need to manually locate the original definition.

Steps to Reproduce:

1. Import a variable from another file into a Svelte component.
2. Use the "Go to Definition" feature on the imported variable.
3. Observe that the feature takes you to the import statement instead of the actual definition.

Expected Behavior:
The "Go to Definition" feature should navigate directly to the location in the original file where the imported variable is defined.

Actual Behavior:
The "Go to Definition" feature takes you to the import statement rather than the variable's definition.

Environment:

• Zed version: Zed Dev 0.151.0
• Extension version: 0.0.3
• Svelte version: 3.59.2
• OS: Microsoft Windows 11 Home 10.0.22631 22631

[Kae7in]

Go to definition has completely stopped working for me in .svelte files in this most release of Zed 0.149.3.

[AlbertMarashi]

This appears to no longer be an issue for me. Could you check whether Go to Definition is working for you in the newer zed versions? @Kae7in @luis11011

[olafurw]

Just installed Zed for the first time, added the svelte extension and hitting F12 or rightclick Go to Definition does nothing. Not on the component name or on the import.

[AlbertMarashi]

have you added the following to your tsconfig.json compilerOptions.plugins and run npm i typescript-svelte-plugin @olafurw

        "plugins": [
            {
                "name": "typescript-svelte-plugin",
                // the following options can be set additionally; they are optional; their default values are listed here
                "enabled": true, // enables this plugin
                "assumeIsSvelteProject": false // if true, skip detection and always assume it's a Svelte project
            }
        ]