Add AI support to the Microsoft.Testing.Platform#6777
Add AI support to the Microsoft.Testing.Platform#6777MarcoRossignoli merged 21 commits intomicrosoft:mainfrom
Microsoft.Testing.Platform#6777Conversation
src/Platform/Microsoft.Testing.Platform.AI/IChatClientFactory.cs
Outdated
Show resolved
Hide resolved
...m/Microsoft.Testing.Extensions.AzureFoundry/Microsoft.Testing.Extensions.AzureFoundry.csproj
Show resolved
Hide resolved
src/Platform/Microsoft.Testing.Extensions.AzFoundry/OpenAIChatClientFactory.cs
Outdated
Show resolved
Hide resolved
src/Platform/Microsoft.Testing.Platform.AI/Microsoft.Testing.Platform.AI.csproj
Show resolved
Hide resolved
Co-authored-by: Youssef Victor <youssefvictor00@gmail.com>
Co-authored-by: Youssef Victor <youssefvictor00@gmail.com>
At the moment nope it's experimental for now...I need to play a bit with the usage to understand, I would not ship the core ai and the extension lib for now, the plumbing is internal and should not affect the normal run if not registered. We can try to add some real test when complete...but not sure if we will be able to test the AI interaction, but let's see maybe we will do it in some way. |
|
Should we avoid publishing it publicly for now until we implement something that uses it and we have more confidence of the design? Maybe we should set |
wonder if we can ship it inside not official nuget feed but in the dogfooding testing one so I can pick it up to do some experiment outside the testfx repo. We can always delist there in case. |
|
I'm fine publishing to test-tools feed and not nuget.org. But I want to ensure we don't end up publishing to nuget.org accidentally. I think setting |
Done! |
src/Platform/Microsoft.Testing.Extensions.AzureFoundry/OpenAIChatClientProvider.cs
Outdated
Show resolved
Hide resolved
src/Platform/Microsoft.Testing.Platform.AI/Microsoft.Testing.Platform.AI.csproj
Outdated
Show resolved
Hide resolved
src/Platform/Microsoft.Testing.Platform/AI/ChatClientManager.cs
Outdated
Show resolved
Hide resolved
…mplementation - Make AzureOpenAIChatClientProvider internal - Add public AddAzureOpenAIChatClientProvider extension method - Extract IEnvironment service and store in field instead of keeping IServiceProvider - Remove unnecessary InternalsVisibleTo from Microsoft.Testing.Platform.AI - Update Playground sample to use new extension method
…nAIChatClientProvider
...m/Microsoft.Testing.Extensions.AzureFoundry/Microsoft.Testing.Extensions.AzureFoundry.csproj
Show resolved
Hide resolved
...m/Microsoft.Testing.Extensions.AzureFoundry/Microsoft.Testing.Extensions.AzureFoundry.csproj
Show resolved
Hide resolved
Evangelink
left a comment
There was a problem hiding this comment.
Playground project is now run during tests and so it fails
src/Platform/Microsoft.Testing.Extensions.AzureFoundry/Resources/ExtensionResources.resx
Outdated
Show resolved
Hide resolved
src/Platform/Microsoft.Testing.Extensions.AzureFoundry/OpenAIChatClientProvider.cs
Outdated
Show resolved
Hide resolved
src/Platform/Microsoft.Testing.Extensions.AzureFoundry/OpenAIChatClientProvider.cs
Outdated
Show resolved
Hide resolved
- Replace string.IsNullOrEmpty with RoslynString.IsNullOrEmpty - Consolidate resource strings into single parameterized entry - Update constructor to take IEnvironment directly instead of IServiceProvider - Add Experimental attribute to IChatClientProvider interface - Rename SupportsToolCalling to HasToolsCapability - Rename IChatClientManager methods for clarity: - AddChatClientProvider -> SetChatClientProviderFactory - RegisterChatClientProvider -> InstantiateChatClientProvider - Add Polyfill package reference to Microsoft.Testing.Platform.AI - Update all documentation and xlf files
src/Platform/Microsoft.Testing.Platform/AI/IChatClientManager.cs
Outdated
Show resolved
Hide resolved
src/Platform/Microsoft.Testing.Platform/AI/IChatClientManager.cs
Outdated
Show resolved
Hide resolved
- Rename SetChatClientProviderFactory to AddChatClientProvider to match other API naming conventions - Rename InstantiateChatClientProvider to BuildChatClients for better clarity
Add AI support to the
Microsoft.Testing.Platform