Skip to content

Configurable per-tool timeouts - #121

Merged
Elias Posen (eliasposen) merged 2 commits into
mainfrom
ws-callback-timeouts
Jul 21, 2026
Merged

Configurable per-tool timeouts#121
Elias Posen (eliasposen) merged 2 commits into
mainfrom
ws-callback-timeouts

Conversation

@eliasposen

@eliasposen Elias Posen (eliasposen) commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Description

Callback tools were hardcoded to 30 sec timeout for responses, this default is now configurable and can be overridden on a per-tool basis

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Performance improvement
  • Code refactoring
  • Test coverage improvement

Changes Made

Testing

Test Coverage

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing performed

Test Commands Run

cargo test
cargo clippy
cargo fmt --check

Breaking Changes

  • This PR includes breaking changes
  • Migration guide updated in CHANGELOG.md

Documentation

  • Code comments added/updated
  • README.md updated (if needed)
  • CHANGELOG.md updated under "Unreleased"
  • API documentation updated

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published

Screenshots/Videos

Additional Context

@eliasposen
Elias Posen (eliasposen) merged commit 8c91d5f into main Jul 21, 2026
15 checks passed
@eliasposen
Elias Posen (eliasposen) deleted the ws-callback-timeouts branch July 21, 2026 15:19
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