stacklok / toolhive / 21041880446

59%

Build Type

push

github

Committed by web-flow

Commit Message

Add proxy_port support to MCP server registry (#3300)

* Add proxy_port support to MCP server registry

This change adds support for specifying proxy_port in the MCP server
registry, allowing servers to define a fixed host port without requiring
CLI flags on every run.

Changes:
- Add ProxyPort field to ImageMetadata in registry schema
- Update config_builder to read and use proxy_port from registry
- Update JSON schema to include proxy_port field (1-65535)

Use case: OAuth flows requiring fixed redirect URIs need consistent
ports (e.g., http://localhost:8976/oauth/callback). This eliminates
the need for users to specify --proxy-port on every run.

Before: thv run server --proxy-port 8976 --target-port 8976
After: thv run server (ports defined in registry)

* update docs

* Add tests for ProxyPort registry functionality

- Added TestRunConfigBuilder_WithRegistryProxyPort with 3 test cases
- Tests registry proxy_port usage, CLI override, and fallback behavior
- Improves code coverage for new proxy_port feature

* Fix linter issues: align struct fields formatting

---------

Co-authored-by: rafael <rmoreno@contractor.indeed.com>

Run Details

12 of 12 new or added lines in 1 file covered. (100.0%)

2 existing lines in 1 file now uncovered.

34553 of 58848 relevant lines covered (58.72%)

82.72 hits per line