Skip to content

feat: add "reasoning" as interleaved field option for vLLM providers - #30477

Merged
rekram1-node merged 10 commits into
anomalyco:devfrom
delta9000:feat/reasoning-interleaved-field
Jun 9, 2026
Merged

feat: add "reasoning" as interleaved field option for vLLM providers#30477
rekram1-node merged 10 commits into
anomalyco:devfrom
delta9000:feat/reasoning-interleaved-field

Conversation

@delta9000

@delta9000 delta9000 commented Jun 2, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Issue for this PR

Closes #19988

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

Allows reasoning as a value for a model's interleaved.field, next to the existing reasoning_content and reasoning_details.

vLLM renamed the assistant reasoning field from reasoning_content to reasoning (≥0.16). On those versions vLLM's chat input parser reads message.reasoning and silently drops reasoning_content (vllm-project/vllm#38488).

How did you verify your code works?

  • Unit test: the config schema rejects field: "reasoning" before this change and accepts it after, and still rejects unknown values (enum stays closed).
  • Live on vLLM 0.20.0 / MiniMax-M2.7: reasoning sent as reasoning reaches the model, confirmed via log trace.

Screenshots / recordings

n/a — no UI change.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR
@github-actions github-actions Bot mentioned this pull request Jun 3, 2026
Open
@delta9000
feat: add "reasoning" as interleaved field option for vLLM providers
d66dc90 
Providers like vLLM use "reasoning" (not "reasoning_content" or
"reasoning_details") as the field name for interleaved reasoning content.
This adds it as a valid option in the schema, OpenAPI spec, and generated types.
@delta9000
delta9000 force-pushed the feat/reasoning-interleaved-field branch from 0a31616 to d66dc90 Compare June 3, 2026 03:10
@github-actions github-actions Bot mentioned this pull request Jun 3, 2026
Open
@klarkc

klarkc commented Jun 3, 2026

Copy link
Copy Markdown

Tested this PR with a self-hosted vLLM setup and it works nicely.

My setup:

  • OpenCode built from this PR via Nix
  • vLLM OpenAI-compatible /v1/chat/completions
  • Qwen3.6 35B A3B
  • --reasoning-parser qwen3
  • OpenCode config using:
{
  "interleaved": {
    "field": "reasoning"
  }
}
bharatv007
bharatv007 reviewed Jun 8, 2026
View reviewed changes

@bharatv007 bharatv007 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems relevant, any way to get this merged ASAP
@github-actions github-actions Bot mentioned this pull request Jun 9, 2026
Open
@rekram1-node
rekram1-node merged commit ab701d2 into anomalyco:dev Jun 9, 2026
6 of 8 checks passed
shoootyou added a commit to shoootyou/opencode that referenced this pull request Jun 10, 2026
@shoootyou @rekram1-node
@opencode-agent @simonklee @Hona @neriousy @fwang @nexxeln @areyouok @remorses @MrMushrooooom @adamdotdevin @thdxr @jlongster @AnthonyMLau @Brendonovich @delta9000 @kommander @songchaow @yui-soul
chore: sync fork with upstream anomalyco/opencode (#4)
4e89d72 
* test(core): cover skill directory output (anomalyco#31263)

* fix(opencode): avoid duplicate skill catalog (anomalyco#31269)

* chore(opencode): update MCP SDK to 1.29.0 (anomalyco#31268)

* chore: generate

* chore: update nix node_modules hashes

* run: make minimal mode more minimal (anomalyco#31227)

* chore: generate

* chore: update web and desktop code owners (anomalyco#31289)

* fix(desktop): few WSL bugs (anomalyco#31095)

* chore: generate

* fix(opencode): respect MCP server capabilities (anomalyco#31271)

* console: update email

* zen: fix

* fix(opencode): include acp pending tool input (anomalyco#31321)

* fix(lsp): resolve JDTLS root to topmost pom.xml in Java Maven multi-module projects (anomalyco#28761)

Co-authored-by: Shoubhit Dash <shoubhit2005@gmail.com>

* chore: generate

* fix(session): merge per-call tool rules into session permission (anomalyco#30529)

Co-authored-by: Simon Klee <hello@simonklee.dk>

* docs(go): update MiniMax M3 pricing (anomalyco#31350)

* fix: speed up fff file search (anomalyco#31366)

* chore: generate

* fix(stats): show new for leaderboard deltas

* fix: stabilize fff file results (anomalyco#31369)

* chore: generate

* fix(core): restore npm proxy agent patch (anomalyco#31373)

* chore: update nix node_modules hashes

* chore: upgrade OpenTUI to v0.3.4 (anomalyco#31326)

* chore: update nix node_modules hashes

* fix(core): disable fff trace logs (anomalyco#31380)

* fix(session): avoid sticky prompt tool overrides (anomalyco#31394)

* fix(opencode): await run event loop (anomalyco#31389)

* refactor(core): replace legacy logger with Effect logging (anomalyco#31310)

* chore: generate

* fix(tui): trim select footer action highlight (anomalyco#31411)

* fix(opencode): support MiniMax M3 thinking toggle (anomalyco#31426)

* fix: adjust item id stripping to happen prior to request signing (anomalyco#31429)

* fix(opencode): generate reasoning variants for all OpenRouter models. (anomalyco#30332)

Co-authored-by: Aiden Cline <63023139+rekram1-node@users.noreply.github.com>
Co-authored-by: Aiden Cline <aidenpcline@gmail.com>

* feat(app): add draft tab support to tabs store (anomalyco#31343)

* chore: generate

* fix(opencode): paginate MCP catalogs (anomalyco#31442)

* fix(opencode): pass abort signal to MCP tool calls (anomalyco#31455)

* feat(app): draft prompt state (anomalyco#31452)

* chore: generate

* feat(app): tabs help button (anomalyco#31454)

* feat: add "reasoning" as interleaved field option for vLLM providers (anomalyco#30477)

Co-authored-by: Ben Sandbrook <1126483+delta9000@users.noreply.github.com>
Co-authored-by: Aiden Cline <63023139+rekram1-node@users.noreply.github.com>

* fix(app): clip rounded session panels (anomalyco#31462)

* core: fix idle CPU use in file logger (anomalyco#31478)

* docs: add uninstall troubleshooting steps (anomalyco#31424)

Co-authored-by: opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com>

* leave a breadcrumb comment about batchWindow zero (anomalyco#31508)

* chore: stats -> data

* test(core): avoid Windows worker close race (anomalyco#31532)

* refactor(tui): centralize application exit (anomalyco#31524)

* chore: generate

* feat(opencode): configure Cohere North model (anomalyco#31536)

* refactor(core): consolidate references (anomalyco#31539)

* feat(tui): show project copy in session list (anomalyco#31421)

* chore: generate

* fix(stats): use data branding assets

* drop citation_options from cohere (anomalyco#31543)

* zen: add north mini code model

* fix(data): timestamp formatting

* fix(opencode): support Claude Fable reasoning (anomalyco#31546)

* fix(mcp): log actionable connection statuses (anomalyco#31544)

* refactor(core): simplify location filesystem (anomalyco#31545)

* chore: generate

* chore: update nix node_modules hashes

* fix(opencode): restore effect error logging (anomalyco#31551)

* chore: generate

* feat(opencode): add typed application layer graph (anomalyco#31531)

* zen: add claude fable 5

* chore: generate

* refactor(mcp): simplify service helpers (anomalyco#31549)

* feat: add X-Session-Id header for proxy cache routing affinity (anomalyco#31511)

* zen: update email

* fix(desktop): update Electron stack and panel layout (anomalyco#31571)

* chore: generate

* chore: update nix node_modules hashes

---------

Co-authored-by: Aiden Cline <63023139+rekram1-node@users.noreply.github.com>
Co-authored-by: opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com>
Co-authored-by: Simon Klee <hello@simonklee.dk>
Co-authored-by: Luke Parker <10430890+Hona@users.noreply.github.com>
Co-authored-by: Filip <34747899+neriousy@users.noreply.github.com>
Co-authored-by: Frank <frank@anoma.ly>
Co-authored-by: Shoubhit Dash <shoubhit2005@gmail.com>
Co-authored-by: huangli <areyouok@gmail.com>
Co-authored-by: Tommy D. Rossi <beats.by.morse@gmail.com>
Co-authored-by: Jack <jack@anoma.ly>
Co-authored-by: Adam <2363879+adamdotdevin@users.noreply.github.com>
Co-authored-by: Dax <mail@thdxr.com>
Co-authored-by: James Long <longster@gmail.com>
Co-authored-by: Anthony Lau <anthony.lau2000@live.com>
Co-authored-by: Aiden Cline <aidenpcline@gmail.com>
Co-authored-by: Brendan Allan <14191578+Brendonovich@users.noreply.github.com>
Co-authored-by: Ben Sandbrook <sandbrookvt@gmail.com>
Co-authored-by: Ben Sandbrook <1126483+delta9000@users.noreply.github.com>
Co-authored-by: opencode-agent[bot] <219766164+opencode-agent[bot]@users.noreply.github.com>
Co-authored-by: Sebastian <hasta84@gmail.com>
Co-authored-by: Songchao Wang <songchaow@outlook.com>
Co-authored-by: yui-soul <yui-soul@users.noreply.github.com>
avion23 pushed a commit to avion23/opencode that referenced this pull request Jun 10, 2026
@delta9000 @rekram1-node
feat: add "reasoning" as interleaved field option for vLLM providers (a…
3fd7916 
…nomalyco#30477)

Co-authored-by: Ben Sandbrook <1126483+delta9000@users.noreply.github.com>
Co-authored-by: Aiden Cline <63023139+rekram1-node@users.noreply.github.com>
isaacfinnegan pushed a commit to isaacfinnegan/whispercode that referenced this pull request Jun 15, 2026
@delta9000 @rekram1-node
feat: add "reasoning" as interleaved field option for vLLM providers (a…
5c2cf64 
…nomalyco#30477)

Co-authored-by: Ben Sandbrook <1126483+delta9000@users.noreply.github.com>
Co-authored-by: Aiden Cline <63023139+rekram1-node@users.noreply.github.com>
leduyphuc1702 pushed a commit to leduyphuc1702/opencode-workflow that referenced this pull request Jun 16, 2026
@delta9000 @rekram1-node Phuc Le
feat: add "reasoning" as interleaved field option for vLLM providers (a…
8e0059a 
…nomalyco#30477)

Co-authored-by: Ben Sandbrook <1126483+delta9000@users.noreply.github.com>
Co-authored-by: Aiden Cline <63023139+rekram1-node@users.noreply.github.com>
(cherry picked from commit ab701d2)
markjaquith pushed a commit to markjaquith/opencode that referenced this pull request Jun 23, 2026
@delta9000 @rekram1-node
feat: add "reasoning" as interleaved field option for vLLM providers (a…
4cef680 
…nomalyco#30477)

Co-authored-by: Ben Sandbrook <1126483+delta9000@users.noreply.github.com>
Co-authored-by: Aiden Cline <63023139+rekram1-node@users.noreply.github.com>
BenGu3 pushed a commit to BenGu3/opencode that referenced this pull request Jun 27, 2026
@delta9000 @rekram1-node @BenGu3
feat: add "reasoning" as interleaved field option for vLLM providers (a…
4693acd 
…nomalyco#30477)

Co-authored-by: Ben Sandbrook <1126483+delta9000@users.noreply.github.com>
Co-authored-by: Aiden Cline <63023139+rekram1-node@users.noreply.github.com>
zackslash pushed a commit to zackslash/opencode that referenced this pull request Jun 28, 2026
@delta9000 @rekram1-node @zackslash
feat: add "reasoning" as interleaved field option for vLLM providers (a…
640f23d 
…nomalyco#30477)

Co-authored-by: Ben Sandbrook <1126483+delta9000@users.noreply.github.com>
Co-authored-by: Aiden Cline <63023139+rekram1-node@users.noreply.github.com>
MarsQiu007 referenced this pull request in MarsQiu007/openNovel Jul 31, 2026
@delta9000 @rekram1-node
feat: add "reasoning" as interleaved field option for vLLM providers …
8c59cb1 
…(#30477)

Co-authored-by: Ben Sandbrook <1126483+delta9000@users.noreply.github.com>
Co-authored-by: Aiden Cline <63023139+rekram1-node@users.noreply.github.com>
banlanzs pushed a commit to banlanzs/opencode that referenced this pull request Aug 6, 2026
@delta9000 @rekram1-node
feat: add "reasoning" as interleaved field option for vLLM providers (a…
68179f3 
…nomalyco#30477)

Co-authored-by: Ben Sandbrook <1126483+delta9000@users.noreply.github.com>
Co-authored-by: Aiden Cline <63023139+rekram1-node@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants

@delta9000 @klarkc @bharatv007 @rekram1-node