log ping timeouts and track consecutive failures#609
log ping timeouts and track consecutive failures#609Agent-Hellboy wants to merge 3 commits intoevalstate:mainfrom
Conversation
Triage resultHuman attention: Quick readThis PR is aimed at issue #607: it makes MCP ping timeouts visible, counts consecutive failures, and escalates repeated timeouts into connection-reset behavior. IntentImprove MCP transport monitoring so ping timeouts are treated as real connection failures, with logging and reconnection after repeated failures. WhyThe reported problem is that the transport layer only handled generic connection errors, so ping-specific failures were not being tracked or escalated per the MCP guidance. The code changes are right-shaped for that goal:
Codex reviewVerdict: good_enough for the underlying bug shape, but this run could not establish the behavior end-to-end. CI/CDBlocked for this handoff step: this workspace did not expose a command execution path, so I could not run a focused repro or validate the fix behaviorally. RecommendationPlease have a human confirm the design/behavioral expectations for ping timeout escalation and then proceed with validation or merge approval. |
fixes #607