feat(completion): conditionally remove "t" flag in 'complete' option

echasnovski · echasnovski · commit eb4d6bd268f7 · 2025-09-16T13:51:03.000+03:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -39,6 +39,10 @@ There are following change types:
 
 ## mini.completion
 
+### Evolve
+
+- Update `setup()` to remove "t" flag from 'complete' option (if not previously set and fallback action is default) as it leads to visible lags.
+
 ### Expand
 
 - Add `MiniCompletionWindowOpen` and `MiniCompletionWindowUpdate` events.
diff --git a/doc/mini-completion.txt b/doc/mini-completion.txt
@@ -88,6 +88,8 @@ Some options are set automatically (if not set before |MiniCompletion.setup()|):
   To enable fuzzy matching, manually set to "menuone,noselect,fuzzy". Consider
   also adding "nosort" flag to preserve initial order when filtering.
 - 'shortmess' is appended with "c" flag for silent <C-n> fallback.
+- 'complete' gets removed "t" flag (if fallback action is default), as it
+  leads to visible lags.
 
 # Snippets ~
 
diff --git a/lua/mini/completion.lua b/lua/mini/completion.lua
@@ -88,6 +88,8 @@
 ---   To enable fuzzy matching, manually set to "menuone,noselect,fuzzy". Consider
 ---   also adding "nosort" flag to preserve initial order when filtering.
 --- - 'shortmess' is appended with "c" flag for silent <C-n> fallback.
+--- - 'complete' gets removed "t" flag (if fallback action is default), as it
+---   leads to visible lags.
 ---
 --- # Snippets ~
 ---
@@ -796,6 +798,10 @@ H.apply_config = function(config)
   local shortmess_flags = 'c' .. (vim.fn.has('nvim-0.10') == 0 and 'C' or '')
   was_set = vim.api.nvim_get_option_info2('shortmess', { scope = 'global' }).was_set
   if not was_set then vim.opt.shortmess:append(shortmess_flags) end
+
+  -- - Remove "t" flag to reduce visible lags
+  was_set = vim.api.nvim_get_option_info2('complete', { scope = 'global' }).was_set
+  if not was_set and config.fallback_action == '<C-n>' then vim.opt.complete:remove('t') end
 end
 
 H.create_autocommands = function(config)
diff --git a/tests/test_completion.lua b/tests/test_completion.lua
@@ -276,13 +276,16 @@ T['setup()']['sets recommended option values'] = function()
   expect.match(child.o.shortmess, 'c')
   expect.match(child.o.shortmess, 'C')
   eq(child.o.completeopt, 'menuone,noselect')
+  eq(child.o.complete:find('t'), nil)
 
   -- Should not set if was previously set
   child.o.shortmess = 'ltToO'
   child.o.completeopt = 'menu,noinsert'
+  child.o.complete = '.,w,b,u,t'
   reload_module()
   eq(child.o.shortmess, 'ltToO')
   eq(child.o.completeopt, 'menu,noinsert')
+  eq(child.o.complete, '.,w,b,u,t')
 end
 
 T['default_process_items()'] = new_set({
