Uh oh!
There was an error while loading. Please reload this page.
Use inquirer prompt to select response mode - #172
Conversation
feat: use inquirer select prompt for response mode instead of text input
🔒 Security Scan Results
⏱️ SLA Breach Summary
✅ BUILD PASSED - All security checks passed |
There was a problem hiding this comment.
Pull request overview
This PR improves the interactive CLI UX by switching the “response mode” prompt from free-text input to an Inquirer list selection, ensuring users can only choose supported modes (buffered or streaming).
Changes:
- Replaced
input+ manual validation/normalization with alistprompt for response mode selection in both GitHub and FileUpload adapters. - Simplified streaming enablement logic to a direct equality check against the selected value.
- Updated adapter unit tests to assert the new prompt shape (
type: 'list',choices) and adjusted response-mode mapping cases accordingly.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/adapters/github.ts | Switches response mode prompt to a list selection and simplifies mapping to isStreamingEnabled. |
| src/adapters/github.test.ts | Updates tests to expect the list prompt and its choices; adjusts mapping/behavior assertions. |
| src/adapters/file-upload.ts | Switches response mode prompt to a list selection and simplifies mapping to isStreamingEnabled. |
| src/adapters/file-upload.test.ts | Updates tests to expect the list prompt and its choices; adjusts mapping/behavior assertions. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Uh oh!
There was an error while loading. Please reload this page.
feat: use inquirer select prompt for response mode instead of text input