fix(spin): properly redirect output

closes #690
charmbracelet · Dec 4, 2024 · 71af32c · 71af32c
1 parent c422e76
commit 71af32c
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/spin/spin.go b/spin/spin.go
@@ -16,13 +16,15 @@ package spin
 
 import (
 	"io"
+	"os"
 	"os/exec"
 	"strings"
 	"syscall"
 	"time"
 
 	"github.com/charmbracelet/gum/internal/exit"
 	"github.com/charmbracelet/gum/timeout"
+	"github.com/charmbracelet/x/term"
 
 	"github.com/charmbracelet/bubbles/spinner"
 	tea "github.com/charmbracelet/bubbletea"
@@ -68,8 +70,12 @@ func commandStart(command []string) tea.Cmd {
 		}
 
 		executing = exec.Command(command[0], args...) //nolint:gosec
-		executing.Stdout = io.MultiWriter(&bothbuf, &outbuf)
-		executing.Stderr = io.MultiWriter(&bothbuf, &errbuf)
+		if term.IsTerminal(os.Stdout.Fd()) {
+			executing.Stdout = io.MultiWriter(&bothbuf, &outbuf)
+			executing.Stderr = io.MultiWriter(&bothbuf, &errbuf)
+		} else {
+			executing.Stdout = os.Stdout
+		}
 		_ = executing.Run()
 		status := executing.ProcessState.ExitCode()
 		if status == -1 {