Skip to content

docs: align reasoning-effort values and model references#1199

Open
TheFactoriousDROID wants to merge 3 commits into
mainfrom
docs/reasoning-effort-values-and-model-refresh
Open

docs: align reasoning-effort values and model references#1199
TheFactoriousDROID wants to merge 3 commits into
mainfrom
docs/reasoning-effort-values-and-model-refresh

Conversation

@TheFactoriousDROID

Copy link
Copy Markdown
Contributor

Summary

Aligns the reasoning-effort and model documentation with the actual implementation (the ReasoningEffort enum, settings schema, and model registry).

docs/cli/configuration/settings.mdx

  • Document the full reasoning-effort scale — off/none, minimal, low, medium, high, xhigh, max — mapped to the labels shown in the model picker and the Available Models Reasoning column.
  • Key model/reasoning settings off sessionDefaultSettings.* (canonical), and note that legacy top-level model, reasoningEffort, autonomyMode, specModeModel, and specModeReasoningEffort are migrated automatically on load.
  • Point spec- and mission-mode reasoning at their respective keys.
  • Note that max may display as Max or Maximum depending on surface.

docs/reference/cli-reference.mdx

  • Reasoning-effort flags (-r, --worker-reasoning-effort, etc.) are model-dependent rather than a fixed value list.
  • Use claude-opus-4-8 in model-ID examples.

Verified against the implementation: the ReasoningEffort enum, SessionDefaultSettingsSchema / ManagedSettingsBaseSchema (session + org defaults), MissionModelSettingsSchema, and the model registry — and confirmed the same sessionDefaultSettings keys are read by both the CLI and desktop/daemon.

Note: an earlier commit on this branch edited choosing-your-model.mdx; that change was reverted, so the net diff is limited to the two files above.

Test plan

Docs-only change; no code paths affected.

TheFactoriousDROID and others added 3 commits June 7, 2026 01:59
Document the label-to-value mapping (e.g. Extra High -> xhigh, Max ->
max) as a single source of truth in settings.mdx, and point the CLI
reference and choosing-your-model pages at it plus /models instead of
duplicating per-model lists that drift. Also fix the stale 'Anthropic
defaults to off' claim, nest model/reasoningEffort under
sessionDefaultSettings, and refresh stale model names/examples.

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
Reverts docs/cli/user-guides/choosing-your-model.mdx to match main so this branch is scoped to the factual reasoning-effort settings and CLI reference updates. The page redesign is parked in a git stash for separate handling.

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
Drops the inaccurate claim that `dynamic` is an internal, non-selectable level (no CLI-available model exposes it, so CLI users never encounter it) and notes that the `max` label can render as Max or Maximum depending on surface.

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
@factory-droid

factory-droid Bot commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

Droid finished @TheFactoriousDROID's task —— View job


Security Review

The doc updates look directionally correct and improve alignment with model-dependent reasoning levels. There are a couple of cross-doc inconsistencies (notably dynamic and the Tab shortcut behavior) that should be reconciled to avoid contradictory guidance.

@mintlify

mintlify Bot commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
factory 🟢 Ready View Preview Jun 7, 2026, 8:59 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

| Label | `settings.json` value |
| ------------ | --------------------- |
| Off / None | `off` / `none` |
| Minimal | `minimal` |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[P1] Reconcile reasoningEffort values across docs (missing dynamic)

docs/enterprise/hierarchical-settings-and-org-control.mdx documents sessionDefaultSettings.reasoningEffort as allowing dynamic, but the new table here omits it; please reconcile by either adding dynamic (if valid for settings.json) or explicitly calling out that dynamic is enterprise/API-only and not accepted in local CLI settings to avoid contradictory guidance.

| `Ctrl+E` | Toggle the approval details view |
| `Ctrl+V` | Paste an image from the clipboard as an attachment |
| `Tab` | Cycle reasoning effort (`low` → `medium` → `high` → `off`) |
| `Tab` | Cycle reasoning effort through the levels supported by the current model |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[P1] Update other shortcut docs to match new Tab reasoning-effort behavior

This changes Tab to “cycle through the levels supported by the current model”, but docs/cli/user-guides/become-a-power-user.mdx still claims a fixed cycle (lowmediumhighoff); aligning those docs will prevent users from assuming missing levels (e.g., minimal, xhigh, max) are unreachable via the TUI.

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