Fix compilation errors for SK 1.74.0 API changes

Author: jncchds

src/ABook.Agents/AgentOrchestrator.cs

@@ -89,7 +89,7 @@ public async Task StartContinuityCheckAsync(int bookId, CancellationToken ct = d
             throw new InvalidOperationException("An agent is already running for this book.");
         try
         {
-            await _continuity.CheckAsync(bookId, ct);
+            await _continuity.CheckAsync(bookId, ct: ct);
             _state.SetStatus(bookId, new AgentRunStatus(AgentRole.ContinuityChecker, "Done", null));
         }
         catch
@@ -147,7 +147,7 @@ await _notifier.NotifyWorkflowProgressAsync(bookId,
             ct.ThrowIfCancellationRequested();
             _state.UpdateRunRole(bookId, AgentRole.ContinuityChecker, null);
             await _notifier.NotifyWorkflowProgressAsync(bookId, "Running final continuity check…", false, ct);
-            await _continuity.CheckAsync(bookId, ct);
+            await _continuity.CheckAsync(bookId, ct: ct);
 
             _state.SetStatus(bookId, new AgentRunStatus(AgentRole.ContinuityChecker, "Done", null));
             await _notifier.NotifyWorkflowProgressAsync(bookId, "Workflow complete!", true, ct);
@@ -260,7 +260,7 @@ await _notifier.NotifyWorkflowProgressAsync(bookId,
             ct.ThrowIfCancellationRequested();
             _state.UpdateRunRole(bookId, AgentRole.ContinuityChecker, null);
             await _notifier.NotifyWorkflowProgressAsync(bookId, "Running continuity check…", false, ct);
-            await _continuity.CheckAsync(bookId, ct);
+            await _continuity.CheckAsync(bookId, ct: ct);
 
             _state.SetStatus(bookId, new AgentRunStatus(AgentRole.ContinuityChecker, "Done", null));
             await _notifier.NotifyWorkflowProgressAsync(bookId, "Workflow complete!", true, ct);

src/ABook.Agents/ContinuityCheckerAgent.cs

@@ -123,7 +123,7 @@ manuscript is consistent so far.
                     $"Chapter {c.Number}: {c.Title}\nOutline: {c.Outline}\nContent excerpt: {c.Content?[..Math.Min(600, c.Content?.Length ?? 0)]}..."))
                 : "(No preceding chapters yet.)";
 
-            var currentExcerpt = c =>
+            var currentExcerpt =
                 $"Chapter {currentChapter.Number}: {currentChapter.Title}\n" +
                 $"Outline: {currentChapter.Outline}\n" +
                 $"Content: {currentChapter.Content}";
@@ -137,7 +137,7 @@ against the preceding chapters. Do NOT report problems between the preceding cha
                     {prevSynopsis}
 
                     ## Chapter Under Review
-                    {currentExcerpt(currentChapter)}
+                    {currentExcerpt}
                     """
                 : $"""
                     Check Chapter {currentChapter.Number} ("{currentChapter.Title}") for continuity issues
@@ -147,7 +147,7 @@ against the preceding chapters. Do NOT report problems between the preceding cha
                     {prevSynopsis}
 
                     ## Chapter Under Review
-                    {currentExcerpt(currentChapter)}
+                    {currentExcerpt}
 
                     ## Detailed Passages (retrieved for continuity review)
                     {ragContext}

src/ABook.Infrastructure/Llm/LlmProviderFactory.cs

@@ -1,11 +1,13 @@
-#pragma warning disable SKEXP0070
+#pragma warning disable SKEXP0070, SKEXP0010
 
 using ABook.Core.Interfaces;
 using ABook.Core.Models;
 using Microsoft.SemanticKernel;
 using Microsoft.SemanticKernel.ChatCompletion;
 using Microsoft.SemanticKernel.Connectors.Ollama;
 using Microsoft.SemanticKernel.Embeddings;
+using OpenAI;
+using System.ClientModel;
 
 namespace ABook.Infrastructure.Llm;
 
@@ -18,8 +20,7 @@ public IChatCompletionService CreateChatCompletion(LlmConfiguration config) =>
             LlmProvider.OpenAI => new Microsoft.SemanticKernel.Connectors.OpenAI.OpenAIChatCompletionService(
                 config.ModelName, config.ApiKey ?? throw new InvalidOperationException("OpenAI requires an API key.")),
             LlmProvider.LMStudio => new Microsoft.SemanticKernel.Connectors.OpenAI.OpenAIChatCompletionService(
-                config.ModelName, config.ApiKey ?? "lm-studio",
-                endpoint: new Uri(config.Endpoint.TrimEnd('/') + "/v1")),
+                config.ModelName, new Uri(config.Endpoint.TrimEnd('/') + "/v1"), config.ApiKey ?? "lm-studio"),
             _ => throw new NotSupportedException($"Provider {config.Provider} is not yet supported.")
         };
 
@@ -32,8 +33,9 @@ public ITextEmbeddingGenerationService CreateEmbeddingGeneration(LlmConfiguratio
             LlmProvider.OpenAI => new Microsoft.SemanticKernel.Connectors.OpenAI.OpenAITextEmbeddingGenerationService(
                 embeddingModel, config.ApiKey ?? throw new InvalidOperationException("OpenAI requires an API key.")),
             LlmProvider.LMStudio => new Microsoft.SemanticKernel.Connectors.OpenAI.OpenAITextEmbeddingGenerationService(
-                embeddingModel, config.ApiKey ?? "lm-studio",
-                endpoint: new Uri(config.Endpoint.TrimEnd('/') + "/v1")),
+                embeddingModel, new OpenAIClient(
+                    new ApiKeyCredential(config.ApiKey ?? "lm-studio"),
+                    new OpenAIClientOptions { Endpoint = new Uri(config.Endpoint.TrimEnd('/') + "/v1") })),
             _ => throw new NotSupportedException($"Provider {config.Provider} is not yet supported for embeddings.")
         };
     }
@@ -57,9 +59,14 @@ public Kernel CreateKernel(LlmConfiguration config)
             case LlmProvider.LMStudio:
                 var lmKey = config.ApiKey ?? "lm-studio";
                 var lmEndpoint = new Uri(config.Endpoint.TrimEnd('/') + "/v1");
-                builder.AddOpenAIChatCompletion(config.ModelName, lmKey, endpoint: lmEndpoint);
+                builder.AddOpenAIChatCompletion(config.ModelName, lmEndpoint, lmKey);
                 if (!string.IsNullOrWhiteSpace(config.EmbeddingModelName))
-                    builder.AddOpenAITextEmbeddingGeneration(config.EmbeddingModelName, lmKey, endpoint: lmEndpoint);
+                {
+                    var lmClient = new OpenAIClient(
+                        new ApiKeyCredential(lmKey),
+                        new OpenAIClientOptions { Endpoint = lmEndpoint });
+                    builder.AddOpenAITextEmbeddingGeneration(config.EmbeddingModelName, lmClient);
+                }
                 break;
             default:
                 throw new NotSupportedException($"Provider {config.Provider} is not yet supported.");