lazy.nvim resets custom mappings when using `keys` about leap.nvim

[ggandor]

I cannot reproduce, the problem is not with this mapping itself, it should work.

[ggandor]

Sure, but I don't fully understand the problem (what Lazy is doing in the background), and cannot really follow the comment in your code snippet, so please walk me through this then.

[ShaolunWang]

@Sleepful This would only work with the first leap. On the second leap it shows:

E5108: Error executing lua: ~/.local/share/nvim/lazy/leap.nvim/lua/leap/main.lua:405: attempt to index local '_local_67_' (a nil value)
stack traceback:
        ~/.local/share/nvim/lazy/leap.nvim/lua/leap/main.lua:405: in function 'leap'
        ~/.config/nvim/lua/plugins/leap.lua:11: in function <~/.config/nvim/lua/plugins/leap.lua:10>
        ```

[cd-a]

see LazyVim/LazyVim#2379 (comment) for solution

[ggandor]

TL;DR: don't use keys, it's unnecessary.

[ShaolunWang]

Right, thanks for confirming that!

[ShaolunWang]

I just narrowed down the issue I had,fyi this is triggered by setting it like this:

require('lazy').setup({
  {
    'ggandor/leap.nvim',
    keys = {'s'} 
    -- with this keys lazy loading option,
    -- at each leap it calls lua/leap/main.lua: line 1202
    -- it also depends on the 
    -- and it will get reset to default vim s key
  }
})

vim.keymap.set({ 'n', 'x', 'o' }, 's', function()
  local current_window = vim.fn.win_getid()
  require('leap').leap({ target_windows = { current_window } })
end, { noremap = true})

Despite said here in the readme that lazy loading isn't necessary, there would potentially still be people like me who interpret the wording as lazy loading leap wouldn't result in unexpected behavior(s). Would it be possible to state this somewhere?

[ShaolunWang]

@ggandor sorry for the ping, I forgot to reopen the issue. Could you please take another look?

[ShaolunWang]

So the problem I had was lazy would reset the keybinds when I use the lazy load on keypress option, if I decided to disable the default keymaps and only map s/S to leap actions. Those keybinds get reset to the vim s/S after the first leap action

[Sleepful]

did you try this?

require('lazy').setup({
  {
    'ggandor/leap.nvim',
    keys = {
        "s",
        function()
          require('leap').leap()
        end,
        mode = { "x", "n" },
        desc = "Leap",
      }
  }
})

[cd-a]

Did anyone find a solution to this?

[ShaolunWang]

I will close it since it's not a leap issue