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>
133 lines
5.2 KiB
C#
133 lines
5.2 KiB
C#
// Copyright (c) Microsoft. All rights reserved.
|
|
|
|
using System;
|
|
using System.Threading.Tasks;
|
|
using Azure.AI.Projects;
|
|
using Azure.Identity;
|
|
using Microsoft.Extensions.Configuration;
|
|
using Shared.IntegrationTests;
|
|
|
|
namespace Microsoft.Agents.AI.FoundryMemory.IntegrationTests;
|
|
|
|
/// <summary>
|
|
/// Integration tests for <see cref="FoundryMemoryProvider"/> against a configured Azure AI Foundry Memory service.
|
|
/// </summary>
|
|
/// <remarks>
|
|
/// These integration tests are skipped by default and require a live Azure AI Foundry Memory service.
|
|
/// The tests need to be updated to use the new AIAgent-based API pattern.
|
|
/// Set <see cref="SkipReason"/> to null to enable them after configuring the service.
|
|
/// </remarks>
|
|
public sealed class FoundryMemoryProviderTests : IDisposable
|
|
{
|
|
private const string SkipReason = "Requires an Azure AI Foundry Memory service configured"; // Set to null to enable.
|
|
|
|
private readonly AIProjectClient? _client;
|
|
private readonly string? _memoryStoreName;
|
|
private readonly string? _deploymentName;
|
|
private bool _disposed;
|
|
|
|
public FoundryMemoryProviderTests()
|
|
{
|
|
IConfigurationRoot configuration = new ConfigurationBuilder()
|
|
.AddJsonFile(path: "testsettings.json", optional: true, reloadOnChange: true)
|
|
.AddJsonFile(path: "testsettings.development.json", optional: true, reloadOnChange: true)
|
|
.AddEnvironmentVariables()
|
|
.AddUserSecrets<FoundryMemoryProviderTests>(optional: true)
|
|
.Build();
|
|
|
|
var foundrySettings = configuration.GetSection("FoundryMemory").Get<FoundryMemoryConfiguration>();
|
|
|
|
if (foundrySettings is not null &&
|
|
!string.IsNullOrWhiteSpace(foundrySettings.Endpoint) &&
|
|
!string.IsNullOrWhiteSpace(foundrySettings.MemoryStoreName))
|
|
{
|
|
this._client = new AIProjectClient(new Uri(foundrySettings.Endpoint), new AzureCliCredential());
|
|
this._memoryStoreName = foundrySettings.MemoryStoreName;
|
|
this._deploymentName = foundrySettings.DeploymentName ?? "gpt-4.1-mini";
|
|
}
|
|
}
|
|
|
|
[Fact(Skip = SkipReason)]
|
|
public async Task CanAddAndRetrieveUserMemoriesAsync()
|
|
{
|
|
// Arrange
|
|
FoundryMemoryProvider memoryProvider = new(
|
|
this._client!,
|
|
this._memoryStoreName!,
|
|
stateInitializer: _ => new(new FoundryMemoryProviderScope("it-user-1")));
|
|
|
|
AIAgent agent = await this._client!.CreateAIAgentAsync(this._deploymentName!,
|
|
options: new ChatClientAgentOptions { AIContextProviders = [memoryProvider] });
|
|
|
|
AgentSession session = await agent.CreateSessionAsync();
|
|
|
|
await memoryProvider.EnsureStoredMemoriesDeletedAsync(session);
|
|
|
|
// Act
|
|
AgentResponse resultBefore = await agent.RunAsync("What is my name?", session);
|
|
Assert.DoesNotContain("Caoimhe", resultBefore.Text);
|
|
|
|
await agent.RunAsync("Hello, my name is Caoimhe.", session);
|
|
await memoryProvider.WhenUpdatesCompletedAsync();
|
|
await Task.Delay(2000);
|
|
|
|
AgentResponse resultAfter = await agent.RunAsync("What is my name?", session);
|
|
|
|
// Cleanup
|
|
await memoryProvider.EnsureStoredMemoriesDeletedAsync(session);
|
|
|
|
// Assert
|
|
Assert.Contains("Caoimhe", resultAfter.Text);
|
|
}
|
|
|
|
[Fact(Skip = SkipReason)]
|
|
public async Task DoesNotLeakMemoriesAcrossScopesAsync()
|
|
{
|
|
// Arrange
|
|
FoundryMemoryProvider memoryProvider1 = new(
|
|
this._client!,
|
|
this._memoryStoreName!,
|
|
stateInitializer: _ => new(new FoundryMemoryProviderScope("it-scope-a")));
|
|
|
|
FoundryMemoryProvider memoryProvider2 = new(
|
|
this._client!,
|
|
this._memoryStoreName!,
|
|
stateInitializer: _ => new(new FoundryMemoryProviderScope("it-scope-b")));
|
|
|
|
AIAgent agent1 = await this._client!.CreateAIAgentAsync(this._deploymentName!,
|
|
options: new ChatClientAgentOptions { AIContextProviders = [memoryProvider1] });
|
|
AIAgent agent2 = await this._client!.CreateAIAgentAsync(this._deploymentName!,
|
|
options: new ChatClientAgentOptions { AIContextProviders = [memoryProvider2] });
|
|
|
|
AgentSession session1 = await agent1.CreateSessionAsync();
|
|
AgentSession session2 = await agent2.CreateSessionAsync();
|
|
|
|
await memoryProvider1.EnsureStoredMemoriesDeletedAsync(session1);
|
|
await memoryProvider2.EnsureStoredMemoriesDeletedAsync(session2);
|
|
|
|
// Act - add memory only to scope A
|
|
await agent1.RunAsync("Hello, I'm an AI tutor and my name is Caoimhe.", session1);
|
|
await memoryProvider1.WhenUpdatesCompletedAsync();
|
|
await Task.Delay(2000);
|
|
|
|
AgentResponse result1 = await agent1.RunAsync("What is your name?", session1);
|
|
AgentResponse result2 = await agent2.RunAsync("What is your name?", session2);
|
|
|
|
// Assert
|
|
Assert.Contains("Caoimhe", result1.Text);
|
|
Assert.DoesNotContain("Caoimhe", result2.Text);
|
|
|
|
// Cleanup
|
|
await memoryProvider1.EnsureStoredMemoriesDeletedAsync(session1);
|
|
await memoryProvider2.EnsureStoredMemoriesDeletedAsync(session2);
|
|
}
|
|
|
|
public void Dispose()
|
|
{
|
|
if (!this._disposed)
|
|
{
|
|
this._disposed = true;
|
|
}
|
|
}
|
|
}
|