Uh oh!
There was an error while loading. Please reload this page.
Send X-CS-CLI header and Enhance prompt for Response mode - #169
Conversation
fix: send populated X-CS-CLI header from launch commands at runtime
feat: prompt for streaming/buffered response mode via 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 ensures the X-CS-CLI header is reliably populated at runtime (even when context.analyticsInfo isn’t available) and improves the interactive UX for choosing buffered vs streaming response mode.
Changes:
- Added
getAnalyticsInfo()helper to reconstruct analytics info from oclif config + persisted config store values, and used it to populateX-CS-CLIheaders and adapter inputs. - Updated GitHub and FileUpload adapters to prompt for response mode via a validated text input (
s/borstreaming/buffered) instead of a confirm prompt. - Added/updated unit tests for analytics reconstruction and response-mode prompting behavior; bumped package version to
1.10.1and refreshed lockfile.
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 unit coverage for getAnalyticsInfo() reconstruction and fallbacks. |
src/util/common-utility.ts | Introduces getAnalyticsInfo() and exports it for use by commands/base command. |
src/commands/launch/index.ts | Passes reconstructed analytics info into adapters and precheck inputs. |
src/base-command.ts | Ensures GraphQL headers and CMA SDK init use reconstructed X-CS-CLI value. |
src/adapters/github.ts | Replaces confirm prompt with validated input prompt for response mode selection. |
src/adapters/github.test.ts | Updates tests to match new response mode prompt + adds mapping/validation tests. |
src/adapters/file-upload.ts | Replaces confirm prompt with validated input prompt for response mode selection. |
src/adapters/file-upload.test.ts | Updates tests to match new response mode prompt + adds mapping/validation tests. |
package.json | Bumps package version to 1.10.1. |
package-lock.json | Updates lockfile for version bump and dependency refresh. |
.talismanrc | Updates checksum to reflect the new package-lock.json. |
💡 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.
fix: send populated X-CS-CLI header from launch commands at runtime
feat: prompt for streaming/buffered response mode via text input