Pit aliases for the new commands, and a real help verb - #20
Merged
Conversation
`cli-tools help` is what people type, and it fell through to the passthrough: "unknown command: help", exit 1, printed before the usage that answers the question. It is a verb now. Three aliases join the set: /web -> ask-web, /speak -> tts, and /aff -> affiliate. The names are chosen around a collision rather than for elegance. /ask and /say are the words you would reach for and both already resolve to something else on a normal box; /tts would shadow our own command. An alias beats PATH, so binding any of those would shadow the real program from inside the pit only — which is about the most confusing failure available, since the same word keeps working in every other shell. The existing invariants already covered the risk and still pass: an alias may not share a name with a command here, and must expand to one that exists. Added a test pinning the three names, so a later tidy-up does not "simplify" /web back to /ask. Suite is 275 tests. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
ThreatCrush Security Scan6 finding(s) MEDIUM: 1 | LOW: 5
Snippets are redacted; ThreatCrush never prints matched credential material. |
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.
Two small gaps found while checking whether auto-update was discoverable from the pit.
cli-tools helpwas an errorIt fell through to the command passthrough:
unknown command: help, exit 1 — printed before the usage that answers the question. It's a verb now, exit 0.Three aliases for the commands that had none
/webask-web/speaktts/affaffiliateThe names are chosen around a collision rather than for elegance.
/askand/sayare the words you'd reach for, and both already resolve to something else on a normal box;/ttswould shadow our own command. An alias beatsPATH, so binding any of those would shadow the real program from inside the pit only — the same word keeps working in every other shell, which is about the most confusing failure available.The repo's existing alias invariants already covered that risk and still pass (an alias may not share a name with a command here, and must expand to one that exists). Added a test pinning the three chosen names so a later tidy-up doesn't "simplify"
/webback to/ask.Alias tables in the README and
/tools:installupdated to match.Verification
pnpm typecheckclean.cli-tools helpexits 0;cli-tools aliaseslists all eight.🤖 Generated with Claude Code