test(web): stop asserting ci scheduling latency in the ws broadcast test

tests/test_web_e2e.py

@@ -116,8 +116,12 @@ def test_reject_without_reason_is_refused(client: TestClient, store: KBStore) ->
 
 def test_websocket_broadcasts_on_approve(client: TestClient, store: KBStore) -> None:
     """A second reviewer's socket receives a refresh frame within the same
-    request that performed the approve — the <1s sync criterion. We assert the
-    frame arrives well within the budget (it's normally single-digit ms)."""
+    request that performed the approve — the #194 sync criterion.
+
+    The load-bearing assertion is that the frame *arrives* at all, and carries
+    the right view and proposal id; the elapsed-time bound below only guards
+    against a stall.
+    """
     pid = _seed(store, "broadcast me")
     with client.websocket_connect("/ws") as ws:
         hello = ws.receive_json()
@@ -130,10 +134,16 @@ def test_websocket_broadcasts_on_approve(client: TestClient, store: KBStore) ->
         assert frame["type"] == "refresh"
         assert frame["view"] == "queue"
         assert frame["proposal_id"] == pid
-        # #194 criterion: two windows sync within 1s. Generous bound — the real
-        # number is milliseconds — but it fails loudly if a regression made the
-        # broadcast block (e.g. a slow-client stall in _Hub.broadcast).
-        assert elapsed < 1.0, f"broadcast took {elapsed:.3f}s, must be <1s"
+        # What this bound is for: catching a broadcast that *stalls* — a
+        # slow-client hang in _Hub.broadcast, which shows up as seconds or a
+        # timeout, not as a near-miss. It is deliberately not a measurement of
+        # #194's 1s sync criterion. The real number here is single-digit
+        # milliseconds, so the criterion has three orders of magnitude of
+        # headroom; what a 1.0s assertion actually measured was whether a
+        # shared CI runner descheduled this thread, which it can do for well
+        # over a second with nothing wrong in the code (observed: 1.479s on
+        # py3.12 while 3.11 and 3.13 passed the same commit).
+        assert elapsed < 10.0, f"broadcast took {elapsed:.3f}s — stalled"
 
 
 def test_healthz_reports_clients_and_auth(client: TestClient) -> None: