Do not crash if the env is nil

Signed-off-by: Alvaro Saurin <[email protected]>

Author: inercia

pkg/model/provider/anthropic/client.go

@@ -103,6 +103,11 @@ func NewClient(ctx context.Context, cfg *latest.ModelConfig, env environment.Pro
 		return nil, errors.New("model type must be 'anthropic'")
 	}
 
+	// Ensure env is not nil - use default provider if nil
+	if env == nil {
+		env = environment.NewDefaultProvider()
+	}
+
 	var globalOptions options.ModelOptions
 	for _, opt := range opts {
 		opt(&globalOptions)

pkg/model/provider/bedrock/client.go

@@ -59,6 +59,11 @@ func NewClient(ctx context.Context, cfg *latest.ModelConfig, env environment.Pro
 		return nil, errors.New("model type must be 'amazon-bedrock'")
 	}
 
+	// Ensure env is not nil - use default provider if nil
+	if env == nil {
+		env = environment.NewDefaultProvider()
+	}
+
 	var globalOptions options.ModelOptions
 	for _, opt := range opts {
 		opt(&globalOptions)

pkg/model/provider/gemini/client.go

@@ -41,6 +41,11 @@ func NewClient(ctx context.Context, cfg *latest.ModelConfig, env environment.Pro
 		return nil, errors.New("model type must be 'google'")
 	}
 
+	// Ensure env is not nil - use default provider if nil
+	if env == nil {
+		env = environment.NewDefaultProvider()
+	}
+
 	var globalOptions options.ModelOptions
 	for _, opt := range opts {
 		opt(&globalOptions)

pkg/model/provider/openai/client.go

@@ -40,6 +40,11 @@ func NewClient(ctx context.Context, cfg *latest.ModelConfig, env environment.Pro
 		return nil, errors.New("model configuration is required")
 	}
 
+	// Ensure env is not nil - use default provider if nil
+	if env == nil {
+		env = environment.NewDefaultProvider()
+	}
+
 	var globalOptions options.ModelOptions
 	for _, opt := range opts {
 		opt(&globalOptions)