[monaco][quick-open] Using fuzzySort: true throws an error

[RomanNikitenko]

Steps to Reproduce:

1. F1 --> Variable:List All
2. You can see empty list and an error in browser console

Actual behavior:

Expected behavior:

Additional Information

This place should be revisited at the next upgrade Monaco version.

The workaround is: use here false instead.

• Operating System: Fedora 31
• Theia Version: 84408de

[akosyakov]

@RomanNikitenko Do you take care about it?

[RomanNikitenko]

@akosyakov
Do you mean apply the workaround or more complex solution?

Within upgrade Monaco I'm going to investigate what VS Code uses instead compareEntries. But my question - should we apply the workaround right now, before upgrade Monaco.
Or maybe another solution is available before upgrade Monaco.

[akosyakov]

Do you mean apply the workaround or more complex solution?

I hoped for the solution. If not I can have a look. Upgrade will take a while.

[RomanNikitenko]

@akosyakov
I didn't find a good solution quickly and I don't see how to fix it before Theia release, that's why I created the issue.
So if you see a solution, please do it.

[danieltomasku]

I am also getting the same error when using the QuickOpenService with fuzzySort: true:

[RomanNikitenko]

I investigated the problem, this function was removed from VS Code.

The function provided ability to sort quick open entries respecting highlight info.
So, it contained two parts:

• label highlights comparison
• compareAnything function

I tried to solve the problem by :

• provide on Theia side logic to compare label highlights
• then use compareAnything function of VS Code

But I faced with the problem: Monaco loader did not bring comparers despite I added the corresponding items to Monaco loader here, so we can not reuse compareAnything function of VS Code.

[akosyakov]

We coudl either drop it if it is not used much or copy missing functions via CQs.