docs: install the kit without --skill '*' - #84
Merged
Conversation
0.14.0 made the picker offer all nine skills under one select-all row, so the flag now only repeats what a single keystroke does. The documented install is `npx skills add svyatov/oss-kit` in the README, the install guide, the site's copy button, `oss-audit`, and the `oss-readme` example. `oss-skill` told readers a whole-collection install wants the flag. It now says a plugin manifest naming each skill directory does that job.
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.
What changed
npx skills add svyatov/oss-kit, in the README, the install guide, the site's copy button,oss-audit's Step 1, and the example README insideoss-readme. The--skill '*'flag existed to preselect nine loose rows in the picker, and 0.14.0 replaced those rows with one select-all row, so the flag now only repeats what a single keystroke does.skills/oss-skill/references/hosts.mdsaid a repository documenting a whole-collection install wants--skill '*'. It now says the CLI groups a repository's skills when a plugin manifest names each skill's own directory, with the caveat that a single./skills/entry matches no skill directory and groups nothing.InstallCommand.astrodrops the logic that joined--skillto its value. One span per word stays, becausepre-wrapbreaks after a hyphen and every hyphen left in the command sits inside a name a reader has to retype.custom.cssjustified layout decisions by naming the flag, and now describe the constraint without it.Why this way
The hero column stays at
0.92fr. That value was measured against the longer command, so a shorter one only adds slack, and the comment says exactly that rather than claiming a measurement nobody took.@latestwas in this change for one revision and came back out. It would have forced npx past a cached CLI old enough to miss the grouping, and it costs eight characters on the line the whole site is arranged around.Historical
CHANGELOG.mdentries that record the flag being added are untouched, as is thehosts.mdsentence documenting what the flag does, which is still true.Verification
bun test328 pass,bun run validate0 errors,bash scripts/check-drift.shandbun scripts/check-ecosystems.mjsclean, and the site builds 114 pages. Read back fromsite/dist/index.html, the hero rendersnpx skills add svyatov/oss-kitas four spans, so the copy handler'stextContentreproduces the command exactly.Known gaps
The grouped row itself was verified against
getPluginGroupingsin the CLI source during 0.14.0, not by running the installer against a published tag on this branch.Affects
hosts.mdno longer recommends--skill '*'for a whole-collection install