"Quick fix" for Import on VSCode seems to modify random code

[Tanin Na Nakorn]

Describe the bug

To Reproduce Steps to reproduce the behavior:

Then, if we choose the second quickfix, the previewed change looks like this:

Expected behavior

It shouldn't modify the line: sealed abstract class Value extends EnumValue at all.

I'm using Vim. I don't know it that matters. But there could be a conflict in shortcut if modifying the code uses a shortcut?

[Tomasz Godzik]

Thanks for reporting! We need to confirm a reproduction, if possible could you paste a self contained example?

[Jakub Cieśluk]

Which Scala and Metals versions do you use?

[Tanin Na Nakorn]

VSCode: 1.87.2, Metal: v1.29.0, Scala: 2.13.12

I'll make a small self-contained example!

Thank you for acknowledging the issue promptly!

[Tanin Na Nakorn]

Here's the POC: https://github.com/tanin47/poc-metals-6232

1. We can see that Queue is not imported correctly:

2. We hover over Queue and click Quick fix...:

3. We choose Import 'Queue' from package 'java.util. Then, we'll see the below:

I've confirmed that, even without the vim extension, the problem still occurs.