fic nil options

Author: qmuntal

go/client.go

@@ -144,9 +144,7 @@ func NewClient(options *ClientOptions) *Client {
 		if options.LogLevel != "" {
 			opts.LogLevel = options.LogLevel
 		}
-		if options.Env == nil {
-			opts.Env = os.Environ()
-		} else {
+		if options.Env != nil {
 			opts.Env = options.Env
 		}
 		if options.AutoStart != nil {
@@ -163,6 +161,11 @@ func NewClient(options *ClientOptions) *Client {
 		}
 	}
 
+	// Default Env to current environment if not set
+	if opts.Env == nil {
+		opts.Env = os.Environ()
+	}
+
 	// Check environment variable for CLI path
 	if cliPath := os.Getenv("COPILOT_CLI_PATH"); cliPath != "" {
 		opts.CLIPath = cliPath

go/client_test.go

@@ -340,6 +340,14 @@ func TestClient_EnvOptions(t *testing.T) {
 		}
 	})
 
+	t.Run("should default to inherit from current process with nil options", func(t *testing.T) {
+		client := NewClient(nil)
+
+		if want := os.Environ(); !reflect.DeepEqual(client.options.Env, want) {
+			t.Errorf("Expected Env to be %v, got %v", want, client.options.Env)
+		}
+	})
+
 	t.Run("should allow empty environment", func(t *testing.T) {
 		client := NewClient(&ClientOptions{
 			Env: []string{},