Skip to content

Use page info for team pagination - #306

Open
hosom wants to merge 1 commit into
mainfrom
hosom/graphql-page-info
Open

hosom wants to merge 1 commit into
mainfrom
hosom/graphql-page-info

Conversation

@hosom

@hosom hosom commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Summary

Use pageInfo.endCursor for GitHub team pagination instead of requesting a cursor on every member edge.

The Entitlements benchmark found that refreshing the 4,835-member Proxima github/employees team requires approximately 49 sequential GraphQL pages. Requesting one cursor per member adds thousands of unused cursor values to the response. This aligns the provider query with the optimized query already used by entitlements-caches.

The parser retains compatibility with cached or older responses that provide only edge cursors.

Validation

  • script/test -d -k - 190 examples, 0 failures, 100% coverage
  • bundle exec rubocop -c .rubocop.yml lib/ spec/ - 31 files, no offenses

Related cache deployment: github/entitlements-caches#85

Tracking: github/entitlements-app#80

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 48d43b26-c466-49f6-ac97-aa304ac378fd
Copilot AI balanced review requested due to automatic review settings September 15, 2026 17:27

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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.

2 participants