Improve error message when using .tap() on undefined plugin about webpack-chain HOT 2 CLOSED

I could definitely see how this would be an issue. Technically this issue can arise from not using tap at all:

neutrino.config.plugin('copy');

This is due to the way the .plugin method works:

https://github.com/mozilla-neutrino/webpack-chain/blob/2445476d0d7f444c34060f383c890597ca0d2e8d/src/Config.js#L51-L57

Using these methods creates an entry with that name, and then expects it to be defined. Obviously in the case of conditional plugins, this creates a problem. We do specifically state in the docs about the conditional nature of the plugins:

https://neutrino.js.org/packages/web/#plugins

Note: Some plugins are only available in certain environments. To override them, they should be modified conditionally.

But I agree the error messaging could be better in this scenario. This would have to be done in webpack-chain.

from webpack-chain.

Yeah I agree this is not expected to work - it's the clearer error message that this issue is about :-)

from webpack-chain.