Skip to content

perf(dashboard): Stop live polls thrashing typing - #1594

Merged
phacops merged 8 commits into
mainfrom
perf/mobile-live-transcript-updates
Aug 16, 2026
Merged

perf(dashboard): Stop live polls thrashing typing#1594
phacops merged 8 commits into
mainfrom
perf/mobile-live-transcript-updates

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

Live 2s polls still forced full transcript rebuilds while typing, even after the earlier off-screen paint pass.

This change keeps the same UI and features. It stops poll work from competing with keystrokes:

  • reuse the previous conversation detail object when a poll only refreshes metadata (generatedAt, duration, lastSeen)
  • version the live bottom pin from the last event only, instead of rebuilding the full transcript model
  • keep the reply footer off the full transcript object; pass only committed message ids for mailbox de-dupe
  • reuse unchanged transcript message objects and memoize message/text rows so only the live tail re-renders

Validation:

  • pnpm --filter @sentry/junior-dashboard typecheck
  • pnpm --filter @sentry/junior-dashboard lint
  • pnpm --filter @sentry/junior-dashboard test (282 passed)
  • git diff --check

Requested by PDPM.

--

View Junior Session in Sentry

Keep no-op conversation poll payloads stable, cheapen live tail versioning,
isolate the reply footer from full transcript rebuilds, and reuse unchanged
message objects so typing stays responsive while messages arrive.

Co-Authored-By: PDPM <pierre.massat@sentry.io>
@vercel

vercel Bot commented Aug 16, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
junior-docs Ready Ready Preview Aug 16, 2026 2:59am

Request Review

@github-actions

github-actions Bot commented Aug 16, 2026

Copy link
Copy Markdown

Dashboard visual evidence

Mode: path-selected
Selected: conversations, conversation-detail, component-gallery

Triggered by:

  • packages/junior-dashboard/e2e/conversations.spec.ts
  • packages/junior-dashboard/src/client/conversations/ConversationComposer.tsx
  • packages/junior-dashboard/src/client/conversations/ConversationPage.tsx
  • packages/junior-dashboard/src/client/conversations/ConversationTranscript.tsx
  • packages/junior-dashboard/src/client/conversations/PendingMailboxStack.tsx
  • packages/junior-dashboard/src/client/conversations/TranscriptMessageView.tsx
  • packages/junior-dashboard/src/client/conversations/TranscriptText.tsx
  • packages/junior-dashboard/src/client/conversations/TranscriptView.tsx

Conversations · desktop

Conversations · desktop

Conversations · mobile

Conversations · mobile

Conversation detail · desktop

Conversation detail · desktop

Conversation detail · mobile

Conversation detail · mobile

Component gallery · desktop

Component gallery · desktop

Full-page screenshots from the mock dashboard. Not a pixel-diff gate.

Comment thread packages/junior-dashboard/src/client/conversations/transcript.ts Outdated
Comment thread packages/junior-dashboard/src/client/conversations/transcriptBottomPinning.ts Outdated
sentry-junior Bot and others added 2 commits August 16, 2026 01:56
Co-Authored-By: PDPM <pierre.massat@sentry.io>
…script-updates

# Conflicts:
#	packages/junior-dashboard/src/client/conversations/TranscriptMessageView.tsx

Co-Authored-By: PDPM <pierre.massat@sentry.io>
Co-Authored-By: PDPM <pierre.massat@sentry.io>
Co-Authored-By: PDPM <pierre.massat@sentry.io>
@blacksmith-sh

This comment has been minimized.

Comment thread packages/junior-dashboard/src/client/conversations/transcriptBottomPinning.ts Outdated
@github-actions github-actions Bot added the risk: medium PR risk score: medium label Aug 16, 2026
Comment thread packages/junior-dashboard/src/client/conversations/transcriptBottomPinning.ts Outdated
Co-Authored-By: PDPM <pierre.massat@sentry.io>

@cursor cursor Bot left a comment

Copy link
Copy Markdown

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 f712cf3. Configure here.

Comment thread packages/junior-dashboard/src/client/conversations/transcriptBottomPinning.ts Outdated
Co-Authored-By: PDPM <pierre.massat@sentry.io>
@phacops
phacops merged commit e924b40 into main Aug 16, 2026
34 checks passed
@phacops
phacops deleted the perf/mobile-live-transcript-updates branch August 16, 2026 03:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: medium PR risk score: medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant