ryan/simbarag
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix mobile conversation launch resetting to homepage #27

Merged
ryan merged 1 commits from fix/mobile-conversation-launch into main 2026-04-09 22:09:55 -04:00
Conversation 0 Commits 1 Files Changed 1 +4 -15
ryan commented 2026-04-09 22:09:32 -04:00
Owner
Copy Link

Summary

  • Remove the useEffect on selectedConversation.id that caused a race condition — it fetched the still-empty conversation from the backend and wiped messages, resetting the UI back to the homepage
  • Refresh conversation list after streaming completes to pick up the auto-generated title

Test plan

  • On mobile viewport, submit a message from the homepage → should immediately show the chat view with the user's message and streaming response
  • After streaming completes, verify the sidebar shows the updated conversation title
  • Clicking an existing conversation in the sidebar still loads its messages correctly
## Summary - Remove the `useEffect` on `selectedConversation.id` that caused a race condition — it fetched the still-empty conversation from the backend and wiped messages, resetting the UI back to the homepage - Refresh conversation list after streaming completes to pick up the auto-generated title ## Test plan - [ ] On mobile viewport, submit a message from the homepage → should immediately show the chat view with the user's message and streaming response - [ ] After streaming completes, verify the sidebar shows the updated conversation title - [ ] Clicking an existing conversation in the sidebar still loads its messages correctly
ryan added 1 commit 2026-04-09 22:09:33 -04:00
Fix mobile conversation launch resetting to homepage f3ae76ce68 
Remove the useEffect on selectedConversation.id that race-conditions
with handleQuestionSubmit — it fetches the (still-empty) conversation
and wipes messages, sending the user back to the empty state. Refresh
conversation list after streaming completes instead to pick up the
auto-generated title.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
ryan merged commit 9ed4ca126a into main 2026-04-09 22:09:55 -04:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
ryan
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: ryan/simbarag#27
Delete Branch "fix/mobile-conversation-launch"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?