Skip to content

ci(pr-automation): broaden automated review - #1885

Merged
Benoît Cortier (CBenoit) merged 3 commits into
masterfrom
cbenoit-lower-reviewer-threshold
Sep 3, 2026
Merged

ci(pr-automation): broaden automated review#1885
Benoît Cortier (CBenoit) merged 3 commits into
masterfrom
cbenoit-lower-reviewer-threshold

Conversation

@CBenoit

@CBenoit Benoît Cortier (CBenoit) commented Sep 2, 2026

Copy link
Copy Markdown
Member

Run automated review for every eligible non-draft pull request and lower contributor history to one same-author pull request merged into master while exempting owners and members.

Replace author quotas with a 50-PR repository-wide fork quota and let the oversized opt-in raise complete diff evidence from 1 MiB to the runtime maximum of 4 MiB.

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

Run automated review for every eligible non-draft pull request and
lower contributor history to one qualifying merge while exempting
owners and members.

Replace author quotas with a 50-PR repository-wide fork quota and let
the oversized opt-in raise complete diff evidence from 1 MiB to the
runtime maximum of 4 MiB.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Treat every pull request from the same immutable human author that
merged into master as qualifying contributor history.

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

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

Removing the oversized opt-in does not revoke its snapshotted 4 MiB allowance from queued runs.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Broadens automated PR review eligibility while revising evidence and fork-quota limits.

Changes:

  • Reviews all eligible non-draft human PRs after exact-head CI.
  • Adds 1/4 MiB evidence limits and oversized retry handling.
  • Replaces author quotas with a 50-PR global fork quota.

Protocol review was skipped because no RDP behavior changes.

File summaries
File Description
.github/workflows/review-pipeline.yml Passes the evidence limit into review preparation.
.github/workflows/labeler.yml Expands routing and propagates evidence limits.
.github/workflows/labeler.intent.md Documents the revised policy intent.
.github/pr-automation/write-state.js Updates quota and evidence-limit guidance.
.github/pr-automation/routing.js Removes size and low-risk review suppression.
.github/pr-automation/resolve-state.js Revises eligibility and state cleanup.
.github/pr-automation/resolve-pr.js Selects the evidence limit from labels.
.github/pr-automation/labels.json Clarifies the oversized-review label.
.github/pr-automation/fork-rate-limit.js Implements the global fork quota.
.github/pr-automation/fetch-pr-evidence.sh Enforces trusted 1/4 MiB limits.
.github/pr-automation/automation.test.js Tests the revised automation policy.
.github/PR_AUTOMATION.md Documents operational policy changes.
Review details
  • Files reviewed: 12/12 changed files
  • Comments generated: 1
  • Review effort level: Balanced

Comment thread .github/pr-automation/resolve-pr.js
Restart classification when the oversized evidence opt-in is removed
so the PR concurrency group cancels queued or running use of 4 MiB.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@github-actions github-actions Bot added maintainer-required Maintainer review or intervention is required risk/unknown Risk could not be determined automatically; needs maintainer-level scrutiny scope/tooling Build, CI, release, or developer tooling size/L Size: up to 899 counted lines and 20 files; exceeds M in either measure labels Sep 2, 2026
@CBenoit
Benoît Cortier (CBenoit) merged commit 2fec684 into master Sep 3, 2026
42 checks passed
@CBenoit
Benoît Cortier (CBenoit) deleted the cbenoit-lower-reviewer-threshold branch September 3, 2026 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer-required Maintainer review or intervention is required risk/unknown Risk could not be determined automatically; needs maintainer-level scrutiny scope/tooling Build, CI, release, or developer tooling size/L Size: up to 899 counted lines and 20 files; exceeds M in either measure

Development

Successfully merging this pull request may close these issues.

2 participants