Plenary Callback issue in dependent plugin about plenary.nvim HOT 8 CLOSED

lazy.nvim just clones the repo in .local/share/nvim/lazy/plenary.nvim it should be enough to open that directory and manually checkout that branch. even with gh pr checkout 589 and after that switch back to the previous branch and later updating with :Lazy. you dont need to get fancy

from plenary.nvim.

thanks for testing

from plenary.nvim.

fixed by #589

my bad for introducing this regression, didnt think that anyone is using the module in an fast event. could you test that PR?

from plenary.nvim.

@Conni2461 thanks so much for getting back so quickly. I would love to test the PR. However, this is my first time foraying into version control with Lazy. Is it possible for your to provide some guidance? I've tried:

  {
    "CopilotC-Nvim/CopilotChat.nvim",
    branch = "canary",
    dependencies = {
      { "zbirenbaum/copilot.lua" },                                       -- or github/copilot.vim
      { "Conni2461/plenary.nvim", branch = "fix/in_fast_loop" }, -- for curl, log wrapper
    },
    opts = {
      debug = true, -- Enable debugging
      -- See Configuration section for rest
    },
    -- See Commands section for default commands if you want to lazy load on them
  },
}

as well as

  {
    "CopilotC-Nvim/CopilotChat.nvim",
    branch = "canary",
    dependencies = {
      { "zbirenbaum/copilot.lua" },
      { "nvim-lua/plenary.nvim", branch = "Conni2461:fix/in_fast_loop" }, 
    },
    opts = {
      debug = true,
    },
  },
}

to try and depend on your branch, but both options give me a refspec error.

I've been just calling nvim-lua/plenary.nvimas a dependency for a few plugins and tried simply changing the dependency on that plugin, so maybe its a namespace issue somehow? Sorry I am not able to test it yet. Later on I can try to install plenary directly and do the same versioning there if no one has any insight.

Thanks so much for your help, apologies again

from plenary.nvim.

Thanks Conni, that made good, plain sense and put my head on straight about the actual relationship Lazy has with plugins. I am no longer receiving the error I opened this issue for when using your branch.

Thanks so much, and again I can confirm this fixes the plenary issue.

from plenary.nvim.

Apologies for the regression @Conni2461

it was my PR that introduced it! I don't really understand what was going wrong, was it the inner function calling before the inline function could return? Id like to understand what I could have done better, and why.

Apologies for the head-scratching!

from plenary.nvim.

Apologies for the regression

don't worry, it can happen. The problem was that your flatten method was called in a fast lua loop, and contained a vim.fn.has function call with is not api-fast and we are not allowed to call that at this point in time. The solution now is to do the vim.fn.has check on module load, which is always safe to do and then either set compat.flatten to the old or the new way of doing things, the version value will also never change while nvim is open, so its safe to do that and cache the result like that

you can read more here :help api-fast and :help vim.in_fast_event()

from plenary.nvim.

@Conni2461 thanks for the fix:)

from plenary.nvim.