From e1d5ff58c08522ced454ad4db2024a99714e2ac0 Mon Sep 17 00:00:00 2001
From: jelveh <nj@puter.com>
Date: Mon, 27 Jul 2026 07:50:02 -0700
Subject: [PATCH] Revert "Keep the ai-chat model-map build inside the server
 lifecycle"

This reverts commit 1b2482dcb8d5c1cb9eac077fcd362f086efc1a79.
---
 .../drivers/ai-chat/ChatCompletionDriver.ts        | 14 +-------------
 src/backend/testUtil.ts                            |  3 ---
 2 files changed, 1 insertion(+), 16 deletions(-)

diff --git a/src/backend/drivers/ai-chat/ChatCompletionDriver.ts b/src/backend/drivers/ai-chat/ChatCompletionDriver.ts
index 1c834b7c9..90ebc3f8b 100644
--- a/src/backend/drivers/ai-chat/ChatCompletionDriver.ts
+++ b/src/backend/drivers/ai-chat/ChatCompletionDriver.ts
@@ -216,22 +216,10 @@ export class ChatCompletionDriver extends PuterDriver {
 
     #providers: Record<string, IChatProvider> = {};
     #modelIdMap: Record<string, IChatModel[]> = {};
-    #modelMapReady: Promise<void> = Promise.resolve();
 
     override onServerStart() {
         this.#registerProviders();
-        // Not awaited — providers fetch model lists over the network and
-        // boot must not block on their latency. Shutdown awaits the
-        // tracked promise so provider I/O (and its error logging) never
-        // outlives the server; a rejection here must not become an
-        // unhandled one.
-        this.#modelMapReady = this.#buildModelMap().catch((e) => {
-            console.error('Failed to build ai-chat model map:', e);
-        });
-    }
-
-    override async onServerShutdown() {
-        await this.#modelMapReady;
+        this.#buildModelMap();
     }
 
     // -- Interface methods -------------------------------------------
diff --git a/src/backend/testUtil.ts b/src/backend/testUtil.ts
index 7373e488d..a6cf752f2 100644
--- a/src/backend/testUtil.ts
+++ b/src/backend/testUtil.ts
@@ -153,9 +153,6 @@ export const setupTestServer = async (
             no_devwatch: true,
             no_browser_launch: true,
             import_ts_extensions: true,
-            // No local-Ollama auto-discovery: every test server would
-            // otherwise fire a doomed model-list fetch at localhost.
-            providers: { ollama: { enabled: false } },
         }),
         configOverrides ?? {},
     ) as IConfig;