feat(lsp): add command to reload lsp settings

[GuillaumeLagrange]

Following the discussion we had in #286, I found out how to reload settings using workspace/didChangeConfiguration.

Turns out the lsp server responds by a workspace/configuration where the server actually pulls the new configuration, since the server can cache configuration and decides whether it needs to actually pull the config.

nvim handles it, but we need to set the client.settings table to the new configuration before sending the workspace/didChangeConfiguration because it seems to be the one used by nvim to send the actual config.

I added this feature as a command because it made sense to me, let me know if it does not for you.

Once again, tested the changes on https://github.com/GuillaumeLagrange/neoconf-repro and it works for me.

Cheers

[github-actions]

Review Checklist

Does this PR follow the Contribution Guidelines? Following is a partial checklist:

Proper conventional commit scoping:

• For example, fix(lsp): some lsp-related bugfix

• Pull request title has the appropriate conventional commit prefix.

If applicable:

• Tested
  • Tests have been added.
  • Tested manually (Steps to reproduce in PR description).
• Updated documentation.
• Updated CHANGELOG.md

[mrcjkb]

Thanks for the PR 😄

This seems like a good approach.
I'll see it I can find some time to test it soon.

[GuillaumeLagrange]

@mrcjkb I'm having troubles with the lua type checker, the CI build fails, and I can reproduce locally by running

nix flake check --print-build-logs

For some reason, it has trouble detecting that client is an lsp.Client class, but my lsp in nvim detects it properly with the cast.
I'm not experienced with nix at all so I cannot dig deeper that this :(

[mrcjkb]

@mrcjkb I'm having troubles with the lua type checker, the CI build fails, and I can reproduce locally by running

nix flake check --print-build-logs

For some reason, it has trouble detecting that client is an lsp.Client class, but my lsp in nvim detects it properly with the cast. I'm not experienced with nix at all so I cannot dig deeper that this :(

Don't worry about that. I'll fix it before merging 😀

[mrcjkb]

Tested it, and it seems to be working well 😄