stacklok / toolhive / 23364212495

64%

Build Type

push

github

Committed by web-flow

Commit Message

Fix inspector URL using wrong transport for proxied stdio servers (#4300)

* Fix inspector URL using wrong transport for proxied stdio servers

`thv inspector` was reading the server's native TransportType (e.g.
"stdio") instead of ProxyMode to determine the inspector URL transport
and path suffix. This caused stdio servers proxied as streamable-http
to get a URL with transport=sse and /sse instead of
transport=streamable-http and /mcp.

Fixes #4298

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Address review feedback: rename function and simplify suffix logic

Rename getServerPortAndTransport to getServerPortAndProxyMode and
replace IIFE with plain if/else for the URL path suffix.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Dan Barr <6922515+danbarr@users.noreply.github.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Run Details

8 of 22 new or added lines in 1 file covered. (36.36%)

24 existing lines in 5 files now uncovered.

50174 of 77849 relevant lines covered (64.45%)

92.24 hits per line