fix(ci): fire pr-review caller on ready_for_review; partition concurrency by draft flag - #20
Conversation
…ency by draft flag Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Auto-merge refused — This PR touches a path category in If this is misclassified, edit the caller repo's |
|
Risk class: This PR touches one of the blocked path categories from Auto-merge is refused by (This is a policy notice, not a code-quality failure. The classify job itself does not fail — required CI checks remain authoritative for "is the code green.") |
Closes the last fail-open residue of the pr-review
ready_for_reviewcampaign (45-repo audit, 2026-07-15; reference fix whois-api-llc/wxa-mcp-server#344): this repo's caller already has the draft-partitioned concurrency group but itspull_requesttypes still lackready_for_review.Why it matters: the reusable
claude-review.ymlgates ondraft == false, so a PR opened as draft skips the requiredreview / Claude Reviewatopened— and without this event, marking the PR ready fires nothing the workflow hears. The check staysskipped, which GitHub counts as a satisfied required context, so the PR can merge green and unreviewed (wxa-mcp-server#339 merged 7s after readying this way; fleet policy makes draft the standard auto-merge opt-out, so manual-merge PRs are precisely the ones hit). It resurfaced concretely tonight on whois-api-llc/disposable-email-domains#167, and the central draft-gate lint (ci-workflows#115) now fails every PR in a repo whose caller has the gap.One line:
types: [opened, synchronize, reopened, ready_for_review]. No behavior change for non-draft PRs.Auto-merge rationale: manual click-merge — touches
.github/workflows/**(fleet policy).🤖 Generated with Claude Code