Skip to content

fix(web): preserve transcripts and surface run failures - #101

Merged
wangjk9527 merged 2 commits into
datagallery-lab:mainfrom
iancaoo:fix/chat-transcript-restore-and-run-errors
Aug 6, 2026
Merged

fix(web): preserve transcripts and surface run failures#101
wangjk9527 merged 2 commits into
datagallery-lab:mainfrom
iancaoo:fix/chat-transcript-restore-and-run-errors

Conversation

@iancaoo

@iancaoo iancaoo commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Summary

  • prevent same-session conversation refreshes from replacing the live transcript with an older persisted snapshot
  • surface persisted and live run failures above the chat composer when the task console is closed

Problem

A background conversation restore could race with a follow-up query and temporarily remove an earlier assistant response until a newer snapshot arrived. Separately, terminal run errors were visible only in the task console, making timed-out or failed runs appear stuck when the console was closed.

Changes

  • only allow message replacement during an intentional thread restore or when the agent transcript is empty
  • keep same-thread background refreshes for checkpoints, usage, branches, and run state without overwriting existing messages
  • pass the current live-run error into the chat input bindings and render it through the existing composer error alert
  • add regression coverage for same-thread stale snapshot replacement

Verification

  • npm --workspace @datafoundry/web run test -- src/app/data-tasks/__tests__/conversation-restore.test.ts src/app/data-tasks/__tests__/run-error-message.test.ts
  • npm run typecheck
  • npm run build:web

Closes #102

@wangjk9527
wangjk9527 self-requested a review August 6, 2026 13:27
@cyber-luoyang

Copy link
Copy Markdown
Collaborator

lgtm

@wangjk9527 wangjk9527 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@wangjk9527
wangjk9527 merged commit 3374b10 into datagallery-lab:main Aug 6, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Chat transcript flickers on follow-up prompts and run failures are hidden

3 participants