chore(starters): add sideEffects: false to the lib template 😎

[gioboa]

Close #7854

Otherwise there might be some side effects imports when building a consumer project, which will be added as static imports to the bundle graph, leading to slight over-prefetching of unnecessary bundles for user events.

[changeset-bot]

🦋 Changeset detected

Latest commit: 71ea2dd

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages

Name	Type
create-qwik	Patch
@builder.io/qwik	Patch
@builder.io/qwik-city	Patch
eslint-plugin-qwik	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[github-actions]

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name	Status	Preview	Last Commit
qwik-docs	✅ Ready (View Log)	Visit Preview	71ea2dd

[wmertens]

Yes great, but the README should include a paraghraph saying that the side-effects are set to false, and that if there are side effects (like a global being set), then it should be removed, possibly only for the file that has it.

That field isn't a part of the npm spec, but it is used by bundlers, and the best documentation is at https://webpack.js.org/guides/tree-shaking/#mark-the-file-as-side-effect-free

[gioboa]

Yes great, but the README should include a paraghraph saying that the side-effects are set to false...

Added

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/@builder.io/qwik@7855

npm i https://pkg.pr.new/@builder.io/qwik-city@7855

npm i https://pkg.pr.new/eslint-plugin-qwik@7855

npm i https://pkg.pr.new/create-qwik@7855

commit: 71ea2dd

[gioboa]

That e2e test is flaky, let me run it again

[wmertens]