Skip to content
This repository has been archived by the owner on Jul 25, 2022. It is now read-only.

Try to launch ENSIME server #3

Merged
merged 5 commits into from
Nov 24, 2017
Merged

Try to launch ENSIME server #3

merged 5 commits into from
Nov 24, 2017

Conversation

@laughedelic laughedelic self-assigned this Nov 10, 2017
@laughedelic laughedelic added this to the ENSIME support milestone Nov 15, 2017
@laughedelic
Copy link
Owner Author

laughedelic commented Nov 20, 2017

This launches the ENSIME server, but I don't see anything working in the editor. It loads the processor, I see requests sent and log messages, but don't get any responses.. ☹️

@laughedelic laughedelic mentioned this pull request Nov 20, 2017
6 tasks
@laughedelic
Copy link
Owner Author

laughedelic commented Nov 21, 2017

I switched to the official ENSIME LSP server, it got recently merged in the 3.0 branch of the ensime-server: https://github.com/ensime/ensime-server/pull/1888 and is a replacement for the dragos' server implementation.

Here's what I got so far:

  • the server launches well with coursier
  • but it doesn't shutdown properly and you need to kill the process (see here). It leads to several problems:
    • newly launched server may fail on trying to access locked cache
    • even when you kill the zombie, new server may fail to launch (details on demand)
  • if it launches well and is running, these things work:
    • textDocument/documentSymbol shows some symbols, but the hierarchy is broken
    • textDocument/definition works quite well and even jumps to the stuff from the project deps
    • textDocument/hover works, but has some glitches (like empty tooltips of showing nonsense when hovering on comments)
    • textDocument/completion seems to work ✨ but breaks once the code doesn't compile and doesn't restore if you fix it
      all of them have problems that have to be fixed in the server, but it's not critical now
  • presentation compiler throws lots of false positives and things like Presentation compiler crashed while type checking this file, but this is probably due to that I didn't set the classpath properly..

@laughedelic
Copy link
Owner Author

I'm going to merge this as a first step towards ENSIME server support. ENSIME LSP is in improvement process: https://github.com/ensime/ensime-server/pull/1894.

Next step will be #13.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant