Sort folders alphabetically; let users pick the merge model directly

Folders now list alphabetically (case-insensitive) everywhere they're
shown — sidebar, advanced conversation list, and the "Move to Folder"
menu — rather than creation order. listFolders() does the sort at the
DB layer; in-memory folder arrays are re-sorted after local
create/rename so newly added or renamed folders don't fall out of
order until the next reload.

Combine Conversations' AI-Assisted Merge no longer silently assumes
settings.defaultModel — added a model picker (reusing ModelSelectorView,
defaulting to the current default model/provider) so users can pick
which model performs the merge per-combine. ConversationMergeService.merge
takes optional mergeModelId/mergeProvider overrides.
This commit is contained in:
2026-07-28 14:39:18 +02:00
parent 8dee77541f
commit e898d4b6db
6 changed files with 75 additions and 17 deletions
+10 -4
View File
@@ -55,6 +55,8 @@ enum ConversationMergeService {
conversationIds: [UUID], conversationIds: [UUID],
name: String, name: String,
mode: CombineMode, mode: CombineMode,
mergeModelId: String? = nil,
mergeProvider: Settings.Provider? = nil,
deleteOriginals: Bool deleteOriginals: Bool
) async throws -> Conversation { ) async throws -> Conversation {
guard conversationIds.count >= 2 else { guard conversationIds.count >= 2 else {
@@ -78,7 +80,7 @@ enum ConversationMergeService {
case .simple: case .simple:
mergedMessages = simpleMerge(sources) mergedMessages = simpleMerge(sources)
case .ai: case .ai:
mergedMessages = try await aiMerge(sources) mergedMessages = try await aiMerge(sources, modelId: mergeModelId, provider: mergeProvider)
} }
let newConversation = try DatabaseService.shared.saveConversation( let newConversation = try DatabaseService.shared.saveConversation(
@@ -108,12 +110,16 @@ enum ConversationMergeService {
let content: String let content: String
} }
private static func aiMerge(_ sources: [(Conversation, [Message])]) async throws -> [Message] { private static func aiMerge(
_ sources: [(Conversation, [Message])],
modelId explicitModelId: String?,
provider explicitProvider: Settings.Provider?
) async throws -> [Message] {
let settings = SettingsService.shared let settings = SettingsService.shared
guard let modelId = settings.defaultModel, !modelId.isEmpty else { guard let modelId = explicitModelId ?? settings.defaultModel, !modelId.isEmpty else {
throw MergeError.noDefaultModel throw MergeError.noDefaultModel
} }
guard let provider = ProviderRegistry.shared.getProvider(for: settings.defaultProvider) else { guard let provider = ProviderRegistry.shared.getProvider(for: explicitProvider ?? settings.defaultProvider) else {
throw MergeError.noAPIKey throw MergeError.noAPIKey
} }
+3 -3
View File
@@ -645,9 +645,9 @@ final class DatabaseService: Sendable {
nonisolated func listFolders() throws -> [Folder] { nonisolated func listFolders() throws -> [Folder] {
try dbQueue.read { db in try dbQueue.read { db in
let records = try FolderRecord // Alphabetical, not creation order (sortOrder) folders should sort
.order(Column("sortOrder")) // predictably by name everywhere they're listed.
.fetchAll(db) let records = try FolderRecord.fetchAll(db, sql: "SELECT * FROM folders ORDER BY name COLLATE NOCASE")
return records.compactMap { record -> Folder? in return records.compactMap { record -> Folder? in
guard let id = UUID(uuidString: record.id), guard let id = UUID(uuidString: record.id),
let createdAt = Self.isoDate(from: record.createdAt) let createdAt = Self.isoDate(from: record.createdAt)
+6
View File
@@ -357,6 +357,7 @@ struct SidebarView: View {
do { do {
let folder = try DatabaseService.shared.createFolder(name: name) let folder = try DatabaseService.shared.createFolder(name: name)
folders.append(folder) folders.append(folder)
sortFolders()
if let conversation = conversation { if let conversation = conversation {
moveConversation(conversation, toFolder: folder.id) moveConversation(conversation, toFolder: folder.id)
} }
@@ -366,6 +367,10 @@ struct SidebarView: View {
#endif #endif
} }
private func sortFolders() {
folders.sort { $0.name.localizedCaseInsensitiveCompare($1.name) == .orderedAscending }
}
private func renameFolderPrompt(_ folder: Folder) { private func renameFolderPrompt(_ folder: Folder) {
#if os(macOS) #if os(macOS)
let alert = NSAlert() let alert = NSAlert()
@@ -385,6 +390,7 @@ struct SidebarView: View {
if let i = folders.firstIndex(where: { $0.id == folder.id }) { if let i = folders.firstIndex(where: { $0.id == folder.id }) {
folders[i].name = newName folders[i].name = newName
} }
sortFolders()
} catch { } catch {
Log.db.error("Failed to rename folder: \(error.localizedDescription)") Log.db.error("Failed to rename folder: \(error.localizedDescription)")
} }
@@ -25,6 +25,7 @@ import SwiftUI
struct CombineConversationsSheet: View { struct CombineConversationsSheet: View {
@Environment(\.dismiss) var dismiss @Environment(\.dismiss) var dismiss
@Environment(ChatViewModel.self) private var chatViewModel
let conversations: [Conversation] let conversations: [Conversation]
var onCompleted: (Conversation) -> Void var onCompleted: (Conversation) -> Void
@@ -34,6 +35,9 @@ struct CombineConversationsSheet: View {
@State private var deleteOriginals = false @State private var deleteOriginals = false
@State private var isProcessing = false @State private var isProcessing = false
@State private var errorMessage: String? @State private var errorMessage: String?
@State private var mergeModel: ModelInfo?
@State private var mergeProvider: Settings.Provider
@State private var showModelPicker = false
private let settings = SettingsService.shared private let settings = SettingsService.shared
@@ -42,9 +46,13 @@ struct CombineConversationsSheet: View {
self.onCompleted = onCompleted self.onCompleted = onCompleted
let joined = conversations.map(\.name).joined(separator: " + ") let joined = conversations.map(\.name).joined(separator: " + ")
_name = State(initialValue: String(joined.prefix(80))) _name = State(initialValue: String(joined.prefix(80)))
_mergeProvider = State(initialValue: SettingsService.shared.defaultProvider)
} }
private var defaultModelLabel: String? { private var mergeModelLabel: String? {
if let mergeModel {
return "\(mergeProvider.displayName) / \(mergeModel.name)"
}
guard let model = settings.defaultModel, !model.isEmpty else { return nil } guard let model = settings.defaultModel, !model.isEmpty else { return nil }
return "\(settings.defaultProvider.displayName) / \(model)" return "\(settings.defaultProvider.displayName) / \(model)"
} }
@@ -52,7 +60,7 @@ struct CombineConversationsSheet: View {
private var isValid: Bool { private var isValid: Bool {
!name.trimmingCharacters(in: .whitespaces).isEmpty !name.trimmingCharacters(in: .whitespaces).isEmpty
&& conversations.count >= 2 && conversations.count >= 2
&& (mode == .simple || defaultModelLabel != nil) && (mode == .simple || mergeModelLabel != nil)
} }
var body: some View { var body: some View {
@@ -109,12 +117,20 @@ struct CombineConversationsSheet: View {
} else { } else {
Text("A model reads all the source messages and rewrites them into one coherent, de-duplicated conversation.") Text("A model reads all the source messages and rewrites them into one coherent, de-duplicated conversation.")
.font(.caption).foregroundStyle(.secondary) .font(.caption).foregroundStyle(.secondary)
if let label = defaultModelLabel { HStack(spacing: 8) {
Label("Uses your default model: \(label)", systemImage: "cpu") if let label = mergeModelLabel {
.font(.caption).foregroundStyle(.secondary) Label(label, systemImage: "cpu")
} else { .font(.caption).foregroundStyle(.secondary)
Label("No default model configured — set one in Settings → General.", systemImage: "exclamationmark.triangle.fill") } else {
.font(.caption).foregroundStyle(.orange) Label("No model selected", systemImage: "exclamationmark.triangle.fill")
.font(.caption).foregroundStyle(.orange)
}
Button("Change…") {
showModelPicker = true
}
.buttonStyle(.link)
.font(.caption)
.disabled(isProcessing)
} }
} }
} }
@@ -156,6 +172,25 @@ struct CombineConversationsSheet: View {
.padding(.horizontal, 24).padding(.vertical, 12) .padding(.horizontal, 24).padding(.vertical, 12)
} }
.frame(minWidth: 520, idealWidth: 560, minHeight: 460, idealHeight: 520) .frame(minWidth: 520, idealWidth: 560, minHeight: 460, idealHeight: 520)
.task {
if chatViewModel.availableModels.count <= 10 {
await chatViewModel.loadAvailableModels()
}
if mergeModel == nil, let defaultModel = settings.defaultModel {
mergeModel = chatViewModel.availableModels.first(where: { $0.id == defaultModel })
}
}
.sheet(isPresented: $showModelPicker) {
ModelSelectorView(
models: chatViewModel.availableModels,
selectedModel: mergeModel,
onSelect: { model in
mergeModel = model
mergeProvider = chatViewModel.inferProviderPublic(from: model.id) ?? mergeProvider
showModelPicker = false
}
)
}
} }
private func combine() { private func combine() {
@@ -165,6 +200,8 @@ struct CombineConversationsSheet: View {
let trimmedName = name.trimmingCharacters(in: .whitespaces) let trimmedName = name.trimmingCharacters(in: .whitespaces)
let selectedMode = mode let selectedMode = mode
let shouldDeleteOriginals = deleteOriginals let shouldDeleteOriginals = deleteOriginals
let selectedModelId = mergeModel?.id
let selectedProvider = mergeModel != nil ? mergeProvider : nil
Task { Task {
do { do {
@@ -172,6 +209,8 @@ struct CombineConversationsSheet: View {
conversationIds: ids, conversationIds: ids,
name: trimmedName, name: trimmedName,
mode: selectedMode, mode: selectedMode,
mergeModelId: selectedModelId,
mergeProvider: selectedProvider,
deleteOriginals: shouldDeleteOriginals deleteOriginals: shouldDeleteOriginals
) )
await MainActor.run { await MainActor.run {
@@ -479,12 +479,17 @@ struct ConversationListView: View {
do { do {
let folder = try DatabaseService.shared.createFolder(name: name) let folder = try DatabaseService.shared.createFolder(name: name)
folders.append(folder) folders.append(folder)
sortFolders()
} catch { } catch {
Log.db.error("Failed to create folder: \(error.localizedDescription)") Log.db.error("Failed to create folder: \(error.localizedDescription)")
} }
#endif #endif
} }
private func sortFolders() {
folders.sort { $0.name.localizedCaseInsensitiveCompare($1.name) == .orderedAscending }
}
private func loadConversations() { private func loadConversations() {
do { do {
conversations = try DatabaseService.shared.listConversations() conversations = try DatabaseService.shared.listConversations()
@@ -525,6 +530,7 @@ struct ConversationListView: View {
if let i = folders.firstIndex(where: { $0.id == folder.id }) { if let i = folders.firstIndex(where: { $0.id == folder.id }) {
folders[i].name = newName folders[i].name = newName
} }
sortFolders()
} catch { } catch {
Log.db.error("Failed to rename folder: \(error.localizedDescription)") Log.db.error("Failed to rename folder: \(error.localizedDescription)")
} }
+3 -2
View File
@@ -240,14 +240,15 @@ struct DatabaseServiceFolderTests {
#expect(second.sortOrder == 1) #expect(second.sortOrder == 1)
} }
@Test("listFolders returns folders ordered by sortOrder") @Test("listFolders returns folders sorted alphabetically, case-insensitive, regardless of creation order")
func listFoldersOrdered() throws { func listFoldersOrdered() throws {
let db = DatabaseService.makeInMemory() let db = DatabaseService.makeInMemory()
_ = try db.createFolder(name: "Work") _ = try db.createFolder(name: "Work")
_ = try db.createFolder(name: "apple")
_ = try db.createFolder(name: "Personal") _ = try db.createFolder(name: "Personal")
let folders = try db.listFolders() let folders = try db.listFolders()
#expect(folders.map(\.name) == ["Work", "Personal"]) #expect(folders.map(\.name) == ["apple", "Personal", "Work"])
} }
@Test("renameFolder updates the stored name") @Test("renameFolder updates the stored name")