-
Notifications
You must be signed in to change notification settings - Fork 16
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
Feature: Support for multiple messages from CodeLens #208
Conversation
val formattedMessages = messages.map { message -> | ||
if (message.role == "user") { | ||
message.copy( | ||
content = replaceVariablesInText(message.content, relativePath, cursor, text) |
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.
Are u sure about that? Is there situations with another roles and replacements of vars?
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.
There is an edge-case when image attached to UserMessage
, but CodeLens should not support images attachment in customization.yaml
(image is basically base64 string). In other cases, special vars should be present in user message since that is a part which is visible for the user in the chat
src/main/kotlin/com/smallcloud/refactai/code_lens/CodeLensAction.kt
Outdated
Show resolved
Hide resolved
… messages directly
Feature: Support for multiple messages from CodeLens
Description
customization.yaml
, and proper sending them to the frontend applicationType of change
Checklist