Skip to content

[withdrawn] - #97749

Closed
n3k0lai wants to merge 1 commit into
NousResearch:mainfrom
n3k0lai:fix/discord-typing-closed-gate
Closed

n3k0lai wants to merge 1 commit into
NousResearch:mainfrom
n3k0lai:fix/discord-typing-closed-gate

Conversation

@n3k0lai

@n3k0lai n3k0lai commented Aug 29, 2026

Copy link
Copy Markdown

Withdrawn. Opened in error before local validation. Please ignore.

…ect it

Discord send_typing starts a persistent POST loop. After the turn ends,
stop_typing cancels it, then _stop_typing_refresh unpauses and the
progress path in gateway/run.py can send_typing again — orphaning a loop
that pulses until gateway restart (NousResearch#85427, lived on 0.20.x after check).

- Base: _typing_closed set on stop, cleared when _keep_typing starts
- Discord send_typing no-ops while closed
- finally only pops this loop (NousResearch#85425 class)
- stop_typing reaps with a 0.5s bound (NousResearch#85435 class)

Closes the recreate race; complementary to open PRs 85425/85435.
@alt-glitch alt-glitch added type/bug Something isn't working P3 Low — cosmetic, nice to have comp/gateway Gateway runner, session dispatch, delivery comp/plugins Plugin system and bundled plugins platform/discord Discord bot adapter sweeper:risk-message-delivery Sweeper risk: may drop, duplicate, misroute, or suppress messages labels Aug 29, 2026
@n3k0lai n3k0lai closed this Aug 29, 2026
@n3k0lai n3k0lai changed the title fix(discord): close the typing loop so late send_typing cannot resurrect it [withdrawn] Aug 29, 2026
@n3k0lai
n3k0lai deleted the fix/discord-typing-closed-gate branch August 29, 2026 07:29
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 comp/plugins Plugin system and bundled plugins P3 Low — cosmetic, nice to have platform/discord Discord bot adapter sweeper:risk-message-delivery Sweeper risk: may drop, duplicate, misroute, or suppress messages type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants