Feat/vm batch operation support (#29)

* feat(vm): add --all, --status, --with-volumes flags to shortcut commands

* feat(vm): add batch validation, fetching, confirmation, and result display

* feat(vm): implement batch execution for all actions including delete

* feat(vm): --status filters interactive picker when --all is not set

* feat(vm): add batch integration tests and help examples

* feat(vm): interactive multi-select for shortcut commands

When running shortcut commands (stop, start, hibernate, delete) without
an instance ID, use MultiSelect picker instead of single Select. Users
can filter by typing, select all with ctrl+a, and pick multiple instances
to batch-operate on.

Single selection continues the existing single-instance flow.
Multiple selections route to the batch execution path.

* fix(vm): remove hint text from multi-select prompt

The hint bar already shows keyboard shortcuts, no need to duplicate
them in the prompt text.

* feat(volume): add delete shortcut with multi-select batch support

* feat(vm): add stop command to help and --hostname glob filter for batch

- Make `stop` a standalone shortcut command visible in `verda vm --help`
  (previously hidden as alias of `shutdown`)
- Add `--hostname` flag to all shortcut commands accepting a glob pattern
  (e.g., "test-*") that implies batch mode without requiring `--all`
- Hostname filter combines with `--status` as AND logic
- Add filterByHostname helper using filepath.Match for glob matching

* feat(vm): add --location filter to vm list

Client-side filtering since the API doesn't support location as a
query parameter. Case-insensitive match.
* deps: upgrade verdagostack to v1.2.0

MultiSelect now supports text filtering and ctrl+a select all,
enabling interactive batch operations in vm and volume commands.
* fix(vm): use verda.ActionDelete constant instead of string literal

Fixes goconst lint error for repeated "delete" string.

* refactor(vm): use SDK action constants instead of string literals

Replace hardcoded action strings with verda.ActionStart,
verda.ActionShutdown, verda.ActionDelete, etc. throughout
shortcuts, action map, and batch code.

Author: hi-lei

go.mod

@@ -9,7 +9,7 @@ require (
 	github.com/spf13/pflag v1.0.10
 	github.com/spf13/viper v1.21.0
 	github.com/verda-cloud/verdacloud-sdk-go v1.4.2
-	github.com/verda-cloud/verdagostack v1.1.3
+	github.com/verda-cloud/verdagostack v1.2.0
 	go.yaml.in/yaml/v3 v3.0.4
 )
 

go.sum

@@ -104,8 +104,8 @@ github.com/subosito/gotenv v1.6.0 h1:9NlTDc1FTs4qu0DDq7AEtTPNw6SVm7uBMsUCUjABIf8
 github.com/subosito/gotenv v1.6.0/go.mod h1:Dk4QP5c2W3ibzajGcXpNraDfq2IrhjMIvMSWPKKo0FU=
 github.com/verda-cloud/verdacloud-sdk-go v1.4.2 h1:oVb8fHVQOY+YPuuMYMee9gYCkPTwAw01LmkqxM21T/Y=
 github.com/verda-cloud/verdacloud-sdk-go v1.4.2/go.mod h1:pmlpiCL9fTSikZ3qWLJPpHOG0E8PKkQVUX5s4Z+SktY=
-github.com/verda-cloud/verdagostack v1.1.3 h1:aJy8ixF5KR3bDy5k5n1fpEHKgZxCIt/FAH+HAvNidSM=
-github.com/verda-cloud/verdagostack v1.1.3/go.mod h1:9uKLNxvH7JRkHj2sxZc4TI5O/JbbveCrLkOkekoY/sA=
+github.com/verda-cloud/verdagostack v1.2.0 h1:HdVFXWyfEMq17ldn3+FrePYYB5Jdquz9D/Y0V+4d2BE=
+github.com/verda-cloud/verdagostack v1.2.0/go.mod h1:h9XR9uCYBYauRyGF4NLlScD5bC2UEMxMkqg6fUVjyDo=
 github.com/xo/terminfo v0.0.0-20220910002029-abceb7e1c41e h1:JVG44RsyaB9T2KIHavMF/ppJZNG9ZpyihvCd0w101no=
 github.com/xo/terminfo v0.0.0-20220910002029-abceb7e1c41e/go.mod h1:RbqR21r5mrJuqunuUZ/Dhy/avygyECGrLceyNeo4LiM=
 github.com/yosida95/uritemplate/v3 v3.0.2 h1:Ed3Oyj9yrmi9087+NczuL5BwkIc4wvTb5zIM+UJPGz4=

internal/verda-cli/cmd/vm/action.go

@@ -15,12 +15,12 @@ import (
 
 // actionNameMap maps CLI flag values to action labels.
 var actionNameMap = map[string]string{
-	"start":          "Start",
-	"shutdown":       "Shutdown",
-	"force_shutdown": "Force shutdown",
-	"force-shutdown": "Force shutdown",
-	"hibernate":      "Hibernate",
-	"delete":         "Delete instance",
+	verda.ActionStart:         "Start",
+	verda.ActionShutdown:      "Shutdown",
+	verda.ActionForceShutdown: "Force shutdown",
+	"force-shutdown":          "Force shutdown", // CLI alias (hyphenated)
+	verda.ActionHibernate:     "Hibernate",
+	verda.ActionDelete:        "Delete instance",
 }
 
 // instanceAction defines a supported action with its display label and executor.
@@ -98,10 +98,14 @@ func availableActions(status string) []instanceAction {
 }
 
 type actionOptions struct {
-	InstanceID string
-	Action     string
-	Yes        bool
-	Wait       cmdutil.WaitOptions
+	InstanceID  string
+	Action      string
+	Yes         bool
+	All         bool
+	Status      string
+	Hostname    string
+	WithVolumes bool
+	Wait        cmdutil.WaitOptions
 }
 
 // NewCmdAction creates the vm action cobra command.
@@ -148,7 +152,7 @@ func runAction(cmd *cobra.Command, f cmdutil.Factory, ioStreams cmdutil.IOStream
 	// In agent mode, --id and --action are required.
 	if f.AgentMode() {
 		var missing []string
-		if opts.InstanceID == "" {
+		if opts.InstanceID == "" && !opts.All && opts.Hostname == "" {
 			missing = append(missing, "--id")
 		}
 		if opts.Action == "" {
@@ -159,6 +163,11 @@ func runAction(cmd *cobra.Command, f cmdutil.Factory, ioStreams cmdutil.IOStream
 		}
 	}
 
+	// Batch mode: --all or --hostname routes to batch execution.
+	if opts.All || opts.Hostname != "" {
+		return runBatchAction(cmd, f, ioStreams, opts)
+	}
+
 	client, err := f.VerdaClient()
 	if err != nil {
 		return err
@@ -167,22 +176,16 @@ func runAction(cmd *cobra.Command, f cmdutil.Factory, ioStreams cmdutil.IOStream
 	prompter := f.Prompter()
 	ctx := cmd.Context()
 
-	// Select instance if not provided.
+	// Select instance(s) if not provided.
 	if opts.InstanceID == "" {
-		// When action is pre-set (shortcut commands), only show instances
-		// with statuses valid for that action.
-		var statusFilter []string
-		if opts.Action != "" {
-			statusFilter = validFromForAction(opts.Action)
-		}
-		selected, err := selectInstance(ctx, f, ioStreams, client, statusFilter...)
-		if err != nil {
-			return err
+		id, batchErr := resolveInstanceInteractive(cmd, f, ioStreams, client, opts)
+		if batchErr != nil {
+			return batchErr
 		}
-		if selected == "" {
-			return nil
+		if id == "" {
+			return nil // canceled or routed to batch
 		}
-		opts.InstanceID = selected
+		opts.InstanceID = id
 	}
 
 	// Fetch instance details.
@@ -307,7 +310,7 @@ func runAction(cmd *cobra.Command, f cmdutil.Factory, ioStreams cmdutil.IOStream
 // runDeleteAgent handles delete in agent mode: requires --yes, deletes all volumes.
 func runDeleteAgent(ctx context.Context, f cmdutil.Factory, ioStreams cmdutil.IOStreams, client *verda.Client, inst *verda.Instance, yes bool) error {
 	if !yes {
-		return cmdutil.NewConfirmationRequiredError("delete")
+		return cmdutil.NewConfirmationRequiredError(verda.ActionDelete)
 	}
 
 	// In agent mode, delete the instance and all attached volumes.
@@ -327,14 +330,46 @@ func runDeleteAgent(ctx context.Context, f cmdutil.Factory, ioStreams cmdutil.IO
 
 	result := map[string]any{
 		"id":              inst.ID,
-		"action":          "delete",
+		"action":          verda.ActionDelete,
 		"status":          "completed",
 		"volumes_deleted": len(volumeIDs),
 	}
 	_, _ = cmdutil.WriteStructured(ioStreams.Out, f.OutputFormat(), result)
 	return nil
 }
 
+// resolveInstanceInteractive handles interactive instance selection.
+// For shortcut commands (action pre-set), uses multi-select so users can pick
+// multiple instances. If multiple are selected, routes to batch and returns "".
+// Returns the selected instance ID, or "" if canceled/routed to batch.
+func resolveInstanceInteractive(cmd *cobra.Command, f cmdutil.Factory, ioStreams cmdutil.IOStreams, client *verda.Client, opts *actionOptions) (string, error) {
+	ctx := cmd.Context()
+
+	var statusFilter []string
+	if opts.Status != "" {
+		statusFilter = []string{opts.Status}
+	} else if opts.Action != "" {
+		statusFilter = validFromForAction(opts.Action)
+	}
+
+	// Shortcut commands use multi-select; generic "vm action" uses single-select.
+	if opts.Action == "" {
+		return selectInstance(ctx, f, ioStreams, client, statusFilter...)
+	}
+
+	selected, err := selectInstances(ctx, f, ioStreams, client, statusFilter...)
+	if err != nil {
+		return "", err
+	}
+	if len(selected) == 0 {
+		return "", nil
+	}
+	if len(selected) > 1 {
+		return "", runBatchWithInstances(cmd, f, ioStreams, client, selected, opts)
+	}
+	return selected[0].ID, nil
+}
+
 func selectInstance(ctx context.Context, f cmdutil.Factory, ioStreams cmdutil.IOStreams, client *verda.Client, statusFilter ...string) (string, error) {
 	var sp interface{ Stop(string) }
 	if status := f.Status(); status != nil {