fix: preview reset race condition, model cache TTL, enhancer auth patterns, JSDoc syntax

Author: zebbern

app/lib/stores/previews.ts

@@ -30,6 +30,7 @@ export class PreviewsStore {
   #REFRESH_DELAY = 300;
   #storageChannel?: BroadcastChannel;
   #disposePortEvents: Disposer | undefined;
+  #resetTimer: ReturnType<typeof setTimeout> | undefined;
 
   previews = atom<PreviewInfo[]>([]);
 
@@ -390,7 +391,8 @@ export class PreviewsStore {
 
     // Defer re-subscription so old runtime teardown completes before we
     // start listening to port events again — prevents stale port capture
-    setTimeout(() => {
+    clearTimeout(this.#resetTimer);
+    this.#resetTimer = setTimeout(() => {
       this.#broadcastChannel = this.#maybeCreateChannel(PREVIEW_CHANNEL);
       this.#storageChannel = this.#maybeCreateChannel('storage-sync-channel');
       this.#setupChannelHandlers();

app/routes/api.enhancer.ts

@@ -113,8 +113,20 @@ async function enhancerAction({ context, request }: ActionFunctionArgs) {
   } catch (error: unknown) {
     logger.error(error);
 
-    if (error instanceof Error && error.message?.includes('API key')) {
-      return errorResponse(new AppError(AppErrorType.UNAUTHORIZED, 'Invalid or missing API key'));
+    if (error instanceof Error) {
+      const msg = error.message.toLowerCase();
+      if (
+        msg.includes('api key') ||
+        msg.includes('api_key') ||
+        msg.includes('unauthorized') ||
+        msg.includes('authentication') ||
+        msg.includes('invalid credentials') ||
+        msg.includes('forbidden') ||
+        msg.includes('access denied') ||
+        msg.includes('401')
+      ) {
+        return errorResponse(new AppError(AppErrorType.UNAUTHORIZED, 'Invalid or missing API key'));
+      }
     }
 
     return errorResponse(error instanceof Error ? error : new AppError(AppErrorType.INTERNAL, 'Internal Server Error'));

app/routes/api.models.ts

@@ -14,12 +14,18 @@ const logger = createScopedLogger('Models');
 
 let cachedProviders: ProviderInfo[] | null = null;
 let cachedDefaultProvider: ProviderInfo | null = null;
+let cachedProvidersTimestamp = 0;
 
 /** Server-side model list cache with short TTL to avoid repeated fetches during startup */
 let cachedModelResponse: { models: ModelInfo[]; timestamp: number } | null = null;
 const MODEL_CACHE_TTL_MS = 30_000; // 30 seconds
 
 function getProviderInfo(llmManager: LLMManager) {
+  if (cachedProviders && Date.now() - cachedProvidersTimestamp > MODEL_CACHE_TTL_MS) {
+    cachedProviders = null;
+    cachedDefaultProvider = null;
+  }
+
   if (!cachedProviders) {
     cachedProviders = llmManager.getAllProviders().map((provider) => ({
       name: provider.name,
@@ -28,6 +34,7 @@ function getProviderInfo(llmManager: LLMManager) {
       labelForGetApiKey: provider.labelForGetApiKey,
       icon: provider.icon,
     }));
+    cachedProvidersTimestamp = Date.now();
   }
 
   if (!cachedDefaultProvider) {

app/utils/previewErrorHandler.ts

@@ -188,7 +188,6 @@ class PreviewErrorHandler {
     logger.info(`Preview error detected [${severity}]: ${title} - ${errorMessage.slice(0, 100)}`);
   }
 
-  /**
   /**
    * Reset the handler state
    * Call this when user clicks "Ask Devonz" so the same error can be caught again