Cleanup

Author: BenjaminMichaelis

EssentialCSharp.Chat.Shared/Services/AIChatService.cs

@@ -76,9 +76,6 @@ public AIChatService(IOptions<AIOptions> options, AISearchService searchService)
         bool enableContextualSearch = false,
         [System.Runtime.CompilerServices.EnumeratorCancellation] CancellationToken cancellationToken = default)
     {
-        // Add logging to help debug the conversation state issue
-        Debug.WriteLine($"GetChatCompletionStream called with previousResponseId: {previousResponseId}");
-
         var responseOptions = await CreateResponseOptionsAsync(previousResponseId, tools, reasoningEffortLevel, mcpClient: mcpClient, cancellationToken: cancellationToken);
         var enrichedPrompt = await EnrichPromptWithContext(prompt, enableContextualSearch, cancellationToken);
 
@@ -113,33 +110,24 @@ private async Task<string> EnrichPromptWithContext(string prompt, bool enableCon
             return prompt;
         }
 
-        try
-        {
-            var searchResults = await _SearchService.ExecuteVectorSearch(prompt);
-            var contextualInfo = new System.Text.StringBuilder();
-
-            contextualInfo.AppendLine("## Contextual Information");
-            contextualInfo.AppendLine("The following information might be relevant to your question:");
-            contextualInfo.AppendLine();
-
-            await foreach (var result in searchResults)
-            {
-                contextualInfo.AppendLine(System.Globalization.CultureInfo.InvariantCulture, $"**From: {result.Record.Heading}**");
-                contextualInfo.AppendLine(result.Record.ChunkText);
-                contextualInfo.AppendLine();
-            }
+        var searchResults = await _SearchService.ExecuteVectorSearch(prompt);
+        var contextualInfo = new System.Text.StringBuilder();
 
-            contextualInfo.AppendLine("## User Question");
-            contextualInfo.AppendLine(prompt);
+        contextualInfo.AppendLine("## Contextual Information");
+        contextualInfo.AppendLine("The following information might be relevant to your question:");
+        contextualInfo.AppendLine();
 
-            return contextualInfo.ToString();
-        }
-        catch (Exception ex)
+        await foreach (var result in searchResults)
         {
-            // Log the error but don't fail the request
-            Debug.WriteLine($"Error enriching prompt with context: {ex.Message}");
-            return prompt;
+            contextualInfo.AppendLine(System.Globalization.CultureInfo.InvariantCulture, $"**From: {result.Record.Heading}**");
+            contextualInfo.AppendLine(result.Record.ChunkText);
+            contextualInfo.AppendLine();
         }
+
+        contextualInfo.AppendLine("## User Question");
+        contextualInfo.AppendLine(prompt);
+
+        return contextualInfo.ToString();
     }
 
     /// <summary>

EssentialCSharp.Web/Controllers/ChatController.cs

@@ -8,7 +8,7 @@ namespace EssentialCSharp.Web.Controllers;
 
 [ApiController]
 [Route("api/[controller]")]
-[Authorize] // Require authentication for all chat endpoints
+[Authorize]
 [EnableRateLimiting("ChatEndpoint")]
 public class ChatController : ControllerBase
 {
@@ -62,7 +62,8 @@ public async Task<IActionResult> SendMessage([FromBody] ChatMessageRequest reque
         }
         catch (Exception ex)
         {
-            _Logger.LogError(ex, "Error processing chat message: {Message}", request.Message);
+            var sanitizedMessage = request.Message?.Replace("\r", "").Replace("\n", "");
+            _Logger.LogError(ex, "Error processing chat message: {Message}", sanitizedMessage);
             return StatusCode(500, new { error = "An error occurred while processing your message. Please try again." });
         }
     }
@@ -130,7 +131,8 @@ public async Task StreamMessage([FromBody] ChatMessageRequest request, Cancellat
         }
         catch (Exception ex)
         {
-            _Logger.LogError(ex, "Error processing streaming chat message: {Message}", request.Message);
+            var sanitizedMessage = request.Message?.Replace("\n", "").Replace("\r", "");
+            _Logger.LogError(ex, "Error processing streaming chat message: {Message}", sanitizedMessage);
             Response.StatusCode = 500;
             await Response.WriteAsync(JsonSerializer.Serialize(new { error = "An error occurred while processing your message. Please try again." }), cancellationToken);
         }

EssentialCSharp.Web/Program.cs

@@ -302,7 +302,7 @@ await context.HttpContext.Response.WriteAsync(
         app.UseMiddleware<ReferralMiddleware>();
 
         app.MapRazorPages();
-        app.MapDefaultControllerRoute().RequireRateLimiting("ChatEndpoint"); // Apply rate limiting to controllers
+        app.MapDefaultControllerRoute();
 
         app.MapFallbackToController("Index", "Home");
 

EssentialCSharp.Web/wwwroot/js/chat-module.js

@@ -13,10 +13,6 @@ export function useChatWidget() {
     const chatMessagesEl = ref(null);
     const chatInputField = ref(null);
     const lastResponseId = ref(null);
-    
-    // Captcha state (currently not used but referenced in template)
-    const showCaptcha = ref(false);
-    const captchaSiteKey = ref(window.HCAPTCHA_SITE_KEY || '');
 
     // Load chat history from localStorage on initialization
     function loadChatHistory() {