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>
277 lines
11 KiB
C#
277 lines
11 KiB
C#
// Copyright (c) Microsoft. All rights reserved.
|
|
|
|
using System;
|
|
using System.Collections.Generic;
|
|
using System.IO;
|
|
using System.Linq;
|
|
using System.Text.Json;
|
|
using System.Threading.Tasks;
|
|
using Microsoft.Agents.AI.Workflows.Declarative.Events;
|
|
using Microsoft.Agents.AI.Workflows.Declarative.IntegrationTests.Agents;
|
|
using Microsoft.Agents.AI.Workflows.Declarative.IntegrationTests.Framework;
|
|
using Microsoft.Agents.AI.Workflows.Declarative.Kit;
|
|
using Microsoft.Agents.AI.Workflows.Declarative.Mcp;
|
|
using Microsoft.Extensions.AI;
|
|
|
|
namespace Microsoft.Agents.AI.Workflows.Declarative.IntegrationTests;
|
|
|
|
/// <summary>
|
|
/// Integration tests for InvokeFunctionTool and InvokeMcpTool actions.
|
|
/// </summary>
|
|
public sealed class InvokeToolWorkflowTest(ITestOutputHelper output) : IntegrationTest(output)
|
|
{
|
|
#region InvokeFunctionTool Tests
|
|
|
|
[Theory]
|
|
[InlineData("InvokeFunctionTool.yaml", new string[] { "GetSpecials", "GetItemPrice" }, "2.95")]
|
|
[InlineData("InvokeFunctionToolWithApproval.yaml", new string[] { "GetItemPrice" }, "4.9")]
|
|
public Task ValidateInvokeFunctionToolAsync(string workflowFileName, string[] expectedFunctionCalls, string? expectedResultContains) =>
|
|
this.RunInvokeFunctionToolTestAsync(workflowFileName, expectedFunctionCalls, expectedResultContains);
|
|
|
|
#endregion
|
|
|
|
#region InvokeMcpTool Tests
|
|
|
|
[Theory]
|
|
[InlineData("InvokeMcpTool.yaml", "Azure OpenAI")]
|
|
public Task ValidateInvokeMcpToolAsync(string workflowFileName, string? expectedResultContains) =>
|
|
this.RunInvokeMcpToolTestAsync(workflowFileName, expectedResultContains, requireApproval: false);
|
|
|
|
[Theory]
|
|
[InlineData("InvokeMcpToolWithApproval.yaml", "Azure OpenAI", true)]
|
|
[InlineData("InvokeMcpToolWithApproval.yaml", "MCP tool invocation was not approved by user", false)]
|
|
public Task ValidateInvokeMcpToolWithApprovalAsync(string workflowFileName, string? expectedResultContains, bool approveRequest) =>
|
|
this.RunInvokeMcpToolTestAsync(workflowFileName, expectedResultContains, requireApproval: true, approveRequest: approveRequest);
|
|
|
|
#endregion
|
|
|
|
#region InvokeFunctionTool Test Helpers
|
|
|
|
/// <summary>
|
|
/// Runs an InvokeFunctionTool workflow test with the specified configuration.
|
|
/// </summary>
|
|
private async Task RunInvokeFunctionToolTestAsync(
|
|
string workflowFileName,
|
|
string[] expectedFunctionCalls,
|
|
string? expectedResultContains = null)
|
|
{
|
|
// Arrange
|
|
string workflowPath = GetWorkflowPath(workflowFileName);
|
|
IEnumerable<AIFunction> functionTools = new MenuPlugin().GetTools();
|
|
Dictionary<string, AIFunction> functionMap = functionTools.ToDictionary(tool => tool.Name, tool => tool);
|
|
DeclarativeWorkflowOptions workflowOptions = await this.CreateOptionsAsync(externalConversation: false);
|
|
Workflow workflow = DeclarativeWorkflowBuilder.Build<string>(workflowPath, workflowOptions);
|
|
|
|
WorkflowHarness harness = new(workflow, runId: Path.GetFileNameWithoutExtension(workflowPath));
|
|
List<string> invokedFunctions = [];
|
|
|
|
// Act - Run workflow and handle function invocations
|
|
WorkflowEvents workflowEvents = await harness.RunWorkflowAsync("start").ConfigureAwait(false);
|
|
|
|
while (workflowEvents.InputEvents.Count > 0)
|
|
{
|
|
RequestInfoEvent inputEvent = workflowEvents.InputEvents[^1];
|
|
ExternalInputRequest? toolRequest = inputEvent.Request.Data.As<ExternalInputRequest>();
|
|
Assert.NotNull(toolRequest);
|
|
|
|
IList<AIContent> functionResults = await this.ProcessFunctionCallsAsync(
|
|
toolRequest,
|
|
functionMap,
|
|
invokedFunctions).ConfigureAwait(false);
|
|
|
|
ChatMessage resultMessage = new(ChatRole.Tool, functionResults);
|
|
WorkflowEvents resumeEvents = await harness.ResumeAsync(
|
|
inputEvent.Request.CreateResponse(new ExternalInputResponse(resultMessage))).ConfigureAwait(false);
|
|
|
|
workflowEvents = new WorkflowEvents([.. workflowEvents.Events, .. resumeEvents.Events]);
|
|
|
|
// Continue processing until there are no more pending input events from the resumed workflow
|
|
if (resumeEvents.InputEvents.Count == 0)
|
|
{
|
|
break;
|
|
}
|
|
}
|
|
|
|
// Assert - Verify function calls were made in expected order
|
|
Assert.Equal(expectedFunctionCalls.Length, invokedFunctions.Count);
|
|
for (int i = 0; i < expectedFunctionCalls.Length; i++)
|
|
{
|
|
Assert.Equal(expectedFunctionCalls[i], invokedFunctions[i]);
|
|
}
|
|
|
|
// Assert - Verify executor and action events
|
|
AssertWorkflowEventsEmitted(workflowEvents);
|
|
|
|
// Assert - Verify expected result if specified
|
|
if (expectedResultContains is not null)
|
|
{
|
|
AssertResultContains(workflowEvents, expectedResultContains);
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Processes function calls from an external input request.
|
|
/// Handles both regular function calls and approval requests.
|
|
/// </summary>
|
|
private async Task<IList<AIContent>> ProcessFunctionCallsAsync(
|
|
ExternalInputRequest toolRequest,
|
|
Dictionary<string, AIFunction> functionMap,
|
|
List<string> invokedFunctions)
|
|
{
|
|
List<AIContent> results = [];
|
|
|
|
foreach (ChatMessage message in toolRequest.AgentResponse.Messages)
|
|
{
|
|
// Handle approval requests if present
|
|
foreach (ToolApprovalRequestContent approvalRequest in message.Contents.OfType<ToolApprovalRequestContent>())
|
|
{
|
|
this.Output.WriteLine($"APPROVAL REQUEST: {((FunctionCallContent)approvalRequest.ToolCall).Name}");
|
|
// Auto-approve for testing
|
|
results.Add(approvalRequest.CreateResponse(approved: true));
|
|
}
|
|
|
|
// Handle function calls
|
|
foreach (FunctionCallContent functionCall in message.Contents.OfType<FunctionCallContent>())
|
|
{
|
|
this.Output.WriteLine($"FUNCTION CALL: {functionCall.Name}");
|
|
|
|
if (!functionMap.TryGetValue(functionCall.Name, out AIFunction? functionTool))
|
|
{
|
|
Assert.Fail($"Function not found: {functionCall.Name}");
|
|
continue;
|
|
}
|
|
|
|
invokedFunctions.Add(functionCall.Name);
|
|
|
|
// Execute the function
|
|
AIFunctionArguments? functionArguments = functionCall.Arguments is null
|
|
? null
|
|
: new(functionCall.Arguments.NormalizePortableValues());
|
|
|
|
object? result = await functionTool.InvokeAsync(functionArguments).ConfigureAwait(false);
|
|
results.Add(new FunctionResultContent(functionCall.CallId, JsonSerializer.Serialize(result)));
|
|
|
|
this.Output.WriteLine($"FUNCTION RESULT: {JsonSerializer.Serialize(result)}");
|
|
}
|
|
}
|
|
|
|
return results;
|
|
}
|
|
|
|
#endregion
|
|
|
|
#region InvokeMcpTool Test Helpers
|
|
|
|
/// <summary>
|
|
/// Runs an InvokeMcpTool workflow test with the specified configuration.
|
|
/// </summary>
|
|
private async Task RunInvokeMcpToolTestAsync(
|
|
string workflowFileName,
|
|
string? expectedResultContains = null,
|
|
bool requireApproval = false,
|
|
bool approveRequest = true)
|
|
{
|
|
// Arrange
|
|
string workflowPath = GetWorkflowPath(workflowFileName);
|
|
DefaultMcpToolHandler mcpToolProvider = new();
|
|
DeclarativeWorkflowOptions workflowOptions = await this.CreateOptionsAsync(
|
|
externalConversation: false,
|
|
mcpToolProvider: mcpToolProvider);
|
|
|
|
Workflow workflow = DeclarativeWorkflowBuilder.Build<string>(workflowPath, workflowOptions);
|
|
WorkflowHarness harness = new(workflow, runId: Path.GetFileNameWithoutExtension(workflowPath));
|
|
|
|
// Act - Run workflow and handle MCP tool invocations
|
|
WorkflowEvents workflowEvents = await harness.RunWorkflowAsync("start").ConfigureAwait(false);
|
|
|
|
while (workflowEvents.InputEvents.Count > 0)
|
|
{
|
|
RequestInfoEvent inputEvent = workflowEvents.InputEvents[^1];
|
|
ExternalInputRequest? toolRequest = inputEvent.Request.Data.As<ExternalInputRequest>();
|
|
Assert.NotNull(toolRequest);
|
|
|
|
IList<AIContent> mcpResults = this.ProcessMcpToolRequests(
|
|
toolRequest,
|
|
approveRequest);
|
|
|
|
ChatMessage resultMessage = new(ChatRole.Tool, mcpResults);
|
|
WorkflowEvents resumeEvents = await harness.ResumeAsync(
|
|
inputEvent.Request.CreateResponse(new ExternalInputResponse(resultMessage))).ConfigureAwait(false);
|
|
|
|
workflowEvents = new WorkflowEvents([.. workflowEvents.Events, .. resumeEvents.Events]);
|
|
|
|
// Continue processing until there are no more pending input events from the resumed workflow
|
|
if (resumeEvents.InputEvents.Count == 0)
|
|
{
|
|
break;
|
|
}
|
|
}
|
|
|
|
// Assert - Verify executor and action events
|
|
AssertWorkflowEventsEmitted(workflowEvents);
|
|
|
|
// Assert - Verify expected result if specified
|
|
if (expectedResultContains is not null)
|
|
{
|
|
AssertResultContains(workflowEvents, expectedResultContains);
|
|
}
|
|
|
|
// Cleanup
|
|
await mcpToolProvider.DisposeAsync().ConfigureAwait(false);
|
|
}
|
|
|
|
/// <summary>
|
|
/// Processes MCP tool requests from an external input request.
|
|
/// Handles approval requests for MCP tools.
|
|
/// </summary>
|
|
private List<AIContent> ProcessMcpToolRequests(
|
|
ExternalInputRequest toolRequest,
|
|
bool approveRequest)
|
|
{
|
|
List<AIContent> results = [];
|
|
|
|
foreach (ChatMessage message in toolRequest.AgentResponse.Messages)
|
|
{
|
|
// Handle MCP approval requests if present
|
|
foreach (ToolApprovalRequestContent approvalRequest in message.Contents.OfType<ToolApprovalRequestContent>())
|
|
{
|
|
this.Output.WriteLine($"MCP APPROVAL REQUEST: {approvalRequest.RequestId}");
|
|
|
|
// Respond based on test configuration
|
|
ToolApprovalResponseContent response = approvalRequest.CreateResponse(approved: approveRequest);
|
|
results.Add(response);
|
|
|
|
this.Output.WriteLine($"MCP APPROVAL RESPONSE: {(approveRequest ? "Approved" : "Rejected")}");
|
|
}
|
|
}
|
|
|
|
return results;
|
|
}
|
|
|
|
#endregion
|
|
|
|
#region Shared Helpers
|
|
|
|
private static void AssertWorkflowEventsEmitted(WorkflowEvents workflowEvents)
|
|
{
|
|
Assert.NotEmpty(workflowEvents.ExecutorInvokeEvents);
|
|
Assert.NotEmpty(workflowEvents.ExecutorCompleteEvents);
|
|
Assert.NotEmpty(workflowEvents.ActionInvokeEvents);
|
|
}
|
|
|
|
private static void AssertResultContains(WorkflowEvents workflowEvents, string expectedResultContains)
|
|
{
|
|
MessageActivityEvent? messageEvent = workflowEvents.Events
|
|
.OfType<MessageActivityEvent>()
|
|
.LastOrDefault();
|
|
|
|
Assert.NotNull(messageEvent);
|
|
Assert.Contains(expectedResultContains, messageEvent.Message, StringComparison.OrdinalIgnoreCase);
|
|
}
|
|
|
|
private static string GetWorkflowPath(string workflowFileName) =>
|
|
Path.Combine(Environment.CurrentDirectory, "Workflows", workflowFileName);
|
|
|
|
#endregion
|
|
}
|