Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix(vim): pass position encoding to Neovim - #4532

Open
tandede wants to merge 1 commit into
TabbyML:mainfrom
tandede:fix/neovim-position-encoding
Open

fix(vim): pass position encoding to Neovim#4532
tandede wants to merge 1 commit into
TabbyML:mainfrom
tandede:fix/neovim-position-encoding

Conversation

@tandede

@tandede tandede commented Aug 22, 2026

Copy link
Copy Markdown

Summary

  • fetch the active Tabby LSP client before building inline-completion parameters
  • pass that client offset encoding to vim.lsp.util.make_position_params
  • return early when the client is no longer available

This removes the Neovim 0.11 position-encoding warning and avoids selecting the encoding of an unrelated first client when several LSP clients are attached.

Test plan

  • luac -p clients/vim/lua/tabby/lsp/nvim_lsp.lua
  • isolated Lua regression scenarios for an active and a missing client
  • verified with Neovim 0.10.4 and 0.11.7

Fixes #4527

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

Successfully merging this pull request may close these issues.

Neovim 0.11 warning because position encoding is omitted from make_position_params

1 participant