Skip to content

Conversation

@ammar-agent
Copy link
Collaborator

Summary

Change thinking level storage from per-workspace to per-model globally.

Changes

  • New storage key: thinkingLevel:model:{modelName} (global) replaces thinkingLevel:{workspaceId} (per-workspace)
  • ThinkingContext: Reads model from workspace storage to derive the per-model key
  • Simplified keybind: Toggle now cycles through allowed levels instead of toggle on/off with memory
  • Removed lastThinkingByModel: No longer needed with cycle behavior
  • No workspace sync: Thinking level not copied on workspace creation (uses global model preference)

Benefits

  • User's thinking preference for a model carries across all workspaces
  • Clearer mental model — thinking is a property of the model, not the workspace
  • Net code reduction despite adding functionality

Testing

  • All 699 browser tests pass
  • Typecheck passes
  • Static checks pass

Generated with mux • Model: anthropic:claude-sonnet-4-20250514 • Thinking: low

- Change thinking level storage from per-workspace to per-model
  Key format: thinkingLevel:model:{modelName}
- Remove lastThinkingByModel toggle memory (replaced by simple cycle)
- Update ThinkingProvider to read model from workspace storage
- Simplify keybind toggle to cycle through allowed levels
- Remove thinking level from workspace storage copy/sync

This makes thinking preferences follow the model rather than the
workspace, which better matches how thinking capability varies per model.

---
_Generated with `mux` • Model: `` • Thinking: ``_
@chatgpt-codex-connector
Copy link

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Repo admins can enable using credits for code reviews in their settings.

@ammario ammario merged commit e6de168 into main Dec 14, 2025
20 checks passed
@ammario ammario deleted the thinking-level-xgnz branch December 14, 2025 02:23
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