thoughtspot / mcp-server / 25576946935

89%

push

github

SCAL-311395 Fix Analytics Engine tenant identity (#150)

## Summary

Fix Analytics Engine tenant identity so `tenant_id` stores the canonical ThoughtSpot `clusterId` instead of `currentOrgId`, cut Analytics Engine over to a clean dataset, and fill in the missing request-context labels on OAuth auth-flow counters.

## What Changed

- source Analytics Engine `tenant_id` from `SessionInfo.clusterId` in `BaseMCPServer.getMetricEventIdentity()`
- keep `currentOrgId` separate so org-level state is not mislabeled as tenant identity
- remove the temporary separate `tenantId` field from `SessionInfo` and update service/server tests to use `clusterId`
- switch the `ANALYTICS` binding target in `wrangler.jsonc` from `mcp_events` to `mcp_events_v2`
- attach `route_group`, `transport`, `auth_mode`, `api_surface`, and `status_class` to OAuth authorize/callback/store-token counters in `handlers.ts`

## Semantics

Analytics Engine `tenant_id` now represents the actual ThoughtSpot cluster/tenant identifier (`clusterId`). Historical rows in `mcp_events` may still contain org ids, so the new deployment writes to `mcp_events_v2` as the clean source of truth for tenant-level analysis. OAuth auth-flow counters now carry the same request-context labels as the rest of the HTTP/auth metrics.

## Validation

- `npm run lint`
- `npm test -- --coverage.enabled=false test/servers/mcp-server-base.spec.ts test/servers/mcp-server.spec.ts test/thoughtspot/thoughtspot-service.spec.ts test/handlers.spec.ts test/metrics/runtime/request-metrics.spec.ts`

713 of 836 branches covered (85.29%)

Branch coverage included in aggregate %.

2 of 2 new or added lines in 2 files covered. (100.0%)

1533 of 1689 relevant lines covered (90.76%)

830.02 hits per line