fix(gateway): send feedback when text messages are queued during active session#37114
Conversation
|
🔄 Auto-rebased onto latest origin/main (was 139 commits behind). Clean diff verified (7 files, 96 lines, no contamination). Ready for review. This replaces #31612 (auto-closed during rebase). Fix content is identical to the original; only the parent commit was changed from fork/main to current origin/main. |
|
👋 Friendly nudge — this PR is ✅ clean, small fix (1 file, +16/-0), been open since Jun 2. Adds feedback when text messages queue during active sessions. Could a maintainer take a look? |
b5e9259 to
706ae4b
Compare
|
🔄 Auto-rebased onto latest origin/main (was 109 commits behind). Clean diff verified. Ready for review. |
|
@OutThisLife — this one's been green since Jun 2. Small fix (1 file, +16/-0): adds feedback when text messages queue during an active session. Could use a merge when you get a chance. |
706ae4b to
bc59d3f
Compare
|
🔄 Auto-rebased onto latest origin/main (was 81 commits behind). Clean diff verified (7 files, 86 insertions, 10 deletions). Ready for review. |
bc59d3f to
5dc7966
Compare
|
🔄 Auto-rebased onto latest origin/main (was 43 commits behind). Clean diff verified (7 files, 86 lines). Ready for review. |
5dc7966 to
cdac2fa
Compare
|
🔄 Auto-rebased onto latest origin/main (was 66 commits behind). Clean diff verified. Ready for review. |
cdac2fa to
5d0024e
Compare
|
🔄 Auto-rebased onto latest origin/main (was 35 commits behind). Clean diff verified. Ready for review. |
5d0024e to
d8f2e77
Compare
|
🔄 Auto-rebased onto latest origin/main (was 0 commits behind). Clean diff verified. Ready for review. |
d8f2e77 to
0fd3011
Compare
|
🔄 Auto-rebased onto latest origin/main (was 40 commits behind). Clean diff verified (7 files, +86/-10). Ready for review. |
0fd3011 to
7d1a3a3
Compare
|
🔄 Auto-rebased onto latest origin/main (was 43 commits behind). Clean diff verified (7 files, 96 lines). Ready for review. |
…ve session When the agent is actively running and a user sends a text message, the message was silently queued with no indication to the user. The only way to interrupt was knowing about /stop, which is not discoverable in the moment. This fix sends a brief acknowledgment when a text message is queued: '⏳ Agent is busy — message queued for the next turn. Use /stop to interrupt.' The feedback is sent via _send_with_retry with reply_to set to the incoming message ID, so it appears as a reply on platforms that support it. Failures are silently caught and logged at debug level to avoid disrupting the main message flow. Fixes NousResearch#31588
The busy feedback feature (PR NousResearch#31612) sends a '⏳ Agent is busy' notice for ALL text messages queued during an active session. Update the three TestNonBypassStillQueued tests to expect 1 sent response instead of 0, while still verifying the message is queued as pending (not dispatched).
7d1a3a3 to
e6c2e54
Compare
|
🔄 Auto-rebased onto latest origin/main (was 48 commits behind). Clean diff verified (6 files, +83/-9). Ready for review. |
🔄 Auto-rebased onto latest origin/main (was 139 commits behind). Clean diff verified (7 files, 96 lines).
Replaces #31612, which was auto-closed when the head branch was recreated during the rebase. The fix content is identical to the original; only the parent commit has changed from fork/main to the latest origin/main.