fix(bedrock): skip passing unsigned reasoning content#15181
Merged
Conversation
4 tasks
aayush-kapoor
added
the
backport
github-actions
Bot
removed
the
backport
Contributor
|
|
aayush-kapoor
added a commit
that referenced
this pull request
May 11, 2026
This is an automated backport of #15181 to the release-v6.0 branch. FYI @aayush-kapoor This backport has conflicts that need to be resolved manually. ### `git cherry-pick` output ``` Auto-merging packages/amazon-bedrock/src/convert-to-bedrock-chat-messages.test.ts CONFLICT (content): Merge conflict in packages/amazon-bedrock/src/convert-to-bedrock-chat-messages.test.ts Auto-merging packages/amazon-bedrock/src/convert-to-bedrock-chat-messages.ts CONFLICT (content): Merge conflict in packages/amazon-bedrock/src/convert-to-bedrock-chat-messages.ts error: could not apply bcbaae6... fix(bedrock): skip passing unsigned reasoning content (#15181) hint: After resolving the conflicts, mark them with hint: "git add/rm <pathspec>", then run hint: "git cherry-pick --continue". hint: You can instead skip this commit with "git cherry-pick --skip". hint: To abort and get back to the state before "git cherry-pick", hint: run "git cherry-pick --abort". hint: Disable this message with "git config set advice.mergeConflict false" ``` --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Aayush Kapoor <83492835+aayush-kapoor@users.noreply.github.com>
Contributor
|
🚀 Published in:
|
|
@aayush-kapoor Hey mate can you please use my email: pasqualecarmine.carbone@gmail.com Thanks |
Collaborator
Author
|
@KiraPC apologies that's the one i saw on github; noted for next time |
|
No problem at all. Maybe I used this in the commit, so it s my fault. Thanks! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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
#14720
passing unsigned reasoning content parts back to bedrock would sometimes lead to the model's thinking/reasoning tags leak into the normal response text.
the core issue was AI SDK passing the unsigned reasoning content back and this was introduced in #13972. The previous code only replayed reasoning when
reasoningMetadata != null, which effectively required a signature or redacted dataSummary
drop the reasoning content from being passed back to bedrock when it has no signature attached to it
Manual Verification
na
Checklist
pnpm changesetin the project root)Related Issues
fixes #14720
Co-authored-by: Pasquale Carmine Carbone pcarbonenearform@ipdanalytics.com