feat(attributes): add attribute transformations#465
Conversation
Semver Impact of This PR🟡 Minor (new features) 📋 Changelog PreviewThis is how your changes will appear in the changelog. New Features ✨Attributes
Other
Internal Changes 🔧
🤖 This preview updates automatically when you update the PR. |
|
The types set for |
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 |
| @@ -0,0 +1,45 @@ | |||
| { | |||
| "id": "gen_ai_request_messages_to_input_messages", | |||
There was a problem hiding this comment.
Is the ID also what we expect the function name to be in Relay, by convention?
There was a problem hiding this comment.
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.
78ef8b7 to
1019ddc
Compare
There was a problem hiding this comment.
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.
|
@alexander-alderman-webb |
|
Well, that's not true 😅. |
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. |

Summary
deprecation._status: "transform"anddeprecation.transformationreferencesThis replaces the previous migration-based approach from #454 and #455.
Contributes to TET-2369.
Closes TET-2586.
Testing
yarn test