Skip to content

Conversation

@roomote
Copy link
Contributor

@roomote roomote bot commented Dec 21, 2025

Fixes issue where messages disappear when sent while Roo is busy.

Changes:

  • Add null check for getCurrentTask() in queueMessage handler
  • Handle edge case where task finishes just as message is queued
  • Preserve user's message if queuing fails

View task on Roo Code Cloud

- Add null check in queueMessage handler to catch edge case where task ends between send click and message processing
- Send queueMessageFailed message to webview to restore user input
- Handle queueMessageFailed in ChatView to restore text and images
- Prevents user frustration from losing typed messages
@roomote
Copy link
Contributor Author

roomote bot commented Dec 21, 2025

Rooviewer Clock   See task on Roo Cloud

I reviewed this PR and found no issues. The implementation correctly handles the edge case where a task ends between when the user clicks send and when the queued message is processed.

Changes look good:

  • Adds null check for getCurrentTask() in the queueMessage handler
  • Sends queueMessageFailed message to restore user input when queueing fails
  • Properly handles the message in ChatView to restore text and images
  • Type union updated correctly in ExtensionMessage.ts

Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.

@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Dec 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels.

Projects

Status: Triage

Development

Successfully merging this pull request may close these issues.

3 participants