Improve treeshaking by setting package.json sideEffects (#1332)

mattcosta7 · web-flow · commit ec11d7b8a589 · 2021-07-19T13:36:05.000-07:00
* fix: mark sideEffects free

* fix: update sideEffects delcaration in package.json to improve treeshaking

* fix: update sideEffects delcaration in package.json to improve treeshaking

* fix: BaseStyles doesnt use sideeffects

* chore: add changeset
diff --git a/.changeset/beige-years-notice.md b/.changeset/beige-years-notice.md
@@ -0,0 +1,5 @@
+---
+'@primer/components': patch
+---
+
+Side effects are properly declared in package.json
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -5,6 +5,10 @@
   "main": "lib/index.js",
   "module": "lib-esm/index.js",
   "typings": "lib/index.d.ts",
+  "sideEffects": [
+    "(lib|lib-esm)/behaviors/focusZone.js",
+    "(lib|lib-esm)/behaviors/focusTrap.js"
+  ],
   "scripts": {
     "start": "cd docs && npm run develop",
     "predist": "rm -rf dist & rm -rf lib & rm -rf lib-esm",

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +'@primer/components': patch
 +---
++
 +Side effects are properly declared in package.json