Skip to content

fix: match exact cancellation sentinel in BashTool UI#172

Open
winterqt wants to merge 1 commit intoboldsoftware:mainfrom
winterqt:fix/bash-cancelled-false-positive
Open

fix: match exact cancellation sentinel in BashTool UI#172
winterqt wants to merge 1 commit intoboldsoftware:mainfrom
winterqt:fix/bash-cancelled-false-positive

Conversation

@winterqt
Copy link
Copy Markdown
Contributor

@winterqt winterqt commented Apr 3, 2026

The 'cancelled' badge was shown for any bash error containing the word 'cancel' (case-insensitive), which false-positives on errors like 'context canceled'. Now we match the exact sentinel string 'Tool execution cancelled by user' that the server records on user-initiated cancellation.

The 'cancelled' badge was shown for any bash error containing the word
'cancel' (case-insensitive), which false-positives on errors like
'context canceled'. Now we match the exact sentinel string
'Tool execution cancelled by user' that the server records on
user-initiated cancellation.

Co-authored-by: Shelley <shelley@exe.dev>
@cla-bot cla-bot bot added the cla-signed label Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant