thoughtspot / mcp-server / 25421235134

88%

push

github

SCAL-308280 Instrument request and auth health metrics (#134)

* SCAL-308280 Instrument request and auth health metrics

## Summary

Add the request-layer metrics for SCAL-308280 so MCP traffic and auth flows
emit bounded counters and latency through the shared runtime recorder.

## What Changed

- record ts_mcp_http_requests_total and ts_mcp_http_request_duration_ms at the worker edge with route, transport, auth mode, api surface, outcome, status class, and canonical api version labels
- add helpers to resolve bounded api_version values through the version registry instead of copying raw query params
- emit auth outcome counters for /authorize, /callback, /store-token, and /bearer/* / /token/* requests
- add focused tests for request metric labeling and auth metric emission
- include the small metrics-runtime review nits from PR #126 while touching the same files

## Notes

- ts_mcp_http_inflight_requests remains intentionally unimplemented because the current request-scoped recorder cannot represent a real inflight gauge safely

## Validation

- npm run lint -- src/index.ts src/bearer.ts src/handlers.ts src/metrics/runtime/request-metrics.ts src/metrics/runtime/grafana-otlp-sink.ts test/metrics/runtime/request-metrics.spec.ts test/bearer.spec.ts test/handlers.spec.ts
- npm test -- --coverage.enabled false test/metrics/runtime/request-metrics.spec.ts test/bearer.spec.ts test/handlers.spec.ts

* SCAL-308280 Fix API version labels in request metrics

## Summary

- map resolved API versions to stable low-cardinality labels
- reuse the parsed request URL in bearer auth handling
- add coverage for date-based API version label mapping

613 of 712 branches covered (86.1%)

Branch coverage included in aggregate %.

125 of 139 new or added lines in 5 files covered. (89.93%)

1426 of 1595 relevant lines covered (89.4%)

631.5 hits per line