thoughtspot / mcp-server / 29208868667

89%

push

github

Keep org tools available when session info fails to load (#182)

isOrgsEnabled() returned false whenever sessionInfo was null. sessionInfo
is fetched once at init via getSessionInfo(), which runs before the
keep-warm global token is reconciled from the token-store DO — so on a
cold-start init after the frozen grant/props token has expired (~24h),
getSessionInfo() authenticates with the expired token, fails, and leaves
sessionInfo null. That silently stripped list_orgs/switch_org from the
tool list even though the DO still held a valid kept-warm token (proven
by other tool calls succeeding, since they reconcile the DO token at call
time).

Default isOrgsEnabled() to true when sessionInfo is absent so org tools
survive a failed getSessionInfo. The tool call paths reconcile the live
token from the DO independently, so the tools still function when the
kept-warm token is healthy. When sessionInfo IS present we still respect
the cluster's real orgsEnabled flag, so orgs-disabled clusters are only
affected during an actual session-info failure. Non-oauth (bearer/token)
clusters are unaffected — the authMode gate short-circuits first.

This is a scoped hotfix; the root-cause fix (reconcile the global token
before getSessionInfo so sessionInfo loads correctly) will follow.

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

806 of 937 branches covered (86.02%)

Branch coverage included in aggregate %.

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

1573 of 1740 relevant lines covered (90.4%)

1094.88 hits per line