Rename generic utility files in pkg/cli to reflect their specific purpose (#3627)

Author: Copilot

pkg/cli/frontmatter_utils.go pkg/cli/frontmatter_editor.gopkg/cli/frontmatter_utils.go renamed to pkg/cli/frontmatter_editor.go

@@ -10,24 +10,24 @@ import (
 	"github.com/githubnext/gh-aw/pkg/workflow"
 )
 
-var frontmatterUtilsLog = logger.New("cli:frontmatter_utils")
+var frontmatterEditorLog = logger.New("cli:frontmatter_editor")
 
 // UpdateFieldInFrontmatter updates a field in the frontmatter while preserving the original formatting
 // when possible. It tries to preserve whitespace, comments, and formatting by working with the raw
 // frontmatter lines, similar to how addSourceToWorkflow works.
 func UpdateFieldInFrontmatter(content, fieldName, fieldValue string) (string, error) {
-	frontmatterUtilsLog.Printf("Updating frontmatter field: %s = %s", fieldName, fieldValue)
+	frontmatterEditorLog.Printf("Updating frontmatter field: %s = %s", fieldName, fieldValue)
 
 	// Parse frontmatter using parser package
 	result, err := parser.ExtractFrontmatterFromContent(content)
 	if err != nil {
-		frontmatterUtilsLog.Printf("Failed to parse frontmatter: %v", err)
+		frontmatterEditorLog.Printf("Failed to parse frontmatter: %v", err)
 		return "", fmt.Errorf("failed to parse frontmatter: %w", err)
 	}
 
 	// Try to preserve original frontmatter formatting by manually updating the field
 	if len(result.FrontmatterLines) > 0 {
-		frontmatterUtilsLog.Printf("Using raw frontmatter lines for field update (%d lines)", len(result.FrontmatterLines))
+		frontmatterEditorLog.Printf("Using raw frontmatter lines for field update (%d lines)", len(result.FrontmatterLines))
 		// Look for existing field in the raw lines
 		fieldUpdated := false
 		frontmatterLines := make([]string, len(result.FrontmatterLines))
@@ -55,7 +55,7 @@ func UpdateFieldInFrontmatter(content, fieldName, fieldValue string) (string, er
 					frontmatterLines[i] = fmt.Sprintf("%s%s: %s", leadingSpace, fieldName, fieldValue)
 				}
 				fieldUpdated = true
-				frontmatterUtilsLog.Printf("Updated existing field %s in place (line %d)", fieldName, i+1)
+				frontmatterEditorLog.Printf("Updated existing field %s in place (line %d)", fieldName, i+1)
 				break
 			}
 		}
@@ -64,7 +64,7 @@ func UpdateFieldInFrontmatter(content, fieldName, fieldValue string) (string, er
 		if !fieldUpdated {
 			newField := fmt.Sprintf("%s: %s", fieldName, fieldValue)
 			frontmatterLines = append(frontmatterLines, newField)
-			frontmatterUtilsLog.Printf("Added new field %s at end of frontmatter", fieldName)
+			frontmatterEditorLog.Printf("Added new field %s at end of frontmatter", fieldName)
 		}
 
 		// Reconstruct the file with preserved formatting

pkg/cli/shared_utils.go pkg/cli/pr_automerge.gopkg/cli/shared_utils.go renamed to pkg/cli/pr_automerge.go

@@ -12,7 +12,7 @@ import (
 	"github.com/githubnext/gh-aw/pkg/logger"
 )
 
-var sharedUtilsLog = logger.New("cli:shared_utils")
+var prAutomergeLog = logger.New("cli:pr_automerge")
 
 // PullRequest represents a GitHub Pull Request
 type PullRequest struct {
@@ -27,7 +27,7 @@ type PullRequest struct {
 // AutoMergePullRequestsCreatedAfter checks for open PRs in the repository created after a specific time and auto-merges them
 // This function filters PRs to only those created after the specified time to avoid merging unrelated PRs
 func AutoMergePullRequestsCreatedAfter(repoSlug string, createdAfter time.Time, verbose bool) error {
-	sharedUtilsLog.Printf("Checking for PRs in repo=%s created after %s", repoSlug, createdAfter.Format(time.RFC3339))
+	prAutomergeLog.Printf("Checking for PRs in repo=%s created after %s", repoSlug, createdAfter.Format(time.RFC3339))
 
 	if verbose {
 		fmt.Fprintln(os.Stderr, console.FormatInfoMessage(fmt.Sprintf("Checking for open pull requests in %s created after %s", repoSlug, createdAfter.Format(time.RFC3339))))
@@ -37,7 +37,7 @@ func AutoMergePullRequestsCreatedAfter(repoSlug string, createdAfter time.Time,
 	listCmd := exec.Command("gh", "pr", "list", "--repo", repoSlug, "--json", "number,title,isDraft,mergeable,createdAt,updatedAt")
 	output, err := listCmd.Output()
 	if err != nil {
-		sharedUtilsLog.Printf("Failed to list pull requests: %v", err)
+		prAutomergeLog.Printf("Failed to list pull requests: %v", err)
 		return fmt.Errorf("failed to list pull requests: %w", err)
 	}
 
@@ -64,14 +64,14 @@ func AutoMergePullRequestsCreatedAfter(repoSlug string, createdAfter time.Time,
 	}
 
 	if len(eligiblePRs) == 0 {
-		sharedUtilsLog.Print("No eligible PRs found for auto-merge")
+		prAutomergeLog.Print("No eligible PRs found for auto-merge")
 		if verbose {
 			fmt.Fprintln(os.Stderr, console.FormatInfoMessage(fmt.Sprintf("No pull requests found created after %s", createdAfter.Format(time.RFC3339))))
 		}
 		return nil
 	}
 
-	sharedUtilsLog.Printf("Found %d eligible PRs for auto-merge", len(eligiblePRs))
+	prAutomergeLog.Printf("Found %d eligible PRs for auto-merge", len(eligiblePRs))
 	fmt.Fprintln(os.Stderr, console.FormatInfoMessage(fmt.Sprintf("Found %d pull request(s) created after workflow start time", len(eligiblePRs))))
 
 	for _, pr := range eligiblePRs {
@@ -118,7 +118,7 @@ func AutoMergePullRequestsLegacy(repoSlug string, verbose bool) error {
 
 // WaitForWorkflowCompletion waits for a workflow run to complete, with a specified timeout
 func WaitForWorkflowCompletion(repoSlug, runID string, timeoutMinutes int, verbose bool) error {
-	sharedUtilsLog.Printf("Waiting for workflow completion: repo=%s, runID=%s, timeout=%d minutes", repoSlug, runID, timeoutMinutes)
+	prAutomergeLog.Printf("Waiting for workflow completion: repo=%s, runID=%s, timeout=%d minutes", repoSlug, runID, timeoutMinutes)
 
 	if verbose {
 		fmt.Fprintln(os.Stderr, console.FormatInfoMessage(fmt.Sprintf("Waiting for workflow completion (timeout: %d minutes)", timeoutMinutes)))