Closed mehalter closed 4 months ago
Also I noticed the optional
specs override the implied "laziness" of a plugin that is only a dependency. Is this intended? It might make more sense for optional
plugin specs not influence implied laziness such as that implied by a plugin only being defined as a dependency to another plugin.
I cant get optional decency working at all. I get errors that the plugin is not found.
When I use this config:
require("lazy").setup({
{
"projekt0n/github-nvim-theme",
priority = 1000,
config = function()
require("github-theme").setup({ options = { transparent = true } })
vim.cmd.colorscheme("github_dark_dimmed")
end,
dependencies = {
{ "echasnovski/mini.nvim", optional = true, lazy= true }, -- should not load
},
},
})
I get this error when i open vim:
Error detected while processing /home/blue/.config/nvim/init.lua:
Plugin mini.nvim not found
@bluebrown I am also getting this behavior. It seems like the feature added in #947 has been broken along the way
@abeldekat any idea what has happened?
@mehalter, I have no idea. My last contribution was in october 2023.
me too.
Also getting this
@folke this issue is still reproducible given the original post instructions. It doesn't seem to be solved at all. Sorry!
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")
-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end
-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath })
end
vim.opt.runtimepath:prepend(lazypath)
-- install plugins
local plugins = {
"folke/tokyonight.nvim",
{
"mfussenegger/nvim-dap",
dependencies = {
{ "hrsh7th/nvim-cmp", optional = true },
},
},
-- add any other plugins here
}
require("lazy").setup(plugins, {
root = root .. "/plugins",
})
vim.cmd.colorscheme("tokyonight")
-- add anything else here
this repro.lua
is also a bug. This is saying nvim-cmp
is an optional dependency but if it is available it needs to be loaded before nvim-dap
and it just throws an error.
So neither of the bugs reported here were resolved in the latest version
You're right. Both should be fixed now. Let me know if you still see any issues!
This looks like it works beautifully with the 2 test cases mentioned here! Thanks so much for the fix! I'll let you know if I run into any issues down the line.
Did you check docs and existing issues?
Neovim version (nvim -v)
NVIM v0.9.5
Operating system/version
Arch Linux
Describe the bug
Given a plugin with some list of dependencies, if the plugin is disabled all of the dependencies should also be disabled if they aren't specified anywhere else.
Similarly if I have a plugin with the
optional = true
key set it should also be ignored unless the plugin is specified explicitly somewhere.If I have a plugin defined as
optional
and also defined as a dependency to a plugin that is disabled the plugin should not be enabled.My guess is the calculation of
optional
is happening before filtering out plugin specs for disabled plugins which breaks the behavior ofoptional
.Steps To Reproduce
nvim -u repro.lua
, run the minimal config below:Lazy
, open lazy and see thatnvim-cmp
is not disabledExpected Behavior
Plugins listed as dependencies of disabled plugins shouldn't trigger
optional
plugin specs to be enabled. Since at the time of calculation it is not valid.Repro