fix(readme): standardize bare slash commands, rename status→health - #85
Merged
Conversation
- Rename skills/status → skills/health so bare /health alias doesn't collide with Claude Code's built-in /status - Delete redundant README "Commands" section (was duplicating Workflows and Skills tables); replace with one-liner under Workflows explaining bare names work and /devkit:<name> is the disambiguation fallback - Fix /tri:review typo in Quick Start → /tri-review - Update ROADMAP.md references to match new names
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.
Summary
## Commandssection in README — it was repeating the Workflows and Skills tables in a third format and was internally inconsistent (/tri-reviewbare vs/devkit:statusnamespaced in the same table)./feature,/bugfix,/tri-review,/health,/setup-rules); document/devkit:<name>as the disambiguation fallback.skills/status→skills/healthso its bare alias/healthdoesn't collide with Claude Code's built-in/status. Namespaced form becomes/devkit:health./tri:reviewtypo in the Quick Start block →/tri-review.Why
/statuswas colliding with the built-in CC/statusin tab-complete (two entries). Renaming the skill tohealthkills the collision without forcing users to type the namespaced form. README was also mixing bare and namespaced forms in the same paragraph, which is what surfaced the confusion in the first place.Test plan
go test ./...insrc/— all packages pass (cmd/engine/lib/mcp/runners)bash hooks/hooks_test.sh— 70 passed, 0 failedskills/statuspath (verified via grep)CHANGELOG.md:150historical/devkit:statusreference intentionally left frozen (managed by release pipeline)/healthtab-completes to the devkit skill and/statusonly shows the built-in