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>
436 lines
20 KiB
C#
436 lines
20 KiB
C#
// Copyright (c) Microsoft. All rights reserved.
|
|
|
|
using System;
|
|
using System.Linq;
|
|
using System.Net.Http;
|
|
using System.Text;
|
|
using System.Text.Json;
|
|
using System.Threading.Tasks;
|
|
using Microsoft.AspNetCore.Builder;
|
|
using Microsoft.AspNetCore.Hosting.Server;
|
|
using Microsoft.AspNetCore.TestHost;
|
|
using Microsoft.Extensions.AI;
|
|
using Microsoft.Extensions.DependencyInjection;
|
|
using Microsoft.Extensions.Hosting;
|
|
|
|
namespace Microsoft.Agents.AI.Hosting.OpenAI.UnitTests;
|
|
|
|
/// <summary>
|
|
/// Integration tests for the HTTP API with in-memory conversation, response, and agent index storage.
|
|
/// Tests create a conversation, create a response, wait for completion, then verify the conversation was updated.
|
|
/// </summary>
|
|
public sealed class OpenAIHttpApiIntegrationTests : IAsyncDisposable
|
|
{
|
|
private WebApplication? _app;
|
|
private HttpClient? _httpClient;
|
|
|
|
[Fact]
|
|
public async Task CreateConversationAndResponse_NonStreaming_NonBackground_UpdatesConversationWithOutputAsync()
|
|
{
|
|
// Arrange
|
|
const string AgentName = "test-agent";
|
|
const string Instructions = "You are a helpful assistant.";
|
|
const string ExpectedResponse = "The capital of France is Paris.";
|
|
const string UserMessage = "What is the capital of France?";
|
|
|
|
HttpClient client = await this.CreateTestServerWithInMemoryStorageAsync(AgentName, Instructions, ExpectedResponse);
|
|
|
|
// Act - Create conversation
|
|
var createConversationRequest = new { metadata = new { agent_id = AgentName } };
|
|
string createConvJson = JsonSerializer.Serialize(createConversationRequest);
|
|
HttpResponseMessage createConvResponse = await this.SendPostRequestAsync(client, "/v1/conversations", createConvJson);
|
|
using var createConvDoc = await this.ParseResponseAsync(createConvResponse);
|
|
string conversationId = createConvDoc.RootElement.GetProperty("id").GetString()!;
|
|
|
|
// Act - Create response (non-streaming, non-background)
|
|
var createResponseRequest = new
|
|
{
|
|
metadata = new { entity_id = AgentName },
|
|
conversation = conversationId,
|
|
input = UserMessage,
|
|
stream = false
|
|
};
|
|
string createRespJson = JsonSerializer.Serialize(createResponseRequest);
|
|
HttpResponseMessage createRespResponse = await this.SendPostRequestAsync(client, $"/{AgentName}/v1/responses", createRespJson);
|
|
using var createRespDoc = await this.ParseResponseAsync(createRespResponse);
|
|
var response = createRespDoc.RootElement;
|
|
|
|
// Assert - Response completed
|
|
Assert.Equal("completed", response.GetProperty("status").GetString());
|
|
string responseId = response.GetProperty("id").GetString()!;
|
|
Assert.NotNull(responseId);
|
|
Assert.StartsWith("resp_", responseId);
|
|
|
|
// Assert - Response has output
|
|
Assert.True(response.TryGetProperty("output", out var output));
|
|
Assert.True(output.GetArrayLength() > 0);
|
|
var outputItem = output[0];
|
|
var content = outputItem.GetProperty("content");
|
|
Assert.True(content.GetArrayLength() > 0);
|
|
var textContent = content[0];
|
|
Assert.Equal("output_text", textContent.GetProperty("type").GetString());
|
|
Assert.Equal(ExpectedResponse, textContent.GetProperty("text").GetString());
|
|
|
|
// Act - List conversation items to verify they were updated
|
|
HttpResponseMessage listItemsResponse = await this.SendGetRequestAsync(client, $"/v1/conversations/{conversationId}/items");
|
|
using var listItemsDoc = await this.ParseResponseAsync(listItemsResponse);
|
|
var itemsList = listItemsDoc.RootElement;
|
|
|
|
// Assert - Conversation items were added
|
|
Assert.Equal("list", itemsList.GetProperty("object").GetString());
|
|
var items = itemsList.GetProperty("data");
|
|
|
|
Assert.True(items.GetArrayLength() > 0, "Conversation should have items after response completion");
|
|
|
|
// Find the assistant message in the items
|
|
bool foundAssistantMessage = items.EnumerateArray()
|
|
.Where(item => item.GetProperty("type").GetString() == "message" &&
|
|
item.GetProperty("role").GetString() == "assistant")
|
|
.Any(item =>
|
|
{
|
|
JsonElement itemContent = item.GetProperty("content");
|
|
if (itemContent.GetArrayLength() > 0)
|
|
{
|
|
JsonElement firstContent = itemContent[0];
|
|
return firstContent.GetProperty("type").GetString() == "output_text" &&
|
|
firstContent.GetProperty("text").GetString() == ExpectedResponse;
|
|
}
|
|
return false;
|
|
});
|
|
|
|
Assert.True(foundAssistantMessage, "Conversation should contain the assistant's response message");
|
|
}
|
|
|
|
[Fact]
|
|
public async Task CreateConversationAndResponse_Streaming_NonBackground_UpdatesConversationWithOutputAsync()
|
|
{
|
|
// Arrange
|
|
const string AgentName = "streaming-agent";
|
|
const string Instructions = "You are a helpful assistant.";
|
|
const string ExpectedResponse = "Hello there! How can I help you today?";
|
|
const string UserMessage = "Hello";
|
|
|
|
HttpClient client = await this.CreateTestServerWithInMemoryStorageAsync(AgentName, Instructions, ExpectedResponse);
|
|
|
|
// Act - Create conversation
|
|
var createConversationRequest = new { metadata = new { agent_id = AgentName } };
|
|
string createConvJson = JsonSerializer.Serialize(createConversationRequest);
|
|
HttpResponseMessage createConvResponse = await this.SendPostRequestAsync(client, "/v1/conversations", createConvJson);
|
|
using var createConvDoc = await this.ParseResponseAsync(createConvResponse);
|
|
string conversationId = createConvDoc.RootElement.GetProperty("id").GetString()!;
|
|
|
|
// Act - Create response (streaming, non-background)
|
|
var createResponseRequest = new
|
|
{
|
|
metadata = new { entity_id = AgentName },
|
|
conversation = conversationId,
|
|
input = UserMessage,
|
|
stream = true
|
|
};
|
|
string createRespJson = JsonSerializer.Serialize(createResponseRequest);
|
|
HttpResponseMessage createRespResponse = await this.SendPostRequestAsync(client, $"/{AgentName}/v1/responses", createRespJson);
|
|
|
|
// Assert - Response is SSE format
|
|
Assert.Equal("text/event-stream", createRespResponse.Content.Headers.ContentType?.MediaType);
|
|
|
|
// Parse SSE events
|
|
string sseContent = await createRespResponse.Content.ReadAsStringAsync();
|
|
var events = this.ParseSseEvents(sseContent);
|
|
|
|
// Assert - Has expected event types
|
|
var eventTypes = events.Select(e => e.GetProperty("type").GetString()).ToList();
|
|
Assert.Contains("response.created", eventTypes);
|
|
Assert.Contains("response.completed", eventTypes);
|
|
|
|
// Collect the full response text from deltas
|
|
var deltaEvents = events.Where(e => e.GetProperty("type").GetString() == "response.output_text.delta").ToList();
|
|
string streamedText = string.Concat(deltaEvents.Select(e => e.GetProperty("delta").GetString()));
|
|
Assert.Equal(ExpectedResponse, streamedText);
|
|
|
|
// Act - List conversation items to verify messages were added
|
|
HttpResponseMessage listItemsResponse = await this.SendGetRequestAsync(client, $"/v1/conversations/{conversationId}/items");
|
|
using var listItemsDoc = await this.ParseResponseAsync(listItemsResponse);
|
|
var itemsList = listItemsDoc.RootElement;
|
|
|
|
// Assert - Conversation items were added
|
|
var items = itemsList.GetProperty("data");
|
|
Assert.True(items.GetArrayLength() > 0, "Conversation should have items after streaming response completion");
|
|
|
|
// Find the assistant message in the items
|
|
bool foundAssistantMessage = items.EnumerateArray()
|
|
.Where(item => item.GetProperty("type").GetString() == "message" &&
|
|
item.GetProperty("role").GetString() == "assistant")
|
|
.Any(item =>
|
|
{
|
|
JsonElement itemContent = item.GetProperty("content");
|
|
if (itemContent.GetArrayLength() > 0)
|
|
{
|
|
JsonElement firstContent = itemContent[0];
|
|
return firstContent.GetProperty("type").GetString() == "output_text" &&
|
|
firstContent.GetProperty("text").GetString() == ExpectedResponse;
|
|
}
|
|
return false;
|
|
});
|
|
|
|
Assert.True(foundAssistantMessage, "Conversation should contain the assistant's response message");
|
|
}
|
|
|
|
[Fact]
|
|
public async Task CreateConversationAndResponse_NonStreaming_Background_UpdatesConversationWhenCompleteAsync()
|
|
{
|
|
// Arrange
|
|
const string AgentName = "background-agent";
|
|
const string Instructions = "You are a helpful assistant.";
|
|
const string ExpectedResponse = "Processing in background...";
|
|
const string UserMessage = "Can you process this?";
|
|
|
|
HttpClient client = await this.CreateTestServerWithInMemoryStorageAsync(AgentName, Instructions, ExpectedResponse);
|
|
|
|
// Act - Create conversation
|
|
var createConversationRequest = new { metadata = new { agent_id = AgentName } };
|
|
string createConvJson = JsonSerializer.Serialize(createConversationRequest);
|
|
HttpResponseMessage createConvResponse = await this.SendPostRequestAsync(client, "/v1/conversations", createConvJson);
|
|
using var createConvDoc = await this.ParseResponseAsync(createConvResponse);
|
|
string conversationId = createConvDoc.RootElement.GetProperty("id").GetString()!;
|
|
|
|
// Act - Create response (non-streaming, background)
|
|
var createResponseRequest = new
|
|
{
|
|
metadata = new { entity_id = AgentName },
|
|
conversation = conversationId,
|
|
input = UserMessage,
|
|
stream = false,
|
|
background = true
|
|
};
|
|
string createRespJson = JsonSerializer.Serialize(createResponseRequest);
|
|
HttpResponseMessage createRespResponse = await this.SendPostRequestAsync(client, $"/{AgentName}/v1/responses", createRespJson);
|
|
using var createRespDoc = await this.ParseResponseAsync(createRespResponse);
|
|
var response = createRespDoc.RootElement;
|
|
|
|
// Assert - Response is in progress or queued
|
|
string status = response.GetProperty("status").GetString()!;
|
|
Assert.True(status == "in_progress" || status == "queued" || status == "completed", $"Expected 'in_progress', 'queued', or 'completed', got '{status}'");
|
|
string responseId = response.GetProperty("id").GetString()!;
|
|
|
|
// Wait for completion by polling
|
|
const int MaxAttempts = 20;
|
|
int attempt = 0;
|
|
string finalStatus = status;
|
|
string? errorMessage = null;
|
|
while (finalStatus != "completed" && finalStatus != "failed" && attempt < MaxAttempts)
|
|
{
|
|
await Task.Delay(100);
|
|
HttpResponseMessage getResponseResponse = await this.SendGetRequestAsync(client, $"/{AgentName}/v1/responses/{responseId}");
|
|
using var getRespDoc = await this.ParseResponseAsync(getResponseResponse);
|
|
finalStatus = getRespDoc.RootElement.GetProperty("status").GetString()!;
|
|
if (getRespDoc.RootElement.TryGetProperty("error", out var error) &&
|
|
error.ValueKind == JsonValueKind.Object &&
|
|
error.TryGetProperty("message", out var messageElement))
|
|
{
|
|
errorMessage = messageElement.GetString();
|
|
}
|
|
|
|
attempt++;
|
|
}
|
|
|
|
// Assert - Response eventually completed
|
|
Assert.Equal("completed", finalStatus + (errorMessage != null ? $" Error: {errorMessage}" : ""));
|
|
|
|
// Act - List conversation items to verify messages were added
|
|
HttpResponseMessage listItemsResponse = await this.SendGetRequestAsync(client, $"/v1/conversations/{conversationId}/items");
|
|
using var listItemsDoc = await this.ParseResponseAsync(listItemsResponse);
|
|
var itemsList = listItemsDoc.RootElement;
|
|
|
|
// Assert - Conversation items were added
|
|
var items = itemsList.GetProperty("data");
|
|
Assert.True(items.GetArrayLength() > 0, "Conversation should have items after background response completion");
|
|
|
|
// Find the assistant message in the items
|
|
bool foundAssistantMessage = items.EnumerateArray()
|
|
.Where(item => item.GetProperty("type").GetString() == "message" &&
|
|
item.GetProperty("role").GetString() == "assistant")
|
|
.Any(item =>
|
|
{
|
|
JsonElement itemContent = item.GetProperty("content");
|
|
if (itemContent.GetArrayLength() > 0)
|
|
{
|
|
JsonElement firstContent = itemContent[0];
|
|
return firstContent.GetProperty("type").GetString() == "output_text" &&
|
|
firstContent.GetProperty("text").GetString() == ExpectedResponse;
|
|
}
|
|
return false;
|
|
});
|
|
|
|
Assert.True(foundAssistantMessage, "Conversation should contain the assistant's response message");
|
|
}
|
|
|
|
[Fact]
|
|
public async Task CreateConversationAndResponse_Streaming_Background_UpdatesConversationWhenCompleteAsync()
|
|
{
|
|
// Arrange
|
|
const string AgentName = "streaming-background-agent";
|
|
const string Instructions = "You are a helpful assistant.";
|
|
const string ExpectedResponse = "Streaming background response";
|
|
const string UserMessage = "Process this with streaming";
|
|
|
|
HttpClient client = await this.CreateTestServerWithInMemoryStorageAsync(AgentName, Instructions, ExpectedResponse);
|
|
|
|
// Act - Create conversation
|
|
var createConversationRequest = new { metadata = new { agent_id = AgentName } };
|
|
string createConvJson = JsonSerializer.Serialize(createConversationRequest);
|
|
HttpResponseMessage createConvResponse = await this.SendPostRequestAsync(client, "/v1/conversations", createConvJson);
|
|
using var createConvDoc = await this.ParseResponseAsync(createConvResponse);
|
|
string conversationId = createConvDoc.RootElement.GetProperty("id").GetString()!;
|
|
|
|
// Act - Create response (streaming, background)
|
|
var createResponseRequest = new
|
|
{
|
|
model = AgentName,
|
|
conversation = conversationId,
|
|
input = UserMessage,
|
|
stream = true,
|
|
background = false // Note: streaming with background=true is typically streaming
|
|
};
|
|
string createRespJson = JsonSerializer.Serialize(createResponseRequest);
|
|
HttpResponseMessage createRespResponse = await this.SendPostRequestAsync(client, $"/{AgentName}/v1/responses", createRespJson);
|
|
|
|
// Assert - Response is SSE format
|
|
Assert.Equal("text/event-stream", createRespResponse.Content.Headers.ContentType?.MediaType);
|
|
|
|
// Parse SSE events
|
|
string sseContent = await createRespResponse.Content.ReadAsStringAsync();
|
|
var events = this.ParseSseEvents(sseContent);
|
|
var eventTypes = events.Select(e => e.GetProperty("type").GetString()).ToList();
|
|
Assert.Contains("response.created", eventTypes);
|
|
Assert.Contains("response.completed", eventTypes);
|
|
|
|
// Act - List conversation items to verify messages were added
|
|
HttpResponseMessage listItemsResponse = await this.SendGetRequestAsync(client, $"/v1/conversations/{conversationId}/items");
|
|
using var listItemsDoc = await this.ParseResponseAsync(listItemsResponse);
|
|
var itemsList = listItemsDoc.RootElement;
|
|
|
|
// Assert - Conversation items were added
|
|
var items = itemsList.GetProperty("data");
|
|
Assert.True(items.GetArrayLength() > 0, "Conversation should have items after streaming response completion");
|
|
|
|
// Find the assistant message in the items
|
|
bool foundAssistantMessage = items.EnumerateArray()
|
|
.Where(item => item.GetProperty("type").GetString() == "message" &&
|
|
item.GetProperty("role").GetString() == "assistant")
|
|
.Any(item =>
|
|
{
|
|
JsonElement itemContent = item.GetProperty("content");
|
|
if (itemContent.GetArrayLength() > 0)
|
|
{
|
|
JsonElement firstContent = itemContent[0];
|
|
return firstContent.GetProperty("type").GetString() == "output_text";
|
|
}
|
|
return false;
|
|
});
|
|
|
|
Assert.True(foundAssistantMessage, "Conversation should contain the assistant's response message");
|
|
}
|
|
|
|
/// <summary>
|
|
/// Creates a test server with in-memory conversation, response, and agent index storage.
|
|
/// </summary>
|
|
private async Task<HttpClient> CreateTestServerWithInMemoryStorageAsync(string agentName, string instructions, string responseText)
|
|
{
|
|
WebApplicationBuilder builder = WebApplication.CreateBuilder();
|
|
builder.WebHost.UseTestServer();
|
|
|
|
// Create mock chat client
|
|
IChatClient mockChatClient = new TestHelpers.SimpleMockChatClient(responseText);
|
|
builder.Services.AddKeyedSingleton("chat-client", mockChatClient);
|
|
|
|
// Add agent
|
|
builder.AddAIAgent(agentName, instructions, chatClientServiceKey: "chat-client");
|
|
|
|
// Add in-memory storage for conversations, responses, and agent index
|
|
builder.AddOpenAIConversations();
|
|
builder.AddOpenAIResponses();
|
|
|
|
this._app = builder.Build();
|
|
|
|
// Map endpoints
|
|
AIAgent agent = this._app.Services.GetRequiredKeyedService<AIAgent>(agentName);
|
|
this._app.MapOpenAIConversations();
|
|
this._app.MapOpenAIResponses(agent);
|
|
|
|
await this._app.StartAsync();
|
|
|
|
TestServer testServer = this._app.Services.GetRequiredService<IServer>() as TestServer
|
|
?? throw new InvalidOperationException("TestServer not found");
|
|
|
|
this._httpClient = testServer.CreateClient();
|
|
return this._httpClient;
|
|
}
|
|
|
|
/// <summary>
|
|
/// Sends a POST request with JSON content to the test server.
|
|
/// </summary>
|
|
private async Task<HttpResponseMessage> SendPostRequestAsync(HttpClient client, string path, string requestJson)
|
|
{
|
|
using StringContent content = new(requestJson, Encoding.UTF8, "application/json");
|
|
return await client.PostAsync(new Uri(path, UriKind.Relative), content);
|
|
}
|
|
|
|
/// <summary>
|
|
/// Sends a GET request to the test server.
|
|
/// </summary>
|
|
private async Task<HttpResponseMessage> SendGetRequestAsync(HttpClient client, string path)
|
|
{
|
|
return await client.GetAsync(new Uri(path, UriKind.Relative));
|
|
}
|
|
|
|
/// <summary>
|
|
/// Parses the response JSON and returns a JsonDocument.
|
|
/// </summary>
|
|
private async Task<JsonDocument> ParseResponseAsync(HttpResponseMessage response)
|
|
{
|
|
string responseJson = await response.Content.ReadAsStringAsync();
|
|
return JsonDocument.Parse(responseJson);
|
|
}
|
|
|
|
/// <summary>
|
|
/// Parses SSE events from streaming response content string.
|
|
/// </summary>
|
|
private JsonElement[] ParseSseEvents(string sseContent)
|
|
{
|
|
var events = new System.Collections.Generic.List<JsonElement>();
|
|
var lines = sseContent.Split('\n');
|
|
|
|
for (int i = 0; i < lines.Length; i++)
|
|
{
|
|
var line = lines[i].TrimEnd('\r');
|
|
|
|
if (line.StartsWith("event: ", StringComparison.Ordinal) && i + 1 < lines.Length)
|
|
{
|
|
var dataLine = lines[i + 1].TrimEnd('\r');
|
|
if (dataLine.StartsWith("data: ", StringComparison.Ordinal))
|
|
{
|
|
var jsonData = dataLine.Substring("data: ".Length);
|
|
if (!string.IsNullOrWhiteSpace(jsonData))
|
|
{
|
|
var doc = JsonDocument.Parse(jsonData);
|
|
events.Add(doc.RootElement.Clone());
|
|
}
|
|
}
|
|
}
|
|
}
|
|
|
|
return events.ToArray();
|
|
}
|
|
|
|
public async ValueTask DisposeAsync()
|
|
{
|
|
this._httpClient?.Dispose();
|
|
if (this._app != null)
|
|
{
|
|
await this._app.DisposeAsync();
|
|
}
|
|
|
|
GC.SuppressFinalize(this);
|
|
}
|
|
}
|