docs: add -y to the npx skills install commands - #146
Merged
Conversation
Running the documented command prompts with a "Select skills to install" multiselect that starts with nothing selected, so a developer has to space-select and confirm before anything installs. Adding -y installs all eight skills straight through.
|
| Name | Type |
|---|---|
| slack | Patch |
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
mwbrooks
marked this pull request as ready for review
September 3, 2026 23:19
WilliamBergamin
approved these changes
Sep 4, 2026
WilliamBergamin
left a comment
Contributor
There was a problem hiding this comment.
-y, I mean yesss do it
Member
Author
|
Thanks @WilliamBergamin! |
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
This pull request removes an interactive prompt from the
npx skillsinstall path, so a developer following the documented command gets the skills installed in one step instead of having to answer a question first.Running the command as documented today stops at a
Select skills to installmultiselect that starts with nothing selected, so the developer has to arrow, space-select, and confirm before anything is written. Adding-yinstalls all eight skills straight through. The flag is applied in all three places the command appears:README.md,docs/slack-skills-plugin.md(the page published to docs.slack.dev), and the unreleased changeset that documents this path.A sentence is added under each code block so the interactive choice is still discoverable: drop
-yto pick from the list, or pass-s <skill>to name skills explicitly.Testing
npx skills add slackapi/slack-skills-plugin -a opencodein a scratch project: stops atSelect skills to install,0/8selected, and writes nothing until confirmed.npx skills add slackapi/slack-skills-plugin -y -a opencodein the same project: printsInstalling all 8 skillsand completes with all eight in.agents/skills/plus askills-lock.json.rumdl check .clean at 19 files.Notes
-yis documented by the tool asSkip confirmation prompts, and with-a <agent>already supplied it is the only remaining prompt on this path..changeset/npx-skills-install-path.mddocuments this same command and has not shipped yet, so its command block is corrected in place rather than adding a second release note about the same flag.Requirements
make testand the tests pass. Docs-only change;rumdl check .run instead, and the evaluation suite needsGEMINI_API_KEYplusSLACK_MCP_TOKEN, which are not available locally.