Restore non-human format rejection for confirmation-gated workflow commands

The JSON confirmation envelope replaced the old Format() != Human guard,
which let --format csv fall through to interactive confirmation on a TTY
for cancel, cutover, and switch-traffic. Reinstate the format error after
the JSON branch so non-JSON machine formats fail fast without --force.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: no-itsbackpack

internal/cmd/workflow/cancel.go

@@ -46,6 +46,10 @@ marks it as cancelled, allowing you to start a new workflow if needed.`,
 						cmdutil.AgentWorkflowActionCmd(wfCtx.Org, db, num, "cancel", "--force"))
 				}
 
+				if ch.Printer.Format() != printer.Human {
+					return fmt.Errorf(`cannot cancel workflow with the output format "%s" (run with --force to override)`, ch.Printer.Format())
+				}
+
 				if !printer.IsTTY {
 					return fmt.Errorf("cannot confirm cancellation (run with --force to override)")
 				}

internal/cmd/workflow/cancel_test.go

@@ -89,6 +89,36 @@ func TestWorkflow_CancelCmd(t *testing.T) {
 	c.Assert(buf.String(), qt.JSONEquals, expectedWorkflow)
 }
 
+func TestWorkflow_CancelCmd_CSVWithoutForce(t *testing.T) {
+	c := qt.New(t)
+
+	var buf bytes.Buffer
+	format := printer.CSV
+	p := printer.NewPrinter(&format)
+	p.SetResourceOutput(&buf)
+
+	svc := &mock.WorkflowsService{}
+
+	ch := &cmdutil.Helper{
+		Printer: p,
+		Config: &config.Config{
+			Organization: "planetscale",
+		},
+		Client: func() (*ps.Client, error) {
+			return &ps.Client{
+				Workflows: svc,
+			}, nil
+		},
+	}
+
+	cmd := CancelCmd(ch)
+	cmd.SetArgs([]string{"planetscale", "123"})
+	err := cmd.Execute()
+
+	c.Assert(err, qt.ErrorMatches, `cannot cancel workflow with the output format "csv".*`)
+	c.Assert(svc.CancelFnInvoked, qt.IsFalse)
+}
+
 func TestWorkflow_CancelCmd_Error(t *testing.T) {
 	c := qt.New(t)
 

internal/cmd/workflow/cutover.go

@@ -45,6 +45,10 @@ func CutoverCmd(ch *cmdutil.Helper) *cobra.Command {
 						cmdutil.AgentWorkflowActionCmd(wfCtx.Org, db, num, "cutover", "--force"))
 				}
 
+				if ch.Printer.Format() != printer.Human {
+					return fmt.Errorf(`cannot cutover with the output format "%s" (run with --force to override)`, ch.Printer.Format())
+				}
+
 				if !printer.IsTTY {
 					return fmt.Errorf("cannot confirm cutover (run with --force to override)")
 				}

internal/cmd/workflow/cutover_test.go

@@ -91,6 +91,36 @@ func TestWorkflow_CutoverCmd(t *testing.T) {
 	c.Assert(buf.String(), qt.JSONEquals, expectedWorkflow)
 }
 
+func TestWorkflow_CutoverCmd_CSVWithoutForce(t *testing.T) {
+	c := qt.New(t)
+
+	var buf bytes.Buffer
+	format := printer.CSV
+	p := printer.NewPrinter(&format)
+	p.SetResourceOutput(&buf)
+
+	svc := &mock.WorkflowsService{}
+
+	ch := &cmdutil.Helper{
+		Printer: p,
+		Config: &config.Config{
+			Organization: "planetscale",
+		},
+		Client: func() (*ps.Client, error) {
+			return &ps.Client{
+				Workflows: svc,
+			}, nil
+		},
+	}
+
+	cmd := CutoverCmd(ch)
+	cmd.SetArgs([]string{"planetscale", "123"})
+	err := cmd.Execute()
+
+	c.Assert(err, qt.ErrorMatches, `cannot cutover with the output format "csv".*`)
+	c.Assert(svc.CutoverFnInvoked, qt.IsFalse)
+}
+
 func TestWorkflow_CutoverCmd_Error(t *testing.T) {
 	c := qt.New(t)
 

internal/cmd/workflow/switch_traffic.go

@@ -54,6 +54,10 @@ By default, this command will route all queries for primary, replica, and read-o
 						cmdutil.AgentWorkflowActionCmd(wfCtx.Org, db, num, "switch-traffic", retryFlags...))
 				}
 
+				if ch.Printer.Format() != printer.Human {
+					return fmt.Errorf(`cannot switch query traffic with the output format "%s" (run with --force to override)`, ch.Printer.Format())
+				}
+
 				if !printer.IsTTY {
 					return fmt.Errorf("cannot confirm switching query traffic (run with --force to override)")
 				}

internal/cmd/workflow/switch_traffic_test.go

@@ -163,6 +163,37 @@ func TestWorkflow_SwitchTrafficCmd_Replicas(t *testing.T) {
 	c.Assert(buf.String(), qt.JSONEquals, expectedWorkflow)
 }
 
+func TestWorkflow_SwitchTrafficCmd_CSVWithoutForce(t *testing.T) {
+	c := qt.New(t)
+
+	var buf bytes.Buffer
+	format := printer.CSV
+	p := printer.NewPrinter(&format)
+	p.SetResourceOutput(&buf)
+
+	svc := &mock.WorkflowsService{}
+
+	ch := &cmdutil.Helper{
+		Printer: p,
+		Config: &config.Config{
+			Organization: "planetscale",
+		},
+		Client: func() (*ps.Client, error) {
+			return &ps.Client{
+				Workflows: svc,
+			}, nil
+		},
+	}
+
+	cmd := SwitchTrafficCmd(ch)
+	cmd.SetArgs([]string{"planetscale", "123"})
+	err := cmd.Execute()
+
+	c.Assert(err, qt.ErrorMatches, `cannot switch query traffic with the output format "csv".*`)
+	c.Assert(svc.SwitchPrimariesFnInvoked, qt.IsFalse)
+	c.Assert(svc.SwitchReplicasFnInvoked, qt.IsFalse)
+}
+
 func TestWorkflow_SwitchTrafficCmd_Error(t *testing.T) {
 	c := qt.New(t)