Skip to content
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

fix(aws): fix Invalid reasoning content exception on empty reasoning chunk #7826

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ChenNima
Copy link

@ChenNima ChenNima commented Mar 11, 2025

When invoking the DeeSeek R1 model on Bedrock, there exists a possibility that the model may generate reasoningContent with an empty text string.

{
  "reasoningContent": {
    "text": ""
  }
}

Modify the code to prevent the Invalid reasoning content exception from being thrown in this scenario.

Copy link

vercel bot commented Mar 11, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
langchainjs-docs ✅ Ready (Inspect) Visit Preview Mar 11, 2025 10:58am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
langchainjs-api-refs ⬜️ Ignored (Inspect) Mar 11, 2025 10:58am

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. auto:bug Related to a bug, vulnerability, unexpected error with an existing feature labels Mar 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto:bug Related to a bug, vulnerability, unexpected error with an existing feature size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant