Skip to content

fix - only destroy subagent conversation, not the main conversation#174

Merged
jdneo merged 1 commit into
mainfrom
tori/destroy-regression
May 12, 2026
Merged

fix - only destroy subagent conversation, not the main conversation#174
jdneo merged 1 commit into
mainfrom
tori/destroy-regression

Conversation

@xinyi-gong

Copy link
Copy Markdown
Member

fix #173
Previously, clicking cancel (stop button) called conversation/destroy on both the main conversation and the subagent conversation. This caused the main conversation to be removed from CLS memory, making it impossible to continue the conversation after cancel.

Copilot AI review requested due to automatic review settings May 12, 2026 03:03

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Fixes a regression where pressing the stop/cancel button destroyed the main CLS conversation, preventing the user from continuing the same chat session afterward (issue #173).

Changes:

  • Remove conversation/destroy call for the main conversationId when canceling.
  • Keep conversation/destroy for the temporary subagentConversationId on cancel.
  • Preserve/persist local conversation state while canceling in-flight futures and resetting UI state.

@jdneo jdneo merged commit b303976 into main May 12, 2026
8 checks passed
@jdneo jdneo deleted the tori/destroy-regression branch May 12, 2026 09:16
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.

[Bug] Regression: cancelled conversation cannot be continued.

3 participants