stacklok / toolhive / 24393619538

66%

Build Type

push

github

Committed by web-flow

Commit Message

Clean up health check dead code and timeout (#4791)

The constructor and functional options guarantee positive values
for healthCheckInterval and healthCheckPingTimeout, making two
zero-value guards unreachable. The redundant context timeout in
checkMCPStatus capped the configurable ping timeout at 5s; the
MCPPinger already enforces its own timeout internally.

Fixes #4790

Signed-off-by: Greg Katz <gkatz@indeed.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Coverage Stats

3 of 4 new or added lines in 2 files covered. (75.0%)

7 existing lines in 3 files now uncovered.

56930 of 86743 relevant lines covered (65.63%)

62.21 hits per line