Skip to content

fix: Bedrock model config update#4

Merged
lgarceau768 merged 2 commits intoflexfrom
bedrock-model-config-update
Apr 21, 2026
Merged

fix: Bedrock model config update#4
lgarceau768 merged 2 commits intoflexfrom
bedrock-model-config-update

Conversation

@sfradkin
Copy link
Copy Markdown

@sfradkin sfradkin commented Apr 20, 2026

Issue for this PR

Closes #

Type of change

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

What does this PR do?

This PR fixes our ability to use OpenCode with the following Bedrock models:

  • Writer Palmyra X4
  • Writer Palmyra X5
  • DeepSeek R1 (A)
  • Mistral Pixtral Large
  • Meta Llama 4 Maverick 17B
  • Meta Llama 4 Scout 17B
  • Amazon Nova 2 Lite

This was done through a combination of some small code fixes plus configuration fixes (in LOCAL_AWS_SETUP.md).

How did you verify your code works?

I tested out the models with a simple prompt.

Screenshots / recordings

N/A

Checklist

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

…ditional models

- Add palmyra and pixtral to US cross-region inference profile prefix list
- Strip reasoning content from message history for non-reasoning models
- Exclude palmyra from reasoning variant generation to prevent unsupported params
@github-actions
Copy link
Copy Markdown

Hey! Your PR title Bedrock model config update 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.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 20, 2026

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.

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.

@sfradkin sfradkin changed the title Bedrock model config update fix: Bedrock model config update Apr 20, 2026
@github-actions
Copy link
Copy Markdown

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@lgarceau768 lgarceau768 merged commit 5e3eb0f into flex Apr 21, 2026
9 of 14 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.

2 participants