fix(minifier): preserve outer paren start when unwrapping SequenceExpression

babu-ch · babu-ch · commit db76b5612ee5 · 2026-04-13T21:29:07.000+09:00
diff --git a/crates/oxc_minifier/src/peephole/normalize.rs b/crates/oxc_minifier/src/peephole/normalize.rs
@@ -6,6 +6,7 @@ use oxc_ecmascript::{
     side_effects::is_valid_regexp,
 };
 use oxc_semantic::IsGlobalReference;
+use oxc_span::GetSpanMut;
 use oxc_syntax::scope::ScopeFlags;
 
 use crate::{ReusableTraverseCtx, Traverse, TraverseCtx, minifier_traverse::traverse_mut_with_ctx};
@@ -86,7 +87,18 @@ impl<'a> Traverse<'a> for Normalize {
 
     fn exit_expression(&mut self, expr: &mut Expression<'a>, ctx: &mut TraverseCtx<'a>) {
         if let Expression::ParenthesizedExpression(paren_expr) = expr {
+            let outer_start = paren_expr.span.start;
+            let is_sequence = matches!(&paren_expr.expression, Expression::SequenceExpression(_));
             *expr = paren_expr.expression.take_in(ctx.ast);
+            // For `SequenceExpression`, the surrounding parens are syntactically required
+            // when re-emitted in expression context, so any comments attached to the outer
+            // `(` should remain reachable via the inner expression's span.
+            if is_sequence {
+                let span = expr.span_mut();
+                if outer_start < span.start {
+                    span.start = outer_start;
+                }
+            }
         }
         if let Some(e) = match expr {
             Expression::Identifier(ident) => Self::try_compress_identifier(ident, ctx),
diff --git a/tasks/coverage/snapshots/minifier_test262.snap b/tasks/coverage/snapshots/minifier_test262.snap
@@ -2,13 +2,7 @@ commit: ff0d1611
 
 minifier_test262 Summary:
 AST Parsed     : 44642/44642 (100.00%)
-Positive Passed: 44632/44642 (99.98%)
-Compress: tasks/coverage/test262/test/built-ins/Array/prototype/forEach/15.4.4.18-7-c-ii-5.js
-
-Compress: tasks/coverage/test262/test/built-ins/Array/prototype/reduce/15.4.4.21-9-c-ii-5.js
-
-Compress: tasks/coverage/test262/test/built-ins/Array/prototype/reduceRight/15.4.4.22-9-c-ii-5.js
-
+Positive Passed: 44635/44642 (99.98%)
 Compress: tasks/coverage/test262/test/intl402/Temporal/PlainDate/prototype/toLocaleString/lone-options-accepted.js
 
 Compress: tasks/coverage/test262/test/intl402/Temporal/PlainMonthDay/prototype/toLocaleString/lone-options-accepted.js
