Skip to content

fix(pr-review): paginate complete file evidence - #1

Open
BigDataDZ wants to merge 1 commit into
ehz0ah:codex/fix-pr-review-body-files-scanfrom
BigDataDZ:codex/pr-4077-file-pagination
Open

fix(pr-review): paginate complete file evidence#1
BigDataDZ wants to merge 1 commit into
ehz0ah:codex/fix-pr-review-body-files-scanfrom
BigDataDZ:codex/pr-4077-file-pagination

Conversation

@BigDataDZ

Copy link
Copy Markdown

Summary

  • compare the GraphQL detail page against the scalar changedFiles count
  • fetch all changed files through the paginated REST endpoint when the detail page is truncated
  • fail closed when the paginated result still does not match the authoritative count
  • move GitHub source reads into the existing PR-review capability boundary so the hot command module stays within its maintainability budget

Validation

  • pytest -q tests/test_pr_review_github_scan.py tests/canary/test_maintainability_ratchet.py — 18 passed
  • ruff check on all changed Python files — passed
  • python examples/pr-review-command-smoke.py — passed
  • real microsoft/vscode#334528 REST pagination — 822/822 files returned
  • public-boundary scan — clean
  • standard premerge gate — passed, including maintainability ratchet

This is a focused follow-up to huangruiteng#4077 and targets its head branch directly.

Signed-off-by: BigDataDZ <76271875+BigDataDZ@users.noreply.github.com>
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.

1 participant