Skip to content

Conversation

@mishraomp
Copy link
Collaborator

@mishraomp mishraomp commented May 17, 2025


Thanks for the PR!

Any successful deployments (not always required) will be available below.
API available
Frontend available

Once merged, code will be promoted and handed off to following workflow run.
Main Merge Workflow

@mishraomp mishraomp requested a review from DerekRoberts as a code owner May 17, 2025 18:26
@mishraomp mishraomp requested review from DerekRoberts and Copilot and removed request for DerekRoberts May 17, 2025 18:26
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds retry logic with exponential backoff and jitter to the GraphQL query function to handle GitHub API rate limits.

  • Introduces getGraphQlResponseOnQuery with configurable retry attempts and backoff
  • Implements detailed logging for rate‐limit retries
  • Documents the new function with JSDoc
Comments suppressed due to low confidence (1)

crawler/src/main.js:161

  • Consider adding unit tests or integration tests that mock rate-limit responses and verify that the retry count, exponential backoff delays, and success/failure paths behave as expected.
async function getGraphQlResponseOnQuery(query) {

@mishraomp mishraomp requested a review from Copilot May 17, 2025 19:15
@mishraomp mishraomp enabled auto-merge (squash) May 17, 2025 19:15
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

A PR to improve rate limit handling for GraphQL queries by implementing exponential backoff with jitter.

  • Introduces new environment-based delay constants (BASE_DELAY, MAX_DELAY, and JITTER_FACTOR).
  • Adds the getGraphQlResponseOnQuery function which implements retry logic with exponential backoff and jitter.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@mishraomp mishraomp merged commit f889bab into main May 17, 2025
13 of 14 checks passed
@mishraomp mishraomp deleted the fix/handle-403-graphql branch May 17, 2025 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants