Merge pull request #641 from numtide/fix/stdin-exlude-behaviour

Fix: stdin global exclude behaviour

Author: brianmcgee

cmd/root_test.go

@@ -2246,6 +2246,30 @@ func TestDeterministicOrderingInPipeline(t *testing.T) {
 			idx++
 		}
 	}
+
+	// test with a file that is in global excludes
+	// all toml files are globally excluded in the test config
+	badToml := `
+	foo = "bla"
+		bar = [ "bla" ]
+	adsfadf;
+`
+	os.Stdin = test.TempFile(t, "", "stdin", &badToml)
+
+	treefmt(t,
+		withArgs("--stdin", "treefmt.toml"),
+		withNoError(t),
+		withStats(t, map[stats.Type]int{
+			stats.Traversed: 1,
+			stats.Matched:   0,
+			stats.Formatted: 0,
+			stats.Changed:   0,
+		}),
+		withStdout(func(out []byte) {
+			// it should not have been modified, simply emitted again
+			as.Equal(badToml, string(out))
+		}),
+	)
 }
 
 func TestRunInSubdir(t *testing.T) {

format/composite.go

@@ -67,6 +67,8 @@ func (c *CompositeFormatter) Apply(ctx context.Context, files []*walk.File) erro
 
 		// if the file is globally excluded, we do not emit a warning
 		if globalExclude {
+			toRelease = append(toRelease, file)
+
 			continue
 		}
 

go.mod

@@ -1,6 +1,6 @@
 module github.com/numtide/treefmt/v2
 
-go 1.24.0
+go 1.24.1
 
 require (
 	github.com/BurntSushi/toml v1.5.0