-
Notifications
You must be signed in to change notification settings - Fork 951
feat(grok): inject per-model reasoning effort into Grok Build config #1756
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
lidge-jun
merged 8 commits into
lidge-jun:dev
from
takltc:feat/grok-inject-reasoning-effort
Aug 29, 2026
Merged
Changes from all commits
Commits
Show all changes
8 commits
Select commit
Hold shift + click to select a range
b64257d
feat(grok): inject per-model reasoning effort into Grok Build config
takltc 50f010a
docs(grok): correct Traditional Chinese ocx restart lifecycle
takltc 8059d38
fix(grok): preserve none and minimal effort tiers
takltc dcd089a
fix(grok): unify effort fallback and French guidance
takltc 10a8b27
docs(grok): clarify French managed-config safety
takltc 8271d69
docs(grok): document complete effort filtering
takltc b92e9af
fix(grok): align examples with native context window
takltc 47fe01f
test(grok): pin ultra effort surface semantics
takltc File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This newly added Japanese paragraph says Grok uses Chat Completions and sends
reasoning_effort, but the managed table setsapi_backend = "responses"and the canonical English page correctly describes Responsesreasoning.summaryand Responses reasoning items. The same contradictory Chat Completions guidance was added to the Korean, Russian, Turkish, Simplified Chinese, and Traditional Chinese pages, so users of those locales are directed to controls such asinclude_reasoningthat do not describe the configured transport. Translate the canonical Responses paragraph consistently across these locales.AGENTS.md reference: docs-site/AGENTS.md:L9-L10
Useful? React with 👍 / 👎.