Skip to content

fix(provider): widen interleaved reasoning fields (#39556) - #8

Merged
DevFlex-AI merged 1 commit into
bolt-builder:devfrom
anomalyco:dev
Jul 30, 2026
Merged

fix(provider): widen interleaved reasoning fields (#39556)#8
DevFlex-AI merged 1 commit into
bolt-builder:devfrom
anomalyco:dev

Conversation

@DevFlex-AI

@DevFlex-AI DevFlex-AI commented Jul 30, 2026

Copy link
Copy Markdown

Issue for this PR

Closes #

Type of change

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

What does this PR do?

Please provide a description of the issue, the changes you made to fix it, and why they work. It is expected that you understand why your changes work and if you do not understand why at least say as much so a maintainer knows how much to value the PR.

If you paste a large clearly AI generated description here your PR may be IGNORED or CLOSED!

How did you verify your code works?

Screenshots / recordings

If this is a UI change, please include a screenshot or recording.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

If you do not follow this template your PR will be automatically rejected.


View with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is enabled.

Summary by CodeRabbit

  • New Features

    • Expanded support for interleaved reasoning fields, including reasoning_content, reasoning_text, and custom field names.
    • Interleaved settings now accept boolean, direct field-name, or structured formats.
    • Provider normalization now consistently converts field names into the supported structured format.
  • Bug Fixes

    • Improved handling of interleaved reasoning configuration across compatible models and providers.
    • Updated API schema validation to reflect the expanded options.

Co-authored-by: Aiden Cline <rekram1-node@users.noreply.github.com>
@vercel

vercel Bot commented Jul 30, 2026

Copy link
Copy Markdown

@opencode-agent[bot] is attempting to deploy a commit to the adevloper152's projects Team on Vercel.

A member of the Team first needs to authorize it.

@DevFlex-AI
DevFlex-AI merged commit 96cc8d8 into bolt-builder:dev Jul 30, 2026
12 of 15 checks passed
@github-actions

Copy link
Copy Markdown

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

  • No "Type of change" checkbox is checked. Please select at least one.
  • No issue referenced. Please add Closes #<number> linking to the relevant issue.
  • "How did you verify your code works?" section is empty. Please explain how you tested.
  • Not all checklist items are checked. Please confirm you have tested locally and have not included unrelated changes.

Please edit this PR description to address the above within 2 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

1 similar comment
@github-actions

Copy link
Copy Markdown

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

  • No "Type of change" checkbox is checked. Please select at least one.
  • No issue referenced. Please add Closes #<number> linking to the relevant issue.
  • "How did you verify your code works?" section is empty. Please explain how you tested.
  • Not all checklist items are checked. Please confirm you have tested locally and have not included unrelated changes.

Please edit this PR description to address the above within 2 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

@github-actions

Copy link
Copy Markdown

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@coderabbitai

coderabbitai Bot commented Jul 30, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: a4efc953-5da9-4ef6-9ab6-e79be9dc1aa0

📥 Commits

Reviewing files that changed from the base of the PR and between f690c35 and a1ab489.

⛔ Files ignored due to path filters (1)
  • packages/sdk/js/src/v2/gen/types.gen.ts is excluded by !**/gen/**
📒 Files selected for processing (5)
  • packages/core/src/models-dev.ts
  • packages/core/src/v1/config/provider.ts
  • packages/opencode/src/provider/provider.ts
  • packages/opencode/test/provider/provider.test.ts
  • packages/sdk/openapi.json

📝 Walkthrough

Walkthrough

Interleaved capability schemas now accept booleans, named fields, and arbitrary strings. Provider normalization converts string values into structured fields, with tests covering configured models and models.dev input. OpenAPI definitions reflect the expanded representations.

Changes

Interleaved capability support

Layer / File(s) Summary
Capability schema contracts
packages/core/src/models-dev.ts, packages/core/src/v1/config/provider.ts, packages/sdk/openapi.json
Interleaved values now support booleans, direct field strings, and structured fields, including reasoning_text and arbitrary strings.
Provider capability normalization
packages/opencode/src/provider/provider.ts
String interleaved values from models.dev and provider configuration are normalized to { field: ... }.
Normalization test coverage
packages/opencode/test/provider/provider.test.ts
Tests cover configured reasoning fields and models.dev normalization to structured interleaved capabilities.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Suggested reviewers: thdxr, rekram1-node

Poem

A rabbit hops through fields of thought,
Where reasoning strings are neatly caught.
Booleans, names, and objects align,
New texty trails now parse just fine.
Tests thump softly: all is right! 🐇

✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Fix failing CI checks
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint install failed. For unrecoverable errors, disable the tool in CodeRabbit configuration.


Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown

The following comment was made by an LLM, it may be inaccurate:

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.

1 participant