Skip to content

Send X-CS-CLI header and allow selection for Response mode via search-list#170

Open
chhavi-mandowara-cstk wants to merge 11 commits into
mainfrom
staging
Open

Send X-CS-CLI header and allow selection for Response mode via search-list#170
chhavi-mandowara-cstk wants to merge 11 commits into
mainfrom
staging

Conversation

@chhavi-mandowara-cstk
Copy link
Copy Markdown
Contributor

@chhavi-mandowara-cstk chhavi-mandowara-cstk commented Jun 2, 2026

fix: send populated X-CS-CLI header from launch commands at runtime
feat: prompt for streaming/buffered response mode via text input
feat: use inquirer select prompt for response mode instead of text input[
feat: use inquirer select prompt for response mode instead of text input

Harshi-Shah-CS and others added 5 commits May 27, 2026 15:56
fix: send populated X-CS-CLI header from launch commands at runtime
feat: prompt for streaming/buffered response mode via text input
Send X-CS-CLI header and Enhance prompt for Response mode
@chhavi-mandowara-cstk chhavi-mandowara-cstk requested review from a team as code owners June 2, 2026 18:19
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 2, 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 improves runtime analytics/header propagation for Launch commands by reconstructing the X-CS-CLI value when context.analyticsInfo is unavailable, and updates the interactive flow to choose streaming vs buffered response mode via text input.

Changes:

  • Add getAnalyticsInfo() helper to reconstruct X-CS-CLI and use it in BaseCommand and Launch adapter construction.
  • Replace the streaming confirm prompt with an input-based “Response Mode” prompt (GitHub + FileUpload adapters), and expand unit tests for mapping/validation.
  • Bump package version to 1.10.1 (and update lockfile/talisman checksum).

Reviewed changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated no comments.

Show a summary per file
File Description
test/unit/util/common-utility.test.ts Adds Mocha/Chai/Sinon unit tests for getAnalyticsInfo() reconstruction behavior.
src/util/common-utility.ts Introduces getAnalyticsInfo() to rebuild X-CS-CLI from oclif config + config store values.
src/commands/launch/index.ts Passes reconstructed analytics info into adapter/precheck inputs.
src/base-command.ts Ensures GraphQL headers and CMA SDK init receive a populated X-CS-CLI value at runtime.
src/adapters/github.ts Switches response-mode prompting to a validated text input and normalizes values.
src/adapters/github.test.ts Updates tests to match the new response-mode input prompt and adds mapping/validation tests.
src/adapters/file-upload.ts Mirrors the GitHub adapter response-mode input prompt behavior.
src/adapters/file-upload.test.ts Updates tests to match the new prompt and adds mapping/validation tests.
package.json Version bump to 1.10.1.
package-lock.json Updates resolved dependency versions and lockfile metadata.
.talismanrc Updates checksum for the modified lockfile.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@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

@chhavi-mandowara-cstk chhavi-mandowara-cstk changed the title Send X-CS-CLI header and Enhance prompt for Response mode Send X-CS-CLI header and allow selection for Response mode via inquirer Jun 3, 2026
feat: use search-list prompt for response mode instead of list
Enhance prompt by using search-list for Response Mode
@chhavi-mandowara-cstk chhavi-mandowara-cstk changed the title Send X-CS-CLI header and allow selection for Response mode via inquirer Send X-CS-CLI header and allow selection for Response mode via search-list Jun 3, 2026
@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

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.

3 participants