-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Suggest Edits should strip CDATA #24564
Comments
i dunno rust (yet) |
Hmm, what models are generating CDATA for you? |
Actually, the LLM using this tag might be a good thing. Right now, the suggested edits better not contain |
Summary
Suggest Edits
prompt uses XML. Some AI models insert CDATA notation. Zed sees the CDATA stuff as content. It shouldn'tWhen the AI suggests:
Zed tries to inject:
<![CDATA[THE NEW TEXT]]>
It should inject:
THE NEW TEXT
insteadZed Version and System Specs
Zed: v0.172.10 (Zed)
OS: macOS 15.1.0
Memory: 96 GiB
Architecture: aarch64
The text was updated successfully, but these errors were encountered: