fix: support all whichkey methods

[tris203]

closes #81

needs further work on method1

I don't really understand the structure though as the section just gets dropped seemingly

@willothy any ideas?

[tris203]

Minimal repro here

local wk = require("which-key.mappings")

local str = [[
 return {
     w = {
         name = "file", -- optional group name
         o = { ":lua print('hello')<CR>", "Find File" }, -- create a binding with label
         n = {
             function()
                 print("bar")
             end,
             "Foobar",
         }, -- you can also pass functions!
     },
 }, { prefix = "<leader>" }
]]

-- local str = [[
--  return ({
--      ["<leader>wf"] = { ':lua print("hello")<CR>', "hello" },
--  })
-- ]]

local obj = loadstring(str)()

print(vim.inspect(obj))

local wkd = wk.parse(obj)

print(vim.inspect(wkd))

[tris203]

Okay, i think the issue is fixed, but I don't know why the test for the which-key message is now failing....

*edit, that took me an embarrassingly long time to fix...

To explain more there was a few issues at play.
First wrapping the return in params caused it not to return the subobjects properly, then additionally the object needs to be wrapped in a table and unpacked, as otherwise return in the loadstring just returns the first table, not both

[tris203]

clearer comment

[willothy]

lgtm - loadstring can definitely be tricky, it would be good if we could figure out an alternative at some point though I'm not sure what that would be.