disable slices on abc variables commands

Author: bahrmichael

cmd/src/abc_variables_delete.go

@@ -16,6 +16,7 @@ var abcVariablesDeleteCommand = clicompat.Wrap(&cli.Command{
 	Name:      "delete",
 	Usage:     "Delete variables on a workflow instance",
 	UsageText: "src abc variables delete [options] <workflow-instance-id> [<name> ...]",
+	DisableSliceFlagSeparator: true,
 	Description: `
 Delete workflow instance variables
 

cmd/src/abc_variables_set.go

@@ -28,6 +28,7 @@ var abcVariablesSetCommand = clicompat.Wrap(&cli.Command{
 	Name:      "set",
 	UsageText: "src abc variables set [options] <workflow-instance-id> [<name>=<value> ...]",
 	Usage:     "Set variables on a workflow instance",
+	DisableSliceFlagSeparator: true,
 	Description: `
 Set workflow instance variables
 

cmd/src/abc_variables_set_test.go

@@ -9,6 +9,7 @@ import (
 	mockapi "github.com/sourcegraph/src-cli/internal/api/mock"
 	"github.com/stretchr/testify/mock"
 	"github.com/stretchr/testify/require"
+	"github.com/urfave/cli/v3"
 )
 
 // If we were to do a json marshalling roundtrip, it may break large integer literals.
@@ -48,6 +49,27 @@ func TestParseABCVariables(t *testing.T) {
 	}
 }
 
+func TestABCVariablesSetVarFlagPreservesCommas(t *testing.T) {
+	t.Parallel()
+
+	var got []string
+	cmd := &cli.Command{
+		Name:                      "set",
+		DisableSliceFlagSeparator: abcVariablesSetCommand.DisableSliceFlagSeparator,
+		Flags: []cli.Flag{
+			&cli.StringSliceFlag{Name: "var"},
+		},
+		Action: func(ctx context.Context, cmd *cli.Command) error {
+			got = cmd.StringSlice("var")
+			return nil
+		},
+	}
+
+	err := cmd.Run(context.Background(), []string{"set", "--var", "checkpoints=[1,2,3]"})
+	require.NoError(t, err)
+	require.Equal(t, []string{"checkpoints=[1,2,3]"}, got)
+}
+
 func TestRunABCVariablesSet(t *testing.T) {
 	t.Parallel()