Merge pull request #7908 from maiieul/set-packages-as-side-effect-free

chore: mark all qwik packages as side effect free

Author: wmertens

.changeset/shaggy-pears-shine.md

@@ -0,0 +1,6 @@
+---
+'@builder.io/qwik': patch
+'@builder.io/qwik-city': patch
+---
+
+FEAT: All qwik packages are now marked as side effect free in their package.json. This should remove a few unecessary empty imports added by rollup and then not tree-shaken like `import "./preloader.js"`.

packages/insights/package.json

@@ -70,5 +70,6 @@
     "test.unit": "vitest",
     "test.watch": "vitest --watch"
   },
-  "type": "module"
+  "type": "module",
+  "sideEffects": false
 }

packages/qwik-city/package.json

@@ -188,5 +188,6 @@
     "build": "cd src/runtime && vite build --mode lib"
   },
   "type": "module",
-  "types": "./lib/index.d.ts"
+  "types": "./lib/index.d.ts",
+  "sideEffects": false
 }

packages/qwik-labs/package.json

@@ -59,5 +59,6 @@
     "test.unit": "vitest"
   },
   "type": "module",
-  "types": "./lib/index.d.ts"
+  "types": "./lib/index.d.ts",
+  "sideEffects": false
 }

packages/qwik-react/package.json

@@ -65,5 +65,6 @@
     "typecheck": "tsc --noEmit"
   },
   "type": "module",
-  "types": "./lib/types/index.qwik.d.ts"
+  "types": "./lib/types/index.qwik.d.ts",
+  "sideEffects": false
 }

packages/qwik-worker/package.json

@@ -36,5 +36,6 @@
     "build": "vite build --mode lib"
   },
   "type": "module",
-  "types": "./lib/types/index.d.ts"
+  "types": "./lib/types/index.d.ts",
+  "sideEffects": false
 }

packages/qwik/package.json

@@ -171,5 +171,6 @@
     "directory": "packages/qwik"
   },
   "type": "module",
-  "types": "./dist/core.d.ts"
+  "types": "./dist/core.d.ts",
+  "sideEffects": false
 }