-
Notifications
You must be signed in to change notification settings - Fork 26
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
Sometimes messages lose code #32
Comments
I can't reproduce locally or on chatcraft.org. I tried with both gpt-3.5 and gpt-4: I wonder if we're hitting react-syntax-highlighter/react-syntax-highlighter#513 upstream. |
yeah it's been hard to repro here too. |
Seems to only happen for me in a production build, which is why I've never hit it (I tend to play with this locally in development). I've confirmed that it's not coming from our code. When we hit it, we aren't passing down It's happening once we hand it off to the component here: https://github.com/tarasglek/chatcraft.org/blob/main/src/components/Markdown.tsx#L161 I'm trying to trace it through, to see if I can spot something. |
Linking your post to react-syntax-highlighter/react-syntax-highlighter#513 |
Unfortunately, the react-syntax-highlighter repo looks unmaintained to me. I cloned it and tried updating some things, but it's very out of date, and APIs have changed all over the place. Probably we should explore other options, but everything I've looked at today is missing pieces that this one has. I'm not sure that there is a good drop-in replacement, so we'll have to get it working again with something new. |
prismjs does seem to have an autoloader plugin. https://prismjs.com/plugins/autoloader/ |
No idea what causes this, guessing it's something to do with serialization/deserialization as it started happening after one of the recent refactors.
The text was updated successfully, but these errors were encountered: