nixd: code actions

[inclyc]

• add undefined variable as top-level formal
• split an subexpression, with import ./file
• with ..; let / inherit

[VTimofeenko]

If I may request, a couple more:

• flatten attrset:

  { foo = { bar = "foz"; } };

  to

  { foo.bar = "foz"; }

• pack into nested attrset:

  foo.bar = "foz";
  foo.baz = "faz";

  to

  foo = { 
    bar = "foz";
    baz = "faz";
  };

[inclyc]

Thanks, any more ideas are welcomed!

[LennyLizowzskiy]

• "remove unused definition" for the warning definition X is not used (sema-def-not-used)

[kpbaks]

I use noogle.dev a lot. A "open in noogle.dev" code action would be neat for any function in the lib attribute set from nixpkgs

[zspher]

from: https://github.com/oxalica/nil/blob/c8e8ce72442a164d89d3fdeaae0bcc405f8c015a/docs/code_actions.md

• add_to_top_level_lambda_param
  Add an undefined name to the top-level lambda.

{ foo }: foo + bar

=>

{ foo, bar }: foo + bar

• quote_attr and unquote_attr
  Rewrite between attribute names and double quoted strings

{ foo = bar; }

<=>

{ "foo" = bar; }

[blackxored]

Is this planned or in development? No pressure, just wanted to know since I currently have to switch LSPs when I need any of these.
In addition one suggestion is converting json to nix which is quite common (think waybar).

[inclyc]

Is this planned or in development? No pressure, just wanted to know since I currently have to switch LSPs when I need any of these. In addition one suggestion is converting json to nix which is quite common (think waybar).

It is something I plan to pursue, but currently, I don’t have enough bandwidth to move it forward. If you're particularly interested in these features, you might consider supporting the project financially. That said, this doesn’t mean I won’t develop them without funding—I’m just not able to prioritize them at the moment.

[nixos-discourse]

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

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

[takeokunn]

Based on the discussions here and nil-lsp, I've created a draft PR implementing all the features that seemed useful.

#755

I plan to release PRs gradually over time.