refine update conv tags

Author: Jicheng Lu

src/Infrastructure/BotSharp.Abstraction/Conversations/ConversationHookBase.cs

@@ -79,6 +79,6 @@ public virtual Task OnBreakpointUpdated(string conversationId, bool resetStates)
     public virtual Task OnNotificationGenerated(RoleDialogModel message)
         => Task.CompletedTask;
 
-    public Task OnUserDisconnected(Conversation conversation)
+    public virtual Task OnUserDisconnected(Conversation conversation)
         => Task.CompletedTask;
 }

src/Infrastructure/BotSharp.Core/Repository/FileRepository/FileRepository.Conversation.cs

@@ -171,7 +171,8 @@ public bool UpdateConversationTags(string conversationId, List<string> toAddTags
         var json = File.ReadAllText(convFile);
         var conv = JsonSerializer.Deserialize<Conversation>(json, _options);
 
-        var tags = conv.Tags.Concat(toAddTags).ToList();
+        var tags = conv.Tags ?? [];
+        tags = tags.Concat(toAddTags).Distinct().ToList();
         conv.Tags = tags.Where(x => !toDeleteTags.Contains(x, StringComparer.OrdinalIgnoreCase)).ToList();
 
         conv.UpdatedTime = DateTime.UtcNow;

src/Plugins/BotSharp.Plugin.MongoStorage/Repository/MongoRepository.Conversation.cs

@@ -142,10 +142,12 @@ public bool UpdateConversationTags(string conversationId, List<string> toAddTags
         var conv = _dc.Conversations.Find(filter).FirstOrDefault();
         if (conv == null) return false;
 
-
+        var tags = conv.Tags ?? [];
+        tags = tags.Concat(toAddTags).Distinct().ToList();
+        tags = tags.Where(x => !toDeleteTags.Contains(x, StringComparer.OrdinalIgnoreCase)).ToList();
 
         var update = Builders<ConversationDocument>.Update
-                                                   .Set(x => x.Tags, tags ?? new())
+                                                   .Set(x => x.Tags, tags)
                                                    .Set(x => x.UpdatedTime, DateTime.UtcNow);
 
         var res = _dc.Conversations.UpdateOne(filter, update);