fix(es/minifier): Drop line comments starting with ! (#7689)

**Related issue:**

 - Closes #7653

Author: kdy1

crates/swc/src/lib.rs

@@ -120,7 +120,11 @@ use std::{
 
 use anyhow::{bail, Context, Error};
 use atoms::JsWord;
-use common::{collections::AHashMap, comments::SingleThreadedComments, errors::HANDLER};
+use common::{
+    collections::AHashMap,
+    comments::{CommentKind, SingleThreadedComments},
+    errors::HANDLER,
+};
 use config::{IsModule, JsMinifyCommentOption, JsMinifyOptions, OutputCharset};
 use jsonc_parser::{parse_to_serde_value, ParseOptions};
 use once_cell::sync::Lazy;
@@ -696,7 +700,15 @@ pub(crate) fn minify_file_comments(
         BoolOr::Data(JsMinifyCommentOption::PreserveSomeComments) => {
             let preserve_excl = |_: &BytePos, vc: &mut Vec<Comment>| -> bool {
                 // Preserve license comments.
-                vc.retain(|c: &Comment| c.text.contains("@license") || c.text.starts_with('!'));
+                //
+                // See https://github.com/terser/terser/blob/798135e04baddd94fea403cfaab4ba8b22b1b524/lib/output.js#L175-L181
+                vc.retain(|c: &Comment| {
+                    c.text.contains("@lic")
+                        || c.text.contains("@preserve")
+                        || c.text.contains("@copyright")
+                        || c.text.contains("@cc_on")
+                        || (c.kind == CommentKind::Block && c.text.starts_with('!'))
+                });
                 !vc.is_empty()
             };
             let (mut l, mut t) = comments.borrow_all_mut();

crates/swc/tests/fixture/issues-7xxx/7653/input/.swcrc

@@ -0,0 +1,15 @@
+{
+    "jsc": {
+        "parser": {
+            "syntax": "typescript",
+            "tsx": true
+        },
+        "transform": null,
+        "target": "es5",
+        "loose": true,
+        "externalHelpers": false,
+        "keepClassNames": false,
+    },
+    "minify": true,
+    "isModule": true
+}

crates/swc/tests/fixture/issues-7xxx/7653/input/1.js

@@ -0,0 +1,12 @@
+/** Add your relevant code here for the issue to reproduce */
+export default function Home() {
+    return <div
+        //TODO: Check if this is still needed after the approval of the new Ad Manager
+        //! The following property might seem like it is doing nothing.
+        //! However, it is important in order to prevent React from discarding
+        //! the node's contents once hydration happens on the client.
+        //! The contents of this node might change between the Server Side Rendering
+        //! and the hydration, due to the registering of the slots that is done before hydration.
+        dangerouslySetInnerHTML={{ __html: 'Hello World' }}
+    />
+}

crates/swc/tests/fixture/issues-7xxx/7653/output/1.js

@@ -0,0 +1 @@
+export default function Home(){return React.createElement("div",{dangerouslySetInnerHTML:{__html:"Hello World"}})}