From 4b5bf988475bb07c174b9fb06ebc8afc75a9e520 Mon Sep 17 00:00:00 2001
From: Thorsten Sommer <SommerEngineering@users.noreply.github.com>
Date: Sun, 13 Sep 2026 17:15:32 +0200
Subject: [PATCH] Sort attachments by folder so each folder is named once

---
 .../Dialogs/ReviewAttachmentsDialog.razor         |  2 +-
 .../Dialogs/ReviewAttachmentsDialog.razor.cs      | 15 +++++++++++++++
 2 files changed, 16 insertions(+), 1 deletion(-)

diff --git a/app/MindWork AI Studio/Dialogs/ReviewAttachmentsDialog.razor b/app/MindWork AI Studio/Dialogs/ReviewAttachmentsDialog.razor
index cf245df1..3528d5fc 100644
--- a/app/MindWork AI Studio/Dialogs/ReviewAttachmentsDialog.razor	
+++ b/app/MindWork AI Studio/Dialogs/ReviewAttachmentsDialog.razor	
@@ -31,7 +31,7 @@
 
             @{
                 var currentFolder = string.Empty;
-                foreach (var fileAttachment in this.DocumentPaths)
+                foreach (var fileAttachment in this.OrderedAttachments)
                 {
                     var folderPath = Path.GetDirectoryName(fileAttachment.FilePath);
                     if (folderPath != currentFolder)
diff --git a/app/MindWork AI Studio/Dialogs/ReviewAttachmentsDialog.razor.cs b/app/MindWork AI Studio/Dialogs/ReviewAttachmentsDialog.razor.cs
index 026a11ef..0efcaa0a 100644
--- a/app/MindWork AI Studio/Dialogs/ReviewAttachmentsDialog.razor.cs	
+++ b/app/MindWork AI Studio/Dialogs/ReviewAttachmentsDialog.razor.cs	
@@ -73,6 +73,21 @@ public partial class ReviewAttachmentsDialog : MSGComponentBase
             : "border-dashed border-2 rounded-lg pa-2 mud-border-lines-default";
     }
 
+    /// <summary>
+    /// The attachments, sorted by their folder and, within it, by their file name.
+    /// </summary>
+    /// <remarks>
+    /// The list below starts a new heading whenever the folder changes from one attachment to the
+    /// next, which names every folder exactly once -- but only as long as the attachments of a
+    /// folder arrive together. The set behind them keeps no order of its own to guarantee that:
+    /// removing one attachment already scrambles it, and one attached while this dialog is open
+    /// lands at its end, giving its folder a second heading further down. Sorting here is what that
+    /// list assumes anyway.
+    /// </remarks>
+    private IEnumerable<FileAttachment> OrderedAttachments => this.DocumentPaths
+        .OrderBy(attachment => Path.GetDirectoryName(attachment.FilePath) ?? string.Empty, StringComparer.OrdinalIgnoreCase)
+        .ThenBy(attachment => attachment.FileName, StringComparer.OrdinalIgnoreCase);
+
     /// <summary>
     /// Attaches what the user dropped onto this dialog and answers which files that became.
     /// </summary>