The previous implementation used `reuse_client = function() return true
end`,
which caused buf_ls to reuse *any* existing LSP client. As a result,
when a
Copilot LSP client was already running, buf_ls attempted to reuse that
client
instead of spawning its own, preventing buf_ls from attaching to .proto
buffers.
Restrict reuse_client so only buf_ls clients can be reused.
This updates the buf_ls command post v1.59.0, where the command was
promoted out of beta to general availability. The command was changed
from buf beta lsp to buf lsp serve.
Problem:
Nvim 0.11 has vim.lsp.config, which mostly replaces the legacy
nvim-lspconfig "framework".
Solution:
Migrate all configs to `lsp/*` variants. The old configs in
`lua/lspconfig/` are "frozen".
The new configs include these changes:
- `commands` field became raw calls to
`vim.api.nvim_buf_create_user_command` inside `on_attach`.
- `root_dir` became:
- `root_markers` whenever the file list was simple didn't need to mach `*`
- if the logic was complicated, or needed to match something like
'\*.c', it was defined as a vim.lsp.Config `root_dir` callback.
- `on_config_change` became `before_init`. I don't actually know if this
is the correct approach, but looking around the documentation of
`nvim-lspconfig` a saw that it was defined as the function that gets
called as soon as the config have `root_dir`, and so I thought
`before_init` might be the closest alternative.
- `docs.description` became a luadoc `@brief` docstring.
- `single_file_support = false`?
Co-authored-by: Aliou Diallo <aliou@users.noreply.github.com>
Co-authored-by: Justin M. Keyes <justinkz@gmail.com>
