Fix two CI flakes surfaced by GitHub Windows runners

1. test_watcher_reloads_after_mtime_change: the test's `_write` helper
   set mtime to time.time() after each write, but two back-to-back
   writes of the same file on the GitHub-runner Windows filesystem
   could land with identical mtimes — defeating the watcher's
   mtime-based reload detection. Force mtime forward past the
   previous value: `now = max(time.time(), previous + 1.0)`.

2. test_ws_viewer_input_reaches_host_dispatcher (and the related
   _authenticates_and_receives_frames / _wrong_token tests): the
   socket-level auth-handshake timeouts were hardcoded at 5 s on
   both the viewer (`_DEFAULT_AUTH_TIMEOUT_S`) and host
   (`_AUTH_TIMEOUT_S` in host.py + `_HANDSHAKE_TIMEOUT_S` in
   ws_host.py). On a slow GitHub Windows runner the handshake
   recvs exceeded that budget — even though the test asked for a
   10 s connect timeout, the per-socket timeout never honored it.
   Bump all three to 15 s, AND make viewer.py honor the caller's
   explicit `connect(timeout=...)` argument:
   `raw_sock.settimeout(max(_DEFAULT_AUTH_TIMEOUT_S, float(timeout)))`.

Both are real defects that would also bite production users on
high-latency / loaded networks; the test failures just gave them
visibility on CI.

Verified: full headless suite 605 passed / 7 skipped / 0 failed
locally after the change.

Author: JE-Chen

je_auto_control/utils/remote_desktop/host.py

@@ -37,7 +37,7 @@
 FrameProvider = Callable[[], bytes]
 InputDispatcher = Callable[[Mapping[str, Any]], Any]
 
-_AUTH_TIMEOUT_S = 5.0
+_AUTH_TIMEOUT_S = 15.0
 _DEFAULT_QUALITY = 70
 
 

je_auto_control/utils/remote_desktop/viewer.py

@@ -26,7 +26,7 @@
 ClipboardCallback = Callable[[str, Any], None]
 ErrorCallback = Callable[[Exception], None]
 
-_DEFAULT_AUTH_TIMEOUT_S = 5.0
+_DEFAULT_AUTH_TIMEOUT_S = 15.0
 _DEFAULT_CONNECT_TIMEOUT_S = 5.0
 _NOT_CONNECTED_MESSAGE = "viewer is not connected"
 
@@ -103,7 +103,11 @@ def connect(self, timeout: float = _DEFAULT_CONNECT_TIMEOUT_S) -> None:
         raw_sock = socket.create_connection(
             (self._host, self._port), timeout=timeout,
         )
-        raw_sock.settimeout(_DEFAULT_AUTH_TIMEOUT_S)
+        # If the caller explicitly asked for a longer connect budget,
+        # honor it for the handshake too — otherwise a slow remote (CI
+        # runners, high-latency links) trips the 5 s default before the
+        # caller's window expires.
+        raw_sock.settimeout(max(_DEFAULT_AUTH_TIMEOUT_S, float(timeout)))
         try:
             sock = self._maybe_wrap_tls(raw_sock)
             channel = self._build_channel(sock)

je_auto_control/utils/remote_desktop/ws_host.py

@@ -17,7 +17,7 @@
     WsProtocolError, server_handshake,
 )
 
-_HANDSHAKE_TIMEOUT_S = 5.0
+_HANDSHAKE_TIMEOUT_S = 15.0
 
 
 class WebSocketDesktopHost(RemoteDesktopHost):

test/unit_test/headless/test_mcp_plugin_watcher.py

@@ -6,10 +6,15 @@
 
 
 def _write(path, body):
-    path.write_text(body, encoding="utf-8")
-    # Bump mtime to ensure the watcher picks it up even on coarse FSes.
-    now = time.time()
     import os
+    # On Windows + GitHub-runner filesystems, mtime resolution can be
+    # coarser than back-to-back test writes — the second write of the
+    # same file can land with the same mtime as the first, defeating
+    # mtime-based reload detection. Always force mtime forward past
+    # any previous value on this path.
+    previous = path.stat().st_mtime if path.exists() else 0.0
+    path.write_text(body, encoding="utf-8")
+    now = max(time.time(), previous + 1.0)
     os.utime(path, (now, now))