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

Merge branch 'main' into focus-fix

Merge pull request #301 from newtonne/loclist-fix

Always switch to correct loclist after running `:LspShowDiags`

update comment

move code to ShowAllDiags

Merge branch 'main' into loclist-fix

default to true and update tests

Merge branch 'main' into focus-fix

Ignore the '$/analyzerStatus' notification message from the dart language server

Merge pull request #321 from andlrc/add-priority-to-props

Add priority to props

Add priority to inline diagnostics

This will render an error sign above warning, info and hint signs.

Add priority to signs

This will render an error sign above warning, info and hint signs.

Get the resolveProvider capability only if completionProvider is present

Merge pull request #320 from andlrc/omni-compl-and-lazy-docs

Add support for lazy documentation and omni completion

Respect when "&completeopt" doesn't contain "preview"

Add support for lazy documentation and omni completion

Merge pull request #306 from andlrc/custom-ref-highlight

Custom ref highlight

Don't override visual selection, hlsearch etc, when highlighting refs

Make it possible to define custom ref highlights

Merge pull request #318 from andlrc/process-diags

Process diags

Merge branch 'main' into process-diags

Update comment to mention the list of unhandled server requests

Merge pull request #310 from andlrc/customrequesthandlers

Add support for custom request handlers

Merge branch 'main' into customrequesthandlers

Merge pull request #309 from girishji/ignore_jdtls_request

Ignore non-standard request from eclipse java LS

Merge pull request #312 from guildem/feature/readme

Updated documentation with options example and reordering

Move the license in the help text to the end

Merge pull request #316 from ubaldot/docs_examples_to_vim9

Examples to vim9, added index and some ascii artwork in docs

Merge pull request #303 from newtonne/fix-link

Link to unit test results from README rather than image

Merge pull request #314 from techntools/fix/return-after-throw

Remove return after throw

Add test for "processDiagHandler"

Add example for how to prefix diagnostics messages

Add example for how to filter diagnostics

Add error checking for the config "processDiagHandler"

Add the server config "processDiagHandler"

Integrated feedbacks 1-5

Removed call for function calls

removed quotes in dicts key and backslashes for new lines

Examples to vim9, added index and some ascii artwork in docs

Remove return after throw

Reorder options by alphabetical order

Add a global example of the features configuration into README

Add support for custom request handlers

Ignore non-standard request from eclipse java LS

Eclipse java language server sends the 'workspace/executeClientCommand'
request (to reload bundles) which is not in the LSP specification.
Ignore this request and suppress the error message.

Fixed typos in lspserver.vim

M  autoload/lsp/handlers.vim
M  autoload/lsp/lspserver.vim

Default keepFocusInDiags to false

make more explicit

add tests

Link to unit test results from README rather than image

Add `keepFocusInDiags` option

This option serves a similar purpose to `keepFocusInReferences` but for
`LspDiagShow` rather than `LspShowReferences`. No changes to default
behaviour since it defaults to true.

Always switch to correct loclist after running `:LspShowDiags`

Fixes #300

Merge pull request #298 from inetic/cygpath

Simplify cygwin path conversions

Update comment

Simplify cygwin path conversions

Update the README file with a link to the omnisharp LSP client plugin.

Merge pull request #297 from inetic/cygwin-file-to-uri

Adapt LspFileTorUri for Cygwin

Style changes

Adapt LspFileTorUri for Cygwin

Convert paths of the form "/cygdrive/c/foo/bar" to "c:/foo/bar" and paths of
the form "/home/pete/foo" to "C:/cygwin64/home/pete/foo"

Merge pull request #293 from berggeist/add-headline-to-language-server-messages

Add headline to language server message

Add headline to language server message

Merge pull request #289 from andlrc/remove-if-around-prop-decl

Remove option checks around prop_ declarations

Merge pull request #290 from andlrc/always-send-initOpts

Always send "initializationOptions"

Always send "initializationOptions"

Some servers expects the object to be there. One being
https://github.com/vuejs/language-tools

Remove option checks around prop_ declarations

There is no reason not supporting enabling these options about
initialization instead.

Merge pull request #284 from greeschenko/main

add words from current buffer to completion list

fix if

Save the server textDocumentSync capability

added requested changes

Update README.md

Update the readme file

Adding inlay hints results in error if the text is modified

add useBufferCompletion option

add words from current buffer to completion list

Merge pull request #283 from berggeist/cleanup-diagnostics-code-duplicates

Cleanup of code duplications for diagnostics.

Cleanup of code duplications for diagnostics.

Use filterText to filter the completion matches.  Move the Ultisnips code to a separate function

In Vim9 scripts, 'ignorecase' is not used for comparison operators