Fix semicolon-separated parameter parsing for git count-contributors (#3052)

utils/cliutils/codegangstautils.go

@@ -20,9 +20,21 @@ func GetIntFlagValue(c *cli.Context, flagName string, defValue int) (int, error)
 	return defValue, nil
 }
 
+// GetStringsArrFlagValue parses semicolon-separated flag values into a string slice.
 func GetStringsArrFlagValue(c *cli.Context, flagName string) (resultArray []string) {
 	if c.IsSet(flagName) {
-		resultArray = append(resultArray, strings.Split(c.String(flagName), ";")...)
+		flagValue := c.String(flagName)
+		if flagValue == "" {
+			return []string{}
+		}
+
+		parts := strings.Split(flagValue, ";")
+		for _, part := range parts {
+			trimmed := strings.TrimSpace(part)
+			if trimmed != "" {
+				resultArray = append(resultArray, trimmed)
+			}
+		}
 	}
 	return
 }

utils/cliutils/semicolon_parsing_test.go

@@ -0,0 +1,106 @@
+package cliutils
+
+import (
+	"flag"
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+	"github.com/urfave/cli"
+)
+
+func TestGetStringsArrFlagValue(t *testing.T) {
+	tests := []struct {
+		name     string
+		flagValue string
+		expected []string
+	}{
+		{
+			name:     "Single value",
+			flagValue: "repo1",
+			expected: []string{"repo1"},
+		},
+		{
+			name:     "Multiple values",
+			flagValue: "repo1;repo2;repo3",
+			expected: []string{"repo1", "repo2", "repo3"},
+		},
+		{
+			name:     "Values with spaces",
+			flagValue: "repo1; repo2 ; repo3",
+			expected: []string{"repo1", "repo2", "repo3"},
+		},
+		{
+			name:     "Empty value",
+			flagValue: "",
+			expected: []string{},
+		},
+		{
+			name:     "Values with empty parts",
+			flagValue: "repo1;;repo2;",
+			expected: []string{"repo1", "repo2"},
+		},
+		{
+			name:     "Complex repository names",
+			flagValue: "my-org/repo1;another-org/repo-with-dashes;third_org/repo_with_underscores",
+			expected: []string{"my-org/repo1", "another-org/repo-with-dashes", "third_org/repo_with_underscores"},
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			flagSet := flag.NewFlagSet("test", flag.ContinueOnError)
+			flagSet.String("test-flag", tt.flagValue, "")
+			err := flagSet.Parse([]string{"--test-flag", tt.flagValue})
+			assert.NoError(t, err)
+
+			c := cli.NewContext(nil, flagSet, nil)
+
+			result := GetStringsArrFlagValue(c, "test-flag")
+			assert.Equal(t, tt.expected, result)
+		})
+	}
+}
+
+func TestOverrideArrayIfSet(t *testing.T) {
+	tests := []struct {
+		name     string
+		flagValue string
+		expected []string
+	}{
+		{
+			name:     "Single repository",
+			flagValue: "my-repo",
+			expected: []string{"my-repo"},
+		},
+		{
+			name:     "Multiple repositories",
+			flagValue: "repo1;repo2;repo3",
+			expected: []string{"repo1", "repo2", "repo3"},
+		},
+		{
+			name:     "Repositories with spaces",
+			flagValue: " repo1 ; repo2 ; repo3 ",
+			expected: []string{"repo1", "repo2", "repo3"},
+		},
+		{
+			name:     "Empty flag value",
+			flagValue: "",
+			expected: []string{},
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			flagSet := flag.NewFlagSet("test", flag.ContinueOnError)
+			flagSet.String("test-flag", tt.flagValue, "")
+			err := flagSet.Parse([]string{"--test-flag", tt.flagValue})
+			assert.NoError(t, err)
+
+			c := cli.NewContext(nil, flagSet, nil)
+
+			var result []string
+			overrideArrayIfSet(&result, c, "test-flag")
+			assert.Equal(t, tt.expected, result)
+		})
+	}
+}

utils/cliutils/utils.go

@@ -385,7 +385,21 @@ func overrideStringIfSet(field *string, c *cli.Context, fieldName string) {
 // If `fieldName` exist in the cli args, read it to `field` as an array split by `;`.
 func overrideArrayIfSet(field *[]string, c *cli.Context, fieldName string) {
 	if c.IsSet(fieldName) {
-		*field = append([]string{}, strings.Split(c.String(fieldName), ";")...)
+		flagValue := c.String(fieldName)
+		if flagValue == "" {
+			*field = []string{}
+			return
+		}
+
+		parts := strings.Split(flagValue, ";")
+		result := []string{}
+		for _, part := range parts {
+			trimmed := strings.TrimSpace(part)
+			if trimmed != "" {
+				result = append(result, trimmed)
+			}
+		}
+		*field = result
 	}
 }