fix: await refreshModels() to ensure the cache is updated before returning from flushModels()

pdecat · pdecat · commit 5da3bb708254 · 2025-12-18T18:17:38.000+01:00
diff --git a/src/api/providers/fetchers/modelCache.ts b/src/api/providers/fetchers/modelCache.ts
@@ -276,9 +276,8 @@ export const flushModels = async (options: GetModelsOptions, refresh: boolean =
 		// Don't delete memory cache - let refreshModels atomically replace it
 		// This prevents a race condition where getModels() might be called
 		// before refresh completes, avoiding a gap in cache availability
-		refreshModels(options).catch((error) => {
-			console.error(`[flushModels] Refresh failed for ${provider}:`, error)
-		})
+		// Await the refresh to ensure the cache is updated before returning
+		await refreshModels(options)
 	} else {
 		// Only delete memory cache when not refreshing
 		memoryCache.del(provider)
