🤖 fix: resolve flaky integration tests in bash runtime and stream resumption (#729)

Generated with `mux`

Fixes two independent flakes:

### 1. `tests/ipcMain/runtimeExecuteBash.test.ts`
**Issue:** The test was flaky because it expected the AI model to
explicitly describe the output in its text response, which is
non-deterministic.
**Fix:** Prioritized verifying the actual tool output (which is
deterministic and proves the runtime works) and made the text response
check optional.

### 2. `tests/ipcMain/resumeStream.test.ts`
**Issue:** The test relied on a `setTimeout(100)` to wait for IPC
subscription, which caused race conditions in CI/loaded environments.
**Fix:** Replaced the sleep with `collector.waitForEvent('caught-up')`,
which is the correct synchronization signal from the backend.

Author: ammar-agent

tests/ipcMain/createWorkspace.test.ts

@@ -818,7 +818,8 @@ exit 1
 
         try {
           // Set up a real origin remote in the test repo
-          const originUrl = "https://github.com/example/test-repo.git";
+          // Use example.com to avoid global git config rewrites (e.g. insteadOf https://github.com/)
+          const originUrl = "https://example.com/example/test-repo.git";
           await execAsync(`git remote add origin ${originUrl}`, {
             cwd: tempGitRepo,
           });

tests/ipcMain/resumeStream.test.ts

@@ -161,8 +161,9 @@ describeIntegration("IpcMain resumeStream integration tests", () => {
         // Subscribe to chat channel to receive events
         env.mockIpcRenderer.send("workspace:chat:subscribe", workspaceId);
 
-        // Wait a moment for subscription to complete
-        await new Promise((resolve) => setTimeout(resolve, 100));
+        // Wait for subscription to complete by waiting for caught-up event
+        const caughtUpEvent = await collector.waitForEvent("caught-up", 5000);
+        expect(caughtUpEvent).toBeDefined();
 
         // Resume the stream (should continue from the summary message)
         const resumeResult = (await env.mockIpcRenderer.invoke(

tests/ipcMain/runtimeExecuteBash.test.ts

@@ -296,10 +296,16 @@ describeIntegration("Runtime Bash Execution", () => {
               const responseText = extractTextFromEvents(events);
 
               // Verify command completed successfully (not timeout)
-              expect(responseText).toContain("test");
+              // We primarily check bashOutput to ensure the tool executed and didn't hang
               const bashOutput = collectToolOutputs(events, "bash");
               expect(bashOutput).toContain('"test": "data"');
 
+              // responseText might be empty if the model decides not to comment on the output
+              // so we make this check optional or less strict if the tool output is correct
+              if (responseText) {
+                expect(responseText).toContain("test");
+              }
+
               // Verify command completed quickly (not hanging until timeout)
               expect(toolDuration).toBeGreaterThan(0);
               const maxDuration = 10000;