fix: unreadable text in console and potentially over the max token #2050
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.
Background
In the process of
google
command, it convertgoogle_result
tosafe_message
byencode('utf-8')
, then return it by juststr()
. This will cause thegoogle_result
becomes a series of hex escaped string, it's hard to read in the process (seefig1
), and COULD cause exception due to the token size of hex-escaped string usually larger than unicode string (seefig2
).(
fig1
: the response of process is hard to read)(
fig2
: InvalidRequestError: token oversize)Changes
return unicode string (ex.:
宋江(演員)
) instead of hex-escaped sequence (ex.:\xe5\xae\×8b\xe6\xb1\x9f
)(
fig3
: expected output)Documentation
Test Plan
PR Quality Checklist