Skip to content

Conversation

@EurFelux
Copy link
Collaborator

What this PR does

Move responseBody display to appear before requestBodyValues for better error flow readability

Fixes #

Why we need it and why it was done in this way

The following tradeoffs were made:

The following alternatives were considered:

Links to places where the discussion took place:

Breaking changes

If this PR introduces breaking changes, please describe the changes and the impact on users.

Special notes for your reviewer

Checklist

This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

Release note


Move responseBody display to appear before requestBodyValues for better error flow readability
@DeJeune
Copy link
Collaborator

DeJeune commented Oct 31, 2025

Note

This issue/comment/review was translated by Claude.

The formatErrorMessage(chunk.error) method cannot extract detailed messages. You can refer to the errors returned in the cherryin stream and optimize it by the way.


Original Content formatErrorMessage(chunk.error)这个方法没办法提取到详细的message,可以参考一下cherryin流中返回的错误,要不顺便优化一下
@EurFelux
Copy link
Collaborator Author

EurFelux commented Oct 31, 2025

Note

This issue/comment/review was translated by Claude.

The formatErrorMessage function itself is not very useful, it can be replaced.


Original Content

formatErrorMessage这个函数本身就不是很好用,可以换掉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants