Skip to content

fix(opencode): remove Qwen sampling defaults - #43310

Merged
rekram1-node merged 1 commit into
devfrom
remove-qwen-defaults
Aug 18, 2026
Merged

fix(opencode): remove Qwen sampling defaults#43310
rekram1-node merged 1 commit into
devfrom
remove-qwen-defaults

Conversation

@opencode-agent

Copy link
Copy Markdown
Contributor

Summary

  • stop forcing temperature: 0.55 and top_p: 1 for every Qwen model
  • leave Qwen sampling unset so provider or server defaults apply, while chat.params plugins can still override all sampling controls
  • add regression coverage for Qwen3.8 and Qwen3-Coder model IDs

Why

Qwen sampling recommendations now vary by model family and thinking mode, so the blanket name-based defaults are stale and can override better provider-side settings, especially for custom and local endpoints.

Fixes #42775

Testing

  • bun test test/provider/transform.test.ts (411 pass)
  • Prettier check on changed files
  • git diff --check
  • bun typecheck attempted, but tsgo was terminated by shared-host resource exhaustion

Requested by: @rekram1-node (Aiden via Slack)

@rekram1-node
rekram1-node merged commit 8b65fa2 into dev Aug 18, 2026
11 checks passed
@rekram1-node
rekram1-node deleted the remove-qwen-defaults branch August 18, 2026 23:00
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.

[BUG]: (Incorrect) sampling parameters are hard-coded based on model name/id

1 participant