prompt for required flags

Author: vegeris

cmd/sandbox/create.go

@@ -21,6 +21,7 @@ import (
 	"time"
 	"unicode"
 
+	"github.com/slackapi/slack-cli/internal/iostreams"
 	"github.com/slackapi/slack-cli/internal/shared"
 	"github.com/slackapi/slack-cli/internal/slackerror"
 	"github.com/slackapi/slack-cli/internal/style"
@@ -98,13 +99,6 @@ func NewCreateCommand(clients *shared.ClientFactory) *cobra.Command {
 	// If one's developer account is managed by multiple Production Slack teams, one of those team IDs must be provided in the command
 	cmd.Flags().StringVar(&createCmdFlags.owningOrgID, "owning-org-id", "", "Enterprise team ID that manages your developer account, if applicable")
 
-	if err := cmd.MarkFlagRequired("name"); err != nil {
-		panic(err)
-	}
-	if err := cmd.MarkFlagRequired("password"); err != nil {
-		panic(err)
-	}
-
 	return cmd
 }
 
@@ -116,6 +110,36 @@ func runCreateCommand(cmd *cobra.Command, clients *shared.ClientFactory) error {
 		return err
 	}
 
+	name := createCmdFlags.name
+	if name == "" {
+		name, err = clients.IO.InputPrompt(
+			ctx,
+			"Enter a name for the sandbox",
+			iostreams.InputPromptConfig{
+				Required: true,
+			},
+		)
+		if err != nil {
+			return err
+		}
+	}
+	createCmdFlags.name = name
+
+	password := createCmdFlags.password
+	if password == "" {
+		password, err = clients.IO.InputPrompt(
+			ctx,
+			"Enter a password for the sandbox",
+			iostreams.InputPromptConfig{
+				Required: true,
+			},
+		)
+		if err != nil {
+			return err
+		}
+	}
+	createCmdFlags.password = password
+
 	domain := createCmdFlags.domain
 	if domain == "" {
 		var err error
@@ -170,7 +194,6 @@ func runCreateCommand(cmd *cobra.Command, clients *shared.ClientFactory) error {
 
 // getEpochFromTTL parses a time-to-live string (e.g., "1d", "2w", "3mo") and returns the Unix epoch
 // when the sandbox will be archived. Supports days (d), weeks (w), and months (mo).
-// Minimum is 1 day, maximum is 6 months.
 func getEpochFromTTL(ttl string) (int64, error) {
 	lower := strings.TrimSpace(strings.ToLower(ttl))
 	if lower == "" {
@@ -206,7 +229,7 @@ func getEpochFromTTL(ttl string) (int64, error) {
 	return target.Unix(), nil
 }
 
-// getEpochFromDate parses a date in yyyy-mm-dd format and returns the Unix epoch at start of that day (UTC).
+// getEpochFromDate parses a date in yyyy-mm-dd format and returns the Unix epoch at the start of that day (UTC)
 func getEpochFromDate(dateStr string) (int64, error) {
 	dateFormat := "2006-01-02"
 	t, err := time.ParseInLocation(dateFormat, dateStr, time.UTC)

internal/slackerror/errors.go

@@ -270,6 +270,7 @@ const (
 	ErrWorkflowNotFound                              = "workflow_not_found"
 	ErrYaml                                          = "yaml_error"
 	ErrSandboxDomainTaken                            = "domain_taken"
+	ErrAtActiveSandboxLimit                          = "at_active_sandbox_limit"
 	ErrInvalidArchiveTTL                             = "invalid_archive_ttl"
 )
 
@@ -1628,6 +1629,11 @@ Otherwise start your app for local development with: %s`,
 		Message: "This domain has been claimed by another sandbox",
 	},
 
+	ErrAtActiveSandboxLimit: {
+		Code:    ErrAtActiveSandboxLimit,
+		Message: "You've reached the maximum number of active sandboxes",
+	},
+
 	ErrInvalidArchiveTTL: {
 		Code:        ErrInvalidArchiveTTL,
 		Message:     "Invalid TTL",