From 9b6cc4085fd0701fbc53126956baab8580ce1992 Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Tue, 12 May 2026 00:50:42 +0100
Subject: [PATCH] test: guard signal autostart mock calls

---
 .../src/monitor.tool-result.autostart.test.ts | 19 ++++++++++++++-----
 1 file changed, 14 insertions(+), 5 deletions(-)

diff --git a/extensions/signal/src/monitor.tool-result.autostart.test.ts b/extensions/signal/src/monitor.tool-result.autostart.test.ts
index a17b77cd6bdc..c0e3b709761c 100644
--- a/extensions/signal/src/monitor.tool-result.autostart.test.ts
+++ b/extensions/signal/src/monitor.tool-result.autostart.test.ts
@@ -52,9 +52,21 @@ async function runMonitorWithMocks(opts: MonitorSignalProviderOptions) {
   });
 }
 
+function requireWaitForTransportReadyOptions(): Record<string, unknown> {
+  const [call] = waitForTransportReadyMock.mock.calls;
+  if (!call) {
+    throw new Error("expected waitForTransportReady call");
+  }
+  const [options] = call;
+  if (!options || typeof options !== "object" || Array.isArray(options)) {
+    throw new Error("expected waitForTransportReady options");
+  }
+  return options as Record<string, unknown>;
+}
+
 function expectWaitForTransportReadyTimeout(timeoutMs: number) {
   expect(waitForTransportReadyMock).toHaveBeenCalledTimes(1);
-  const options = waitForTransportReadyMock.mock.calls[0]?.[0] as { timeoutMs?: number };
+  const options = requireWaitForTransportReadyOptions();
   expect(options.timeoutMs).toBe(timeoutMs);
 }
 
@@ -71,10 +83,7 @@ describe("monitorSignalProvider autostart", () => {
     });
 
     expect(waitForTransportReadyMock).toHaveBeenCalledTimes(1);
-    const options = waitForTransportReadyMock.mock.calls[0]?.[0] as {
-      abortSignal?: unknown;
-      check?: unknown;
-    } & Record<string, unknown>;
+    const options = requireWaitForTransportReadyOptions();
     expect(options).toEqual({
       label: "signal daemon",
       timeoutMs: 30_000,