Add skill support to the installer, plus the pick-pr-review skill - #3
Open
gregjkal wants to merge 2 commits into
Open
Add skill support to the installer, plus the pick-pr-review skill#3gregjkal wants to merge 2 commits into
gregjkal wants to merge 2 commits into
Conversation
gregjkal
force-pushed
the
add-skill-support
branch
from
August 5, 2026 16:09
9da0a47 to
ee438f0
Compare
A command is a single prompt file copied to ~/.claude/commands. A skill is a directory: a SKILL.md plus whatever scripts it calls. The whole directory has to travel together, which is why skills need their own list and their own path. Skills install to ~/.claude/skills and ~/.cursor/skills, since Claude Code and Cursor read the same SKILL.md format and one directory serves both. Override with SKILL_DEST, or point it at a single path to install for one agent only. Register a skill by adding its directory to the SKILLS array. Entries resolve from the repo root rather than tools/, since skills live outside it. The array is empty here; the first skill follows. Re-install clears the previous copy before writing, so a file dropped upstream does not linger. Uninstall removes a directory only if it still contains a SKILL.md.
Scans the open PRs across wg21-website and wg21-paperflow and recommends the single one worth reviewing next, to remove the friction of choosing. Ranks by who is waiting on whom: PRs you reviewed where the author has since replied or pushed, then PRs where your review was requested, then unreviewed ones. Your own PRs, PRs you approved with nothing changed since, and drafts you have no involvement with are dropped. Ties break on how close the PR sits to areas you have committed to in the last 90 days, then oldest wait first. The skill offers to run the review rather than starting one, and posts findings as a pending draft review for the human to submit.
gregjkal
force-pushed
the
add-skill-support
branch
from
August 5, 2026 17:01
ee438f0 to
17843a9
Compare
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
1 participant
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.
Makes the installer able to ship directory-based tools, not just single prompt files, and adds the first one.
.mdin~/.claude/commands. A skill is a directory:SKILL.mdplus the scripts it calls. Register one by adding its directory to the newSKILLSarray.SKILL.mdformat, so one directory serves both. Destinations override withSKILL_DEST=a:b./pick-pr-review. Scans open PRs acrosswg21-websiteandwg21-paperflowand names the single one worth reviewing next, ranked by who is waiting on whom. It offers to run the review rather than starting one, and posts findings as a pending draft for the human to submit.Tested on bash 3.2, the macOS system bash that
curl | bashactually uses: fresh install, re-install, uninstall, single-destination override, and the real tarball download path.One call for you: no paired image. AGENTS.md says to stop and ask when one is missing, so I am asking. I left it out because
SKILL.mdis agent-facing, loaded into the model's context rather than read by a person, so an<img>tag is dead weight and the PNG ships to every install. Say the word and I will add one, either attools-wg21/images/for the README or inside the skill directory.Follow-up in #4, stacked on this: the
tools-wg21reorg had left the installer silently shipping 21 commands instead of 35, and the README WG21 links dead. Skills were unaffected.