Add private local telemetry writer - #134
Merged
Merged
Conversation
Doctacon
marked this pull request as draft
August 19, 2026 21:16
Doctacon
marked this pull request as ready for review
August 19, 2026 21:42
Doctacon
added a commit
that referenced
this pull request
Aug 20, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
trace-envelope/v1payloads before idempotent replay into the unchanged DuckDB v1 schemabuoy telemetry statusand boundedbuoy telemetry flush~/.buoy/telemetry/telemetry.duckdbanalytical surfaceWhy
The first local telemetry slice appended to DuckDB inside each retrieval process. That made fresh-process DuckDB startup latency part of retrieval completion and could drop observations when multiple processes contended for the single-writer database. The private writer keeps the same local ownership and content-free schema while making producer handoff durable, bounded, eventual, and observable.
User impact
With telemetry disabled, behavior remains a zero-side-effect no-op. With
BUOY_TELEMETRY=local, retrieval now returns after a private atomic envelope publication instead of waiting for DuckDB persistence. Users can inspect backlog and health withbuoy telemetry statusor request a bounded drain withbuoy telemetry flush.Packaging stop and repair
The first PR head correctly built
buoy = buoy_search.entrypoint:main, but the repository's distribution validator still required the formerbuoy_search.cli:maintarget. The first exact-head Python jobs passed, the distribution job failed on that mismatch, and the PR returned to draft before merge. The repaired validator now uses one exact source/wheel contract, requires the lightweight entry point plus all six telemetry modules, and rejects old, missing, additional, malformed, or non-UTF-8 entry-point metadata. The original packaging PASS is visibly superseded in the records rather than rewritten.Validation
55f41fd9e2f98fed83fd0619c29a9a5549ae4052; packaging repair:33c6124180120bb6711e3463556a672789bc134c; repaired closure head:fe40aadf88e6fbe8ad702225a111d2f787291689Records
The completed ticket, diagnostic findings, corrected evidence, specifications, decision, superseded original review, and independent packaging-repair PASS review are included under
.10x/. The repaired closure commit changes only evidence, reviews, and the ticket location.