Skip to content

feat(cron): match reasoning levels to selected model#251

Merged
su-fen merged 2 commits into
Stack-Cairn:mainfrom
AlphaCatMeow:feat/cron-dynamic-reasoning-levels
Jul 23, 2026
Merged

feat(cron): match reasoning levels to selected model#251
su-fen merged 2 commits into
Stack-Cairn:mainfrom
AlphaCatMeow:feat/cron-dynamic-reasoning-levels

Conversation

@AlphaCatMeow

@AlphaCatMeow AlphaCatMeow commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Summary

  • derive Cron reasoning options from the selected provider and model capabilities
  • hide off for always-on thinking models and reset unsupported selections to medium
  • keep desktop GUI and Gateway WebUI behavior mirrored, including corrected Chinese level labels

Validation

  • manual Tauri acceptance
  • pnpm --dir crates/agent-gui build
  • pnpm --dir crates/agent-gateway/web build
  • GUI settings/providers tests: 329 passed
  • Gateway WebUI tests: 403 passed
  • node scripts/check-mirror.mjs
  • git diff --check

Lint baseline

  • Full Biome lint remains blocked by existing repository-wide formatting and accessibility diagnostics in unchanged code; no unrelated lint cleanup is included in this PR.

Depends-On: none
Stack-Root: #251

AlphaCatMeow and others added 2 commits July 23, 2026 23:18
Dynamically derive Cron reasoning options from the selected provider model, hide off for always-on models, and normalize unsupported selections. Keep the desktop and Gateway WebUI mirrored and align Chinese reasoning labels with their runtime levels.
Switching a cron task to a non-reasoning model reset the reasoning
value to the hardcoded `medium` default, which such models don't
offer — the select rendered a value missing from its options and
saving persisted an unsupported level. Coerce to `medium` only when
the model offers it, otherwise the first supported level.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@su-fen
su-fen merged commit 4176280 into Stack-Cairn:main Jul 23, 2026
7 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.

2 participants