Skip to content

feat: use inquirer select prompt for response mode instead of text input#171

Merged
chhavi-mandowara-cstk merged 1 commit into
developmentfrom
CL-4136
Jun 3, 2026
Merged

feat: use inquirer select prompt for response mode instead of text input#171
chhavi-mandowara-cstk merged 1 commit into
developmentfrom
CL-4136

Conversation

@chhavi-mandowara-cstk
Copy link
Copy Markdown
Contributor

No description provided.

@chhavi-mandowara-cstk chhavi-mandowara-cstk requested review from a team as code owners June 3, 2026 06:38
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 3, 2026

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check Type Count (with fixes) Without fixes Threshold Result
🔴 Critical Severity 0 0 10 ✅ Passed
🟠 High Severity 0 0 25 ✅ Passed
🟡 Medium Severity 0 0 500 ✅ Passed
🔵 Low Severity 0 0 1000 ✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

Severity Breaches (with fixes) Breaches (no fixes) SLA Threshold (with/no fixes) Status
🔴 Critical 0 0 15 / 30 days ✅ Passed
🟠 High 0 0 30 / 120 days ✅ Passed
🟡 Medium 0 0 90 / 365 days ✅ Passed
🔵 Low 0 0 180 / 365 days ✅ Passed

✅ BUILD PASSED - All security checks passed

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 a list prompt with explicit Buffered / Streaming choices in both GitHub and FileUpload adapters.
  • Simplified response mode handling logic by removing manual normalization/validation for s/b shortcuts.
  • 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.

@chhavi-mandowara-cstk chhavi-mandowara-cstk merged commit 704d876 into development Jun 3, 2026
7 checks passed
@chhavi-mandowara-cstk chhavi-mandowara-cstk deleted the CL-4136 branch June 3, 2026 06:44
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.

4 participants