Repository navigation
Conversation
The second Escape reset the footer straight back to "esc interrupt", and nothing else changed until the server finished cancelling the turn and the session status went idle. Under load that gap is long enough that the press looks ignored, so users keep pressing. The double press now sets an "aborting…" state for the session before the abort request is sent, shown in place of the interrupt hint. It clears when the session goes idle. If the request fails it clears with an error toast, and if the session is still running 15s later it clears with a warning so the user knows to press again. AI-Tool: opencode 1.18.34-vt-126-907b3bc518 AI-Model: anthropic/claude-opus-5-5 AI-Variant: high AI-Platform: linux AI-Harness: Vibeterm 52e9db3 AI-Session-ID: ses_eec446a24ffebz6buEl7LB2m3O
Contributor
|
The following comment was made by an LLM, it may be inaccurate: |
4 of 6 tasks
This branch has not been deployed
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
1 participant
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.
Issue for this PR
Closes #53652
Type of change
What does this PR do?
When the double Escape fires, it now sets an
aborting…state for the session before sending the abort request. That state replaces theesc interrupthint in the footer:The state is keyed by session ID, so switching sessions doesn't carry it over. The change is limited to
component/prompt/index.tsx. The first press already had feedback (esc again to interrupt) and is unchanged.How did you verify your code works?
bun typecheckandbun testinpackages/tui.Ran the TUI from source with isolated XDG dirs in a throwaway tmux server, against a local fake OpenAI-compatible provider that streams a long reply and logs when the client disconnects. A script sent the keys and polled the pane every 10ms. It ran with and without four CPU hogs pinned to the same cores as the TUI.
aborting…: 54-168msScreenshots / recordings
Footer captured on the first frame after the second Escape:
Checklist