Skip to content

refactor(r-lib): prefix all skill names with r-* - #90

Merged
gadenbuie merged 3 commits into
mainfrom
refactor/prefix-r-skills
Oct 6, 2026
Merged

gadenbuie merged 3 commits into
mainfrom
refactor/prefix-r-skills

Conversation

@gadenbuie

@gadenbuie gadenbuie commented Oct 6, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Renames all r-lib/ skills to use a consistent r-* prefix so names are self-describing across categories, and documents the naming convention in CONTRIBUTING.md.

Renames

  • cli → r-cli
  • cran-extrachecks → r-cran-extrachecks
  • lifecycle → r-lifecycle
  • mirai → r-mirai
  • testing-r-packages → r-testthat (matches the package-named pattern of r-cli, r-lifecycle, r-mirai)

Migration

Verify the migration

Run the repo's new find-stale-references.py utility against any project to find lingering references to the old names:

./find-stale-references.py ~/my-project
# or, high-confidence path references only
./find-stale-references.py --kinds path ~/my-project

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-lib plugin. Update the plugin and the new skill names replace the old ones:

claude plugin marketplace update posit-dev-skills
claude plugin update r-lib@posit-dev-skills

Or in a Claude Code session, run /plugin and choose Update on the r-lib plugin (restart Claude Code afterwards to pick up the changes).

If you installed with npx skills, use the commands below — add -a claude-code to target Claude Code only.

If you installed the renamed skills globally with the skills CLI, remove the old names and install the new ones:

npx skills remove -g cli cran-extrachecks lifecycle mirai testing-r-packages
npx skills add posit-dev/skills -g \
  --skill r-cli \
  --skill r-cran-extrachecks \
  --skill r-lifecycle \
  --skill r-mirai \
  --skill r-testthat

The other r-lib skills (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:

npx skills remove -g --skill '*' -y
npx skills add posit-dev/skills -g --skill '*' -y

Reference updates

  • name: frontmatter in each renamed SKILL.md
  • .claude-plugin/marketplace.json, root README.md, CLAUDE.md
  • Cross-references in r-lib/r-tidyverse-style and posit-dev/review-testing

r-lib/README.md audit

  • Added missing sections for r-cli-app and r-package-development
  • Corrected the alt-text note (lives at repo root; shared across r-lib and quarto plugins)
  • Refreshed the Potential Skills list (removed items now covered by existing skills)

CI fix

  • .github/scripts/validate-skills.sh only 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.
  • Rewrote the r-testthat description as prose so it no longer reads as a keyword list.

CONTRIBUTING.md

  • Added guidance to prefer category-based name prefixes (r-*, shiny-*, quarto-*, connect-*) when reasonable

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
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.
@gadenbuie
gadenbuie merged commit 0300946 into main Oct 6, 2026
1 check passed
@gadenbuie
gadenbuie deleted the refactor/prefix-r-skills branch October 6, 2026 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant