potiuk opened a new pull request, #286:
URL: https://github.com/apache/airflow-steward/pull/286
## Summary
Adds an optional variant of the `request-author-confirmation`
template body for projects that prefer a maintainer-sweep
handback over the default reviewer-ping handback. Selected via
a new `confirmation_handback_mode` key in
`<project-config>/pr-management-config.md` (default
`reviewer-ping`, current behaviour unchanged).
## The two flows side-by-side
| | `reviewer-ping` (default) | `maintainer-sweep` (new) |
|---|---|---|
| What the author does | Marks threads resolved + `@`-pings the reviewer for
a final look | Replies with a short `yes / ready` |
| Who advances the PR next | The reviewer applies the `ready for maintainer
review` label | The next triage sweep promotes via
`author_confirmation_received` → `mark-ready` |
| Best fit | Reviewers themselves apply the label, or no maintainer-sweep
cadence exists | Project runs a regular maintainer triage cadence and prefers
lightweight contributor confirmation |
The downstream flow (sweep detects the author's affirmative
reply via `author_confirmation_received` and offers the
maintainer the `mark-ready` action) is identical between the
variants.
## What changed
- **`.claude/skills/pr-management-triage/comment-templates.md`**
— adds a `### Variant: maintainer-sweep handback` subsection
under the existing `## Request author confirmation` section.
Documents the alternative body + notes on why it differs
(no `<reviewer_logins>` in para 1; `<PROJECT>` placeholder in
the "If yes" sentence; same marker string).
- **`projects/_template/pr-management-config.md`** — adds a new
`## Workflow choices` section with the
`confirmation_handback_mode` key. Default `reviewer-ping` so
existing adopters keep the current behaviour.
No changes to `actions.md`, `classify-and-act.md`, or
`stale-sweeps.md`. The action name (`request-author-confirmation`)
is the same; the agent picks which body to render based on the
config value.
## Motivation
The apache/airflow project's
`.apache-steward-overrides/pr-management-triage-comment-templates.md`
has been carrying a custom `request-author-confirmation` body
with the maintainer-sweep "if yes" branch, working as a
project-local hack since the two-sweep flow landed. Promoting
it to an optional framework variant lets airflow (and any
adopter with a similar triage cadence) configure rather than
override.
## Test plan
- [x] prek passes (markdownlint, skill-validator, doctoc,
typos, check-placeholders)
- Post-merge follow-up (apache/airflow side): set
`confirmation_handback_mode: maintainer-sweep` in its
pr-management-config override and drop the airflow-specific
body from
`.apache-steward-overrides/pr-management-triage-comment-templates.md`
Generated-by: Claude (Opus 4.7)
--
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]
