feat(cli): support cobra GroupID and set error prefix

Commander layout now reads both cobra's native GroupID and the
legacy Annotations["group"] for command grouping. Projects can
use either approach while keeping salt's custom help formatting.

cli.Init sets SetErrPrefix(name + ":") for consistent error messages
across all raystack CLIs (e.g. "Error: frontier: unknown command").

Author: ravisuhag

cli/cli.go

@@ -43,6 +43,9 @@ func Init(rootCmd *cobra.Command, opts ...Option) {
 		opt(cfg)
 	}
 
+	// Set error prefix for consistent error messages.
+	rootCmd.SetErrPrefix(rootCmd.Name() + ":")
+
 	// Inject shared output and prompter into command context.
 	existing := rootCmd.PersistentPreRun
 	rootCmd.PersistentPreRun = func(cmd *cobra.Command, args []string) {

cli/commander/layout.go

@@ -95,17 +95,22 @@ func buildHelpEntries(cmd *cobra.Command) []helpEntry {
 		}
 
 		entry := fmt.Sprintf("%s%s", rpad(c.Name(), c.NamePadding()+3), c.Short)
-		if group, ok := c.Annotations["group"]; ok {
-			switch group {
-			case "core":
-				coreCommands = append(coreCommands, entry)
-			case "help":
-				helpCommands = append(helpCommands, entry)
-			default:
-				groupCommands[group] = append(groupCommands[group], entry)
-			}
-		} else {
+
+		// Support both cobra's native GroupID and legacy Annotations["group"].
+		group := c.GroupID
+		if group == "" {
+			group = c.Annotations["group"]
+		}
+
+		switch group {
+		case "core":
+			coreCommands = append(coreCommands, entry)
+		case "help":
+			helpCommands = append(helpCommands, entry)
+		case "":
 			otherCommands = append(otherCommands, entry)
+		default:
+			groupCommands[group] = append(groupCommands[group], entry)
 		}
 	}