openclaw - 💡(How to fix) Fix Feishu: ackReaction removed before reply message is sent, causing user confusion [1 comments, 1 participants]

TL;DR

Modify the removeAckReactionAfterReply() function in ack-reactions.ts to only remove the ackReaction after the reply message has been sent.

Guidance

• Check the removeAckAfterReply flag support for each channel and only call removeAckReactionAfterReply() if the flag is supported and set to true.
• Consider adding a delay between sending the reply message and removing the ackReaction to ensure the user sees the reaction until the reply arrives.
• Review the message handling loop to ensure it correctly handles the timing of sending reply messages and removing ackReactions.
• Investigate adding a try-catch block around the code that sends the reply message to handle any errors that may occur, preventing the ackReaction from being removed prematurely.

Example

if (channel.supportsRemoveAckAfterReply && removeAckAfterReply) {
  // Send reply message
  sendMessage(reply);
  // Remove ackReaction after reply message has been sent
  removeAckReactionAfterReply();
} else {
  // Handle channels that do not support removeAckAfterReply
  // ...
}

Notes

The provided solution assumes that the removeAckAfterReply flag is correctly set for each channel and that the removeAckReactionAfterReply() function is the only place where the ackReaction is removed. Additional debugging may be necessary to ensure the correct behavior.

Recommendation

Apply workaround: Modify the removeAckReactionAfterReply() function to correctly handle the timing of removing the ackReaction, as this will provide a more accurate "message received" visual confirmation to users.