nixd/configuration: mandatory field "args" is confusing for new users about nixd HOT 4 CLOSED

I don't think it's doing anything. I'm pretty new to lsp and elgot, I have no idea how to debug this case.

Did eglot show you nixd logs? Typically we have logging outputs in stderr.

for example, it completes builtins.fromTOML but obviously missing builtins.fromJSON somehow, maybe this is a separate
issue.

Looks like you are writting a module, and builtins is the attribute definition inside { }, (do you want to define an option named builtin?). Basically LSP cannot decide that builtin is the same as top-level builtin, the former is an attribute and the latter is a top-level variable. Maybe this is confusing for our new users, but I'm suspicious about whether or not to fix.

from nixd.

This completion list here explicitly annotated the items are "fields", can you reproduce this on emacs? Just create a file that only contains builtin, then type . (dot) to see the completion list

from nixd.

The problem here is "target" field has a mandatory member "args", which may be a trouble for new users who does not write "args": [ ] in the configuration.

from nixd.

This looks like a good issue for new contributors. Please:

1. use std::optional on this structure.

2. rewrite implementation here

3. fix & add regression & unit tests

4. (optional, but recommended) update the docs.

from nixd.