Improved doc

M  doc/lsp.txt

Improved doc

M  doc/lsp.txt

Optimize regex comparison in omnifunc

M  autoload/lsp/completion.vim

API to find out if LSP server has replied

Add a function to find out if LSP server has replied with completion
matches. This is needed in case of using external completion engines
that expect asynchronous callback. LSP client simply acts as a source
of completion items through omnifunc (g:LspOmniFunc). Before calling
the omnifunc the second time g:LspOmniCompletePending can be checked
to see if LSP server is ready to provide completion items.

M  autoload/lsp/completion.vim
M  doc/lsp.txt

Update the client capabilities.  Pass buffer number to autocmd handler functions.  Send pending file changes to the language server before updating inlay hints and highlights

Optimize the diags handling code

Markdown is not applied in the preview window.  Fixes #359

Pass buffer number as the argument to the autocmd handlers

Enable line number in the symbol peek popup window

Diagnostic balloon doesn't work properly

Merge pull request #358 from vimpostor/diagsbufreload

Respect autoHighlightDiags when reloading a buffer

Do not rely on vim's locale in tests

The language of the user running the tests may be different, which means
that the error message may be different as it is translated.

Instead match only against the error code, which is safer.

Enforce disabled autoHighlightDiags when aleSupport is set

When someone enables Ale support, it semantically implies disabling
automatic highlighting of diagnostics from this plugin.

Thus we can simplify later checks by automatically disabling
autoHighlightDiags.

Never show diagnostics when they are disabled

A regression was introduced in 6e4ba228eaa0770470060d8d47a360bd05a04ee5
that causes diagnostics to be displayed on buffer reload, even when the
user disabled them.

Fix this by moving the option check directly to DiagsRefresh.

Add the ":LspDiag" command and add sub-commands for the various Lsp
diagnostic functionality.  Update the test scripts to use the new set of
commands.  Add support for dynamically changing the value of the
diagnostic options.  Remove the ":LspDiagHighlightEnable" and the
":LspDiagHighlightDisable" commands (these are replaced by the ":LspDiag
highlight" command)

Diags are not highlighted after a buffer is reloaded

Fix test failure

Add support for enabling/disabling inlay hints

Test fail with older Vim versions

Error loading the plugin if OptionsSet() function is not used

Ignore unsupported pyright language server notifications.  Fixes #355

Add basic support for InsertReplaceEdit in completion reply.  Fixes #354

Dict key access optimizations

Use the current encoding for the completion popup menu.  Fixes #352

Minor optimizations

When applying text edits and restoring eol, ignore empty lines. Optimize use of range Dict by using local variables

Don't check the capabilities for language servers which are not ready.  Update help text

Add an option (showDiagWithSign) to enable/disable placing signs on lines with diagnostics.  Add separate virtual text highlight groups for each type of diagnostic

Select the current symbol when opening the peek symbol popup menu.  Open closed folds when jumping to a symbol location

Use the FileType autocmd to start the LSP server for a buffer

When using multiple language servers, check for the server capabilities in more places

Merge pull request #347 from girishji/buffer-completion-timeout

Make buffer completion responsive for large files

Optimize dict lookup
Move dict lookup outside of for loop to avoid repetition.

M  autoload/lsp/completion.vim

Improved

M  doc/lsp.txt

Make buffer completion responsive for large files

Since buffer completion processes the current buffer everytime user
types something, it will degrade the experience for large files.

This change adds a timeout to buffer completor function. Processing
current buffer is cut short when timeout is exceeded. Setting
timeout to 0 will revert back to existing behaviour.

Default is set to 100 ms, good for scanning 6000 lines on M1 macbook.

It is possible to get fancy by scanning locality of cursor first
but such complication may not be worth the complexity.

Tested on >20k line files (I have to open these large C files
filled with hw specs occasionally).

M  autoload/lsp/completion.vim
M  autoload/lsp/options.vim
M  doc/lsp.txt

Minor optimization

Add support for displaying symbols in the current file in a popup menu

Move snippet related code to a separate file

Support using a particular offset encoding in LSP messages

Silence the messages when loading new buffers

Update the check for the presence of a diags location list and update comments

Decode symbol location only when needed

Cursor not positioned correctly when jumping to a symbol in another file

:LspSymbolSeach command doesn't support command modifiers. When only one symbol is found, jump to it

Not able to set language server trace level during initialization

Add a function to get all the diagnostics in a buffer

Update diags location list when the diags for the buffer changes. Update comments

Fix hover test failure

Not able to use :LspHover with 'keywordprg'

Merge pull request #343 from Shane-XB-Qian/fix_test_conflict_failure

fix: test conflict failure

fix: test conflict failure

Add support for UTF-8 and UTF-16 offset encoding

Merge pull request #340 from Shane-XB-Qian/fix_super_is_keyword

fix: 'super' is a keyword cannot use

fix: 'super' is a keyword cannot use

Merge pull request #334 from vimpostor/ale

Implement Ale integration support

Merge branch 'main' into ale

Some lines in the plugin help text exceed 79 characters

Merge pull request #336 from Shane-XB-Qian/fix_util_getbufline_useless

fix: util.GetBufOneLine is a bit useless

Merge pull request #335 from Shane-XB-Qian/fix_correct_test_fs_ignore

fix: correct test fs gitignore

Merge pull request #337 from Shane-XB-Qian/fix_test_conflict_failure

fix: test conflict failure

fix: test conflict failure

fix: util.GetBufOneLine is a bit useless

fix: correct test fs gitignore

Fix test failure

Fix test failures

Properly support multibyte characters with composing characters

Add Ale integration support

Ale [0] is a famous plugin to show linter warnings and diagnostics. It
already provides extensive functionality and configurability to display
those diagnostics.
In order to allow diagnostics from other plugins (e.g. LSP plugins) to
be shown together with those linter warnings, Ale has API so that those
diagnostics can be sent to Ale.

Ale will then display them as if they came from Ale itself. This is
useful for many reasons, e.g. this ensures a consistent look and allows
the location list for a window to be populated with all diagnostics,
regardless from which plugin they come.

There are other famous LSP plugins, that already integrate with Ale,
e.g. coc.nvim [1] allows this with "diagnostic.displayByAle".

This patch implements the same functionality by adding an "aleSupport"
option that is off by default. If the option is turned on, diagnostics
will be sent to Ale instead of being displayed by this plugin directly.

[0] https://github.com/dense-analysis/ale
[1] https://github.com/neoclide/coc.nvim

Use getbufoneline() in GetLineByteFromPos

This makes this more robust and also decreases the indentation level.

Merge pull request #332 from girishji/vim-vsnip-support

Add support for hrsh7th/vim-vsnip plugin

Use string evaluation

M  autoload/lsp/completion.vim

Correction

M  autoload/lsp/completion.vim

Change to '->' method call syntax

M  autoload/lsp/completion.vim

Merge pull request #333 from girishji/buffer-compl

Fix bug where buffer completion spams

Fix bug where buffer completion spams
Steps to reproduce:
- Enable buffer completion
- Search for 'xyz' where 'xy' has completions
- Backspace to remove 'z'
- You'll see all items from current buffer shown that are not related
  to 'xy'

M  autoload/lsp/completion.vim

Enchance documentation

M  doc/lsp.txt

Enhance documentation

M  doc/lsp.txt

Fix merge issue

M  autoload/lsp/completion.vim

Add support for hrsh7th/vim-vsnip snippets

Summary:

- Query vim-vsnip plugin for snippet completions and add the items
  to the completion list.
- Add a boolean flag to set vsnip completion option
- Update documentation
- Without this enhancement vsnip could only expand LSP provided
  snippets
- Inteface to vim-vsnip plugin is based on hrsh7th/cmp-vsnip

Following plugins need to be installed:

[required] hrsh7th/vim-vsnip
[required] hrsh7th/vim-vsnip-integ
[optional] rafamadriz/friendly-snippets

Options should be set as follows:

completionTextEdit: false,
snippetSupport: true,
vsnipSupport: true,
ultisnipsSupport: false,

Shortcoming:

Completion is not triggered for non-keyword characters like '#' (ex. #
before #if in C) even though vsnip is capable of such completion,
unless LSP server sets special characters as trigger characters. This
is not a major shortcoming since it will expand once a keyword
character is typed (say, #i in #if). To address this issue some code
reorganization is needed, and perhaps separation of completion
mechanism from the LSP client core.

User can define keymaps for Tab completion as follows:
(<Space> will expand the snippet)

def! g:LspCleverTab(): string
    return pumvisible() ? "\<c-n>" : vsnip#jumpable(1) ?
\ "\<Plug>(vsnip-jump-next)" : g:WhitespaceOnly() ?
\ "\<tab>" : "\<c-n>"
enddef
def! g:LspCleverSTab(): string
    return pumvisible() ? "\<c-p>" : vsnip#jumpable(-1) ?
\ "\<Plug>(vsnip-jump-prev)" : g:WhitespaceOnly() ?
\ "\<s-tab>" : "\<c-p>"
enddef
autocmd FileType java,c,cpp,python
            \| iunmap <silent> <tab>
            \| iunmap <silent> <s-tab>
            \| inoremap <expr> <tab> g:LspCleverTab()
            \| snoremap <expr> <tab> g:LspCleverTab()
            \| inoremap <expr> <S-Tab> g:LspCleverSTab()
            \| snoremap <expr> <S-Tab> g:LspCleverSTab()

M  autoload/lsp/completion.vim
M  autoload/lsp/options.vim
M  doc/lsp.txt

Fix test failure

getbufoneline() is supported only starting from Vim 9.0.0916

Merge pull request #331 from vimpostor/completepopup

Fix completepopup colors

Merge pull request #330 from vimpostor/getbufoneline

Fix rare out-of-bounds issue with getbufline()

Merge pull request #329 from vimpostor/continue-missing-server

Skip invalid servers instead of aborting

Merge pull request #328 from inetic/cygpath-cache

Speed up cygwin path transformation.

Unify completepopup settings

There is not really a reason to have different completepopup settings
depending on if the LSP server supports lazy documentation or not:

In both cases info may be displayed, the only difference being that with
"resolveProvider" the info is to be fetched on demand.
But for both scenarios the formatting of the info popup should be the
same.

This fixes a problem, where in the non-lazydoc branch the info popup is
colored with the PmenuSel highlight group, which is a little bit ugly
when the content is syntax highlighted as well.

Use getbufoneline() where applicable

This has better performance than getbufline() and as a sideeffect also
fixes a rare issue where the LSP sends invalid data such that
getbufline() reads an empty list and causes the array access to get out
of bounds. With getbufoneline() this neatly falls back to returning the
empty string.

Skip invalid servers instead of aborting in LspAddServer()

If the user calls LspAddServer() with a list of servers, where the very
first entry does not exist, but all the remaining ones do, then no
servers are added at all. This doesn't make much sense, as the other
entries might be valid entries.

Fix this by simply continuing with the next entry instead of returning
when encountering an error.

Minor refactor

Use method calling syntax

Markdown text in a completion popup window is not properly rendered.  In a Vim9 script, string index is a character index and not a byte index.  Use strpart() instead of indexing a string to extract characters

Cache full paths in LspFileToUri

Speed up cygwin path transformation.

Fix test failures

In the LSP client capabilities, use 'markdown' first before 'plaintext'

Error message about a language server not supporting a particular feature is not correct

Merge pull request #326 from andlrc/conditional-prop-override

Conditionally override cursor for props when "&cursorline" is set

Conditionally override cursor for props when "&cursorline" is set

Merge pull request #302 from newtonne/focus-fix

Add `keepFocusInDiags` option

Merge pull request #325 from andlrc/refactor/golsp-test

Save bufnr() in a variable instead of inlining it

Save bufnr() in a variable instead of inlining it