fix: stabilize bun tests for timers and module mocks

Change-Id: Ib691f6831627e0e03ecfb26339a6bd9b4a4c310c
Signed-off-by: Thomas Kosiewski <tk@coder.com>

# Conflicts:
#	src/browser/utils/RefreshController.test.ts

Author: ThomasK33

src/browser/contexts/API.test.tsx

@@ -66,6 +66,7 @@ void mock.module("@orpc/client/message-port", () => ({
 }));
 
 void mock.module("@/browser/components/AuthTokenModal", () => ({
+  AuthTokenModal: () => null,
   getStoredAuthToken: () => null,
   // eslint-disable-next-line @typescript-eslint/no-empty-function
   clearStoredAuthToken: () => {},

src/browser/hooks/useVoiceInput.ts

@@ -57,7 +57,8 @@ export interface UseVoiceInputResult {
  */
 function hasTouchDictation(): boolean {
   if (typeof window === "undefined") return false;
-  const hasTouch = "ontouchstart" in window || navigator.maxTouchPoints > 0;
+  const hasTouch =
+    "ontouchstart" in window || (typeof navigator !== "undefined" && navigator.maxTouchPoints > 0);
   // Touch-only check: most touch devices have native dictation.
   // We don't check screen size because iPads are large but still have dictation.
   return hasTouch;
@@ -66,7 +67,9 @@ function hasTouchDictation(): boolean {
 const HAS_TOUCH_DICTATION = hasTouchDictation();
 const HAS_MEDIA_RECORDER = typeof window !== "undefined" && typeof MediaRecorder !== "undefined";
 const HAS_GET_USER_MEDIA =
-  typeof window !== "undefined" && typeof navigator.mediaDevices?.getUserMedia === "function";
+  typeof window !== "undefined" &&
+  typeof navigator !== "undefined" &&
+  typeof navigator.mediaDevices?.getUserMedia === "function";
 
 // =============================================================================
 // Global Key State Tracking
@@ -79,7 +82,7 @@ const HAS_GET_USER_MEDIA =
  */
 let isSpaceCurrentlyHeld = false;
 
-if (typeof window !== "undefined") {
+if (typeof window !== "undefined" && typeof window.addEventListener === "function") {
   window.addEventListener(
     "keydown",
     (e) => {

src/browser/utils/RefreshController.test.ts

@@ -1,4 +1,4 @@
-import { describe, it, expect, beforeEach, afterEach, jest } from "@jest/globals";
+import { describe, it, expect, beforeEach, afterEach, jest } from "bun:test";
 import { RefreshController } from "./RefreshController";
 
 describe("RefreshController", () => {

src/node/services/tools/task.test.ts

@@ -75,6 +75,7 @@ describe("task tool", () => {
       taskId: "child-task",
       reportMarkdown: "Hello from child",
       title: "Result",
+      agentId: "explore",
       agentType: "explore",
     });
   });