feat(core): handle display oneof (scaleway#3728)

Author: jremy42

cmd/scw/testdata/test-all-usage-alias-create-usage.golden

@@ -13,8 +13,9 @@ EXAMPLES:
     scw alias create c command=create
 
 ARGS:
-  alias       Alias name
-  [command]   Command to create an alias for
+  alias   Alias name
+  command (one of):
+    [command]   Command to create an alias for
 
 FLAGS:
   -h, --help   help for create

cmd/scw/testdata/test-all-usage-baremetal-server-install-usage.golden

@@ -10,16 +10,17 @@ EXAMPLES:
     scw baremetal server install 11111111-1111-1111-1111-111111111111 os-id=11111111-1111-1111-1111-111111111111 ssh-key-ids.0=11111111-1111-1111-1111-111111111111
 
 ARGS:
-  server-id             Server ID to install
-  os-id                 ID of the OS to installation on the server
-  hostname              Hostname of the server
-  [all-ssh-keys]        Add all SSH keys on your baremetal instance (cannot be used with ssh-key-ids)
-  ssh-key-ids.{index}   SSH key IDs authorized on the server (cannot be used with all-ssh-keys)
-  [user]                User used for the installation
-  [password]            Password used for the installation
-  [service-user]        User used for the service to install
-  [service-password]    Password used for the service to install
-  [zone=fr-par-1]       Zone to target. If none is passed will use default zone from the config (fr-par-1 | fr-par-2 | nl-ams-1 | nl-ams-2)
+  server-id   Server ID to install
+  os-id       ID of the OS to installation on the server
+  hostname    Hostname of the server
+  ssh (one of):
+    [all-ssh-keys]        Add all SSH keys on your baremetal instance (cannot be used with ssh-key-ids)
+    ssh-key-ids.{index}   SSH key IDs authorized on the server (cannot be used with all-ssh-keys)
+  [user]                  User used for the installation
+  [password]              Password used for the installation
+  [service-user]          User used for the service to install
+  [service-password]      Password used for the service to install
+  [zone=fr-par-1]         Zone to target. If none is passed will use default zone from the config (fr-par-1 | fr-par-2 | nl-ams-1 | nl-ams-2)
 
 FLAGS:
   -h, --help   help for install

cmd/scw/testdata/test-all-usage-instance-ssh-remove-key-usage.golden

@@ -8,9 +8,10 @@ USAGE:
   scw instance ssh remove-key [arg=value ...]
 
 ARGS:
-  server-id         Server to add your key to
-  [name]            Name of the key you want to remove, has to be the key comment or the index
-  [public-key]      Public key you want to remove
+  server-id   Server to add your key to
+  identifier (one of):
+    [name]          Name of the key you want to remove, has to be the key comment or the index
+    [public-key]    Public key you want to remove
   [zone=fr-par-1]   Zone to target. If none is passed will use default zone from the config (fr-par-1 | fr-par-2 | fr-par-3 | nl-ams-1 | nl-ams-2 | nl-ams-3 | pl-waw-1 | pl-waw-2 | pl-waw-3)
 
 FLAGS:

internal/core/cobra_usage_builder.go

@@ -65,9 +65,27 @@ func BuildUsageArgs(ctx context.Context, cmd *Command, deprecated bool) string {
 // _buildUsageArgs builds the arg usage list.
 // This should not be called directly.
 func _buildUsageArgs(ctx context.Context, w io.Writer, argSpecs ArgSpecs) error {
+	inOneOfGroup := false
+	lastOneOfGroup := ""
+
 	for _, argSpec := range argSpecs {
 		argSpecUsageLeftPart := argSpec.Name
 		argSpecUsageRightPart := _buildArgShort(argSpec)
+
+		if argSpec.OneOfGroup != "" {
+			if argSpec.OneOfGroup != lastOneOfGroup {
+				inOneOfGroup = true
+				lastOneOfGroup = argSpec.OneOfGroup
+				_, err := fmt.Fprintf(w, "  %s (one of):\n", argSpec.OneOfGroup)
+				if err != nil {
+					return err
+				}
+			}
+		} else {
+			inOneOfGroup = false
+			lastOneOfGroup = ""
+		}
+
 		if argSpec.Default != nil {
 			_, doc := argSpec.Default(ctx)
 			argSpecUsageLeftPart = fmt.Sprintf("%s=%s", argSpecUsageLeftPart, doc)
@@ -78,6 +96,9 @@ func _buildUsageArgs(ctx context.Context, w io.Writer, argSpecs ArgSpecs) error
 		if argSpec.CanLoadFile {
 			argSpecUsageRightPart += " (Support file loading with @/path/to/file)"
 		}
+		if inOneOfGroup {
+			argSpecUsageLeftPart = "  " + argSpecUsageLeftPart
+		}
 
 		_, err := fmt.Fprintf(w, "  %s\t%s\n", argSpecUsageLeftPart, argSpecUsageRightPart)
 		if err != nil {