Add Windows compatibility check for SSH multiplexing

Skip SSH multiplexing on Windows due to potential compatibility issues
with OpenSSH's ControlMaster/ControlPath implementation. On Windows,
BuildSSHMultiplexArgs now returns an empty slice, allowing the extension
to work without multiplexing on that platform.

- Add runtime.GOOS check to detect Windows
- Return empty args slice on Windows to disable multiplexing
- Add test to verify platform-specific behavior
- Multiplexing still works on Linux/macOS as before

Co-authored-by: scaryrawr <661373+scaryrawr@users.noreply.github.com>

Author: Copilot

args.go

@@ -5,6 +5,7 @@ import (
 	"fmt"
 	"os"
 	"path/filepath"
+	"runtime"
 	"strings"
 )
 
@@ -179,7 +180,14 @@ func sanitizeCodespaceNameForControl(name string) string {
 }
 
 // BuildSSHMultiplexArgs builds SSH multiplexing arguments for a given control path and mode
+// On Windows, SSH multiplexing may not be fully supported, so this returns an empty slice
 func BuildSSHMultiplexArgs(controlPath string, isMaster bool) []string {
+	// Skip SSH multiplexing on Windows due to potential compatibility issues
+	// with OpenSSH's ControlMaster/ControlPath implementation
+	if runtime.GOOS == "windows" {
+		return []string{}
+	}
+	
 	var args []string
 
 	if isMaster {

args_test.go

@@ -3,6 +3,7 @@ package main
 import (
 	"fmt"
 	"os"
+	"runtime"
 	"strings"
 	"testing"
 )
@@ -434,3 +435,55 @@ func TestSanitizeCodespaceNameForControl(t *testing.T) {
 		})
 	}
 }
+
+// TestBuildSSHMultiplexArgsWindows tests that multiplexing is disabled on Windows
+func TestBuildSSHMultiplexArgsWindows(t *testing.T) {
+// This test verifies the behavior on different platforms
+controlPath := "/tmp/test-socket"
+
+// Get args for both master and slave
+masterArgs := BuildSSHMultiplexArgs(controlPath, true)
+slaveArgs := BuildSSHMultiplexArgs(controlPath, false)
+
+// On Windows, both should return empty slices
+// On other platforms, they should contain multiplexing options
+if runtime.GOOS == "windows" {
+if len(masterArgs) != 0 {
+t.Errorf("On Windows, BuildSSHMultiplexArgs(master) should return empty slice, got %v", masterArgs)
+}
+if len(slaveArgs) != 0 {
+t.Errorf("On Windows, BuildSSHMultiplexArgs(slave) should return empty slice, got %v", slaveArgs)
+}
+} else {
+// On non-Windows platforms, verify multiplexing args are present
+if len(masterArgs) == 0 {
+t.Error("On non-Windows platforms, BuildSSHMultiplexArgs(master) should return multiplexing args")
+}
+if len(slaveArgs) == 0 {
+t.Error("On non-Windows platforms, BuildSSHMultiplexArgs(slave) should return multiplexing args")
+}
+
+// Verify ControlMaster settings
+foundMaster := false
+for _, arg := range masterArgs {
+if arg == "ControlMaster=auto" {
+foundMaster = true
+break
+}
+}
+if !foundMaster {
+t.Error("Master args should contain ControlMaster=auto on non-Windows platforms")
+}
+
+foundNo := false
+for _, arg := range slaveArgs {
+if arg == "ControlMaster=no" {
+foundNo = true
+break
+}
+}
+if !foundNo {
+t.Error("Slave args should contain ControlMaster=no on non-Windows platforms")
+}
+}
+}