on main help sign in
refactor(catwalk): remove stale helper to avoid confusion
commit449de92d283aa41575befa57fee49a9210034f31 authorAndrey Nering <andreynering@users.noreply.github.com> dateMon 3 Nov 2025, 14:03 (1 year ago) parentfc128be
1 file changed+2−12
internal/config/provider.go+2−12
internal/config/provider.go+2−12
@@ -10,7 +10,6 @@ import (
1010 "runtime"
1111 "strings"
1212 "sync"
13− "time"
1413
1514 "github.com/charmbracelet/catwalk/pkg/catwalk"
1615 "github.com/charmbracelet/catwalk/pkg/embedded"
@@ -126,8 +125,6 @@ func Providers(cfg *Config) ([]catwalk.Provider, error) {
126125}
127126
128127func loadProviders(autoUpdateDisabled bool, client ProviderClient, path string) ([]catwalk.Provider, error) {
129− _, cacheExists := isCacheStale(path)
130−
131128 catwalkGetAndSave := func() ([]catwalk.Provider, error) {
132129 providers, err := client.GetProviders()
133130 if err != nil {
@@ -141,11 +138,12 @@ func loadProviders(autoUpdateDisabled bool, client ProviderClient, path string)
141138 }
142139 return providers, nil
143140 }
141+
144142 switch {
145143 case autoUpdateDisabled:
146144 slog.Warn("Providers auto-update is disabled")
147145
148− if cacheExists {
146+ if _, err := os.Stat(path); err == nil {
149147 slog.Warn("Using locally cached providers")
150148 return loadProvidersFromCache(path)
151149 }
@@ -168,11 +166,3 @@ func loadProviders(autoUpdateDisabled bool, client ProviderClient, path string)
168166 return providers, nil
169167 }
170168}
171−
172−func isCacheStale(path string) (stale, exists bool) {
173− info, err := os.Stat(path)
174− if err != nil {
175− return true, false
176− }
177− return time.Since(info.ModTime()) > 24*time.Hour, true
178−}