-
-
Notifications
You must be signed in to change notification settings - Fork 23.2k
Fixed: Variable syntax highlighting lost after copy-paste from external editor in Agentflow V2 #5513
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
Fixed: Variable syntax highlighting lost after copy-paste from external editor in Agentflow V2 #5513
Conversation
Summary of ChangesHello @DebottamMandal123, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request addresses a critical issue in Agentflow V2 where variable syntax highlighting was lost when content containing variables like Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
Summary of ChangesHello @DebottamMandal123, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request resolves an issue in Agentflow V2 where variable syntax highlighting was lost when content containing variables, such as Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
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.
Code Review
This pull request effectively resolves the issue of variable syntax highlighting being lost on copy-paste by creating a custom Tiptap Mention extension. The implementation with renderText and addPasteRules is well done. However, the new CustomMention extension is duplicated in both ExpandRichInputDialog.jsx and RichInput.jsx. To improve maintainability and adhere to the DRY (Don't Repeat Yourself) principle, this duplicated code should be extracted into a single, reusable module.
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.
Code Review
This pull request introduces a CustomMention Tiptap extension to fix an issue with copy-pasting variables. The approach of using renderText and addPasteRules is correct and effectively solves the problem.
My main feedback is regarding code duplication. The CustomMention component is defined identically in two separate files (ExpandRichInputDialog.jsx and RichInput.jsx). I've left comments suggesting to extract this logic into a shared module to improve maintainability. Addressing this will make the codebase cleaner and easier to manage in the long run.
HenryHengZJ
left a comment
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.
wow this is awesome thank you!
Added a custom PasteRule and renderText override to the Tiptap Mention extension. It ensures variables like {{variable}} are correctly converted into Mention nodes on paste and copied with the correct closing braces.