fix: use getFinalMessage() instead of getDeltas() in resumeStream test

The test was flaky because stream-delta events can be batched or arrive
before the collector starts collecting. Using getFinalMessage().content
is more reliable since it's captured from the stream-end event.

Author: ammar-agent

tests/ipcMain/resumeStream.test.ts

@@ -192,26 +192,22 @@ describeIntegration("IpcMain resumeStream integration tests", () => {
           .filter((e) => "role" in e && e.role === "assistant");
         expect(assistantMessages.length).toBeGreaterThan(0);
 
-        // Verify we received content deltas
-        const deltas = collector.getDeltas();
-        expect(deltas.length).toBeGreaterThan(0);
-
         // Verify no stream errors
         const streamErrors = collector
           .getEvents()
           .filter((e) => "type" in e && e.type === "stream-error");
         expect(streamErrors.length).toBe(0);
 
-        // Verify the assistant responded with actual content and said the verification word
-        const allText = deltas
-          .filter((d) => "delta" in d)
-          .map((d) => ("delta" in d ? d.delta : ""))
-          .join("");
-        expect(allText.length).toBeGreaterThan(0);
+        // Get the final message content (more reliable than checking deltas which can be batched)
+        const finalMessage = collector.getFinalMessage();
+        expect(finalMessage).toBeDefined();
+        const finalContent =
+          finalMessage && "content" in finalMessage ? String(finalMessage.content) : "";
+        expect(finalContent.length).toBeGreaterThan(0);
 
         // Verify the assistant followed the instruction and said the verification word
         // This proves resumeStream properly loaded history and continued from it
-        expect(allText).toContain(verificationWord);
+        expect(finalContent).toContain(verificationWord);
       } finally {
         await cleanup();
       }