fix(chat): don't send on Enter while the agent is running#100
Closed
viniciussouzax wants to merge 1 commit into
Closed
fix(chat): don't send on Enter while the agent is running#100viniciussouzax wants to merge 1 commit into
viniciussouzax wants to merge 1 commit into
Conversation
Pressing Enter while a response was streaming called sendMessage() unconditionally. A new message aborts the in-flight query (chat-bridge stopSession), so a user's follow-up (often an impatient "are you there?") silently killed the very answer they were waiting for, which surfaced as the recurring "Claude Code process aborted by user" message. Guard the Enter handler with `status !== 'running'`, mirroring the Send button that is already disabled through `canSend` while the agent is running. Shift+Enter (newline) is unaffected. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
There was a problem hiding this comment.
Sorry @viniciussouzax, you have reached your weekly rate limit of 500000 diff characters.
Please try again later or upgrade to continue using Sourcery
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
While an agent response is streaming, pressing Enter sends the typed text
anyway. Sending a new message aborts the in-flight query (chat-bridge
stopSession), so a user's follow-up — often an impatient "are you there?" —silently kills the very answer they were waiting for. This shows up as the
recurring "Claude Code process aborted by user" message and the impression
that the chat is broken.
Root cause
The keydown handler in
AgentChat.tsxcallssendMessage()unconditionally:The Send button is already guarded — it's disabled via
canSend = ... && status !== 'running'— but the Enter shortcut bypasses thatguard entirely.
Fix
Mirror the button's guard in the Enter handler:
Shift+Enter(newline) is unaffected.Testing
response is aborted ("Claude Code process aborted by user").
button); the streaming response completes normally. Sending works as before
once the agent is idle.