Skip to content

fix(opencode): request summaries in Bedrock GPT-5 variants - #46501

Open
rekram1-node wants to merge 3 commits into
devfrom
bedrock-reasoning
Open

fix(opencode): request summaries in Bedrock GPT-5 variants#46501
rekram1-node wants to merge 3 commits into
devfrom
bedrock-reasoning

Conversation

@rekram1-node

@rekram1-node rekram1-node commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add additionalModelRequestFields.reasoning.summary: "auto" to Bedrock Converse GPT-5 variants, in both catalog-driven and fallback variant generation.
  • Mantle already sets reasoningSummary: "auto"; leave it unchanged.
  • Leave effort mapping, defaults, configuration merging, and request preparation unchanged.

This replaces the earlier, broader implementation. The final diff contains only the variant summary setting and two focused regression tests.

Verification

  • Both added tests failed without the summary setting and pass with it.
  • Provider transform and Bedrock suites: 448 passed, 0 failed.
  • Package typecheck, Prettier check, and git diff --check passed.

This requests summaries; it does not guarantee AWS returns readable summary text or address the separately identified effort-serialization issues.

@rekram1-node rekram1-node changed the title fix(opencode): normalize Bedrock effort and request summaries fix(opencode): normalize Bedrock reasoning defaults and variants Sep 1, 2026
@rekram1-node rekram1-node changed the title fix(opencode): normalize Bedrock reasoning defaults and variants fix(opencode): request summaries in Bedrock GPT-5 variants Sep 1, 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.

1 participant