diff --git a/internal/config/provider.go b/internal/config/provider.go index be96e8844132efd12910597c504f66e153073a3e..7a98caaaa92a1c0401adccd79c3e945f813fbfc7 100644 --- a/internal/config/provider.go +++ b/internal/config/provider.go @@ -10,7 +10,6 @@ import ( "runtime" "strings" "sync" - "time" "github.com/charmbracelet/catwalk/pkg/catwalk" "github.com/charmbracelet/catwalk/pkg/embedded" @@ -126,8 +125,6 @@ func Providers(cfg *Config) ([]catwalk.Provider, error) { } func loadProviders(autoUpdateDisabled bool, client ProviderClient, path string) ([]catwalk.Provider, error) { - _, cacheExists := isCacheStale(path) - catwalkGetAndSave := func() ([]catwalk.Provider, error) { providers, err := client.GetProviders() if err != nil { @@ -141,11 +138,12 @@ func loadProviders(autoUpdateDisabled bool, client ProviderClient, path string) } return providers, nil } + switch { case autoUpdateDisabled: slog.Warn("Providers auto-update is disabled") - if cacheExists { + if _, err := os.Stat(path); err == nil { slog.Warn("Using locally cached providers") return loadProvidersFromCache(path) } @@ -168,11 +166,3 @@ func loadProviders(autoUpdateDisabled bool, client ProviderClient, path string) return providers, nil } } - -func isCacheStale(path string) (stale, exists bool) { - info, err := os.Stat(path) - if err != nil { - return true, false - } - return time.Since(info.ModTime()) > 24*time.Hour, true -}