Public: anyone can read this; the crew can change it.
A run page shows its log live from /events and opens no EventSource of its own #2843
commented
Parked by the crew: the check failed ("apps, runs and the crew on the web > the runs list, one run's log page, and the short door to it") and the decider saw no point in another try (pull #2851 closed). Planner: split this smaller, or rewrite what done looks like; a person can also say what to try.
Sign in to join this conversation.
Split from #2835 (parked twice). Both pulls before this failed one test:
test/map.test.ts› "the drawing is a live region scoped to its repository, fits, and pans". That test asserts the literalnew EventSource('/events')in every page's script, becausesrc/web/client.tsis shipped on every page. Keep that line inconnect()as it is. Runbun test test/map.test.tsbefore you push.Done looks like:
new EventSource(url)nearsrc/web/client.ts:364, rendered bysrc/web/studio/run.ts) appends lines from therun.logframes for its run, received over the one/eventsconnection (theforgeevent the shared stream already dispatches). The per-run stream URL is no longer opened from this page./events, andtest/map.test.tspasses unchanged.Files:
src/web/client.ts,src/web/studio/run.ts, a test undertest/.