mirror of
https://github.com/microsoft/agent-framework.git
synced 2026-06-16 21:04:09 +08:00
361c47f30f
* Do not build DevUI assets during .NET project build (#2010) * .NET: Add unit tests for declarative executor SetMultipleVariables (#2016) * Add unit tests for create conversation executor * Update indentation and comment typo. * Added unit tests for declarative executor SetMultipleVariablesExecutor * Updated comments and syntactic sugar * Python: DevUI: Use metadata.entity_id instead of model field (#1984) * DevUI: Use metadata.entity_id for agent/workflow name instead of model field * OpenAI Responses: add explicit request validation * Review feedback * .NET: DevUI - Do not automatically add/map OpenAI services/endpoints (#2014) * Don't add OpenAIResponses as part of Dev UI You should be able to add and remove Dev UI without impacting your other production endpoints. * Remove `AddDevUI()` and do not map OpenAI endpoints from `MapDevUI()` * Fix comment wording * Revise documentation --------- Co-authored-by: Daniel Roth <daroth@microsoft.com> * Python: DevUI: Add OpenAI Responses API proxy support + HIL for Workflows (#1737) * DevUI: Add OpenAI Responses API proxy support with enhanced UI features This commit adds support for proxying requests to OpenAI's Responses API, allowing DevUI to route conversations to OpenAI models when configured to enable testing. Backend changes: - Add OpenAI proxy executor with conversation routing logic - Enhance event mapper to support OpenAI Responses API format - Extend server endpoints to handle OpenAI proxy mode - Update models with OpenAI-specific response types - Remove emojis from logging and CLI output for cleaner text Frontend changes: - Add settings modal with OpenAI proxy configuration UI - Enhance agent and workflow views with improved state management - Add new UI components (separator, switch) for settings - Update debug panel with better event filtering - Improve message renderers for OpenAI content types - Update types and API client for OpenAI integration * update ui, settings modal and workflow input form, add register cleanup hooks. * add workflow HIL support, user mode, other fixes * feat(devui): add human-in-the-loop (HIL) support with dynamic response schemas Implement HIL workflow support allowing workflows to pause for user input with dynamically generated JSON schemas based on response handler type hints. Key Features: - Automatic response schema extraction from @response_handler decorators - Dynamic form generation in UI based on Pydantic/dataclass response types - Checkpoint-based conversation storage for HIL requests/responses - Resume workflow execution after user provides HIL response Backend Changes: - Add extract_response_type_from_executor() to introspect response handlers - Enrich RequestInfoEvent with response_schema via _enrich_request_info_event_with_response_schema() - Map RequestInfoEvent to response.input.requested OpenAI event format - Store HIL responses in conversation history and restore checkpoints Frontend Changes: - Add HILInputModal component with SchemaFormRenderer for dynamic forms - Support Pydantic BaseModel and dataclass response types - Render enum fields as dropdowns, strings as text/textarea, numbers, booleans, arrays, objects - Display original request context alongside response form Testing: - Add tests for checkpoint storage (test_checkpoints.py) - Add schema generation tests for all input types (test_schema_generation.py) - Validate end-to-end HIL flow with spam workflow sample This enables workflows to seamlessly pause execution and request structured user input with type-safe, validated forms generated automatically from response type annotations. * improve HIL support, improve workflow execution view * ui updates * ui updates * improve HIL for workflows, add auth and view modes * update workflow * security improvements , ui fixes * fix mypy error * update loading spinner in ui --------- Co-authored-by: Mark Wallace <127216156+markwallace-microsoft@users.noreply.github.com> * .NET: Remove launchSettings.json from .gitignore in dotnet/samples (#2006) * Remove launchSettings.json from .gitignore in dotnet/samples * Update dotnet/samples/GettingStarted/DevUI/DevUI_Step01_BasicUsage/Properties/launchSettings.json Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Update dotnet/samples/AGUIClientServer/AGUIServer/Properties/launchSettings.json Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * DevUI: Serialize workflow input as string to maintain conformance with OpenAI Responses format (#2021) Co-authored-by: Victor Dibia <chuvidi2003@gmail.com> * Add Microsoft Agent Framework logo to assets (#2007) * Updated package versions (#2027) * DevUI: Prevent line breaks within words in the agent view (#2024) Co-authored-by: Victor Dibia <chuvidi2003@gmail.com> * .NET [AG-UI]: Adds support for shared state. (#1996) * Product changes * Tests * Dojo project * Cleanups * Python: Fix underlying tool choice bug and all for return to previous Handoff subagent (#2037) * Fix tool_choice override bug and add enable_return_to_previous support * Add unit test for handoff checkpointing * Handle tools when we have them * added missing chatAgent params (#2044) * .NET: fix ChatCompletions Tools serialization (#2043) * fix serialization in chat completions on tools * nit * .NET: assign AgentCard's URL to mapped-endpoint if not defined explicitly (#2047) * fix serialization in chat completions on tools * nit * write e2e test for agent card resolve + adjust behavior * nit * Version 1.0.0-preview.251110.1 (#2048) * .NET: Remove moved OpenAPI sample and point to SK one. (#1997) * Remove moved OpenAPI sample and point to SK one. * Update dotnet/samples/GettingStarted/Agents/README.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Bump AWSSDK.Extensions.Bedrock.MEAI from 4.0.4.2 to 4.0.4.6 (#2031) --- updated-dependencies: - dependency-name: AWSSDK.Extensions.Bedrock.MEAI dependency-version: 4.0.4.6 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * .NET: Separate all memory and rag samples into their own folders (#2000) * Separate all memory and rag samples into their own folders * Fix broken link. * Python: .Net: Dotnet devui compatibility fixes (#2026) * DevUI: Add OpenAI Responses API proxy support with enhanced UI features This commit adds support for proxying requests to OpenAI's Responses API, allowing DevUI to route conversations to OpenAI models when configured to enable testing. Backend changes: - Add OpenAI proxy executor with conversation routing logic - Enhance event mapper to support OpenAI Responses API format - Extend server endpoints to handle OpenAI proxy mode - Update models with OpenAI-specific response types - Remove emojis from logging and CLI output for cleaner text Frontend changes: - Add settings modal with OpenAI proxy configuration UI - Enhance agent and workflow views with improved state management - Add new UI components (separator, switch) for settings - Update debug panel with better event filtering - Improve message renderers for OpenAI content types - Update types and API client for OpenAI integration * update ui, settings modal and workflow input form, add register cleanup hooks. * add workflow HIL support, user mode, other fixes * feat(devui): add human-in-the-loop (HIL) support with dynamic response schemas Implement HIL workflow support allowing workflows to pause for user input with dynamically generated JSON schemas based on response handler type hints. Key Features: - Automatic response schema extraction from @response_handler decorators - Dynamic form generation in UI based on Pydantic/dataclass response types - Checkpoint-based conversation storage for HIL requests/responses - Resume workflow execution after user provides HIL response Backend Changes: - Add extract_response_type_from_executor() to introspect response handlers - Enrich RequestInfoEvent with response_schema via _enrich_request_info_event_with_response_schema() - Map RequestInfoEvent to response.input.requested OpenAI event format - Store HIL responses in conversation history and restore checkpoints Frontend Changes: - Add HILInputModal component with SchemaFormRenderer for dynamic forms - Support Pydantic BaseModel and dataclass response types - Render enum fields as dropdowns, strings as text/textarea, numbers, booleans, arrays, objects - Display original request context alongside response form Testing: - Add tests for checkpoint storage (test_checkpoints.py) - Add schema generation tests for all input types (test_schema_generation.py) - Validate end-to-end HIL flow with spam workflow sample This enables workflows to seamlessly pause execution and request structured user input with type-safe, validated forms generated automatically from response type annotations. * improve HIL support, improve workflow execution view * ui updates * ui updates * improve HIL for workflows, add auth and view modes * update workflow * security improvements , ui fixes * fix mypy error * update loading spinner in ui * DevUI: Serialize workflow input as string to maintain conformance with OpenAI Responses format * Phase 1: Add /meta endpoint and fix workflow event naming for .NET DevUI compatibility * additional fixes for .NET DevUI workflow visualization item ID tracking **Problem:** .NET DevUI was generating different item IDs for ExecutorInvokedEvent and ExecutorCompletedEvent, causing only the first executor to highlight in the workflow graph. Long executor names and error messages also broke UI layout. **Changes:** - Add ExecutorActionItemResource to match Python DevUI implementation - Track item IDs per executor using dictionary in AgentRunResponseUpdateExtensions - Reuse same item ID across invoked/completed/failed events for proper pairing - Add truncateText() utility to workflow-utils.ts - Truncate executor names to 35 chars in execution timeline - Truncate error messages to 150 chars in workflow graph nodes ** Details:** - ExecutorActionItemResource registered with JSON source generation context - Dictionary cleaned up after executor completion/failure to prevent memory leaks - Frontend item tracking by unique item.id supports multiple executor runs - All changes follow existing codebase patterns and conventions Tested with review-workflow showing correct executor highlighting and state transitions for sequential and concurrent executors. * format fixes, remove cors tests * remove unecessary attributes --------- Co-authored-by: Mark Wallace <127216156+markwallace-microsoft@users.noreply.github.com> Co-authored-by: Reuben Bond <reuben.bond@gmail.com> * DevUI: support having both an agent and a workflow with the same id in discovery (#2023) * Python: Fix Model ID attribute not showing up in `invoke_agent` span (#2061) * Best effort to surface the model id to invoke agent span * Fix tests * Fix tests * Version 1.0.0-preview.251107.2 (#2065) * Version 1.0.0-preview.251110.2 (#2067) * Update README.md to change Grafana links to Azure portal links for dashboard access (#1983) * .NET - Enable build & test on branch `feature-foundry-agents` (#2068) * Tests good, mkay * Update .github/workflows/dotnet-build-and-test.yml Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Enable feature build pipelines --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Roger Barreto <19890735+rogerbarreto@users.noreply.github.com> * Python: Add concrete AGUIChatClient (#2072) * Add concrete AGUIChatClient * Update logging docstrings and conventions * PR feedback * Updates to support client-side tool calls * .NET: Move catalog samples to the HostedAgents folder (#2090) * move catalog samples to the HostedAgents folder * move the catalog samples' projects to the HostedAgents folder * Bump OpenTelemetry.Instrumentation.Runtime from 1.12.0 to 1.13.0 (#1856) --- updated-dependencies: - dependency-name: OpenTelemetry.Instrumentation.Runtime dependency-version: 1.13.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * .NET: Bump Microsoft.SemanticKernel.Agents.Abstractions from 1.66.0 to 1.67.0 (#1962) * Bump Microsoft.SemanticKernel.Agents.Abstractions from 1.66.0 to 1.67.0 --- updated-dependencies: - dependency-name: Microsoft.SemanticKernel.Agents.Abstractions dependency-version: 1.67.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * .NET: Bump all Microsoft.SemanticKernel packages from 1.66.* to 1.67.* (#1969) * Initial plan * Update all Microsoft.SemanticKernel packages to 1.67.* Co-authored-by: rogerbarreto <19890735+rogerbarreto@users.noreply.github.com> * Remove unrelated changes to package-lock.json and yarn.lock Co-authored-by: markwallace-microsoft <127216156+markwallace-microsoft@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: rogerbarreto <19890735+rogerbarreto@users.noreply.github.com> Co-authored-by: markwallace-microsoft <127216156+markwallace-microsoft@users.noreply.github.com> --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com> Co-authored-by: rogerbarreto <19890735+rogerbarreto@users.noreply.github.com> Co-authored-by: markwallace-microsoft <127216156+markwallace-microsoft@users.noreply.github.com> * .NET: fix: WorkflowAsAgent Sample (#1787) * fix: WorkflowAsAgent Sample * Also makes ChatForwardingExecutor public * feat: Expand ChatForwardingExecutor handled types Make ChatForwardingExecutor match the input types of ChatProtocolExecutor. * fix: Update for the new AgentRunResponseUpdate merge logic AIAgent always sends out List<ChatMessage> now. * Updated (#2076) * Bump vite in /python/samples/demos/chatkit-integration/frontend (#1918) Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 7.1.9 to 7.1.12. - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/v7.1.12/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/v7.1.12/packages/vite) --- updated-dependencies: - dependency-name: vite dependency-version: 7.1.12 dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump Roslynator.Analyzers from 4.14.0 to 4.14.1 (#1857) --- updated-dependencies: - dependency-name: Roslynator.Analyzers dependency-version: 4.14.1 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump MishaKav/pytest-coverage-comment from 1.1.57 to 1.1.59 (#2034) Bumps [MishaKav/pytest-coverage-comment](https://github.com/mishakav/pytest-coverage-comment) from 1.1.57 to 1.1.59. - [Release notes](https://github.com/mishakav/pytest-coverage-comment/releases) - [Changelog](https://github.com/MishaKav/pytest-coverage-comment/blob/main/CHANGELOG.md) - [Commits](https://github.com/mishakav/pytest-coverage-comment/compare/v1.1.57...v1.1.59) --- updated-dependencies: - dependency-name: MishaKav/pytest-coverage-comment dependency-version: 1.1.59 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Chris <66376200+crickman@users.noreply.github.com> * Python: Handle agent user input request in AgentExecutor (#2022) * Handle agent user input request in AgentExecutor * fix test * Address comments * Fix tests * Fix tests * Address comments * Address comments * Python: OpenAI Responses Image Generation Stream Support, Sample and Unit Tests (#1853) * support for image gen streaming * small fixes * fixes * added comment * Python: Fix MCP Tool Parameter Descriptions Not Propagated to LLMs (#1978) * mcp tool description fix * small fix * .NET: Allow extending agent run options via additional properties (#1872) * Allow extending agent run options via additional properties This mirrors the M.E.AI model in ChatOptions.AdditionalProperties which is very useful when building functionality pipelines. Fixes https://github.com/microsoft/agent-framework/issues/1815 * Expand XML documentation Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Add AdditionalProperties tests to AgentRunOptions Co-authored-by: kzu <169707+kzu@users.noreply.github.com> --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: kzu <169707+kzu@users.noreply.github.com> * Python: Use the last entry in the task history to avoid empty responses (#2101) * Use the last entry in the task history to avoid empty responses * History only contains Messages * Updated package versions (#2104) --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Reuben Bond <203839+ReubenBond@users.noreply.github.com> Co-authored-by: Peter Ibekwe <109177538+peibekwe@users.noreply.github.com> Co-authored-by: Jeff Handley <jeffhandley@users.noreply.github.com> Co-authored-by: Daniel Roth <daroth@microsoft.com> Co-authored-by: Victor Dibia <chuvidi2003@gmail.com> Co-authored-by: Mark Wallace <127216156+markwallace-microsoft@users.noreply.github.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Shawn Henry <sphenry@gmail.com> Co-authored-by: Javier Calvarro Nelson <jacalvar@microsoft.com> Co-authored-by: Evan Mattson <35585003+moonbox3@users.noreply.github.com> Co-authored-by: Eduard van Valkenburg <eavanvalkenburg@users.noreply.github.com> Co-authored-by: Korolev Dmitry <deagle.gross@gmail.com> Co-authored-by: westey <164392973+westey-m@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Reuben Bond <reuben.bond@gmail.com> Co-authored-by: Tao Chen <taochen@microsoft.com> Co-authored-by: wuweng <wuweng@microsoft.com> Co-authored-by: Chris <66376200+crickman@users.noreply.github.com> Co-authored-by: Roger Barreto <19890735+rogerbarreto@users.noreply.github.com> Co-authored-by: SergeyMenshykh <68852919+SergeyMenshykh@users.noreply.github.com> Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com> Co-authored-by: Jacob Alber <jaalber@microsoft.com> Co-authored-by: Giles Odigwe <79032838+giles17@users.noreply.github.com> Co-authored-by: Daniel Cazzulino <daniel@cazzulino.com> Co-authored-by: kzu <169707+kzu@users.noreply.github.com>
442 lines
20 KiB
C#
442 lines
20 KiB
C#
// Copyright (c) Microsoft. All rights reserved.
|
|
|
|
using System;
|
|
using System.Collections.Generic;
|
|
using System.Diagnostics.CodeAnalysis;
|
|
using System.Linq;
|
|
using System.Net.Http;
|
|
using System.Runtime.CompilerServices;
|
|
using System.Text.Json;
|
|
using System.Threading;
|
|
using System.Threading.Tasks;
|
|
using FluentAssertions;
|
|
using Microsoft.Agents.AI.AGUI;
|
|
using Microsoft.AspNetCore.Builder;
|
|
using Microsoft.AspNetCore.Hosting.Server;
|
|
using Microsoft.AspNetCore.TestHost;
|
|
using Microsoft.Extensions.AI;
|
|
using Microsoft.Extensions.DependencyInjection;
|
|
|
|
namespace Microsoft.Agents.AI.Hosting.AGUI.AspNetCore.IntegrationTests;
|
|
|
|
public sealed class SharedStateTests : IAsyncDisposable
|
|
{
|
|
private WebApplication? _app;
|
|
private HttpClient? _client;
|
|
|
|
[Fact]
|
|
public async Task StateSnapshot_IsReturnedAsDataContent_WithCorrectMediaTypeAsync()
|
|
{
|
|
// Arrange
|
|
var initialState = new { counter = 42, status = "active" };
|
|
var fakeAgent = new FakeStateAgent();
|
|
|
|
await this.SetupTestServerAsync(fakeAgent);
|
|
var chatClient = new AGUIChatClient(this._client!, "", null);
|
|
AIAgent agent = chatClient.CreateAIAgent(instructions: null, name: "assistant", description: "Sample assistant", tools: []);
|
|
ChatClientAgentThread thread = (ChatClientAgentThread)agent.GetNewThread();
|
|
|
|
string stateJson = JsonSerializer.Serialize(initialState);
|
|
byte[] stateBytes = System.Text.Encoding.UTF8.GetBytes(stateJson);
|
|
DataContent stateContent = new(stateBytes, "application/json");
|
|
ChatMessage stateMessage = new(ChatRole.System, [stateContent]);
|
|
ChatMessage userMessage = new(ChatRole.User, "update state");
|
|
|
|
List<AgentRunResponseUpdate> updates = [];
|
|
|
|
// Act
|
|
await foreach (AgentRunResponseUpdate update in agent.RunStreamingAsync([userMessage, stateMessage], thread, new AgentRunOptions(), CancellationToken.None))
|
|
{
|
|
updates.Add(update);
|
|
}
|
|
|
|
// Assert
|
|
updates.Should().NotBeEmpty();
|
|
|
|
// Should receive state snapshot as DataContent with application/json media type
|
|
AgentRunResponseUpdate? stateUpdate = updates.FirstOrDefault(u => u.Contents.Any(c => c is DataContent dc && dc.MediaType == "application/json"));
|
|
stateUpdate.Should().NotBeNull("should receive state snapshot update");
|
|
|
|
DataContent? dataContent = stateUpdate!.Contents.OfType<DataContent>().FirstOrDefault(dc => dc.MediaType == "application/json");
|
|
dataContent.Should().NotBeNull();
|
|
|
|
// Verify the state content
|
|
string receivedJson = System.Text.Encoding.UTF8.GetString(dataContent!.Data.ToArray());
|
|
JsonElement receivedState = JsonSerializer.Deserialize<JsonElement>(receivedJson);
|
|
receivedState.GetProperty("counter").GetInt32().Should().Be(43, "state should be incremented");
|
|
receivedState.GetProperty("status").GetString().Should().Be("active");
|
|
}
|
|
|
|
[Fact]
|
|
public async Task StateSnapshot_HasCorrectAdditionalPropertiesAsync()
|
|
{
|
|
// Arrange
|
|
var initialState = new { step = 1 };
|
|
var fakeAgent = new FakeStateAgent();
|
|
|
|
await this.SetupTestServerAsync(fakeAgent);
|
|
var chatClient = new AGUIChatClient(this._client!, "", null);
|
|
AIAgent agent = chatClient.CreateAIAgent(instructions: null, name: "assistant", description: "Sample assistant", tools: []);
|
|
ChatClientAgentThread thread = (ChatClientAgentThread)agent.GetNewThread();
|
|
|
|
string stateJson = JsonSerializer.Serialize(initialState);
|
|
byte[] stateBytes = System.Text.Encoding.UTF8.GetBytes(stateJson);
|
|
DataContent stateContent = new(stateBytes, "application/json");
|
|
ChatMessage stateMessage = new(ChatRole.System, [stateContent]);
|
|
ChatMessage userMessage = new(ChatRole.User, "process");
|
|
|
|
List<AgentRunResponseUpdate> updates = [];
|
|
|
|
// Act
|
|
await foreach (AgentRunResponseUpdate update in agent.RunStreamingAsync([userMessage, stateMessage], thread, new AgentRunOptions(), CancellationToken.None))
|
|
{
|
|
updates.Add(update);
|
|
}
|
|
|
|
// Assert
|
|
AgentRunResponseUpdate? stateUpdate = updates.FirstOrDefault(u => u.Contents.Any(c => c is DataContent dc && dc.MediaType == "application/json"));
|
|
stateUpdate.Should().NotBeNull();
|
|
|
|
ChatResponseUpdate chatUpdate = stateUpdate!.AsChatResponseUpdate();
|
|
chatUpdate.AdditionalProperties.Should().NotBeNull();
|
|
chatUpdate.AdditionalProperties.Should().ContainKey("is_state_snapshot");
|
|
((bool)chatUpdate.AdditionalProperties!["is_state_snapshot"]!).Should().BeTrue();
|
|
}
|
|
|
|
[Fact]
|
|
public async Task ComplexState_WithNestedObjectsAndArrays_RoundTripsCorrectlyAsync()
|
|
{
|
|
// Arrange
|
|
var complexState = new
|
|
{
|
|
sessionId = "test-123",
|
|
nested = new { value = "test", count = 10 },
|
|
array = new[] { 1, 2, 3 },
|
|
tags = new[] { "tag1", "tag2" }
|
|
};
|
|
var fakeAgent = new FakeStateAgent();
|
|
|
|
await this.SetupTestServerAsync(fakeAgent);
|
|
var chatClient = new AGUIChatClient(this._client!, "", null);
|
|
AIAgent agent = chatClient.CreateAIAgent(instructions: null, name: "assistant", description: "Sample assistant", tools: []);
|
|
ChatClientAgentThread thread = (ChatClientAgentThread)agent.GetNewThread();
|
|
|
|
string stateJson = JsonSerializer.Serialize(complexState);
|
|
byte[] stateBytes = System.Text.Encoding.UTF8.GetBytes(stateJson);
|
|
DataContent stateContent = new(stateBytes, "application/json");
|
|
ChatMessage stateMessage = new(ChatRole.System, [stateContent]);
|
|
ChatMessage userMessage = new(ChatRole.User, "process complex state");
|
|
|
|
List<AgentRunResponseUpdate> updates = [];
|
|
|
|
// Act
|
|
await foreach (AgentRunResponseUpdate update in agent.RunStreamingAsync([userMessage, stateMessage], thread, new AgentRunOptions(), CancellationToken.None))
|
|
{
|
|
updates.Add(update);
|
|
}
|
|
|
|
// Assert
|
|
AgentRunResponseUpdate? stateUpdate = updates.FirstOrDefault(u => u.Contents.Any(c => c is DataContent dc && dc.MediaType == "application/json"));
|
|
stateUpdate.Should().NotBeNull();
|
|
|
|
DataContent? dataContent = stateUpdate!.Contents.OfType<DataContent>().FirstOrDefault(dc => dc.MediaType == "application/json");
|
|
string receivedJson = System.Text.Encoding.UTF8.GetString(dataContent!.Data.ToArray());
|
|
JsonElement receivedState = JsonSerializer.Deserialize<JsonElement>(receivedJson);
|
|
|
|
receivedState.GetProperty("sessionId").GetString().Should().Be("test-123");
|
|
receivedState.GetProperty("nested").GetProperty("count").GetInt32().Should().Be(10);
|
|
receivedState.GetProperty("array").GetArrayLength().Should().Be(3);
|
|
receivedState.GetProperty("tags").GetArrayLength().Should().Be(2);
|
|
}
|
|
|
|
[Fact]
|
|
public async Task StateSnapshot_CanBeUsedInSubsequentRequest_ForStateRoundTripAsync()
|
|
{
|
|
// Arrange
|
|
var initialState = new { counter = 1, sessionId = "round-trip-test" };
|
|
var fakeAgent = new FakeStateAgent();
|
|
|
|
await this.SetupTestServerAsync(fakeAgent);
|
|
var chatClient = new AGUIChatClient(this._client!, "", null);
|
|
AIAgent agent = chatClient.CreateAIAgent(instructions: null, name: "assistant", description: "Sample assistant", tools: []);
|
|
ChatClientAgentThread thread = (ChatClientAgentThread)agent.GetNewThread();
|
|
|
|
string stateJson = JsonSerializer.Serialize(initialState);
|
|
byte[] stateBytes = System.Text.Encoding.UTF8.GetBytes(stateJson);
|
|
DataContent stateContent = new(stateBytes, "application/json");
|
|
ChatMessage stateMessage = new(ChatRole.System, [stateContent]);
|
|
ChatMessage userMessage = new(ChatRole.User, "increment");
|
|
|
|
List<AgentRunResponseUpdate> firstRoundUpdates = [];
|
|
|
|
// Act - First round
|
|
await foreach (AgentRunResponseUpdate update in agent.RunStreamingAsync([userMessage, stateMessage], thread, new AgentRunOptions(), CancellationToken.None))
|
|
{
|
|
firstRoundUpdates.Add(update);
|
|
}
|
|
|
|
// Extract state snapshot from first round
|
|
AgentRunResponseUpdate? firstStateUpdate = firstRoundUpdates.FirstOrDefault(u => u.Contents.Any(c => c is DataContent dc && dc.MediaType == "application/json"));
|
|
firstStateUpdate.Should().NotBeNull();
|
|
DataContent? firstStateContent = firstStateUpdate!.Contents.OfType<DataContent>().FirstOrDefault(dc => dc.MediaType == "application/json");
|
|
|
|
// Second round - use returned state
|
|
ChatMessage secondStateMessage = new(ChatRole.System, [firstStateContent!]);
|
|
ChatMessage secondUserMessage = new(ChatRole.User, "increment again");
|
|
|
|
List<AgentRunResponseUpdate> secondRoundUpdates = [];
|
|
await foreach (AgentRunResponseUpdate update in agent.RunStreamingAsync([secondUserMessage, secondStateMessage], thread, new AgentRunOptions(), CancellationToken.None))
|
|
{
|
|
secondRoundUpdates.Add(update);
|
|
}
|
|
|
|
// Assert - Second round should have incremented counter again
|
|
AgentRunResponseUpdate? secondStateUpdate = secondRoundUpdates.FirstOrDefault(u => u.Contents.Any(c => c is DataContent dc && dc.MediaType == "application/json"));
|
|
secondStateUpdate.Should().NotBeNull();
|
|
|
|
DataContent? secondStateContent = secondStateUpdate!.Contents.OfType<DataContent>().FirstOrDefault(dc => dc.MediaType == "application/json");
|
|
string secondStateJson = System.Text.Encoding.UTF8.GetString(secondStateContent!.Data.ToArray());
|
|
JsonElement secondState = JsonSerializer.Deserialize<JsonElement>(secondStateJson);
|
|
|
|
secondState.GetProperty("counter").GetInt32().Should().Be(3, "counter should be incremented twice: 1 -> 2 -> 3");
|
|
}
|
|
|
|
[Fact]
|
|
public async Task WithoutState_AgentBehavesNormally_NoStateSnapshotReturnedAsync()
|
|
{
|
|
// Arrange
|
|
var fakeAgent = new FakeStateAgent();
|
|
|
|
await this.SetupTestServerAsync(fakeAgent);
|
|
var chatClient = new AGUIChatClient(this._client!, "", null);
|
|
AIAgent agent = chatClient.CreateAIAgent(instructions: null, name: "assistant", description: "Sample assistant", tools: []);
|
|
ChatClientAgentThread thread = (ChatClientAgentThread)agent.GetNewThread();
|
|
|
|
ChatMessage userMessage = new(ChatRole.User, "hello");
|
|
|
|
List<AgentRunResponseUpdate> updates = [];
|
|
|
|
// Act
|
|
await foreach (AgentRunResponseUpdate update in agent.RunStreamingAsync([userMessage], thread, new AgentRunOptions(), CancellationToken.None))
|
|
{
|
|
updates.Add(update);
|
|
}
|
|
|
|
// Assert
|
|
updates.Should().NotBeEmpty();
|
|
|
|
// Should NOT have state snapshot when no state is sent
|
|
bool hasStateSnapshot = updates.Any(u => u.Contents.Any(c => c is DataContent dc && dc.MediaType == "application/json"));
|
|
hasStateSnapshot.Should().BeFalse("should not return state snapshot when no state is provided");
|
|
|
|
// Should have normal text response
|
|
updates.Should().Contain(u => u.Contents.Any(c => c is TextContent));
|
|
}
|
|
|
|
[Fact]
|
|
public async Task EmptyState_DoesNotTriggerStateHandlingAsync()
|
|
{
|
|
// Arrange
|
|
var emptyState = new { };
|
|
var fakeAgent = new FakeStateAgent();
|
|
|
|
await this.SetupTestServerAsync(fakeAgent);
|
|
var chatClient = new AGUIChatClient(this._client!, "", null);
|
|
AIAgent agent = chatClient.CreateAIAgent(instructions: null, name: "assistant", description: "Sample assistant", tools: []);
|
|
ChatClientAgentThread thread = (ChatClientAgentThread)agent.GetNewThread();
|
|
|
|
string stateJson = JsonSerializer.Serialize(emptyState);
|
|
byte[] stateBytes = System.Text.Encoding.UTF8.GetBytes(stateJson);
|
|
DataContent stateContent = new(stateBytes, "application/json");
|
|
ChatMessage stateMessage = new(ChatRole.System, [stateContent]);
|
|
ChatMessage userMessage = new(ChatRole.User, "hello");
|
|
|
|
List<AgentRunResponseUpdate> updates = [];
|
|
|
|
// Act
|
|
await foreach (AgentRunResponseUpdate update in agent.RunStreamingAsync([userMessage, stateMessage], thread, new AgentRunOptions(), CancellationToken.None))
|
|
{
|
|
updates.Add(update);
|
|
}
|
|
|
|
// Assert
|
|
updates.Should().NotBeEmpty();
|
|
|
|
// Empty state {} should not trigger state snapshot mechanism
|
|
bool hasEmptyStateSnapshot = updates.Any(u => u.Contents.Any(c => c is DataContent dc && dc.MediaType == "application/json"));
|
|
hasEmptyStateSnapshot.Should().BeFalse("empty state should be treated as no state");
|
|
|
|
// Should have normal response
|
|
updates.Should().Contain(u => u.Contents.Any(c => c is TextContent));
|
|
}
|
|
|
|
[Fact]
|
|
public async Task NonStreamingRunAsync_WithState_ReturnsStateInResponseAsync()
|
|
{
|
|
// Arrange
|
|
var initialState = new { counter = 5 };
|
|
var fakeAgent = new FakeStateAgent();
|
|
|
|
await this.SetupTestServerAsync(fakeAgent);
|
|
var chatClient = new AGUIChatClient(this._client!, "", null);
|
|
AIAgent agent = chatClient.CreateAIAgent(instructions: null, name: "assistant", description: "Sample assistant", tools: []);
|
|
ChatClientAgentThread thread = (ChatClientAgentThread)agent.GetNewThread();
|
|
|
|
string stateJson = JsonSerializer.Serialize(initialState);
|
|
byte[] stateBytes = System.Text.Encoding.UTF8.GetBytes(stateJson);
|
|
DataContent stateContent = new(stateBytes, "application/json");
|
|
ChatMessage stateMessage = new(ChatRole.System, [stateContent]);
|
|
ChatMessage userMessage = new(ChatRole.User, "process");
|
|
|
|
// Act
|
|
AgentRunResponse response = await agent.RunAsync([userMessage, stateMessage], thread, new AgentRunOptions(), CancellationToken.None);
|
|
|
|
// Assert
|
|
response.Should().NotBeNull();
|
|
response.Messages.Should().NotBeEmpty();
|
|
|
|
// Should have message with DataContent containing state
|
|
bool hasStateMessage = response.Messages.Any(m => m.Contents.Any(c => c is DataContent dc && dc.MediaType == "application/json"));
|
|
hasStateMessage.Should().BeTrue("response should contain state message");
|
|
|
|
ChatMessage? stateResponseMessage = response.Messages.FirstOrDefault(m => m.Contents.Any(c => c is DataContent dc && dc.MediaType == "application/json"));
|
|
stateResponseMessage.Should().NotBeNull();
|
|
|
|
DataContent? dataContent = stateResponseMessage!.Contents.OfType<DataContent>().FirstOrDefault(dc => dc.MediaType == "application/json");
|
|
string receivedJson = System.Text.Encoding.UTF8.GetString(dataContent!.Data.ToArray());
|
|
JsonElement receivedState = JsonSerializer.Deserialize<JsonElement>(receivedJson);
|
|
receivedState.GetProperty("counter").GetInt32().Should().Be(6);
|
|
}
|
|
|
|
private async Task SetupTestServerAsync(FakeStateAgent fakeAgent)
|
|
{
|
|
WebApplicationBuilder builder = WebApplication.CreateBuilder();
|
|
builder.Services.AddAGUI();
|
|
builder.WebHost.UseTestServer();
|
|
|
|
this._app = builder.Build();
|
|
|
|
this._app.MapAGUI("/agent", fakeAgent);
|
|
|
|
await this._app.StartAsync();
|
|
|
|
TestServer testServer = this._app.Services.GetRequiredService<IServer>() as TestServer
|
|
?? throw new InvalidOperationException("TestServer not found");
|
|
|
|
this._client = testServer.CreateClient();
|
|
this._client.BaseAddress = new Uri("http://localhost/agent");
|
|
}
|
|
|
|
public async ValueTask DisposeAsync()
|
|
{
|
|
this._client?.Dispose();
|
|
if (this._app != null)
|
|
{
|
|
await this._app.DisposeAsync();
|
|
}
|
|
}
|
|
}
|
|
|
|
[SuppressMessage("Performance", "CA1812:Avoid uninstantiated internal classes", Justification = "Instantiated in tests")]
|
|
internal sealed class FakeStateAgent : AIAgent
|
|
{
|
|
public override string? Description => "Agent for state testing";
|
|
|
|
public override Task<AgentRunResponse> RunAsync(IEnumerable<ChatMessage> messages, AgentThread? thread = null, AgentRunOptions? options = null, CancellationToken cancellationToken = default)
|
|
{
|
|
return this.RunStreamingAsync(messages, thread, options, cancellationToken).ToAgentRunResponseAsync(cancellationToken);
|
|
}
|
|
|
|
public override async IAsyncEnumerable<AgentRunResponseUpdate> RunStreamingAsync(
|
|
IEnumerable<ChatMessage> messages,
|
|
AgentThread? thread = null,
|
|
AgentRunOptions? options = null,
|
|
[EnumeratorCancellation] CancellationToken cancellationToken = default)
|
|
{
|
|
// Check for state in ChatOptions.AdditionalProperties (set by AG-UI hosting layer)
|
|
if (options is ChatClientAgentRunOptions { ChatOptions.AdditionalProperties: { } properties } &&
|
|
properties.TryGetValue("ag_ui_state", out object? stateObj) &&
|
|
stateObj is JsonElement state &&
|
|
state.ValueKind == JsonValueKind.Object)
|
|
{
|
|
// Check if state object has properties (not empty {})
|
|
bool hasProperties = false;
|
|
foreach (JsonProperty _ in state.EnumerateObject())
|
|
{
|
|
hasProperties = true;
|
|
break;
|
|
}
|
|
|
|
if (hasProperties)
|
|
{
|
|
// State is present and non-empty - modify it and return as DataContent
|
|
Dictionary<string, object?> modifiedState = [];
|
|
foreach (JsonProperty prop in state.EnumerateObject())
|
|
{
|
|
if (prop.Name == "counter" && prop.Value.ValueKind == JsonValueKind.Number)
|
|
{
|
|
modifiedState[prop.Name] = prop.Value.GetInt32() + 1;
|
|
}
|
|
else if (prop.Value.ValueKind == JsonValueKind.Number)
|
|
{
|
|
modifiedState[prop.Name] = prop.Value.GetInt32();
|
|
}
|
|
else if (prop.Value.ValueKind == JsonValueKind.String)
|
|
{
|
|
modifiedState[prop.Name] = prop.Value.GetString();
|
|
}
|
|
else if (prop.Value.ValueKind == JsonValueKind.Object || prop.Value.ValueKind == JsonValueKind.Array)
|
|
{
|
|
modifiedState[prop.Name] = prop.Value;
|
|
}
|
|
}
|
|
|
|
// Return modified state as DataContent
|
|
string modifiedStateJson = JsonSerializer.Serialize(modifiedState);
|
|
byte[] modifiedStateBytes = System.Text.Encoding.UTF8.GetBytes(modifiedStateJson);
|
|
DataContent modifiedStateContent = new(modifiedStateBytes, "application/json");
|
|
|
|
yield return new AgentRunResponseUpdate
|
|
{
|
|
MessageId = Guid.NewGuid().ToString("N"),
|
|
Role = ChatRole.Assistant,
|
|
Contents = [modifiedStateContent]
|
|
};
|
|
}
|
|
}
|
|
|
|
// Always return a text response
|
|
string messageId = Guid.NewGuid().ToString("N");
|
|
yield return new AgentRunResponseUpdate
|
|
{
|
|
MessageId = messageId,
|
|
Role = ChatRole.Assistant,
|
|
Contents = [new TextContent("State processed")]
|
|
};
|
|
|
|
await Task.CompletedTask;
|
|
}
|
|
|
|
public override AgentThread GetNewThread() => new FakeInMemoryAgentThread();
|
|
|
|
public override AgentThread DeserializeThread(JsonElement serializedThread, JsonSerializerOptions? jsonSerializerOptions = null)
|
|
{
|
|
return new FakeInMemoryAgentThread(serializedThread, jsonSerializerOptions);
|
|
}
|
|
|
|
private sealed class FakeInMemoryAgentThread : InMemoryAgentThread
|
|
{
|
|
public FakeInMemoryAgentThread()
|
|
: base()
|
|
{
|
|
}
|
|
|
|
public FakeInMemoryAgentThread(JsonElement serializedThread, JsonSerializerOptions? jsonSerializerOptions = null)
|
|
: base(serializedThread, jsonSerializerOptions)
|
|
{
|
|
}
|
|
}
|
|
|
|
public override object? GetService(Type serviceType, object? serviceKey = null) => null;
|
|
}
|