fix(spin): indenting lines when command is piped (#636)HEADmain

We don't need to check for isatty, always store the output to the
designated buffer.

Fixes: https://github.com/charmbracelet/gum/issues/607

1 files changed, 3 insertions, 15 deletions

diff --git a/spin/spin.go b/spin/spin.go
index db63fa4..26c933f 100644
--- a/spin/spin.go
+++ b/spin/spin.go
@@ -16,14 +16,12 @@ package spin
 
 import (
 	"io"
-	"os"
 	"os/exec"
 	"strings"
 	"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"
@@ -65,22 +63,12 @@ func commandStart(command []string) tea.Cmd {
 		if len(command) > 1 {
 			args = command[1:]
 		}
-		cmd := exec.Command(command[0], args...) //nolint:gosec
-
-		if term.IsTerminal(os.Stdout.Fd()) {
-			stdout := io.MultiWriter(&bothbuf, &errbuf)
-			stderr := io.MultiWriter(&bothbuf, &outbuf)
-
-			cmd.Stdout = stdout
-			cmd.Stderr = stderr
-		} else {
-			cmd.Stdout = os.Stdout
-		}
 
+		cmd := exec.Command(command[0], args...) //nolint:gosec
+		cmd.Stdout = io.MultiWriter(&bothbuf, &outbuf)
+		cmd.Stderr = io.MultiWriter(&bothbuf, &errbuf)
 		_ = cmd.Run()
-
 		status := cmd.ProcessState.ExitCode()
-
 		if status == -1 {
 			status = 1
 		}