Repository navigation
refactor(r-lib): prefix all skill names with r-* - #90
Merged
Merged
Conversation
Rename r-lib skills to use a consistent r-* prefix so names are self-describing across categories: - cli -> r-cli - cran-extrachecks -> r-cran-extrachecks - lifecycle -> r-lifecycle - mirai -> r-mirai - testing-r-packages -> r-testthat Update the name frontmatter in each renamed SKILL.md, plus all references in marketplace.json, README.md, r-lib/README.md, CLAUDE.md, and cross-references in other skills. Also audit r-lib/README.md: add missing sections for r-cli-app and r-package-development, correct the alt-text shared-skill note, and refresh the Potential Skills list. Document the naming convention in CONTRIBUTING.md: prefer a category-based prefix (r-*, shiny-*, quarto-*, connect-*) when reasonable.
skill-validator exits 2 for warnings-only results and 1 for errors, but validate-skills.sh treated any non-zero exit as failure, so pre-existing warnings could fail unrelated PRs. Now only exit code 1 fails CI; warnings still appear in the job summary. Also rewrite the r-testthat description as prose so it no longer reads as a keyword list (validator frontmatter warning).
gadenbuie
marked this pull request as ready for review
October 6, 2026 20:41
Scans text files for references to renamed skills, reporting high-confidence path references (e.g. "r-lib/cli/SKILL.md") separately from bare name mentions that need human review (the old name may be an R package, website, or ordinary word). Defaults to the r-lib r-* prefix renames; --rename old=new supports other migrations. Exits non-zero on matches so it can gate CI. Document it in CLAUDE.md, and add a renaming checklist to CONTRIBUTING.md's naming guidance.
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
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
Renames all
r-lib/skills to use a consistentr-*prefix so names are self-describing across categories, and documents the naming convention in CONTRIBUTING.md.Renames
cli→r-clicran-extrachecks→r-cran-extracheckslifecycle→r-lifecyclemirai→r-miraitesting-r-packages→r-testthat(matches the package-named pattern ofr-cli,r-lifecycle,r-mirai)Migration
Verify the migration
Run the repo's new
find-stale-references.pyutility against any project to find lingering references to the old names:Claude Code
If you installed via the plugin marketplace (Methods 1–2 in the README), no manual work is needed — the renames ship as part of the
r-libplugin. Update the plugin and the new skill names replace the old ones:Or in a Claude Code session, run
/pluginand choose Update on ther-libplugin (restart Claude Code afterwards to pick up the changes).If you installed with
npx skills, use the commands below — add-a claude-codeto target Claude Code only.If you installed the renamed skills globally with the skills CLI, remove the old names and install the new ones:
The other
r-libskills (r-cli-app,r-cran-status,r-package-development,r-tidyverse-style) are unchanged and need no action. Alternatively, remove all globally installed Posit skills and re-add the ones you want:Reference updates
name:frontmatter in each renamed SKILL.md.claude-plugin/marketplace.json, rootREADME.md,CLAUDE.mdr-lib/r-tidyverse-styleandposit-dev/review-testingr-lib/README.md audit
r-cli-appandr-package-developmentalt-textnote (lives at repo root; shared acrossr-libandquartoplugins)CI fix
.github/scripts/validate-skills.shonly failed CI on the validator's error exit code (1); it previously treated warnings-only runs (exit 2) as failures, so pre-existing warnings could fail unrelated PRs. Warnings still appear in the job summary.r-testthatdescription as prose so it no longer reads as a keyword list.CONTRIBUTING.md
r-*,shiny-*,quarto-*,connect-*) when reasonable