potiuk opened a new pull request, #430:
URL: https://github.com/apache/airflow-steward/pull/430
## What
A new `pr-management` family skill, **`pr-management-quick-merge`**, that
screens the `ready for maintainer review` queue for the trivial tail — tiny PRs
in supplementary areas (docs, changelog, translations, tests) that pass *every*
quality gate — so a maintainer can clear them in seconds instead of letting
them age behind the heavyweight PRs.
## Boundary (the important part)
- **Surface-first.** It ranks candidates and prints the file list, a
per-gate attestation, and the exact `gh pr merge` command — for the maintainer
to run themselves.
- **It never merges.** Narrowly-scoped auto-merge is the framework's
**`mode:D`**, deliberately off until modes A/B/C have a two-quarter track
record (`docs/labels-and-capabilities.md`). This PR ships only the Mode-A
identification half; a "Why the skill does not merge" section documents that
merge belongs in a separate, Mode-D-labelled change when the gate lifts.
- **One mutation only:** an explicitly-confirmed, per-PR **APPROVE** review
— the same assistant-proposes / maintainer-fires act
`pr-management-code-review` already performs — so a maintainer can add their
approval to an all-green PR that has none yet, then merge. Never batched, never
auto, requires viewing the diff first, with a live gate re-check + head-SHA
optimistic lock before submitting. Gated by `enable_approve` (default on).
## Family fit
The three `pr-management` queue skills compose and drain the `ready` queue
from both ends:
- `pr-management-triage` fills the `ready for maintainer review` queue.
- **`pr-management-quick-merge`** (this PR) skims the trivial top.
- `pr-management-code-review` does the line-level read of the substantive
remainder — and quick-merge's **Step 5 explicitly hands its non-candidates**
(`too-large` / consequential-path PRs) off to it (a pointer, not an
auto-invocation).
## Capability
`capability:triage` + `capability:review` (added to the capability-to-skill
map in `docs/labels-and-capabilities.md`).
## Files
- `.claude/skills/pr-management-quick-merge/{SKILL.md,candidate-rules.md}`
- `projects/_template/pr-management-quick-merge-config.md` (thresholds,
allow/deny path globs, merge-command template, approve knobs)
- `docs/labels-and-capabilities.md` (map row)
Reuses the family's prerequisites / fetch-and-batch / Real-CI guard rather
than re-implementing them.
## Note for reviewers
`skill-and-tool-validate` reports one **pre-existing, unrelated** violation
— `tools/egress-gateway` missing its `README.md` — already present on `main`
and untouched by this PR (hence the `--no-verify` commit). This PR's own files
validate clean: zero violations, zero soft warnings.
Suggested labels: `area:pr-management`, `capability:triage`,
`capability:review`, `kind:dx`.
---
Generated-by: Claude Code (Opus 4.8)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
