update(manifest): serverPath default to nixd

[Malix-Labs]

Since https://github.com/nix-community/nixd has become the https://github.com/nix-community supported nix lsp, and the seemingly overwhelming preference for nixd, it might be time to prefer it compared to https://github.com/oxalica/nil

[rszyma]

Isn't nil working better out-of-the-box? iirc nixd requires configuration regarding nixpkgs location to be usable

the seemingly overwhelming preference for nixd

source?

[Malix-Labs]

i have not configured nixd and it is properly working out of the box indeed (using nixpkgs as a flake input)

source?

The research I made quite some time ago now in various Nix spaces, including the discourse and discord.

I have started a poll there: https://discourse.nixos.org/t/poll-nix-lsp-language-server-protocol/70658

it is also possible to combine lsps

---

regardless, I think the https://github.com/nix-community supported one should be prefered

[nixos-discourse]

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/poll-nix-lsp-language-server-protocol/70658/1

[rszyma]

i have not configured nixd and it is properly working out of the box indeed (using nixpkgs as a flake input)

I tried it now too and seems to work great (I like the pkgs definitions on hover, I'm switching to it now :D )

[jnoortheen]

As I have mentioned in the other PR, I am Ok with having a dynamic option where the available binary is picked first. The order of preference can start from nixd.

[Malix-Labs]

@jnoortheen neither nixd and nil are installed by default

This PR simply proposes to change the default to nixd instead of nil

Can this PR be reopened?

Or do you have a refactoring of this option or its implementation in mind, or perhaps making it so you can do nixd, nil?

[jnoortheen]

I would like #506 to be worked to support to choose the first available binary if you are up for that.

[Malix-Labs]

I guess then #505 and #506 needs to be one congruent PR, and enableLanguageServer, serverPath, and their implementation reworked