From afc05f85b73b32d9c4896cccf49aa522607ef958 Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Tue, 12 May 2026 04:45:42 +0100
Subject: [PATCH] test: guard message hook mock calls

---
 src/hooks/message-hooks.test.ts | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/src/hooks/message-hooks.test.ts b/src/hooks/message-hooks.test.ts
index 6a1e35f1edd..5cbf46e1486 100644
--- a/src/hooks/message-hooks.test.ts
+++ b/src/hooks/message-hooks.test.ts
@@ -106,6 +106,14 @@ const actionCases: ActionCase[] = [
   },
 ];
 
+function requireHookEvent(handler: ReturnType<typeof vi.fn>): InternalHookEvent {
+  const call = handler.mock.calls.at(0);
+  if (!call) {
+    throw new Error("expected hook handler call");
+  }
+  return call[0] as InternalHookEvent;
+}
+
 describe("message hooks", () => {
   beforeEach(() => {
     clearInternalHooks();
@@ -126,7 +134,7 @@ describe("message hooks", () => {
         );
 
         expect(handler).toHaveBeenCalledOnce();
-        const event = handler.mock.calls[0][0] as InternalHookEvent;
+        const event = requireHookEvent(handler);
         expect(event.type).toBe("message");
         expect(event.action).toBe(testCase.action);
         testCase.assertContext(event.context);
@@ -244,7 +252,7 @@ describe("message hooks", () => {
       );
       const after = new Date();
 
-      const event = handler.mock.calls[0][0] as InternalHookEvent;
+      const event = requireHookEvent(handler);
       expect(event.timestamp).toBeInstanceOf(Date);
       expect(event.timestamp.getTime()).toBeGreaterThanOrEqual(before.getTime());
       expect(event.timestamp.getTime()).toBeLessThanOrEqual(after.getTime());