feat(resume-build): add optional telemetry/tracing support#2345
feat(resume-build): add optional telemetry/tracing support#2345
Conversation
Wire telemetry.New() into resume-build so traces and metrics are exported when OTEL_COLLECTOR_GRPC_ENDPOINT is set. When unset, behavior is unchanged (noop client, zero overhead).
PR SummaryLow Risk Overview Reviewed by Cursor Bugbot for commit c418c9a. Bugbot is set up for automated code reviews on this repo. Configure here. |
…l pointer panic With a real OTEL provider, instrument registration can fail, leaving the Metrics struct with nil fields that cause panics on first use. Matches the pattern in the production orchestrator.
…ew() Move the env var documentation from the resume-build call site to the telemetry.New() function definition where it belongs.
…metadata Add telemetry.NewAnonymous(ctx, serviceName) for CLI tools that lack build-time injected values. Uses hostname as nodeID and sensible defaults for the rest. Update resume-build to use it.
Wire telemetry.New() into resume-build so traces and metrics are exported when
OTEL_COLLECTOR_GRPC_ENDPOINTis set. When unset, behavior is unchanged (noop client, zero overhead).This is useful for sandbox debugging.