[ENHANCEMENT] Add queue and steer delivery modes for running conversations - #138
[ENHANCEMENT] Add queue and steer delivery modes for running conversations#138allquixotic wants to merge 2 commits into
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
This PR has had no activity for 60 days and will be automatically closed in 7 days unless there is new activity. If you'd like to continue, please rebase and leave a comment. |
|
Closing due to 60+ days of inactivity. Feel free to reopen if you'd like to resume this work. |
Related GitHub Issue
Closes: #133
Description
This PR adds explicit queue and steer delivery modes for messages sent while a conversation is running. It is stacked on the concurrent-conversations work from PR #128.
Dependency PR: #128. Please review/merge #128 first; until then this PR is intentionally opened as a stacked draft and its GitHub diff against
mainmay include the concurrent-conversations prerequisite.Key changes:
QueuedMessageDeliveryModeand extends queued-message payloads withcreatedAt,updatedAt, anddeliveryMode.MessageQueueServiceinto separatequeueandsteerlanes with steer-first combined ordering, lane-specific dequeue, delivery-mode switching, restore support, and tests.QueuedMessagesUI to show separate "Steer next" and "Queued" lanes with edit, remove, and Queue/Steer toggle actions.processQueuedMessages()method as a no-op compatibility shim so queued messages are not drained from unsafe call sites.Reviewer focus:
Test Procedure
From the Zoo Code worktree after the concurrent-conversations branch is present:
Manual verification recommended:
Pre-Submission Checklist
Screenshots / Videos
TODO: Attach a short video showing messages being queued, edited, switched between delivery modes, and delivered during a running conversation.
Documentation Updates
Yes. User-facing docs should explain the difference between queued input and steer input.
Additional Notes
This should be reviewed after or on top of PR #128. If opened before #128 merges, the GitHub diff against Zoo
mainwill include prerequisite concurrent-conversation changes.Get in Touch
Discord:
coorbin