Skip to content

feat(embed): add TelemetrySettings provider bundle + NopTelemetry (PIPE-1066) - #284

Open
Dylan-M wants to merge 1 commit into
mainfrom
dylanmyers/pipe-1066-spine-foundation
Open

feat(embed): add TelemetrySettings provider bundle + NopTelemetry (PIPE-1066)#284
Dylan-M wants to merge 1 commit into
mainfrom
dylanmyers/pipe-1066-spine-foundation

Conversation

@Dylan-M

@Dylan-M Dylan-M commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Proposed Change

Adds embed.TelemetrySettings, the single bundle blitz threads from a host (or the
standalone CLI) down through module construction. Every generator, output, and the
runtime records to the providers and logger it carries. Every field is optional with a
nil-safe fallback to the process globals, so a zero-value bundle behaves as blitz did
before providers were injectable. NopTelemetry() returns a bundle wired to no-op
providers for tests and record-nothing paths.

Nothing consumes the bundle yet. Later PRs thread metrics, traces, and logs through it.

How to validate

This PR adds a type and changes no runtime behavior, so there is nothing to run yet. To
confirm it stands on its own:

  • go build ./... succeeds, and a built binary behaves identically to main.
  • go test ./embed/ exercises the nil-provider fallbacks and NopTelemetry.
Checklist
  • Changes are tested
  • CI has passed

@Dylan-M
Dylan-M requested review from a team as code owners August 6, 2026 15:50
@Dylan-M
Dylan-M force-pushed the dylanmyers/pipe-1066-spine-foundation branch from d27466e to 36f0408 Compare August 14, 2026 19:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant