feat(core)!: Remove sentry.sdk_meta.gen_ai.input.messages.original_length attribute#22546
Closed
nicohrubec wants to merge 1 commit into
Closed
Conversation
…ength` attribute This internal SDK-meta span attribute only ever signaled how many gen_ai input messages existed before truncation dropped some. It carries no meaning for consumers on its own and is being removed ahead of the truncation removal. Text truncation itself is unchanged by this commit. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
size-limit report 📦
|
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
1 participant
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.
Stacked on #22495. Removes the internal
sentry.sdk_meta.gen_ai.input.messages.original_lengthspan attribute across@sentry/coreand@sentry/server-utils.This SDK-meta attribute only ever recorded how many gen_ai input messages existed before truncation dropped some — it carries no standalone meaning for consumers. It is removed here, ahead of the truncation removal (stacked in #22516) so that change stays focused.
Text truncation itself is unchanged by this PR:
enableTruncation,getTruncatedJsonString, and the truncation machinery all remain. Only the attribute (its definition,server-exportsre-export, all set-sites, and test assertions) is removed. Truncation tests that usedoriginal_lengthto distinguish spans now key offgen_ai.system_instructionspresence instead.