stacklok / toolhive / 23509622591

65%

Build Type

push

github

Committed by web-flow

Commit Message

Add RegisterHandlers and wire embedded AS routes on vMCP mux (#4348)

* Add RegisterHandlers and wire embedded AS routes on vMCP mux

Phase 2 of the vMCP embedded authorization server (#4141):

- Add RegisterHandlers(mux) to EmbeddedAuthServer — the AS owns its
  route paths (/oauth/, /.well-known/openid-configuration, etc.)
- Add AuthServer field to vMCP server Config
- Replace /.well-known/ catch-all with explicit registrations:
  /.well-known/oauth-protected-resource (exact + subpath) is always
  registered; Mode B conditionally adds AS discovery/JWKS routes
- No commands.go changes — CLI path deferred to Phase 4

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

* Extract Routes() method to eliminate AS route duplication

The proxy runner and vMCP server both registered the same four
authorization server routes independently. Extract a Routes() method
on EmbeddedAuthServer that returns the canonical route map, and have
both RegisterHandlers and the proxy runner consume it.

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

---------

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

Run Details

18 of 27 new or added lines in 3 files covered. (66.67%)

353 existing lines in 6 files now uncovered.

50892 of 79068 relevant lines covered (64.36%)

151.25 hits per line