Issues with lsp imports

[ry]

Version: Deno 1.45.5

No quickfix for installing npm dependencies

No auto-complete for node:fs/pro

Use "install" instead of "cache" everywhere. For example - change terminology from "Uncached or missing jsr dependency" to "Uninstalled". Or in the quickfix, instead of "Cache jsr:@std/fs/walk and its dependencies" say "Install jsr:@std/fs/walk and its dependencies"

Uncached deps red squiggly should probably be a yellow squiggly. Red is too aggressive.

also https://github.com/denoland/deno/issues/24951

[lucacasonato]

I agree with everything here, except:

Uncached deps red squiggly should probably be a yellow squiggly. Red is too aggressive.

It should be a red squiggly - it's the error for when you are importing something that does not exist. import "./doesnotexist.ts" should not be a yellow squiggly either.

[guy-borderless]

May be nice to also be able to deno add and have the name rewritten

[ry]

@lucacasonato

It should be a red squiggly - it's the error for when you are importing something that does not exist. import "./doesnotexist.ts" should not be a yellow squiggly either.

With auto-install common in deno, I don't think this matches the user's experience. jsr:@std/fs/walk does exist, unlike ./doesnotexist.ts. The analogy doesn't hold.

[luizfilipe]

that would be nice to be on LSP, and also with things on "imports" section on deno.json conf

[nayeemrmn]

There is an npm install quick fix when using "nodeModulesDir": "auto", which I think matches reasonable expectations. Let's close this.