1
66%
main: 66%

Ran 08 Apr 2026 08:42AM UTC

Files 607

Run time 17s

Badge

Embed ▾

Committed 08 Apr 2026 08:36AM UTC coverage: 65.145% (-0.05%) from 65.19%

Job # 24126081599.1

Build Type

push

github

Committed by

web-flow

Commit Message

fix(telemetry): warn instead of crashing when OTLP endpoint has no enabled exporters (#4648)

* fix(telemetry): warn instead of crashing when OTLP endpoint has no enabled exporters

When an OTLP endpoint is configured via `thv config otel set-endpoint`
but both tracing and metrics are disabled, `thv serve` crashes with a
fatal validation error preventing the server from starting.

Instead of failing, log a warning and clear the endpoint so the server
starts normally. This avoids a confusing "Health check failed" error
in ToolHive Studio.

Fixes #4647

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* refactor(telemetry): extract handleUnusedEndpoint to reduce cyclomatic complexity

Move the unused endpoint check into a dedicated helper function to
keep NewServeProvider under the gocyclo threshold (15).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix(telemetry): enable tracing by default when OTLP endpoint has no enabled exporters

Instead of clearing the endpoint (which would silently drop telemetry
for users who configured an endpoint expecting it to work), enable
tracing by default so the endpoint is actually used.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

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

Coverage Stats

54845 of 84189 relevant lines covered (65.15%)

63.56 hits per line

stacklok / toolhive / 24126081599 / 1
66%
main: 66%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 24126081599.1

stacklok / toolhive / 24126081599 / 1 66% main: 66%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 24126081599.1

stacklok / toolhive / 24126081599 / 1
66%
main: 66%

README BADGES
x