You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Investigate why terser/webpack defaults causes SDK code to break #11212
I wonder if this may be related to #10566 - ember uses terser under the hood 🤔 there the problem seemed to be related to tree shaking & minifying, somehow.
Activity
AbhiPrasad commentedon Mar 20, 2024
From Tim on discord
timfish commentedon Mar 20, 2024
Terser itself doesn't appear to have minify levels, all the different options are configured individually:
https://github.com/terser/terser?tab=readme-ov-file#compress-options
What level option is being referred to in the webpack config?
https://webpack.js.org/plugins/terser-webpack-plugin/
AbhiPrasad commentedon Mar 21, 2024
webpack config: https://gist.github.com/tristanbes/7e2eb2f00637e908ba0fc6f703ae52ea
timfish commentedon Mar 21, 2024
Ah ok so it's the inline levels!
options: { compress: { inline: 1 } }timfish commentedon Mar 23, 2024
I've added tests for both webpack 4 and 5 and backported them to the v7 branch to see if there are any failures.
We don't see any issues with the default terser config #11257
The webpack config looks like this:
mydea commentedon Mar 25, 2024
I wonder if this may be related to #10566 - ember uses terser under the hood 🤔 there the problem seemed to be related to tree shaking & minifying, somehow.
timfish commentedon Mar 25, 2024
Maybe my test cases should at least access more of the SDK API?
mydea commentedon Jul 4, 2024
I think this is hopefully fixed, let's close this for now.