Skip to content

Fix/chat history not rendering on return#1690

Closed
markturansky wants to merge 3 commits into
ambient-code:mainfrom
markturansky:fix/chat-history-not-rendering-on-return
Closed

Fix/chat history not rendering on return#1690
markturansky wants to merge 3 commits into
ambient-code:mainfrom
markturansky:fix/chat-history-not-rendering-on-return

Conversation

@markturansky

Copy link
Copy Markdown
Contributor

No description provided.

user and others added 2 commits June 15, 2026 15:46
Replace async params.then() + useState pattern with synchronous
React.use(params) so projectName and sessionName are available from the
first render. The previous pattern left both values as empty strings on
mount, causing the AG-UI EventSource connection to be deferred until a
second render cycle. On fast re-navigation this race could result in
historical SSE events being missed and chat history not rendering.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
The between-run listener was calling GET /events without a thread_id
path parameter, causing the runner to return 404 on every attempt.
The runner endpoint requires GET /events/{thread_id}. Use the session
name as the thread_id (matching the convention in HandleAGUIRunProxy).

🤖 Generated with [Claude Code](https://claude.ai/code)

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

netlify Bot commented Jun 15, 2026

Copy link
Copy Markdown

Deploy Preview for cheerful-kitten-f556a0 canceled.

Name Link
🔨 Latest commit 0e0db04
🔍 Latest deploy log https://app.netlify.com/projects/cheerful-kitten-f556a0/deploys/6a306dd53969eb0008b58903

@coderabbitai

coderabbitai Bot commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Caution

Review failed

The head commit changed during the review from 9bfcaec to 0e0db04.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
✨ Simplify code
  • Create PR with simplified code

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant