Skip to content

fix(bedrock): forward default model from provider - #2447

Closed
ShiroKSH wants to merge 1 commit into
567-labs:mainfrom
ShiroKSH:fix/forward-bedrock-default-model
Closed

fix(bedrock): forward default model from provider#2447
ShiroKSH wants to merge 1 commit into
567-labs:mainfrom
ShiroKSH:fix/forward-bedrock-default-model

Conversation

@ShiroKSH

Copy link
Copy Markdown

Summary

  • Forward the model parsed by the Bedrock auto-client to the Bedrock client factory.
  • Add a regression assertion for the forwarded default model.

Root cause

The Bedrock auto-client selected a mode from the model string but never passed that string as the factory's default model. Calls that omitted model therefore reached Bedrock without the required modelId.

Validation

  • Targeted auto-client and Bedrock coverage tests: 31 passed.
  • Ruff check for the two changed files passed.

@jxnl

jxnl commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

Consolidated and shipped in #2495. Closing this focused patch as superseded; thank you for the contribution.

@jxnl jxnl closed this Jul 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants