feat: ✨ some refactoring in the agents

Author: mehdihadeli

Directory.Packages.props

@@ -57,6 +57,10 @@ Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Recommendation", "src\Servi
 EndProject
 Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Reviews", "src\Services\Reviews\Reviews.csproj", "{AC102A54-FBB1-4BD6-90B4-CE62D5F4E328}"
 EndProject
+Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "UI", "UI", "{407B9EA3-0148-4611-AFD5-D92E7C10DE51}"
+EndProject
+Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "ConsoleApp", "src\UI\ConsoleApp\ConsoleApp.csproj", "{2D4DADFB-AA8E-4664-BA9B-E9FE40874E48}"
+EndProject
 Global
 	GlobalSection(SolutionConfigurationPlatforms) = preSolution
 		Debug|Any CPU = Debug|Any CPU
@@ -83,6 +87,8 @@ Global
 		{510C732A-FD95-4DE4-8187-76A7AD012633} = {14C757F9-E1E3-429C-AF7E-E14F28E6F852}
 		{B9F571EB-1AF4-4343-B763-5DE23EE8FC28} = {14C757F9-E1E3-429C-AF7E-E14F28E6F852}
 		{AC102A54-FBB1-4BD6-90B4-CE62D5F4E328} = {14C757F9-E1E3-429C-AF7E-E14F28E6F852}
+		{407B9EA3-0148-4611-AFD5-D92E7C10DE51} = {5DF8C01D-D181-498E-97DE-3535FFD4FFB5}
+		{2D4DADFB-AA8E-4664-BA9B-E9FE40874E48} = {407B9EA3-0148-4611-AFD5-D92E7C10DE51}
 	EndGlobalSection
 	GlobalSection(ProjectConfigurationPlatforms) = postSolution
 		{15C0604F-9107-44C1-9CDB-B6C407084C52}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
@@ -125,5 +131,9 @@ Global
 		{AC102A54-FBB1-4BD6-90B4-CE62D5F4E328}.Debug|Any CPU.Build.0 = Debug|Any CPU
 		{AC102A54-FBB1-4BD6-90B4-CE62D5F4E328}.Release|Any CPU.ActiveCfg = Release|Any CPU
 		{AC102A54-FBB1-4BD6-90B4-CE62D5F4E328}.Release|Any CPU.Build.0 = Release|Any CPU
+		{2D4DADFB-AA8E-4664-BA9B-E9FE40874E48}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
+		{2D4DADFB-AA8E-4664-BA9B-E9FE40874E48}.Debug|Any CPU.Build.0 = Debug|Any CPU
+		{2D4DADFB-AA8E-4664-BA9B-E9FE40874E48}.Release|Any CPU.ActiveCfg = Release|Any CPU
+		{2D4DADFB-AA8E-4664-BA9B-E9FE40874E48}.Release|Any CPU.Build.0 = Release|Any CPU
 	EndGlobalSection
 EndGlobal

gen-ai-app-samples.sln

@@ -1,6 +1,6 @@
 {
   "sdk": {
-    "version": "9.0.303",
+    "version": "9.0.305",
     "rollForward": "latestFeature",
     "allowPrerelease": true
   }

global.json

@@ -36,6 +36,7 @@ string agentPath
         // https://github.com/microsoft/semantic-kernel/blob/main/dotnet/src/Agents/A2A/A2AAgent.cs#L178
         builder.Services.AddHttpClient(agentName, client => client.BaseAddress = agentUri);
 
+        // use external service a2a based on http protocol, not json/rpc
         builder.Services.AddKeyedSingleton<A2AAgent>(
             agentName,
             (sp, _) =>
@@ -62,7 +63,7 @@ private static async Task<A2AAgent> CreateAgentAsync(HttpClient httpClient, Uri
         ArgumentNullException.ThrowIfNull(agentHostUri);
 
         var client = new A2AClient(agentHostUri, httpClient);
-        // resolve `agent card` from a2a discovery endpoint `.well-known/agent-card.json`
+        // resolve `agent card` from a2a http discovery endpoint `.well-known/agent-card.json` in the external service
         var cardResolver = new A2ACardResolver(
             baseUrl: agentHostUri,
             httpClient: httpClient,

src/BuildingBlocks/AI/A2A/A2AClientExtensions.cs

@@ -195,6 +195,7 @@ public static void AddStdioMcpClient(
                         WorkingDirectory = workingDirectory,
                     };
 
+                // use JSON-RPC 2.0 over stdio
                 StdioClientTransport stdioTransport = new(stdioOptions, loggerFactory);
 
                 McpClient mcpClient = McpClient

src/BuildingBlocks/AI/MCP/DependencyInjectionExtensions.cs

@@ -18,8 +18,6 @@ namespace BuildingBlocks.AI.SemanticKernel;
 
 public static class DependencyInjectionExtensions
 {
-    private const string ActivitySourceName = "Microsoft.SemanticKernel*";
-
     public static IHostApplicationBuilder AddSemanticKernel(this IHostApplicationBuilder builder)
     {
         var options = builder.Configuration.BindOptions<SemanticKernelOptions>();
@@ -147,10 +145,12 @@ private static void AddOpenTelemetry(IHostApplicationBuilder builder)
         // https://learn.microsoft.com/en-us/semantic-kernel/concepts/enterprise-readiness/observability/telemetry-with-aspire-dashboard?tabs=Powershell&pivots=programming-language-csharp
         AppContext.SetSwitch("Microsoft.SemanticKernel.Experimental.GenAI.EnableOTelDiagnosticsSensitive", true);
 
+        var activitySourceName = "Microsoft.SemanticKernel*";
+
         builder
             .Services.AddOpenTelemetry()
-            .WithTracing(x => x.AddSource(ActivitySourceName))
-            .WithMetrics(x => x.AddMeter(ActivitySourceName));
+            .WithTracing(x => x.AddSource(activitySourceName))
+            .WithMetrics(x => x.AddMeter(activitySourceName));
 
         builder
             .Services.AddOpenTelemetry()

src/BuildingBlocks/AI/SemanticKernel/DependencyInjectionExtensions.cs

@@ -53,6 +53,9 @@ public static PromptExecutionSettings GetProviderExecutionSettings(SemanticKerne
 
                 return new OllamaPromptExecutionSettings
                 {
+                    FunctionChoiceBehavior = FunctionChoiceBehavior.Auto(
+                        options: new FunctionChoiceBehaviorOptions { RetainArgumentTypes = true }
+                    ),
                     Temperature = semanticKernelOptions.Temperature,
                     ExtensionData = ollamaMergedExtensionData,
                 };

src/BuildingBlocks/AI/SemanticKernel/SemanticKernelExecutionSettings.cs

@@ -16,6 +16,7 @@ public class SemanticKernelOptions
     public ProviderType EmbeddingProviderType { get; set; } = ProviderType.Ollama;
     public float Temperature { get; set; } = 0.8f;
     public double? SearchThreshold { get; set; }
+    public int MaximumInvocationCount { get; set; }
 
     public Dictionary<string, object> ChatExtensionData { get; set; } = new();
 }

src/BuildingBlocks/AI/SemanticKernel/SemanticKernelOptions.cs

@@ -16,6 +16,7 @@
         <PackageReference Include="Microsoft.AspNetCore.OpenApi"/>
         <PackageReference Include="Npgsql.EntityFrameworkCore.PostgreSQL" />
         <PackageReference Include="Scalar.AspNetCore"/>
+        <PackageReference Include="Spectre.Console" />
         <PackageReference Include="Swashbuckle.AspNetCore.SwaggerUI"/>
         <PackageReference Include="System.Net.Http.Json" />
         <PackageReference Include="System.Text.Json" />

src/BuildingBlocks/BuildingBlocks.csproj

@@ -10,6 +10,7 @@
 using Microsoft.Extensions.AI;
 using Microsoft.Extensions.Options;
 using Microsoft.Extensions.VectorData;
+using Microsoft.SemanticKernel;
 using Microsoft.SemanticKernel.ChatCompletion;
 
 namespace BuildingBlocks.VectorDB.SearchServices;
@@ -18,14 +19,15 @@ public sealed class SemanticSearch(
     IChatCompletionService chatCompletionService,
     IEmbeddingGenerator<string, Embedding<float>> embeddingGenerator,
     VectorStore vectorStore,
+    Kernel kernel,
     IOptions<SemanticKernelOptions> semanticKernelOptions
 ) : ISemanticSearch
 {
     private readonly SemanticKernelOptions _semanticKernelOptions = semanticKernelOptions.Value;
 
     private const string InitPrompt =
         @"
-You are an intelligent, friendly e-commerce assistant designed to help shoppers understand their search results. 
+You are an intelligent, friendly GenAI-Eshop assistant designed to help users understand their search results. 
 
 Follow these guidelines in every response:
 - Always respond in a warm, professional, and conversational tone.
@@ -58,19 +60,20 @@ public async Task<VectorSearchResult<TEntity>> HybridSearchAsync<TVectorEntity,
 
         var collection = vectorStore.GetCollection<Guid, TVectorEntity>(typeof(TVectorEntity).Name.Underscore());
         await collection.EnsureCollectionExistsAsync(cancellationToken);
-        var vectorCollection = (IKeywordHybridSearchable<TVectorEntity>)collection;
 
         // https://learn.microsoft.com/en-us/semantic-kernel/concepts/vector-store-connectors/hybrid-search?pivots=programming-language-csharp#top-and-skip
         var options = new HybridSearchOptions<TVectorEntity>
         {
             VectorProperty = r => r.Vector,
+            // field to search on the full-text search column with `IsFullTextIndexed=true` attribute
             AdditionalProperty = fullTextSearchFiled ?? (r => r.Description),
             Skip = skip,
             IncludeVectors = false,
             // https://learn.microsoft.com/en-us/semantic-kernel/concepts/vector-store-connectors/vector-search?pivots=programming-language-csharp#filter
             Filter = filter,
         };
 
+        var vectorCollection = (IKeywordHybridSearchable<TVectorEntity>)collection;
         // https://learn.microsoft.com/en-us/semantic-kernel/concepts/vector-store-connectors/hybrid-search?pivots=programming-language-csharp
         // keywords will use to search on full-text search column with `IsFullTextIndexed=true` attribute
         var searchResults = vectorCollection.HybridSearchAsync(
@@ -146,7 +149,8 @@ public async Task<VectorSearchResult<TEntity>> HybridSearchAsync<TVectorEntity,
             var aiResponse = await chatCompletionService.GetChatMessageContentAsync(
                 chatHistory: history,
                 // https://ollama.com/blog/thinking
-                executionSettings: SemanticKernelExecutionSettings.GetDefaultSettings(_semanticKernelOptions),
+                executionSettings: SemanticKernelExecutionSettings.GetProviderExecutionSettings(_semanticKernelOptions),
+                kernel: kernel,
                 cancellationToken: cancellationToken
             );
 
@@ -173,9 +177,6 @@ public async Task<VectorSearchResult<TEntity>> SemanticSearchAsync<TVectorEntity
         where TVectorEntity : VectorEntityBase
         where TEntity : Entity
     {
-        // https://learn.microsoft.com/en-us/semantic-kernel/concepts/ai-services/embedding-generation/?tabs=csharp-Ollama&pivots=programming-language-csharp#using-text-embedding-generation-services
-        var vector = await embeddingGenerator.GenerateVectorAsync(searchTerm, cancellationToken: cancellationToken);
-
         var collection = vectorStore.GetCollection<Guid, TVectorEntity>(typeof(TVectorEntity).Name.Underscore());
         await collection.EnsureCollectionExistsAsync(cancellationToken);
 
@@ -188,6 +189,8 @@ public async Task<VectorSearchResult<TEntity>> SemanticSearchAsync<TVectorEntity
             Filter = filter,
         };
 
+        // https://learn.microsoft.com/en-us/semantic-kernel/concepts/ai-services/embedding-generation/?tabs=csharp-Ollama&pivots=programming-language-csharp#using-text-embedding-generation-services
+        var vector = await embeddingGenerator.GenerateVectorAsync(searchTerm, cancellationToken: cancellationToken);
         var searchResults = collection.SearchAsync(
             searchValue: vector,
             top: take,
@@ -265,7 +268,8 @@ Make the response sound natural and helpful to the user.
                 // https://ollama.com/blog/thinking
                 // - in ollama cli we can `ollama run qwen3:0.6b --think=false` to turn off thinking
                 // - in ollama api with passing `think=false` as parameter
-                executionSettings: SemanticKernelExecutionSettings.GetDefaultSettings(_semanticKernelOptions),
+                executionSettings: SemanticKernelExecutionSettings.GetProviderExecutionSettings(_semanticKernelOptions),
+                kernel: kernel,
                 cancellationToken: cancellationToken
             );