fix: persist URL across status_set calls until explicitly replaced

Previously, calling status_set without a URL would clear the stored URL.
Now, the URL persists until explicitly replaced with a different URL.

- Updated StreamingMessageAggregator to preserve previous URL
- Updated test to verify URL persistence behavior
- Clarified tool description

Author: ammar-agent

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

@@ -584,7 +584,7 @@ describe("StreamingMessageAggregator - Agent Status", () => {
     expect(status?.url).toBe(testUrl);
   });
 
-  it("should persist URL until replaced with new status", () => {
+  it("should persist URL across status updates until explicitly replaced", () => {
     const aggregator = new StreamingMessageAggregator("2024-01-01T00:00:00.000Z");
     const messageId = "msg1";
 
@@ -621,7 +621,7 @@ describe("StreamingMessageAggregator - Agent Status", () => {
 
     expect(aggregator.getAgentStatus()?.url).toBe(testUrl);
 
-    // Second status without URL - should clear URL
+    // Second status without URL - should keep previous URL
     aggregator.handleToolCallStart({
       type: "tool-call-start",
       workspaceId: "workspace1",
@@ -642,9 +642,36 @@ describe("StreamingMessageAggregator - Agent Status", () => {
       result: { success: true, emoji: "✅", message: "Done" },
     });
 
+    const statusAfterUpdate = aggregator.getAgentStatus();
+    expect(statusAfterUpdate?.emoji).toBe("✅");
+    expect(statusAfterUpdate?.message).toBe("Done");
+    expect(statusAfterUpdate?.url).toBe(testUrl); // URL persists
+
+    // Third status with different URL - should replace
+    const newUrl = "https://github.com/owner/repo/pull/456";
+    aggregator.handleToolCallStart({
+      type: "tool-call-start",
+      workspaceId: "workspace1",
+      messageId,
+      toolCallId: "tool3",
+      toolName: "status_set",
+      args: { emoji: "🔄", message: "New PR", url: newUrl },
+      tokens: 10,
+      timestamp: Date.now(),
+    });
+
+    aggregator.handleToolCallEnd({
+      type: "tool-call-end",
+      workspaceId: "workspace1",
+      messageId,
+      toolCallId: "tool3",
+      toolName: "status_set",
+      result: { success: true, emoji: "🔄", message: "New PR", url: newUrl },
+    });
+
     const finalStatus = aggregator.getAgentStatus();
-    expect(finalStatus?.emoji).toBe("✅");
-    expect(finalStatus?.message).toBe("Done");
-    expect(finalStatus?.url).toBeUndefined();
+    expect(finalStatus?.emoji).toBe("🔄");
+    expect(finalStatus?.message).toBe("New PR");
+    expect(finalStatus?.url).toBe(newUrl); // URL replaced
   });
 });

src/utils/messages/StreamingMessageAggregator.ts

@@ -523,10 +523,12 @@ 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;
       this.agentStatus = {
         emoji: result.emoji,
         message: result.message,
-        ...(result.url && { url: result.url }),
+        url: result.url ?? previousUrl,
       };
     }
   }

src/utils/tools/toolDefinitions.ts

@@ -197,7 +197,7 @@ export const TOOL_DEFINITIONS = {
       "URL PARAMETER:\n" +
       "- Optional 'url' parameter links to external resources (e.g., PR URL: 'https://github.com/owner/repo/pull/123')\n" +
       "- Prefer stable URLs that don't change often - saving the same URL twice is a no-op\n" +
-      "- URL persists until replaced by a new status",
+      "- URL persists until replaced by a new status with a different URL",
     schema: z
       .object({
         emoji: z.string().describe("A single emoji character representing the current activity"),