Repository navigation
Add task telemetry diagnostics tech spec - #171
Draft
bholmesdev wants to merge 1 commit into
Draft
bholmesdev wants to merge 1 commit into
bholmesdev wants to merge 1 commit into
Conversation
Co-Authored-By: Warp <agent@warp.dev>
Author
|
This PR was generated with Warp. Comment |
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.


Adds a TECH.md proposal for a shared sanitized diagnostic contract around task telemetry exporter failures. The spec is motivated by recent OTLP task telemetry changes where log export failures can preserve collector response bodies in worker logs.
Review guide
Start with
specs/task-telemetry-sanitized-diagnostics/TECH.md. The key decision is the sanitizer boundary: keep useful status/category/request diagnostics while prohibiting raw collector responses, credentials, endpoints with user info, and nested provider error strings.What changed
internal/tasktelemetry/otlpsanitizer for all task telemetry exporter errors.Validation
TECH.mdafter writing it.