fix: RunWithOutput stdout conflict and clipboard security warning

Author: dlevy-msft-sql

cmd/modern/root/open/clipboard.go

@@ -32,6 +32,6 @@ func copyPasswordToClipboard(user *sqlconfig.User, out *output.Output) bool {
 		return false
 	}
 
-	out.Info(localizer.Sprintf("Password copied to clipboard - paste it when prompted"))
+	out.Info(localizer.Sprintf("Password copied to clipboard - paste it when prompted, then clear your clipboard"))
 	return true
 }

internal/tools/tool/tool.go

@@ -4,6 +4,7 @@
 package tool
 
 import (
+	"bytes"
 	"fmt"
 	"strings"
 
@@ -75,12 +76,12 @@ func (t *tool) RunWithOutput(args []string) (string, int, error) {
 	}
 
 	cmd := t.generateCommandLine(args)
-	output, err := cmd.Output()
+	err := cmd.Run()
 
 	exitCode := 0
 	if cmd.ProcessState != nil {
 		exitCode = cmd.ProcessState.ExitCode()
 	}
 
-	return string(output), exitCode, err
+	return cmd.Stdout.(*bytes.Buffer).String(), exitCode, err
 }