Avoid duplicating logic across the codebase. Instead, extract common functionality into well-named helper functions or standalone utilities. This improves code readability, simplifies maintenance, and centralizes logic for easier updates.
Avoid duplicating logic across the codebase. Instead, extract common functionality into well-named helper functions or standalone utilities. This improves code readability, simplifies maintenance, and centralizes logic for easier updates.
Examples where this applies:
Before:
if (provider.toLowerCase() === "githubcopilot" && !apiKey) {
apiKey = await fetchGithubCopilotApiKey();
// Additional logic for handling the API key...
} else if (cli.flags.login) {
if (provider.toLowerCase() === "githubcopilot") {
apiKey = await fetchGithubCopilotApiKey();
// Duplicate logic...
}
}
After:
function handleGithubCopilotLogin() {
// Centralized login logic
}
if (provider.toLowerCase() === "githubcopilot" && !apiKey) {
apiKey = await handleGithubCopilotLogin();
} else if (cli.flags.login) {
if (provider.toLowerCase() === "githubcopilot") {
apiKey = await handleGithubCopilotLogin();
}
}
Enter the URL of a public GitHub repository