Address code review feedback - simplify delete operations

Co-authored-by: MikeAlhayek <24724371+MikeAlhayek@users.noreply.github.com>

Author: Copilot

src/Core/CrestApps.OrchardCore.AI.Core/Services/DefaultAIChatSessionManager.cs

@@ -169,11 +169,14 @@ public async Task<int> DeleteAllAsync(string profileId)
             collection: AIConstants.CollectionName)
             .ListAsync();
 
+        var totalDeleted = 0;
+
         foreach (var session in sessions)
         {
             _session.Delete(session, collection: AIConstants.CollectionName);
+            totalDeleted++;
         }
 
-        return sessions.Count;
+        return totalDeleted;
     }
 }

src/Modules/CrestApps.OrchardCore.AI.Chat/Controllers/AdminController.cs

@@ -209,7 +209,7 @@ public IActionResult Chat(string profileId)
         => RedirectToAction(nameof(Index), new { profileId });
 
     [Admin("ai/chat/session/delete/{sessionId}", "DeleteChatSession")]
-    public async Task<IActionResult> Delete(string sessionId, string profileId)
+    public async Task<IActionResult> Delete(string sessionId)
     {
         var chatSession = await _sessionManager.FindAsync(sessionId);
 
@@ -239,12 +239,7 @@ public async Task<IActionResult> Delete(string sessionId, string profileId)
             await _notifier.ErrorAsync(H["Unable to delete the chat session."]);
         }
 
-        if (!string.IsNullOrEmpty(profileId))
-        {
-            return RedirectToAction(nameof(History), new { profileId });
-        }
-
-        return RedirectToAction(nameof(Index), new { profileId = chatSession.ProfileId });
+        return RedirectToAction(nameof(History), new { profileId = chatSession.ProfileId });
     }
 
     [HttpPost]

src/Modules/CrestApps.OrchardCore.AI.Chat/Views/AIChatSessionHeader.cshtml

@@ -5,13 +5,13 @@
 
 <div class="card text-bg-theme">
     <div class="card-body d-flex justify-content-between">
-        <form asp-action="DeleteAll" asp-controller="Admin" asp-route-profileId="@Model.Session.ProfileId" method="post" class="d-inline">
-            <button type="submit"
-                    class="btn btn-danger"
-                    data-url-af="RemoveUrl UnsafeUrl">
-                <i class="fa-solid fa-trash"></i> @T["Delete All History"]
-            </button>
-        </form>
+        <a asp-action="DeleteAll"
+           asp-controller="Admin"
+           asp-route-profileId="@Model.Session.ProfileId"
+           class="btn btn-danger"
+           data-url-af="RemoveUrl UnsafeUrl">
+            <i class="fa-solid fa-trash"></i> @T["Delete All History"]
+        </a>
 
         <div>
             <a class="btn btn-secondary me-1" asp-action="Chat" asp-controller="Admin" asp-route-profileId="@Model.Session.ProfileId">

src/Modules/CrestApps.OrchardCore.AI.Chat/Views/AIChatSessionsList.cshtml

@@ -35,7 +35,6 @@
                         <a asp-action="Delete"
                            asp-controller="Admin"
                            asp-route-sessionId="@session.SessionId"
-                           asp-route-profileId="@Model.ProfileId"
                            class="btn btn-danger btn-sm"
                            data-url-af="RemoveUrl UnsafeUrl">@T["Delete"]</a>
                     </div>