feat(oxfmt): Respect ignore settings for --stdin-filepath (#21625)

leaysgur · leaysgur · commit 144f27a042f1 · 2026-04-22T03:29:24.000Z
Fixes #20545
diff --git a/apps/oxfmt/src/cli/mod.rs b/apps/oxfmt/src/cli/mod.rs
@@ -4,7 +4,7 @@ mod reporter;
 mod result;
 mod runner;
 mod service;
-mod walk;
+pub(crate) mod walk;
 
 pub use crate::core::utils::init_tracing;
 #[cfg(feature = "napi")]
diff --git a/apps/oxfmt/src/stdin/mod.rs b/apps/oxfmt/src/stdin/mod.rs
@@ -4,6 +4,9 @@ use std::{
     path::PathBuf,
 };
 
+use ignore::gitignore::Gitignore;
+
+use crate::cli::walk::resolve_ignore_paths;
 use crate::cli::{CliRunResult, FormatCommand, Mode};
 use crate::core::{
     ConfigResolver, ExternalFormatter, FormatFileStrategy, FormatResult, JsConfigLoaderCb,
@@ -40,7 +43,7 @@ impl StdinRunner {
         let stderr = &mut BufWriter::new(io::stderr());
 
         let cwd = self.cwd;
-        let FormatCommand { mode, config_options, .. } = self.options;
+        let FormatCommand { mode, config_options, ignore_options, .. } = self.options;
 
         let Mode::Stdin(filepath) = mode else {
             unreachable!("`StdinRunner::run()` called with non-Stdin mode");
@@ -98,6 +101,24 @@ impl StdinRunner {
             return CliRunResult::InvalidOptionConfig;
         };
 
+        // Check if the file is ignored by global ignores or config`.ignorePatterns`
+        let resolved_ignore_paths = match resolve_ignore_paths(&cwd, &ignore_options.ignore_path) {
+            Ok(paths) => paths,
+            Err(err) => {
+                utils::print_and_flush(stderr, &format!("{err}\n"));
+                return CliRunResult::InvalidOptionConfig;
+            }
+        };
+        let is_ignored = resolved_ignore_paths.iter().any(|ignore_path| {
+            let (gitignore, _) = Gitignore::new(ignore_path);
+            gitignore.matched_path_or_any_parents(strategy.path(), false).is_ignore()
+        }) || config_resolver.is_path_ignored(strategy.path(), false);
+
+        if is_ignored {
+            utils::print_and_flush(stdout, &source_text);
+            return CliRunResult::FormatSucceeded;
+        }
+
         // Resolve options for the stdin file entry
         let resolved_options = config_resolver.resolve(&strategy);
 
diff --git a/apps/oxfmt/test/cli/stdin_ignore_patterns/0.snap.md b/apps/oxfmt/test/cli/stdin_ignore_patterns/0.snap.md
@@ -0,0 +1,32 @@
+# Input
+
+## Command
+```
+oxfmt --stdin-filepath test.ts
+```
+
+## File tree
+```
+- fixtures/ <CWD>
+  - .oxfmtrc.json
+  - .prettierignore
+  - input.ts
+```
+
+# Result
+
+## Exit code
+0
+
+## stdout
+```
+const x: number = 1;
+```
+
+## stderr
+```
+
+```
+
+## File changes
+No changes
diff --git a/apps/oxfmt/test/cli/stdin_ignore_patterns/1.snap.md b/apps/oxfmt/test/cli/stdin_ignore_patterns/1.snap.md
@@ -0,0 +1,32 @@
+# Input
+
+## Command
+```
+oxfmt --stdin-filepath ignored/test.ts
+```
+
+## File tree
+```
+- fixtures/ <CWD>
+  - .oxfmtrc.json
+  - .prettierignore
+  - input.ts
+```
+
+# Result
+
+## Exit code
+0
+
+## stdout
+```
+const   x:number=1
+```
+
+## stderr
+```
+
+```
+
+## File changes
+No changes
diff --git a/apps/oxfmt/test/cli/stdin_ignore_patterns/2.snap.md b/apps/oxfmt/test/cli/stdin_ignore_patterns/2.snap.md
@@ -0,0 +1,32 @@
+# Input
+
+## Command
+```
+oxfmt --stdin-filepath prettierignored/test.ts
+```
+
+## File tree
+```
+- fixtures/ <CWD>
+  - .oxfmtrc.json
+  - .prettierignore
+  - input.ts
+```
+
+# Result
+
+## Exit code
+0
+
+## stdout
+```
+const   x:number=1
+```
+
+## stderr
+```
+
+```
+
+## File changes
+No changes
diff --git a/apps/oxfmt/test/cli/stdin_ignore_patterns/fixtures/.oxfmtrc.json b/apps/oxfmt/test/cli/stdin_ignore_patterns/fixtures/.oxfmtrc.json
@@ -0,0 +1,3 @@
+{
+  "ignorePatterns": ["ignored/"]
+}
diff --git a/apps/oxfmt/test/cli/stdin_ignore_patterns/fixtures/.prettierignore b/apps/oxfmt/test/cli/stdin_ignore_patterns/fixtures/.prettierignore
@@ -0,0 +1 @@
+prettierignored/
diff --git a/apps/oxfmt/test/cli/stdin_ignore_patterns/fixtures/input.ts b/apps/oxfmt/test/cli/stdin_ignore_patterns/fixtures/input.ts
@@ -0,0 +1 @@
+const   x:number=1
diff --git a/apps/oxfmt/test/cli/stdin_ignore_patterns/options.json b/apps/oxfmt/test/cli/stdin_ignore_patterns/options.json
@@ -0,0 +1,5 @@
+[
+  { "args": ["--stdin-filepath", "test.ts"], "stdin": "input.ts" },
+  { "args": ["--stdin-filepath", "ignored/test.ts"], "stdin": "input.ts" },
+  { "args": ["--stdin-filepath", "prettierignored/test.ts"], "stdin": "input.ts" }
+]

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+{`
	`2`	`+ "ignorePatterns": ["ignored/"]`
	`3`	`+}`