• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

thoughtspot / mcp-server / 25127974304
89%

Build:
DEFAULT BRANCH: main
Ran 29 Apr 2026 06:58PM UTC
Jobs 1
Files 31
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

29 Apr 2026 06:56PM UTC coverage: 88.187% (+1.5%) from 86.653%
25127974304

push

github

web-flow
SCAL-308277 Add metrics recorder core (#119)

* SCAL-308277 Add metrics recorder core

## Summary

Introduce the request-scoped metrics runtime core for the Worker and add
focused tests so the new runtime metrics files do not regress repo
coverage.

## What Changed

- add metric definitions, label normalization, and runtime config for sink
  selection
- add the request-scoped recorder, sink interfaces, composite/noop sinks,
  and context helpers
- wrap the Worker fetch entrypoint so every request gets a recorder and
  non-blocking flush behavior
- add targeted tests for metric context, sink behavior, recorder branches,
  runtime config, and request-scoped recorder helpers

## Notes

- this PR only adds the runtime core, request wrapper plumbing, and tests
- external Analytics Engine and Grafana emission lands in follow-up PRs

## Validation

- `npm test -- --coverage.enabled=false test/metrics/runtime`
- `npm test`

* SCAL-308277 Fix recorder flush race

## Summary

Close the recorder as soon as flush begins, add coverage for the in-flight
flush case, and address the follow-up lint and label-policy feedback.

## What Changed

- mark the recorder flushed before awaiting the sink and snapshot observations
  once
- add a test that rejects metrics recorded after flush has started
- document why forbidden metric label keys stay explicit
- remove the redundant `both` switch case flagged by Biome

## Validation

- `npm run lint`
- `npm test`

* Refine request metrics routing context

## Summary

Strip tracing headers before metrics setup and centralize route metric
classification so request metrics are easier to maintain as routes evolve.

## What Changed

- move tracing header stripping ahead of `withRequestMetrics(...)` in the
  Worker entrypoint
- replace duplicated metric path classification helpers with a shared explicit
  route-context table and resolver
- expand focused metric-context tests to cover the shared route metadata and
  fallback behavior

#... (continued)

469 of 550 branches covered (85.27%)

Branch coverage included in aggregate %.

186 of 193 new or added lines in 10 files covered. (96.37%)

1151 of 1287 relevant lines covered (89.43%)

343.21 hits per line

Uncovered Changes

Lines Coverage ∆ File
4
94.12
src/metrics/runtime/metric-context.ts
2
91.78
src/metrics/runtime/runtime-config.ts
1
95.92
src/metrics/runtime/metric-types.ts
Jobs
ID Job ID Ran Files Coverage
1 25127974304.1 29 Apr 2026 06:58PM UTC 31
88.19
GitHub Action Run
Source Files on build 25127974304
  • Tree
  • List 31
  • Changed 3
  • Source Changed 3
  • Coverage Changed 3
Coverage ∆ File Lines Relevant Covered Missed Hits/Line Branch Hits Branch Misses
  • Back to Repo
  • Github Actions Build #25127974304
  • 575dc94b on github
  • Prev Build on main (#25126825354)
  • Next Build on main (#25141515918)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc