fix(pyright): command :LspPyrightOrganizeImports fails

[michelesr]

Problem:

The pyright.organizeimports is private, so client:exec_cmd()
fails because it refuses to run commands that are not advertised
as capabilities.

Solution:

Call client.request() to side-step the check in client:exec_cmd().
YOLO

[justinmk]

Ok, I did not realize that exec_cmd() is actively doing a check: https://github.com/neovim/neovim/blob/c333d64663d3b6e0dd9aa440e433d346af4a3d81/runtime/lua/vim/lsp/client.lua#L1024-L1027

and that fails because "basedpyright.organizeimports" is a private command (not advertised via capabilities).

Whereas client.request() just doesn't do that check.

So, we can merge this after:

• add a comment above client.request() with the above information
• rebase

[michelesr]

@justinmk what about the failing linting checks, is there a way to disable them for the relevant LoC?

lint.sh: Use client:exec_cmd() instead of calling request("workspace/executeCommand") directly. Example: lsp/pyright.lua

[justinmk]

To pass the lint you can change this line

nvim-lspconfig/.github/ci/lint.sh

Line 56 in 408cf07

local exclude='eslint'

to:

local exclude='eslint\|pyright\|basedpyright'

i'll think about how to make it more granular later.

[michelesr]

@justinmk rebased and added the lint exclusion.

P.S: added the comments.

[justinmk]

Thanks for your help. I updated the description to be 100% clear, please consider that next time.