fix(pr-review): include full review metadata in scan#5373
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates scripts/pr_review.py to set include_full_reviews to True in the GraphQL query fields. The reviewer notes that fetching full reviews during bulk scans could lead to GraphQL payload bloat and potential rate limits or timeouts, and recommends reverting this change.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| + pr_node_fields( | ||
| comments_last=15, | ||
| include_full_reviews=False, | ||
| include_full_reviews=True, |
There was a problem hiding this comment.
[MED] GraphQL query payload bloat in bulk scan. Evidence:
scripts/pr_review.py:2300.\n> Why it matters: Fetching up to 50 reviews for every open PR during a bulk scan significantly increases the GraphQL payload and execution time, risking rate limits or query timeouts on large repositories.\n> Suggested fix: Revert toinclude_full_reviews=Falsefor the scan query and rely on the single-PR query's full fetch when acting on candidates.
| include_full_reviews=True, | |
| include_full_reviews=False, |
1 participant
No description provided.