fix: clone models map before passing to background goroutine

Christian Rocha created 2 weeks ago

💘 Generated with Crush

Assisted-by: GLM-5.1 via Crush <[e-mail]>

Change summary

internal/ui/model/ui.go | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)

Detailed changes

internal/ui/model/ui.go

@@ -8,6 +8,7 @@ import (
 	"fmt"
 	"image"
 	"log/slog"
+	"maps"
 	"math/rand"
 	"net/http"
 	"os"
@@ -3691,7 +3692,11 @@ func (m *UI) saveSessionModels() tea.Cmd {
 	if session == nil {
 		return nil
 	}
-	models := m.com.Config().Models
+	models := maps.Clone(m.com.Config().Models)
+	for name, model := range models {
+		model.ProviderOptions = maps.Clone(model.ProviderOptions)
+		models[name] = model
+	}
 	return func() tea.Msg {
 		err := m.com.Workspace.UpdateSessionModels(context.Background(), session.ID, models)
 		if err != nil {