Persist Codex Thinking Level and Fast Mode#1004
Persist Codex Thinking Level and Fast Mode#1004maskdotdev wants to merge 3 commits intopingdotgg:mainfrom
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
juliusmarminge
left a comment
There was a problem hiding this comment.
should behave similar to the draft state for model selected
What Changed
Moved Codex reasoning effort and fast mode from thread local draft state to persisted global last used settings. ChatView, app settings, and composer draft persistence were updated so the composer follows the shared preference across threads and sessions, and the reasoning labels were centralized in one helper.
Why
resolve #384
Checklist
Video
t3code-persist-thinking.mp4
Note
Persist Codex reasoning effort and fast mode as global user preferences
defaultCodexReasoningEffortandlastUsedCodexFastModetoAppSettingsSchemaso selections survive across threads and sessions.codexFastModeinComposerThreadDraftStatefrombooleantoboolean | null, wherenullmeans "follow global preference" rather than an explicit off.null).CODEX_REASONING_EFFORT_LABELSexport inpackages/contracts/src/model.ts; bothCodexTraitsPickerandCompactComposerControlsMenunow receive adefaultEffortprop to mark the current global default.codexFastMode === falsewere treated identically to unset; they are now persisted as an explicit override, and onlynullsignals absence.Macroscope summarized deb11c8.