Skip to content

feat: implement interactive model picker for CLI chat#58

Merged
cidrblock merged 4 commits into
mainfrom
AAP-79615-implement-interactive-model-picker-for-aby-chat
Jun 22, 2026
Merged

feat: implement interactive model picker for CLI chat#58
cidrblock merged 4 commits into
mainfrom
AAP-79615-implement-interactive-model-picker-for-aby-chat

Conversation

@sathyapramod

Copy link
Copy Markdown
Contributor
  • Updated the CLI chat command to allow model selection via an interactive picker when the --model option is omitted.
  • Added selectModel function to manage model selection and daemon state.
  • Introduced promptModelPicker for displaying available models and handling user input.
  • Created tests for model selection and input parsing to ensure functionality and error handling.
  • Updated documentation to reflect changes in command usage.

Ref : https://redhat.atlassian.net/browse/AAP-79615

- Updated the CLI chat command to allow model selection via an interactive picker when the --model option is omitted.
- Added `selectModel` function to manage model selection and daemon state.
- Introduced `promptModelPicker` for displaying available models and handling user input.
- Created tests for model selection and input parsing to ensure functionality and error handling.
- Updated documentation to reflect changes in command usage.
- Changed the mock implementation of DaemonState in select-model tests to use a function instead of an object for better clarity and consistency.
- Ensured that the listModels method returns a resolved promise with an empty array or predefined models as needed for various test cases.
- Updated the `selectModel` function to return both the selected model and the associated DaemonState, allowing for reuse without restarting the daemon.
- Modified the CLI chat command to accept the new state parameter from the model picker.
- Adjusted tests to verify that the model and state are correctly returned and utilized.
@sathyapramod sathyapramod requested a review from cidrblock June 22, 2026 13:27
@cidrblock cidrblock merged commit d255a13 into main Jun 22, 2026
6 checks passed
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