fix(es/minifier): Change the default of mangle.toplevel to false (#6439)

Author: kdy1

crates/swc_ecma_minifier/src/option/mod.rs

@@ -58,7 +58,7 @@ pub struct MangleOptions {
     #[serde(default, alias = "properties")]
     pub props: Option<ManglePropertiesOptions>,
 
-    #[serde(default = "true_by_default", alias = "toplevel")]
+    #[serde(default, alias = "toplevel")]
     pub top_level: bool,
 
     #[serde(default, alias = "keep_classnames")]

crates/swc_ecma_minifier/tests/fixture/issues/6344/1/mangle.json

@@ -1 +1,3 @@
-{}
+{
+    "toplevel": true
+}

crates/swc_ecma_minifier/tests/fixture/issues/6344/2/mangle.json

@@ -1 +1,3 @@
-{}
+{
+    "toplevel": true
+}

crates/swc_html_minifier/tests/fixture/element/script-group/output.min.html

@@ -27,11 +27,11 @@
 
 
 <div>breaker</div>
-<script type=module>var o="test";console.log(o)</script><script>var o="test";console.log(o)</script>
+<script type=module>var a="test";console.log(a)</script><script>var o="test";console.log(o)</script>
 
 
 <div>breaker</div>
-<script type=module>var o="test";console.log(o);var e="test";console.log(e)</script>
+<script type=module>var a="test";console.log(a);var b="test";console.log(b)</script>
 
 
 <div>breaker</div>
@@ -51,28 +51,28 @@
 <script>var o="test";console.log(o);var e="test";console.log(e)</script>
 
 <div>breaker</div>
-<script type=module>var o="test";console.log(o);var e="test";console.log(e)</script>
+<script type=module>var a="test";console.log(a);var a1="test";console.log(a1)</script>
 
 <div>breaker</div>
-<script type=module>var o="test";console.log(o)</script><script>var o="test";console.log(o)</script>
+<script type=module>var a="test";console.log(a)</script><script>var o="test";console.log(o)</script>
 
 <div>breaker</div>
-<script type=module>var o="test";console.log(o);var e="test";console.log(e)</script>
+<script type=module>var a="test";console.log(a);var a1="test";console.log(a1)</script>
 
 <div>breaker</div>
-<script type=module>var o="test";console.log(o);var e="test";console.log(e)</script>
+<script type=module>var a="test";console.log(a);var a1="test";console.log(a1)</script>
 
 <div>breaker</div>
 <script>let e="test";console.log(e);let l="test";console.log(l);let e="test";console.log(e);let l="test";console.log(l)</script>
 
 <div>breaker</div>
-<script type=module>let t="test";console.log(t);const o="test";console.log("test");let e="test";console.log(e);const s="test";console.log("test")</script>
+<script type=module>let a="test";console.log(a);const b="test";console.log("test");let a1="test";console.log(a1);const b1="test";console.log("test")</script>
 
 <div>breaker</div>
 <script>let e=[1,2,3];[()=>{console.log("test")},2,3][0]()</script>
 
 <div>breaker</div>
-<script type=module>let o="test";console.log(o),console.log(o),console.log(o);let l="test";console.log(l),console.log(l),console.log(l)</script>
+<script type=module>let a="test";console.log(a),console.log(a),console.log(a);let a1="test";console.log(a1),console.log(a1),console.log(a1)</script>
 
 <div>breaker</div>
 <script>let o="test";console.log(o);console.log(o);console.log(o);let o="test";console.log(o);console.log(o);console.log(o)</script>
@@ -84,10 +84,10 @@
 <script>var o="test";console.log(o);console.log(o);console.log(o);var o="test";console.log(o);console.log(o);console.log(o);var o="test";console.log(o);console.log(o);console.log(o)</script>
 
 <div>breaker</div>
-<script type=module>var o="test";console.log(o),console.log(o),console.log(o);var l="test";console.log(l),console.log(l),console.log(l);var e="test";console.log(e),console.log(e),console.log(e)</script>
+<script type=module>var a="test";console.log(a),console.log(a),console.log(a);var a1="test";console.log(a1),console.log(a1),console.log(a1);var a2="test";console.log(a2),console.log(a2),console.log(a2)</script>
 
 <div>breaker</div>
-<script type=module>var o="test";console.log(o),console.log(o),console.log(o)</script><script>var o="test";console.log(o);console.log(o);console.log(o)</script><script type=module>var o="test";console.log(o),console.log(o),console.log(o)</script>
+<script type=module>var a="test";console.log(a),console.log(a),console.log(a)</script><script>var o="test";console.log(o);console.log(o);console.log(o)</script><script type=module>var a="test";console.log(a),console.log(a),console.log(a)</script>
 
 <div>breaker</div>
-<script>var o="test";console.log(o);console.log(o);console.log(o)</script><script type=module>var o="test";console.log(o),console.log(o),console.log(o)</script><script>var o="test";console.log(o);console.log(o);console.log(o)</script>
+<script>var o="test";console.log(o);console.log(o);console.log(o)</script><script type=module>var a="test";console.log(a),console.log(a),console.log(a)</script><script>var o="test";console.log(o);console.log(o);console.log(o)</script>

crates/swc_html_minifier/tests/fixture/element/script-options-1/output.min.html

@@ -1 +1 @@
-<!doctype html><html lang=en><title>Document</title><body><script type=module>const t="bar";function foo(){return"bartest"}</script>
+<!doctype html><html lang=en><title>Document</title><body><script type=module>const my="bar";function foo(){return"bartest"}</script>

crates/swc_html_minifier/tests/fixture/element/script/output.min.html

@@ -19,7 +19,7 @@
 
         </script><script id=data type=application/json>{"foo":"bar"}</script><script>
             broken    broken
-        </script><script>var a=3,a=9</script><script type=module>import t from"foo.js";let o=58800,e="testtest"</script><script>let e=new Date;alert("Today's date is "+e)</script><script src=javascript.js>
+        </script><script>var a=3,a=9</script><script type=module>import t from"foo.js";let test=58800,test1="testtest"</script><script>let e=new Date;alert("Today's date is "+e)</script><script src=javascript.js>
 
 
 

crates/swc_plugin_runner/src/memory_interop.rs

@@ -32,9 +32,7 @@ where
     let serialized_len = serialized_bytes.as_ptr().1;
 
     let ptr_start: u32 = get_allocated_ptr(serialized_len);
-    let ptr_start_size: u32 = ptr_start
-        .try_into()
-        .unwrap_or_else(|_| panic!("Should be able to convert the value {} to u32", ptr_start));
+    let ptr_start_size: u32 = ptr_start;
     let serialized_len_size: u32 = serialized_len.try_into().unwrap_or_else(|_| {
         panic!(
             "Should be able to convert the value {} to u32",