Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lsp server not started properly #65

Open
teto opened this issue Nov 4, 2023 · 2 comments
Open

lsp server not started properly #65

teto opened this issue Nov 4, 2023 · 2 comments

Comments

@teto
Copy link

teto commented Nov 4, 2023

I hit the following error:

"Requesting completion for a detached buffer, check enable_suggestions_on_files' value",

1/ I think the message could be improved, aka "no LSP server enabled for this buffer"
2/ LspInfo gives me:

 Client: llm-ls (id: 1, bufnr: [1, 6])
 	filetypes:       
 	autostart:       false
 	root directory:  /home/teto/nixpkgs
 	cmd:             /nix/store/ngay8qxw0bnirwsnjsk84bdcsbd2q9kc-llm-ls-0.4.0/bin/llm-ls

which means the config here

local client_id = lsp.start({
is probably not enough (it should autostart maybe ?).
Turns out :LspStart llm-ls seems to start it (I see "codellama currently loading", I hope it's not too long took 1mn and I could accept the proposition with the binding in accept_keymap).

Interested in adding llm-ls to https://github.com/neovim/nvim-lspconfig ? ping me if you open a PR and I can review it.

@teto
Copy link
Author

teto commented Nov 5, 2023

also the plugin spams Requesting completion for a detached buffer, check enable_suggestions_on_files' value notifications which is not a good design IMO (aka it does not scale if every plugin does the same). I believe users should go through checkhealth / :LspInfo in case it doesn't work, like for other servers.

@McPatate
Copy link
Member

McPatate commented Nov 5, 2023

Interested in adding llm-ls to https://github.com/neovim/nvim-lspconfig ?

Why not though not a priority for me atm. Thanks for offering a review.

also the plugin spams

Yes, I'll remove that log as it doesn't add much value. Manual :checkhealth seems like the way to go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants