Rearrange workspaceConfiguration call

Merge pull request #407 from vimpostor/textedit_cursor

Avoid moving the cursor when applying textedits

Avoid moving the cursor when applying textedits

In 33c9dc37bd354391579f669a142699801b13d571 we removed a workaround that
resets the cursor to the previous location after applying an
workspaceEdit, because it interfered with auto-imports.

Turns out the workaround was there for a reason: When we apply general
textedits (e.g. when we rename a variable), then we are always moving
the cursor by accident, because internally (to apply a textedit) we
delete the entire text range and then append the new range.
Obviously this loses the cursor position, hence the reason for the
original manual cursor reset workaround.

Instead of reintroducing that workaround, we now avoid moving the cursor
altogether: We accomplish this by no longer deleting the entire range
and inserting it again. Instead we just remove lines that actually need
to be deleted, add lines that actually need to be added and then handle
the rest with a normal setbufline() call.

This will cause the cursor to never lose the correct position, because
we never remove the entire region where the cursor is located in.

We have to be careful with off-by-one errors, therefore this also adds
an extra test for the scenario where we have to delete some lines. The
two other scenarios already had comprehensive tests (adding lines, and
keeping the amount of lines).

Access the save key in textDocumentSync only if it is a Dict

Merge pull request #405 from mhanberg/didsavetext

fix: send buffer text on textDocument/didSave

fixup! fix: send buffer text on textDocument/didSave

fixup! fix: send buffer text on textDocument/didSave

fix: send buffer text on textDocument/didSave

This sends the full buffer text on the textDocument/didSave notification
if the server's sync options specify it.

Merge pull request #393 from andlrc/fix-doc-error

Fix filetype error in docs

Fix filetype error in docs

Merge pull request #390 from vimpostor/completionitem_fix

Fix additionalTextEdits being ignored

Fix additionalTextEdits being ignored

Regression was introduced in 6f4fdc7bcc3a1ba041c6196b85b506bd0c207cf0.
The logic was still fine if the LSP server delayed the
additionalTextEdits field, but if we already got it, we ended up
ignoring it.

Fix this by checking for additionalTextEdits outside the delayed-resolve
condition.

Also see #389

Merge pull request #388 from matthias-margush/fix/initialize-lspserver-with-needOffsetEncoding

fix: initialize needOffsetEncoding in NewLspServer

Merge pull request #389 from vimpostor/completionitem_command

Add support for CompletionItem commands

Do not reset cursor after workspaceEdit

The workspace edit may add or delete lines, so resetting it to the last
linenumber and column will very likely not match the last logical line
before the workspaceEdit operation.

For example an action might add an auto-import statement at the top of
the file, effectively adding one line in-between. If we now reset the
cursor, we will end up one logical line before the one where we actually
started at.

This behaviour is automatically fixed if we just don't reset the cursor
at all.

Add support for CompletionItem commands

When we resolve a completion item with a "completionItem/resolve"
request, the LSP server returns a CompletionItem object [0], which may
contain an optional "command" field. This field describes a command that
must be called **after** inserting the completion.

For example the haskell-language-server may respond with the following
command after autocompleting an unimported function (arguments are left
out for brevity):

{'command': '10619:ghcide-extend-import-action:extendImport', 'arguments': [], 'title': 'extend import'}

Then when we send the matching "workspace/executeCommand" request, the
server will respond with a "workspace/applyEdit" request, that
autoimports the function.

Technically the specification dictates that servers should prefer to set
"additionalTextEdits" for that usecase, but there is still some use for
doing it this way, e.g. haskell-language-server also wants to display an
info message to the user, which doesn't happen with
"additionalTextEdits".

In any case it is important to support the "command" field, as it may
also contain actions unrelated to "additionalTextEdits".

[0] https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#completionItem

Merge pull request #387 from maximyurevich/main

fix(handlers): ignore `sqlLanguageServer.finishSetup`

fix: initialize needOffsetEncoding in NewLspServer

fix(handlers): ignore `sqlLanguageServer.finishSetup`

Merge pull request #383 from maximyurevich/main

fix: ignore `eslint/status` and `taplo/didChangeSchemaAssociation`

fix: ignore `eslint/status` and `taplo/didChangeSchemaAssociation`

Merge pull request #375 from girishji/main

omnifunc documentation popup should not throw error

Silenty ignore a file with empty file type

omnifunc documentation popup not throw error
When using Omni completion with other sources of completion items
that do not have 'user_data' dictionary, documentation popup should
not throw error.

I am using my own completion plugin with g:LspOmniFunc and I need
LSP to grab lazy-doc from lsp server. But it should not throw
error when user_data does not contain dictionary.

M  autoload/lsp/completion.vim

Options changed autocmd is not invoked

Option changes are not handled properly

Auto import is not working with typescript language server.  Fixes #367.

Move the code for displaying the current diag in the status line to diag.vim

Update help text

Add option to enalbe/disable omni-completion

Skip displaying inlay hints if it is disabled for a language server

When signature help is disabled for a lsp server, don't map the trigger characters.  Update the help text to use the space character (32)

Merge pull request #365 from rcasta74/md_parser

Markdown parser fix for contents with multibyte characters

Markdown parser: add tests for line breaks

Markdown parser: fix SplitLine with multibyte characters

Markdown parser: strip leading and trailing whitespaces in heading

Fix test failure

Add markdown parser tests

Newline character is not removed from the completion info popup menu text

Merge pull request #363 from rcasta74/md_parser

Fix markdown parser

Fix markdown parser

Merge pull request #362 from rcasta74/md_parser

Markdown parser enhancement

Handle soft/hard line breaks in markdown parser

Expand tabs used to define block structure in markdown parser

Merge pull request #361 from girishji/omnifunc

Omnifunc optimization and readiness indicator

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