| 1234567891011121314151617181920212223242526272829303132333435363738394041424344454647484950515253545556575859606162636465666768697071727374757677787980818283848586878889909192939495969798991001011021031041051061071081091101111121131141151161171181191201211221231241251261271281291301311321331341351361371381391401411421431441451461471481491501511521531541551561571581591601611621631641651661671681691701711721731741751761771781791801811821831841851861871881891901911921931941951961971981992002012022032042052062072082092102112122132142152162172182192202212222232242252262272282292302312322332342352362372382392402412422432442452462472482492502512522532542552562572582592602612622632642652662672682692702712722732742752762772782792802812822832842852862872882892902912922932942952962972982993003013023033043053063073083093103113123133143153163173183193203213223233243253263273283293303313323333343353363373383393403413423433443453463473483493503513523533543553563573583593603613623633643653663673683693703713723733743753763773783793803813823833843853863873883893903913923933943953963973983994004014024034044054064074084094104114124134144154164174184194204214224234244254264274284294304314324334344354364374384394404414424434444454464474484494504514524534544554564574584594604614624634644654664674684694704714724734744754764774784794804814824834844854864874884894904914924934944954964974984995005015025035045055065075085095105115125135145155165175185195205215225235245255265275285295305315325335345355365375385395405415425435445455465475485495505515525535545555565575585595605615625635645655665675685695705715725735745755765775785795805815825835845855865875885895905915925935945955965975985996006016026036046056066076086096106116126136146156166176186196206216226236246256266276286296306316326336346356366376386396406416426436446456466476486496506516526536546556566576586596606616626636646656666676686696706716726736746756766776786796806816826836846856866876886896906916926936946956966976986997007017027037047057067077087097107117127137147157167177187197207217227237247257267277287297307317327337347357367377387397407417427437447457467477487497507517527537547557567577587597607617627637647657667677687697707717727737747757767777787797807817827837847857867877887897907917927937947957967977987998008018028038048058068078088098108118128138148158168178188198208218228238248258268278288298308318328338348358368378388398408418428438448458468478488498508518528538548558568578588598608618628638648658668678688698708718728738748758768778788798808818828838848858868878888898908918928938948958968978988999009019029039049059069079089099109119129139149159169179189199209219229239249259269279289299309319329339349359369379389399409419429439449459469479489499509519529539549559569579589599609619629639649659669679689699709719729739749759769779789799809819829839849859869879889899909919929939949959969979989991000100110021003100410051006100710081009101010111012101310141015101610171018101910201021102210231024102510261027102810291030103110321033103410351036103710381039104010411042104310441045104610471048104910501051105210531054105510561057105810591060106110621063106410651066106710681069107010711072107310741075107610771078107910801081108210831084108510861087108810891090109110921093109410951096109710981099110011011102110311041105110611071108110911101111111211131114111511161117111811191120112111221123112411251126112711281129113011311132113311341135113611371138113911401141114211431144114511461147114811491150115111521153115411551156115711581159116011611162116311641165116611671168116911701171117211731174117511761177117811791180118111821183118411851186 |
- package config
- import (
- "io"
- "log/slog"
- "os"
- "path/filepath"
- "strings"
- "testing"
- "github.com/charmbracelet/catwalk/pkg/catwalk"
- "github.com/charmbracelet/crush/internal/csync"
- "github.com/charmbracelet/crush/internal/env"
- "github.com/stretchr/testify/require"
- )
- func TestMain(m *testing.M) {
- slog.SetDefault(slog.New(slog.NewTextHandler(io.Discard, nil)))
- exitVal := m.Run()
- os.Exit(exitVal)
- }
- func TestConfig_LoadFromReaders(t *testing.T) {
- data1 := strings.NewReader(`{"providers": {"openai": {"api_key": "key1", "base_url": "https://api.openai.com/v1"}}}`)
- data2 := strings.NewReader(`{"providers": {"openai": {"api_key": "key2", "base_url": "https://api.openai.com/v2"}}}`)
- data3 := strings.NewReader(`{"providers": {"openai": {}}}`)
- loadedConfig, err := loadFromReaders([]io.Reader{data1, data2, data3})
- require.NoError(t, err)
- require.NotNil(t, loadedConfig)
- require.Equal(t, 1, loadedConfig.Providers.Len())
- pc, _ := loadedConfig.Providers.Get("openai")
- require.Equal(t, "key2", pc.APIKey)
- require.Equal(t, "https://api.openai.com/v2", pc.BaseURL)
- }
- func TestConfig_setDefaults(t *testing.T) {
- cfg := &Config{}
- cfg.setDefaults("/tmp")
- require.NotNil(t, cfg.Options)
- require.NotNil(t, cfg.Options.TUI)
- require.NotNil(t, cfg.Options.ContextPaths)
- require.NotNil(t, cfg.Providers)
- require.NotNil(t, cfg.Models)
- require.NotNil(t, cfg.LSP)
- require.NotNil(t, cfg.MCP)
- require.Equal(t, filepath.Join("/tmp", ".crush"), cfg.Options.DataDirectory)
- for _, path := range defaultContextPaths {
- require.Contains(t, cfg.Options.ContextPaths, path)
- }
- require.Equal(t, "/tmp", cfg.workingDir)
- }
- func TestConfig_configureProviders(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: "openai",
- APIKey: "$OPENAI_API_KEY",
- APIEndpoint: "https://api.openai.com/v1",
- Models: []catwalk.Model{{
- ID: "test-model",
- }},
- },
- }
- cfg := &Config{}
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{
- "OPENAI_API_KEY": "test-key",
- })
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- require.Equal(t, 1, cfg.Providers.Len())
- // We want to make sure that we keep the configured API key as a placeholder
- pc, _ := cfg.Providers.Get("openai")
- require.Equal(t, "$OPENAI_API_KEY", pc.APIKey)
- }
- func TestConfig_configureProvidersWithOverride(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: "openai",
- APIKey: "$OPENAI_API_KEY",
- APIEndpoint: "https://api.openai.com/v1",
- Models: []catwalk.Model{{
- ID: "test-model",
- }},
- },
- }
- cfg := &Config{
- Providers: csync.NewMap[string, ProviderConfig](),
- }
- cfg.Providers.Set("openai", ProviderConfig{
- APIKey: "xyz",
- BaseURL: "https://api.openai.com/v2",
- Models: []catwalk.Model{
- {
- ID: "test-model",
- Name: "Updated",
- },
- {
- ID: "another-model",
- },
- },
- })
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{
- "OPENAI_API_KEY": "test-key",
- })
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- require.Equal(t, 1, cfg.Providers.Len())
- // We want to make sure that we keep the configured API key as a placeholder
- pc, _ := cfg.Providers.Get("openai")
- require.Equal(t, "xyz", pc.APIKey)
- require.Equal(t, "https://api.openai.com/v2", pc.BaseURL)
- require.Len(t, pc.Models, 2)
- require.Equal(t, "Updated", pc.Models[0].Name)
- }
- func TestConfig_configureProvidersWithNewProvider(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: "openai",
- APIKey: "$OPENAI_API_KEY",
- APIEndpoint: "https://api.openai.com/v1",
- Models: []catwalk.Model{{
- ID: "test-model",
- }},
- },
- }
- cfg := &Config{
- Providers: csync.NewMapFrom(map[string]ProviderConfig{
- "custom": {
- APIKey: "xyz",
- BaseURL: "https://api.someendpoint.com/v2",
- Models: []catwalk.Model{
- {
- ID: "test-model",
- },
- },
- },
- }),
- }
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{
- "OPENAI_API_KEY": "test-key",
- })
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- // Should be to because of the env variable
- require.Equal(t, cfg.Providers.Len(), 2)
- // We want to make sure that we keep the configured API key as a placeholder
- pc, _ := cfg.Providers.Get("custom")
- require.Equal(t, "xyz", pc.APIKey)
- // Make sure we set the ID correctly
- require.Equal(t, "custom", pc.ID)
- require.Equal(t, "https://api.someendpoint.com/v2", pc.BaseURL)
- require.Len(t, pc.Models, 1)
- _, ok := cfg.Providers.Get("openai")
- require.True(t, ok, "OpenAI provider should still be present")
- }
- func TestConfig_configureProvidersBedrockWithCredentials(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: catwalk.InferenceProviderBedrock,
- APIKey: "",
- APIEndpoint: "",
- Models: []catwalk.Model{{
- ID: "anthropic.claude-sonnet-4-20250514-v1:0",
- }},
- },
- }
- cfg := &Config{}
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{
- "AWS_ACCESS_KEY_ID": "test-key-id",
- "AWS_SECRET_ACCESS_KEY": "test-secret-key",
- })
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- require.Equal(t, cfg.Providers.Len(), 1)
- bedrockProvider, ok := cfg.Providers.Get("bedrock")
- require.True(t, ok, "Bedrock provider should be present")
- require.Len(t, bedrockProvider.Models, 1)
- require.Equal(t, "anthropic.claude-sonnet-4-20250514-v1:0", bedrockProvider.Models[0].ID)
- }
- func TestConfig_configureProvidersBedrockWithoutCredentials(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: catwalk.InferenceProviderBedrock,
- APIKey: "",
- APIEndpoint: "",
- Models: []catwalk.Model{{
- ID: "anthropic.claude-sonnet-4-20250514-v1:0",
- }},
- },
- }
- cfg := &Config{}
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{})
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- // Provider should not be configured without credentials
- require.Equal(t, cfg.Providers.Len(), 0)
- }
- func TestConfig_configureProvidersBedrockWithoutUnsupportedModel(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: catwalk.InferenceProviderBedrock,
- APIKey: "",
- APIEndpoint: "",
- Models: []catwalk.Model{{
- ID: "some-random-model",
- }},
- },
- }
- cfg := &Config{}
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{
- "AWS_ACCESS_KEY_ID": "test-key-id",
- "AWS_SECRET_ACCESS_KEY": "test-secret-key",
- })
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.Error(t, err)
- }
- func TestConfig_configureProvidersVertexAIWithCredentials(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: catwalk.InferenceProviderVertexAI,
- APIKey: "",
- APIEndpoint: "",
- Models: []catwalk.Model{{
- ID: "gemini-pro",
- }},
- },
- }
- cfg := &Config{}
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{
- "VERTEXAI_PROJECT": "test-project",
- "VERTEXAI_LOCATION": "us-central1",
- })
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- require.Equal(t, cfg.Providers.Len(), 1)
- vertexProvider, ok := cfg.Providers.Get("vertexai")
- require.True(t, ok, "VertexAI provider should be present")
- require.Len(t, vertexProvider.Models, 1)
- require.Equal(t, "gemini-pro", vertexProvider.Models[0].ID)
- require.Equal(t, "test-project", vertexProvider.ExtraParams["project"])
- require.Equal(t, "us-central1", vertexProvider.ExtraParams["location"])
- }
- func TestConfig_configureProvidersVertexAIWithoutCredentials(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: catwalk.InferenceProviderVertexAI,
- APIKey: "",
- APIEndpoint: "",
- Models: []catwalk.Model{{
- ID: "gemini-pro",
- }},
- },
- }
- cfg := &Config{}
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{
- "GOOGLE_GENAI_USE_VERTEXAI": "false",
- "GOOGLE_CLOUD_PROJECT": "test-project",
- "GOOGLE_CLOUD_LOCATION": "us-central1",
- })
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- // Provider should not be configured without proper credentials
- require.Equal(t, cfg.Providers.Len(), 0)
- }
- func TestConfig_configureProvidersVertexAIMissingProject(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: catwalk.InferenceProviderVertexAI,
- APIKey: "",
- APIEndpoint: "",
- Models: []catwalk.Model{{
- ID: "gemini-pro",
- }},
- },
- }
- cfg := &Config{}
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{
- "GOOGLE_GENAI_USE_VERTEXAI": "true",
- "GOOGLE_CLOUD_LOCATION": "us-central1",
- })
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- // Provider should not be configured without project
- require.Equal(t, cfg.Providers.Len(), 0)
- }
- func TestConfig_configureProvidersSetProviderID(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: "openai",
- APIKey: "$OPENAI_API_KEY",
- APIEndpoint: "https://api.openai.com/v1",
- Models: []catwalk.Model{{
- ID: "test-model",
- }},
- },
- }
- cfg := &Config{}
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{
- "OPENAI_API_KEY": "test-key",
- })
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- require.Equal(t, cfg.Providers.Len(), 1)
- // Provider ID should be set
- pc, _ := cfg.Providers.Get("openai")
- require.Equal(t, "openai", pc.ID)
- }
- func TestConfig_EnabledProviders(t *testing.T) {
- t.Run("all providers enabled", func(t *testing.T) {
- cfg := &Config{
- Providers: csync.NewMapFrom(map[string]ProviderConfig{
- "openai": {
- ID: "openai",
- APIKey: "key1",
- Disable: false,
- },
- "anthropic": {
- ID: "anthropic",
- APIKey: "key2",
- Disable: false,
- },
- }),
- }
- enabled := cfg.EnabledProviders()
- require.Len(t, enabled, 2)
- })
- t.Run("some providers disabled", func(t *testing.T) {
- cfg := &Config{
- Providers: csync.NewMapFrom(map[string]ProviderConfig{
- "openai": {
- ID: "openai",
- APIKey: "key1",
- Disable: false,
- },
- "anthropic": {
- ID: "anthropic",
- APIKey: "key2",
- Disable: true,
- },
- }),
- }
- enabled := cfg.EnabledProviders()
- require.Len(t, enabled, 1)
- require.Equal(t, "openai", enabled[0].ID)
- })
- t.Run("empty providers map", func(t *testing.T) {
- cfg := &Config{
- Providers: csync.NewMap[string, ProviderConfig](),
- }
- enabled := cfg.EnabledProviders()
- require.Len(t, enabled, 0)
- })
- }
- func TestConfig_IsConfigured(t *testing.T) {
- t.Run("returns true when at least one provider is enabled", func(t *testing.T) {
- cfg := &Config{
- Providers: csync.NewMapFrom(map[string]ProviderConfig{
- "openai": {
- ID: "openai",
- APIKey: "key1",
- Disable: false,
- },
- }),
- }
- require.True(t, cfg.IsConfigured())
- })
- t.Run("returns false when no providers are configured", func(t *testing.T) {
- cfg := &Config{
- Providers: csync.NewMap[string, ProviderConfig](),
- }
- require.False(t, cfg.IsConfigured())
- })
- t.Run("returns false when all providers are disabled", func(t *testing.T) {
- cfg := &Config{
- Providers: csync.NewMapFrom(map[string]ProviderConfig{
- "openai": {
- ID: "openai",
- APIKey: "key1",
- Disable: true,
- },
- "anthropic": {
- ID: "anthropic",
- APIKey: "key2",
- Disable: true,
- },
- }),
- }
- require.False(t, cfg.IsConfigured())
- })
- }
- func TestConfig_configureProvidersWithDisabledProvider(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: "openai",
- APIKey: "$OPENAI_API_KEY",
- APIEndpoint: "https://api.openai.com/v1",
- Models: []catwalk.Model{{
- ID: "test-model",
- }},
- },
- }
- cfg := &Config{
- Providers: csync.NewMapFrom(map[string]ProviderConfig{
- "openai": {
- Disable: true,
- },
- }),
- }
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{
- "OPENAI_API_KEY": "test-key",
- })
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- // Provider should be removed from config when disabled
- require.Equal(t, cfg.Providers.Len(), 0)
- _, exists := cfg.Providers.Get("openai")
- require.False(t, exists)
- }
- func TestConfig_configureProvidersCustomProviderValidation(t *testing.T) {
- t.Run("custom provider with missing API key is allowed, but not known providers", func(t *testing.T) {
- cfg := &Config{
- Providers: csync.NewMapFrom(map[string]ProviderConfig{
- "custom": {
- BaseURL: "https://api.custom.com/v1",
- Models: []catwalk.Model{{
- ID: "test-model",
- }},
- },
- "openai": {
- APIKey: "$MISSING",
- },
- }),
- }
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{})
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, []catwalk.Provider{})
- require.NoError(t, err)
- require.Equal(t, cfg.Providers.Len(), 1)
- _, exists := cfg.Providers.Get("custom")
- require.True(t, exists)
- })
- t.Run("custom provider with missing BaseURL is removed", func(t *testing.T) {
- cfg := &Config{
- Providers: csync.NewMapFrom(map[string]ProviderConfig{
- "custom": {
- APIKey: "test-key",
- Models: []catwalk.Model{{
- ID: "test-model",
- }},
- },
- }),
- }
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{})
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, []catwalk.Provider{})
- require.NoError(t, err)
- require.Equal(t, cfg.Providers.Len(), 0)
- _, exists := cfg.Providers.Get("custom")
- require.False(t, exists)
- })
- t.Run("custom provider with no models is removed", func(t *testing.T) {
- cfg := &Config{
- Providers: csync.NewMapFrom(map[string]ProviderConfig{
- "custom": {
- APIKey: "test-key",
- BaseURL: "https://api.custom.com/v1",
- Models: []catwalk.Model{},
- },
- }),
- }
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{})
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, []catwalk.Provider{})
- require.NoError(t, err)
- require.Equal(t, cfg.Providers.Len(), 0)
- _, exists := cfg.Providers.Get("custom")
- require.False(t, exists)
- })
- t.Run("custom provider with unsupported type is removed", func(t *testing.T) {
- cfg := &Config{
- Providers: csync.NewMapFrom(map[string]ProviderConfig{
- "custom": {
- APIKey: "test-key",
- BaseURL: "https://api.custom.com/v1",
- Type: "unsupported",
- Models: []catwalk.Model{{
- ID: "test-model",
- }},
- },
- }),
- }
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{})
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, []catwalk.Provider{})
- require.NoError(t, err)
- require.Equal(t, cfg.Providers.Len(), 0)
- _, exists := cfg.Providers.Get("custom")
- require.False(t, exists)
- })
- t.Run("valid custom provider is kept and ID is set", func(t *testing.T) {
- cfg := &Config{
- Providers: csync.NewMapFrom(map[string]ProviderConfig{
- "custom": {
- APIKey: "test-key",
- BaseURL: "https://api.custom.com/v1",
- Type: catwalk.TypeOpenAI,
- Models: []catwalk.Model{{
- ID: "test-model",
- }},
- },
- }),
- }
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{})
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, []catwalk.Provider{})
- require.NoError(t, err)
- require.Equal(t, cfg.Providers.Len(), 1)
- customProvider, exists := cfg.Providers.Get("custom")
- require.True(t, exists)
- require.Equal(t, "custom", customProvider.ID)
- require.Equal(t, "test-key", customProvider.APIKey)
- require.Equal(t, "https://api.custom.com/v1", customProvider.BaseURL)
- })
- t.Run("custom anthropic provider is supported", func(t *testing.T) {
- cfg := &Config{
- Providers: csync.NewMapFrom(map[string]ProviderConfig{
- "custom-anthropic": {
- APIKey: "test-key",
- BaseURL: "https://api.anthropic.com/v1",
- Type: catwalk.TypeAnthropic,
- Models: []catwalk.Model{{
- ID: "claude-3-sonnet",
- }},
- },
- }),
- }
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{})
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, []catwalk.Provider{})
- require.NoError(t, err)
- require.Equal(t, cfg.Providers.Len(), 1)
- customProvider, exists := cfg.Providers.Get("custom-anthropic")
- require.True(t, exists)
- require.Equal(t, "custom-anthropic", customProvider.ID)
- require.Equal(t, "test-key", customProvider.APIKey)
- require.Equal(t, "https://api.anthropic.com/v1", customProvider.BaseURL)
- require.Equal(t, catwalk.TypeAnthropic, customProvider.Type)
- })
- t.Run("disabled custom provider is removed", func(t *testing.T) {
- cfg := &Config{
- Providers: csync.NewMapFrom(map[string]ProviderConfig{
- "custom": {
- APIKey: "test-key",
- BaseURL: "https://api.custom.com/v1",
- Type: catwalk.TypeOpenAI,
- Disable: true,
- Models: []catwalk.Model{{
- ID: "test-model",
- }},
- },
- }),
- }
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{})
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, []catwalk.Provider{})
- require.NoError(t, err)
- require.Equal(t, cfg.Providers.Len(), 0)
- _, exists := cfg.Providers.Get("custom")
- require.False(t, exists)
- })
- }
- func TestConfig_configureProvidersEnhancedCredentialValidation(t *testing.T) {
- t.Run("VertexAI provider removed when credentials missing with existing config", func(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: catwalk.InferenceProviderVertexAI,
- APIKey: "",
- APIEndpoint: "",
- Models: []catwalk.Model{{
- ID: "gemini-pro",
- }},
- },
- }
- cfg := &Config{
- Providers: csync.NewMapFrom(map[string]ProviderConfig{
- "vertexai": {
- BaseURL: "custom-url",
- },
- }),
- }
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{
- "GOOGLE_GENAI_USE_VERTEXAI": "false",
- })
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- require.Equal(t, cfg.Providers.Len(), 0)
- _, exists := cfg.Providers.Get("vertexai")
- require.False(t, exists)
- })
- t.Run("Bedrock provider removed when AWS credentials missing with existing config", func(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: catwalk.InferenceProviderBedrock,
- APIKey: "",
- APIEndpoint: "",
- Models: []catwalk.Model{{
- ID: "anthropic.claude-sonnet-4-20250514-v1:0",
- }},
- },
- }
- cfg := &Config{
- Providers: csync.NewMapFrom(map[string]ProviderConfig{
- "bedrock": {
- BaseURL: "custom-url",
- },
- }),
- }
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{})
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- require.Equal(t, cfg.Providers.Len(), 0)
- _, exists := cfg.Providers.Get("bedrock")
- require.False(t, exists)
- })
- t.Run("provider removed when API key missing with existing config", func(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: "openai",
- APIKey: "$MISSING_API_KEY",
- APIEndpoint: "https://api.openai.com/v1",
- Models: []catwalk.Model{{
- ID: "test-model",
- }},
- },
- }
- cfg := &Config{
- Providers: csync.NewMapFrom(map[string]ProviderConfig{
- "openai": {
- BaseURL: "custom-url",
- },
- }),
- }
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{})
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- require.Equal(t, cfg.Providers.Len(), 0)
- _, exists := cfg.Providers.Get("openai")
- require.False(t, exists)
- })
- t.Run("known provider should still be added if the endpoint is missing the client will use default endpoints", func(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: "openai",
- APIKey: "$OPENAI_API_KEY",
- APIEndpoint: "$MISSING_ENDPOINT",
- Models: []catwalk.Model{{
- ID: "test-model",
- }},
- },
- }
- cfg := &Config{
- Providers: csync.NewMapFrom(map[string]ProviderConfig{
- "openai": {
- APIKey: "test-key",
- },
- }),
- }
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{
- "OPENAI_API_KEY": "test-key",
- })
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- require.Equal(t, cfg.Providers.Len(), 1)
- _, exists := cfg.Providers.Get("openai")
- require.True(t, exists)
- })
- }
- func TestConfig_defaultModelSelection(t *testing.T) {
- t.Run("default behavior uses the default models for given provider", func(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: "openai",
- APIKey: "abc",
- DefaultLargeModelID: "large-model",
- DefaultSmallModelID: "small-model",
- Models: []catwalk.Model{
- {
- ID: "large-model",
- DefaultMaxTokens: 1000,
- },
- {
- ID: "small-model",
- DefaultMaxTokens: 500,
- },
- },
- },
- }
- cfg := &Config{}
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{})
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- large, small, err := cfg.defaultModelSelection(knownProviders)
- require.NoError(t, err)
- require.Equal(t, "large-model", large.Model)
- require.Equal(t, "openai", large.Provider)
- require.Equal(t, int64(1000), large.MaxTokens)
- require.Equal(t, "small-model", small.Model)
- require.Equal(t, "openai", small.Provider)
- require.Equal(t, int64(500), small.MaxTokens)
- })
- t.Run("should error if no providers configured", func(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: "openai",
- APIKey: "$MISSING_KEY",
- DefaultLargeModelID: "large-model",
- DefaultSmallModelID: "small-model",
- Models: []catwalk.Model{
- {
- ID: "large-model",
- DefaultMaxTokens: 1000,
- },
- {
- ID: "small-model",
- DefaultMaxTokens: 500,
- },
- },
- },
- }
- cfg := &Config{}
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{})
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- _, _, err = cfg.defaultModelSelection(knownProviders)
- require.Error(t, err)
- })
- t.Run("should error if model is missing", func(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: "openai",
- APIKey: "abc",
- DefaultLargeModelID: "large-model",
- DefaultSmallModelID: "small-model",
- Models: []catwalk.Model{
- {
- ID: "not-large-model",
- DefaultMaxTokens: 1000,
- },
- {
- ID: "small-model",
- DefaultMaxTokens: 500,
- },
- },
- },
- }
- cfg := &Config{}
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{})
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- _, _, err = cfg.defaultModelSelection(knownProviders)
- require.Error(t, err)
- })
- t.Run("should configure the default models with a custom provider", func(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: "openai",
- APIKey: "$MISSING", // will not be included in the config
- DefaultLargeModelID: "large-model",
- DefaultSmallModelID: "small-model",
- Models: []catwalk.Model{
- {
- ID: "not-large-model",
- DefaultMaxTokens: 1000,
- },
- {
- ID: "small-model",
- DefaultMaxTokens: 500,
- },
- },
- },
- }
- cfg := &Config{
- Providers: csync.NewMapFrom(map[string]ProviderConfig{
- "custom": {
- APIKey: "test-key",
- BaseURL: "https://api.custom.com/v1",
- Models: []catwalk.Model{
- {
- ID: "model",
- DefaultMaxTokens: 600,
- },
- },
- },
- }),
- }
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{})
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- large, small, err := cfg.defaultModelSelection(knownProviders)
- require.NoError(t, err)
- require.Equal(t, "model", large.Model)
- require.Equal(t, "custom", large.Provider)
- require.Equal(t, int64(600), large.MaxTokens)
- require.Equal(t, "model", small.Model)
- require.Equal(t, "custom", small.Provider)
- require.Equal(t, int64(600), small.MaxTokens)
- })
- t.Run("should fail if no model configured", func(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: "openai",
- APIKey: "$MISSING", // will not be included in the config
- DefaultLargeModelID: "large-model",
- DefaultSmallModelID: "small-model",
- Models: []catwalk.Model{
- {
- ID: "not-large-model",
- DefaultMaxTokens: 1000,
- },
- {
- ID: "small-model",
- DefaultMaxTokens: 500,
- },
- },
- },
- }
- cfg := &Config{
- Providers: csync.NewMapFrom(map[string]ProviderConfig{
- "custom": {
- APIKey: "test-key",
- BaseURL: "https://api.custom.com/v1",
- Models: []catwalk.Model{},
- },
- }),
- }
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{})
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- _, _, err = cfg.defaultModelSelection(knownProviders)
- require.Error(t, err)
- })
- t.Run("should use the default provider first", func(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: "openai",
- APIKey: "set",
- DefaultLargeModelID: "large-model",
- DefaultSmallModelID: "small-model",
- Models: []catwalk.Model{
- {
- ID: "large-model",
- DefaultMaxTokens: 1000,
- },
- {
- ID: "small-model",
- DefaultMaxTokens: 500,
- },
- },
- },
- }
- cfg := &Config{
- Providers: csync.NewMapFrom(map[string]ProviderConfig{
- "custom": {
- APIKey: "test-key",
- BaseURL: "https://api.custom.com/v1",
- Models: []catwalk.Model{
- {
- ID: "large-model",
- DefaultMaxTokens: 1000,
- },
- },
- },
- }),
- }
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{})
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- large, small, err := cfg.defaultModelSelection(knownProviders)
- require.NoError(t, err)
- require.Equal(t, "large-model", large.Model)
- require.Equal(t, "openai", large.Provider)
- require.Equal(t, int64(1000), large.MaxTokens)
- require.Equal(t, "small-model", small.Model)
- require.Equal(t, "openai", small.Provider)
- require.Equal(t, int64(500), small.MaxTokens)
- })
- }
- func TestConfig_configureSelectedModels(t *testing.T) {
- t.Run("should override defaults", func(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: "openai",
- APIKey: "abc",
- DefaultLargeModelID: "large-model",
- DefaultSmallModelID: "small-model",
- Models: []catwalk.Model{
- {
- ID: "larger-model",
- DefaultMaxTokens: 2000,
- },
- {
- ID: "large-model",
- DefaultMaxTokens: 1000,
- },
- {
- ID: "small-model",
- DefaultMaxTokens: 500,
- },
- },
- },
- }
- cfg := &Config{
- Models: map[SelectedModelType]SelectedModel{
- "large": {
- Model: "larger-model",
- },
- },
- }
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{})
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- err = cfg.configureSelectedModels(knownProviders)
- require.NoError(t, err)
- large := cfg.Models[SelectedModelTypeLarge]
- small := cfg.Models[SelectedModelTypeSmall]
- require.Equal(t, "larger-model", large.Model)
- require.Equal(t, "openai", large.Provider)
- require.Equal(t, int64(2000), large.MaxTokens)
- require.Equal(t, "small-model", small.Model)
- require.Equal(t, "openai", small.Provider)
- require.Equal(t, int64(500), small.MaxTokens)
- })
- t.Run("should be possible to use multiple providers", func(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: "openai",
- APIKey: "abc",
- DefaultLargeModelID: "large-model",
- DefaultSmallModelID: "small-model",
- Models: []catwalk.Model{
- {
- ID: "large-model",
- DefaultMaxTokens: 1000,
- },
- {
- ID: "small-model",
- DefaultMaxTokens: 500,
- },
- },
- },
- {
- ID: "anthropic",
- APIKey: "abc",
- DefaultLargeModelID: "a-large-model",
- DefaultSmallModelID: "a-small-model",
- Models: []catwalk.Model{
- {
- ID: "a-large-model",
- DefaultMaxTokens: 1000,
- },
- {
- ID: "a-small-model",
- DefaultMaxTokens: 200,
- },
- },
- },
- }
- cfg := &Config{
- Models: map[SelectedModelType]SelectedModel{
- "small": {
- Model: "a-small-model",
- Provider: "anthropic",
- MaxTokens: 300,
- },
- },
- }
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{})
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- err = cfg.configureSelectedModels(knownProviders)
- require.NoError(t, err)
- large := cfg.Models[SelectedModelTypeLarge]
- small := cfg.Models[SelectedModelTypeSmall]
- require.Equal(t, "large-model", large.Model)
- require.Equal(t, "openai", large.Provider)
- require.Equal(t, int64(1000), large.MaxTokens)
- require.Equal(t, "a-small-model", small.Model)
- require.Equal(t, "anthropic", small.Provider)
- require.Equal(t, int64(300), small.MaxTokens)
- })
- t.Run("should override the max tokens only", func(t *testing.T) {
- knownProviders := []catwalk.Provider{
- {
- ID: "openai",
- APIKey: "abc",
- DefaultLargeModelID: "large-model",
- DefaultSmallModelID: "small-model",
- Models: []catwalk.Model{
- {
- ID: "large-model",
- DefaultMaxTokens: 1000,
- },
- {
- ID: "small-model",
- DefaultMaxTokens: 500,
- },
- },
- },
- }
- cfg := &Config{
- Models: map[SelectedModelType]SelectedModel{
- "large": {
- MaxTokens: 100,
- },
- },
- }
- cfg.setDefaults("/tmp")
- env := env.NewFromMap(map[string]string{})
- resolver := NewEnvironmentVariableResolver(env)
- err := cfg.configureProviders(env, resolver, knownProviders)
- require.NoError(t, err)
- err = cfg.configureSelectedModels(knownProviders)
- require.NoError(t, err)
- large := cfg.Models[SelectedModelTypeLarge]
- require.Equal(t, "large-model", large.Model)
- require.Equal(t, "openai", large.Provider)
- require.Equal(t, int64(100), large.MaxTokens)
- })
- }
|