You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "potiuk (via GitHub)" <gi...@apache.org> on 2026/05/02 12:29:48 UTC
[PR] refactor(skills): rename skills with category prefixes (setup-/security-) [airflow-steward]
potiuk opened a new pull request, #31:
URL: https://github.com/apache/airflow-steward/pull/31
## Summary
- Rename 13 skills with `setup-` / `security-` category prefixes (flat-on-disk — Claude Code only discovers flat `.claude/skills/<name>/`, nested subdirs verified empirically as not picked up).
- Drop `apache-` from `*-apache-steward` skill names (pre-empts the framework rename to `apache/steward`); pluralise batch-style skills (`security-import-issues`, `security-sync-issues`, `security-deduplicate-issues`).
- Update cross-references in 46 files (README, AGENTS.md, CONTRIBUTING.md, secure-agent-setup.md, secure-agent-internals.md, projects/_template/, tools/, `generate-cve-json` docstrings). 353/353 insertions/deletions — pure rename, no semantic change.
Preparation for adding the `pr-*` skill family (`pr-triage`, `pr-stats`, `pr-maintainer-review`) lifted from `apache/airflow` in a follow-up PR.
## Rename table
| Old | New |
|---|---|
| `setup-secure-config` | *(unchanged — already prefixed)* |
| `verify-secure-config` | `setup-verify-secure-config` |
| `update-secure-config` | `setup-update-secure-config` |
| `upgrade-apache-steward` | `setup-upgrade-steward` |
| `verify-apache-steward` | `setup-verify-steward` |
| `sync-shared-config` | `setup-sync-shared-config` |
| `import-security-issue` | `security-import-issues` |
| `import-security-issue-from-md` | `security-import-issues-from-md` |
| `import-security-issue-from-pr` | `security-import-issue-from-pr` |
| `sync-security-issue` | `security-sync-issues` |
| `fix-security-issue` | `security-fix-issue` |
| `allocate-cve` | `security-allocate-cve` |
| `deduplicate-security-issue` | `security-deduplicate-issues` |
| `invalidate-security-issue` | `security-invalidate-issue` |
## Breaking change for adopters
Adopters using the framework via submodule + symlinks (e.g. the `airflow-s` tracker) need to:
1. Bump their submodule pointer to the merge commit of this PR.
2. Update any project-side `.claude/skills/<name>` symlinks to the new names.
## Test plan
- [x] Claude Code discovers all renamed skills (live-change detection picked them up in-session)
- [x] No bare references to old names remain in tracked files (grep audit)
- [x] No double-prefix collisions (`setup-setup-`, `security-security-`)
- [x] Pre-commit hook passes locally
- [ ] CI link-check (lychee) passes
- [ ] Reviewer eyeballs the rename table for plural/singular consistency
🤖 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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] refactor(skills): rename skills with category prefixes (setup-/security-) [airflow-steward]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk merged PR #31:
URL: https://github.com/apache/airflow-steward/pull/31
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org