feat(ui): add /theme picker (rebased from #543)#567
Merged
Conversation
Surfaces a SingleSelect modal for the terminal theme preference. Bare \`/theme\` opens the picker; \`/theme <name>\` keeps its existing "persist and report" behaviour. Moves the slash entry from the advanced group to setup and adds high-contrast to the arg completer. Adapted from J3y0r's PR — repointed onto current main (which had since reformatted App.tsx around the InflightProvider wrap from #566) and the count test in tests/ui-slash-suggestions.test.tsx updated from 12 → 11 advanced commands now that /theme has moved out of the advanced bucket. Thanks @J3y0r for the contribution.
Closed
ChasLui
pushed a commit
to ChasLui/DeepSeek-Reasonix
that referenced
this pull request
May 23, 2026
Surfaces a SingleSelect modal for the terminal theme preference. Bare \`/theme\` opens the picker; \`/theme <name>\` keeps its existing "persist and report" behaviour. Moves the slash entry from the advanced group to setup and adds high-contrast to the arg completer. Adapted from J3y0r's PR — repointed onto current main (which had since reformatted App.tsx around the InflightProvider wrap from esengine#566) and the count test in tests/ui-slash-suggestions.test.tsx updated from 12 → 11 advanced commands now that /theme has moved out of the advanced bucket. Thanks @J3y0r for the contribution.
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.
Re-land of @J3y0r's #543 on top of current main.
Why a fresh PR
The original #543 branched from before #549 / #552 / #555 / #560 /
#566 landed; #566 in particular reformatted `src/cli/ui/App.tsx`
around a new `` wrap, which made GitHub flag #543
as DIRTY and produced a ~400-line conflict on local merge (mostly
indentation, but with the wrap interleaved). Rebasing onto current
main was simpler than resolving the merge in-place.
What's in the PR
Same shape as #543:
`SingleSelect` listing `auto` + every registered theme, marking
the current preference and the active theme.
on bare invocation; `/theme ` keeps its existing persist-
and-report behaviour.
and `high-contrast` added to the arg completer.
inclusion, slash-result branch, and the picker render branch in
the modal-conditional chain.
Delta vs #543
wrap is preserved.
expectation updated 12 → 11 (the contributor caught the total going
37 → 38 but missed the advanced count dropping by one when /theme
moved out of advanced) and the matching `"+ 12 advanced"` string.
Test plan
Thanks @J3y0r for the contribution. Closing #543 in favour of this rebased version.