Skip to content

Commit 5dc0f72

Browse files
authoredJun 23, 2022
feat: enable tree-shaking for lib es (#8737)
1 parent 31348b5 commit 5dc0f72

File tree

1 file changed

+5
-7
lines changed

1 file changed

+5
-7
lines changed
 

‎packages/vite/src/node/plugins/esbuild.ts

+5-7
Original file line numberDiff line numberDiff line change
@@ -237,12 +237,7 @@ export const buildEsbuildPlugin = (config: ResolvedConfig): Plugin => {
237237
}
238238

239239
const target = config.build.target
240-
const minify =
241-
config.build.minify === 'esbuild' &&
242-
// Do not minify ES lib output since that would remove pure annotations
243-
// and break tree-shaking
244-
// https://github.com/vuejs/core/issues/2860#issuecomment-926882793
245-
!(config.build.lib && opts.format === 'es')
240+
const minify = config.build.minify === 'esbuild'
246241

247242
if ((!target || target === 'esnext') && !minify) {
248243
return null
@@ -253,7 +248,10 @@ export const buildEsbuildPlugin = (config: ResolvedConfig): Plugin => {
253248
target: target || undefined,
254249
...(minify
255250
? {
256-
minify,
251+
// Do not minify ES lib output since that would remove pure annotations
252+
// and break tree-shaking
253+
// https://github.com/vuejs/core/issues/2860#issuecomment-926882793
254+
minify: !(config.build.lib && opts.format === 'es'),
257255
treeShaking: true,
258256
format: rollupToEsbuildFormatMap[opts.format]
259257
}

0 commit comments

Comments
 (0)
Please sign in to comment.