fix: export not generated due to single API failure

[amenocal]

Description

This pull request improves error handling and code maintainability in the repository validation process. It refactors the API retrieval logic to handle partial failures gracefully, ensuring that the validation process continues even if some API requests fail.

In addition introduces utility methods for selecting the appropriate API client to reduce code duplication

Checklist

• I have added the appropriate label to this PR according to the type of change
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• I have added tests

Additional Context

Improved Error Handling and User Feedback:

• Refactored the retrieveSource and retrieveTarget methods in validator.go to handle individual API failures gracefully: instead of failing on the first error, the code now logs the error, sets a default value, and continues with remaining requests. At the end, it summarizes the number of successful and failed requests, and only fails if all requests fail. [1] [2] [3] [4]

Refactoring and Code Simplification:

• Added getGraphQLClient and getRESTClient helper methods to GitHubAPI for consistent and centralized selection of the appropriate API client based on ClientType, replacing repeated switch statements throughout the code.
• Updated all repository data retrieval methods (e.g., GetIssueCount, GetPRCounts, GetTagCount, etc.) in api.go to use the new helper methods, reducing code duplication and improving maintainability. [1] [2] [3] [4] [5] [6] [7] [8]

These changes make the migration validation process more resilient to partial API failures and easier to maintain.

[Copilot]

Pull Request Overview

This pull request improves error handling in the repository validation process by making it more resilient to individual API failures. Instead of failing the entire export when a single API request fails, the code now logs errors, uses default values, and continues processing.

• Enhanced error handling to gracefully handle partial API failures during data retrieval
• Added utility methods to reduce code duplication in API client selection
• Improved user feedback with detailed success/failure reporting

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
internal/validator/validator.go	Enhanced error handling in retrieveSource and retrieveTarget methods to continue processing despite individual API failures
internal/api/api.go	Added helper methods getGraphQLClient and getRESTClient to centralize client selection logic and reduce duplication

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[pmartindev]

LGTM