Plugin introducing a (control?) character into vim buffers about codeium.vim HOT 4 CLOSED

@toofar thanks for looking into this. Happy to accept this as a PR if you'd like to create one, otherwise we'll likely make this change ourselves then.

from codeium.vim.

@fortenforge I can also reproduce this on macOS but not Ubuntu

from codeium.vim.

I can reproduce this (long line of control characters printed over the buffer as pictured in #126) with wezterm and vim 9.0 (patches 1-1658). On the same system using urxvt it only puts a couple of characters there instead of a huge list.

I did the old "follow the logic flow and comment out functionality until the symptoms change" trick and traced it to these two system() calls: https://github.com/Exafunction/codeium.vim/blob/7e0054a/autoload/codeium/server.vim#L123-L124

Removing the substitute and using echom on the raw string returned from system() it looks like the "garbage" on the screen is control characters that are read after the end of the output from uname. Interestingly, when I comment out the call to StartLanguageServer that is triggering that code path the garbage doesn't show up when I later call CodeiumEnable, so possibly it's something that only happens when system() is called while vim is starting up before it's initialized the terminal fully.

Anyway, :help system says to prepend the calls with silent when you don't need user input because it "avoids stray characters showing up on the screen", and that seems to resolve the issue here just fine:

diff --git i/autoload/codeium/server.vim w/autoload/codeium/server.vim
index 569c4abf5171..9cd65ea9fa73 100644
--- i/autoload/codeium/server.vim
+++ w/autoload/codeium/server.vim
@@ -120,8 +120,8 @@ function! s:SendHeartbeat(timer) abort
 endfunction

 function! codeium#server#Start(...) abort
-  let os = substitute(system('uname'), '\n', '', '')
-  let arch = substitute(system('uname -m'), '\n', '', '')
+  silent let os = substitute(system('uname'), '\n', '', '')
+  silent let arch = substitute(system('uname -m'), '\n', '', '')
   let is_arm = stridx(arch, 'arm') == 0 || stridx(arch, 'aarch64') == 0

   if os ==# 'Linux' && is_arm

from codeium.vim.

Wow really good find

from codeium.vim.