Warning: This build has drifted.
The coverage report for this pull request build may be inaccurate because its base commit is no longer the HEAD of its target branch.
This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

    • Learn more: For more information on this, see Tracking coverage changes for pull request builds.
    • Fix now: For a quick fix, rebase this PR at GitHub. Your next report should be accurate.
    • Prevent going forward: To avoid this issue with future PRs, see these Recommended CI Configurations.

fix(startup): address Copilot review feedback

- `toolhive-manager.ts`: capture `toolhiveSocketPath` into a local const
  before the readiness probe runs (it can be cleared by `child.on('exit')`
  on fast-fail). Unref the `setTimeout` timers in `probeSocket` and the
  `waitForSocketReady` polling loop so they don't keep the Electron main
  event loop alive during shutdown. Export both functions so they can be
  tested independently.

- `toolhive-manager.test.ts`: add 6 tests covering the readiness probe —
  connect/error/timeout for `probeSocket`, success/early-exit/eventual-
  success for `waitForSocketReady`. Mock `net.connect` with a default
  ENOENT-on-next-tick stub so existing `startToolhive` tests (which now
  trigger a detached readiness probe inside the Sentry.withScope mock)
  don't fail with `undefined.once` access.

- `renderer/index.html`: add `role="status"`, `aria-live="polite"`,
  `aria-busy="true"` to the splash container so screen readers announce
  the loading state. Mark the decorative spinner `aria-hidden="true"`.
  Respect `prefers-reduced-motion: reduce` by disabling the spin
  animation for users with vestibular sensitivities.