feat: persist status URL across stream boundaries

Introduced lastStatusUrl field to track the most recent URL set via
status_set, ensuring it persists even when agentStatus is cleared by
new user messages.

- URL now persists across stream boundaries until explicitly replaced
- Added TDD test to verify URL persistence after status clear
- All 15 status tests passing

This ensures the URL remains available throughout a conversation, even
when status is cleared between user/assistant turns.

Author: ammar-agent

src/utils/messages/StreamingMessageAggregator.status.test.ts

@@ -674,4 +674,89 @@ describe("StreamingMessageAggregator - Agent Status", () => {
     expect(finalStatus?.message).toBe("New PR");
     expect(finalStatus?.url).toBe(newUrl); // URL replaced
   });
+
+  it("should persist URL even after status is cleared by new stream start", () => {
+    const aggregator = new StreamingMessageAggregator("2024-01-01T00:00:00.000Z");
+    const messageId1 = "msg1";
+
+    // Start first stream
+    aggregator.handleStreamStart({
+      type: "stream-start",
+      workspaceId: "workspace1",
+      messageId: messageId1,
+      model: "test-model",
+      historySequence: 1,
+    });
+
+    // Set status with URL in first stream
+    const testUrl = "https://github.com/owner/repo/pull/123";
+    aggregator.handleToolCallStart({
+      type: "tool-call-start",
+      workspaceId: "workspace1",
+      messageId: messageId1,
+      toolCallId: "tool1",
+      toolName: "status_set",
+      args: { emoji: "🔗", message: "PR submitted", url: testUrl },
+      tokens: 10,
+      timestamp: Date.now(),
+    });
+
+    aggregator.handleToolCallEnd({
+      type: "tool-call-end",
+      workspaceId: "workspace1",
+      messageId: messageId1,
+      toolCallId: "tool1",
+      toolName: "status_set",
+      result: { success: true, emoji: "🔗", message: "PR submitted", url: testUrl },
+    });
+
+    expect(aggregator.getAgentStatus()?.url).toBe(testUrl);
+
+    // User sends a new message, which clears the status
+    const userMessage = {
+      id: "user1",
+      role: "user" as const,
+      parts: [{ type: "text" as const, text: "Continue" }],
+      metadata: { timestamp: Date.now(), historySequence: 2 },
+    };
+    aggregator.handleMessage(userMessage);
+
+    expect(aggregator.getAgentStatus()).toBeUndefined(); // Status cleared
+
+    // Start second stream
+    const messageId2 = "msg2";
+    aggregator.handleStreamStart({
+      type: "stream-start",
+      workspaceId: "workspace1",
+      messageId: messageId2,
+      model: "test-model",
+      historySequence: 2,
+    });
+
+    // Set new status WITHOUT URL - should use the last URL ever seen
+    aggregator.handleToolCallStart({
+      type: "tool-call-start",
+      workspaceId: "workspace1",
+      messageId: messageId2,
+      toolCallId: "tool2",
+      toolName: "status_set",
+      args: { emoji: "✅", message: "Tests passed" },
+      tokens: 10,
+      timestamp: Date.now(),
+    });
+
+    aggregator.handleToolCallEnd({
+      type: "tool-call-end",
+      workspaceId: "workspace1",
+      messageId: messageId2,
+      toolCallId: "tool2",
+      toolName: "status_set",
+      result: { success: true, emoji: "✅", message: "Tests passed" },
+    });
+
+    const finalStatus = aggregator.getAgentStatus();
+    expect(finalStatus?.emoji).toBe("✅");
+    expect(finalStatus?.message).toBe("Tests passed");
+    expect(finalStatus?.url).toBe(testUrl); // URL from previous stream persists!
+  });
 });

src/utils/messages/StreamingMessageAggregator.ts

@@ -73,6 +73,10 @@ export class StreamingMessageAggregator {
   // Unlike todos, this persists after stream completion to show last activity
   private agentStatus: { emoji: string; message: string; url?: string } | undefined = undefined;
 
+  // Last URL set via status_set - persists even when agentStatus is cleared
+  // This ensures URL stays available across stream boundaries
+  private lastStatusUrl: string | undefined = undefined;
+
   // Workspace init hook state (ephemeral, not persisted to history)
   private initState: {
     status: "running" | "success" | "error";
@@ -523,12 +527,17 @@ export class StreamingMessageAggregator {
     // Use output instead of input to get the truncated message
     if (toolName === "status_set" && hasSuccessResult(output)) {
       const result = output as { success: true; emoji: string; message: string; url?: string };
-      // Preserve the previous URL if the new status doesn't provide one
-      const previousUrl = this.agentStatus?.url;
+      
+      // Update lastStatusUrl if a new URL is provided
+      if (result.url) {
+        this.lastStatusUrl = result.url;
+      }
+      
+      // Use the provided URL, or fall back to the last URL ever set
       this.agentStatus = {
         emoji: result.emoji,
         message: result.message,
-        url: result.url ?? previousUrl,
+        url: result.url ?? this.lastStatusUrl,
       };
     }
   }