Skip to content

fix: render the [sandbox] tag on speak/dub help instead of eating it#150

Merged
alexkroman merged 1 commit into
mainfrom
claude/command-consistency-wjmftu
Jun 13, 2026
Merged

fix: render the [sandbox] tag on speak/dub help instead of eating it#150
alexkroman merged 1 commit into
mainfrom
claude/command-consistency-wjmftu

Conversation

@alexkroman

Copy link
Copy Markdown
Collaborator

The speak and dub docstrings begin with a '[sandbox]' marker (adopted in
#146 to flag the two TTS commands that only work against --sandbox). With
rich_markup_mode="rich", Rich parsed '[sandbox]' as a style tag and dropped
it, leaving a stray leading space and an invisible marker — so these two
commands rendered with different indentation from every other command in
both the root --help table and their own help body.

Escape the bracket (raw docstring + [sandbox]), matching the same
bracket-escaping convention help_text.examples_epilog already uses, so the
marker renders literally and the help lines line up with the rest.

The speak and dub docstrings begin with a '[sandbox]' marker (adopted in
#146 to flag the two TTS commands that only work against --sandbox). With
rich_markup_mode="rich", Rich parsed '[sandbox]' as a style tag and dropped
it, leaving a stray leading space and an invisible marker — so these two
commands rendered with different indentation from every other command in
both the root --help table and their own help body.

Escape the bracket (raw docstring + \[sandbox]), matching the same
bracket-escaping convention help_text.examples_epilog already uses, so the
marker renders literally and the help lines line up with the rest.
@alexkroman alexkroman enabled auto-merge June 13, 2026 15:07
@alexkroman alexkroman added this pull request to the merge queue Jun 13, 2026
Merged via the queue into main with commit 4564f93 Jun 13, 2026
16 checks passed
@alexkroman alexkroman deleted the claude/command-consistency-wjmftu branch June 13, 2026 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants