mirror of
https://github.com/microsoft/agent-framework.git
synced 2026-06-16 21:04:09 +08:00
88ea9d08c7
* Initial plan * Update code for Microsoft.Extensions.AI.Abstractions 10.4.0 breaking changes - Rename FunctionApprovalRequestContent → ToolApprovalRequestContent - Rename FunctionApprovalResponseContent → ToolApprovalResponseContent - Rename UserInputRequestContent → ToolApprovalRequestContent - Rename UserInputResponseContent → ToolApprovalResponseContent - Update .FunctionCall property → .ToolCall with FunctionCallContent casts where needed - Update .Id property → .RequestId on the renamed types - Rename FunctionApprovalRequestEventGenerator → ToolApprovalRequestEventGenerator - Rename FunctionApprovalResponseEventGenerator → ToolApprovalResponseEventGenerator Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Update OpenAI 2.9.1, ME.AI 10.4.0, fix breaking API changes Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com> * Fix remaining ME.AI 10.4.0 breaking changes: MCP approval types, .Output→.Outputs Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com> * Use pattern matching with `when` for ToolApprovalRequestContent/FunctionCallContent Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com> * Update Azure.AI.OpenAI to 2.9.0-beta.1 Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com> * Fix remaining GetResponsesClient(model) build failures for Azure.AI.OpenAI 2.9.0-beta.1 Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com> * Address review feedback: remove redundant type checks in TestRequestAgent.cs and fix error message in AIAgentHostExecutor.cs Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com> * Update Azure.AI.Projects to 2.0.0-beta.2 with namespace migration - Azure.AI.Projects 2.0.0-beta.1 → 2.0.0-beta.2 - Azure.AI.Projects.OpenAI → Azure.AI.Extensions.OpenAI (transitive) - Agent types moved to Azure.AI.Projects.Agents namespace - AgentRecord.Versions.Latest → AgentRecord.GetLatestVersion() - OpenAPIFunctionDefinition → OpenApiFunctionDefinition - BingCustomSearchToolParameters → BingCustomSearchToolOptions - MemorySearchPreviewTool.UpdateDelay → UpdateDelayInSecs - Azure.Identity 1.17.1 → 1.19.0 - Microsoft.Identity.Client.Extensions.Msal 4.78.0 → 4.83.1 Co-authored-by: rogerbarreto <19890735+rogerbarreto@users.noreply.github.com> * Fix remaining type renames for Azure.AI.Projects 2.0.0-beta.2 - BrowserAutomationToolParameters → BrowserAutomationToolOptions - MemoryUpdateOptions.UpdateDelay stays as UpdateDelay (not renamed) - WaitForMemoriesUpdateAsync parameter order: pollingInterval before options - AIProjectAgentsOperations → AgentsClient Co-authored-by: rogerbarreto <19890735+rogerbarreto@users.noreply.github.com> * Fix format errors and OpenTelemetry test for ME.AI 10.4.0 - Remove unused 'using Azure.AI.Extensions.OpenAI' and fix import ordering in Agent_With_AzureAIProject/Program.cs - Update OpenTelemetryAgentTests: gen_ai.tool.definitions is now always emitted regardless of EnableSensitiveData per ME.AI 10.4.0 change (dotnet/extensions#7346). Tool definitions are not considered sensitive. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Fix GetRepoFolder() to work in git worktrees Use 'workflow-samples' directory as repo root marker instead of '.git', which fails in worktrees (.git is a file) and also matches too early when a '.github' folder exists in subdirectories. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Fix formatting: remove unused usings and fix import ordering dotnet format applied across 59 impacted projects. Primarily removes unnecessary 'using Azure.AI.Projects' where Azure.AI.Projects.Agents provides all needed types, and fixes import ordering per editorconfig. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Disable AzureAIAgentsPersistent integration tests (#4769) Azure.AI.Agents.Persistent 1.2.0-beta.9 references McpServerToolApprovalResponseContent which was removed in ME.AI 10.4.0 (renamed to ToolApprovalResponseContent), causing TypeLoadException at runtime. Mark all 6 test classes with IntegrationDisabled trait until Persistent ships a version targeting ME.AI 10.4.0+. Upstream fix: https://github.com/Azure/azure-sdk-for-net/pull/56929 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Add README with compatibility note for AzureAI.Persistent (#4769) Documents that Azure.AI.Agents.Persistent 1.2.0-beta.9 is only compatible with ME.AI ≤10.3.0 and OpenAI ≤2.8.0 due to type renames in ME.AI 10.4.0. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Fix file encoding: restore UTF-8 BOM on Persistent test files Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Mark AzureAI.Persistent as IsPackable=false (#4769) Prevent shipping until Azure.AI.Agents.Persistent targets ME.AI 10.4.0+. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Moving IsPackable after import --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com> Co-authored-by: rogerbarreto <19890735+rogerbarreto@users.noreply.github.com>
279 lines
11 KiB
C#
279 lines
11 KiB
C#
// Copyright (c) Microsoft. All rights reserved.
|
|
|
|
using System;
|
|
using System.Collections.Generic;
|
|
using System.Linq;
|
|
using System.Threading.Tasks;
|
|
using FluentAssertions;
|
|
using Microsoft.Agents.AI.Workflows.Specialized;
|
|
using Microsoft.Extensions.AI;
|
|
|
|
namespace Microsoft.Agents.AI.Workflows.UnitTests;
|
|
|
|
public class AIAgentHostExecutorTests
|
|
{
|
|
private const string TestAgentId = nameof(TestAgentId);
|
|
private const string TestAgentName = nameof(TestAgentName);
|
|
|
|
private static readonly string[] s_messageStrings = [
|
|
"",
|
|
"Hello world!",
|
|
"Lorem ipsum dolor sit amet, consectetur adipiscing elit.",
|
|
"Quisque dignissim ante odio, at facilisis orci porta a. Duis mi augue, fringilla eu egestas a, pellentesque sed lacus."
|
|
];
|
|
|
|
private static List<ChatMessage> TestMessages => TestReplayAgent.ToChatMessages(s_messageStrings);
|
|
|
|
[Theory]
|
|
[InlineData(null, null)]
|
|
[InlineData(null, true)]
|
|
[InlineData(null, false)]
|
|
[InlineData(true, null)]
|
|
[InlineData(true, true)]
|
|
[InlineData(true, false)]
|
|
[InlineData(false, null)]
|
|
[InlineData(false, true)]
|
|
[InlineData(false, false)]
|
|
public async Task Test_AgentHostExecutor_EmitsStreamingUpdatesIFFConfiguredAsync(bool? executorSetting, bool? turnSetting)
|
|
{
|
|
// Arrange
|
|
TestRunContext testContext = new();
|
|
TestReplayAgent agent = new(TestMessages, TestAgentId, TestAgentName);
|
|
AIAgentHostExecutor executor = new(agent, new() { EmitAgentUpdateEvents = executorSetting });
|
|
testContext.ConfigureExecutor(executor);
|
|
|
|
// Act
|
|
await executor.TakeTurnAsync(new(turnSetting), testContext.BindWorkflowContext(executor.Id));
|
|
|
|
// Assert
|
|
// The rules are: TurnToken overrides Agent, if set. Default to false, if both unset.
|
|
bool expectingEvents = turnSetting ?? executorSetting ?? false;
|
|
|
|
AgentResponseUpdateEvent[] updates = testContext.Events.OfType<AgentResponseUpdateEvent>().ToArray();
|
|
if (expectingEvents)
|
|
{
|
|
// The way TestReplayAgent is set up, it will emit one update per non-empty AIContent
|
|
List<AIContent> expectedUpdateContents = TestMessages.SelectMany(message => message.Contents).ToList();
|
|
|
|
updates.Should().HaveCount(expectedUpdateContents.Count);
|
|
for (int i = 0; i < updates.Length; i++)
|
|
{
|
|
AgentResponseUpdateEvent updateEvent = updates[i];
|
|
AIContent expectedUpdateContent = expectedUpdateContents[i];
|
|
|
|
updateEvent.ExecutorId.Should().Be(agent.GetDescriptiveId());
|
|
|
|
AgentResponseUpdate update = updateEvent.Update;
|
|
update.AuthorName.Should().Be(TestAgentName);
|
|
update.AgentId.Should().Be(TestAgentId);
|
|
update.Contents.Should().HaveCount(1);
|
|
update.Contents[0].Should().BeEquivalentTo(expectedUpdateContent);
|
|
}
|
|
}
|
|
else
|
|
{
|
|
updates.Should().BeEmpty();
|
|
}
|
|
}
|
|
|
|
[Theory]
|
|
[InlineData(true)]
|
|
[InlineData(false)]
|
|
public async Task Test_AgentHostExecutor_EmitsResponseIFFConfiguredAsync(bool executorSetting)
|
|
{
|
|
// Arrange
|
|
TestRunContext testContext = new();
|
|
TestReplayAgent agent = new(TestMessages, TestAgentId, TestAgentName);
|
|
AIAgentHostExecutor executor = new(agent, new() { EmitAgentResponseEvents = executorSetting });
|
|
testContext.ConfigureExecutor(executor);
|
|
|
|
// Act
|
|
await executor.TakeTurnAsync(new(), testContext.BindWorkflowContext(executor.Id));
|
|
|
|
// Assert
|
|
AgentResponseEvent[] updates = testContext.Events.OfType<AgentResponseEvent>().ToArray();
|
|
if (executorSetting)
|
|
{
|
|
updates.Should().HaveCount(1);
|
|
|
|
AgentResponseEvent responseEvent = updates[0];
|
|
responseEvent.ExecutorId.Should().Be(agent.GetDescriptiveId());
|
|
|
|
AgentResponse response = responseEvent.Response;
|
|
response.AgentId.Should().Be(TestAgentId);
|
|
response.Messages.Should().HaveCount(TestMessages.Count - 1);
|
|
|
|
for (int i = 0; i < response.Messages.Count; i++)
|
|
{
|
|
ChatMessage responseMessage = response.Messages[i];
|
|
ChatMessage expectedMessage = TestMessages[i + 1]; // Skip the first empty message
|
|
|
|
responseMessage.AuthorName.Should().Be(TestAgentName);
|
|
responseMessage.Text.Should().Be(expectedMessage.Text);
|
|
}
|
|
}
|
|
else
|
|
{
|
|
updates.Should().BeEmpty();
|
|
}
|
|
}
|
|
|
|
private static ChatMessage UserMessage => new(ChatRole.User, "Hello from User!") { AuthorName = "User" };
|
|
private static ChatMessage AssistantMessage => new(ChatRole.Assistant, "Hello from Assistant!") { AuthorName = "User" };
|
|
private static ChatMessage TestAgentMessage => new(ChatRole.Assistant, $"Hello from {TestAgentName}!") { AuthorName = TestAgentName };
|
|
|
|
[Theory]
|
|
[InlineData(true, true, false, false)]
|
|
[InlineData(true, true, false, true)]
|
|
[InlineData(true, true, true, false)]
|
|
[InlineData(true, true, true, true)]
|
|
[InlineData(true, false, false, false)]
|
|
[InlineData(true, false, false, true)]
|
|
[InlineData(true, false, true, false)]
|
|
[InlineData(true, false, true, true)]
|
|
[InlineData(false, true, false, false)]
|
|
[InlineData(false, true, false, true)]
|
|
[InlineData(false, true, true, false)]
|
|
[InlineData(false, true, true, true)]
|
|
[InlineData(false, false, false, false)]
|
|
[InlineData(false, false, false, true)]
|
|
[InlineData(false, false, true, false)]
|
|
[InlineData(false, false, true, true)]
|
|
public async Task Test_AgentHostExecutor_ReassignsRolesIFFConfiguredAsync(bool executorSetting, bool includeUser, bool includeSelfMessages, bool includeOtherMessages)
|
|
{
|
|
// Arrange
|
|
TestRunContext testContext = new();
|
|
RoleCheckAgent agent = new(false, TestAgentId, TestAgentName);
|
|
AIAgentHostExecutor executor = new(agent, new() { ReassignOtherAgentsAsUsers = executorSetting });
|
|
testContext.ConfigureExecutor(executor);
|
|
|
|
List<ChatMessage> messages = [];
|
|
|
|
if (includeUser)
|
|
{
|
|
messages.Add(UserMessage);
|
|
}
|
|
|
|
if (includeSelfMessages)
|
|
{
|
|
messages.Add(TestAgentMessage);
|
|
}
|
|
|
|
if (includeOtherMessages)
|
|
{
|
|
messages.Add(AssistantMessage);
|
|
}
|
|
|
|
// Act
|
|
await executor.Router.RouteMessageAsync(messages, testContext.BindWorkflowContext(executor.Id));
|
|
|
|
Func<Task> act = async () => await executor.TakeTurnAsync(new(), testContext.BindWorkflowContext(executor.Id));
|
|
|
|
// Assert
|
|
bool shouldThrow = includeOtherMessages && !executorSetting;
|
|
|
|
if (shouldThrow)
|
|
{
|
|
await act.Should().ThrowAsync<InvalidOperationException>();
|
|
}
|
|
else
|
|
{
|
|
await act.Should().NotThrowAsync();
|
|
}
|
|
}
|
|
|
|
[Theory]
|
|
[InlineData(true, TestAgentRequestType.FunctionCall)]
|
|
[InlineData(false, TestAgentRequestType.FunctionCall)]
|
|
//[InlineData(true, TestAgentRequestType.UserInputRequest)] TODO: Enable when we support polymorphic routing
|
|
[InlineData(false, TestAgentRequestType.UserInputRequest)]
|
|
public async Task Test_AgentHostExecutor_InterceptsRequestsIFFConfiguredAsync(bool intercept, TestAgentRequestType requestType)
|
|
{
|
|
const int UnpairedRequestCount = 2;
|
|
const int PairedRequestCount = 3;
|
|
|
|
// Arrange
|
|
TestRunContext testContext = new();
|
|
TestRequestAgent agent = new(requestType, UnpairedRequestCount, PairedRequestCount, TestAgentId, TestAgentName);
|
|
AIAgentHostOptions agentHostOptions = requestType switch
|
|
{
|
|
TestAgentRequestType.FunctionCall =>
|
|
new()
|
|
{
|
|
EmitAgentResponseEvents = true,
|
|
InterceptUnterminatedFunctionCalls = intercept
|
|
},
|
|
TestAgentRequestType.UserInputRequest =>
|
|
new()
|
|
{
|
|
EmitAgentResponseEvents = true,
|
|
InterceptUserInputRequests = intercept
|
|
},
|
|
_ => throw new NotSupportedException()
|
|
};
|
|
|
|
AIAgentHostExecutor executor = new(agent, agentHostOptions);
|
|
testContext.ConfigureExecutor(executor);
|
|
|
|
// Act
|
|
await executor.TakeTurnAsync(new(), testContext.BindWorkflowContext(executor.Id));
|
|
|
|
// Assert
|
|
List<object> responses;
|
|
if (intercept)
|
|
{
|
|
// We expect to have a sent message containing the requests as an ExternalRequest
|
|
switch (requestType)
|
|
{
|
|
case TestAgentRequestType.FunctionCall:
|
|
responses = ExtractAndValidateRequestContents<FunctionCallContent>();
|
|
break;
|
|
case TestAgentRequestType.UserInputRequest:
|
|
responses = ExtractAndValidateRequestContents<ToolApprovalRequestContent>();
|
|
break;
|
|
default:
|
|
throw new NotSupportedException();
|
|
}
|
|
|
|
List<object> ExtractAndValidateRequestContents<TRequest>() where TRequest : AIContent
|
|
{
|
|
IEnumerable<TRequest> requests = testContext.QueuedMessages.Should().ContainKey(executor.Id)
|
|
.WhoseValue
|
|
.Select(envelope => envelope.Message as TRequest)
|
|
.Where(item => item is not null)
|
|
.Select(item => item!);
|
|
|
|
return agent.ValidateUnpairedRequests(requests).ToList();
|
|
}
|
|
}
|
|
else
|
|
{
|
|
responses = agent.ValidateUnpairedRequests([.. testContext.ExternalRequests]).ToList<object>();
|
|
}
|
|
|
|
// Act 2
|
|
foreach (object response in responses.Take(UnpairedRequestCount - 1))
|
|
{
|
|
await executor.Router.RouteMessageAsync(response, testContext.BindWorkflowContext(executor.Id));
|
|
}
|
|
|
|
// Assert 2
|
|
// Since we are not finished, we expect the agent to not have produced a final response (="Remaining: 1")
|
|
AgentResponseEvent lastResponseEvent = testContext.Events.OfType<AgentResponseEvent>().Should().NotBeEmpty()
|
|
.And.Subject.Last();
|
|
|
|
lastResponseEvent.Response.Text.Should().Be("Remaining: 1");
|
|
|
|
// Act 3
|
|
object finalResponse = responses.Last();
|
|
await executor.Router.RouteMessageAsync(finalResponse, testContext.BindWorkflowContext(executor.Id));
|
|
|
|
// Assert 3
|
|
// Now that we are finished, we expect the agent to have produced a final response
|
|
lastResponseEvent = testContext.Events.OfType<AgentResponseEvent>().Should().NotBeEmpty()
|
|
.And.Subject.Last();
|
|
|
|
lastResponseEvent.Response.Text.Should().Be("Done");
|
|
}
|
|
}
|