chore: capture exit code in tests

Co-authored-by: fuxingloh <4266087+fuxingloh@users.noreply.github.com>

Author: Copilot

packages/use-agently/src/bin.ts

@@ -11,8 +11,8 @@ installTelemetry(cli);
 function resolveOutputFormat(): "tui" | "json" {
   try {
     return getOutputFormat(cli);
-  } catch {
-    // If option parsing fails, still show the underlying CLI error without double-logging here.
+  } catch (err) {
+    console.warn("Falling back to TUI output after failing to resolve --output option.", err);
     return "tui";
   }
 }

packages/use-agently/src/errors.test.ts

@@ -5,10 +5,13 @@ describe("handleCliError", () => {
   let exitSpy: ReturnType<typeof spyOn>;
   let errorSpy: ReturnType<typeof spyOn>;
   let originalIsTTY: boolean | undefined;
+  let lastExitCode: number | undefined;
 
   beforeEach(() => {
     originalIsTTY = process.stderr.isTTY;
+    lastExitCode = undefined;
     exitSpy = spyOn(process, "exit").mockImplementation((code?: number) => {
+      lastExitCode = code;
       // Bubble a sentinel error so tests can assert the exit while preventing the process from terminating.
       throw new Error("process.exit");
     });
@@ -24,6 +27,7 @@ describe("handleCliError", () => {
   test("prints json error when format is json", () => {
     expect(() => handleCliError(new Error("boom"), "json")).toThrow("process.exit");
     expect(exitSpy).toHaveBeenCalledWith(1);
+    expect(lastExitCode).toBe(1);
     expect(errorSpy).toHaveBeenCalledWith(JSON.stringify({ error: { message: "boom" } }));
   });
 
@@ -32,6 +36,7 @@ describe("handleCliError", () => {
 
     expect(() => handleCliError(new Error("oops"), "tui")).toThrow("process.exit");
     expect(exitSpy).toHaveBeenCalledWith(1);
+    expect(lastExitCode).toBe(1);
     expect(errorSpy).toHaveBeenCalledWith("Error: oops");
   });
 
@@ -40,6 +45,7 @@ describe("handleCliError", () => {
 
     expect(() => handleCliError(new Error("boxed"), "tui")).toThrow("process.exit");
     expect(exitSpy).toHaveBeenCalledWith(1);
+    expect(lastExitCode).toBe(1);
     const output = errorSpy.mock.calls.map((call) => call[0]).join("");
     expect(output).toContain("boxed");
     expect(output).toContain("Error");

packages/use-agently/src/errors.ts

@@ -3,7 +3,7 @@ import type { OutputFormat } from "./output.js";
 
 function isProcessExitError(err: unknown): err is Error {
   // Commander calls process.exit() internally; in tests we mock it to throw an error
-  // with this message so the test runner can intercept the exit.
+  // with this message so the test runner can intercept the exit without double-formatting.
   return err instanceof Error && err.message === "process.exit";
 }