mirror of
https://github.com/microsoft/agent-framework.git
synced 2026-06-16 21:04:09 +08:00
0e2fcb1c7f
* Add Azure AI Foundry Memory Context Provider with unit tests * Add FoundryMemory integration tests and sample application * Fix ClearStoredMemoriesAsync to handle 404 gracefully and rename to EnsureStoredMemoriesDeletedAsync * Refactor FoundryMemory: simplify architecture and add memory store creation - Remove IFoundryMemoryOperations interface (was only for test mocking) - Remove AIProjectClientMemoryOperations wrapper class - Provider now directly uses AIProjectClient with internal extension methods - Extension methods return actual response models instead of extracted values - Remove WaitForUpdateCompletionAsync from provider (sample uses delay) - Simplify EnsureMemoryStoreCreatedAsync to return Task instead of Task<bool> - Add memory store creation with chat_model and embedding_model - Add UpdateMemoriesResponse with SupersededBy and Error fields - Simplify unit tests to focus on constructor validation and serialization - Update sample to use simple delay for memory processing wait * Add waiting operation for memory store updates * Fix UTF-8 BOM encoding for FoundryMemory csproj files * Update copilot instructions for UTF-8 BOM and fix sample API rename * Fix UTF-8 BOM encoding for TestableAIProjectClient.cs * Add missing response headers for TS * Changing default embedding * Using the SDK Models * Program update * Remove debugging code from sample * Adapt FoundryMemoryProvider to new AIContextProvider API and add UTF-8 BOM instruction - Override ProvideAIContextAsync/StoreAIContextAsync instead of removed virtual InvokingAsync/InvokedAsync - Use ProviderSessionState<State> for session-scoped state management (matching Mem0Provider pattern) - Replace constructor-based scope with stateInitializer delegate - Remove Serialize method (no longer on base class) - Add SearchInputMessageFilter, StorageInputMessageFilter, StateKey to options - Update sample to use AIContextProviders list instead of AIContextProviderFactory - Update unit and integration tests for new API - Add UTF-8 BOM encoding and --tl:off instructions to dotnet/AGENTS.md Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Use DefaultAzureCredential in Foundry Memory sample Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Address PR review comments for FoundryMemoryProvider - Move memoryStoreName from options to required constructor parameter - Make FoundryMemoryProviderScope require non-null/whitespace scope in constructor - Make Scope property read-only (getter only) - Replace ConcurrentQueue with single last update ID to fix memory leak - Only clear pending update ID after successful completion - Add delete success logging - Mark FoundryMemoryProvider with [Experimental] attribute - Update unit tests for new API signatures Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Use Throw.IfNullOrWhitespace for scope and memoryStoreName validation Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
197 lines
6.7 KiB
C#
197 lines
6.7 KiB
C#
// Copyright (c) Microsoft. All rights reserved.
|
|
|
|
using System;
|
|
using System.ClientModel.Primitives;
|
|
using System.Net;
|
|
using System.Net.Http;
|
|
using System.Text;
|
|
using System.Text.Json.Serialization;
|
|
using System.Threading;
|
|
using System.Threading.Tasks;
|
|
using Azure.AI.Projects;
|
|
using Azure.Core;
|
|
|
|
namespace Microsoft.Agents.AI.FoundryMemory.UnitTests;
|
|
|
|
/// <summary>
|
|
/// Creates a testable AIProjectClient with a mock HTTP handler.
|
|
/// </summary>
|
|
internal sealed class TestableAIProjectClient : IDisposable
|
|
{
|
|
private readonly HttpClient _httpClient;
|
|
|
|
public TestableAIProjectClient(
|
|
string? searchMemoriesResponse = null,
|
|
string? updateMemoriesResponse = null,
|
|
HttpStatusCode? searchStatusCode = null,
|
|
HttpStatusCode? updateStatusCode = null,
|
|
HttpStatusCode? deleteStatusCode = null,
|
|
HttpStatusCode? createStoreStatusCode = null,
|
|
HttpStatusCode? getStoreStatusCode = null)
|
|
{
|
|
this.Handler = new MockHttpMessageHandler(
|
|
searchMemoriesResponse,
|
|
updateMemoriesResponse,
|
|
searchStatusCode,
|
|
updateStatusCode,
|
|
deleteStatusCode,
|
|
createStoreStatusCode,
|
|
getStoreStatusCode);
|
|
|
|
this._httpClient = new HttpClient(this.Handler);
|
|
|
|
AIProjectClientOptions options = new()
|
|
{
|
|
Transport = new HttpClientPipelineTransport(this._httpClient)
|
|
};
|
|
|
|
// Using a valid format endpoint
|
|
this.Client = new AIProjectClient(
|
|
new Uri("https://test.services.ai.azure.com/api/projects/test-project"),
|
|
new MockTokenCredential(),
|
|
options);
|
|
}
|
|
|
|
public AIProjectClient Client { get; }
|
|
|
|
public MockHttpMessageHandler Handler { get; }
|
|
|
|
public void Dispose()
|
|
{
|
|
this._httpClient.Dispose();
|
|
this.Handler.Dispose();
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Mock HTTP message handler for testing.
|
|
/// </summary>
|
|
internal sealed class MockHttpMessageHandler : HttpMessageHandler
|
|
{
|
|
private readonly string? _searchMemoriesResponse;
|
|
private readonly string? _updateMemoriesResponse;
|
|
private readonly HttpStatusCode _searchStatusCode;
|
|
private readonly HttpStatusCode _updateStatusCode;
|
|
private readonly HttpStatusCode _deleteStatusCode;
|
|
private readonly HttpStatusCode _createStoreStatusCode;
|
|
private readonly HttpStatusCode _getStoreStatusCode;
|
|
|
|
public MockHttpMessageHandler(
|
|
string? searchMemoriesResponse = null,
|
|
string? updateMemoriesResponse = null,
|
|
HttpStatusCode? searchStatusCode = null,
|
|
HttpStatusCode? updateStatusCode = null,
|
|
HttpStatusCode? deleteStatusCode = null,
|
|
HttpStatusCode? createStoreStatusCode = null,
|
|
HttpStatusCode? getStoreStatusCode = null)
|
|
{
|
|
this._searchMemoriesResponse = searchMemoriesResponse ?? """{"memories":[]}""";
|
|
this._updateMemoriesResponse = updateMemoriesResponse ?? """{"update_id":"test-update-id","status":"queued"}""";
|
|
this._searchStatusCode = searchStatusCode ?? HttpStatusCode.OK;
|
|
this._updateStatusCode = updateStatusCode ?? HttpStatusCode.OK;
|
|
this._deleteStatusCode = deleteStatusCode ?? HttpStatusCode.NoContent;
|
|
this._createStoreStatusCode = createStoreStatusCode ?? HttpStatusCode.Created;
|
|
this._getStoreStatusCode = getStoreStatusCode ?? HttpStatusCode.NotFound;
|
|
}
|
|
|
|
public string? LastRequestUri { get; private set; }
|
|
public string? LastRequestBody { get; private set; }
|
|
public HttpMethod? LastRequestMethod { get; private set; }
|
|
|
|
protected override async Task<HttpResponseMessage> SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
|
|
{
|
|
this.LastRequestUri = request.RequestUri?.ToString();
|
|
this.LastRequestMethod = request.Method;
|
|
|
|
if (request.Content != null)
|
|
{
|
|
#if NET472
|
|
this.LastRequestBody = await request.Content.ReadAsStringAsync().ConfigureAwait(false);
|
|
#else
|
|
this.LastRequestBody = await request.Content.ReadAsStringAsync(cancellationToken).ConfigureAwait(false);
|
|
#endif
|
|
}
|
|
|
|
string path = request.RequestUri?.AbsolutePath ?? "";
|
|
|
|
// Route based on path and method
|
|
if (path.Contains("/memory-stores/") && path.Contains("/search") && request.Method == HttpMethod.Post)
|
|
{
|
|
return CreateResponse(this._searchStatusCode, this._searchMemoriesResponse);
|
|
}
|
|
|
|
if (path.Contains("/memory-stores/") && path.Contains("/memories") && request.Method == HttpMethod.Post)
|
|
{
|
|
return CreateResponse(this._updateStatusCode, this._updateMemoriesResponse);
|
|
}
|
|
|
|
if (path.Contains("/memory-stores/") && path.Contains("/scopes") && request.Method == HttpMethod.Delete)
|
|
{
|
|
return CreateResponse(this._deleteStatusCode, "");
|
|
}
|
|
|
|
if (path.Contains("/memory-stores") && request.Method == HttpMethod.Post)
|
|
{
|
|
return CreateResponse(this._createStoreStatusCode, """{"name":"test-store","status":"active"}""");
|
|
}
|
|
|
|
if (path.Contains("/memory-stores/") && request.Method == HttpMethod.Get)
|
|
{
|
|
return CreateResponse(this._getStoreStatusCode, """{"name":"test-store","status":"active"}""");
|
|
}
|
|
|
|
// Default response
|
|
return CreateResponse(HttpStatusCode.NotFound, "{}");
|
|
}
|
|
|
|
private static HttpResponseMessage CreateResponse(HttpStatusCode statusCode, string? content)
|
|
{
|
|
return new HttpResponseMessage(statusCode)
|
|
{
|
|
Content = new StringContent(content ?? "{}", Encoding.UTF8, "application/json")
|
|
};
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Mock token credential for testing.
|
|
/// </summary>
|
|
internal sealed class MockTokenCredential : TokenCredential
|
|
{
|
|
public override AccessToken GetToken(TokenRequestContext requestContext, CancellationToken cancellationToken)
|
|
{
|
|
return new AccessToken("mock-token", DateTimeOffset.UtcNow.AddHours(1));
|
|
}
|
|
|
|
public override ValueTask<AccessToken> GetTokenAsync(TokenRequestContext requestContext, CancellationToken cancellationToken)
|
|
{
|
|
return new ValueTask<AccessToken>(new AccessToken("mock-token", DateTimeOffset.UtcNow.AddHours(1)));
|
|
}
|
|
}
|
|
|
|
/// <summary>
|
|
/// Source-generated JSON serializer context for unit test types.
|
|
/// </summary>
|
|
[JsonSourceGenerationOptions(PropertyNamingPolicy = JsonKnownNamingPolicy.CamelCase)]
|
|
[JsonSerializable(typeof(TestState))]
|
|
[JsonSerializable(typeof(TestScope))]
|
|
internal sealed partial class TestJsonContext : JsonSerializerContext
|
|
{
|
|
}
|
|
|
|
/// <summary>
|
|
/// Test state class for deserialization tests.
|
|
/// </summary>
|
|
internal sealed class TestState
|
|
{
|
|
public TestScope? Scope { get; set; }
|
|
}
|
|
|
|
/// <summary>
|
|
/// Test scope class for deserialization tests.
|
|
/// </summary>
|
|
internal sealed class TestScope
|
|
{
|
|
public string? Scope { get; set; }
|
|
}
|