Skip to content

Commit

Permalink
feat: deep resolve side effects
Browse files Browse the repository at this point in the history
  • Loading branch information
mihkeleidast committed Jan 24, 2023
1 parent 8a40ea4 commit 9133b4c
Showing 1 changed file with 14 additions and 1 deletion.
15 changes: 14 additions & 1 deletion packages/vite/src/node/packages.ts
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,20 @@ export function loadPackageData(
if (typeof sideEffects === 'boolean') {
hasSideEffects = () => sideEffects
} else if (Array.isArray(sideEffects)) {
hasSideEffects = createFilter(sideEffects, null, { resolve: pkgDir })
const finalPackageSideEffects = sideEffects.map((sideEffect) => {
/*
* The array accepts simple glob patterns to the relevant files... Patterns like *.css, which do not include a /, will be treated like **\/*.css.
* https://webpack.js.org/guides/tree-shaking/
*/
if (sideEffect.includes('/')) {
return sideEffect
}
return `**/${sideEffect}`
})

hasSideEffects = createFilter(finalPackageSideEffects, null, {
resolve: pkgDir,
})
} else {
hasSideEffects = () => true
}
Expand Down

0 comments on commit 9133b4c

Please sign in to comment.