Better error message if Desktop not available while running using docker agent the gateway

Signed-off-by: Guillaume Tardif <guillaume.tardif@gmail.com>

Author: gtardif

pkg/model/provider/anthropic/client.go

@@ -96,7 +96,7 @@ func NewClient(ctx context.Context, cfg *latest.ModelConfig, env environment.Pro
 			// Query a fresh auth token each time the client is used
 			authToken, _ := env.Get(ctx, environment.DockerDesktopTokenEnv)
 			if authToken == "" {
-				return anthropic.Client{}, errors.New("failed to get Docker Desktop token for Gateway")
+				return anthropic.Client{}, errors.New(base.NoDesktopTokenErrorMessage)
 			}
 
 			url, err := url.Parse(gateway)

pkg/model/provider/base/base.go

@@ -6,6 +6,8 @@ import (
 	"github.com/docker/docker-agent/pkg/model/provider/options"
 )
 
+const NoDesktopTokenErrorMessage = "failed to get Docker Desktop token for Gateway. Is Docker Desktop running and are you signed in?"
+
 // Config is a common base configuration shared by all provider clients.
 // It can be embedded in provider-specific Client structs to avoid code duplication.
 type Config struct {

pkg/model/provider/gemini/client.go

@@ -130,7 +130,7 @@ func NewClient(ctx context.Context, cfg *latest.ModelConfig, env environment.Pro
 			// Query a fresh auth token each time the client is used
 			authToken, _ := env.Get(ctx, environment.DockerDesktopTokenEnv)
 			if authToken == "" {
-				return nil, errors.New("failed to get Docker Desktop token for Gateway")
+				return nil, errors.New(base.NoDesktopTokenErrorMessage)
 			}
 
 			url, err := url.Parse(gateway)

pkg/model/provider/openai/client.go

@@ -115,7 +115,7 @@ func NewClient(ctx context.Context, cfg *latest.ModelConfig, env environment.Pro
 			// Query a fresh auth token each time the client is used
 			authToken, _ := env.Get(ctx, environment.DockerDesktopTokenEnv)
 			if authToken == "" {
-				return nil, errors.New("failed to get Docker Desktop token for Gateway")
+				return nil, errors.New(base.NoDesktopTokenErrorMessage)
 			}
 
 			url, err := url.Parse(gateway)