Skip to content

feat(web): add reasoningSummary support for OpenAI and Azure providers#1102

Merged
brendan-kellam merged 2 commits intomainfrom
brendan/add-reasoning-summary
Apr 9, 2026
Merged

feat(web): add reasoningSummary support for OpenAI and Azure providers#1102
brendan-kellam merged 2 commits intomainfrom
brendan/add-reasoning-summary

Conversation

@brendan-kellam
Copy link
Copy Markdown
Contributor

@brendan-kellam brendan-kellam commented Apr 9, 2026

Summary

  • Adds reasoningSummary configuration option to the OpenAI and Azure OpenAI language model schemas
  • Defaults to auto (condensed summary). Set to detailed for comprehensive reasoning or none to disable.
  • Passes reasoningSummary through to the AI SDK's OpenAIResponsesProviderOptions

🤖 Generated with Claude Code

Summary by CodeRabbit

  • New Features

    • Added a reasoning summary configuration option for OpenAI and Azure OpenAI providers to control whether and how the model returns its reasoning output (options: auto, detailed, or none).
  • Documentation

    • Updated configuration documentation for OpenAI and Azure OpenAI providers to include examples of the new reasoning summary setting.

brendan-kellam and others added 2 commits April 9, 2026 11:01
Adds a `reasoningSummary` configuration option that controls whether reasoning
models return their reasoning process. Defaults to 'auto', can be set to
'detailed' for comprehensive reasoning or 'none' to disable.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions

This comment has been minimized.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 9, 2026

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

This PR extends OpenAI and Azure OpenAI provider configurations by adding a new reasoningSummary field that controls whether the model returns reasoning output. The field accepts values "auto", "detailed", or "none" (defaulting to "auto"), and is implemented consistently across documentation, schemas, type definitions, and runtime handling code.

Changes

Cohort / File(s) Summary
Changelog & Documentation
CHANGELOG.md, docs/docs/configuration/language-model-providers.mdx
Added reasoningSummary field to changelog and updated provider documentation with field descriptions, allowed values, and example configurations for both OpenAI and Azure OpenAI providers.
Type Definitions
packages/schemas/src/v3/index.type.ts, packages/schemas/src/v3/languageModel.type.ts
Added optional reasoningSummary?: string property to AzureLanguageModel and OpenAILanguageModel interfaces with documentation of accepted values and default behavior.
JSON Schema Definitions
packages/schemas/src/v3/index.schema.ts, packages/schemas/src/v3/languageModel.schema.ts, docs/snippets/schemas/v3/*.schema.mdx, schemas/v3/languageModel.json
Extended reasoningEffort examples to include "none" value and added new reasoningSummary string property with description and examples ("auto", "detailed", "none") across all applicable provider configuration schemas.
Runtime Implementation
packages/web/src/features/chat/utils.server.ts
Added conditional handling to derive reasoningSummary from configuration (defaulting to 'auto') and include it in OpenAI-style provider options only when value is not 'none'.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested reviewers

  • msukkari
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title accurately summarizes the main change: adding reasoningSummary support for OpenAI and Azure providers, which aligns with the primary objective across all modified files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch brendan/add-reasoning-summary

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@brendan-kellam brendan-kellam merged commit 59b5971 into main Apr 9, 2026
8 of 9 checks passed
@brendan-kellam brendan-kellam deleted the brendan/add-reasoning-summary branch April 9, 2026 18:06
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.

1 participant