Skip to content

Revert #92: pr-gate authorize auto-closes private org members (CONTRIBUTOR) - #93

Merged
noahgift merged 1 commit into
mainfrom
revert/92-author-association
Oct 22, 2026
Merged

noahgift merged 1 commit into
mainfrom
revert/92-author-association

Conversation

@noahgift

Copy link
Copy Markdown
Contributor

Revert rule (two-way door): #92 (5841c07, merged 18:03:45Z) breaks the gate.

  • Why: authorize decides from the payload's author_association alone. A private org member's PR arrives as CONTRIBUTOR, so pr-gate closes it.
  • Number: since the merge, two org-owner PRs have been closed: ci: push-to-main tier diffs event.before..after and fails closed to full (PMAT-5952) aprender#5981 (closed 18:36:00Z, auto-close comment) and #5982 (18:38:14Z). No new PR from a private member can open.
  • Trade-off: this restores the ambient-auth membership probe. Its known failure on a runner with no gh login (gx10) is a red "indeterminate", not a closed PR.
  • Re-land: with a members/<login> probe for private members, after a fresh sign-off round.

🤖 Generated with Claude Code

Revert rule (two-way door): #92 breaks the gate. Private org members
arrive in the payload as CONTRIBUTOR, so pr-gate auto-closed noahgift's
aprender#5981 (18:36:00Z) with the "organization members" comment, and
#5982 closed at 18:38:14Z. Number: 2 org-owner PRs closed since the 18:03:45Z
merge; 0 can open. Re-land with a members/<login> probe for private members.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@noahgift
noahgift enabled auto-merge (squash) October 22, 2026 19:03
@noahgift
noahgift merged commit 4587f12 into main Oct 22, 2026
2 checks passed
@noahgift
noahgift deleted the revert/92-author-association branch October 22, 2026 19:40
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