Altinity / altinity-mcp / 24507656808

77%

main: 77%

Build Type

push

github

Committed by BorisTyshkevich

Commit Message

Defer dynamic tool discovery until credentials are available

In forward-OAuth mode with blank static credentials, EnsureDynamicTools
failed during the tools/list handshake because no Bearer token was present
yet. Discovery now checks for available credentials (JWE, OAuth, or static)
and skips gracefully if none exist, retrying on the next request.

Also fixes discovery to use GetClickHouseClientFromCtx which handles both
JWE and OAuth tokens, instead of only checking for JWE tokens.

The MCP SDK's AddTool automatically sends notifications/tools/list_changed,
so the client re-fetches the tool list after deferred discovery completes.

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

Coverage Stats

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

244 existing lines in 1 file now uncovered.

4302 of 5393 relevant lines covered (79.77%)

1.14 hits per line