(mini.jump) Update ; to not ask for target in Operator-pending mode.

Author: echasnovski

lua/mini/jump.lua

@@ -354,9 +354,11 @@ H.make_expr_jump = function(backward, till)
   return function()
     if H.is_disabled() then return '' end
 
-    -- Always ask for `target` as this will be used only in operator-pending
-    -- mode. Dot-repeat will be implemented via expression-mapping.
-    local target = H.get_target()
+    -- Ask for `target` for non-repeating jump as this will be used only in
+    -- operator-pending mode. Dot-repeat is supported via expression-mapping.
+    local is_repeat_jump = backward == nil or till == nil
+    local target = is_repeat_jump and MiniJump.state.target or H.get_target()
+
     -- Stop if user supplied invalid target
     if target == nil then return '<Esc>' end
     H.update_state(target, backward, till, vim.v.count1)

tests/test_jump.lua

@@ -811,7 +811,7 @@ T['Repeat jump with ;']['works in Normal and Visual mode'] = new_set({
   end,
 })
 
-T['Repeat jump with ;']['works in Operator-pending mode'] = function()
+T['Repeat jump with ;']['works after jump in Operator-pending mode'] = function()
   -- It doesn't repeat actual operation, just performs same jump
   set_lines({ '1e2e3e4e5e' })
   set_cursor(1, 0)
@@ -829,6 +829,16 @@ T['Repeat jump with ;']['works in Operator-pending mode'] = function()
   eq(get_cursor(), { 1, 3 })
 end
 
+T['Repeat jump with ;']['works in Operator-pending mode'] = function()
+  set_lines({ '1e2e3e4e5e' })
+  set_cursor(1, 0)
+
+  -- Should repeat without asking for target
+  type_keys('f', 'e', 'd', ';')
+  eq(get_lines(), { '13e4e5e' })
+  eq(get_cursor(), { 1, 1 })
+end
+
 T['Repeat jump with ;']['works with different mapping'] = function()
   child.api.nvim_del_keymap('n', ';')
   reload_module({ mappings = { repeat_jump = 'g;' } })