Skip to content

fix(weixin): avoid cross-loop send session reuse - #16540

Closed
yannisxu wants to merge 1 commit into
NousResearch:mainfrom
yannisxu:fix/weixin-cross-loop-send-session-reuse
Closed

fix(weixin): avoid cross-loop send session reuse#16540
yannisxu wants to merge 1 commit into
NousResearch:mainfrom
yannisxu:fix/weixin-cross-loop-send-session-reuse

Conversation

@yannisxu

Copy link
Copy Markdown

Bug Description

Sending messages from Hermes to Weixin could fail intermittently when an async tool execution ran on a different event loop than the live Weixin adapter session.

Root Cause

The live Weixin adapter kept a long-lived aiohttp send session, but `send_

@alt-glitch alt-glitch added type/bug Something isn't working P2 Medium — degraded but workaround exists comp/gateway Gateway runner, session dispatch, delivery platform/wecom WeCom / WeChat Work adapter labels Apr 27, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Likely duplicate of #13520 (and #14384, #13350, #14873) — same root cause: cross-event-loop aiohttp session reuse in Weixin adapter send path.

@teknium1

Copy link
Copy Markdown
Contributor

Thanks for the focused diagnosis. This is already implemented on current main.

Automated hermes-sweeper review verified:

Closing as implemented on main.

@teknium1 teknium1 closed this Jul 12, 2026
@teknium1 teknium1 added the sweeper:implemented-on-main Sweeper: behavior already present on current main label Jul 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/gateway Gateway runner, session dispatch, delivery P2 Medium — degraded but workaround exists platform/wecom WeCom / WeChat Work adapter sweeper:implemented-on-main Sweeper: behavior already present on current main type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants