fix: ask interactive auto-pause in create

Author: BhautikChudasama

cmd/sandbox/create.go

@@ -109,13 +109,24 @@ func runCreate(c *cli.Context) error {
 	// Lets users walk through name / rootfs / network / ssh keys without
 	// remembering every flag. Headless callers continue to get the
 	// "use --shape" error.
+	// Pre-parse --auto-pause so the wizard can skip the step when supplied.
+	var autoPauseSecs *int
+	if raw := strings.TrimSpace(c.String("auto-pause")); raw != "" {
+		secs, parseErr := parseDurationToSeconds(raw)
+		if parseErr != nil {
+			return fmt.Errorf("--auto-pause %q: %w", raw, parseErr)
+		}
+		autoPauseSecs = &secs
+	}
+
 	if shape == "" {
 		w, werr := runCreateWizard(c, client, wizardSeed{
-			name:    name,
-			rootfs:  rootfs,
-			ingress: ingress,
-			netIDs:  netIDs,
-			sshKeys: sshKeys,
+			name:          name,
+			rootfs:        rootfs,
+			ingress:       ingress,
+			netIDs:        netIDs,
+			sshKeys:       sshKeys,
+			autoPauseSecs: autoPauseSecs,
 		})
 		if werr != nil {
 			return werr
@@ -138,6 +149,9 @@ func runCreate(c *cli.Context) error {
 			sshKeys = w.sshKeys
 		}
 		ingress = ingress || w.ingress
+		if autoPauseSecs == nil {
+			autoPauseSecs = w.autoPauseSecs
+		}
 	}
 
 	req := api.SandboxCreateReq{
@@ -177,12 +191,8 @@ func runCreate(c *cli.Context) error {
 		req.Disks = disks
 	}
 
-	if raw := strings.TrimSpace(c.String("auto-pause")); raw != "" {
-		secs, parseErr := parseDurationToSeconds(raw)
-		if parseErr != nil {
-			return fmt.Errorf("--auto-pause %q: %w", raw, parseErr)
-		}
-		req.AutoPauseAfterSeconds = &secs
+	if autoPauseSecs != nil {
+		req.AutoPauseAfterSeconds = autoPauseSecs
 	}
 
 	spinner, _ := pterm.DefaultSpinner.Start("Creating sandbox…") //nolint:errcheck

cmd/sandbox/wizard.go

@@ -17,22 +17,24 @@ import (
 // wizardSeed lets the caller pre-fill some answers so already-supplied
 // flags aren't asked for again.
 type wizardSeed struct {
-	name    string
-	rootfs  string
-	ingress bool
-	netIDs  []string
-	sshKeys []string // canonicalised key strings (NOT paths)
+	name          string
+	rootfs        string
+	ingress       bool
+	netIDs        []string
+	sshKeys       []string // canonicalised key strings (NOT paths)
+	autoPauseSecs *int     // nil = not supplied via flag
 }
 
 // wizardResult is what runCreateWizard returns when the user finished.
 // nil result = user cancelled (caller exits quietly).
 type wizardResult struct {
-	shape   string
-	name    string
-	rootfs  string
-	ingress bool
-	netIDs  []string
-	sshKeys []string // canonicalised key strings (NOT paths)
+	shape         string
+	name          string
+	rootfs        string
+	ingress       bool
+	netIDs        []string
+	sshKeys       []string // canonicalised key strings (NOT paths)
+	autoPauseSecs *int     // nil = disabled
 }
 
 // runCreateWizard walks the user through shape → name → rootfs → ingress
@@ -50,11 +52,12 @@ func runCreateWizard(c *cli.Context, client *api.SandboxClient, seed wizardSeed)
 		return nil, fmt.Errorf("please choose a size with --shape\n\n  To see the options, run:\n    createos sandbox shapes")
 	}
 	out := &wizardResult{
-		name:    seed.name,
-		rootfs:  seed.rootfs,
-		ingress: seed.ingress,
-		netIDs:  append([]string{}, seed.netIDs...),
-		sshKeys: append([]string{}, seed.sshKeys...),
+		name:          seed.name,
+		rootfs:        seed.rootfs,
+		ingress:       seed.ingress,
+		netIDs:        append([]string{}, seed.netIDs...),
+		sshKeys:       append([]string{}, seed.sshKeys...),
+		autoPauseSecs: seed.autoPauseSecs,
 	}
 
 	// ── 1. Shape (required) ─────────────────────────────────────────
@@ -125,6 +128,16 @@ func runCreateWizard(c *cli.Context, client *api.SandboxClient, seed wizardSeed)
 		}
 	}
 
+	// ── 7. Auto-pause (optional; skip if already supplied via --auto-pause) ──
+	if out.autoPauseSecs == nil {
+		secs, err := wizardPickAutoPause()
+		if err != nil {
+			pterm.Println(pterm.Gray(fmt.Sprintf("  Auto-pause step skipped (%v).", err)))
+		} else {
+			out.autoPauseSecs = secs
+		}
+	}
+
 	return out, nil
 }
 
@@ -339,6 +352,33 @@ func relToHome(path, home string) string {
 	return path
 }
 
+// wizardPickAutoPause asks whether the sandbox should pause itself when idle.
+// Returns nil (no auto-pause) if the user skips or types nothing.
+func wizardPickAutoPause() (*int, error) {
+	pterm.Println()
+	pterm.NewStyle(pterm.FgCyan).Println("  Auto-pause")
+	pterm.Println(pterm.Gray("  Your sandbox can pause itself automatically when you're not using it,"))
+	pterm.Println(pterm.Gray("  saving resources. It resumes instantly the next time you need it."))
+	pterm.Println(pterm.Gray("  Leave blank to keep it running until you stop it manually."))
+
+	input, err := pterm.DefaultInteractiveTextInput.
+		WithDefaultText("Pause after how long with no activity? (e.g. 10m, 1h — leave blank to skip)").
+		Show()
+	if err != nil {
+		return nil, err
+	}
+	input = strings.TrimSpace(input)
+	if input == "" {
+		return nil, nil
+	}
+	secs, err := parseDurationToSeconds(input)
+	if err != nil {
+		pterm.Println(pterm.Gray(fmt.Sprintf("  Didn't recognise %q — skipping auto-pause.", input)))
+		return nil, nil
+	}
+	return &secs, nil
+}
+
 // stringSliceCleanup trims and drops empty entries — pulled out of
 // runCreate so the wizard plumbing can reuse it.
 func stringSliceCleanup(raw []string) []string {