ruby-lsp icon indicating copy to clipboard operation
ruby-lsp copied to clipboard

Add simple completion to the `ruby-lsp`

Open vinistock opened this issue 3 years ago • 9 comments

Although our LSP won't offer first class completion like Sorbet's LSP (since it would require complete typechecking), we can still offer some simple convenience auto completes.

Some ideas, but not an exhaustive list:

  • on typing do, offer to complete with two options |arg| \n end or just end
  • on proc, lambda or -> offer to complete with { |arg| } or just {}
  • on if, offer else, elsif and complete the end
  • offer to add end on other keywords, such as while, until, for
  • on require or require_relative, offer a filtered list of possible files

Documentation: https://microsoft.github.io/language-server-protocol/specification#textDocument_completion

In addition, we can investigate how IRB completion works and evaluate if we can somehow use it from inside the Ruby LSP.

vinistock avatar Mar 29 '22 18:03 vinistock

Sorry if this is the wrong place to ask (or a silly question in general), but what is the likelihood of ruby-lsp completions becoming as thorough as the completions in irb and rails console? I find both of those tools have incredibly fast and detailed completions. After using them, I am often left longing for my text editor to work in the same way.

Example of irb giving an immense list of methods I can call on a string:

irb

Example of rails console giving a list of all the methods I can call on my Active Record model:

rails-console

Is this something that ruby-lsp and vscode-ruby-lsp could eventually achieve? Or are completions like this the responsibility of a different sort of tool?

inkyvoxel avatar Aug 30 '22 12:08 inkyvoxel

Not a silly question at all! I'm not sure how these new IRB completions work internally, but they are indeed very useful - and it would be a great experience for the Ruby LSP.

I think it could be achieved, but it does require some exploration to understand how it's done. Maybe it could even be shared between IRB and the Ruby LSP? I'll add this idea to the issue description.

vinistock avatar Aug 30 '22 13:08 vinistock

@vinistock is https://github.com/Shopify/ruby-lsp/pull/253 part of this ticket?

st0012 avatar Aug 30 '22 13:08 st0012

@st0012 no, onTypeFormatting is for formatting code as the user is typing - like automatically adding the end or adjusting indentation. This ticket is for completion on method invocations.

vinistock avatar Aug 30 '22 14:08 vinistock

offer to add end on other keywords, such as while, until, for

Ah ok. I saw #253 adds end keyword so I assumed they are the same as the above. I guess the difference is #253 always append a word based on pattern, but this one is to provide completion options?

st0012 avatar Aug 30 '22 14:08 st0012

Yes, one is for automatic formatting and the other one is to provide completion options (since we can't know ahead of time which method the user is looking for).

vinistock avatar Aug 30 '22 14:08 vinistock

Not a silly question at all! I'm not sure how these new IRB completions work internally, but they are indeed very useful - and it would be a great experience for the Ruby LSP.

I think it could be achieved, but it does require some exploration to understand how it's done. Maybe it could even be shared between IRB and the Ruby LSP? I'll add this idea to the issue description.

That is exciting! Thanks for considering it.

I was curious how the completions work in the tools I mentioned:

  • rails console loads in Rails specific libraries, then starts a new IRB instance. That is cool, I did not realise you could do that. https://github.com/rails/rails/blob/main/tools/console
  • IRB is where all the magic happens, and the completions look quite complex. It parses user input, searches some hard coded values/searches RDoc, and uses RDoc to return the output for IRB session. https://github.com/ruby/irb/blob/master/lib/irb/completion.rb

I am new to Ruby, so this may just be fluff :grin: Hopefully it can start some discussion in the future. I am interested in learning more!

inkyvoxel avatar Aug 30 '22 15:08 inkyvoxel

I think a while ago JetBrains had an experimental completion plugin for Rubymine that instantiated the application in the background and offered completions based on what is available in the runtime. This was actually painfully slow. But given Ruby's character of meta programming this is probably the only approach that can offer 100% accurate code completion. Feel free to correct me when there are better approaches available to have exact autocompletion.

gobijan avatar Sep 22 '22 20:09 gobijan

It would indeed be much more accurate, but I think it would be unbearably slow for large Rails applications. Also, it would require the LSP to know how to boot the application, since it may not necessarily be a Rails app.

If we can figure out a way to offer completions based on static analysis only, it should be a lot faster, more maintainable and application agnostic.

vinistock avatar Sep 23 '22 13:09 vinistock

it would be unbearably slow for large Rails applications

Wouldn't it be possible to cache all models and their methods, and just modify the cache on changes? doesn't sound like a huge list.

eyar avatar Jul 05 '23 20:07 eyar