patch: remove response_format for bedrock top level

[narengogi]

#1084

[matter-code-review]

Important

PR Review Skipped

PR review skipped as per the configuration setting. Run a manually review by commenting /matter review

💡Tips to use Matter AI

Command List

• /matter summary: Generate AI Summary for the PR
• /matter review: Generate AI Reviews for the PR
• /matter release-notes: Generate AI release-notes for the PR
• /matter <ask-question>: Chat with your PR with Matter AI Agent
• /matter remember <recommendation>: Generate AI memories for the PR

[matter-code-review]

Summary By MatterAI

🔄 What Changed

Removed top_k and response_format parameters from the BedrockConverseChatCompleteConfig object in the Bedrock provider implementation.

🔍 Impact of the Change

Simplifies the Bedrock chat completion configuration by removing parameters that were likely unnecessary or causing issues. This change streamlines the configuration object and potentially fixes compatibility issues with the Bedrock API.

📁 Total Files Changed

1 file changed with 0 additions and 10 deletions.

🧪 Test Added

N/A - No new tests were added in this PR. This change likely requires manual testing to verify that Bedrock chat completions still function correctly without these parameters.

🔒Security Vulnerabilities

N/A - No security vulnerabilities were introduced or addressed in this change.

Motivation

To fix compatibility issues or remove unused parameters in the Bedrock provider implementation.

Type of Change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Documentation update
• Refactoring (no functional changes)

How Has This Been Tested?

• Unit Tests
• Integration Tests
• Manual Testing

Screenshots (if applicable)

N/A

Checklist

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes

Related Issues

N/A

Sequence Diagram

sequenceDiagram
    participant Client
    participant BedrockConverseChatCompleteConfig
    participant transformAdditionalModelRequestFields
    
    Client->>BedrockConverseChatCompleteConfig: Request with parameters
    Note over BedrockConverseChatCompleteConfig: Before: Had top_k and response_format
    Note over BedrockConverseChatCompleteConfig: After: Removed top_k and response_format
    BedrockConverseChatCompleteConfig->>transformAdditionalModelRequestFields: Transform remaining parameters
    transformAdditionalModelRequestFields-->>BedrockConverseChatCompleteConfig: Return transformed parameters
    BedrockConverseChatCompleteConfig-->>Client: Return configured response

Loading