.Net: Add fix for plugin naming collisions (#12371)

### Motivation and Context

#12320

### Description

Add code to ensure that naming collisions are avoided when using ai
context provider tools.

### Contribution Checklist

<!-- Before submitting this PR, please make sure: -->

- [ ] The code builds clean without any errors or warnings
- [ ] The PR follows the [SK Contribution
Guidelines](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md#development-scripts)
raises no violations
- [ ] All unit tests pass, and I have added new tests where possible
- [ ] I didn't break anyone 😄

Author: westey-m

dotnet/src/SemanticKernel.Core/CompatibilitySuppressions.xml

@@ -1,6 +1,13 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!-- https://learn.microsoft.com/dotnet/fundamentals/package-validation/diagnostic-ids -->
 <Suppressions xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xsd="http://www.w3.org/2001/XMLSchema">
+  <Suppression>
+    <DiagnosticId>CP0002</DiagnosticId>
+    <Target>M:Microsoft.SemanticKernel.AIContextExtensions.AddFromAIContext(System.Collections.Generic.ICollection{Microsoft.SemanticKernel.KernelPlugin},Microsoft.SemanticKernel.AIContext,System.String)</Target>
+    <Left>lib/net8.0/Microsoft.SemanticKernel.Core.dll</Left>
+    <Right>lib/net8.0/Microsoft.SemanticKernel.Core.dll</Right>
+    <IsBaselineSuppression>true</IsBaselineSuppression>
+  </Suppression>
   <Suppression>
     <DiagnosticId>CP0002</DiagnosticId>
     <Target>M:Microsoft.SemanticKernel.Data.TextSearchProvider.#ctor(Microsoft.SemanticKernel.Data.ITextSearch,Microsoft.SemanticKernel.Data.TextSearchProviderOptions)</Target>
@@ -22,6 +29,13 @@
     <Right>lib/net8.0/Microsoft.SemanticKernel.Core.dll</Right>
     <IsBaselineSuppression>true</IsBaselineSuppression>
   </Suppression>
+  <Suppression>
+    <DiagnosticId>CP0002</DiagnosticId>
+    <Target>M:Microsoft.SemanticKernel.AIContextExtensions.AddFromAIContext(System.Collections.Generic.ICollection{Microsoft.SemanticKernel.KernelPlugin},Microsoft.SemanticKernel.AIContext,System.String)</Target>
+    <Left>lib/netstandard2.0/Microsoft.SemanticKernel.Core.dll</Left>
+    <Right>lib/netstandard2.0/Microsoft.SemanticKernel.Core.dll</Right>
+    <IsBaselineSuppression>true</IsBaselineSuppression>
+  </Suppression>
   <Suppression>
     <DiagnosticId>CP0002</DiagnosticId>
     <Target>M:Microsoft.SemanticKernel.Data.TextSearchProvider.#ctor(Microsoft.SemanticKernel.Data.ITextSearch,Microsoft.SemanticKernel.Data.TextSearchProviderOptions)</Target>

dotnet/src/SemanticKernel.Core/Memory/AIContextExtensions.cs

@@ -18,12 +18,24 @@ public static class AIContextExtensions
     /// </summary>
     /// <param name="plugins">The plugins collection to register the <see cref="AIFunction"/> objects on.</param>
     /// <param name="aiContext">The <see cref="AIContext"/> to get plugins from.</param>
-    /// <param name="pluginName">The name to give to the plugin.</param>
-    public static void AddFromAIContext(this ICollection<KernelPlugin> plugins, AIContext aiContext, string pluginName)
+    /// <param name="pluginName">The name to give to the plugin. This will be appended with _x where x is an ascending number, until a unique plugin name is found.</param>
+    /// <returns>The chosen plugin name.</returns>
+    public static string AddFromAIContext(this ICollection<KernelPlugin> plugins, AIContext aiContext, string pluginName)
     {
         if (aiContext.AIFunctions is { Count: > 0 })
         {
+            var originalPluginName = pluginName;
+            var counter = 1;
+
+            // Find a unique plugin name by appending a counter if necessary.
+            while (plugins.Any(x => x.Name == pluginName))
+            {
+                pluginName = $"{originalPluginName}_{counter++}";
+            }
+
             plugins.AddFromFunctions(pluginName, aiContext.AIFunctions.Select(x => x.AsKernelFunction()));
         }
+
+        return pluginName;
     }
 }

dotnet/src/SemanticKernel.UnitTests/Memory/AIContextProviderTests.cs

@@ -1,5 +1,6 @@
 // Copyright (c) Microsoft. All rights reserved.
 
+using System.Collections.Generic;
 using System.Threading;
 using System.Threading.Tasks;
 using Microsoft.Extensions.AI;
@@ -64,4 +65,28 @@ public async Task ResumingAsyncBaseImplementationSucceeds()
         // Act & Assert.
         await mockPart.Object.ResumingAsync("threadId", CancellationToken.None);
     }
+
+    [Fact]
+    public void ExtensionCanAddPluginsFromAIContextProvider()
+    {
+        var plugins = new List<KernelPlugin>();
+
+        var aiContext = new AIContext
+        {
+            AIFunctions = new List<AIFunction>
+            {
+                AIFunctionFactory.Create(() => Task.FromResult("Function1 Result"), "Function1"),
+                AIFunctionFactory.Create(() => Task.FromResult("Function2 Result"), "Function2")
+            }
+        };
+
+        Assert.Equal("TestPlugin", plugins.AddFromAIContext(aiContext, "TestPlugin"));
+        Assert.Equal("TestPlugin_1", plugins.AddFromAIContext(aiContext, "TestPlugin"));
+        Assert.Equal("TestPlugin_2", plugins.AddFromAIContext(aiContext, "TestPlugin"));
+
+        Assert.Equal(3, plugins.Count);
+        Assert.Contains(plugins, p => p.Name == "TestPlugin");
+        Assert.Contains(plugins, p => p.Name == "TestPlugin_1");
+        Assert.Contains(plugins, p => p.Name == "TestPlugin_2");
+    }
 }