Add side effects to package.json

[denisx]

Add side effects to package.json to help bundlers cut unused code. Good with named imports.

[WebReflection]

I am not sure about this one ... I don't mind adding that extra flags but the underlying issue seems to be bundlers incapable of tree-shaking? 🤔

Or better: what scenario are we trying to solve in here?

[denisx]

Package allready has "type": "module". sideEffects just a full settings for bundlers.
For example, if i use named '{serialize} from index', bundler can cut other exports from client bundle.

[WebReflection]

@denisx again, that's what my bundler of choice (rollup) does already for me ... what is this change trying to solve? which bundler is incapable of proper tree-shaking and requires this change? 🤔

[denisx]

yes, more better treeshake. https://rollupjs.org/configuration-options/#treeshake-modulesideeffects