Skip to content

docs(triage): verify maintainer access before surfacing PRs#106507

Merged
steipete merged 2 commits into
mainfrom
codex/triage-maintainer-access
Jul 13, 2026
Merged

docs(triage): verify maintainer access before surfacing PRs#106507
steipete merged 2 commits into
mainfrom
codex/triage-maintainer-access

Conversation

@steipete

@steipete steipete commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

What Problem This Solves

Generic triage could surface recent PRs from newly added maintainers when the static suppression list lagged current repository access, causing agents to pick up work maintainers already owned.

Why This Change Was Made

Live repository permission is now the source of truth before a finalist is surfaced. The existing login list remains a fast path, @pgondhi987 and @mmaps are added explicitly, and read-only or triage-only access otherwise remains outside the suppression rule.

User Impact

Generic triage prioritizes external contributors without accidentally selecting recent maintainer-authored PRs merely because a local list is stale. Named PR requests and the existing 14-day exception still work unchanged.

Evidence

  • Verified the reported public PR: fix: keep LINE group allowlists scoped [AI] #106056 was authored by @pgondhi987 and merged on 2026-07-13.
  • Verified current repository permission through the GitHub collaborator permission API before adding the access-derived fast-path entry; added the second explicit identity on maintainer request.
  • git diff --check
  • Documentation/skill-only change; no runtime code changed.

@openclaw-barnacle openclaw-barnacle Bot added size: XS maintainer Maintainer-authored PR labels Jul 13, 2026
@steipete steipete self-assigned this Jul 13, 2026
@steipete

Copy link
Copy Markdown
Contributor Author

Land-ready proof for head 34e94dfbf5d3dd0d1b2384f78d28f3fcbc6f05a7:

  • Change: make live repository permission authoritative for generic triage suppression; add @pgondhi987 and @mmaps to the explicit fast path.
  • Review: repo-native artifacts validated with zero findings.
  • Static proof: git diff --check origin/main...HEAD.
  • Prepare proof: OPENCLAW_TESTBOX=1 scripts/pr prepare-run 106507 passed hosted exact/recent-rebase gates at the current PR head.
  • Gap: no runtime tests; Markdown skill-only change.

Proceeding to land per maintainer instruction.

@steipete
steipete merged commit 260a5c6 into main Jul 13, 2026
53 checks passed
@steipete
steipete deleted the codex/triage-maintainer-access branch July 13, 2026 15:43
@steipete

Copy link
Copy Markdown
Contributor Author

Merged via squash.

wm0018 pushed a commit to wm0018/openclaw that referenced this pull request Jul 14, 2026
…#106507)

* docs(triage): verify maintainer access

* docs(triage): suppress mmaps PRs
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request Jul 14, 2026
…#106507)

* docs(triage): verify maintainer access

* docs(triage): suppress mmaps PRs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer Maintainer-authored PR size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant