Skip to content

feat(traces): Use new transcript in AI trace tabs#119039

Open
obostjancic wants to merge 5 commits into
masterfrom
ognjenbostjancic/tet-2604-use-new-transcript-in-ai-trace-tabs
Open

feat(traces): Use new transcript in AI trace tabs#119039
obostjancic wants to merge 5 commits into
masterfrom
ognjenbostjancic/tet-2604-use-new-transcript-in-ai-trace-tabs

Conversation

@obostjancic

@obostjancic obostjancic commented Jul 6, 2026

Copy link
Copy Markdown
Member

Behind the gen-ai-conversations-redesign flag, the trace AI tab uses the redesigned transcript and span detail components. The conversation panel renders MessagesPanelNew on the left and the redesigned ConversationSpanDetail on the right, reusing the existing ConversationSplitLayout resizable panel — no hardcoded pixel sizes.

Tab order is [Timeline, Transcript(s)] with Timeline selected by default. The flag-off path is unchanged: Spans first, legacy MessagesPanel.

Closes TET-2604

Behind the gen-ai-conversations-redesign flag, the trace AI tab now leads
with the redesigned transcript instead of the Timeline/Spans tab, and
renders the Seer Explorer-styled transcript (MessagesPanelNew) with the
redesigned span detail panel that opens on span selection.

The flag-off path is unchanged: Spans first, legacy MessagesPanel.

Refs TET-2604
@linear-code

linear-code Bot commented Jul 6, 2026

Copy link
Copy Markdown

TET-2604

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jul 6, 2026
Replaces the copied fixed-width responsive layout (320px/430px hardcoded
sizes) with ConversationSplitLayout, which is already used by the legacy
path and handles sizing via a resizable split panel. Drops the manual
open/close detail state in favour of ConversationSplitLayout's right-panel
empty state. Removes the explanatory comments and the now-unused
MessagesPanelSkeleton import.

Refs TET-2604
@obostjancic obostjancic marked this pull request as ready for review July 6, 2026 10:35
@obostjancic obostjancic requested a review from a team as a code owner July 6, 2026 10:35
@obostjancic obostjancic requested a review from a team July 6, 2026 12:36
Put Timeline tab before Transcript tab(s) and default to Timeline
on mount, regardless of the redesign flag.

Co-Authored-By: Claude Sonnet 4 <noreply@example.com>

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

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.

Reviewed by Cursor Bugbot for commit e09df7f. Configure here.

ConversationSpanDetail already sets flex=1, minHeight=0, and
background=primary on its root Stack, so the outer Flex was redundant.

Co-Authored-By: Claude Sonnet 4 <noreply@example.com>
…ing state

The transcript was still using the old ConversationViewSkeleton.
Switch to MessagesPanelSkeleton which mirrors the conversation bubble
layout of MessagesPanelNew.

Co-Authored-By: Claude Sonnet 4 <noreply@example.com>
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