Change build minification to keep function names for better flamegraph/callstacks

[GiantRobots]

When profiling my application Paste component calls (which become a pretty large number of components) component names become anonymous (see pic below)

I'm pretty sure if we update the config below we can keep our function names.

paste/packages/paste-core/core-bundle/tools/minify.js

Line 12 in 1bcb8b3

const result = await Terser.minify(fs.readFileSync(filePath, 'utf8'), {});

const result = await Terser.minify(fs.readFileSync(filePath, 'utf8'), { keep_fnames:true });

[TheSisb]

Hi @GiantRobots,

Thanks for the suggestion! This is tricky; we want to ship the absolute smallest bundle for developers using Paste, but at the same time understand the debugging concern.

My ideal approach would be to ship two versions: minified (*.min.js) and an unminified (*.js) version of our packages. While I can easily build both versions in our dist folders, I'm not sure yet how developers can toggle between both versions. What do you think?

[GiantRobots]

I thought (and we should double check the assumption) That when built in production mode webpack (and other bundlers) minifies all the assets when making chunks so when a project is ran in production mode the code ends up minified either way.

[GiantRobots]

https://stackoverflow.com/a/37901120/6190714

Seems to indicate that minification runs on dependencies so we should be gucci to keep fnames may want to keep classnames as well, or just not minify.

Here's a better example of where this becomes a problem. If I hover over the name of the file I get a path but not much more information than that making it difficult to profile my application and speed things up in the appropriate places.

[TheSisb]

After some discussion and tests internally, we're going to distribute a min/non-minified dist JS in our packages. A ticket has been made into our backlog. We will report back when the work has been completed. Thanks for the feedback!

[gloriliale]

The work on this has been completed and merged! Let us know if it helps!