Skip to content

Add supported reasoning efforts to provider models - #2858

Open
burkeholland wants to merge 1 commit into
mainfrom
burkeholland-provider-reasoning-efforts
Open

burkeholland wants to merge 1 commit into
mainfrom
burkeholland-provider-reasoning-efforts

Conversation

@burkeholland

Copy link
Copy Markdown
Contributor

Summary

  • Add optional supported reasoning effort levels to Node.js and Rust provider model configs.
  • Preserve omission compatibility and serialize the JSON field as supportedReasoningEfforts.

Validation

  • Rust formatting and focused serialization tests.
  • Node.js TypeScript 5.9 type-check.

The canonical schema/codegen inputs are external to this SDK checkout; this updates the checked-in Node.js and Rust projections.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI max review requested due to automatic review settings October 21, 2026 16:25
@burkeholland
burkeholland requested a review from a team as a code owner October 21, 2026 16:26

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🔵 Needs a closer look

The generated additions need schema-backed regeneration to satisfy repository requirements and prevent later regeneration from removing them.

No open issues 🎉

What changed in this PR

Adds optional reasoning-effort levels to Node.js and Rust provider model configurations while preserving omission compatibility.

Changes:

  • Extends public and generated model types with supportedReasoningEfforts.
  • Adds a Rust builder and serialization compatibility tests.
File Description
rust/​src/​types/​tests.rs Tests serialization and omitted-field compatibility.
rust/​src/​types.rs Adds the optional field and builder.
rust/​src/​generated/​api_types.rs Extends the generated Rust model type.
nodejs/​src/​types.ts Extends the public Node.js model interface.
nodejs/​src/​generated/​rpc.ts Extends the generated Node.js model interface.

Review effort: Max

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants