Merge branch 'master' into scroll-border

prabirshrestha · web-flow · commit 0235b1dd7c86 · 2024-08-04T17:02:49.000-07:00
diff --git a/.github/workflows/mac_neovim.yml b/.github/workflows/mac_neovim.yml
@@ -51,19 +51,11 @@ jobs:
           repository: thinca/vim-themis
           path: ./vim-themis
           ref: v1.5.5
-      - name: Cache gopls
-        id: cache-gopls
-        uses: actions/cache@v4
-        with:
-          path: bin/gopls
-          key: ${{ runner.os }}-${{ env.VIM_LSP_GO_VERSION }}-${{ env.VIM_LSP_GOPLS_VERSION }}-${{ env.VIM_LSP_GOPLS_CACHE_VER }}-gopls
       - name: Install Go for gopls
-        if: steps.cache-gopls.outputs.cache-hit != 'true'
         uses: actions/setup-go@v5
         with:
           go-version: ${{ env.VIM_LSP_GO_VERSION }}
       - name: Install gopls
-        if: steps.cache-gopls.outputs.cache-hit != 'true'
         shell: bash
         run: |
           go install golang.org/x/tools/gopls@v${{ env.VIM_LSP_GOPLS_VERSION }}
diff --git a/.github/workflows/mac_vim.yml b/.github/workflows/mac_vim.yml
@@ -27,19 +27,11 @@ jobs:
           repository: thinca/vim-themis
           path: ./vim-themis
           ref: v1.5.5
-      - name: Cache gopls
-        id: cache-gopls
-        uses: actions/cache@v4
-        with:
-          path: bin/gopls
-          key: ${{ runner.os }}-${{ env.VIM_LSP_GO_VERSION }}-${{ env.VIM_LSP_GOPLS_VERSION }}-${{ env.VIM_LSP_GOPLS_CACHE_VER }}-gopls
       - name: Install Go for gopls
-        if: steps.cache-gopls.outputs.cache-hit != 'true'
         uses: actions/setup-go@v5
         with:
           go-version: ${{ env.VIM_LSP_GO_VERSION }}
       - name: Install gopls
-        if: steps.cache-gopls.outputs.cache-hit != 'true'
         shell: bash
         run: |
           go install golang.org/x/tools/gopls@v${{ env.VIM_LSP_GOPLS_VERSION }}
diff --git a/autoload/health/lsp.vim b/autoload/health/lsp.vim
@@ -8,7 +8,7 @@ endf
 
 
 function! health#lsp#check() abort
-    call health#report_start('server status')
+    call s:report_start('server status')
     let l:server_status = lsp#collect_server_status()
 
     let l:has_printed = v:false
@@ -17,21 +17,21 @@ function! health#lsp#check() abort
 
         let l:status_msg = printf('%s: %s', l:k, l:report.status)
         if l:report.status == 'running'
-            call health#report_ok(l:status_msg)
+            call s:report_ok(l:status_msg)
         elseif l:report.status == 'failed'
             call health#report_error(l:status_msg, 'See :help g:lsp_log_verbose to debug server failure.')
         else
-            call health#report_warn(l:status_msg)
+            call s:report_warn(l:status_msg)
         endif
         let l:has_printed = v:true
     endfor
 
     if !l:has_printed
-        call health#report_warn('no servers connected')
+        call s:report_warn('no servers connected')
     endif
 
     for l:k in sort(keys(l:server_status))
-        call health#report_start(printf('server configuration: %s', l:k))
+        call s:report_start(printf('server configuration: %s', l:k))
         let l:report = l:server_status[l:k]
 
         let l:msg = "\t\n"
@@ -54,12 +54,35 @@ function! health#lsp#check() abort
         call health#report_info(l:msg)
     endfor
 
-    call health#report_start('Performance')
+    call s:report_start('Performance')
     if lsp#utils#has_lua() && g:lsp_use_lua
-        call health#report_ok('Using lua for faster performance.')
+        call s:report_ok('Using lua for faster performance.')
     else
-        call health#report_warn('Missing requirements to enable lua for faster performance.')
+        call s:report_warn('Missing requirements to enable lua for faster performance.')
     endif
 
 endf
 
+function! s:report_start(report) abort
+  if has('nvim-0.10')
+    call v:lua.vim.health.start(a:report)
+  else
+    call health#report_start(a:report)
+  endif
+endf
+
+function! s:report_warn(report) abort
+  if has('nvim-0.10')
+    call v:lua.vim.health.warn(a:report)
+  else
+    call health#report_warn(a:report)
+  endif
+endf
+
+function! s:report_ok(report) abort
+  if has('nvim-0.10')
+    call v:lua.vim.health.ok(a:report)
+  else
+    call health#report_ok(a:report)
+  endif
+endf
diff --git a/autoload/lsp.vim b/autoload/lsp.vim
@@ -750,7 +750,7 @@ function! s:text_changes(buf, server_name) abort
     endif
 
     " When syncKind is Incremental and previous content is saved.
-    if l:sync_kind == 2 && has_key(s:file_content, a:buf)
+    if l:sync_kind == 2 && has_key(s:file_content, a:buf) && has_key(s:file_content[a:buf], a:server_name)
         " compute diff
         let l:old_content = s:get_last_file_content(a:buf, a:server_name)
         let l:new_content = lsp#utils#buffer#_get_lines(a:buf)
diff --git a/autoload/lsp/ui/vim/output.vim b/autoload/lsp/ui/vim/output.vim
@@ -102,6 +102,7 @@ function! s:get_float_positioning(height, width) abort
     let l:height = min([l:height, max([&lines - &cmdheight - l:row, &previewheight])])
 
     let l:style = 'minimal'
+    let l:border = 'double'
     " Positioning is not window but screen relative
     let l:opts = {
         \ 'relative': 'editor',
@@ -110,6 +111,7 @@ function! s:get_float_positioning(height, width) abort
         \ 'width': l:width,
         \ 'height': l:height,
         \ 'style': l:style,
+        \ 'border': l:border,
         \ }
     return l:opts
 endfunction
diff --git a/autoload/lsp/utils/text_edit.vim b/autoload/lsp/utils/text_edit.vim
@@ -200,18 +200,17 @@ function! s:_compare(text_edit1, text_edit2) abort
     return a:text_edit1.range.start.character - a:text_edit2.range.start.character
   endif
   return l:diff
-endfunction
+endfunction 
 
 "
 " _switch
 "
 function! s:_switch(path) abort
-  if bufnr(a:path) >= 0
-    execute printf('keepalt keepjumps %sbuffer!', bufnr(a:path))
-  else
-    execute printf('keepalt keepjumps edit! %s', fnameescape(a:path))
+  if bufnr(a:path) == -1
+    execute printf('badd %s', fnameescape(a:path))
   endif
-endfunction
+  execute printf('keepalt keepjumps %sbuffer!', bufnr(a:path))
+endfunction 
 
 "
 " delete
diff --git a/doc/vim-lsp.txt b/doc/vim-lsp.txt
@@ -58,9 +58,9 @@ CONTENTS                                                  *vim-lsp-contents*
       g:lsp_diagnostics_virtual_text_wrap
 			     |g:lsp_diagnostics_virtual_text_wrap|
       g:lsp_document_code_action_signs_enabled
-			     |g:lsp_document_code_actions_signs_enabled|
+			     |g:lsp_document_code_action_signs_enabled|
       g:lsp_document_code_action_signs_delay
-			     |g:lsp_document_code_actions_signs_delay|
+			     |g:lsp_document_code_action_signs_delay|
       g:lsp_inlay_hints_enabled
 			     |g:lsp_inlay_hints_enabled|
       g:lsp_inlay_hints_delay
@@ -271,7 +271,7 @@ You can use tcp to connect to LSP servers that don't support stdio. Set host
 and port to tcp. The Godot game engine uses 6008 as its LSP port and godot
 ftplugins define gdscript or gdscript3 filetype: >
 
-	au User lsp_setup 
+	au User lsp_setup
 	    \ call lsp#register_server({
 	    \     'name': 'godot',
 	    \     'tcp': "localhost:6008",
@@ -290,7 +290,7 @@ vim-lsp supports the |:CheckHealth| command which can be useful when debugging
 lsp configuration issues.
 
 This command is implemented in vim with the
-[vim-healthcheck](https://github.com/rhysd/vim-healthcheck) plugin. 
+[vim-healthcheck](https://github.com/rhysd/vim-healthcheck) plugin.
 
 WIKI                                               *vim-lsp-configure-wiki*
 For documentation on how to configure other language servers refer
@@ -905,7 +905,7 @@ g:lsp_max_buffer_size                               *g:lsp_max_buffer_size*
     `g:lsp_max_buffer_size` (measured in bytes), the following features
     are disabled:
     * Semantic highlighting
-    
+
     This functionality can be disabled by setting `g:lsp_max_buffer_size`
     to a negative value.
 
@@ -1402,6 +1402,13 @@ The vim |dict| containing information about the server.
 	    Example: >
 		'config': { 'diagnostics': v:false }
 <
+  * env:
+    optional vim |dict|
+    Used to pass environment variables to the cmd.
+    Example: >
+	'env': { 'GOFLAGS': '-tags=wireinject' }
+<
+
 refresh_pattern                                    *vim-lsp-refresh_pattern*
     Type: |String| (|pattern|)
     Default: `'\k*$'`
@@ -2245,7 +2252,7 @@ Popup Formatting                                      *vim-lsp-popup-format*
 
 Popup windows use the |gq| operator for formatting content to the window.
 
-For customization, see 
+For customization, see
 |formatprg|.
 
 ==============================================================================
diff --git a/test/lsp/utils/text_edit.vimspec b/test/lsp/utils/text_edit.vimspec
@@ -2,6 +2,7 @@ function! s:set_text(lines)
     % delete _
     put =a:lines
     execute 'normal ggdd'
+    execute 'file my-file'
 endfunction
 
 function! s:get_text()
@@ -642,6 +643,45 @@ Describe lsp#utils#text_edit
 
             Assert Equals(getbufline(l:target, 1), ['aiueo'])
         End
+
+        It should apply edits to buffer and unloaded file
+            let l:text = ['plop']
+            call s:set_text(l:text)
+            let l:buffer_text = s:get_text()
+            Assert Equals(l:buffer_text, ['plop', ''])
+            let l:target = globpath(&runtimepath, 'test/lsp/utils/text_edit.vimspec')
+            call lsp#utils#text_edit#apply_text_edits(
+              \ lsp#utils#path_to_uri(expand('%')),
+              \  [{
+              \   'range': {
+              \     'start': {
+              \       'line': 0,
+              \       'character': 0,
+              \     },
+              \     'end': {
+              \       'line': 1,
+              \       'character': 0,
+              \     }
+              \   },
+              \   'newText': "buffer\n"
+              \ }])
+            call lsp#utils#text_edit#apply_text_edits(lsp#utils#path_to_uri(l:target), [{
+              \   'range': {
+              \     'start': {
+              \       'line': 0,
+              \       'character': 0,
+              \     },
+              \     'end': {
+              \       'line': 1,
+              \       'character': 0,
+              \     }
+              \   },
+              \   'newText': "unloaded\n"
+              \ }])
+
+            Assert Equals(getbufline(l:target, 1), ['unloaded'])
+            Assert Equals(getbufline(expand('%'), 1), ['buffer'])
+        End
     End
 End
 
