Merge pull request #425 from DanielViberg/main

Changed default value of missing workspaceConfig from [{}] to [null]

Changed default value of missing workspaceConfig from [{}] to [null]

Merge pull request #422 from Angluca/main

Copy to my vimrc Error!!

Copy to my vimrc Error!!

Newbies only copy and paste settings
T. T

Merge pull request #421 from Shane-XB-Qian/fix_semantic_flag_typo

fix: fix semantic flag typo

fix: fix semantic flag typo

Signed-off-by: shane.xb.qian <shane.qian@foxmail.com>

Add the initial support for semantic highlighting

Merge pull request #420 from Shane-XB-Qian/refine_compl_kind_trig_char

none: get trig chars always

none: get trig chars always

Signed-off-by: shane.xb.qian <shane.qian@foxmail.com>

Merge pull request #418 from girishji/main

Make omnifunc() complete keywords based on LSP triggerCharacters

Improve doc

M  autoload/lsp/completion.vim

Merge branch 'yegappan:main' into main

Changes as requested by reviwers

M  autoload/lsp/completion.vim
M  test/clangd_tests.vim

Merge pull request #419 from CoelacanthusHex/fix/list

fix(inlayhint): fix format when label is a list in Rust Analyzer

In recent Vim versions, prop_list() doesn't return the length item for virtual text

fix(inlayhint): fix format when label is a list in Rust Analyzer

resolve #416

Fix some of the test failures
- fixed omnifunc related tests
- diag related tests are still failing

M  autoload/lsp/completion.vim
M  test/clangd_tests.vim

Propagate triggerCharacters to omni-completion
Fixed 2 bugs:
- add triggerCharacters to lsp list for omniComplete option
- process triggerCharacters based keywords in omnifunc()

M  autoload/lsp/completion.vim
M  autoload/lsp/lspserver.vim

Merge pull request #415 from Konfekt/patch-1

update doc on default options

Merge pull request #357 from Shane-XB-Qian/fix_inlay_maybe_delayed

fix: inlay hints maybe a bit delayed if sync init

update default options

the documentation no longer reflected autoload/lsp/options.vim (lines 9-162):

export var lspOptions: dict<any> = {
  # Enable ale diagnostics support.
  # If true, diagnostics will be sent to ale, which will be responsible for
  # showing them.
  aleSupport: false,

  # In insert mode, complete the current symbol automatically
  # Otherwise, use omni-completion
  autoComplete: true,

  # In normal mode, highlight the current symbol automatically
  autoHighlight: false,

  # Automatically highlight diagnostics messages from LSP server
  autoHighlightDiags: true,

  # Automatically populate the location list with new diagnostics
  autoPopulateDiags: false,

  # icase | fuzzy | case match for language servers that replies with a full
  # list of completion items
  completionMatcher: 'case',

  # Due to a bug in the earlier versions of Vim, cannot use the
  # COMPLETIONMATCHER_CASE constant here for initialization.
  completionMatcherValue: 1,

  # diagnostics signs options
  diagSignErrorText: 'E>',
  diagSignHintText: 'H>',
  diagSignInfoText: 'I>',
  diagSignWarningText: 'W>',

  # In insert mode, echo the current symbol signature in the status line
  # instead of showing it in a popup
  echoSignature: false,

  # hide disabled code actions
  hideDisabledCodeActions: false,

  # Highlight diagnostics inline
  highlightDiagInline: true,

  # Show the symbol documentation in the preview window instead of in a popup
  hoverInPreview: false,

  # Don't print message when a configured language server is missing.
  ignoreMissingServer: false,

  # Focus on the location list window after ":LspDiag show"
  keepFocusInDiags: true,

  # Focus on the location list window after LspShowReferences
  keepFocusInReferences: true,

  # If true, apply the LSP server supplied text edits after a completion.
  # If a snippet plugin is going to apply the text edits, then set this to
  # false to avoid applying the text edits twice.
  completionTextEdit: true,

  # Alignment of virtual diagnostic text, when showDiagWithVirtualText is true
  # Allowed values: 'above' | 'below' | 'after' (default is 'above')
  diagVirtualTextAlign: 'above',

  # Suppress adding a new line on completion selection with <CR>
  noNewlineInCompletion: false,

  # Omni-completion support.  To keep backward compatibility, this option is
  # set to null by default instead of false.
  omniComplete: null,

  # Open outline window on right side
  outlineOnRight: false,

  # Outline window size
  outlineWinSize: 20,

  # Show diagnostic text in a balloon when the mouse is over the diagnostic
  showDiagInBalloon: true,

  # Make diagnostics show in a popup instead of echoing
  showDiagInPopup: true,

  # Suppress diagnostic hover from appearing when the mouse is over the line
  # Show a diagnostic message on a status line
  showDiagOnStatusLine: false,

  # Show a diagnostic messages using signs
  showDiagWithSign: true,

  # Show a diagnostic messages with virtual text
  showDiagWithVirtualText: false,

  # enable inlay hints
  showInlayHints: false,

  # In insert mode, show the current symbol signature automatically
  showSignature: true,

  # enable snippet completion support
  snippetSupport: false,

  # enable SirVer/ultisnips completion support
  ultisnipsSupport: false,

  # add to autocomplition list current buffer words
  useBufferCompletion: false,

  # Use a floating menu to show the code action menu instead of asking for
  # input
  usePopupInCodeAction: false,

  # ShowReferences in a quickfix list instead of a location list`
  useQuickfixForLocations: false,

  # enable hrsh7th/vim-vsnip completion support
  vsnipSupport: false,

  # Limit the time autocompletion searches for words in current buffer (in
  # milliseconds)
  bufferCompletionTimeout: 100,

  # Enable support for custom completion kinds
  customCompletionKinds: false,

  # A dictionary with all completion kinds that you want to customize
  completionKinds: {}
}

# set the LSP plugin options from the user provided option values
export def OptionsSet(opts: dict<any>)
  lspOptions->extend(opts)
  if !has('patch-9.0.0178')
    lspOptions.showInlayHints = false
  endif
  if !has('patch-9.0.1157')
    lspOptions.showDiagWithVirtualText = false
  endif

  # For faster comparison, convert the 'completionMatcher' option value from a
  # string to a number.
  if lspOptions.completionMatcher == 'icase'
    lspOptions.completionMatcherValue = COMPLETIONMATCHER_ICASE
  elseif lspOptions.completionMatcher == 'fuzzy'
    lspOptions.completionMatcherValue = COMPLETIONMATCHER_FUZZY
  else
    lspOptions.completionMatcherValue = COMPLETIONMATCHER_CASE
  endif

  # Apply the changed options
  if exists('#LspCmds#User#LspOptionsChanged')
    :doautocmd <nomodeline> LspCmds User LspOptionsChanged
  endif
enddef

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.

none: change to use 'LspInlayHintsUpdateNow'

fix: inlay hints maybe a bit delayed if sync init

Signed-off-by: shane.xb.qian <shane.qian@foxmail.com>

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