feat(ts/fast-strip): Remove line numbers (#10254)

**Description:**

We use string search to replace line numbers

Author: kdy1

bindings/Cargo.lock

@@ -36,15 +36,15 @@ resolver = "2"
   swc_atoms            = "5.0.0"
   swc_cached           = "2.0.0"
   swc_cli_impl         = "16.0.0"
-  swc_common           = "8.0.0"
+  swc_common           = "8.0.1"
   swc_compiler_base    = "13.0.1"
   swc_config           = "2.0.0"
-  swc_core             = "16.8.0"
+  swc_core             = "16.9.1"
   swc_css_ast          = "8.0.0"
   swc_css_codegen      = "8.0.0"
   swc_css_minifier     = "8.0.0"
   swc_css_parser       = "8.0.0"
-  swc_error_reporters  = "9.1.0"
+  swc_error_reporters  = "9.1.1"
   swc_fast_ts_strip    = "12.1.0"
   swc_html             = "12.0.0"
   swc_html_ast         = "8.0.0"

bindings/Cargo.toml

@@ -9,7 +9,7 @@ exports[`transform in strip-only mode should not emit 'Caused by: failed to pars
   "message": "await isn't allowed in non-async function",
   "snippet": "   
         
- 1   function foo() { await Promise.resolve(1); }
+     function foo() { await Promise.resolve(1); }
                       ^^^^^
         
 ",
@@ -53,7 +53,7 @@ exports[`transform in strip-only mode should report correct error for syntax err
   "message": "Expected ';', '}' or <eof>",
   "snippet": "   
         
- 1   function foo() { invalid syntax }
+     function foo() { invalid syntax }
                               ^^^^^^
         
 ",
@@ -70,10 +70,10 @@ exports[`transform in strip-only mode should report correct error for unsupporte
   "filename": "test.ts",
   "message": "TypeScript enum is not supported in strip-only mode",
   "snippet": "   
-     [1:1]
- 1     > enum Foo {
- 2                           a, b    
- 3     >                     }
+          
+       > enum Foo {
+                             a, b    
+       >                     }
         
 ",
   "startColumn": 0,
@@ -143,7 +143,7 @@ exports[`transform in strip-only mode should throw an error when it encounters a
   "message": "\`module\` keyword is not supported. Use \`namespace\` instead.",
   "snippet": "   
         
- 1   module foo { }
+     module foo { }
      ^^^^^^^^^^
         
 ",
@@ -161,7 +161,7 @@ exports[`transform in strip-only mode should throw an error when it encounters a
   "message": "\`module\` keyword is not supported. Use \`namespace\` instead.",
   "snippet": "   
         
- 1   declare module foo { }
+     declare module foo { }
              ^^^^^^^^^^
         
 ",
@@ -179,7 +179,7 @@ exports[`transform in strip-only mode should throw an error when it encounters a
   "message": "TypeScript namespace declaration is not supported in strip-only mode",
   "snippet": "   
         
- 1   namespace Foo { export const m = 1; }
+     namespace Foo { export const m = 1; }
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
         
 ",
@@ -197,7 +197,7 @@ exports[`transform in strip-only mode should throw an error when it encounters a
   "message": "TypeScript enum is not supported in strip-only mode",
   "snippet": "   
         
- 1   enum Foo {}
+     enum Foo {}
      ^^^^^^^^^^^
         
 ",
@@ -215,7 +215,7 @@ exports[`transform in transform mode shoud throw an object even with deprecatedT
   "message": "\`module\` keyword is not supported. Use \`namespace\` instead.",
   "snippet": "   
         
- 1   module F { export type x = number }
+     module F { export type x = number }
      ^^^^^^^^
         
 ",
@@ -233,7 +233,7 @@ exports[`transform in transform mode should throw an error when it encounters a
   "message": "\`module\` keyword is not supported. Use \`namespace\` instead.",
   "snippet": "   
         
- 1   declare module foo { }
+     declare module foo { }
              ^^^^^^^^^^
         
 ",
@@ -251,7 +251,7 @@ exports[`transform in transform mode should throw an error when it encounters a
   "message": "\`module\` keyword is not supported. Use \`namespace\` instead.",
   "snippet": "   
         
- 1   module foo { }
+     module foo { }
      ^^^^^^^^^^
         
 ",

bindings/binding_typescript_wasm/__tests__/__snapshots__/transform.js.snap

@@ -139,7 +139,10 @@ impl Emitter for JsonErrorWriter {
                     span,
                 },
             ) {
-                Ok(()) => Some(snippet),
+                Ok(()) => Some({
+                    remove_line_number(&mut snippet);
+                    snippet
+                }),
                 Err(_) => None,
             }
         });
@@ -261,3 +264,88 @@ impl std::fmt::Debug for Snippet<'_> {
         Ok(())
     }
 }
+
+/// Finds for `[\d+\s+]` and `\[\d+:\d+\]` and remove them.
+fn remove_line_number(snippet: &mut String) {
+    let mut result = String::with_capacity(snippet.len());
+    let lines = snippet.lines();
+
+    for line in lines {
+        let mut new_line = String::with_capacity(line.len());
+
+        // Check for [n:m] pattern
+        if let Some(pos) = line.find('[') {
+            if let Some(end_pos) = line[pos..].find(']') {
+                let bracket_content = &line[pos + 1..pos + end_pos];
+                if bracket_content
+                    .chars()
+                    .all(|c| c.is_ascii_digit() || c == ':')
+                {
+                    // Replace [n:m] with spaces
+                    for _ in 0..pos {
+                        new_line.push(' ');
+                    }
+                    for _ in 0..(end_pos + 1) {
+                        new_line.push(' ');
+                    }
+                    new_line.push_str(&line[pos + end_pos + 1..]);
+                    result.push_str(&new_line);
+                    result.push('\n');
+                    continue;
+                }
+            }
+        }
+
+        // Check for digit-space pattern (e.g., " 1   ")
+        let trimmed = line.trim_start();
+
+        if !trimmed.is_empty() && trimmed.chars().next().unwrap().is_ascii_digit() {
+            let leading_space_count = line.len() - trimmed.len();
+            let mut digit_end = 0;
+
+            // Find where the digits end
+            for (i, c) in trimmed.char_indices() {
+                if !c.is_ascii_digit() {
+                    digit_end = i;
+                    break;
+                }
+            }
+
+            if digit_end == 0 && trimmed.chars().all(|c| c.is_ascii_digit()) {
+                digit_end = trimmed.len();
+            }
+
+            // Check if digits are followed by whitespace
+            if digit_end < trimmed.len()
+                && trimmed[digit_end..].starts_with(|c: char| c.is_whitespace())
+            {
+                // Count trailing whitespace
+                let trailing_space_end = trimmed[digit_end..]
+                    .chars()
+                    .take_while(|&c| c.is_whitespace())
+                    .count();
+
+                // Replace number and surrounding spaces with spaces
+                for _ in 0..(leading_space_count + digit_end + trailing_space_end) {
+                    new_line.push(' ');
+                }
+
+                new_line.push_str(&trimmed[digit_end + trailing_space_end..]);
+                result.push_str(&new_line);
+                result.push('\n');
+                continue;
+            }
+        }
+
+        // If no pattern matched, keep the line as is
+        result.push_str(line);
+        result.push('\n');
+    }
+
+    // Remove the trailing newline if the original didn't have one
+    if !snippet.ends_with('\n') && !result.is_empty() {
+        result.pop();
+    }
+
+    *snippet = result;
+}