n8n:community-pr-readiness-check

Source excerpt starting at line 1.

1# Community PR Readiness Check — triage label flow2 3A PR should carry **exactly one** `triage:<state>` label at any time. The skill drives the PR through this state machine.4 5## States6 7| State                  | Meaning                                                                |8|------------------------|------------------------------------------------------------------------|9| `triage:pending`       | Auto-applied by n8n-assistant when the PR opens. Skill removes it.     |10| `triage:in-progress`   | Skill is actively reviewing. Set in step 2, replaced before exit.      |11| `triage:complete`      | PR has been triaged to a team (or closed). Terminal.                   |12| `triage:needs-info`    | Comment posted; contributor needs to address something non-test.       |13| `triage:tests-needed`  | Comment posted; contributor needs to add tests (priority over info).   |14 15## Transitions16 17### Skill entry (step 2)18 19Before transitioning, **check for an existing `triage:in-progress` label**. If present, another reviewer is mid-triage — bail out without touching labels, Linear, or comments. This prevents two parallel runs from double-processing the same PR.20 21Strip any existing `triage:*` state label before adding `triage:in-progress`, so the single-state invariant holds even when re-reviewing a PR that was previously sent back with `triage:needs-info` or `triage:tests-needed`:22 23```bash24gh pr edit <number> --repo n8n-io/n8n \25  --remove-label "triage:pending" \26  --remove-label "triage:needs-info" \27  --remove-label "triage:tests-needed" \28  --remove-label "triage:complete" \29  --add-label "triage:in-progress"30```31 32Only one of those `triage:*` labels will actually be present; `--remove-label` errors when a label is missing, so run each removal as its own call (or batch and ignore errors) and then do the add.33 34### Skill exit — branch by outcome35 36#### Triaged to a team (action path 7B)37 38```bash39gh pr edit <number> --repo n8n-io/n8n \40  --remove-label "triage:in-progress" \41  --remove-label "status:pending-assignment" \42  --add-label "team:<slug>" \43  --add-label "status:team-assigned" \44  --add-label "triage:complete"45```46 47The `team:<slug>` is taken from `reference/teams.md`. Apply only after the Linear `save_issue` call succeeds — if Linear fails, leave the PR on `triage:in-progress` for the next loop run. `status:pending-assignment` may not be present (e.g. older PRs that predate the convention) — if `--remove-label` errors, drop the flag and retry, or run it as a separate call.48 49#### Closed (action path 7D)50 51```bash52gh pr edit <number> --repo n8n-io/n8n \53  --remove-label "triage:in-progress" \54  --remove-label "status:pending-assignment" \55  --add-label "status:internal-closed" \56  --add-label "triage:complete"57```58 59Apply alongside `gh pr close <number>` and the close-comment template (see SKILL.md). If the PR is already closed (`gh pr close` reports "already closed"), still apply the labels and the Linear cancellation.60 61#### Comment posted, contributor must act (action path 7C)62 63Pick **one** terminal label using this priority:64 65| Condition                                                              | Label                  |66|------------------------------------------------------------------------|------------------------|67| `TestsNeeded === true` **and** `TestsIncluded === false`               | `triage:tests-needed`  |68| Any other failing check (`CLA`, `Title`, `Description`, `CubicIssues`) | `triage:needs-info`    |69 70When both conditions hold, `triage:tests-needed` wins — it's the more specific signal and the comment already covers everything else.71 72```bash73gh pr edit <number> --repo n8n-io/n8n \74  --remove-label "triage:in-progress" \75  --add-label "triage:tests-needed"   # or triage:needs-info76```77 78#### Skill aborted before posting79 80If the user picks `Skip` at the "post this comment?" prompt, leave the PR on `triage:in-progress` — don't apply a terminal label without an accompanying contributor message. The next loop run will pick it up.81 82## status:* labels83 84These are independent of the triage state and signal where the PR sits in the n8n internal flow:85 86| Label                       | Applied when                                       |87|-----------------------------|----------------------------------------------------|88| `status:team-assigned`      | PR triaged to a Linear team (path 7B).             |89| `status:internal-closed`    | PR closed via path 7D.                             |90| `status:pending-assignment` | Default before the skill runs (set elsewhere).     |91 92The skill writes `status:team-assigned` (path 7B) or `status:internal-closed` (path 7D) and clears `status:pending-assignment` at the same time, so a triaged or closed PR drops out of any `status:pending-assignment` queue. It does not touch other `status:*` labels — reconciliation between team-assigned/internal-closed happens elsewhere if needed.93