./$types resolution not working in Neovim

[AndreasNasman]

I'm looking for pointers in the right direction. I'm pretty sure this is a nvim-lspconfig problem, but I'll ask here first if the fix is something I've missed. Maybe @bartlomieju, @nayeemrmn or @vladjerca can help?

With #17248 and #27958 closed ./$types in e.g. routes/+page.ts resolves correctly to module "/<path-to-project>/.svelte-kit/types/src/routes/$types" in VS Code! 🎉

However, I can't get the same functionality working in Neovim. I've tried cloning kickstart.nvim and only adding denols in the LSP setup to cancel out the chance of me doing something stupid in my regular config, but I get the same result:

Do you have to pass some custom configuration to denols for it to resolve ./$types correctly? I compared the config defaults of vscode_deno with the config defaults of denols, but I can't find anything missing.

[thobiasn]

Experiencing the exact same issue

[AndreasNasman]

Will most likely be resolved when #28599 and #30324 are fixed.

[nayeemrmn]

@AndreasNasman Can you share what version of Deno and lsp config you're using?

[nayeemrmn]

This should work now with tsconfig detection

[alexanderameye]

I have a similar issue where it works in VS Code but not in Zed. Should I open a separate issue? Might just be an issue with Zed an unrelated to Deno.

VS Code

Zed

[alexanderameye]

It works, thanks!

[abalmos]

After a deno upgrade --canary I still see this error in SvelteKit .ts files with NeoVIM. Is there something more I need to get the deno lsp on canary?

Actually, I was able to confirm that I am running the canary. Output of `:LspInfo:'

<...snip...>
vim.lsp: Active Clients ~
- denols (id: 2)
  - Version: 2.4.5+54bda9f (release, x86_64-unknown-linux-gnu)
  - Root directory: ~/git/app/conductor
  - Command: { "/home/abalmos/.local/share/nvim/mason/bin/deno", "lsp" }
  - Settings: {
      deno = {
        enable = true,
        suggest = {
          imports = {
            hosts = {
              ["https://crux.land"] = true,
              ["https://deno.land"] = true,
              ["https://x.nest.land"] = true
            }
          }
        }
      }
    }
  - Attached buffers: 4

[nayeemrmn]

@abalmos It should work on neovim too, could you share your package.json, deno.json and tsconfig files if they exist?

[AndreasNasman]

I got it working in Neovim now with v2.5.0 of Deno and the commands below 🎉

deno run -A npm:sv create --template minimal --types ts --no-add-ons --install deno deno-sveltekit
cd deno-sveltekit
git init # Creates a root directory for Deno
deno run dev # Creates and updates `.svelte-kit` where our types live

Create a +page.ts file (e.g. this one) and save. You'll get a deno-ts: Module '"./$types"' has no exported member 'PageLoad'. [2305] error, but doing a :LspRestart fixes that. You're now only left with a deno-lint: Sloppy imports are not allowed. [no-sloppy-imports] warning.

[AndreasNasman]

deno check is still broken in SvelteKit projects: #29385