-
Notifications
You must be signed in to change notification settings - Fork 3.2k
CLI: Missing smart_approve in /mode help text and error message #4097
Copy link
Copy link
Closed
Labels
good first issueGood for newcomersGood for newcomershelp wantedGreat issue for non-Block contributorsGreat issue for non-Block contributors
Description
Describe the bug
The CLI help text and error message for the /mode command are missing smart_approve as a valid option, even though the functionality is fully implemented and working.
To Reproduce
Steps to reproduce the behavior:
- Run
goose session - Type
/helpto see available commands - Observe the
/modecommand description shows only:('auto', 'approve', 'chat') - Try an invalid mode like
/mode invalid - See error message lists only:
auto, approve, chat
Expected behavior
Both the help text and error message should include smart_approve as a valid mode option, consistent with the actual functionality and tab completion.
Screenshots
N/A
Please provide following information:
- OS & Arch: Any
- Interface: CLI
- Version: Current main branch
- Extensions enabled: N/A
- Provider & Model: N/A
Additional context
The issue is in two locations:
crates/goose-cli/src/session/input.rsline ~245: Help text inprint_help()crates/goose-cli/src/session/mod.rs: Error message for invalid modes
Tab completion and validation logic already correctly support smart_approve.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
good first issueGood for newcomersGood for newcomershelp wantedGreat issue for non-Block contributorsGreat issue for non-Block contributors