* fix(python): add timeout mechanism to ConnectionManager.stop()
Add timeout parameter (default: 30s) to stop() method to prevent infinite hangs
during application shutdown. Resolves issue #562.
Changes:
- Add timeout parameter to stop() with 30 second default
- Wrap task wait with asyncio.wait_for() to enforce timeout
- Wrap cleanup event wait with asyncio.wait_for() to enforce timeout
- Force cleanup on timeout by clearing connection and setting done event
- Add appropriate warning/error logging on timeout
Tests:
- Add 10 comprehensive unit tests for timeout scenarios
- All 158 unit tests pass
- Manual end-to-end testing verified
This ensures graceful shutdown even when cleanup hangs in production.
* updated with copilot suggested edits
* ruff checks passed
* Improve timeout handling in ConnectionManager.stop()
- Move 'import time' to module level
- Use CancelledError consistently (Python 3.11+ style)
- Treat timeout <= 0 or None as infinite wait (no timeout)
- Add elapsed time to timeout warning logs
- Add tests for negative/zero/None timeout behavior
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
* docs: update ConnectionManager.stop() API reference with timeout parameter
---------
Co-authored-by: pietrozullo <pietro.zullo@gmail.com>
Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
Inchara J