🤖 Fix integration test race condition in AI SDK provider imports

Restore preloadAISDKProviders() functionality to eagerly load @ai-sdk/anthropic
and @ai-sdk/openai modules before tests run concurrently. This prevents Jest's
module sandboxing from blocking dynamic imports with 'You are trying to import
a file outside of the scope of the test code' errors.

The function was previously converted to a no-op under the assumption that
'Jest concurrency has been stabilized elsewhere', but tests still failed when
run together. Preloading ensures providers are in the module cache before
concurrent tests access them via dynamic imports in createModel().

Also add note to AGENTS.md warning that tests/ipcMain takes a long time locally.

Author: ammar-agent

docs/AGENTS.md

@@ -205,6 +205,7 @@ This project uses **Make** as the primary build orchestrator. See `Makefile` for
 - **Integration tests:**
   - Run specific integration test: `TEST_INTEGRATION=1 bun x jest tests/ipcMain/sendMessage.test.ts -t "test name pattern"`
   - Run all integration tests: `TEST_INTEGRATION=1 bun x jest tests` (~35 seconds, runs 40 tests)
+  - **⚠️ Running `tests/ipcMain` locally takes a very long time.** Prefer running specific test files or use `-t` to filter to specific tests.
   - **Performance**: Tests use `test.concurrent()` to run in parallel within each file
   - **NEVER bypass IPC in integration tests** - Integration tests must use the real IPC communication paths (e.g., `mockIpcRenderer.invoke()`) even when it's harder. Directly accessing services (HistoryService, PartialService, etc.) or manipulating config/state directly bypasses the integration layer and defeats the purpose of the test.
 

src/services/aiService.ts

@@ -97,12 +97,9 @@ if (typeof globalFetchWithExtras.certificate === "function") {
  * In production, providers are lazy-loaded on first use to optimize startup time.
  * In tests, we preload them once during setup to ensure reliable concurrent execution.
  */
-// eslint-disable-next-line @typescript-eslint/require-await
 export async function preloadAISDKProviders(): Promise<void> {
-  // No-op: Providers are lazy-loaded in createModel().
-  // Preloading was previously used to avoid race conditions in concurrent tests,
-  // but Jest concurrency has been stabilized elsewhere and this is no longer necessary.
-  return;
+  // Preload providers to ensure they're in the module cache before concurrent tests run
+  await Promise.all([import("@ai-sdk/anthropic"), import("@ai-sdk/openai")]);
 }
 
 export class AIService extends EventEmitter {

tests/ipcMain/renameWorkspace.test.ts

@@ -20,12 +20,16 @@ if (shouldRunIntegrationTests()) {
 }
 
 describeIntegration("IpcMain rename workspace integration tests", () => {
-  // Load tokenizer modules once before all tests (takes ~14s)
-  // This ensures accurate token counts for API calls without timing out individual tests
+  // Load tokenizer modules and AI SDK providers once before all tests
+  // This ensures accurate token counts for API calls and prevents race conditions
+  // when tests import providers concurrently
   beforeAll(async () => {
-    const { loadTokenizerModules } = await import("../../src/utils/main/tokenizer");
-    await loadTokenizerModules();
-  }, 30000); // 30s timeout for tokenizer loading
+    const [{ loadTokenizerModules }, { preloadAISDKProviders }] = await Promise.all([
+      import("../../src/utils/main/tokenizer"),
+      import("../../src/services/aiService"),
+    ]);
+    await Promise.all([loadTokenizerModules(), preloadAISDKProviders()]);
+  }, 30000); // 30s timeout for tokenizer and provider loading
 
   test.concurrent(
     "should successfully rename workspace and update all paths",
@@ -300,6 +304,9 @@ describeIntegration("IpcMain rename workspace integration tests", () => {
         // Send a message to create some history
         env.sentEvents.length = 0;
         const result = await sendMessageWithModel(env.mockIpcRenderer, workspaceId, "What is 2+2?");
+        if (!result.success) {
+          console.error("Send message failed:", result.error);
+        }
         expect(result.success).toBe(true);
 
         // Wait for response
@@ -370,6 +377,9 @@ describeIntegration("IpcMain rename workspace integration tests", () => {
           "anthropic",
           "claude-sonnet-4-5"
         );
+        if (!result.success) {
+          console.error("Send message failed:", result.error);
+        }
         expect(result.success).toBe(true);
 
         // Wait for response