Skip to content

Conversation

@deansheather
Copy link
Member

@deansheather deansheather commented Nov 18, 2025

Generated with mux

Improves the bash tool timeout error message to include the actual timeout value that was exceeded, making it clearer for LLMs how to adjust their tool calls.

Before:

Command exceeded timeout

After:

Command exceeded timeout of 3 seconds. You can increase the timeout by setting the `timeout_secs` parameter on the tool call. Do not use the `timeout` bash command to increase the timeout.

The error message now:

  • Shows the actual timeout value that was exceeded
  • Explains how to increase the timeout via the timeout_secs parameter
  • Warns against using the bash timeout command as a workaround

Relates to #652

@deansheather deansheather requested a review from ammario November 18, 2025 02:46
Copy link
Member

@ammario ammario left a comment

Choose a reason for hiding this comment

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

Good approach. Way better to send the tokens once when the model transgresses vs. on every request.

@deansheather deansheather added this pull request to the merge queue Nov 19, 2025
Merged via the queue into main with commit 6c3b655 Nov 19, 2025
13 checks passed
@deansheather deansheather deleted the improve-bash-timeout-error branch November 19, 2025 07:26
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