Skip to content

feat(opencode): Add Venice support in temperature, topP, topK and smallOption #13553

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
rekram1-node merged 3 commits into from
Feb 16, 2026
Merged

feat(opencode): Add Venice support in temperature, topP, topK and smallOption #13553

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
2b02e2b
[provider] add Venice support in temperature, topP, topK and smallOpt…
dpuyosa Feb 14, 2026
62a0c18
Run prettier & fix inconsistency with model name
dpuyosa Feb 14, 2026
62c8917
Resolve minor review suggestions
dpuyosa Feb 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 10 additions & 5 deletions packages/opencode/src/provider/transform.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -297,9 +297,9 @@ export namespace ProviderTransform {
if (id.includes("glm-4.7")) return 1.0
if (id.includes("minimax-m2")) return 1.0
if (id.includes("kimi-k2")) {
// kimi-k2-thinking & kimi-k2.5 && kimi-k2p5
if (id.includes("thinking") || id.includes("k2.") || id.includes("k2p")) {
return 1.0
// kimi-k2-thinking & kimi-k2.5 && kimi-k2p5 && kimi-k2-5
if (["thinking", "k2.", "k2p", "k2-"].some(s => id.includes(s))) {
Comment thread
dpuyosa marked this conversation as resolved.
Outdated
return 1.0;
Comment thread
dpuyosa marked this conversation as resolved.
Outdated
}
return 0.6
}
Expand All @@ -309,7 +309,7 @@ export namespace ProviderTransform {
export function topP(model: Provider.Model) {
const id = model.id.toLowerCase()
if (id.includes("qwen")) return 1
if (id.includes("minimax-m2") || id.includes("kimi-k2.5") || id.includes("kimi-k2p5") || id.includes("gemini")) {
if (["minimax-m2", "gemini", "kimi-k2.5", "kimi-k2p5", "kimi-k2-5"].some(s => id.includes(s))) {
return 0.95
}
return undefined
Expand All @@ -318,7 +318,7 @@ export namespace ProviderTransform {
export function topK(model: Provider.Model) {
const id = model.id.toLowerCase()
if (id.includes("minimax-m2")) {
if (id.includes("m2.1")) return 40
if (id.includes("m2.") || id.includes("m25") || id.includes("m21")) return 40 // m2.1 and m2.5
Comment thread
rekram1-node marked this conversation as resolved.
Outdated
Comment thread
dpuyosa marked this conversation as resolved.
Outdated
return 20
}
if (id.includes("gemini")) return 64
Expand Down Expand Up @@ -750,6 +750,11 @@ export namespace ProviderTransform {
}
return { reasoningEffort: "minimal" }
}

if (model.providerID === "venice") {
return { veniceParameters: { disableThinking: true } }
}

return {}
}

Expand Down
Loading