Skip to content

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

Merged
chhavi-mandowara-cstk merged 11 commits into
mainfrom
staging
Jun 9, 2026
Merged

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

Conversation

@chhavi-mandowara-cstk

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

Copy link
Copy Markdown
Contributor

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-CSand others added 5 commits May 27, 2026 15:56
@github-actions

Copy link
Copy Markdown

🔒 Security Scan Results

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

Check TypeCount (with fixes)Without fixesThresholdResult
🔴 Critical Severity0010✅ Passed
🟠 High Severity0025✅ Passed
🟡 Medium Severity00500✅ Passed
🔵 Low Severity001000✅ Passed

⏱️ SLA Breach Summary

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

SeverityBreaches (with fixes)Breaches (no fixes)SLA Threshold (with/no fixes)Status
🔴 Critical0015 / 30 days✅ Passed
🟠 High0030 / 120 days✅ Passed
🟡 Medium0090 / 365 days✅ Passed
🔵 Low00180 / 365 days✅ Passed

✅ BUILD PASSED - All security checks passed

CopilotAI left a comment

Copy link
Copy Markdown

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
FileDescription
test/unit/util/common-utility.test.tsAdds Mocha/Chai/Sinon unit tests for getAnalyticsInfo() reconstruction behavior.
src/util/common-utility.tsIntroduces getAnalyticsInfo() to rebuild X-CS-CLI from oclif config + config store values.
src/commands/launch/index.tsPasses reconstructed analytics info into adapter/precheck inputs.
src/base-command.tsEnsures GraphQL headers and CMA SDK init receive a populated X-CS-CLI value at runtime.
src/adapters/github.tsSwitches response-mode prompting to a validated text input and normalizes values.
src/adapters/github.test.tsUpdates tests to match the new response-mode input prompt and adds mapping/validation tests.
src/adapters/file-upload.tsMirrors the GitHub adapter response-mode input prompt behavior.
src/adapters/file-upload.test.tsUpdates tests to match the new prompt and adds mapping/validation tests.
package.jsonVersion bump to 1.10.1.
package-lock.jsonUpdates resolved dependency versions and lockfile metadata.
.talismanrcUpdates 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

🔒 Security Scan Results

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

Check TypeCount (with fixes)Without fixesThresholdResult
🔴 Critical Severity0010✅ Passed
🟠 High Severity0025✅ Passed
🟡 Medium Severity00500✅ Passed
🔵 Low Severity001000✅ Passed

⏱️ SLA Breach Summary

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

SeverityBreaches (with fixes)Breaches (no fixes)SLA Threshold (with/no fixes)Status
🔴 Critical0015 / 30 days✅ Passed
🟠 High0030 / 120 days✅ Passed
🟡 Medium0090 / 365 days✅ Passed
🔵 Low00180 / 365 days✅ Passed

✅ BUILD PASSED - All security checks passed

@chhavi-mandowara-cstkchhavi-mandowara-cstk changed the title Send X-CS-CLI header and Enhance prompt for Response modeSend X-CS-CLI header and allow selection for Response mode via inquirerJun 3, 2026
@chhavi-mandowara-cstkchhavi-mandowara-cstk changed the title Send X-CS-CLI header and allow selection for Response mode via inquirerSend X-CS-CLI header and allow selection for Response mode via search-listJun 3, 2026
@github-actions

Copy link
Copy Markdown

🔒 Security Scan Results

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

Check TypeCount (with fixes)Without fixesThresholdResult
🔴 Critical Severity0010✅ Passed
🟠 High Severity0025✅ Passed
🟡 Medium Severity00500✅ Passed
🔵 Low Severity001000✅ Passed

⏱️ SLA Breach Summary

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

SeverityBreaches (with fixes)Breaches (no fixes)SLA Threshold (with/no fixes)Status
🔴 Critical0015 / 30 days✅ Passed
🟠 High0030 / 120 days✅ Passed
🟡 Medium0090 / 365 days✅ Passed
🔵 Low00180 / 365 days✅ Passed

✅ BUILD PASSED - All security checks passed

@chhavi-mandowara-cstk
chhavi-mandowara-cstk merged commit d7d0f9a into mainJun 9, 2026
6 checks passed
Sign up for freeto 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.

7 participants

@chhavi-mandowara-cstk@dhruvparekh12@sanikapareek-contentstack@naeem-shakh-cs@AryanBansal-launch@Harshi-Shah-CS