Space multi-line command replies 5 s apart (v0.21.5) #48

Merged
kamilkrzywda merged 1 commits from fix/multi-send-spacing into main 2026-09-26 12:31:22 +02:00
Owner

#rss get lublin sent its 4 items back-to-back within the same second and 3 of 4 were dropped in RF — owner saw only the last one ("Culture:...").

  • CommandService's array-of-lines loop (the shared multi-send path used by #rss get) now awaits a delay between consecutive lines before respond()/sendText
  • New env MESH_MULTI_SEND_DELAY_MS, default 5000 ms; no delay before the first line; zero value skips the await entirely

Tests 172 → 173 (3-line reply with 10 ms override: all sent, first out immediately, gaps ≥10 ms measured). Build clean, oxlint clean. Lockfile untouched.

`#rss get lublin` sent its 4 items back-to-back within the same second and 3 of 4 were dropped in RF — owner saw only the last one ("Culture:..."). - CommandService's array-of-lines loop (the shared multi-send path used by `#rss get`) now awaits a delay between consecutive lines before respond()/sendText - New env MESH_MULTI_SEND_DELAY_MS, default **5000** ms; no delay before the first line; zero value skips the await entirely Tests 172 → 173 (3-line reply with 10 ms override: all sent, first out immediately, gaps ≥10 ms measured). Build clean, oxlint clean. Lockfile untouched.
kamilkrzywda added 1 commit 2026-09-26 12:31:15 +02:00
Production incident: '#rss get lublin' returned an array of lines and
CommandService sent each one back-to-back within the same second — on the real
mesh 3 of 4 items were dropped in RF (the owner saw only the last one).

CommandService.onRx now waits MESH_MULTI_SEND_DELAY_MS (new env, default 5000 =
5 s, read at construction like the other timing knobs) between consecutive
lines of an array reply — no delay before the first line. The delay sits in the
shared per-line loop, so both the respond() path and the legacy sendText
fallback are covered by one await.
kamilkrzywda merged commit 847c3d47d0 into main 2026-09-26 12:31:22 +02:00
kamilkrzywda deleted branch fix/multi-send-spacing 2026-09-26 12:31:22 +02:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: OpenCode/MeshBot#48