mirror of
https://github.com/microsoft/agent-framework.git
synced 2026-06-16 21:04:09 +08:00
c9321b9028
* Initial plan * Allow developer to specify custom formatter for ToolResultCompactionStrategy Co-authored-by: crickman <66376200+crickman@users.noreply.github.com> * Refine shape * Fix test expectation * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: crickman <66376200+crickman@users.noreply.github.com> Co-authored-by: Chris Rickman <crickman@microsoft.com> Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
439 lines
16 KiB
C#
439 lines
16 KiB
C#
// Copyright (c) Microsoft. All rights reserved.
|
|
|
|
using System;
|
|
using System.Collections.Generic;
|
|
using System.Threading.Tasks;
|
|
using Microsoft.Agents.AI.Compaction;
|
|
using Microsoft.Extensions.AI;
|
|
|
|
namespace Microsoft.Agents.AI.UnitTests.Compaction;
|
|
|
|
/// <summary>
|
|
/// Contains tests for the <see cref="ToolResultCompactionStrategy"/> class.
|
|
/// </summary>
|
|
public class ToolResultCompactionStrategyTests
|
|
{
|
|
[Fact]
|
|
public async Task CompactAsyncTriggerNotMetReturnsFalseAsync()
|
|
{
|
|
// Arrange — trigger requires > 1000 tokens
|
|
ToolResultCompactionStrategy strategy = new(CompactionTriggers.TokensExceed(1000));
|
|
|
|
ChatMessage toolCall = new(ChatRole.Assistant, [new FunctionCallContent("call1", "get_weather")]);
|
|
ChatMessage toolResult = new(ChatRole.Tool, "Sunny");
|
|
|
|
CompactionMessageIndex groups = CompactionMessageIndex.Create(
|
|
[
|
|
new ChatMessage(ChatRole.User, "What's the weather?"),
|
|
toolCall,
|
|
toolResult,
|
|
]);
|
|
|
|
// Act
|
|
bool result = await strategy.CompactAsync(groups);
|
|
|
|
// Assert
|
|
Assert.False(result);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task CompactAsyncCollapsesOldToolGroupsAsync()
|
|
{
|
|
// Arrange — always trigger
|
|
ToolResultCompactionStrategy strategy = new(
|
|
trigger: _ => true,
|
|
minimumPreservedGroups: 1);
|
|
|
|
CompactionMessageIndex groups = CompactionMessageIndex.Create(
|
|
[
|
|
new ChatMessage(ChatRole.User, "Q1"),
|
|
new ChatMessage(ChatRole.Assistant, [new FunctionCallContent("call1", "get_weather")]),
|
|
new ChatMessage(ChatRole.Tool, "Sunny and 72°F"),
|
|
new ChatMessage(ChatRole.User, "Q2"),
|
|
]);
|
|
|
|
// Act
|
|
bool result = await strategy.CompactAsync(groups);
|
|
|
|
// Assert
|
|
Assert.True(result);
|
|
|
|
List<ChatMessage> included = [.. groups.GetIncludedMessages()];
|
|
// Q1 + collapsed tool summary + Q2
|
|
Assert.Equal(3, included.Count);
|
|
Assert.Equal("Q1", included[0].Text);
|
|
Assert.Equal("[Tool Calls]\nget_weather:\n - Sunny and 72°F", included[1].Text);
|
|
Assert.Equal("Q2", included[2].Text);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task CompactAsyncPreservesRecentToolGroupsAsync()
|
|
{
|
|
// Arrange — protect 2 recent non-system groups (the tool group + Q2)
|
|
ToolResultCompactionStrategy strategy = new(
|
|
trigger: _ => true,
|
|
minimumPreservedGroups: 3);
|
|
|
|
CompactionMessageIndex groups = CompactionMessageIndex.Create(
|
|
[
|
|
new ChatMessage(ChatRole.User, "Q1"),
|
|
new ChatMessage(ChatRole.Assistant, [new FunctionCallContent("call1", "search")]),
|
|
new ChatMessage(ChatRole.Tool, "Results"),
|
|
new ChatMessage(ChatRole.User, "Q2"),
|
|
]);
|
|
|
|
// Act
|
|
bool result = await strategy.CompactAsync(groups);
|
|
|
|
// Assert — all groups are in the protected window, nothing to collapse
|
|
Assert.False(result);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task CompactAsyncPreservesSystemMessagesAsync()
|
|
{
|
|
// Arrange
|
|
ToolResultCompactionStrategy strategy = new(
|
|
trigger: _ => true,
|
|
minimumPreservedGroups: 1);
|
|
|
|
CompactionMessageIndex groups = CompactionMessageIndex.Create(
|
|
[
|
|
new ChatMessage(ChatRole.System, "You are helpful."),
|
|
new ChatMessage(ChatRole.User, "Q1"),
|
|
new ChatMessage(ChatRole.Assistant, [new FunctionCallContent("call1", "fn")]),
|
|
new ChatMessage(ChatRole.Tool, "result"),
|
|
new ChatMessage(ChatRole.User, "Q2"),
|
|
]);
|
|
|
|
// Act
|
|
await strategy.CompactAsync(groups);
|
|
|
|
// Assert
|
|
List<ChatMessage> included = [.. groups.GetIncludedMessages()];
|
|
Assert.Equal("You are helpful.", included[0].Text);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task CompactAsyncExtractsMultipleToolNamesAsync()
|
|
{
|
|
// Arrange — assistant calls two tools
|
|
ToolResultCompactionStrategy strategy = new(
|
|
trigger: _ => true,
|
|
minimumPreservedGroups: 1);
|
|
|
|
ChatMessage multiToolCall = new(ChatRole.Assistant,
|
|
[
|
|
new FunctionCallContent("c1", "get_weather"),
|
|
new FunctionCallContent("c2", "search_docs"),
|
|
]);
|
|
|
|
CompactionMessageIndex groups = CompactionMessageIndex.Create(
|
|
[
|
|
new ChatMessage(ChatRole.User, "Q1"),
|
|
multiToolCall,
|
|
new ChatMessage(ChatRole.Tool, "Sunny"),
|
|
new ChatMessage(ChatRole.Tool, "Found 3 docs"),
|
|
new ChatMessage(ChatRole.User, "Q2"),
|
|
]);
|
|
|
|
// Act
|
|
await strategy.CompactAsync(groups);
|
|
|
|
// Assert
|
|
List<ChatMessage> included = [.. groups.GetIncludedMessages()];
|
|
string collapsed = included[1].Text!;
|
|
Assert.Equal("[Tool Calls]\nget_weather:\n - Sunny\nsearch_docs:\n - Found 3 docs", collapsed);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task CompactAsyncNoToolGroupsReturnsFalseAsync()
|
|
{
|
|
// Arrange — trigger fires but no tool groups to collapse
|
|
ToolResultCompactionStrategy strategy = new(
|
|
trigger: _ => true,
|
|
minimumPreservedGroups: 0);
|
|
|
|
CompactionMessageIndex groups = CompactionMessageIndex.Create(
|
|
[
|
|
new ChatMessage(ChatRole.User, "Hello"),
|
|
new ChatMessage(ChatRole.Assistant, "Hi!"),
|
|
]);
|
|
|
|
// Act
|
|
bool result = await strategy.CompactAsync(groups);
|
|
|
|
// Assert
|
|
Assert.False(result);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task CompactAsyncCompoundTriggerRequiresTokensAndToolCallsAsync()
|
|
{
|
|
// Arrange — compound: tokens > 0 AND has tool calls
|
|
ToolResultCompactionStrategy strategy = new(
|
|
CompactionTriggers.All(
|
|
CompactionTriggers.TokensExceed(0),
|
|
CompactionTriggers.HasToolCalls()),
|
|
minimumPreservedGroups: 1);
|
|
|
|
CompactionMessageIndex groups = CompactionMessageIndex.Create(
|
|
[
|
|
new ChatMessage(ChatRole.User, "Q1"),
|
|
new ChatMessage(ChatRole.Assistant, [new FunctionCallContent("c1", "fn")]),
|
|
new ChatMessage(ChatRole.Tool, "result"),
|
|
new ChatMessage(ChatRole.User, "Q2"),
|
|
]);
|
|
|
|
// Act
|
|
bool result = await strategy.CompactAsync(groups);
|
|
|
|
// Assert
|
|
Assert.True(result);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task CompactAsyncTargetStopsCollapsingEarlyAsync()
|
|
{
|
|
// Arrange — 2 tool groups, target met after first collapse
|
|
int collapseCount = 0;
|
|
bool TargetAfterOne(CompactionMessageIndex _) => ++collapseCount >= 1;
|
|
|
|
ToolResultCompactionStrategy strategy = new(
|
|
trigger: _ => true,
|
|
minimumPreservedGroups: 1,
|
|
target: TargetAfterOne);
|
|
|
|
CompactionMessageIndex index = CompactionMessageIndex.Create(
|
|
[
|
|
new ChatMessage(ChatRole.User, "Q1"),
|
|
new ChatMessage(ChatRole.Assistant, [new FunctionCallContent("c1", "fn1")]),
|
|
new ChatMessage(ChatRole.Tool, "result1"),
|
|
new ChatMessage(ChatRole.User, "Q2"),
|
|
new ChatMessage(ChatRole.Assistant, [new FunctionCallContent("c2", "fn2")]),
|
|
new ChatMessage(ChatRole.Tool, "result2"),
|
|
new ChatMessage(ChatRole.User, "Q3"),
|
|
]);
|
|
|
|
// Act
|
|
bool result = await strategy.CompactAsync(index);
|
|
|
|
// Assert — only first tool group collapsed, second left intact
|
|
Assert.True(result);
|
|
|
|
// Count collapsed tool groups (excluded with ToolCall kind)
|
|
int collapsedToolGroups = 0;
|
|
foreach (CompactionMessageGroup group in index.Groups)
|
|
{
|
|
if (group.IsExcluded && group.Kind == CompactionGroupKind.ToolCall)
|
|
{
|
|
collapsedToolGroups++;
|
|
}
|
|
}
|
|
|
|
Assert.Equal(1, collapsedToolGroups);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task CompactAsyncSkipsPreExcludedAndSystemGroupsAsync()
|
|
{
|
|
// Arrange — pre-excluded and system groups in the enumeration
|
|
ToolResultCompactionStrategy strategy = new(CompactionTriggers.Always, minimumPreservedGroups: 0);
|
|
|
|
List<ChatMessage> messages =
|
|
[
|
|
new ChatMessage(ChatRole.System, "System prompt"),
|
|
new ChatMessage(ChatRole.User, "Q0"),
|
|
new ChatMessage(ChatRole.Assistant, [new FunctionCallContent("c1", "fn")]),
|
|
new ChatMessage(ChatRole.Tool, "Result 1"),
|
|
new ChatMessage(ChatRole.User, "Q1"),
|
|
];
|
|
|
|
CompactionMessageIndex index = CompactionMessageIndex.Create(messages);
|
|
// Pre-exclude the last user group
|
|
index.Groups[index.Groups.Count - 1].IsExcluded = true;
|
|
|
|
// Act
|
|
bool result = await strategy.CompactAsync(index);
|
|
|
|
// Assert — system never excluded, pre-excluded skipped
|
|
Assert.True(result);
|
|
Assert.False(index.Groups[0].IsExcluded); // System stays
|
|
}
|
|
|
|
[Fact]
|
|
public async Task CompactAsyncDeduplicatesDuplicateToolNamesAsync()
|
|
{
|
|
// Arrange — same tool called multiple times
|
|
ToolResultCompactionStrategy strategy = new(
|
|
trigger: _ => true,
|
|
minimumPreservedGroups: 1);
|
|
|
|
CompactionMessageIndex groups = CompactionMessageIndex.Create(
|
|
[
|
|
new ChatMessage(ChatRole.User, "Q1"),
|
|
new ChatMessage(ChatRole.Assistant,
|
|
[
|
|
new FunctionCallContent("c1", "get_weather"),
|
|
new FunctionCallContent("c2", "get_weather"),
|
|
]),
|
|
new ChatMessage(ChatRole.Tool, "Sunny"),
|
|
new ChatMessage(ChatRole.Tool, "Rainy"),
|
|
new ChatMessage(ChatRole.User, "Q2"),
|
|
]);
|
|
|
|
// Act
|
|
await strategy.CompactAsync(groups);
|
|
|
|
// Assert — duplicate names listed once with all results
|
|
List<ChatMessage> included = [.. groups.GetIncludedMessages()];
|
|
Assert.Equal("[Tool Calls]\nget_weather:\n - Sunny\n - Rainy", included[1].Text);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task CompactAsyncIncludesResultsFromFunctionResultContentAsync()
|
|
{
|
|
// Arrange — tool results provided as FunctionResultContent (matched by CallId)
|
|
ToolResultCompactionStrategy strategy = new(
|
|
trigger: _ => true,
|
|
minimumPreservedGroups: 1);
|
|
|
|
CompactionMessageIndex groups = CompactionMessageIndex.Create(
|
|
[
|
|
new ChatMessage(ChatRole.User, "Q1"),
|
|
new ChatMessage(ChatRole.Assistant,
|
|
[
|
|
new FunctionCallContent("c1", "get_weather"),
|
|
new FunctionCallContent("c2", "search_docs"),
|
|
]),
|
|
new ChatMessage(ChatRole.Tool, [new FunctionResultContent("c1", "Sunny and 72°F")]),
|
|
new ChatMessage(ChatRole.Tool, [new FunctionResultContent("c2", "Found 3 docs")]),
|
|
new ChatMessage(ChatRole.User, "Q2"),
|
|
]);
|
|
|
|
// Act
|
|
await strategy.CompactAsync(groups);
|
|
|
|
// Assert — results matched by CallId and included in summary
|
|
List<ChatMessage> included = [.. groups.GetIncludedMessages()];
|
|
Assert.Equal("[Tool Calls]\nget_weather:\n - Sunny and 72°F\nsearch_docs:\n - Found 3 docs", included[1].Text);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task CompactAsyncDeduplicatesWithFunctionResultContentAsync()
|
|
{
|
|
// Arrange — same tool called multiple times with FunctionResultContent
|
|
ToolResultCompactionStrategy strategy = new(
|
|
trigger: _ => true,
|
|
minimumPreservedGroups: 1);
|
|
|
|
CompactionMessageIndex groups = CompactionMessageIndex.Create(
|
|
[
|
|
new ChatMessage(ChatRole.User, "Q1"),
|
|
new ChatMessage(ChatRole.Assistant,
|
|
[
|
|
new FunctionCallContent("c1", "get_weather"),
|
|
new FunctionCallContent("c2", "get_weather"),
|
|
new FunctionCallContent("c3", "search_docs"),
|
|
]),
|
|
new ChatMessage(ChatRole.Tool, [new FunctionResultContent("c1", "Sunny")]),
|
|
new ChatMessage(ChatRole.Tool, [new FunctionResultContent("c2", "Rainy")]),
|
|
new ChatMessage(ChatRole.Tool, [new FunctionResultContent("c3", "Found 3 docs")]),
|
|
new ChatMessage(ChatRole.User, "Q2"),
|
|
]);
|
|
|
|
// Act
|
|
await strategy.CompactAsync(groups);
|
|
|
|
// Assert — duplicate tool name results listed under same key
|
|
List<ChatMessage> included = [.. groups.GetIncludedMessages()];
|
|
Assert.Equal("[Tool Calls]\nget_weather:\n - Sunny\n - Rainy\nsearch_docs:\n - Found 3 docs", included[1].Text);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task CompactAsyncUsesCustomFormatterAsync()
|
|
{
|
|
// Arrange — custom formatter that produces a collapsed message count
|
|
static string CustomFormatter(CompactionMessageGroup group) =>
|
|
$"[Collapsed: {group.Messages.Count} messages]";
|
|
|
|
ToolResultCompactionStrategy strategy = new(
|
|
trigger: _ => true,
|
|
minimumPreservedGroups: 1)
|
|
{
|
|
ToolCallFormatter = CustomFormatter,
|
|
};
|
|
|
|
CompactionMessageIndex groups = CompactionMessageIndex.Create(
|
|
[
|
|
new ChatMessage(ChatRole.User, "Q1"),
|
|
new ChatMessage(ChatRole.Assistant, [new FunctionCallContent("c1", "get_weather")]),
|
|
new ChatMessage(ChatRole.Tool, "Sunny"),
|
|
new ChatMessage(ChatRole.User, "Q2"),
|
|
]);
|
|
|
|
// Act
|
|
bool result = await strategy.CompactAsync(groups);
|
|
|
|
// Assert — custom formatter output used instead of default YAML-like format
|
|
Assert.True(result);
|
|
List<ChatMessage> included = [.. groups.GetIncludedMessages()];
|
|
Assert.Equal("[Collapsed: 2 messages]", included[1].Text);
|
|
}
|
|
|
|
[Fact]
|
|
public void ToolCallFormatterPropertyIsNullWhenNoneProvided()
|
|
{
|
|
// Arrange
|
|
ToolResultCompactionStrategy strategy = new(CompactionTriggers.Always);
|
|
|
|
// Assert — ToolCallFormatter is null when no custom formatter is provided
|
|
Assert.Null(strategy.ToolCallFormatter);
|
|
}
|
|
|
|
[Fact]
|
|
public void ToolCallFormatterPropertyReturnsCustomFormatterWhenProvided()
|
|
{
|
|
// Arrange
|
|
Func<CompactionMessageGroup, string> customFormatter = static _ => "custom";
|
|
ToolResultCompactionStrategy strategy = new(
|
|
CompactionTriggers.Always)
|
|
{
|
|
ToolCallFormatter = customFormatter
|
|
};
|
|
|
|
// Assert — ToolCallFormatter is the injected custom function
|
|
Assert.Same(customFormatter, strategy.ToolCallFormatter);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task CompactAsyncCustomFormatterCanDelegateToDefaultAsync()
|
|
{
|
|
// Arrange — custom formatter that wraps the default output
|
|
static string WrappingFormatter(CompactionMessageGroup group) =>
|
|
$"CUSTOM_PREFIX\n{ToolResultCompactionStrategy.DefaultToolCallFormatter(group)}";
|
|
|
|
ToolResultCompactionStrategy strategy = new(
|
|
trigger: _ => true,
|
|
minimumPreservedGroups: 1)
|
|
{
|
|
ToolCallFormatter = WrappingFormatter
|
|
};
|
|
|
|
CompactionMessageIndex groups = CompactionMessageIndex.Create(
|
|
[
|
|
new ChatMessage(ChatRole.User, "Q1"),
|
|
new ChatMessage(ChatRole.Assistant, [new FunctionCallContent("c1", "fn")]),
|
|
new ChatMessage(ChatRole.Tool, "result"),
|
|
new ChatMessage(ChatRole.User, "Q2"),
|
|
]);
|
|
|
|
// Act
|
|
await strategy.CompactAsync(groups);
|
|
|
|
// Assert — wrapped default output
|
|
List<ChatMessage> included = [.. groups.GetIncludedMessages()];
|
|
Assert.Equal("CUSTOM_PREFIX\n[Tool Calls]\nfn:\n - result", included[1].Text);
|
|
}
|
|
}
|