Skip to content

Conversation

@mrubens
Copy link
Collaborator

@mrubens mrubens commented Jan 25, 2025

From Discord: "found a bug. When I approved the mode change it doesnt change the model like it does when i manually switch"


Important

Fix mode switch bug by centralizing logic in handleModeSwitch() in ClineProvider, ensuring correct API provider updates.

  • Behavior:
    • Fix bug where API provider wasn't updated after mode switch in Cline.ts and ClineProvider.ts.
    • Introduce handleModeSwitch() in ClineProvider to centralize mode switching logic.
  • Tests:
    • Add tests for handleModeSwitch() in ClineProvider.test.ts to verify correct API config loading and saving.
  • Misc:
    • Update Cline.ts to use handleModeSwitch() for mode changes.

This description was created by Ellipsis for 8c929ba. It will automatically update as commits are pushed.

@mrubens mrubens merged commit bf49b3e into main Jan 25, 2025
6 checks passed
@mrubens mrubens deleted the fix_switch_mode_command branch January 25, 2025 04:30
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.

3 participants