refactor(skills): rename to type-what_you_operate_on-action convention#35
Merged
potiuk merged 1 commit intoMay 3, 2026
Merged
Conversation
Standardise the 17 skill folder names + 4 adopter-config scaffold
files on `type-what_you_operate_on-action` for consistency and to
make the responsible noun + verb visible at a glance.
Setup family — `setup-{thing}-{action}`:
setup-secure-config → setup-isolated-setup-install
setup-verify-secure-config → setup-isolated-setup-verify
setup-update-secure-config → setup-isolated-setup-update
setup-upgrade-steward → setup-steward-upgrade
setup-verify-steward → setup-steward-verify
setup-sync-shared-config → setup-shared-config-sync
(`secure-config` becomes `isolated-setup` to more accurately name
what is being installed — an isolated agent setup, not just a
secure config.)
Security family — `security-{issue,cve}-{action}`, singular noun:
security-import-issues → security-issue-import
security-import-issues-from-md → security-issue-import-from-md
security-import-issue-from-pr → security-issue-import-from-pr
security-sync-issues → security-issue-sync
security-fix-issue → security-issue-fix
security-allocate-cve → security-cve-allocate
security-deduplicate-issues → security-issue-deduplicate
security-invalidate-issue → security-issue-invalidate
PR family — `pr-management-{action}` (type prefix promoted from
plain `pr-` to `pr-management-` to disambiguate from
`pr` as an enum value elsewhere):
pr-triage → pr-management-triage
pr-stats → pr-management-stats
pr-maintainer-review → pr-management-code-review
Adopter-config scaffolds renamed to match:
pr-triage-config.md → pr-management-config.md
pr-triage-comment-templates.md → pr-management-triage-comment-templates.md
pr-triage-ci-check-map.md → pr-management-triage-ci-check-map.md
pr-maintainer-review-criteria.md → pr-management-code-review-criteria.md
62 files touched, 488/488 ins/del — pure rename. Cross-references
updated everywhere (README, AGENTS.md, CONTRIBUTING.md,
secure-agent-setup.md, secure-agent-internals.md,
projects/_template/, .claude/skills/*/, tools/, generate-cve-json
docstrings). doctoc TOCs regenerated.
Generated-by: Claude Code (Claude Opus 4.7)
6 tasks
potiuk
added a commit
that referenced
this pull request
May 4, 2026
…sedes #30) (#44) * docs: enable markdownlint MD040 and tag all fenced code blocks Follow-up to #18. Flips MD040 from `false` to `true` and tags the 64 previously untagged fences across the tree. Most fences ended up `text` (MCP call sketches, URL examples, dir trees, plain output, commit trailers). 3 got `html` for HTML-comment idempotency markers and the <details> envelope, 3 `markdown` for the AI-disclosure block and rollup body samples, 1 `yaml` for the subagent return block in sync-security-issue. One nested-fence case in allocate-cve/SKILL.md needed the outer fence promoted from 3 to 4 backticks so the inner 3-backtick block renders as an actual nested code block instead of breaking the outer one. Signed-off-by: André Ahlert <andre@aex.partners> * docs: extend MD040 tagging to skills added since #30 #30 was opened against an earlier tree state; the pr-management skill family (lifted in #33, renamed to type-what-action in #35) added 9 new skill supporting files with 20 untagged fences that fail markdownlint MD040 once it's enabled. This commit applies the same tagging convention #30 established for the security family + tools to the new pr-management files: pr-management-triage/fetch-and-batch.md 1 fence pr-management-triage/comment-templates.md 2 fences → markdown pr-management-triage/interaction-loop.md 4 fences → text (UI mockups) pr-management-triage/workflow-approval.md 2 fences → text (UI mockups) pr-management-stats/fetch.md 3 fences → text (search queries) pr-management-stats/render.md 3 fences → text (output samples) pr-management-stats/classify.md 3 fences → text (pseudocode) pr-management-code-review/review-flow.md 1 fence → text (CLI mockup) pr-management-code-review/prerequisites.md 1 fence → text (HTTP error) Most fences ended up `text` (the same catch-all #30's commit message used for "MCP call sketches, URL examples, dir trees, plain output"). Two `markdown` fences in `pr-management-triage/comment-templates.md` because the content is a markdown link / list item example that GitHub should render as markdown. prek run --all-files clean. MD040 reports zero violations across the tree. Generated-by: Claude Code (Claude Opus 4.7) --------- Signed-off-by: André Ahlert <andre@aex.partners> Co-authored-by: André Ahlert <andre@aex.partners>
andreahlert
added
the
mode:platform
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
type-what_you_operate_on-actionshape (follow-up to refactor(skills): rename skills with category prefixes (setup-/security-) #31's category-prefix rename). 62 files touched, 488/488 ins/del — pure rename, no semantic change.setup-{thing}-{action}(e.g.setup-isolated-setup-installwassetup-secure-config).secure-configbecomesisolated-setupto more accurately name what's being installed.security-{issue,cve}-{action}with singular noun (issues→issue).pr-management-{action}— type prefix promoted from plainpr-topr-management-.projects/_template/renamed to match.Full rename table
Setup family
setup-secure-configsetup-isolated-setup-installsetup-verify-secure-configsetup-isolated-setup-verifysetup-update-secure-configsetup-isolated-setup-updatesetup-upgrade-stewardsetup-steward-upgradesetup-verify-stewardsetup-steward-verifysetup-sync-shared-configsetup-shared-config-syncSecurity family
security-import-issuessecurity-issue-importsecurity-import-issues-from-mdsecurity-issue-import-from-mdsecurity-import-issue-from-prsecurity-issue-import-from-prsecurity-sync-issuessecurity-issue-syncsecurity-fix-issuesecurity-issue-fixsecurity-allocate-cvesecurity-cve-allocatesecurity-deduplicate-issuessecurity-issue-deduplicatesecurity-invalidate-issuesecurity-issue-invalidatePR family
pr-triagepr-management-triagepr-statspr-management-statspr-maintainer-reviewpr-management-code-reviewAdopter-config scaffolds
pr-triage-config.mdpr-management-config.mdpr-triage-comment-templates.mdpr-management-triage-comment-templates.mdpr-triage-ci-check-map.mdpr-management-triage-ci-check-map.mdpr-maintainer-review-criteria.mdpr-management-code-review-criteria.mdBreaking change
Adopters using the framework via submodule + symlinks (the
airflow-stracker; the in-flightapache/airflow#66283adopter-config PR) need to:<project-config>/files that referenced the old scaffold names.For
apache/airflow#66283specifically: the four.apache-steward/pr-triage-*.mdand.apache-steward/pr-maintainer-review-criteria.mdfiles will need to be renamed during the next rebase — non-blocking for this PR since #66283 is unmerged and the user is redesigning that adoption mechanism anyway.Test plan
setup-isolated-setup-*is intentional but verbose)🤖 Generated with Claude Code