Skip to content

Fix/bedrock mediaTool support#16

Merged
lgarceau768 merged 2 commits intoflexfrom
fix/bedrock-pdf-support
Apr 28, 2026
Merged

Fix/bedrock mediaTool support#16
lgarceau768 merged 2 commits intoflexfrom
fix/bedrock-pdf-support

Conversation

@lgarceau768
Copy link
Copy Markdown
Collaborator

Issue for this PR

Closes #

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

Corrects an error with not allowing models from the bedrock package to leverage Media Tools, which was made in an effort to support more bedrock models, but degraded performance with reliable Anthropic Models.

If you paste a large clearly AI generated description here your PR may be IGNORED or CLOSED!

How did you verify your code works?

Screenshots / recordings

If this is a UI change, please include a screenshot or recording.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

If you do not follow this template your PR will be automatically rejected.

AWS Bedrock does not support the pdfs-2024-09-25 beta feature even for
Claude models. When supportsMediaInToolResults=true, PDFs from tool
results are sent directly with the beta flag, which Bedrock rejects.

This change sets supportsMediaInToolResults=false for Bedrock, causing
PDFs/images to be extracted and sent as synthetic user messages instead.
@lgarceau768 lgarceau768 self-assigned this Apr 27, 2026
@lgarceau768 lgarceau768 changed the title Fix/bedrock pdf support Fix/bedrock mediaTool support Apr 27, 2026
@github-actions
Copy link
Copy Markdown

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

  • No issue referenced. Please add Closes #<number> linking to the relevant issue.
  • "How did you verify your code works?" section is empty. Please explain how you tested.

Please edit this PR description to address the above within 2 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

@github-actions
Copy link
Copy Markdown

Hey! Your PR title Fix/bedrock pdf support doesn't follow conventional commit format.

Please update it to start with one of:

  • feat: or feat(scope): new feature
  • fix: or fix(scope): bug fix
  • docs: or docs(scope): documentation changes
  • chore: or chore(scope): maintenance tasks
  • refactor: or refactor(scope): code refactoring
  • test: or test(scope): adding or updating tests

Where scope is the package name (e.g., app, desktop, opencode).

See CONTRIBUTING.md for details.

@lgarceau768 lgarceau768 merged commit 0db1e6a into flex Apr 28, 2026
9 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant