Add terser format options (#1258)

ediamin · web-flow · commit c4093bfb167d · 2021-07-20T11:17:53.000+02:00
diff --git a/packages/optimizer/lib/transformers/MinifyHtml.js b/packages/optimizer/lib/transformers/MinifyHtml.js
@@ -139,7 +139,14 @@ class MinifyHtml {
     }
     try {
       const {minify} = require('terser');
-      const result = await minify(child.data, {});
+      const result = await minify(child.data, {
+        format: {
+          // Always use the original quotes.
+          quote_style: 3,
+          // Do not wrap function expressions that are passed as arguments in parenthesis.
+          wrap_func_args: false,
+        },
+      });
       if (result.error) {
         this.log.warn(
           'Could not minify inline amp-script',
diff --git a/packages/optimizer/spec/transformers/valid/MinifyHtml/minifies_inline_amp-script/expected_output.html b/packages/optimizer/spec/transformers/valid/MinifyHtml/minifies_inline_amp-script/expected_output.html
@@ -1,4 +1,4 @@
 <!doctype html><html ⚡><head><script async src="https://cdn.ampproject.org/v0.js"></script><script async custom-element="amp-img" src="https://cdn.ampproject.org/v0/amp-img-0.1.js"></script><style amp-boilerplate>body{-webkit-animation:-amp-start 8s steps(1,end) 0s 1 normal both;
--moz-animation:-amp-start 8s steps(1,end) 0s 1 normal both;-ms-animation:-amp-start 8s steps(1,end) 0s 1 normal both;animation:-amp-start 8s steps(1,end) 0s 1 normal both}@-webkit-keyframes-amp-start{from{visibility:hidden}to{visibility:visible}}@-moz-keyframes-amp-start{from{visibility:hidden}to{visibility:visible}}@-ms-keyframes-amp-start{from{visibility:hidden}to{visibility:visible}}@-o-keyframes-amp-start{from{visibility:hidden}to{visibility:visible}}@keyframes -amp-start{from{visibility:hidden}to{visibility:visible}}</style><noscript><style amp-boilerplate>body{-webkit-animation:none;-moz-animation:none;-ms-animation:none;animation:none}</style></noscript><link href="https://example.com/favicon.ico" rel="icon"></head><body>  <script id="hello-world" type="text/plain" target="amp-script">const btn=document.querySelector("button");btn.addEventListener("click",(()=>{document.body.textContent="Hello World!"}));</script>  <script id="hello-world" type="text/plain" target="amp-script">
+-moz-animation:-amp-start 8s steps(1,end) 0s 1 normal both;-ms-animation:-amp-start 8s steps(1,end) 0s 1 normal both;animation:-amp-start 8s steps(1,end) 0s 1 normal both}@-webkit-keyframes-amp-start{from{visibility:hidden}to{visibility:visible}}@-moz-keyframes-amp-start{from{visibility:hidden}to{visibility:visible}}@-ms-keyframes-amp-start{from{visibility:hidden}to{visibility:visible}}@-o-keyframes-amp-start{from{visibility:hidden}to{visibility:visible}}@keyframes -amp-start{from{visibility:hidden}to{visibility:visible}}</style><noscript><style amp-boilerplate>body{-webkit-animation:none;-moz-animation:none;-ms-animation:none;animation:none}</style></noscript><link href="https://example.com/favicon.ico" rel="icon"></head><body>  <script id="hello-world" type="text/plain" target="amp-script">const btn=document.querySelector('button');btn.addEventListener('click',()=>{document.body.textContent='Hello World!'});</script>  <script id="hello-world" type="text/plain" target="amp-script">
       const const;
     </script> </body></html>
