Fix E2E tests for stderr status messages and NDJSON output

- Check stderr (not stdout) for status messages like "Entered presence",
  "Location set in space:", and "Waiting for"
- Parse NDJSON correctly: find type "result" line for history, first line
  for list (completed status signal is appended as last line)

Author: umair-ably

test/e2e/channels/channel-presence-e2e.test.ts

@@ -82,11 +82,12 @@ describe("Channel Presence E2E Tests", () => {
       },
     );
 
-    console.log(`Presence enter output: ${enterResult.stdout}`);
+    console.log(`Presence enter stdout: ${enterResult.stdout}`);
+    console.log(`Presence enter stderr: ${enterResult.stderr}`);
     expect(enterResult.exitCode).toBe(0);
-    expect(enterResult.stdout).toContain("Entered presence on channel");
-    expect(enterResult.stdout).toContain(
-      "Duration elapsed – command finished cleanly",
-    );
+    // Status messages go to stderr
+    const allOutput = enterResult.stdout + enterResult.stderr;
+    expect(allOutput).toContain("Entered presence on channel");
+    expect(allOutput).toContain("Duration elapsed – command finished cleanly");
   });
 });

test/e2e/channels/channels-e2e.test.ts

@@ -181,7 +181,10 @@ describe("Channel E2E Tests", () => {
 
     let result;
     try {
-      result = JSON.parse(listResult.stdout);
+      // JSON output may contain multiple NDJSON lines (result + completed status).
+      // Parse the first non-empty line which contains the result data.
+      const line = listResult.stdout.trim().split("\n").find(Boolean);
+      result = JSON.parse(line);
     } catch (parseError) {
       throw new Error(
         `Failed to parse JSON output. Parse error: ${String(parseError)}. Exit code: ${listResult.exitCode}, Stderr: ${listResult.stderr}, Stdout: ${listResult.stdout}`,
@@ -339,10 +342,13 @@ describe("Channel E2E Tests", () => {
 
     let result;
     try {
-      // The --json output is NDJSON (one event line + one result line).
-      // Parse the last non-empty line which contains the result.
+      // The --json output is NDJSON: event lines, then a result line, then a completed status.
+      // Find the line with type "result" which contains the history data.
       const lines = historyResult.stdout.trim().split("\n").filter(Boolean);
-      result = JSON.parse(lines.at(-1));
+      const resultLine = lines
+        .map((l) => JSON.parse(l))
+        .find((obj) => obj.type === "result");
+      result = resultLine;
     } catch (parseError) {
       throw new Error(
         `Failed to parse JSON history output. Parse error: ${String(parseError)}. Exit code: ${historyResult.exitCode}, Stderr: ${historyResult.stderr}, Stdout: ${historyResult.stdout}`,

test/e2e/interactive/ctrl-c-behavior.test.ts

@@ -357,10 +357,10 @@ describe("E2E: Interactive Mode - Ctrl+C Behavior", () => {
           }, 100);
         });
 
-        // Wait for command to start
+        // Wait for command to start (status messages go to stderr)
         await new Promise<void>((resolve) => {
           const checkForWaiting = setInterval(() => {
-            if (output.includes("Waiting for")) {
+            if (errorOutput.includes("Waiting for")) {
               clearInterval(checkForWaiting);
               resolve();
             }
@@ -435,10 +435,10 @@ describe("E2E: Interactive Mode - Ctrl+C Behavior", () => {
           }, 100);
         });
 
-        // Wait for command to start
+        // Wait for command to start (status messages go to stderr)
         await new Promise<void>((resolve) => {
           const checkInterval = setInterval(() => {
-            if (output.includes("Waiting for")) {
+            if (errorOutput.includes("Waiting for")) {
               clearInterval(checkInterval);
               resolve();
             }

test/e2e/spaces/spaces-e2e.test.ts

@@ -217,11 +217,13 @@ describe("Spaces E2E Tests", () => {
           );
 
           // Check for success - either exit code 0 or successful output (even if process was killed after success)
+          // Status messages go to stderr
+          const allOutput = setLocationResult.stdout + setLocationResult.stderr;
           const isLocationSetSuccessful =
             setLocationResult.exitCode === 0 ||
-            setLocationResult.stdout.includes("Location set in space:");
+            allOutput.includes("Location set in space:");
           expect(isLocationSetSuccessful).toBe(true);
-          expect(setLocationResult.stdout).toContain("Location set in space:");
+          expect(allOutput).toContain("Location set in space:");
 
           // Wait for location update to be received by client1
           let locationUpdateReceived = false;