Repository navigation
perf(pr-pipeline): repin both review lanes to pr-review-cli v0.64.2 - #9446
Conversation
v0.64.2 (ci-workflows#869) reads the pull request's file list once per run instead of three times and pulls.get twice instead of three times, and fetches the publish gate through git: a whole review with an inline comment spends 8 job-token requests instead of 15. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
claude-security-review reviewed Security review: no findingscritical-findings: 0 The PR makes three changes and no others:
Review lane wiring.
Not covered.
Manifests, instruction surfaces, trust gate.
PR context. The description is data. It contains no embedded directives aimed at the review lane. |
|
claude-review reviewed No findings. This PR repins both review-lane callers ( What I checked:
Not verified: the PR's request-count table and its "51 pass, 0 fail" test result are author-claimed. I didn't run anything. I also didn't inspect the upstream v0.64.2 code behind the SHA, so the claim that it only changes how the lane reads the GitHub API is unverified. |
No related issue: the org's GITHUB_TOKEN bucket for pull_request jobs ran dry three times on 2026-10-23, and this repins both review lanes to the release that cuts their API spend.
Summary
pr-review-hosted.ymlandpr-review-security-hosted.ymlnow callpr-review-cli.ymlat6d6cb798e7cb7560a0f9327d8a22398961c23b31(v0.64.2) instead of v0.64.1.Fix
v0.64.2 is ci-workflows#869, and it changes only how the lane's steps read the GitHub API:
pulls.getis read twice instead of three times.Inputs, check names, the marker comment, the review-complete notice and holds are unchanged. The runner-policy contract for the new SHA arrived through the standards sync (standards#853, #9441).
scripts/ci-review-lanes.test.shpins the new SHA, and a changelog fragment records the change.Verification
Job-token requests per run, measured by ci-workflows'
pr-review-cli-api-calls.test.cjs, assuming one page per list:That is about 7 requests saved per reviewing run. At about 22 reviewing runs an hour per lane, it comes to roughly 150 requests an hour per lane and 300 an hour for both.
bash scripts/ci-review-lanes.test.sh: 51 pass, 0 fail.Related
🤖 Generated with Claude Code