stacklok / toolhive / 27379407427

67%

Build Type

push

github

Committed by web-flow

Commit Message

Carry RateLimitMiddleware through vMCP buildServeConfig (#5500)

#5276 added Config.RateLimitMiddleware and wired it on the New/Config
path, but the parallel Serve/ServerConfig migration API was not extended:
ServerConfig had no RateLimitMiddleware field and buildServeConfig did
not map it, so TestBuildServeConfigMapsSharedFields fails on main and the
Serve path would silently drop rate limiting.

Add the field to ServerConfig, map it in buildServeConfig, and set it in
the completeness test's source config so a dropped mapping keeps failing
the test.

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Coverage Stats

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

201 existing lines in 7 files now uncovered.

67606 of 101697 relevant lines covered (66.48%)

64.05 hits per line