Support trailing ignore comments

crates/oxc_formatter/src/formatter/comments.rs

@@ -389,6 +389,16 @@ impl<'a> Comments<'a> {
         self.comments_before(start).iter().any(|comment| self.is_suppression_comment(comment))
     }
 
+    /// Checks if there is a trailing line suppression comment on the same line.
+    ///
+    /// This supports patterns like:
+    /// `statement(); // prettier-ignore`
+    pub fn has_line_suppression_comment_at_end_of_line(&self, pos: u32) -> bool {
+        self.end_of_line_comments_after(pos)
+            .iter()
+            .any(|comment| comment.is_line() && self.is_suppression_comment(comment))
+    }
+
     /// Checks if a comment is a suppression comment (`oxfmt-ignore`).
     ///
     /// `prettier-ignore` is also supported for compatibility.

crates/oxc_formatter/src/print/mod.rs

@@ -74,6 +74,7 @@ use crate::{
         object::{format_property_key, should_preserve_quote},
         statement_body::FormatStatementBody,
         string::{FormatLiteralStringToken, StringLiteralParentKind},
+        suppressed::FormatSuppressedNode,
         tailwindcss::{tailwind_context_for_string_literal, write_tailwind_string_literal},
     },
     write,
@@ -556,13 +557,21 @@ fn expression_statement_needs_semicolon<'a>(
 
 impl<'a> FormatWrite<'a> for AstNode<'a, ExpressionStatement<'a>> {
     fn write(&self, f: &mut Formatter<'_, 'a>) {
+        let span = self.span();
         // Check if we need a leading semicolon to prevent ASI issues
         if f.options().semicolons == Semicolons::AsNeeded
             && expression_statement_needs_semicolon(self, f)
         {
             write!(f, ";");
         }
 
+        if f.comments().has_line_suppression_comment_at_end_of_line(span.end) {
+            // Preserve original text when the statement has an inline suppression comment:
+            // `stmt(); // prettier-ignore`
+            write!(f, [FormatSuppressedNode(span)]);
+            return;
+        }
+
         write!(f, [self.expression(), OptionalSemicolon]);
     }
 }

crates/oxc_formatter/tests/fixtures/js/ignore/expression-statement.js

@@ -7,4 +7,6 @@ Object.defineProperties    (    exports    , {
   c +
     b +
   d
-)
+)
+
+logger({ level: 'debug', message: `Really long debugging message about how ${user} called a certain part of our application with a certain ${payload}` }); // oxfmt-ignore

crates/oxc_formatter/tests/fixtures/js/ignore/expression-statement.js.snap

@@ -12,6 +12,7 @@ Object.defineProperties    (    exports    , {
     b +
   d
 )
+logger({ level: 'debug', message: `Really long debugging message about how ${user} called a certain part of our application with a certain ${payload}` }); // oxfmt-ignore
 ==================== Output ====================
 ------------------
 { printWidth: 80 }
@@ -26,6 +27,7 @@ Object.defineProperties    (    exports    , {
     b +
   d
 )
+logger({ level: 'debug', message: `Really long debugging message about how ${user} called a certain part of our application with a certain ${payload}` }); // oxfmt-ignore
 
 -------------------
 { printWidth: 100 }
@@ -40,5 +42,6 @@ Object.defineProperties    (    exports    , {
     b +
   d
 )
+logger({ level: 'debug', message: `Really long debugging message about how ${user} called a certain part of our application with a certain ${payload}` }); // oxfmt-ignore
 
 ===================== End =====================

crates/oxc_formatter/tests/fixtures/js/ignore/oxfmt.js.snap

@@ -95,7 +95,7 @@ function a() {
   };
 
   function giveMeSome() {
-    a(a); // oxfmt-ignore
+    a(  a  ); // oxfmt-ignore
     // shouldn't I return something?  :shrug:
   }
 
@@ -159,7 +159,7 @@ function a() {
   };
 
   function giveMeSome() {
-    a(a); // oxfmt-ignore
+    a(  a  ); // oxfmt-ignore
     // shouldn't I return something?  :shrug:
   }