fix: correct linewrap think display in generic openai api #13260
+26
−440
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Follow up #13234.
I've got keys from deepseek and siliconflow. Thus I tested and removed duplicated code.
I also found two corner cases and one bug:
\n\n..\n
may be there. That said you can't just replace\n\n
but also\n
in the same line. I made a regexp replace.reasoning_content
andcontent
may be empty sometimes in the thinking phase. the current code will terminate the reasoning phase, append> think_icon
again.delta_content
is not assigned correctly in the generic openai api module.Since official deepseek api does not respond after fixing
problem 2
, I don't have screen shots for it.Screenshots
Checklist
Important
Please review the checklist below before submitting your pull request.
dev/reformat
(backend) andcd web && npx lint-staged
(frontend) to appease the lint gods