potiuk opened a new pull request, #442:
URL: https://github.com/apache/airflow-steward/pull/442
## Summary
- All 38 framework skills install under a `magpie-` namespace prefix
(e.g. `skills/issue-triage` → `.claude/skills/magpie-issue-triage`),
but each `SKILL.md` `name:` frontmatter still carried the bare
directory name — a mismatch with the installed skill name. This
aligns every `name:` with its installed `magpie-<dir>` name.
- Adds a HARD validator check (`validate_name_convention`) enforcing
`name: magpie-<directory-name>` for every `SKILL.md`, so the
convention can't silently drift again.
## Type of change
- [x] Skill change (`.claude/skills/<name>/`)
- [x] CI / dev loop (`prek`, workflows, validators)
## Test plan
- [x] `prek run --all-files` passes (all hooks green on commit, incl.
`skill-and-tool-validate` + workspace `pytest`)
- [x] `uv run pytest` for the validator: 218 passed (5 new tests in
`TestValidateNameConvention`); `ruff check` + `mypy` clean
- Scope is `name:`-only: directory names and the install-time
symlink/prefix logic are untouched.
## RFC-AI-0004 compliance
- [x] **Vendor neutrality** — placeholders preserved; no project names
introduced
## Notes for reviewers
- The `magpie-` prefix is still applied at install time. This PR only
makes the source `name:` match the installed name and adds a guard so
the two can't diverge. No behaviour change to any skill, so no eval
fixtures were needed.
--
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]
