[Leela Mahalakshmi (Leela0o5)]

Motivation

BedrockChatClient was not handling reasoningContent properly. Because of this, reasoning was not shown, and tool calls with extended thinking could fail because the required reasoning signature was missing.

Changes

• Convert Bedrock reasoningContent to text_reasoning.
• Preserve the reasoning signature in protected_data.
• Convert text_reasoning back to reasoningContent in requests.
• Handle reasoning updates from ConverseStream.
• Follow the approach used by the AI client.

Impact

Reasoning is now available to callers, and extended thinking with tool calls should work correctly. Non-reasoning requests are unchanged.

Testing

• Added unit tests using stubbed Bedrock responses.
• I have not tested with live Bedrock yet.
• redactedContent is still not handled.

Related Issue

Fixes #8935

Checklist

• Builds without errors or warnings
• Unit tests pass
• Added tests
• No breaking changes

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[Eduard van Valkenburg (eavanvalkenburg)]

/review

[github-actions]

MAF Automated Review — Iteration 1

Result: Findings reported
Scope: full PR (1 commit(s)): 6b49c10fe5ea
Model: gpt-5.6-sol

Overview

The change maps Bedrock reasoning text and signatures into text_reasoning, preserves their order with tool calls, and replays signed reasoning only from assistant messages. The new tests cover non-streaming parsing, single-block streaming aggregation, replay shape, and role gating. Streaming still loses reasoning-block boundaries when a response contains consecutive reasoning blocks, which can pair modified text with the wrong signature during tool-result continuation.

Reviewed the supplied pull-request change set across correctness, security/reliability, architecture, and failure behavior.
1 verified finding remained after source verification (1 medium) across 1 file. Details are attached to the affected lines below.

Affected areas: python/packages/bedrock/agent_framework_bedrock/_chat_client.py