Skip to content

ci: label pull requests with suiflex-bot - #62

Merged
mulhamna merged 1 commit into
mainfrom
ci/suiflex-bot-triage
Aug 28, 2026
Merged

ci: label pull requests with suiflex-bot#62
mulhamna merged 1 commit into
mainfrom
ci/suiflex-bot-triage

Conversation

@mulhamna

Copy link
Copy Markdown
Member

Summary

  • Pull requests here carry no automatic signal about who opened them or what they contain, so triage starts by reading the author list and the commits by hand.
  • Adopts the organization-wide labeler defined in suiflex/.github. All logic lives there; this repository only calls it.

Changes

  • Adds .github/workflows/pr-triage.yml, a caller stub. Pull requests get maintainer, first contribution, or bot for the author, plus one commit: <type> label per conventional-commit type on the branch, and needs: conventional commit when a message is one release-please cannot parse.

The trigger is pull_request_target rather than pull_request, and the file carries a comment saying so. A fork's pull request runs with a read-only token that cannot apply labels — which is the case this is most needed for. It is safe because the workflow never checks out or executes the pull request's code; it only passes GitHub-provided event metadata to pinned actions. The same pattern is already used by cla.yml in this repository.

Test plan

  • Workflow parses as valid YAML
  • Upstream logic covered by scripts/test-pr-triage.mjs in suiflex/.github — 8 checks pass
  • Merge suiflex/.github#9 first; this stub references it at @main and fails until then
  • Create the suiflex-bot App and set SUIFLEX_BOT_APP_ID / SUIFLEX_BOT_PRIVATE_KEY as org secrets
  • Run sync-labels.sh against this repository so the labels exist
  • Confirm on a live PR that labels appear and the actor reads suiflex-bot

@mulhamna
mulhamna requested a review from a team August 28, 2026 15:25
Adds the organization's PR triage labeler so pull requests here are
marked with who opened them and which conventional-commit types the
branch carries.

Uses pull_request_target because a fork's pull request runs with a
read-only token that cannot apply labels, which is the case this is
most needed for. No repository code is checked out or executed.
@mulhamna
mulhamna force-pushed the ci/suiflex-bot-triage branch from 287a7bb to a2106ed Compare August 28, 2026 16:04
@mulhamna
mulhamna merged commit 12f8824 into main Aug 28, 2026
3 checks passed
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