Add detection of redirected stdin for proper interactive mode detection

Co-authored-by: shueybubbles <[email protected]>

Author: Copilot

cmd/sqlcmd/sqlcmd.go

@@ -717,8 +717,24 @@ func setConnect(connect *sqlcmd.ConnectSettings, args *SQLCmdArguments, vars *sq
 }
 
 func isConsoleInitializationRequired(connect *sqlcmd.ConnectSettings, args *SQLCmdArguments) bool {
+	// Password input always requires console initialization
+	if connect.RequiresPassword() {
+		return true
+	}
+
+	// Check if stdin is from a terminal or a redirection
+	file, err := os.Stdin.Stat()
+	if err == nil {
+		// If stdin is not a character device, it's coming from a pipe or redirect
+		if (file.Mode() & os.ModeCharDevice) == 0 {
+			// Non-interactive: stdin is redirected
+			return false
+		}
+	}
+
+	// If we get here, stdin is from a terminal or we couldn't determine
 	iactive := args.InputFile == nil && args.Query == "" && len(args.ChangePasswordAndExit) == 0
-	return iactive || connect.RequiresPassword()
+	return iactive
 }
 
 func run(vars *sqlcmd.Variables, args *SQLCmdArguments) (int, error) {

cmd/sqlcmd/stdin_console_test.go

@@ -0,0 +1,68 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT license.
+
+package sqlcmd
+
+import (
+	"os"
+	"testing"
+
+	"github.com/microsoft/go-sqlcmd/pkg/sqlcmd"
+	"github.com/stretchr/testify/assert"
+)
+
+func TestIsConsoleInitializationRequiredWithRedirectedStdin(t *testing.T) {
+	// Create a temp file to simulate redirected stdin
+	tempFile, err := os.CreateTemp("", "stdin-test-*.txt")
+	if err != nil {
+		t.Fatalf("Failed to create temp file: %v", err)
+	}
+	defer os.Remove(tempFile.Name())
+	defer tempFile.Close()
+
+	// Write some data to it
+	_, err = tempFile.WriteString("SELECT 1;\nGO\n")
+	if err != nil {
+		t.Fatalf("Failed to write to temp file: %v", err)
+	}
+
+	// Remember the original stdin
+	originalStdin := os.Stdin
+	defer func() { os.Stdin = originalStdin }()
+
+	// Test with a file redirection
+	stdinFile, err := os.Open(tempFile.Name())
+	if err != nil {
+		t.Fatalf("Failed to open temp file: %v", err)
+	}
+	defer stdinFile.Close()
+
+	// Replace stdin with our redirected file
+	os.Stdin = stdinFile
+
+	// Set up a connect settings instance for SQL authentication
+	connectConfig := sqlcmd.ConnectSettings{
+		UserName: "testuser", // This will trigger SQL authentication, requiring a password
+	}
+
+	// Test regular args
+	args := &SQLCmdArguments{}
+
+	// Print file stat mode for debugging
+	fileStat, _ := os.Stdin.Stat()
+	t.Logf("File mode: %v", fileStat.Mode())
+	t.Logf("Is character device: %v", (fileStat.Mode()&os.ModeCharDevice) != 0)
+	t.Logf("Connection config: %+v", connectConfig)
+	t.Logf("RequiresPassword() returns: %v", connectConfig.RequiresPassword())
+
+	// Test with SQL authentication that requires a password
+	res := isConsoleInitializationRequired(&connectConfig, args)
+	// Should be true since password is required, even with redirected stdin
+	assert.True(t, res, "Console initialization should be required when SQL authentication is used")
+
+	// Now test with no authentication (no password required)
+	connectConfig = sqlcmd.ConnectSettings{}
+	res = isConsoleInitializationRequired(&connectConfig, args)
+	// Should be false since stdin is redirected and no password is required
+	assert.False(t, res, "Console initialization should not be required with redirected stdin and no password")
+}