Apply suggestions from code review

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This commit is contained in:
westey
2025-11-27 18:16:21 +00:00
committed by GitHub
co-authored by Copilot
parent d9ce6ee292
commit 938961d2cc
3 changed files with 3 additions and 3 deletions
@@ -151,7 +151,7 @@ public sealed class ContextualFunctionProvider : AIContextProvider
return await this._options.ContextEmbeddingValueProvider.Invoke(recentMessages, newMessages, cancellationToken).ConfigureAwait(false);
}
// Build context from the recent messages that already include the new messages
// Build context by concatenating the recent messages and the new messages
return string.Join(
Environment.NewLine,
this._recentMessages
@@ -20,7 +20,7 @@ public sealed class ContextualFunctionProviderOptions
public string? CollectionName { get; set; }
/// <summary>
/// Gets or sets the number of recent messages(messages from previous model/agent invocations) the provider uses to form a context.
/// Gets or sets the number of recent messages (messages from previous model/agent invocations) the provider uses to form a context.
/// The provider collects all messages from all model/agent invocations, up to this number,
/// and prepends them to the new messages of the current model/agent invocation to build a context.
/// While collecting new messages, the provider will remove the oldest messages
@@ -58,7 +58,7 @@ public sealed class FunctionStoreTests
}
[Fact]
public async Task SaveAsync_ShouldUpsertFunctions_Async()
public async Task SaveAsync_ShouldUpsertFunctionsAsync()
{
// Arrange
var functions = new List<AIFunction>