stacklok / toolhive / 30458409923
70%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 866
Run time 2min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 70.027% (+0.02%) from 70.01%
30458409923

push

github

web-flow 
Fix port-selection TOCTOU race in e2e tests (#6142)

networking.FindOrUsePort probes a port by binding then immediately
releasing it, so anything else on the machine can steal the number
before the real bind happens later. Under sharded CI this is lost
often enough to flake (observed on PR #6088 as
"OIDC mock server error: listen tcp :49278: bind: address already in use").

Two fixes depending on where the real bind happens:

- In-process mock servers (OIDCMockServer, LLMGatewayMock) now open
  their own listener at construction time instead of accepting a
  pre-selected port, closing the window entirely.
- Subprocess-launched proxies (thv serve, thv llm proxy start) get a
  shared RetryOnPortConflict helper that retries once on a detected
  bind conflict, generalizing the pattern already used for Docker
  workload port races in the vMCP dual-era test helpers. thv proxy
  needed a different approach: it silently substitutes a different
  port on conflict for a nonzero request (no error to retry on) and
  its real bind can lag port selection by a full OAuth exchange, so
  those tests instead pass --port 0 and read the real bound port back
  from the subprocess's own stdout.

Production callers with the same root cause (Docker port bindings,
in-process OAuth/proxy binders) are tracked separately in #6141.

Generated with Claude Code (https://claude.com/claude-code)

82707 of 118108 relevant lines covered (70.03%)

97.13 hits per line

Coverage Regressions

Lines Coverage ∆ File
4
66.64
 0.0% pkg/workloads/manager.go
Jobs
ID Job ID Ran Files Coverage
1 30458409923.1 866
70.03
 GitHub Action Run
Source Files on build 30458409923