Skip to content

Commit 8a5560f

Browse files
committed
fix: await refreshModels() to ensure the cache is updated before returning from flushModels()
1 parent 0d40375 commit 8a5560f

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

src/api/providers/fetchers/modelCache.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -281,9 +281,8 @@ export const flushModels = async (options: GetModelsOptions, refresh: boolean =
281281
// Don't delete memory cache - let refreshModels atomically replace it
282282
// This prevents a race condition where getModels() might be called
283283
// before refresh completes, avoiding a gap in cache availability
284-
refreshModels(options).catch((error) => {
285-
console.error(`[flushModels] Refresh failed for ${provider}:`, error)
286-
})
284+
// Await the refresh to ensure the cache is updated before returning
285+
await refreshModels(options)
287286
} else {
288287
// Only delete memory cache when not refreshing
289288
memoryCache.del(provider)

0 commit comments

Comments
 (0)