feat(attributes): add attribute transformations by constantinius · Pull Request #465 · getsentry/sentry-conventions

constantinius · 2026-07-07T10:34:20Z

Summary

add attribute transformation documents for deprecated attributes whose replacement values need reshaping
add deprecation._status: "transform" and deprecation.transformation references
document how to add attribute transformations
expose transformation references in generated JS and Python metadata

This replaces the previous migration-based approach from #454 and #455.

Contributes to TET-2369.
Closes TET-2586.

Testing

yarn test

linear-code · 2026-07-07T10:34:23Z

TET-2369

TET-2586

github-actions · 2026-07-07T10:34:34Z

Semver Impact of This PR

🟡 Minor (new features)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

New Features ✨

Attributes

Add attribute transformations by constantinius in #465

Add stable app vitals attributes by buenaflor in #493
Deprecate sentry.frames.* attributes by buenaflor in #500
Add missing AI legacy aliases from Relay SpanData by vgrozdanic in #498

Internal Changes 🔧

(attrs) Backport attributes and normalizations from Relay by Dav1dde in #497
(codeowners) Assign telemetry-experience to gen_ai/ai attributes by vgrozdanic in #506
(release) Show version in workflow run title by constantinius in #504

_{🤖 This preview updates automatically when you update the PR.}

alexander-alderman-webb · 2026-07-07T12:24:54Z

The types set for gen_ai.request.messages and gen_ai.response.text are inconsistent in the SDKs.
You yourself have added the litellm integration that sets a list of objects for gen_ai.response.text.
How should the suggested transformations be implemented if the source attribute does not have a consistent type?

constantinius · 2026-07-07T14:05:14Z

How should the suggested transformations be implemented if the source attribute does not have a consistent type?

This is true and indeed an issue. In the frontend, we have several paths to get the input/output messages, I will match them in Relay, but I think I should also mention them here in the sentry-conventions

jjbayer · 2026-07-08T10:28:35Z

@@ -0,0 +1,45 @@
+{
+  "id": "gen_ai_request_messages_to_input_messages",


Is the ID also what we expect the function name to be in Relay, by convention?

No, this is just the ID we link to from the attribute where we want to apply the transformation.

Introduce transformation documents for deprecated attributes whose replacement values need to be reshaped rather than copied. The attribute schema now supports a transform deprecation status with a transformation reference, and generated metadata exposes that contract for downstream consumers like Relay.

cursor

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

^{Reviewed by Cursor Bugbot for commit 1019ddc. Configure here.}

Document the common gen_ai request message shapes seen in the wild, including string content, array content, and preserved response metadata.

constantinius · 2026-07-10T07:41:19Z

@alexander-alderman-webb
This now also includes the transformation from various output formats we encounter across the SDK integrations.

alexander-alderman-webb · 2026-07-10T08:47:50Z

Well, that's not true 😅.
For example, the LangChain integration sets an array of arrays as gen_ai.response.text.

constantinius · 2026-07-10T09:58:23Z

@alexander-alderman-webb

For example, the LangChain integration sets an array of arrays as gen_ai.response.text.

I can't confirm this. In the testing framework I get for Python/JS LangChain only strings or array of strings across all tests. Do you have a working example?

Also, at some point we have to stop chasing wrong implementations.

loewenheim

LGTM. The extensive specification and examples are nice.

Resolve the generated attributes conflict after updating the attribute transformations branch with latest main. Regenerated outputs so CI's clean-working-tree check passes.

constantinius requested review from a team, Lms24, cleptric, mjq and nsdeschenes as code owners July 7, 2026 10:34

This was referenced Jul 7, 2026

feat(attributes): add data migration utilities #454

Closed

feat(attributes): add gen ai data migrations #455

Closed

cursor Bot reviewed Jul 7, 2026

View reviewed changes

Comment thread model/attribute_transformations/gen_ai_response_to_output_messages.json

Comment thread model/attribute_transformations/gen_ai_response_to_output_messages.json

Comment thread model/attributes/gen_ai/gen_ai__request__messages.json

jjbayer reviewed Jul 8, 2026

View reviewed changes

constantinius force-pushed the constantinius/feat/attributes/TET-2369-transformations branch from 78ef8b7 to 1019ddc Compare July 9, 2026 11:01

cursor Bot reviewed Jul 9, 2026

View reviewed changes

Comment thread model/attribute_transformations/gen_ai_request_messages_to_input_messages.json

docs(attributes): expand input message transformation examples

d8be4e4

Document the common gen_ai request message shapes seen in the wild, including string content, array content, and preserved response metadata.

constantinius mentioned this pull request Jul 9, 2026

feat(eap): Implement gen_ai attribute transformations getsentry/relay#6201

Draft

loewenheim approved these changes Jul 14, 2026

View reviewed changes

Lms24 mentioned this pull request Jul 14, 2026

Flag type divergence in attribute deprecation chains #499

Open

chore: merge main into attribute transformations

2547f36

Resolve the generated attributes conflict after updating the attribute transformations branch with latest main. Regenerated outputs so CI's clean-working-tree check passes.

constantinius requested a review from a team as a code owner July 16, 2026 10:49

constantinius enabled auto-merge (squash) July 16, 2026 10:50

constantinius merged commit aa03b0d into main Jul 16, 2026
16 checks passed

constantinius deleted the constantinius/feat/attributes/TET-2369-transformations branch July 16, 2026 10:51

constantinius mentioned this pull request Aug 3, 2026

chore(conventions): Bump sentry-conventions to include transform status getsentry/relay#6281

Merged

		@@ -0,0 +1,45 @@
		{
		"id": "gen_ai_request_messages_to_input_messages",

Uh oh!

Uh oh!

Conversation

constantinius commented Jul 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Testing

Uh oh!

linear-code Bot commented Jul 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions Bot commented Jul 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Semver Impact of This PR

New Features ✨

Attributes

Internal Changes 🔧

Uh oh!

Uh oh!

Uh oh!

Uh oh!

alexander-alderman-webb commented Jul 7, 2026

Uh oh!

constantinius commented Jul 7, 2026

Uh oh!

jjbayer Jul 8, 2026

Choose a reason for hiding this comment

Uh oh!

constantinius Jul 8, 2026

Choose a reason for hiding this comment

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

constantinius commented Jul 10, 2026

Uh oh!

alexander-alderman-webb commented Jul 10, 2026

Uh oh!

constantinius commented Jul 10, 2026

Uh oh!

loewenheim left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

constantinius commented Jul 7, 2026 •

edited

Loading

linear-code Bot commented Jul 7, 2026 •

edited

Loading

github-actions Bot commented Jul 7, 2026 •

edited

Loading