Add simple completion to the `ruby-lsp`
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 endor justend - on
proc,lambdaor->offer to complete with{ |arg| }or just{} - on
if, offerelse,elsifand complete theend - offer to add
endon other keywords, such aswhile,until,for - on
requireorrequire_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.
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:

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

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?
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 is https://github.com/Shopify/ruby-lsp/pull/253 part of this ticket?
@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.
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?
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).
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 consoleloads 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!
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.
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.
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.