Fix duplicate Session header in command palette on macOS

The speak command (macOS-only) was appended after the Settings commands, causing the category sequence Session → Settings → Session. This made buildLines render a second Session header.

Extract Settings into its own builtInSettingsCommands function and reorder categories in BuildCommandCategories so that Settings appears before Feedback, which is always last.

Assisted-By: cagent

Author: dgageot

pkg/tui/commands/commands.go

@@ -212,6 +212,18 @@ func builtInSessionCommands() []Item {
 				return core.CmdHandler(messages.AttachFileMsg{FilePath: arg})
 			},
 		},
+	}
+
+	// Add speak command on supported platforms (macOS only)
+	if speak := speakCommand(); speak != nil {
+		cmds = append(cmds, *speak)
+	}
+
+	return cmds
+}
+
+func builtInSettingsCommands() []Item {
+	return []Item{
 		{
 			ID:           "settings.theme",
 			Label:        "Theme",
@@ -233,13 +245,6 @@ func builtInSessionCommands() []Item {
 			},
 		},
 	}
-
-	// Add speak command on supported platforms (macOS only)
-	if speak := speakCommand(); speak != nil {
-		cmds = append(cmds, *speak)
-	}
-
-	return cmds
 }
 
 func builtInFeedbackCommands() []Item {
@@ -298,10 +303,6 @@ func BuildCommandCategories(ctx context.Context, application *app.App) []Categor
 			Name:     "Session",
 			Commands: sessionCommands,
 		},
-		{
-			Name:     "Feedback",
-			Commands: builtInFeedbackCommands(),
-		},
 	}
 
 	agentCommands := application.CurrentAgentCommands(ctx)
@@ -428,6 +429,18 @@ func BuildCommandCategories(ctx context.Context, application *app.App) []Categor
 		})
 	}
 
+	// Settings and Feedback are always last, in that order.
+	categories = append(categories,
+		Category{
+			Name:     "Settings",
+			Commands: builtInSettingsCommands(),
+		},
+		Category{
+			Name:     "Feedback",
+			Commands: builtInFeedbackCommands(),
+		},
+	)
+
 	return categories
 }
 
@@ -449,5 +462,11 @@ func ParseSlashCommand(input string) tea.Cmd {
 		}
 	}
 
+	for _, item := range builtInSettingsCommands() {
+		if item.SlashCommand == cmd {
+			return item.Execute(arg)
+		}
+	}
+
 	return nil
 }