[Skype] Group conversations: mention is not removed from Text in RemoveRecipientMention

[nrobert]

System Information

• SDK Language: C#
• SDK Version: BotBuilder 3.8.3 (latest NuGet package)
• Development Environment: Localhost.
• Channel: Bug on Skype, OK on Slack

Issue Description

There is a bug on the way the Mentions are managed in Skype that is linked to the <at> tag that is added in the text of the message.

When you want to get the text of a message without the recipient mention by calling RemoveRecipientMention, the mention will not be removed from the text.

Code Example

The following code is used for the demo below:

var incomingMessage = context.Activity.AsMessageActivity();

var mentionFound = (incomingMessage.Entities?.Where(entity => String.Compare(entity.Type, "mention", ignoreCase: true) == 0).Select(e => e.Properties.ToObject<Mention>()).ToArray() ?? new Mention[0]).Where(mention => mention.Mentioned.Id == context.Activity.Recipient.Id).FirstOrDefault();
if (mentionFound != null)
{ 
    await context.PostAsync($"Activity.Text= {incomingMessage.Text}" +
        $"\n\nMention.Text= {mentionFound.Text}" +
        $"\n\nAfter RemoveRecipientMention()= { incomingMessage.RemoveRecipientMention() }" +
        $"\n\n\n\nChannelData= { Newtonsoft.Json.JsonConvert.SerializeObject(incomingMessage.ChannelData).Replace("<", "&lt;").Replace(">", "&gt;") }" +
        $"\n\nMention json= {Newtonsoft.Json.JsonConvert.SerializeObject(mentionFound).Replace("<", "&lt;").Replace(">", "&gt;")}"
        );
}

Steps to Reproduce the Issue

1. Deploy your bot on Skype
2. Connect to https://web.skype.com/en/?ecsoverride=developer (or use Skype App but in that case put your mention in the beginning of the message)
3. Create a group conversation
4. In this conversation, type a message with a mention to the bot
5. In the bot code treating this message, call RemoveRecipientMention() on the messageText

Expected Behavior

The Text should not contain the bot name mentioned on Skype when using RemoveRecipientMention() on the message text.

Actual Results

Demo:
Valid on Slack (mention removed successfully):

Invalid on Skype:

The problem seems to be the mismatch between the Activity.Text where the <at> tag is not present and the Mention.Text which contains it.
As a consequence, the method Regex called in RemoveRecipientMention cannot do its work.

[nrobert]

I have a doubt on where this bug should be fixed: can you confirm that it is more a Bot Connector problem than on Bot Builder side?

[helgeh]

This seems to be an issue with the Node SDK as well. I can not get the bot to work as expected in a group conversation on Skype. The bot recieves messages with the "@botName" prepended and changing the text property of the session/context in a middleware does not help.

[helgeh]

Sorry, my bad. I was actually able to strip the mention after all.

[fanidamj-zz]

Closing the issue assuming it as resolved. Please reopen a new issue if it still persists.

[nrobert]

Why should it be resolved without testing and verifying it is?
Please, issues should not be closed if issue is not checked

[tgckpg]

This is NOT resolved. I can reproduce this issue.

Edit [12 July, 2018]:
Here's more information I've gathered.

This usually happens on UWP Skype. When you first add the bot to your contact. The mention name would be shown as a long string of uuid, something like this: c0074553-463e-4056-ad3b-1d5a51541618. The uuid has not been removed.

After a while, it would sometimes be the bot's resource name. The resources name is also not replaced. It looked something like this: test_AADnLytkwbB3YXKf