Algorithm5838

I found a solution: https://github.com/ChatGPTNextWeb/ChatGPT-Next-Web/pull/4354 I tested it with different examples, and it worked.

I came up with it with the help of LLMs. Here is an explanation: 1. Ensure that the dollar sign (`$`) is not preceded by a backtick (`` ` ``)....

I noticed an issue with the regex and fixed it. It is now: ``` /(?

It is now: ``` /(?

Update: I went about it the wrong. The new PR is the one to use, it has a better and short regex, covering all cases. https://github.com/ChatGPTNextWeb/ChatGPT-Next-Web/pull/4363 ``` /(?

Unfortunately, there are still some issues: 1. If you did not use the `Inject System Prompt`, the issues will persist, as the LLM might still use single dollar signs for...

@daiaji Did you try my workaround? If so, how did you find it?

Per my understanding, the original is correct; it is for the gpt-3.5-* models.

You are correct. I encountered it before and solved it by commenting out this part: ``` i >= contextStartIndex;// && tokenCount < maxTokenThreshold; ``` The issue with the logic is...

Unless the maintainers come up with a solution, I recommend you use my workaround I made in this fork: https://github.com/Algorithm5838/NextChat/tree/dollar-sign You can try it here: https://nextchat-git-dollar-sign-algorithm5838s-projects.vercel.app/