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

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