Improve autocompletion order

[WeetHet]

Check for existing issues

• Completed

Describe the feature

In VSCode rust-ananlyzer puts Vec type before vec! macro when autocompleting something like

fn main() {
    let x: V
}

In Zed autocomplete order also definitely should respect at least the case of item being autocompleted, and if possible also try rearanging completions arrording to some heuristic or maybe even some pretrained neural network, which would assign weight to completions

Also, using macros for types is quite rare, so they can be moved down

If applicable, add mockups / screenshots to help present your vision of the feature

[TENX-S]

same issue with golang:

[as-cii]

A fix for this will go out with the next release. Thanks so much for the feedback! 🙏

[mikayla-maki]

Strangely, this seems to still not be fixed.

From the discord, I verified this behavior as well:

[victor-teles]

I think it would be great if it suggest typescript primitives types first too (for me at least, I use primitives more often)

[Snap45dragon]

Similar issue in vue template files while autocompleting imported components

[DannyJJK]

Similar issue with PHP (in this case using the Laravel framework).

I start typing whereBetween however the first result on the autocomplete is orWhereBetween and the actual one I want is further down.

[mrnugget]

Just to record this here: Theo also mentioned this in his video

[DannyJJK]

This seems to affect the project file search (cmd+p) as well. Here I search for the file mi.php and it's the third option down when it should really be the first one. The first suggestion here doesn't make sense given it's only a partial match:

[mrnugget]

@DannyJJK that last one, that's a different issue: we rank recent items higher. There's a very thin line after the first 2 results. Admittedly, it's really hard to spot and we should probably improve that UI.

---

In #7727 we just made some improvements that should help with the bug that Theo reported and some of the others reported in here (string/String and UIAlert). The vec vs. Vec thing in Rust is still unchanged (and matches VS Code btw.!)

[mrnugget]

We just merged #7727 and it attempts to fix the order.

If problems like this pop up again, I think it would be best to have separate issues with clear descriptions on how to reproduce the problem and what the expected/actual behavior are, since there's a lot of nuance in how these completions are sorted and we don't want to make something worse by loosely making something else better :)

[Dorjderem2002]

Same problem happens with C++ too.

[mrnugget]

Same problem happens with C++ too.

Can you provide more information? i.e. what did you type, what did you expect to see, what did you see instead, etc.?

[Dorjderem2002]

Same problem happens with C++ too.

Can you provide more information? i.e. what did you type, what did you expect to see, what did you see instead, etc.?

Sure, when I write "for" it does not show for loop instead it shows for_each function. I assume for loop should be most used. And I never used for_each in my project or file so it is recommending it out of nowhere. Zed version 0.125.3

[mrnugget]

I can't see that screenshot, but that does sound like an issue. Like I wrote above:

I think it would be best to have separate issues with clear descriptions on how to reproduce the problem and what the expected/actual behavior are

Can you create a new ticket for this?

[bernardinorafael]

Same here, not fixed yet...

[mrnugget]

@bernardinorafael can you create a ticket with a minimal reproducible example, what you expect to see, what you get instead?