Skip to content

Show default reasoning effort in the prompt for the live model - #493

Merged
TheGreatAxios merged 1 commit into
mainfrom
cl-6572-show-a-default-reasoning-effort-in-the-prompt-when-the-model
Aug 22, 2026
Merged

Show default reasoning effort in the prompt for the live model#493
TheGreatAxios merged 1 commit into
mainfrom
cl-6572-show-a-default-reasoning-effort-in-the-prompt-when-the-model

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Summary

  • When the live model supports reasoning, the prompt always shows an effort
  • Unset session effort stays unset in config and omits on the wire so the provider default applies
  • Leftover unsupported effort after a model switch is treated as the family default for cycle and the request
  • Shift+Tab cycles from the visible default instead of snapping to the first catalog rung
  • Non-reasoning models still omit the segment
  • xAI sources forward chosen effort and keep thinking-stream summary detailed

Verification

  • bun run typecheck, bun run build, and bun run test pass

Fixes CL-6572

The prompt only showed effort after an explicit pick, so grok and gpt-5
looked like they had none. Resolve the family default for display and
Shift+Tab without writing it into session config. Forward a chosen
effort through xAI sources; the grok adapter never invents high.

Leftover unsupported effort is treated as the family default for cycle
and wire. Unset still omits.
@linear-code

linear-code Bot commented Aug 22, 2026

Copy link
Copy Markdown

CL-6572

@TheGreatAxios TheGreatAxios left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Self-review of this branch.

When the live model supports reasoning, the prompt shows an effort. Unset config stays unset and the wire still omits so the provider default applies. Leftover unsupported effort after a model switch (xhigh on gpt-5) is treated as the family default for cycle and the request. Shift+Tab walks from that visible default, not the first catalog rung. Non-reasoning models omit the segment. The grok adapter forwards effort when set and does not invent high.

No blocking issues in this diff.

Merge after #492 so grok-4.6 xhigh is on the catalog first.

File-for-later: TUI/exec grok fallbacks still forward raw config effort if liveSources is empty, so a leftover invalid value would skip the clamp on that path.

@TheGreatAxios
TheGreatAxios merged commit 03d164b into main Aug 22, 2026
2 checks passed
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