Merge pull request #363 from hchen2020/master

Delay persisting message to allow message content to be changed in hook.
SciSharp · Mar 26, 2024 · 71eeb01 · 71eeb01
2 parents 098aa92 + 691c3b2
commit 71eeb01
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/Infrastructure/BotSharp.Core/Conversations/Services/ConversationService.SendMessage.cs b/src/Infrastructure/BotSharp.Core/Conversations/Services/ConversationService.SendMessage.cs
@@ -32,8 +32,6 @@ public partial class ConversationService
             message.SenderId = _user.Id;
         }
 
-        _storage.Append(_conversationId, message);
-
         var conv = _services.GetRequiredService<IConversationService>();
         var dialogs = conv.GetDialogHistory();
 
@@ -72,6 +70,12 @@ public partial class ConversationService
             }
         }
 
+        // Persist to storage
+        _storage.Append(_conversationId, message);
+
+        // Add to thread
+        dialogs.Add(message);
+
         if (!stopCompletion)
         {
             // Routing with reasoning