textDocument/definition not working

Hi nice work,

I have some problems using the goto definition feature. My setup include nvim + LanguageClient (or even builtin LSP) and jedi-language-server.

The problem in my case is that when calling LanguageClient#textDocument_definition() the cursor do not move, nvim complains as:

[LC] data did not match any variant of untagged enum GotoDefinitionResponse

and the client log is as follows:

17:04:44 DEBUG reader-Some("python") src/rpcclient.rs:207 <= Some("python") {"id": 1, "jsonrpc": "2.0", "result": [{"uri": null, "range": {"start": {"line": 0, "character": 4}, "end": {"line": 0, "character": 7}}}]}
17:04:44 ERROR unnamed src/rpchandler.rs:27 Error handling message: data did not match any variant of untagged enum GotoDefinitionResponse
Message: {"jsonrpc":"2.0","method":"textDocument/definition","params":{"bufnr":1,"character":0,"filename":"/home/mauro/t.py","gotoCmd":null,"handle":true,"languageId":"python","line":4,"text":["def foo():","    print('FOO')","","","foo()",""]},"id":1}
Error: data did not match any variant of untagged enum GotoDefinitionResponse

In particular note that it seams that the server do not send back the URI to the file, instead there is a null entry, and that with the method textDocument/publishDiagnostics, the server send back the right URI to the file position.

I found the same problem with anakin https://github.com/muffinmad/anakin-language-server/issues/17 but not with other servers such as python-language-server.

[pappasam]

On second thought, since this tool targets the latest version of Jedi, I’ll just wait for the next version of Jedi and use the path objects directly (pinning the latest Jedi version in that release). from_fs_path is unnecessary when working with pathlib.Path objects; it provides its own as_uri method.

Anyway, thanks for raising this and I look forward to Jedi’s next version’s stabilization!

[pappasam]

resolved in the latest published release (0.22.0)!