-
Notifications
You must be signed in to change notification settings - Fork 3.7k
fix(cli): unskip first slash command output #8482
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(cli): unskip first slash command output #8482
Conversation
| // TODO: Properly separate LLM system messages from UI informational messages | ||
| // using discriminated union types. For now, skip displaying the first system | ||
| // message which is typically the LLM's system prompt. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this does not seem to cause any problem as tested with baseSystemMessage set
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No issues found across 1 file
|
🎉 This PR is included in version 1.31.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 1.4.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 1.28.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Description
The first slash command in the cli is skipped due to the if condition. This PR fixes it.
AI Code Review
@continue-reviewChecklist
Screen recording or screenshot
before.mp4
after.mp4
Tests
[ What tests were added or updated to ensure the changes work as expected? ]
Summary by cubic
Shows the first slash command output in the CLI instead of skipping it. Removes the early return that hid the first system message in MemoizedMessage.tsx.