From dcf03db072bcb1f7f067c35b889f7411517c94f1 Mon Sep 17 00:00:00 2001 From: Andrey Nering Date: Fri, 20 Feb 2026 10:44:53 -0300 Subject: [PATCH] refactor: simply code --- internal/config/load.go | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/internal/config/load.go b/internal/config/load.go index 077598509796fa13b47dd0e0f05a3dbaeb4975db..3fba44aa9142c52b8966b1dbe994cef0ae654c48 100644 --- a/internal/config/load.go +++ b/internal/config/load.go @@ -219,15 +219,19 @@ func (c *Config) configureProviders(env env.Env, resolver VariableResolver, know switch p.ID { // Handle specific providers that require additional configuration case catwalk.InferenceProviderVertexAI: - if !hasVertexCredentials(env) { + var ( + project = env.Get("VERTEXAI_PROJECT") + location = env.Get("VERTEXAI_LOCATION") + ) + if project == "" || location == "" { if configExists { slog.Warn("Skipping Vertex AI provider due to missing credentials") c.Providers.Del(string(p.ID)) } continue } - prepared.ExtraParams["project"] = env.Get("VERTEXAI_PROJECT") - prepared.ExtraParams["location"] = env.Get("VERTEXAI_LOCATION") + prepared.ExtraParams["project"] = project + prepared.ExtraParams["location"] = location case catwalk.InferenceProviderAzure: endpoint, err := resolver.ResolveValue(p.APIEndpoint) if err != nil || endpoint == "" { @@ -677,12 +681,6 @@ func loadFromBytes(configs [][]byte) (*Config, error) { return &config, nil } -func hasVertexCredentials(env env.Env) bool { - hasProject := env.Get("VERTEXAI_PROJECT") != "" - hasLocation := env.Get("VERTEXAI_LOCATION") != "" - return hasProject && hasLocation -} - func hasAWSCredentials(env env.Env) bool { if env.Get("AWS_BEARER_TOKEN_BEDROCK") != "" { return true