Skip to content

fix(ai-insights): Use AIContentRenderer for conversation table tooltips#111285

Merged
obostjancic merged 2 commits intomasterfrom
ognjenbostjancic/tet-2129-agree-on-where-to-use-the-new-input-output-render
Mar 23, 2026
Merged

fix(ai-insights): Use AIContentRenderer for conversation table tooltips#111285
obostjancic merged 2 commits intomasterfrom
ognjenbostjancic/tet-2129-agree-on-where-to-use-the-new-input-output-render

Conversation

@obostjancic
Copy link
Member

@obostjancic obostjancic requested a review from a team as a code owner March 23, 2026 12:02
@linear-code
Copy link

linear-code bot commented Mar 23, 2026

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 23, 2026
Copy link
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Fix All in Cursor

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

return (
<TooltipTextContainer>
<MarkedText text={ellipsize(text, TOOLTIP_MAX_CHARS)} />
<AIContentRenderer text={ellipsize(text, TOOLTIP_MAX_CHARS)} inline />
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

JSON content renders interactive widget inside tooltip

Medium Severity

AIContentRenderer ignores the inline prop for json, python-dict, and fixed-json content types. When conversation input/output happens to be JSON, the tooltip will render a full TraceDrawerComponents.MultilineJSON widget — which includes useState, useHover, a Pretty/Raw SegmentedControl, and expandable StructuredData trees — inside a tooltip. This creates a broken UX with interactive controls fighting the tooltip's hover behavior.

Fix in Cursor Fix in Web

@obostjancic obostjancic merged commit c7e0983 into master Mar 23, 2026
70 checks passed
@obostjancic obostjancic deleted the ognjenbostjancic/tet-2129-agree-on-where-to-use-the-new-input-output-render branch March 23, 2026 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants