Repository navigation
docs(skill): run the command's help before naming what it takes - #80
Merged
Merged
Conversation
A skill is written once and read for as long as it is installed, so the options it lists are the options that existed the day it was written. An agent reading only the skill reported that `art concept` had no transparent-background option; `--transparent` has been there all along, and one free `--help` would have said so. The rule goes in four places, because an agent may read any one of them alone: the entry skill, each of the six category skills where the option lists live, and the block `setup` writes into the file a harness reads every session. The skills keep their option lists — a list nobody wrote is not an improvement on a list a version behind — but they are now framed as what the page knows, with the command's own help named as what the installed version accepts. From the code review: `CATEGORY_SKILLS` in the tests never listed the animation skill, so five of the six were checked and the sixth was tested by nothing at all — the same staleness one layer up. It is in the list now, and `TestNothingShipsUntested` binds that list to what `packaged_skills` ships, so the next skill cannot be added without being checked. specs/agent-skill R5.12. Verified with scc validate --checks --pr: 1736 passed, 58 skipped, coverage 95.28%, ruff check and format clean, artifacts clean.
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.
No description provided.