Skip to content
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/codelens support multiple messages #169

Merged
merged 4 commits into from
Jan 7, 2025

Conversation

alashchev17
Copy link
Collaborator

@alashchev17 alashchev17 commented Jan 6, 2025

Relies on #275 pull request of refact-chat-js

@alashchev17 alashchev17 changed the base branch from main to dev January 6, 2025 12:12
@MarcMcIntosh
Copy link
Contributor

Looks like there's a couple of things going wrong here :/

Screenshot 2025-01-06 at 14 28 33

%PROMPT_EXPLORATION_TOOLS% isn't being replaced and the markdown isn't being detected :/

@MarcMcIntosh
Copy link
Contributor

It looks like the chat is being called twice so there's two streams

@alashchev17
Copy link
Collaborator Author

alashchev17 commented Jan 6, 2025

%PROMPT_EXPLORATION_TOOLS%

this one is tricky, needs a bit of reconsideration on how to get it or maybe this one is getting parsed in LSP binary

@alashchev17 alashchev17 self-assigned this Jan 6, 2025
@alashchev17 alashchev17 marked this pull request as ready for review January 7, 2025 17:13
if (('type' in content) && content.type === 'text') {
return replaceVariablesInText(content.text, relative_path, cursor, text);
}
}).join("\n");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will join concat all the objects to a string ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

only if there's images, I don't think code lens will be adding images :)

if (('type' in content) && content.type === 'text') {
return replaceVariablesInText(content.text, relative_path, cursor, text);
}
}).join("\n");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

only if there's images, I don't think code lens will be adding images :)

@alashchev17 alashchev17 merged commit 36ce9e7 into dev Jan 7, 2025
6 checks passed
@alashchev17 alashchev17 deleted the fix/codelens-support-multiple-messages branch January 7, 2025 17:26
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.

2 participants