Skip to content

Expand xAI reasoning effort to medium and xhigh - #35

Merged
rafaeelricco merged 1 commit into
mainfrom
rafaeelricco/expand-xai-effort-options
Aug 12, 2026
Merged

Expand xAI reasoning effort to medium and xhigh#35
rafaeelricco merged 1 commit into
mainfrom
rafaeelricco/expand-xai-effort-options

Conversation

@rafaeelricco

@rafaeelricco rafaeelricco commented Aug 12, 2026

Copy link
Copy Markdown
Owner

Motivation

No motivation was provided.

What's New

xAI Effort Catalog

  • XAI_EFFORTS now accepts low, medium, high, and xhigh in config and the effort picker.
  • The xAI slider defaults to high when no effort is stored.
  • Config tests treat xhigh as valid and none as rejected for xAI.

Changed Files

File Change Type Summary
src/domain/config/config.ts Modified Widen XAI_EFFORTS
src/infra/ui/effort-picker.ts Modified Default xAI effort to high
test/domain/config/config.test.ts Modified Valid xhigh; reject none
test/infra/ui/effort-picker.test.ts Modified selectXaiEffort options

Testing & Feedback

Confirm commit effort on an xAI model lists medium and xhigh, and that a saved xhigh config still loads.

If you find any bugs or have recommendations for improvements, please open an issue and assign it to me.


Note

Cursor Bugbot is generating a summary for commit 87579fa. Configure here.

Review in cubic

- Widen `XAI_EFFORTS` from `low|high` to `low|medium|high|xhigh`.
- Default the xAI effort slider to `high` when none is set.
- Cover the new catalog in config and picker tests.
@rafaeelricco rafaeelricco self-assigned this Aug 12, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

All reported issues were addressed across 4 files

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

Comment thread test/infra/ui/effort-picker.test.ts
@rafaeelricco
rafaeelricco merged commit 3812366 into main Aug 12, 2026
6 checks passed
@rafaeelricco rafaeelricco mentioned this pull request Sep 1, 2026
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