potiuk opened a new pull request, #33: URL: https://github.com/apache/airflow-steward/pull/33
## Summary - Lift three maintainer-facing PR-queue skills from `apache/airflow` into the framework: `pr-triage`, `pr-stats`, `pr-maintainer-review` (renamed from `maintainer-review` per the category-prefix convention). 7522 lines of skill content moved. - Genericise: all `apache/airflow` → `<upstream>`, `Apache Airflow` → `<PROJECT>` per the framework's placeholder convention (enforced by `tools/dev/check-placeholders.sh`). - Add adopter-contract scaffolds in `projects/_template/`: `pr-triage-config.md`, `pr-triage-comment-templates.md`, `pr-triage-ci-check-map.md`, `pr-maintainer-review-criteria.md`. Each skill's `SKILL.md` adds a new `## Adopter configuration` section pointing at these. - README's `PR triage and review` section graduates from a placeholder to the actual skill listing + adopter contract pointers. ## What the skills do | Skill | Purpose | |---|---| | `pr-triage` | First-pass triage: classify, propose disposition (draft / comment / close / rebase / rerun / mark ready / ping), execute on confirmation. Successor to `breeze pr auto-triage`. | | `pr-stats` | Read-only summary tables of the open PR backlog grouped by area label. | | `pr-maintainer-review` | Deep code review one PR at a time. Reads diff, applies project's review criteria, drafts `APPROVE` / `REQUEST_CHANGES` / `COMMENT` review with inline findings. | ## Known follow-up work The framework currently ships with airflow-flavored defaults inline in the supporting files of each skill (e.g. `pr-triage/comment-templates.md` references airflow's PR-quality-criteria URL; `pr-maintainer-review/criteria.md` lists airflow's `code-review.instructions.md` and `AGENTS.md` paths). The new template scaffolds in `projects/_template/` document the contract; a **follow-up PR will complete the extraction** so the skills read exclusively from `<project-config>/`. **Until then, non-airflow adopters override by forking the relevant supporting file into their own `.claude/skills/`.** This is documented in each skill's new "Adopter configuration" section. ## Two minor pre-commit-config additions - `.typos.toml`: allowlist bracket-prefix CLI prompt fragments (`ody` from `[B]ody`, `equest` from `[R]equest`, etc.) used in the skills' interactive prompts. - `pr-triage-ci-check-map.md` heading uses `to` instead of `→`: markdownlint MD051 doesn't recognise UTF-8 arrows in heading-derived anchors. ## Test plan - [x] All pre-commit hooks pass (markdownlint, typos, doctoc, `check-placeholders`) - [x] No `apache/airflow` / `Apache Airflow` literals in framework files - [x] Slash command `/maintainer-review` → `/pr-maintainer-review` renamed everywhere - [x] Internal cross-refs (`pr-triage` ↔ `pr-stats`, `pr-triage` → `pr-maintainer-review`) updated - [ ] Reviewer eyeballs whether the four template scaffolds capture the right contract knobs (committers team / area-label prefix / CI-check map / review-criteria source files) - [ ] CI link-check (lychee) passes 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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]
