Skip to content

fix(aws-bedrock, microsoft-foundry): remove unsupported params and features flagged by live checks - #5755

Merged
bhaveshpatel640 merged 4 commits into
mainfrom
fix/model-yaml-unsupported-params
Oct 27, 2026
Merged

bhaveshpatel640 merged 4 commits into
mainfrom
fix/model-yaml-unsupported-params

Conversation

@bhaveshpatel640

@bhaveshpatel640 bhaveshpatel640 commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Fixes model YAMLs that the SDK live checks flagged and that still fail on main (confirmed in run 40497772917).

  • Bedrock gpt-5.6-luna, gpt-5.6-terra, gpt-6.1-sol, claude-opus-5-5, grok-4.6 (us. and global.): removeParams: [temperature, top_p]. responses/params and chat/params return 400 for these (same pattern as gpt-6.1-terra in fix(sdk-live-test): skip :batch variants, drop temperature for bedrock gpt-6.1-terra #5753).
  • Bedrock grok-4.6 (us. and global.): drop prompt_caching; the API returns 403 "did not allow prompt caching".
  • Microsoft Foundry claude-opus-4-7, claude-sonnet-4-5: add n to removeParams; chat/params returns 400 "n: Extra inputs are not permitted".

Not covered: OpenAI gpt-5.6/gpt-6 sampling params, Mistral assistant_prefill (needs prefix: true), Bedrock au./eu. Claude IDs, undeployed Foundry models, and account-side issues (credits, access, secrets).

Cross-check against LiteLLM and provider docs

Model Removed LiteLLM Provider docs Live test
Bedrock gpt-5.6-luna, gpt-5.6-terra, gpt-6.1-sol temperature, top_p sampling params not supported Not checked (LiteLLM already covers it) pass
Bedrock claude-opus-5-5 temperature, top_p sampling params not supported Not checked (LiteLLM already covers it) pass
Bedrock grok-4.6 temperature, top_p no data Not supported: Converse rejects temperature/topP with a ValidationException; send maxTokens only pass
Bedrock grok-4.6 prompt caching prompt caching not supported Not checked (LiteLLM already covers it) pass
Foundry claude-opus-4-7 temperature, top_p sampling params not supported Not checked (LiteLLM already covers it) pass
Foundry claude-opus-4-7, claude-sonnet-4-5 n no data Not supported: Foundry exposes the Claude Messages API, which has no n pass

"no data" means LiteLLM has no value for that setting; the provider docs were checked and also do not support it.

Test plan

  • yamllint and prettier --check pass on the 12 changed files
  • Live run on this branch: 40499443827, all scenarios, in progress

Made with Cursor


Note

Low Risk
Metadata-only changes to model catalog YAML; no runtime logic, but they affect which request parameters clients may omit when routing to these models.

Reviewed by Cursor Bugbot for commit 6012c97. Bugbot is set up for automated code reviews on this repo. Configure here.

bhaveshpatel640 and others added 3 commits October 27, 2026 17:22
…s and features flagged by live checks

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

github-actions Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

SDK live feature checks

Result: passed (414 scenarios)

Informational — does not block merge.

Model Pass Fail
aws-bedrock/global.anthropic.claude-opus-5-5 22 0
aws-bedrock/global.openai.gpt-5.6-luna 46 0
aws-bedrock/global.openai.gpt-5.6-terra 46 0
aws-bedrock/global.openai.gpt-6.1-sol 44 0
aws-bedrock/global.xai.grok-4.6 28 0
aws-bedrock/us.anthropic.claude-opus-5-5 22 0
aws-bedrock/us.openai.gpt-5.6-luna 46 0
aws-bedrock/us.openai.gpt-5.6-terra 46 0
aws-bedrock/us.openai.gpt-6.1-sol 44 0
aws-bedrock/us.xai.grok-4.6 28 0
microsoft-foundry/claude-opus-4-7 24 0
microsoft-foundry/claude-sonnet-4-5 18 0

Co-authored-by: Cursor <cursoragent@cursor.com>
@bhaveshpatel640 bhaveshpatel640 changed the title fix(aws-bedrock, openai, microsoft-foundry): remove unsupported params and features flagged by live checks fix(aws-bedrock, microsoft-foundry): remove unsupported params and features flagged by live checks Oct 27, 2026
@bhaveshpatel640
bhaveshpatel640 merged commit e460176 into main Oct 27, 2026
5 checks passed
@bhaveshpatel640
bhaveshpatel640 deleted the fix/model-yaml-unsupported-params branch October 27, 2026 12:51
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