Uh oh!
There was an error while loading. Please reload this page.
feat: use inquirer select prompt for response mode instead of text input - #171
Conversation
🔒 Security Scan Results
⏱️ SLA Breach Summary
✅ BUILD PASSED - All security checks passed |
There was a problem hiding this comment.
Pull request overview
This PR updates the interactive “response mode” selection in the Launch CLI adapters to use a discrete selection prompt (instead of free-form text input), aligning the prompt UX with the existing buffered|streaming flag options.
Changes:
- Replaced the Response Mode prompt from
input+ validation to alistprompt with explicitBuffered/Streamingchoices in both GitHub and FileUpload adapters. - Simplified response mode handling logic by removing manual normalization/validation for
s/bshortcuts. - Updated Jest tests to assert the new prompt shape (
type: 'list',choices) and adjusted parametrized 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 maps selection to isStreamingEnabled. |
| src/adapters/github.test.ts | Updates prompt assertions and selection-mapping tests for the new list prompt. |
| src/adapters/file-upload.ts | Switches Response Mode prompt to a list selection and maps selection to isStreamingEnabled. |
| src/adapters/file-upload.test.ts | Updates prompt assertions and selection-mapping tests for the new list prompt. |
💡 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.
No description provided.