From 8f4aab7ecfa34f5c139ebf54f1cf26c75d90a875 Mon Sep 17 00:00:00 2001
From: Ayman Bagabas <ayman.bagabas@gmail.com>
Date: Fri, 22 Nov 2024 11:47:36 -0500
Subject: [PATCH] fix(query): windows: explicitly open CONIN/CONOUT when I/O is
 not a terminal

This change explicitly opens CONIN/CONOUT when either in/out files
is not a terminal. Otherwise, the background color query fails because
the streams are not console devices.

Fixes: https://github.com/charmbracelet/sequin/issues/28
---
 query.go | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/query.go b/query.go
index 4d678fb7..1faa2574 100644
--- a/query.go
+++ b/query.go
@@ -33,6 +33,23 @@ func backgroundColor(in *os.File, out *os.File) (color.Color, error) {
 // Bubble Tea, listen for tea.BackgroundColorMsg in your update function.
 func BackgroundColor(in *os.File, out *os.File) (bg color.Color, err error) {
 	if runtime.GOOS == "windows" {
+		// On Windows, when the input/output is redirected or piped, we need to
+		// open the console explicitly.
+		// See https://learn.microsoft.com/en-us/windows/console/getstdhandle#remarks
+		if !term.IsTerminal(in.Fd()) {
+			f, err := os.OpenFile("CONIN$", os.O_RDWR, 0o644)
+			if err != nil {
+				return nil, fmt.Errorf("error opening CONIN$: %w", err)
+			}
+			in = f
+		}
+		if !term.IsTerminal(out.Fd()) {
+			f, err := os.OpenFile("CONOUT$", os.O_RDWR, 0o644)
+			if err != nil {
+				return nil, fmt.Errorf("error opening CONOUT$: %w", err)
+			}
+			out = f
+		}
 		return backgroundColor(in, out)
 	}