Give crispctl a conventional help screen and uuid selectors - #107
Merged
Conversation
The help text read like a man page pasted into a terminal: full invocations under a heading, then four paragraphs. It now prints what every comparable tool prints, a usage line, an aligned command table with one-line descriptions, and a short footer with the output format and exit codes. The long explanations live in the README section, which already said most of it. Two changes ride along, both cheap while nothing has shipped. `displays list` becomes `display list`, so every command uses the same singular noun as the display commands proposed in #97. And `brightness get` and `brightness set` take a uuid as well as a runtime id, through a `selector` field on the request that the app resolves id first, then uuid in any case; the numeric `display` field still works for clients that already send it. The resolver is written to match #97's, so that PR can adopt it on rebase. Verified live on the Dell U2412M: get and set by id, by uuid in both cases, unknown selectors refused with exit 3, and the old command name rejected with the usage line. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016NDzD18GySiCwEGfWJiT4J
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.
The help text read like a man page pasted into a terminal: full invocations under a heading, then four paragraphs. It now prints what every comparable tool prints: a usage line, an aligned command table with one-line descriptions, and a short footer with the output format and exit codes. The long explanations live in the README section, which already said most of it.
Two changes ride along, both cheap while nothing has shipped.
displays listbecomesdisplay list, so every command uses the same singular noun as the display commands in #97. Andbrightness getandbrightness settake a uuid as well as a runtime id, through aselectorfield the app resolves id first, then uuid in any case. The numericdisplayfield still works for clients that already send it. The resolver matches the one in #97 so that PR can adopt it on rebase.Verified live on the Dell U2412M: get and set by id and by uuid in both cases, unknown selectors refused with exit 3, the old command name rejected with the usage line.
🤖 Generated with Claude Code
https://claude.ai/code/session_016NDzD18GySiCwEGfWJiT4J