Hi guys, sometimes I try something new to compare with already used packages, this weekend I wanted to try Eglot, however, I ran into a number of problems. One of them is autocomplete.
For example, when I write in vue, I want to autocomplete a watch function. This function is available in the following packages
import { watch } from 'vue';
import { watch } from 'fs';
import { watch } from 'fs/promises'
... etc
But when I use autocomplete in eglot I only see the first candidate - importing from the fs package. Is there any possibility or workaround to display all possible candidates?
Example of autocomplete with eglot
Example of autocomplete with lsp-mode
I also know that lsp-mode in conjunction with corfu has a similar problem, but it is impossible to solve it there as the authors of both packages think that the problem is not on their side 😅 Is the situation with eglot the same?
/u/hvis Would you consider changing this, since this way of deduplication in the frontend is quite inefficient? Given that the default completion UI demands that candidates are unique with respect to
equaland even deletesequalduplicates, the backend should better produce unique candidates.Deduplicating in the backend is not difficult, all that is needed is adding a suffix like (1), (2), (3), … In fact, this is what I am doing in my Consult package, which also is a backend and provides completion tables. For example for the
consult-linecommand I was facing the same issue, that candidates must be unique with respect toequaland not toeq.cc /u/JDRiverRun
I’d also worry that an eq test is a bit fragile, and could go mysteriously wrong if any step in the chain decided down the road to copy or duplicate the string.
Something like that would be generally easy to see very soon (completion popup without both icons and annotations). This approach has worked for years for both Emacs’s default UI and company-mode, so it’s hard to call it fragile.
I suppose it might have been a cause for investigation for some backend authors at some point, but backends would generally avoid internal copying anyway, for performance reasons if nothing else.
It is not only fragile, it is also wrong, since the default completion UI even deletes duplicate candidates.
Actually, I forgot how this works in most cases X-D.
And it happens through text properties, which backends add to their completion strings. Then the
annotationandkindoperations look them up.Makes sense. I thought in the past the default completion system stripped out text properties? But I dimly recall that situation changed recently (v29?).
I simply consider it part of the requirements (see the other message). And there has been some effort made to ensure that the annotation/kind functions are only called when equal strings are encountered.
Indeed, when the list is long deduplication does show up on the graph, but the impact also depends on the shape of the data, and there probably are some untapped code optimizations still.
I’m not sure this is very easy (e.g. for LSP clients), and in general it would require a scan across all completions of comparable complexity. Also, showing method overloads with suffixes like 1/2/3 would look rather odd, I think.