goose icon indicating copy to clipboard operation
goose copied to clipboard

fix: when changing modes mid chat or in settings need to notify

Open michaelneale opened this issue 1 week ago • 8 comments

When changing chat modes - you need to currently restart sessions. This will tell you via a simple toast to do that if you change when in active chat or in settings.

michaelneale avatar Dec 18 '25 01:12 michaelneale

Thanks! I looked into it and we don't need to change mid chat it applies on the next message reply currently so we only need to do this on the settings page.

zanesq avatar Dec 18 '25 01:12 zanesq

FWIW I have seen on model change that it doesn't show this in the UI until after reload

DOsinga avatar Dec 18 '25 13:12 DOsinga

@DOsinga oh so we should add the warning for model change as well you mean?

(this is partly only temporary until multi chat lands)

michaelneale avatar Dec 18 '25 20:12 michaelneale

model change seems to work fine mid chat for me, example switching to gemini (model response fails for other reasons but you can see it working)

https://github.com/user-attachments/assets/3fd4fac6-5730-4e60-83c7-f97cf9e14b56

zanesq avatar Dec 18 '25 20:12 zanesq

so may be ok as is @zanesq ?

michaelneale avatar Dec 18 '25 22:12 michaelneale

I think model switching mid chat is ok so no need for a notice there. Mode change mid chat also works mid chat currently so no need for it there also. Can you remove it from mid chat and only keep it in settings?

zanesq avatar Dec 18 '25 23:12 zanesq

hrm - when I tried it, it didn't seem to actually change it mid chat - you sure @zanesq ? (I just put it in chat only - and it still called tools)

michaelneale avatar Dec 19 '25 00:12 michaelneale

I haven't tried it just looking at the code I see it applies the mode on the next reply.. will try it first thing tomorrow to verify

zanesq avatar Dec 19 '25 01:12 zanesq

@michaelneale strange it works for me on main see below. Also for some reason if I test on this branch it never gets past the first prompt "create a test file in my desktop" so there might be a bug in this branch?

https://github.com/user-attachments/assets/d34d90b0-5c83-4f2e-8910-b569e4e1854c

zanesq avatar Dec 19 '25 19:12 zanesq

hrm yeah - not sure what has gone wrong but will take another run at it.

michaelneale avatar Dec 22 '25 23:12 michaelneale