Apply suggestions from code review

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

Author: qmuntal

go/client.go

@@ -112,7 +112,7 @@ func NewClient(options *ClientOptions) *Client {
 
 	if options != nil {
 		// Validate mutually exclusive options
-		if options.CLIUrl != "" && ((options.UseStdio != nil && *options.UseStdio) || options.CLIPath != "") {
+		if options.CLIUrl != "" && ((options.UseStdio != nil) || options.CLIPath != "") {
 			panic("CLIUrl is mutually exclusive with UseStdio and CLIPath")
 		}
 

go/e2e/client_test.go

@@ -54,7 +54,7 @@ func TestClient(t *testing.T) {
 	t.Run("should start and connect to server using tcp", func(t *testing.T) {
 		client := copilot.NewClient(&copilot.ClientOptions{
 			CLIPath:  cliPath,
-			UseStdio: copilot.Bool(true),
+			UseStdio: copilot.Bool(false),
 		})
 		t.Cleanup(func() { client.ForceStop() })
 

go/types.go

@@ -18,7 +18,8 @@ type ClientOptions struct {
 	Cwd string
 	// Port for TCP transport (default: 0 = random port)
 	Port int
-	// UseStdio enables stdio transport instead of TCP (default: true)
+	// UseStdio controls whether to use stdio transport instead of TCP.
+	// Default: nil (use default = true, i.e. stdio). Use Bool(false) to explicitly select TCP.
 	UseStdio *bool
 	// CLIUrl is the URL of an existing Copilot CLI server to connect to over TCP
 	// Format: "host:port", "http://host:port", or just "port" (defaults to localhost)