fix(assistant): store the transcript before message.end so the panel shows the answer #85

Merged

james.maes merged 1 commits from fix/transcript-stored-before-message-end into release/0.1 2026-11-05 10:17:11 +00:00

Conversation 0 Commits 1 Files Changed 4

+73

james.maes commented 2026-11-05 01:17:26 +00:00

Owner

Copy link

Copy source

The panel reloads the conversation the moment the stream ends. The redacted transcript (message parts, pending actions, stored events) was flushed only after message.end was sent, so that reload read the "[]" placeholders and showed the newest question and answer as empty bubbles until the panel was closed and reopened.

Persist and flush before message.end is emitted. Regression test: TurnEngineTest.testMessagesAreStoredBeforeMessageEnd (fails without the fix).

./mvnw verify -pl qbit-assistant -am: 352 tests, coverage gate met (release/0.1 copy).

🤖 Generated with Claude Code

The panel reloads the conversation the moment the stream ends. The redacted transcript (message parts, pending actions, stored events) was flushed only after `message.end` was sent, so that reload read the `"[]"` placeholders and showed the newest question and answer as empty bubbles until the panel was closed and reopened. Persist and flush before `message.end` is emitted. Regression test: `TurnEngineTest.testMessagesAreStoredBeforeMessageEnd` (fails without the fix). `./mvnw verify -pl qbit-assistant -am`: 352 tests, coverage gate met (release/0.1 copy). 🤖 Generated with [Claude Code](https://claude.com/claude-code)

james.maes added 1 commit 2026-11-05 01:17:26 +00:00

fix(assistant): store the transcript before message.end so the panel shows the answer ... c5125f9788

The panel reloads the conversation the moment the stream ends. The
redacted transcript (message parts, pending actions, stored events) was
flushed only after message.end was sent, so that reload read the "[]"
placeholders and showed the newest question and answer as empty bubbles
until the panel was closed and reopened. Persist and flush first, then
emit message.end.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

james.maes merged commit 2d25f0b769 into release/0.1 2026-11-05 10:17:11 +00:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

architecture

Architecture review: coupling, cohesion, UI API

bug

Something isn't working

dependencies

documentation

Improvements or additions to documentation

duplicate

This issue or pull request already exists

enhancement

New feature or request

github_actions

Pull requests that update GitHub Actions code

good first issue

Good for newcomers

help wanted

Extra attention is needed

invalid

This doesn't seem right

java

Pull requests that update java code

question

Further information is requested

release:rc

Merging this develop PR cuts the next release candidate

secuurity

ux

Admin UX / information architecture

wontfix

This will not be worked on

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

forge-recovery (Forge Recovery) james.maes (James Maes)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: QRun-IO/qbit-assistant#85