Add newlines after notifications

Author: benbrandt

src/CodexEventHandler.ts

@@ -283,7 +283,7 @@ export class CodexEventHandler {
             sessionUpdate: "agent_message_chunk",
             content: {
                 type: "text",
-                text,
+                text: `${text}\n\n`,
             },
         };
     }
@@ -315,7 +315,7 @@ export class CodexEventHandler {
             sessionUpdate: "agent_message_chunk",
             content: {
                 type: "text",
-                text: "Goal cleared.",
+                text: "Goal cleared.\n\n",
             },
         };
     }

src/__tests__/CodexACPAgent/data/thread-goal-cleared.json

@@ -7,7 +7,7 @@
         "sessionUpdate": "agent_message_chunk",
         "content": {
           "type": "text",
-          "text": "Goal cleared."
+          "text": "Goal cleared.\n\n"
         }
       }
     }

src/__tests__/CodexACPAgent/data/thread-goal-updated-multiline.json

@@ -7,7 +7,7 @@
         "sessionUpdate": "agent_message_chunk",
         "content": {
           "type": "text",
-          "text": "Goal updated (budget limited):\nFirst task\nSecond task"
+          "text": "Goal updated (budget limited):\nFirst task\nSecond task\n\n"
         }
       }
     }

src/__tests__/CodexACPAgent/data/thread-goal-updated.json

@@ -7,7 +7,7 @@
         "sessionUpdate": "agent_message_chunk",
         "content": {
           "type": "text",
-          "text": "Goal updated (active): Ship the goal update"
+          "text": "Goal updated (active): Ship the goal update\n\n"
         }
       }
     }

src/__tests__/CodexACPAgent/thread-goal-events.test.ts

@@ -123,7 +123,7 @@ describe("CodexEventHandler - thread goal events", () => {
 
         const events = mockFixture.getAcpConnectionEvents([]);
         expect(events).toHaveLength(1);
-        expect(events[0]!.args[0].update.content.text).toBe("Goal updated (active): Ship the goal update");
+        expect(events[0]!.args[0].update.content.text).toBe("Goal updated (active): Ship the goal update\n\n");
     });
 
     it("should suppress duplicate thread goal cleared notifications", async () => {
@@ -141,7 +141,7 @@ describe("CodexEventHandler - thread goal events", () => {
 
         const events = mockFixture.getAcpConnectionEvents([]);
         expect(events).toHaveLength(1);
-        expect(events[0]!.args[0].update.content.text).toBe("Goal cleared.");
+        expect(events[0]!.args[0].update.content.text).toBe("Goal cleared.\n\n");
     });
 
     function createSessionState(): SessionState {