ada_language_server listed twice

[cloudlena]

Summary

When running hx --health, ada_language_server is listed twice:

The same goes for the documentation

helix/book/src/generated/lang-support.md

Line 3 in cf99615

| ada | ✓ | ✓ | | `ada_language_server`, `ada_language_server` |

.

To me, this is quite confusing. Is it correct?

Reproduction Steps

1. Run hx --health ada

Helix log

No response

Platform

Linux

Terminal Emulator

alacritty

Installation Method

nixpkgs

Helix Version

helix 24.03

[archseer]

Please submit a PR

[cloudlena]

Thanks, @archseer. I'm happy to provide a PR.
However, the issue seems to be that the documentation is generated from https://github.com/helix-editor/helix/blob/master/languages.toml. It only takes the command property of the language servers which, in case of ada is the same for both options because they only differ in the args. I'm not sure how to sustainably propagate that differentiation.

Ideally, it would be done in an automated way but ignoring the args for all other language servers to avoid cluttering the docs. What do you think? Do you have any guidance on how to proceed?

[the-mikedavis]

Deduplicating the command names seems like the right choice to me here. Or we could switch that column from the name of the binary to the name of the language server. That column predates #2507 which refactored language servers so that they have names in the config

[timothyhutz]

As a new Rust Dev, can I take this fix? I'll send you a PR from my fork.

[timothyhutz]

Fixed and tested #10563