[plan] Review parseRunURL() wrapper necessity in CLI package

[github-actions]

Objective

Assess whether the parseRunURL() wrapper in pkg/cli/audit.go adds value or just creates unnecessary indirection.

Context

Location: pkg/cli/audit.go:103

The function is a simple wrapper around parser.ParseRunURL() that converts the return values into a struct:

func parseRunURL(input string) (RunURLInfo, error) {
    runID, owner, repo, hostname, err := parser.ParseRunURL(input)
    if err != nil {
        return RunURLInfo{}, err
    }
    return RunURLInfo{
        RunID: runID,
        Owner: owner,
        Repo: repo,
        Hostname: hostname,
    }, nil
}

Analysis Required

1. Review all usages of parseRunURL() in pkg/cli/audit.go
2. Determine if the RunURLInfo struct provides value over individual return values
3. Check if this pattern is used elsewhere in the CLI package
4. Consider whether consolidating return values into a struct improves readability

Options

Option A: Keep the wrapper if it improves code readability
Option B: Remove wrapper and use parser.ParseRunURL() directly if indirection adds no value
Option C: Move RunURLInfo struct to parser package if it's generally useful

Files to Review

• Review: pkg/cli/audit.go (usage of parseRunURL wrapper)
• Review: pkg/parser/github_urls.go (original implementation)

Acceptance Criteria

• Analysis complete on whether wrapper adds value
• Decision documented (keep, remove, or refactor)
• If removing: wrapper eliminated and direct calls used
• If keeping: justification documented in code comments
• All tests pass (make test)
• No functionality changes

Estimated Effort

1 hour
Related to #5677

AI generated by Plan Command for #5506