Skip to content

fix: isolate New Chat from in-progress web sessions#788

Merged
SamSaffron merged 1 commit into
SamSaffron:mainfrom
sam-saffron-jarvis:feat/web-new-chat-reset
Jun 9, 2026
Merged

fix: isolate New Chat from in-progress web sessions#788
SamSaffron merged 1 commit into
SamSaffron:mainfrom
sam-saffron-jarvis:feat/web-new-chat-reset

Conversation

@sam-saffron-jarvis

Copy link
Copy Markdown
Contributor

What

Fix New Chat state leakage from in-progress web sessions.

  • Treat a send as an interjection only when the currently active, non-draft session owns the busy state.
  • Keep inactive session active_run status in the sidebar without flipping the global draft composer into streaming mode.
  • Add regression coverage for:
    • New Chat draft sends with stale global streaming=true
    • late active-run sync responses after switching to New Chat

Verification

mise exec node@24 -- node internal/serveui/static/app_stream_test.js
mise exec node@24 -- node internal/serveui/static/app_sessions_test.js
go test ./...
go build ./...

@SamSaffron SamSaffron merged commit a51f13b into SamSaffron:main Jun 9, 2026
1 check 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.

2 participants