Feature Request: Fine-tuning Terser

[aigoncharov]

Is this a bug report?

No

Proposal:
Support a new environment variable TERSER_CONFIG. A dev could set it to a JSON with config values supported by Terser and CRA would merge default this JSON with its default Terser config.

Example:

1. Set TERSER_CONFIG to { "keep_classnames": true }
2. keep_classnames set to true added to Terser options

Reasoning:
Certain libraries rely on class names which are minified by default by Terser. It would be nice to change that without maintaining a fork of CRA for one line of code only.
I'm pretty sure there're other cases as well. Anyway it's pretty simple to add this, it would give developers more freedom, so what's the harm, right? :)

Implementation considerations:
If TERSER_CONFIG environment variable is present, do JSON.parse of its value and add it to Terser options here. Add TERSER_CONFIG to this list.

Would you be willing to accept a PR for this functionality?

[aigoncharov]

@gaearon @Timer @iansu @ianschmitz any thoughts? Would you be willing to merge a PR for this feature if I come up with one?

[stale]

This issue has been automatically marked as stale because it has not had any recent activity. It will be closed in 5 days if no further activity occurs.