[mini.deps] Add pre/post_remove hooks?

[jamestansx]

Since we have pre/post_install hooks, I think it would be useful to have a hook that run before/after removing the package. The usecase is that one might need to cleanup the stuff that was added in the install hook (example).

[echasnovski]

Thanks for the suggestion!

They were considered, but they have shortcomings which I find significant:

• I would like users to be able to remove plugins by just deleting its directory. The reason is that usually plugin-to-be-removed is obsolete and not used, so removing from outside of Neovim doesn't make much sense to me.
• In order for those hooks to be called, plugin should be loaded. So to remove plugin user should first load it with proper hooks. As, again, plugin-to-be-removed is usually obsolete and not loaded to begin with, it makes the whole approach quite unintuitive to me.

I am afraid this won't be added. Unless, probably, the upstream (Neovim) finalizes the packspec which will include those hooks.

[jamestansx]

Understood. Actually this is because I need some way to cleanup the luarocks packages when the associated plugins are removed. The plugins don't need to be loaded.

Anyhow, I totally understand your decision. Thanks for the awesome plugins!

[echasnovski]

The plugins don't need to be loaded.

They need to be loaded in a sense that the only way to define hooks (including potential "remove" hooks) is inside add(), which adds and loads a plugin.