Better behavour for anthropic with multiple messages and mixed bot us…

…age (#235)

server/ai/anthropic/anthropic.go

@@ -32,17 +32,32 @@ func conversationToMessages(conversation ai.BotConversation) (string, []InputMes
 	systemMessage := ""
 	messages := make([]InputMessage, 0, len(conversation.Posts))
 	for _, post := range conversation.Posts {
+		previousRole := ""
+		previousContent := ""
+		if len(messages) > 0 {
+			previous := messages[len(messages)-1]
+			previousRole = previous.Role
+			previousContent = previous.Content
+		}
 		switch post.Role {
 		case ai.PostRoleSystem:
 			systemMessage += post.Message
 		case ai.PostRoleBot:
+			if previousRole == RoleAssistant {
+				previousContent += post.Message
+				continue
+			}
 			messages = append(messages,
 				InputMessage{
 					Role:    RoleAssistant,
 					Content: post.Message,
 				},
 			)
 		case ai.PostRoleUser:
+			if previousRole == RoleUser {
+				previousContent += post.Message
+				continue
+			}
 			messages = append(messages,
 				InputMessage{
 					Role:    RoleUser,

server/ai/conversation.go

@@ -179,13 +179,6 @@ func (b *BotConversation) Truncate(maxTokens int, countTokens func(string) int)
 	return false
 }
 
-func GetPostRole(botID string, post *model.Post) PostRole {
-	if post.UserId == botID {
-		return PostRoleBot
-	}
-	return PostRoleUser
-}
-
 func FormatPostBody(post *model.Post) string {
 	attachments := post.Attachments()
 	if len(attachments) > 0 {

server/post_processing.go

@@ -404,8 +404,13 @@ func (p *Plugin) PostToAIPost(bot *Bot, post *model.Post) ai.Post {
 		}
 	}
 
+	role := ai.PostRoleUser
+	if p.IsAnyBot(post.UserId) {
+		role = ai.PostRoleBot
+	}
+
 	return ai.Post{
-		Role:    ai.GetPostRole(bot.mmBot.UserId, post),
+		Role:    role,
 		Message: ai.FormatPostBody(post),
 		Files:   files,
 	}