potiuk opened a new pull request, #437: URL: https://github.com/apache/airflow-steward/pull/437
## Summary First phase of the Magpie restructure (skills layout & namespace). **Pure relocation + tooling retarget — no skill renames, no dotfile renames, no adoption-semantics changes.** - Move all **38** framework skills from `.claude/skills/<name>/` to a top-level **`skills/<name>/`** (the source of truth, committed). The framework no longer keeps active skills in `.claude/skills/`; a maintainer self-adopts the framework into itself to wire the now-gitignored `.claude/skills/` symlinks. - Retarget every tool that addressed the old location: `skill-and-tool-validator` `SKILLS_DIR`, `check-placeholders.sh` `SCAN_PATHS`, the three pre-commit globs (doctoc / check-placeholders / skill-and-tool-validate), `list_skills.py`'s discovery comment, and every eval `step-config.json` `skill_md` path (120 files). - Fix relative-link **depth** inside the moved skill files (`../../../` -> `../../`, one level shallower now -- 813 occurrences) and **repoint** every markdown link that targeted `.claude/skills/<skill>/` to `skills/<skill>/` (285 links across 62 files). - Gitignore `/.claude/skills/`. ## Type of change - [x] Cross-cutting (RFC, AGENTS.md, sandbox, privacy-LLM) — framework layout - [x] CI / dev loop (`prek`, workflows, validators) - [x] Skill change (`.claude/skills/<name>/`) — relocation only; **no behaviour/eval-fixture content change** (only `step-config.json` `skill_md` paths) ## Test plan - [x] `uv run --project tools/skill-and-tool-validator skill-and-tool-validate` → passes scanning the new `skills/` tree (5 pre-existing soft warnings only) - [x] `uv run --project tools/skill-and-tool-validator pytest` (validator unit tests) → green with retargeted `skills/` paths - [x] `prek run --from-ref HEAD~1 --to-ref HEAD` → doctoc, markdownlint, typos, check-placeholders, ruff, mypy, pytest(workspace), skill-and-tool-validate all **Passed** - [x] `python3 skills/list-steward-skills/scripts/list_skills.py` discovery resolves to `skills/` and finds all 38 SKILL.md - [x] Local relative-link sweep: no new broken links vs `main` (lychee is file-relative; the only residue is pre-existing code-block/adopter-path examples lychee already tolerates) - Eval suite: the two pre-existing `security-issue-sync` / `setup-isolated-setup-verify` heading-drift failures exist on `main` and are unrelated to this move. ## Scope notes / deferred to later phases - **Adopter-facing `.claude/skills/` references stay unchanged** (adopters still install into `.claude/skills/`). Only framework-*source*-location references moved. - Skill renames (`setup-steward`->`setup`, `list-steward-skills`->`list-skills`), the `magpie-` adoption prefix, the `.apache-steward*` -> `.apache-magpie*` dotfile rename, the `/setup-steward upgrade` migration logic, and the broader docs cross-reference sweep land in subsequent PRs. 🤖 Prepared with AI assistance (Claude Code, Opus 4.8); reviewed by the human author before submission. -- 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]
