stacklok / toolhive / 23141866436

64%

Build Type

push

github

Committed by web-flow

Commit Message

fix(vmcp): /status uses live health monitor state (#4135)

The /status endpoint was reading backend health from the static
registry snapshot (set at discovery time and never updated), while
/api/backends/health correctly read from the live health monitor.
This caused the two endpoints to report inconsistent state for the
same backend (issue #4103).

Fix buildStatusResponse() to call GetAllBackendHealthStates() and
prefer the monitor's runtime health over the registry's initial
value. When health monitoring is disabled the registry value is
used as before, preserving backwards compatibility.

Add unit tests that assert /status reflects live monitor state for
both healthy and unhealthy transitions, and an e2e It block in the
circuit breaker lifecycle suite that compares both endpoints
side-by-side once the unstable backend's circuit breaker opens.

Closes: #4103

Co-authored-by: taskbot <taskbot@users.noreply.github.com>

Run Details

19 of 20 new or added lines in 1 file covered. (95.0%)

11 existing lines in 4 files now uncovered.

48950 of 75906 relevant lines covered (64.49%)

162.63 hits per line