stacklok / toolhive-studio / 26939594552

71%

main: 71%

Build Type

Pull #2304

github

Committed by peppescg

Commit Message

fix(startup): eliminate cold-start window flash by awaiting socket readiness

Three coupled changes that together remove the dark-window flash users see
between launching the app and the renderer becoming interactive.

1. `startToolhive()` now actually awaits a real socket connection (via
   `net.connect()` polling, cross-platform for UNIX sockets and Windows
   named pipes) before resolving. Previously the function had a missing
   `return` on its `Sentry.withScope` wrapper and only awaited `spawn()`,
   which returns synchronously — so the renderer could fire its first
   `getHealth` request while thv was still binding the socket, producing
   an ENOENT in main logs and a retry-driven black flash on cold start.

2. `BrowserWindow` is now constructed with a `backgroundColor` that mirrors
   the Tailwind theme `--background` token (`#f9faf9` light / `#2e2e2e`
   dark, sampled via `nativeTheme.shouldUseDarkColors`). Without this the
   OS shows a default white/black window for the brief gap between window
   creation and the first paint of `index.html`.

3. `renderer/index.html` now contains a static splash painted from the
   first parse of the HTML, before the Vite bundle is fetched and React
   mounts. Colors mirror the Tailwind theme so the transition into the
   first React paint is seamless. `createRoot().render()` replaces the
   splash contents on first commit.

Pull Request Pull Request #2304: fix(startup): eliminate cold-start window flash by awaiting socket readiness

Coverage Stats

5213 of 8012 branches covered (65.06%)

22 of 28 new or added lines in 2 files covered. (78.57%)

5 existing lines in 1 file now uncovered.

7731 of 10962 relevant lines covered (70.53%)

115.23 hits per line