Skip to content

feat: Add WebSocket heartbeat configuration support with enhanced logging - #118

Closed
Million-mo wants to merge 5 commits into
develop/agenticfrom
feat/websocket-heartbeat-config
Closed

feat: Add WebSocket heartbeat configuration support with enhanced logging#118
Million-mo wants to merge 5 commits into
develop/agenticfrom
feat/websocket-heartbeat-config

Conversation

@Million-mo

Copy link
Copy Markdown
Owner

Summary

Add configurable WebSocket heartbeat support with enhanced logging and MCP timeout improvements.

Changes

  • feat: Add WebSocket heartbeat configuration support — Enable configurable heartbeat intervals for WebSocket connections
  • feat: Enhance WebSocket heartbeat logging with detailed info and debug messages — Improve observability of heartbeat mechanism
  • fix(acp): increase timeout for mcp/connect and mcp/message to 300 seconds — Prevent timeout issues with slow MCP connections
  • fix(acp): increase default read timeout for MCP transport and session handling — Increase default read timeout to 600s for MCP transport

备份迁移自 SRC-PR#51 · 作者 @Leoyzen · 创建于 2026-06-22T02:40:10Z · head=feat/websocket-heartbeat-config base=develop/agentic
源状态: merged · merge_commit_sha=1557dda502c781f0473b0c48610947842e7480eb

A-Qiang and others added 5 commits June 22, 2026 10:48
- Add heartbeat parameters (ping_interval, pong_timeout, max_missed_pongs) to transport classes
- Implement _websocket_heartbeat function for proactive connection monitoring
- Update CLI with --ws-ping-interval, --ws-pong-timeout, --ws-max-missed-pongs flags
- Add ACPPoolServerConfig heartbeat fields for YAML configuration
- Integrate heartbeat logic into _serve_websocket and _serve_streamable_http
- Support disabling heartbeat via ping_interval=None

Defaults: ping_interval=60s, pong_timeout=30s, max_missed_pongs=3
…duce MCP transport fallback timeout

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
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.

3 participants