fix: avoid double copy when ui fails and pin golangci-lint v2

Author: programmersd21

.github/workflows/ci.yml

@@ -82,6 +82,6 @@ jobs:
         go-version: '1.26.5'
 
     - name: run golangci-lint
-      uses: golangci/golangci-lint-action@v4
+      uses: golangci/golangci-lint-action@v9
       with:
-        version: latest
+        version: v2.12.2

cmd/zap/main.go

@@ -104,6 +104,13 @@ func run(ctx context.Context, args []string) error {
 }
 
 func runCopy(ctx context.Context, sources []string, dest string) error {
+	if !term.IsTerminal(os.Stdout.Fd()) {
+		return runCopyDirect(sources, dest)
+	}
+	if w, _, err := term.GetSize(os.Stdout.Fd()); err != nil || w == 0 {
+		return runCopyDirect(sources, dest)
+	}
+
 	stats, err := walk.ComputeStats(sources)
 	if err != nil {
 		return err
@@ -143,7 +150,9 @@ func runCopyWithProgress(ctx context.Context, sources []string, dest string, sta
 	model := ui.NewModel(ui.ThemeMocha, ui.OpCopy, flagVerbose, stats.TotalBytes, stats.TotalFiles)
 	p := tea.NewProgram(model)
 
+	done := make(chan struct{})
 	go func() {
+		defer close(done)
 		var cumulBytes, cumulFiles int64
 		for _, src := range sources {
 			dst := destPath(dest, src)
@@ -162,8 +171,10 @@ func runCopyWithProgress(ctx context.Context, sources []string, dest string, sta
 	}()
 
 	if _, err := p.Run(); err != nil {
-		return runCopyDirect(sources, dest)
+		<-done
+		return summarizeErrors(ec)
 	}
+	<-done
 
 	if ctx.Err() != nil {
 		printInterrupted("copy", stats.TotalFiles)