(mini.clue) reduce changes to minimum

Author: hbiel

lua/mini/clue.lua

@@ -601,7 +601,6 @@ MiniClue.enable_all_triggers = function()
     -- Map only inside valid listed buffers
     if vim.fn.buflisted(buf_id) == 1 then H.map_buf_triggers(buf_id) end
   end
-  H.state.disable_autocmd_triggers = false
 end
 
 --- Enable triggers in buffer
@@ -618,7 +617,6 @@ MiniClue.disable_all_triggers = function()
   for _, buf_id in ipairs(vim.api.nvim_list_bufs()) do
     H.unmap_buf_triggers(buf_id)
   end
-  H.state.disable_autocmd_triggers = true
 end
 
 --- Disable triggers in buffer
@@ -672,6 +670,26 @@ MiniClue.set_mapping_desc = function(mode, lhs, desc)
   if not ok_set then H.error(vim.inspect(desc) .. ' is not a valid description.') end
 end
 
+MiniClue.start_query_process = function(trigger)
+  -- Don't act if for some reason entered another trigger is already active
+  local is_in_exec = type(H.exec_trigger) == 'table'
+    and H.exec_trigger.mode == trigger.mode
+    and H.exec_trigger.keys == trigger.keys
+  if is_in_exec then
+    H.exec_trigger = nil
+    return
+  end
+
+  -- Start user query
+  H.state_set(trigger, { trigger.keys })
+
+  -- Do not advance if no other clues to query. NOTE: it is `<= 1` and not
+  -- `<= 0` because the "init query" mapping should match.
+  if vim.tbl_count(H.state.clues) <= 1 then return H.state_exec() end
+
+  H.state_advance()
+end
+
 --- Generate pre-configured clues
 ---
 --- This is a table with function elements. Call to actually get array of clues.
@@ -1192,10 +1210,6 @@ H.create_autocommands = function(config)
   au('RecordingLeave', '*', MiniClue.enable_all_triggers, 'Enable all triggers')
 
   au('VimResized', '*', H.window_update, 'Update window on resize')
-
-  if vim.fn.has('nvim-0.10') == 1 then
-    au('ModeChanged', 'n:no', function() H.start_query({ mode = "o", keys = "" }) end, 'Trigger on change to operator-pending mode')
-  end
 end
 
 --stylua: ignore
@@ -1235,31 +1249,6 @@ H.get_buf_var = function(buf_id, name)
 end
 
 -- Triggers -------------------------------------------------------------------
-H.start_query = function(trigger)
-  if vim.fn.has('nvim-0.10') == 1 and vim.fn.state('m') ~= '' then
-    return
-  end
-
-  if H.state.disable_autocmd_triggers then
-    return
-  end
-
-  -- Don't act if for some reason entered another trigger is already active
-  local is_in_exec = type(H.exec_trigger) == 'table'
-  if is_in_exec then
-    return
-  end
-
-  -- Start user query
-  H.state_set(trigger, { trigger.keys })
-
-  -- Do not advance if no other clues to query. NOTE: it is `<= 1` and not
-  -- `<= 0` because the "init query" mapping should match.
-  if vim.tbl_count(H.state.clues) <= 1 then return H.state_exec() end
-
-  H.state_advance()
-end
-
 H.map_buf_triggers = function(buf_id)
   if not H.is_valid_buf(buf_id) or H.is_disabled(buf_id) then return end
 
@@ -1277,7 +1266,7 @@ H.unmap_buf_triggers = function(buf_id)
 end
 
 H.map_trigger = function(buf_id, trigger)
-  if not H.is_valid_buf(buf_id) then return end
+  if not H.is_valid_buf(buf_id) or trigger.keys == '' then return end
 
   -- Compute mapping RHS
   trigger.keys = H.replace_termcodes(trigger.keys)
@@ -1289,11 +1278,11 @@ H.map_trigger = function(buf_id, trigger)
   local opts = { buffer = buf_id, nowait = true, desc = desc }
 
   -- Create mapping. Use translated variant to make it work with <F*> keys.
-  vim.keymap.set(trigger.mode, keys_trans, function() H.start_query(trigger) end, opts)
+  vim.keymap.set(trigger.mode, keys_trans, function() MiniClue.start_query_process(trigger) end, opts)
 end
 
 H.unmap_trigger = function(buf_id, trigger)
-  if not H.is_valid_buf(buf_id) then return end
+  if not H.is_valid_buf(buf_id) or trigger.keys == '' then return end
 
   trigger.keys = H.replace_termcodes(trigger.keys)
 
@@ -1383,20 +1372,20 @@ H.state_exec = function()
   local has_postkeys = (clue or {}).postkeys ~= nil
   H.state_reset(has_postkeys)
 
-  -- Disable triggers !!!VERY IMPORTANT!!!
+  -- Disable trigger !!!VERY IMPORTANT!!!
   -- This is a workaround against infinite recursion (like if `g` is trigger
   -- then typing `gg`/`g~` would introduce infinite recursion).
   local buf_id = vim.api.nvim_get_current_buf()
-  MiniClue.disable_all_triggers()
+  H.unmap_trigger(buf_id, trigger)
 
   -- Execute keys. The `i` flag is used to fully support Operator-pending mode.
   -- Flag `t` imitates keys as if user typed, which is reasonable but has small
   -- downside with edge cases of 'langmap' (like ':\;;\;:') as it "inverts" key
   -- meaning second time (at least in Normal mode).
   vim.api.nvim_feedkeys(keys_to_type, 'mit', false)
 
-  -- Enable triggers back after it can no longer harm
-  vim.schedule(function() MiniClue.enable_all_triggers() end)
+  -- Enable trigger back after it can no longer harm
+  vim.schedule(function() H.map_trigger(buf_id, trigger) end)
 
   -- Apply postkeys (in scheduled fashion)
   if has_postkeys then H.state_apply_postkeys(clue.postkeys) end
@@ -1450,7 +1439,7 @@ H.compute_exec_keys = function()
   -- Normal/Insert mode so extra work should be done to rebuild all keys
   if vim.startswith(cur_mode, 'no') then
     res = H.get_forced_submode() .. res
-    if H.state.trigger.keys ~= "" then
+    if H.state.trigger.keys ~= '' then
       local operator_tweak = H.operator_tweaks[vim.v.operator] or function(x) return x end
       res = operator_tweak(vim.v.operator .. res)
     end