thoughtspot / mcp-server / 31232452540

89%

Build Type

push

github

Committed by web-flow

Commit Message

Add raw Spotter output mode behind a flag (#200)

* Add raw Spotter output mode behind a flag

- Client can specify via URL flag if they want the full raw Spotter
  output instead of the simplified version we are providing
- Pass along the unmodified Spotter event stream if this flag has
  been set
- No impact when using v1 tools, this is only for v2 tools
- Update tests and add coverage

* Fix raw session-updates review findings

1. Synthesize answer_id on RAW answer events, so the get_session_updates
   -> create_dashboard handoff keeps working when
   enable-raw-session-updates is on (callCreateDashboard unconditionally
   JSON.parses answer_id).
2. Split the get_session_updates tool description into structured vs
   raw variants instead of sharing one description across both output
   schemas; the raw description now reflects the unfiltered z.unknown()
   contract instead of promising complete/filtered results.
3. Raw mode now still classifies each event by type to record
   recordUpstreamStreamMessageMetric counts and flag span errors on
   upstream error events, closing the observability blind spot where
   raw-mode metrics/error spans went dark. Messages are still passed
   through unmodified otherwise.
4. Widen Message[] to (Message | RawMessage)[] through the
   streaming-utils -> thoughtspot-service -> storage-service ->
   conversation-storage-server chain, since raw mode's pass-through
   items are not guaranteed to match the Message shape.

Updated test/streaming-utils.spec.ts to match the new raw-mode
behavior for answer_id synthesis, error observability, and metrics.

---------

Co-authored-by: Mourya Balabhadra <mourya.balabhadra@thoughtspot.com>

Coverage Stats

1010 of 1168 branches covered (86.47%)

Branch coverage included in aggregate %.

35 of 41 new or added lines in 6 files covered. (85.37%)

2 existing lines in 1 file now uncovered.

1802 of 1980 relevant lines covered (91.01%)

1128.86 hits per line