anomalyco/opencode
1
0
Fork 0
mirror of https://github.com/anomalyco/opencode.git synced 2026-07-21 17:56:33 -04:00
Code Issues 5.9k Packages Projects Releases 19 Wiki Activity

[PR #12772] fix: enable thinking for all reasoning models on alibaba-cn (DashScope) #14366

New Issue
Closed
opened 2026-02-16 18:19:10 -05:00 by yindo · 0 comments
yindo commented 2026-02-16 18:19:10 -05:00
Owner
Copy Link
Copy Source

Original Pull Request: https://github.com/anomalyco/opencode/pull/12772

State: closed
Merged: Yes

Summary

Fixes all alibaba-cn reasoning models (kimi-k2.5, qwen-plus, qwen3, qwq, deepseek-r1, etc.) not producing reasoning/thinking output by adding enable_thinking: true to the request body

Closes #12771

Problem

DashScope's OpenAI-compatible API requires enable_thinking: true in the request body for reasoning models to return reasoning_content. Without it, the field is either absent or empty and tokens.reasoning is always 0.

This affects all reasoning-capable models on the alibaba-cn provider, not just kimi-k2.5:

  • alibaba-cn/kimi-k2.5
  • alibaba-cn/qwen-plus
  • alibaba-cn/qwen-turbo
  • alibaba-cn/qwen-flash
  • alibaba-cn/qwen3-* (8b, 14b, 32b, 235b)
  • alibaba-cn/qwq-plus, alibaba-cn/qwq-32b
  • alibaba-cn/deepseek-r1*
  • alibaba-cn/qvq-max
  • etc.

The only exception is alibaba-cn/kimi-k2-thinking, which returns reasoning_content by default without the parameter.

The existing code in transform.ts options() only handled thinking for:

  1. providerID === "opencode" (via chat_template_args.enable_thinking)
  2. @ai-sdk/anthropic SDK (via thinking.type: "enabled")

But alibaba-cn uses @ai-sdk/openai-compatible, so neither path was taken.

Change

Added a condition in ProviderTransform.options() that sets enable_thinking: true for all reasoning-capable models on alibaba-cn using @ai-sdk/openai-compatible, excluding kimi-k2-thinking (which works without it).

Temporary Workaround (until PR merged)

Add this to your ~/.config/opencode/opencode.json:

{
  "provider": {
    "alibaba-cn": {
      "models": {
        "kimi-k2.5": { "options": { "enable_thinking": true } },
        "qwen-plus": { "options": { "enable_thinking": true } },
        "qwen-turbo": { "options": { "enable_thinking": true } },
        "qwen-flash": { "options": { "enable_thinking": true } },
        "qwen3-8b": { "options": { "enable_thinking": true } },
        "qwen3-14b": { "options": { "enable_thinking": true } },
        "qwen3-32b": { "options": { "enable_thinking": true } },
        "qwen3-235b-a22b": { "options": { "enable_thinking": true } },
        "qwv-plus": { "options": { "enable_thinking": true } },
        "qwq-32b": { "options": { "enable_thinking": true } },
        "qvq-max": { "options": { "enable_thinking": true } },
        "deepseek-r1": { "options": { "enable_thinking": true } },
        "deepseek-r1-0528": { "options": { "enable_thinking": true } },
        "deepseek-r1-distill-llama-8b": { "options": { "enable_thinking": true } },
        "deepseek-r1-distill-llama-70b": { "options": { "enable_thinking": true } },
        "deepseek-r1-distill-qwen-1-5b": { "options": { "enable_thinking": true } },
        "deepseek-r1-distill-qwen-7b": { "options": { "enable_thinking": true } },
        "deepseek-r1-distill-qwen-14b": { "options": { "enable_thinking": true } },
        "deepseek-r1-distill-qwen-32b": { "options": { "enable_thinking": true } },
        "qwen3-next-80b-a3b-thinking": { "options": { "enable_thinking": true } },
        "qwen3-omni-flash": { "options": { "enable_thinking": true } },
        "qwen3-vl-plus": { "options": { "enable_thinking": true } },
        "qwen3-vl-30b-a3b": { "options": { "enable_thinking": true } },
        "qwen3-vl-235b-a22b": { "options": { "enable_thinking": true } }
      }
    }
  }
}

Note: alibaba-cn/kimi-k2-thinking does NOT need this workaround as it returns reasoning content by default.

Verified

Tested via raw DashScope API:

Model Without enable_thinking With enable_thinking: true
kimi-k2.5 reasoning_content: "" Has reasoning content
qwen-plus No reasoning_content field Has reasoning content
kimi-k2-thinking Has reasoning content (always) N/A

Tested in OpenCode (opencode run -m alibaba-cn/kimi-k2.5 --format json --thinking):

  • Before: tokens.reasoning: 0, no type: "reasoning" events
  • After: tokens.reasoning: 54, reasoning content visible
**Original Pull Request:** https://github.com/anomalyco/opencode/pull/12772 **State:** closed **Merged:** Yes --- ## Summary Fixes all `alibaba-cn` reasoning models (kimi-k2.5, qwen-plus, qwen3, qwq, deepseek-r1, etc.) not producing reasoning/thinking output by adding `enable_thinking: true` to the request body Closes #12771 ## Problem DashScope's OpenAI-compatible API requires `enable_thinking: true` in the request body for reasoning models to return `reasoning_content`. Without it, the field is either absent or empty and `tokens.reasoning` is always 0. This affects **all** reasoning-capable models on the `alibaba-cn` provider, not just kimi-k2.5: - `alibaba-cn/kimi-k2.5` - `alibaba-cn/qwen-plus` - `alibaba-cn/qwen-turbo` - `alibaba-cn/qwen-flash` - `alibaba-cn/qwen3-*` (8b, 14b, 32b, 235b) - `alibaba-cn/qwq-plus`, `alibaba-cn/qwq-32b` - `alibaba-cn/deepseek-r1*` - `alibaba-cn/qvq-max` - etc. The only exception is `alibaba-cn/kimi-k2-thinking`, which returns `reasoning_content` by default without the parameter. The existing code in `transform.ts` `options()` only handled thinking for: 1. `providerID === "opencode"` (via `chat_template_args.enable_thinking`) 2. `@ai-sdk/anthropic` SDK (via `thinking.type: "enabled"`) But `alibaba-cn` uses `@ai-sdk/openai-compatible`, so neither path was taken. ## Change Added a condition in `ProviderTransform.options()` that sets `enable_thinking: true` for all reasoning-capable models on `alibaba-cn` using `@ai-sdk/openai-compatible`, excluding `kimi-k2-thinking` (which works without it). ## Temporary Workaround (until PR merged) Add this to your `~/.config/opencode/opencode.json`: ```json { "provider": { "alibaba-cn": { "models": { "kimi-k2.5": { "options": { "enable_thinking": true } }, "qwen-plus": { "options": { "enable_thinking": true } }, "qwen-turbo": { "options": { "enable_thinking": true } }, "qwen-flash": { "options": { "enable_thinking": true } }, "qwen3-8b": { "options": { "enable_thinking": true } }, "qwen3-14b": { "options": { "enable_thinking": true } }, "qwen3-32b": { "options": { "enable_thinking": true } }, "qwen3-235b-a22b": { "options": { "enable_thinking": true } }, "qwv-plus": { "options": { "enable_thinking": true } }, "qwq-32b": { "options": { "enable_thinking": true } }, "qvq-max": { "options": { "enable_thinking": true } }, "deepseek-r1": { "options": { "enable_thinking": true } }, "deepseek-r1-0528": { "options": { "enable_thinking": true } }, "deepseek-r1-distill-llama-8b": { "options": { "enable_thinking": true } }, "deepseek-r1-distill-llama-70b": { "options": { "enable_thinking": true } }, "deepseek-r1-distill-qwen-1-5b": { "options": { "enable_thinking": true } }, "deepseek-r1-distill-qwen-7b": { "options": { "enable_thinking": true } }, "deepseek-r1-distill-qwen-14b": { "options": { "enable_thinking": true } }, "deepseek-r1-distill-qwen-32b": { "options": { "enable_thinking": true } }, "qwen3-next-80b-a3b-thinking": { "options": { "enable_thinking": true } }, "qwen3-omni-flash": { "options": { "enable_thinking": true } }, "qwen3-vl-plus": { "options": { "enable_thinking": true } }, "qwen3-vl-30b-a3b": { "options": { "enable_thinking": true } }, "qwen3-vl-235b-a22b": { "options": { "enable_thinking": true } } } } } } ``` Note: `alibaba-cn/kimi-k2-thinking` does NOT need this workaround as it returns reasoning content by default. ## Verified Tested via raw DashScope API: | Model | Without `enable_thinking` | With `enable_thinking: true` | |-------|--------------------------|------------------------------| | kimi-k2.5 | `reasoning_content: ""` | Has reasoning content | | qwen-plus | No `reasoning_content` field | Has reasoning content | | kimi-k2-thinking | Has reasoning content (always) | N/A | Tested in OpenCode (`opencode run -m alibaba-cn/kimi-k2.5 --format json --thinking`): - **Before**: `tokens.reasoning: 0`, no `type: "reasoning"` events - **After**: `tokens.reasoning: 54`, reasoning content visible
yindo added the pull-request label 2026-02-16 18:19:10 -05:00
yindo closed this issue 2026-02-16 18:19:10 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
acp
bad
bug
discussion
docs
documentation
enhancement
extension
good first issue
growth
help-wanted
mcp
model-problem
needs:compliance
nix
opentui
perf
pull-request
Mirrored from GitHub Pull Request
 tui
web
windows
zen
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: anomalyco/opencode#14366
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?