Refactor: Extract duplicate parsing logic into shared helper functions

pkg/workflow/add_comment.go

@@ -127,16 +127,13 @@ func (c *Compiler) parseCommentsConfig(outputMap map[string]any) *AddCommentsCon
 				}
 			}
 
-			// Parse target-repo
-			if targetRepoSlug, exists := configMap["target-repo"]; exists {
-				if targetRepoStr, ok := targetRepoSlug.(string); ok {
-					// Validate that target-repo is not "*" - only definite strings are allowed
-					if targetRepoStr == "*" {
-						return nil // Invalid configuration, return nil to cause validation error
-					}
-					commentsConfig.TargetRepoSlug = targetRepoStr
-				}
+			// Parse target-repo using shared helper
+			targetRepoSlug := parseTargetRepoFromConfig(configMap)
+			// Validate that target-repo is not "*" - only definite strings are allowed
+			if targetRepoSlug == "*" {
+				return nil // Invalid configuration, return nil to cause validation error
 			}
+			commentsConfig.TargetRepoSlug = targetRepoSlug
 
 			// Parse discussion
 			if discussion, exists := configMap["discussion"]; exists {

pkg/workflow/config.go

@@ -0,0 +1,46 @@
+package workflow
+
+// parseLabelsFromConfig extracts and validates labels from a config map
+// Returns a slice of label strings, or nil if labels is not present or invalid
+func parseLabelsFromConfig(configMap map[string]any) []string {
+	if labels, exists := configMap["labels"]; exists {
+		if labelsArray, ok := labels.([]any); ok {
+			var labelStrings []string
+			for _, label := range labelsArray {
+				if labelStr, ok := label.(string); ok {
+					labelStrings = append(labelStrings, labelStr)
+				}
+			}
+			// Return the slice even if empty (to distinguish from not provided)
+			if labelStrings == nil {
+				return []string{}
+			}
+			return labelStrings
+		}
+	}
+	return nil
+}
+
+// parseTitlePrefixFromConfig extracts and validates title-prefix from a config map
+// Returns the title prefix string, or empty string if not present or invalid
+func parseTitlePrefixFromConfig(configMap map[string]any) string {
+	if titlePrefix, exists := configMap["title-prefix"]; exists {
+		if titlePrefixStr, ok := titlePrefix.(string); ok {
+			return titlePrefixStr
+		}
+	}
+	return ""
+}
+
+// parseTargetRepoFromConfig extracts and validates target-repo from a config map
+// Returns the target repository slug, or empty string if not present or invalid
+// Returns error string "*" if the wildcard value is used (which is invalid for target-repo)
+// Callers should check for "*" and handle it as an error condition
+func parseTargetRepoFromConfig(configMap map[string]any) string {
+	if targetRepoSlug, exists := configMap["target-repo"]; exists {
+		if targetRepoStr, ok := targetRepoSlug.(string); ok {
+			return targetRepoStr
+		}
+	}
+	return ""
+}