Skip to content

docs: correct JSON completion reasoning behavior in configuration - #261

Open
detail-app[bot] wants to merge 1 commit into
devfrom
detail/fix-docs/docs-correct-json-completion-reasoning-behavior-in-261ef0
Open

detail-app[bot] wants to merge 1 commit into
devfrom
detail/fix-docs/docs-correct-json-completion-reasoning-behavior-in-261ef0

Conversation

@detail-app

@detail-app detail-app Bot commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

The configuration doc claimed JSON-object completions (reranker,
enrichment) 'always inherit the gateway/model default' for reasoning,
but commit 2a740a3 changed OpenAiRequestFactory to explicitly pin
reasoning to none on JSON completions so reasoning tokens cannot
starve their small output budgets. A developer reading the doc would
expect omitting reasoning is sufficient and could re-introduce the
budget-starvation regression the commit fixed.

Introduced by commit 2a740a3 (@WilliamAGH, #259)


Doc Drift PRs can be configured here.

@coderabbitai

coderabbitai Bot commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 53737721-fdb6-406d-bb89-265435a44592

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

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.

1 participant