-
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
feat(agents): Render markdown and allow switching to raw values #104850
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(agents): Render markdown and allow switching to raw values #104850
Conversation
static/app/views/performance/newTraceDetails/traceDrawer/details/styles.tsx
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Missing key props cause inconsistent state reset behavior
The PR adds key={getNodeId(node)} to MultilineText and MessagesArrayRenderer components to reset internal state when switching nodes (lines 239, 245), but the MultilineJSON for toolArgs and MultilineText for embeddingsInput don't have this key prop. Since these components now have internal showRaw and clipped state that should reset when nodes change, the missing keys cause inconsistent behavior where some views reset when switching nodes but these don't, potentially showing stale raw/formatted toggle states from a previously viewed node.
static/app/views/performance/newTraceDetails/traceDrawer/details/span/eapSections/aiInput.tsx#L249-L256
sentry/static/app/views/performance/newTraceDetails/traceDrawer/details/span/eapSections/aiInput.tsx
Lines 249 to 256 in 40113c9
| ) : null} | |
| {toolArgs ? ( | |
| <TraceDrawerComponents.MultilineJSON value={toolArgs} maxDefaultDepth={1} /> | |
| ) : null} | |
| {embeddingsInput ? ( | |
| <TraceDrawerComponents.MultilineText> | |
| {embeddingsInput.toString()} | |
| </TraceDrawerComponents.MultilineText> |
…-render-markdown-nicely-in-span-details
Render messages as markdown.
Allow switching to raw values for text and json messages.
Adjust the collapsing mechanism to also work with rich content.
Screen.Recording.2025-12-12.at.12.56.43.mov
Screen.Recording.2025-12-12.at.12.47.16.mov