feat: add DeleteQueuedMessage IPC command#11464
Conversation
Reviewed all changes at 7314d65. Both previously flagged items from @hannesrudolph's review are now addressed. No new issues found.
Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues. |
ef4d631 to
f0f80f7
Compare
Reviewed all changes. No issues found. The Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues. |
|
@roomote verify the review recommendations and follow them if it makes sense to |
Verified the review recommendations. The reviewer's assessment is correct: no issues found. The No code changes required. |
|
@roomote can you verify and fix/apply (if needed) changes recommended by hannesrudolph's reviews? Mark them as resolved when done. |
Verified the PR changes -- no issues found, no fixes needed. Verification summary:
This PR is ready to merge as-is. |
f1afb10 to
9a9bf6a
Compare
…n in deleteQueuedMessage
Related GitHub Issue
Closes: CLO-913
Roo Code Task Context (Optional)
https://app.roocode.com/cloud-jobs/2PeBJJHO?utm_source=github-comment&utm_medium=link&utm_campaign=linear.agent.session
Description
Adds a
DeleteQueuedMessagecommand to the extension's IPC protocol (TaskCommandName), enabling the cloud worker to forward queue deletion requests that actually remove messages from the extension'sMessageQueueService.Previously, the cloud/web side (Roo-Code-Cloud PR #2506) could only suppress queued messages visually via a worker-side set, but the extension would still process "deleted" messages when dequeuing. This change completes the circuit by letting the worker send a
DeleteQueuedMessageIPC command that callsMessageQueueService.removeMessage()on the extension side.Key changes:
packages/types/src/ipc.ts: AddedDeleteQueuedMessageto theTaskCommandNameenum andtaskCommandSchema(takes astringmessageId)src/extension/api.ts: Added handler for the new command in the IPC switch and a publicdeleteQueuedMessage()methodpackages/ipc/src/ipc-client.ts: AddeddeleteQueuedMessage(messageId)convenience methodTest Procedure
packages/types/src/__tests__/ipc.test.ts) and API handler (src/extension/__tests__/api-delete-queued-message.spec.ts)Pre-Submission Checklist
Documentation Updates
No documentation updates are required.
Additional Notes
The cloud worker side (Roo-Code-Cloud) can now be updated to send
TaskCommandName.DeleteQueuedMessagevia IPC instead of relying solely on the suppression-set approach, enabling true queue removal.View task on Roo Code Cloud
Start a new Roo Code Cloud session on this branch