The namedExports option from "@rollup/plugin-commonjs" is deprecated

[elpupi]

Stencil version:

 @stencil/[email protected]

I'm submitting a:

[x] bug report
[x] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://stencil-worldwide.herokuapp.com/ or https://forum.ionicframework.com/

Current behavior:

stencil.config.ts allows to specify commonjs plugin option. Firstly, only namedExports is allowed in commonjs?: BundlingConfig. Secondly, the namedExportsoption is deprecated in@rollup/plugin-commonjs`.

Running stencil build, the following message is shown:

[ WARN  ]  Bundling Warning PLUGIN_WARNING
           The namedExports option from "@rollup/plugin-commonjs" is deprecated. Named exports are now handled

Looking at the @rollup/plugin-commonjs code commonjs/src/index.js, the option is ignored. So it is really misleading and error prone when updating the version up as it was working before.

Expected behavior:

Remove namedExports from BundlingConfig and show an error message. And if possible, allow the other commonjs options.

[rwaskiewicz]

The fix for this issue has been released as a part of today's Stencil v4.0.2 release. We've officially deprecated the field and marked it as such in our documentation site. To avoid a (small) breaking change, we will not remove this field until Stencil v5