8.3 KiB
status, contact, date, deciders, consulted, informed
| status | contact | date | deciders | consulted | informed |
|---|---|---|---|---|---|
| accepted | dmytrostruk | 2023-12-08 | SergeyMenshykh, markwallace, rbarreto, mabolan, stephentoub, dmytrostruk |
Chat Models
Context and Problem Statement
In latest OpenAI API, content property of chat message object can accept two types of values string or array (Documentation).
We should update current implementation of ChatMessageContent class with string Content property to support this API.
Decision Drivers
- New design should not be coupled to OpenAI API and should work for other AI providers.
- Naming of classes and properties should be consistent and intuitive.
Considered Options
Some of the option variations can be combined.
Option #1: Naming updates and new data type for chat message content
Since chat message content can be an object now instead of string, it requires reserved name for better understanding in domain.
ChatMessageContentwill be renamed toChatMessage. (Same forStreamingChatMessageContent).GetChatMessageContentmethods will be renamed toGetChatMessage.- New abstract class
ChatMessageContentthat will have propertyChatMessageContentType Typewith valuestext,image. (Will be extended withaudio,videoin the future). ChatMessagewill contain collection ofChatMessageContentobjectsIList<ChatMessageContent> Contents.- There will be concrete implementations of
ChatMessageContent-ChatMessageTextContentandChatMessageImageContent.
New ChatMessageContentType.cs
public readonly struct ChatMessageContentType : IEquatable<ChatMessageContentType>
{
public static ChatMessageContentType Text { get; } = new("text");
public static ChatMessageContentType Image { get; } = new("image");
public string Label { get; }
// Implementation of `IEquatable`...
}
New ChatMessageContent.cs
public abstract class ChatMessageContent
{
public ChatMessageContentType Type { get; set; }
public ChatMessageContent(ChatMessageContentType type)
{
this.Type = type;
}
}
Updated ChatMessage.cs:
public class ChatMessage : ContentBase
{
public AuthorRole Role { get; set; }
public IList<ChatMessageContent> Contents { get; set; }
New ChatMessageTextContent.cs
public class ChatMessageTextContent : ChatMessageContent
{
public string Text { get; set; }
public ChatMessageTextContent(string text) : base(ChatMessageContentType.Text)
{
this.Text = text;
}
}
New ChatMessageImageContent.cs
public class ChatMessageImageContent : ChatMessageContent
{
public Uri Uri { get; set; }
public ChatMessageImageContent(Uri uri) : base(ChatMessageContentType.Image)
{
this.Uri = uri;
}
}
Usage:
var chatHistory = new ChatHistory("You are friendly assistant.");
// Construct request
var userContents = new List<ChatMessageContent>
{
new ChatMessageTextContent("What's in this image?"),
new ChatMessageImageContent(new Uri("https://link-to-image.com"))
};
chatHistory.AddUserMessage(userContents);
// Get response
var message = await chatCompletionService.GetChatMessageAsync(chatHistory);
foreach (var content in message.Contents)
{
// Possibility to get content type (text or image).
var contentType = content.Type;
// Cast for specific content type
// Extension methods can be provided for better usability
// (e.g. message GetContent<ChatMessageTextContent>()).
if (content is ChatMessageTextContent textContent)
{
Console.WriteLine(textContent);
}
if (content is ChatMessageImageContent imageContent)
{
Console.WriteLine(imageContent.Uri);
}
}
Option #2: Avoid renaming and new data type for chat message content
Same as Option #1, but without naming changes. In order to differentiate actual chat message and chat message content:
Chat Messagewill beChatMessageContent(as it is right now).Chat Message Contentwill beChatMessageContentItem.
- New abstract class
ChatMessageContentItemthat will have propertyChatMessageContentItemType Typewith valuestext,image. (Will be extended withaudio,videoin the future). ChatMessageContentwill contain collection ofChatMessageContentItemobjectsIList<ChatMessageContentItem> Items.- There will be concrete implementations of
ChatMessageContentItem-ChatMessageTextContentItemandChatMessageImageContentItem.
New ChatMessageContentItemType.cs
public readonly struct ChatMessageContentItemType : IEquatable<ChatMessageContentItemType>
{
public static ChatMessageContentItemType Text { get; } = new("text");
public static ChatMessageContentItemType Image { get; } = new("image");
public string Label { get; }
// Implementation of `IEquatable`...
}
New ChatMessageContentItem.cs
public abstract class ChatMessageContentItem
{
public ChatMessageContentItemType Type { get; set; }
public ChatMessageContentItem(ChatMessageContentItemType type)
{
this.Type = type;
}
}
Updated ChatMessageContent.cs:
public class ChatMessageContent : ContentBase
{
public AuthorRole Role { get; set; }
public IList<ChatMessageContentItem> Items { get; set; }
New ChatMessageTextContentItem.cs
public class ChatMessageTextContentItem : ChatMessageContentItem
{
public string Text { get; set; }
public ChatMessageTextContentItem(string text) : base(ChatMessageContentType.Text)
{
this.Text = text;
}
}
New ChatMessageImageContent.cs
public class ChatMessageImageContentItem : ChatMessageContentItem
{
public Uri Uri { get; set; }
public ChatMessageImageContentItem(Uri uri) : base(ChatMessageContentType.Image)
{
this.Uri = uri;
}
}
Usage:
var chatHistory = new ChatHistory("You are friendly assistant.");
// Construct request
var userContentItems = new List<ChatMessageContentItem>
{
new ChatMessageTextContentItem("What's in this image?"),
new ChatMessageImageContentItem(new Uri("https://link-to-image.com"))
};
chatHistory.AddUserMessage(userContentItems);
// Get response
var message = await chatCompletionService.GetChatMessageContentAsync(chatHistory);
foreach (var contentItem in message.Items)
{
// Possibility to get content type (text or image).
var contentItemType = contentItem.Type;
// Cast for specific content type
// Extension methods can be provided for better usability
// (e.g. message GetContent<ChatMessageTextContentItem>()).
if (contentItem is ChatMessageTextContentItem textContentItem)
{
Console.WriteLine(textContentItem);
}
if (contentItem is ChatMessageImageContentItem imageContentItem)
{
Console.WriteLine(imageContentItem.Uri);
}
}
Option #3: Add new property to ChatMessageContent - collection of content items
This option will keep string Content property as it is, but will add new property - collection of ContentBase items.
Updated ChatMessageContent.cs
public class ChatMessageContent : ContentBase
{
public AuthorRole Role { get; set; }
public string? Content { get; set; }
public ChatMessageContentItemCollection? Items { get; set; }
}
New ChatMessageContentItemCollection.cs
public class ChatMessageContentItemCollection : IList<ContentBase>, IReadOnlyList<ContentBase>
{
// Implementation of IList<ContentBase>, IReadOnlyList<ContentBase> to catch null values.
}
Usage:
var chatCompletionService = kernel.GetRequiredService<IChatCompletionService>();
var chatHistory = new ChatHistory("You are a friendly assistant.");
chatHistory.AddUserMessage(new ChatMessageContentItemCollection
{
new TextContent("What’s in this image?"),
new ImageContent(new Uri(ImageUri))
});
var reply = await chatCompletionService.GetChatMessageContentAsync(chatHistory);
Console.WriteLine(reply.Content);
Decision Outcome
Option #3 was preferred as it requires small amount of changes to existing hierarchy and provides clean usability for end-user.
