bugfix: Don't deduplicate all options when using Metals

[tgodzik]

PReviously, we would use .distinct on all project scalac options which would remove values for different options if the happened to be the same. Now, we only deduplicate semanticdb options.

Fixes scalameta/metals#5400

[ckipp01]

Makes sense and this seems to be safer, although it makes me wish there was an easier way to actually detect duplicates. However I know it's hard since some will be -key:value and others key, value so detecting which shape they are isn't easy. Will bloop emit a bunch of warnings in situations where options are duplicated now though?

Either way, LGTM

[tgodzik]

Makes sense and this seems to be safer, although it makes me wish there was an easier way to actually detect duplicates. However I know it's hard since some will be -key:value and others key, value so detecting which shape they are isn't easy. Will bloop emit a bunch of warnings in situations where options are duplicated now though?

Either way, LGTM

We should not get any duplicates really, the only ones we could have are the ones from adding semanticdb, which are handled here.

[tgodzik]

Anyway, one of the changes test was testing that, but the logic was reworked so the exact order changed.