Support inlay hints

[kitsonk]

TypeScript 4.4 which will be in Deno 1.14 supports inlay hints: https://devblogs.microsoft.com/typescript/announcing-typescript-4-4/#inlay-hints

We need to plumb that through in the LSP and integrate it into the extension.

[kitsonk]

Sadly, InlayHints are "experimental" in vscode at the moment, and there is no way to publish an extension that uses them, until the APIs are moved to stable. Even then, they are currently not part of the language server protocol. That isn't specifically a limitation as we can likely add the feature as a custom extension until it gets added.

[kitsonk]

Now vscode has shipped, but the API is still proposed, meaning we can't publish an extension that uses it.

Issue requesting it be made available to other extension authors: microsoft/vscode#132197
There was an already existing issue to add it to the language server protocol, though it doesn't look like there was any movement of adding the vscode implementation to the specification ATM: microsoft/language-server-protocol#956

[ghost]

I have been using inlay hints in vscode for node projects and it's really useful. Any plans on supporting this is deno?

[kitsonk]

It is blocked by it being added to the LSP specification.

[MihaelBercic]

It is blocked by it being added to the LSP specification.

What is the solution to be able to use inlay hints with deno in the end?

[sno2]

The next LSP specification version will include inlay hints.

[kitsonk]

The new APIs need to be added to lsp-types and then the changes need to be integrated into lspower. Once that is done, it should be possible to make them available in the lsp. I would get around to doing that eventually, but it is a good opportunity for community contribution, and fairly straight forward changes. Also contributions to lsp-types and lspower benefit a lot of Rust community language server implementations, not just Deno.

[sno2]

@kitsonk It seems that lspower is now deprecated as of 12 days ago in favor of tower-lsp FYI

[kitsonk]

ah, ok... I should have paid closer attention... anyways, it needs to be integrated into tower-lsp then!

[kitsonk]

Closed by denoland/deno#16287, #739 and #740