mis-set my API key, and it's a bit unclear how I r...
# proj-cli
w
mis-set my API key, and it's a bit unclear how I reset thonk
had to ctrl+p to find settings
not sure why that's not /settings
q
@nutritious-balloon-65098 please add
/settings
as a command it should open the settings modal
n
I'm on it! Rohit Malhotra can track my progress here.
## Summary I've successfully added the
/settings
slash command that opens the settings modal, addressing the user's request to make settings more discoverable (instead of requiring Ctrl+P or the command palette). ### Checklist - ✅ Added
/settings - Open settings
to the
COMMANDS
list in
commands.py
- ✅ Added
/settings
to the help text shown by
/help
command - ✅ Added command handler case and
_command_settings()
method in
input_area.py
- ✅ Updated all test assertions (count from 8→9, added
/settings
to parametrized tests) - ✅
make lint
passes - ✅
make test
passes (1260 tests) - ✅ Changes are minimal and focused only on the requested feature ### Files Changed -
openhands_cli/tui/core/commands.py
(+2 lines) -
openhands_cli/tui/widgets/input_area.py
(+6 lines) -
tests/tui/test_commands.py
(+8 lines, -2 lines) ### Commit The changes have been committed locally with message:
feat: add /settings slash command to open settings modal
Note: The changes still need to be pushed to the remote branch. I can do that if you'd like me to proceed.
q
@nutritious-balloon-65098 open a pr with the changes
n
I'm on it! Rohit Malhotra can continue tracking my progress here.
## Summary I've completed the task of adding
/settings
as a slash command that opens the settings modal. ### Checklist - ✅ Added
/settings
command to the
COMMANDS
list in
commands.py
- ✅ Added
/settings
to the help text shown by
/help
- ✅ Added command handler in
input_area.py
that calls
action_open_settings()
- ✅ Updated all relevant tests to include
/settings
- ✅
make lint
passes - ✅
make test
passes (1260 tests) - ✅ Changes pushed to remote branch - ✅ PR created ### Files Changed 1.
openhands_cli/tui/core/commands.py
- Added
/settings - Open settings
to COMMANDS list and help text 2.
openhands_cli/tui/widgets/input_area.py
- Added
case "settings":
handler and
_command_settings()
method 3.
tests/tui/test_commands.py
- Updated test expectations from 8 to 9 commands, added
/settings
to parametrized tests ### Pull Request PR #542: https://github.com/OpenHands/OpenHands-CLI/pull/542 The changes are minimal and focused solely on adding the
/settings
command as requested. Users can now type
/settings
(with autocomplete support) to open the settings modal instead of needing to discover Ctrl+P.
w
High-level, why have both the ctrl+p menu and slash commands? does it make sense to unify them?
q
Yeah we should We should try to nest commands the way codex does, to avoid flooding the slash command menu with too many items
👍 1
💯 1