Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

URI must contain a scheme #355

Closed
frangio opened this issue Jan 3, 2023 · 1 comment · Fixed by #360
Closed

URI must contain a scheme #355

frangio opened this issue Jan 3, 2023 · 1 comment · Fixed by #360
Assignees
Labels
status:ready This issue is ready to be worked on type:bug Something isn't working

Comments

@frangio
Copy link

frangio commented Jan 3, 2023

I know this isn't yet fully supported but I'm trying to run the server directly with Neovim's built in LSP client.

I'm seeing this error that seems to prevent the server from working at all:

URI must contain a scheme: /home/[...]/AccessControl.sol

This seems to be triggered by:

[TRACE][2023-01-03 17:34:03] .../lua/vim/lsp.lua:1045	"notification"	"textDocument/publishDiagnostics"	{  diagnostics = {},  uri = "/home/[...]/AccessControl.sol"}

Neovim seems to expect that the uri field will start with file:// or some other scheme.

@antico5
Copy link
Collaborator

antico5 commented Jan 3, 2023

Good catch. We should be using URIs instead of paths on the publishDiagnostics notification. My guess is that both vscode and coc.nvim assume the scheme is file but we should make it explicit to be more compliant with the protocol.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:ready This issue is ready to be worked on type:bug Something isn't working
Projects
Status: Done
3 participants