LanguageServer only works with VSCode #34523

onur · 2019-12-21T20:26:27Z

Godot version: Godot Engine v3.2.beta4.official

OS/device including version: Debian GNU/Linux sid

Issue description:

I tried to use Godot Language Server with coc.nvim and LanguageClient-neovim clients. Godot is returning this error when clients sends initialize method:

ERROR: _parse_request: Not enough response headers, got: 1, expected >= 4.
   At: modules/websocket/wsl_server.cpp:49.

This is the assert line it's failing: wsl_server.cpp:49. Both clients are only sending Content-Lenght header and nothing else and godot is expecting more.

This is the request client is sending:

T 127.0.0.1:54458 -> 127.0.0.1:6008 [AP] #94
  Content-Length: 2601....                                                                                                                                                                  
##
T 127.0.0.1:54458 -> 127.0.0.1:6008 [AP] #96
  {"jsonrpc":"2.0","id":0,"method":"initialize","params":{"processId":1522387,"rootPath":"/home/onur/gamedev/04-tycoon","rootUri":"file:///home/onur/gamedev/04-tycoon","capabilities":{"wor
  kspace":{"applyEdit":true,"workspaceEdit":{"documentChanges":true,"resourceOperations":["create","rename","delete"],"failureHandling":"textOnlyTransactional"},"didChangeConfiguration":{"
  dynamicRegistration":true},"didChangeWatchedFiles":{"dynamicRegistration":true},"symbol":{"dynamicRegistration":true,"symbolKind":{"valueSet":[1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,1
  8,19,20,21,22,23,24,25,26]}},"executeCommand":{"dynamicRegistration":true},"configuration":true,"workspaceFolders":true},"textDocument":{"publishDiagnostics":{"relatedInformation":true},
  "synchronization":{"dynamicRegistration":true,"willSave":true,"willSaveWaitUntil":true,"didSave":true},"completion":{"dynamicRegistration":true,"contextSupport":true,"completionItem":{"s
  nippetSupport":true,"commitCharactersSupport":true,"documentationFormat":["markdown","plaintext"],"deprecatedSupport":true,"preselectSupport":true},"completionItemKind":{"valueSet":[1,2,
  3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21,22,23,24,25]}},"hover":{"dynamicRegistration":true,"contentFormat":["markdown","plaintext"]},"signatureHelp":{"dynamicRegistration":true
  ,"signatureInformation":{"documentationFormat":["markdown","plaintext"],"parameterInformation":{"labelOffsetSupport":true}}},"definition":{"dynamicRegistration":true},"references":{"dyna
  micRegistration":true},"documentHighlight":{"dynamicRegistration":true},"documentSymbol":{"dynamicRegistration":true,"symbolKind":{"valueSet":[1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,1
  8,19,20,21,22,23,24,25,26]}},"codeAction":{"dynamicRegistration":true,"codeActionLiteralSupport":{"codeActionKind":{"valueSet":["","quickfix","refactor","refactor.extract","refactor.inli
  ne","refactor.rewrite","source","source.organizeImports"]}}},"codeLens":{"dynamicRegistration":true},"formatting":{"dynamicRegistration":true},"rangeFormatting":{"dynamicRegistration":tr
  ue},"onTypeFormatting":{"dynamicRegistration":true},"rename":{"dynamicRegistration":true,"prepareSupport":true},"documentLink":{"dynamicRegistration":true},"typeDefinition":{"dynamicRegi
  stration":true},"implementation":{"dynamicRegistration":true},"declaration":{"dynamicRegistration":true},"colorProvider":{"dynamicRegistration":true},"foldingRange":{"dynamicRegistration
  ":true,"rangeLimit":5000,"lineFoldingOnly":true}}},"initializationOptions":{},"trace":"verbose","workspaceFolders":[{"uri":"file:///home/onur/gamedev/04-tycoon","name":"04-tycoon"}]}}   
#

cc @Faless

The text was updated successfully, but these errors were encountered:

Faless · 2019-12-22T14:04:16Z

@onur I don't know much about the LSP, but the WebSocketServer, expects a valid websocket client handshake. A valid and minimal WebSocket handshake is like this:

GET /chat HTTP/1.1
Host: example.com:8000
Upgrade: websocket
Connection: Upgrade
Sec-WebSocket-Key: dGhlIHNhbXBsZSBub25jZQ==
Sec-WebSocket-Version: 13

Maybe your LanguageClient does not use websocket as a transport for LSP?

onur · 2019-12-22T18:12:41Z

@Faless that's the issue right there, Godot's Language Server is not actually a Language Server, it is a websocket server. According to LSP Specification a Language Server only supports two headers: Content-Lenght and Content-Type. IDK why godot requires a websocketserver on top of this.

None of the clients (except godot's vscode plugin) using a protocol like this to communicate a Language Server.

Faless · 2019-12-22T20:25:29Z

Again, not an expert on LSP, but for what I understand the protocol only specifies the message format (with one header) but does not really specify what kind of connection it uses. It could be raw tcp, websocket, a Unix pipe etc. The chosen connection for Godot was websocket. I can't say if that was the best choice. For what I see other editors have websocket support too, but again I don't know much about it. Maybe @Geequlim can help here.

…

On Sun, Dec 22, 2019, 19:12 Onur Aslan ***@***.***> wrote: @Faless <https://github.com/Faless> that's the issue right there, Godot's Language Server is not actually a Language Server, it is a websocket server. According to LSP Specification <https://microsoft.github.io/language-server-protocol/specifications/specification-3-14/> a Language Server only supports two headers: Content-Lenght and Content-Type. IDK why godot requires a websocketserver on top of this. None of the clients (except godot's vscode plugin) using a protocol like this to communicate a Language Server. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#34523?email_source=notifications&email_token=AAM4C3QVCQTDB6U6B7XADOLQZ6UZVA5CNFSM4J6IRXX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHPWSEY#issuecomment-568289555>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAM4C3TUFMGGGSHUAX2DJXTQZ6UZVANCNFSM4J6IRXXQ> .

Zylann · 2019-12-23T19:17:09Z

Maybe websocket ended up being the choice because the subject was VSCode, which is built with web technology?

Geequlim · 2019-12-26T01:55:59Z

@Faless that's the issue right there, Godot's Language Server is not actually a Language Server, it is a websocket server. According to LSP Specification a Language Server only supports two headers: Content-Lenght and Content-Type. IDK why godot requires a websocketserver on top of this.

I think the LSP only specifies the message content format. It doesn't force implementations to use what kind of the connection method. I choose the websocket as it is quite simple for the implementation of both the server and the client (The VSCode extention). With websocket is is easy to allow multi-clients.

The LSP is designed to reduce the work of editor plugins which doesn't mean the client don't need to do anything.

Geequlim · 2019-12-26T02:06:13Z

And the GDScript Language Server is not work only with VSCode but also with Atom
https://github.com/PrestonKnopp/language-gdscript

If your client does not support Websocket there are two main ways to connect to the GDScript language servser

Make a middleware for connecting to GDScript language service through websocket, and connect to the client through the methods supported by your editor.
If most of client support TCP connecting out of box. We can consider replacing the websocket server with a TCP server, but the existing clients must be refactored.

habamax · 2020-01-26T12:11:45Z

If your client does not support Websocket there are two main ways to connect to the GDScript language servser

That is sad. I was waiting for godot language server thinking I would be able to use it with vim.

I guess emacs folks will not be able to use it as well. (don't remember if emacs has websocket implementation)

PS, don't get me wrong, I use vim now for gdscript and will continue using it no matter if godot language server would be implemented in a websocket only way.

PPS, thanks for language server!

Calinou · 2020-01-27T20:17:22Z

@Geequlim How much work would it be to add LSP communication using HTTP requests? It's probably a bit slower than WebSocket, but it should be far easier to integrate into most editors. If the latency difference is large enough, we can keep both implementations.

Geequlim · 2020-01-28T04:03:30Z

@Calinou I'm not sure it is possible with HTTP because both the server and the client need to send messages to each other realtime

habamax · 2020-01-28T09:22:21Z

Not sure about HTTP, but there are language servers that does both stdio and socket communications. For example, solargraph of ruby https://github.com/castwide/solargraph

 Clients can connect using either stdio or TCP

I don't know how websocket is different with regular socket, though

howdoicomputer · 2020-01-29T22:51:11Z

emacs-lsp seems to only support stdio and TCP connections

https://github.com/emacs-lsp/lsp-mode/blob/6d4c54e869a6f43ab035d6ed91f58a45aab1bee1/lsp-mode.el#L974-L980

NathanLovato · 2020-01-31T21:53:59Z

I can confirm the issue. @ofrank123 told me that according to the Language Server specification, the server should accept messages starting with the Content-Length: header, but that the GDScript LSP server does not.

lsp-mode for Emacs does only support TCP right now, but the maintainer was saying he was okay with the idea of adding web socket support. On the other hand, most language servers I've seen out there communicate over TCP. So I'd expect also most editors to expect and support TCP rather than web-sockets by default.

CyanBlob · 2020-01-31T21:56:37Z

I think moving to TCP makes more sense, given how much more popular it seems for language clients. I could potentially work on this but I wouldn't expect to get anything done anytime soon. Unfortunately, I've been pretty busy recently

NathanLovato · 2020-01-31T22:00:13Z

If there's a consensus to move to TCP and there's someone who wants to take on that, I'd gladly sponsor the work through GDQuest. That is, up to some amount at least, depending on the scope of the task, as we are running on a limited budget.

@Geequlim @Faless may either of you be interested, or may you know someone who could do it?

Geequlim · 2020-02-01T01:56:08Z

@NathanLovato For me it is about two full day of work to replace it to raw tcp for the server and the vscode plugin. But I can't find time to do it before February 21.

NathanLovato · 2020-02-01T02:17:56Z

@Geequlim Thanks for the info. 🙂

@ofrank123 said he would get a fork and PR started for that moments ago.

avahe-kellenberger · 2020-02-01T05:49:00Z

Very excited to see this being looked into, thanks everyone. If there's a way to help sponsor/support the work specifically for the language server, I'd like to know.

CyanBlob · 2020-02-03T18:34:21Z

Work is being tracked in this PR: #35864
@ofrank123 seems to be almost finished with it. Their current PR doesn't support multiple clients, but we're not sure if anybody would mind losing that functionality or not. If you do, what's your use case?

Faless · 2020-02-03T19:15:40Z

What about running two instances of vim?

…

On Mon, Feb 3, 2020, 19:34 Andrew ***@***.***> wrote: Work is being tracked in this PR: #35864 <#35864> @ofrank123 <https://github.com/ofrank123> seems to be almost finished with it. Their current PR doesn't support multiple clients, but we're not sure if anybody would mind losing that functionality or not. If you do, what's your use case? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#34523?email_source=notifications&email_token=AAM4C3WRITSEDXTKQJ4CHWDRBBPS7A5CNFSM4J6IRXX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKU46DI#issuecomment-581553933>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAM4C3QZ2PD24D3HK5I2OFLRBBPS7ANCNFSM4J6IRXXQ> .

avahe-kellenberger · 2020-02-03T19:17:49Z

Is that a common use case?

CyanBlob · 2020-02-03T19:17:56Z

@Faless Why do you need/want to run two instances of vim? Do other language servers allow this?

habamax · 2020-02-03T19:20:51Z

@Faless Why do you need/want to run two instances of vim? Do other language servers allow this?

Well this is quite common. I usually do have more than 1 vim instances open.

And yes other, I would say most of the servers are ok with it.

NathanLovato · 2020-02-03T19:22:36Z

What about running two instances of vim?

For vim it's normal to run multiple instances, e.g. use tmux or a program like tilix to have several tabs or sessions running in parallel, and to pop in and out of vim.

For emacs you typically use a daemon, as with many packages it becomes slow to load. But you can run several servers or open a single gui process as well. So no problem with that.

If that's enough to have two Godot projects open in parallel with completion, that sounds good to me.

Faless · 2020-02-03T19:40:03Z

I simply have multiple terminals tab and sometimes run multiple vim instances, this happens every day for me honestly...

…

On Mon, Feb 3, 2020, 20:17 Andrew ***@***.***> wrote: @Faless <https://github.com/Faless> Why do you need/want to run two instances of vim? Do other language servers allow thi?s — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#34523?email_source=notifications&email_token=AAM4C3WNFBEHW5NLGPFPUYDRBBUWNA5CNFSM4J6IRXX2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKVBXEY#issuecomment-581573523>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAM4C3TYDZLCKBBPGVBASXTRBBUWNANCNFSM4J6IRXXQ> .

avahe-kellenberger · 2020-02-03T19:52:32Z

I could see that when working on multiple projects at once.

CyanBlob · 2020-02-04T22:22:05Z

I just tested your new changes and it seems to be working great, multiple clients and all! Thanks for implementing this :)
I'll go ahead and add instructions for coc.nvim, since this wasn't very clear to me:
You need to add a custom language server (in CocConfig)

"languageserver": {
    "godot": {
      "host": "127.0.0.1",
      "filetypes": ["gd", "gdscript3"],
      "port": 6008
    }
  }

The filetype is not necessarily the same as the file extension. By default, vim doesn't assign a filetype to .gd files. You can set that up however you want, but this plugin does it for you, along with adding syntax highlighting: https://github.com/calviken/vim-gdscript3

ofrank123 · 2020-02-04T23:03:54Z

Awesome! Glad to hear it's working for you! I might try messing with the VSCode and Atom plugins to see if I can get them to work, but no promises as I've never worked with those editors.

0xh007 · 2020-02-10T04:41:50Z

Fantastic work everyone! I've been keeping an eye out for anything like this for probably a year now. Great to hear we've got it now. Thanks!

Shatur · 2020-02-15T23:13:32Z

@CyanBlob, it works for me too, but I also see [coc.nvim] Command: not found error after completion. Do you have this behavior too?

mlvl42 · 2020-04-18T12:22:35Z

I was making a quick and dirty port of the vscode extension to coc.nvim when I stumbled upon this issue, switching from websocket to tcp is awesome news !
While we wait for the 3.2.2 release, perhaps this could help some vim users https://github.com/j3d42/coc-godot

rafaeldelboni · 2020-04-18T12:34:32Z

I've been using Godot + Nvim, since this merge #35864, and is being a lovely experience!

Rybadour · 2020-04-22T04:36:39Z

As a fellow Vim user this is excellent news! Thanks everyone!

Calinou · 2020-07-18T21:03:44Z

This was resolved in 3.2.2 by #35864, closing. Please create new issues in editor extensions' repositories if you can encounter any.

TobiasDev · 2020-07-29T12:29:03Z

I've been using Godot + Nvim, since this merge #35864, and is being a lovely experience!

@rafaeldelboni sorry to call you out like this, but I can't get auto-complete to work in my NVIM, would you mind sharing your how you got it to work?

I added this to my Exec Flags like stated in the documentation: "+call cursor({line}, {col})" {file} and changed Exec Path to my nvim-qt.exe.

Not sure if I need anything else? I have tried both with coc-godot installed and without it. Neither of them works for me. :(

rafaeldelboni · 2020-07-29T12:36:33Z

Hey hello Tobias, how you doing? No problem, could you double check your godot version is above 3.2.2 and if your coc-settings is looking similar to mine? Btw you could double check my init.nvim, I don't use coc-godot, just vannila coc.nvim and some other vim plugins.

I tested my setup yesterday with the current godot and everything was working fine :)

https://github.com/rafaeldelboni/dotfiles/blob/299e8a9f32446b272613ff1f133dc0b0db30b2fb/config/nvim/coc-settings.json#L12

TobiasDev · 2020-07-29T12:52:46Z

Hey hello Tobias, how you doing? No problem, could you double check your godot version is above 3.2.2 and if your coc-settings is looking similar to mine? Btw you could double check my init.nvim, I don't use coc-godot, just vannila coc.nvim and some other vim plugins.

I tested my setup yesterday with the current godot and everything was working fine :)

https://github.com/rafaeldelboni/dotfiles/blob/299e8a9f32446b272613ff1f133dc0b0db30b2fb/config/nvim/coc-settings.json#L12

Wow! Thanks! Using your CocConfing instantly fixed the issue for me. :) And even started showing me warnings and errors etc.!

Thank you once again! Now I can finally use Godot and NeoVim together!!! :) This made my day.

khalid151 · 2020-08-16T20:27:31Z

I'm using neovim with coc-nvim and suggestions work but I get Command: not found upon completion (C-y).

Edit:
This is a workaround for the time being, for my <CR> mapping:

inoremap <expr> <CR> complete_info().selected != -1 ?
            \ &filetype == "gdscript" ? (coc#expandable() ?  "\<C-y>" : "\<Esc>a") : "\<C-y>"
            \ : "\<C-g>u\<CR>"

sQu1rr · 2020-11-29T10:50:55Z

@khalid151 how does it work? In particular, what does the C-g u\ does? And what does coc#expandable() do (which didn't work for me)?
I modified it to be

inoremap <expr> <CR> complete_info().selected != -1 ? "\<Esc>a" : "\<C-g>u\<CR>"

and placed it in after/ftplugin/gdscript.vim

khalid151 · 2020-11-29T11:44:23Z

@sQu1rr It first checks if there's a completion menu open and there's an item selected with complete_info().
If there is, coc#expandable() will check if the current entry is a snippet to expand it. When there is no item selected and <CR> is needed, "\<C-g>u\<CR>" is run.

As I understood, <C-g>u is there to help with undo sequence. (here's the patch info that introduced this mapping)

kovasap · 2021-06-15T01:02:34Z

Has anyone gotten this to work with neovim's native LSP (not CoC), for instance like these language servers: https://github.com/neovim/nvim-lspconfig?

beyarkay · 2023-09-03T08:35:45Z

Anyone coming from Godot 4, note that the port has changed to 6005 and you'll want to use the filetype "gdscript":

  "languageserver": {
    "godot": {
      "host": "127.0.0.1",
      "filetypes": ["gd", "gdscript"],
      "port": 6005
    }
  }

You can change your LSP settings for godot by going to Editor > Editor Settings > Network > Language Server.

And you can setup neovim as an external editor for godot by:

Navigating to the directory containing project.godot and running the command nvim --listen ./godothost
Going to the settings menu at Editor > Editor Settings > General > Text Editor > External
Tick Use external editor
Set Exec Path to either nvim or the full nvim path. My full nvim path is /opt/homebrew/bin/nvim
Set the Exec Flags to --server ./godothost --remote-send "<C-\><C-N>:n {file}<CR>{line}G{col}|"

Now, when you click to open a gdscript file in Godot, neovim will jump to that location.

Calinou added bug topic:editor labels Dec 21, 2019

akien-mga assigned Geequlim Dec 22, 2019

habamax mentioned this issue Jan 28, 2020

Support servers using different transports than stdio prabirshrestha/vim-lsp#469

Closed

howdoicomputer mentioned this issue Jan 29, 2020

Add support for GDScript emacs-lsp/lsp-mode#1361

Closed

rafaeldelboni mentioned this issue Feb 15, 2020

Switched the language server from websockets to TCP #35864

Merged

habamax mentioned this issue Feb 18, 2020

vim-lsp 2020 prabirshrestha/vim-lsp#624

Closed

15 tasks

Calinou closed this as completed Jul 18, 2020

Calinou added archived and removed archived labels Jul 18, 2020

Calinou added this to the 3.2 milestone Jul 18, 2020

This was referenced Sep 9, 2020

Now that godot 3.2.2 has been realeased should I install this extension? mlvl42/coc-godot#2

Closed

Display WebSocket error messages mlvl42/coc-godot#1

Closed

LikeLakers2 mentioned this issue Sep 16, 2020

Add a Vim mode to the script editor godotengine/godot-proposals#622

Closed

excalios mentioned this issue Dec 20, 2020

CoC Command Not Found habamax/vim-godot#15

Closed

fannheyward mentioned this issue Mar 9, 2021

Auto completion not working with godot language server. neoclide/coc.nvim#2955

Closed

avahe-kellenberger mentioned this issue Apr 9, 2021

Autocomplete erases text (using CoC) clktmr/vim-gdscript3#10

Closed

VPavliashvili mentioned this issue Jul 3, 2022

Godot section in Language Servers wiki page is suggesting obsolete plugin neoclide/coc.nvim#3910

Closed

tkgalk mentioned this issue Jan 14, 2024

LSP: textDocument/hover is not working correctly in neovim #87192

Closed

LanguageServer only works with VSCode #34523

LanguageServer only works with VSCode #34523

Comments

onur commented Dec 21, 2019

Faless commented Dec 22, 2019

onur commented Dec 22, 2019

Faless commented Dec 22, 2019 via email

Zylann commented Dec 23, 2019 • edited Loading

Geequlim commented Dec 26, 2019

Geequlim commented Dec 26, 2019

habamax commented Jan 26, 2020 • edited Loading

Calinou commented Jan 27, 2020 • edited Loading

Geequlim commented Jan 28, 2020 • edited Loading

habamax commented Jan 28, 2020

howdoicomputer commented Jan 29, 2020

NathanLovato commented Jan 31, 2020

CyanBlob commented Jan 31, 2020

NathanLovato commented Jan 31, 2020

Geequlim commented Feb 1, 2020

NathanLovato commented Feb 1, 2020

avahe-kellenberger commented Feb 1, 2020

CyanBlob commented Feb 3, 2020

Faless commented Feb 3, 2020 via email

avahe-kellenberger commented Feb 3, 2020

CyanBlob commented Feb 3, 2020 • edited Loading

habamax commented Feb 3, 2020

NathanLovato commented Feb 3, 2020 • edited Loading

Faless commented Feb 3, 2020 via email

avahe-kellenberger commented Feb 3, 2020

CyanBlob commented Feb 4, 2020 • edited Loading

ofrank123 commented Feb 4, 2020

0xh007 commented Feb 10, 2020

Shatur commented Feb 15, 2020

mlvl42 commented Apr 18, 2020

rafaeldelboni commented Apr 18, 2020

Rybadour commented Apr 22, 2020 • edited Loading

Calinou commented Jul 18, 2020

TobiasDev commented Jul 29, 2020

rafaeldelboni commented Jul 29, 2020

TobiasDev commented Jul 29, 2020

khalid151 commented Aug 16, 2020 • edited Loading

sQu1rr commented Nov 29, 2020 • edited Loading

khalid151 commented Nov 29, 2020

kovasap commented Jun 15, 2021

beyarkay commented Sep 3, 2023 • edited Loading

Zylann commented Dec 23, 2019 •

edited

Loading

habamax commented Jan 26, 2020 •

edited

Loading

Calinou commented Jan 27, 2020 •

edited

Loading

Geequlim commented Jan 28, 2020 •

edited

Loading

CyanBlob commented Feb 3, 2020 •

edited

Loading

NathanLovato commented Feb 3, 2020 •

edited

Loading

CyanBlob commented Feb 4, 2020 •

edited

Loading

Rybadour commented Apr 22, 2020 •

edited

Loading

khalid151 commented Aug 16, 2020 •

edited

Loading

sQu1rr commented Nov 29, 2020 •

edited

Loading

beyarkay commented Sep 3, 2023 •

edited

Loading