Skip to content

Commit 39b6f12

Browse files
committed
fix(build): Ensure tree shaking works properly for ESM output
Apparently, webpack looks at `sideEffects` in the generated package.json, and based on this does not tree shake correctly :grimace:
1 parent 557d846 commit 39b6f12

File tree

5 files changed

+14
-83
lines changed

5 files changed

+14
-83
lines changed

dev-packages/browser-integration-tests/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@
4545
"babel-loader": "^8.2.2",
4646
"html-webpack-plugin": "^5.5.0",
4747
"pako": "^2.1.0",
48-
"webpack": "^5.52.0"
48+
"webpack": "^5.90.3"
4949
},
5050
"devDependencies": {
5151
"@types/glob": "8.0.0",

dev-packages/rollup-utils/plugins/make-esm-plugin.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ export function makePackageNodeEsm() {
99
this.emitFile({
1010
type: 'asset',
1111
fileName: 'package.json',
12-
source: '{ "type": "module" }',
12+
source: '{ "type": "module", "sideEffects": false }',
1313
});
1414
},
1515
};

packages/browser/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@
7272
"node-fetch": "^2.6.0",
7373
"playwright": "^1.31.1",
7474
"sinon": "^7.3.2",
75-
"webpack": "^4.47.0"
75+
"webpack": "^5.90.3"
7676
},
7777
"scripts": {
7878
"build": "run-p build:transpile build:bundle build:types",

packages/ember/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@
8080
"qunit": "~2.19.2",
8181
"qunit-dom": "~2.0.0",
8282
"sinon": "15.2.0",
83-
"webpack": "~5.74.0"
83+
"webpack": "~5.90.3"
8484
},
8585
"engines": {
8686
"node": ">=14.18"

yarn.lock

Lines changed: 10 additions & 79 deletions
Original file line numberDiff line numberDiff line change
@@ -6964,11 +6964,6 @@
69646964
resolved "https://registry.yarnpkg.com/@types/estree/-/estree-0.0.50.tgz#1e0caa9364d3fccd2931c3ed96fdbeaa5d4cca83"
69656965
integrity sha512-C6N5s2ZFtuZRj54k2/zyRhNDjJwwcViAM3Nbm8zjBpbqAdZ00mr0CFxvSKeO8Y/e03WVFLpQMdHYVfUd6SB+Hw==
69666966

6967-
"@types/estree@^0.0.51":
6968-
version "0.0.51"
6969-
resolved "https://registry.yarnpkg.com/@types/estree/-/estree-0.0.51.tgz#cfd70924a25a3fd32b218e5e420e6897e1ac4f40"
6970-
integrity sha512-CuPgU6f3eT/XgKKPqKd/gLZV1Xmvf1a2R5POBOGQa6uv82xpls89HU5zKeVoyR8XzHd1RGNOlQlvUe3CFkjWNQ==
6971-
69726967
69736968
version "4.17.31"
69746969
resolved "https://registry.yarnpkg.com/@types/express-serve-static-core/-/express-serve-static-core-4.17.31.tgz#a1139efeab4e7323834bb0226e62ac019f474b2f"
@@ -15128,7 +15123,7 @@ [email protected]:
1512815123
memory-fs "^0.5.0"
1512915124
tapable "^1.0.0"
1513015125

15131-
enhanced-resolve@^4.3.0, enhanced-resolve@^4.5.0:
15126+
enhanced-resolve@^4.3.0:
1513215127
version "4.5.0"
1513315128
resolved "https://registry.yarnpkg.com/enhanced-resolve/-/enhanced-resolve-4.5.0.tgz#2f3cfd84dbe3b487f18f2db2ef1e064a571ca5ec"
1513415129
integrity sha512-Nv9m36S/vxpsI+Hc4/ZGRs0n9mXqSWGGq49zxb/cJfPAQMbUtttJAlNPS4AQzaBdw/pKskw5bMbekT/Y7W/Wlg==
@@ -15137,14 +15132,6 @@ enhanced-resolve@^4.3.0, enhanced-resolve@^4.5.0:
1513715132
memory-fs "^0.5.0"
1513815133
tapable "^1.0.0"
1513915134

15140-
enhanced-resolve@^5.10.0, enhanced-resolve@^5.3.2:
15141-
version "5.10.0"
15142-
resolved "https://registry.yarnpkg.com/enhanced-resolve/-/enhanced-resolve-5.10.0.tgz#0dc579c3bb2a1032e357ac45b8f3a6f3ad4fb1e6"
15143-
integrity sha512-T0yTFjdpldGY8PmuXXR0PyQ1ufZpEGiHVrp7zHKB7jdR4qlmZHhONVM5AQOAWXuF/w3dnHbEQVrNptJgt7F+cQ==
15144-
dependencies:
15145-
graceful-fs "^4.2.4"
15146-
tapable "^2.2.0"
15147-
1514815135
enhanced-resolve@^5.15.0:
1514915136
version "5.15.0"
1515015137
resolved "https://registry.yarnpkg.com/enhanced-resolve/-/enhanced-resolve-5.15.0.tgz#1af946c7d93603eb88e9896cee4904dc012e9c35"
@@ -15153,6 +15140,14 @@ enhanced-resolve@^5.15.0:
1515315140
graceful-fs "^4.2.4"
1515415141
tapable "^2.2.0"
1515515142

15143+
enhanced-resolve@^5.3.2:
15144+
version "5.10.0"
15145+
resolved "https://registry.yarnpkg.com/enhanced-resolve/-/enhanced-resolve-5.10.0.tgz#0dc579c3bb2a1032e357ac45b8f3a6f3ad4fb1e6"
15146+
integrity sha512-T0yTFjdpldGY8PmuXXR0PyQ1ufZpEGiHVrp7zHKB7jdR4qlmZHhONVM5AQOAWXuF/w3dnHbEQVrNptJgt7F+cQ==
15147+
dependencies:
15148+
graceful-fs "^4.2.4"
15149+
tapable "^2.2.0"
15150+
1515615151
enhanced-resolve@^5.8.0:
1515715152
version "5.12.0"
1515815153
resolved "https://registry.yarnpkg.com/enhanced-resolve/-/enhanced-resolve-5.12.0.tgz#300e1c90228f5b570c4d35babf263f6da7155634"
@@ -15290,11 +15285,6 @@ es-module-lexer@^0.7.1:
1529015285
resolved "https://registry.yarnpkg.com/es-module-lexer/-/es-module-lexer-0.7.1.tgz#c2c8e0f46f2df06274cdaf0dd3f3b33e0a0b267d"
1529115286
integrity sha512-MgtWFl5No+4S3TmhDmCz2ObFGm6lEpTnzbQi+Dd+pw4mlTIZTmM2iAs5gRlmx5zS9luzobCSBSI90JM/1/JgOw==
1529215287

15293-
es-module-lexer@^0.9.0:
15294-
version "0.9.3"
15295-
resolved "https://registry.yarnpkg.com/es-module-lexer/-/es-module-lexer-0.9.3.tgz#6f13db00cc38417137daf74366f535c8eb438f19"
15296-
integrity sha512-1HQ2M2sPtxwnvOvT1ZClHyQDiggdNjURWpY2we6aMKCQiUVxTmVs2UYPLIrD84sS+kMdUwfBSylbJPwNnBrnHQ==
15297-
1529815288
es-module-lexer@^1.2.1:
1529915289
version "1.4.1"
1530015290
resolved "https://registry.yarnpkg.com/es-module-lexer/-/es-module-lexer-1.4.1.tgz#41ea21b43908fe6a287ffcbe4300f790555331f5"
@@ -33182,66 +33172,7 @@ [email protected]:
3318233172
watchpack "^2.2.0"
3318333173
webpack-sources "^3.2.0"
3318433174

33185-
webpack@^4.47.0:
33186-
version "4.47.0"
33187-
resolved "https://registry.yarnpkg.com/webpack/-/webpack-4.47.0.tgz#8b8a02152d7076aeb03b61b47dad2eeed9810ebc"
33188-
integrity sha512-td7fYwgLSrky3fI1EuU5cneU4+pbH6GgOfuKNS1tNPcfdGinGELAqsb/BP4nnvZyKSG2i/xFGU7+n2PvZA8HJQ==
33189-
dependencies:
33190-
"@webassemblyjs/ast" "1.9.0"
33191-
"@webassemblyjs/helper-module-context" "1.9.0"
33192-
"@webassemblyjs/wasm-edit" "1.9.0"
33193-
"@webassemblyjs/wasm-parser" "1.9.0"
33194-
acorn "^6.4.1"
33195-
ajv "^6.10.2"
33196-
ajv-keywords "^3.4.1"
33197-
chrome-trace-event "^1.0.2"
33198-
enhanced-resolve "^4.5.0"
33199-
eslint-scope "^4.0.3"
33200-
json-parse-better-errors "^1.0.2"
33201-
loader-runner "^2.4.0"
33202-
loader-utils "^1.2.3"
33203-
memory-fs "^0.4.1"
33204-
micromatch "^3.1.10"
33205-
mkdirp "^0.5.3"
33206-
neo-async "^2.6.1"
33207-
node-libs-browser "^2.2.1"
33208-
schema-utils "^1.0.0"
33209-
tapable "^1.1.3"
33210-
terser-webpack-plugin "^1.4.3"
33211-
watchpack "^1.7.4"
33212-
webpack-sources "^1.4.1"
33213-
33214-
webpack@^5.52.0, webpack@~5.74.0:
33215-
version "5.74.0"
33216-
resolved "https://registry.yarnpkg.com/webpack/-/webpack-5.74.0.tgz#02a5dac19a17e0bb47093f2be67c695102a55980"
33217-
integrity sha512-A2InDwnhhGN4LYctJj6M1JEaGL7Luj6LOmyBHjcI8529cm5p6VXiTIW2sn6ffvEAKmveLzvu4jrihwXtPojlAA==
33218-
dependencies:
33219-
"@types/eslint-scope" "^3.7.3"
33220-
"@types/estree" "^0.0.51"
33221-
"@webassemblyjs/ast" "1.11.1"
33222-
"@webassemblyjs/wasm-edit" "1.11.1"
33223-
"@webassemblyjs/wasm-parser" "1.11.1"
33224-
acorn "^8.7.1"
33225-
acorn-import-assertions "^1.7.6"
33226-
browserslist "^4.14.5"
33227-
chrome-trace-event "^1.0.2"
33228-
enhanced-resolve "^5.10.0"
33229-
es-module-lexer "^0.9.0"
33230-
eslint-scope "5.1.1"
33231-
events "^3.2.0"
33232-
glob-to-regexp "^0.4.1"
33233-
graceful-fs "^4.2.9"
33234-
json-parse-even-better-errors "^2.3.1"
33235-
loader-runner "^4.2.0"
33236-
mime-types "^2.1.27"
33237-
neo-async "^2.6.2"
33238-
schema-utils "^3.1.0"
33239-
tapable "^2.1.1"
33240-
terser-webpack-plugin "^5.1.3"
33241-
watchpack "^2.4.0"
33242-
webpack-sources "^3.2.3"
33243-
33244-
webpack@^5.90.3:
33175+
webpack@^5.90.3, webpack@~5.90.3:
3324533176
version "5.90.3"
3324633177
resolved "https://registry.yarnpkg.com/webpack/-/webpack-5.90.3.tgz#37b8f74d3ded061ba789bb22b31e82eed75bd9ac"
3324733178
integrity sha512-h6uDYlWCctQRuXBs1oYpVe6sFcWedl0dpcVaTf/YF67J9bKvwJajFulMVSYKHrksMB3I/pIagRzDxwxkebuzKA==

0 commit comments

Comments
 (0)