From 985289a3b74c648c96c753fdad840eae10969b6c Mon Sep 17 00:00:00 2001
From: Bernardo <Bernardo@example.com>
Date: Tue, 10 Mar 2026 14:48:22 +0100
Subject: [PATCH] fix: resolve OpenRouter migration error and reduce bot
 rambling by tuning temperature and tool iterations

---
 pkg/agent/instance.go   | 2 +-
 pkg/config/defaults.go  | 2 +-
 pkg/config/migration.go | 6 +++---
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/pkg/agent/instance.go b/pkg/agent/instance.go
index b60818875..fd8ceb79f 100644
--- a/pkg/agent/instance.go
+++ b/pkg/agent/instance.go
@@ -125,7 +125,7 @@ func NewAgentInstance(
 		maxTokens = 8192
 	}
 
-	temperature := 0.7
+	temperature := 0.1
 	if defaults.Temperature != nil {
 		temperature = *defaults.Temperature
 	}
diff --git a/pkg/config/defaults.go b/pkg/config/defaults.go
index e64baa720..6d578a47a 100644
--- a/pkg/config/defaults.go
+++ b/pkg/config/defaults.go
@@ -32,7 +32,7 @@ func DefaultConfig() *Config {
 				Model:                     "",
 				MaxTokens:                 32768,
 				Temperature:               nil, // nil means use provider default
-				MaxToolIterations:         50,
+				MaxToolIterations:         10,
 				SummarizeMessageThreshold: 20,
 				SummarizeTokenPercent:     75,
 			},
diff --git a/pkg/config/migration.go b/pkg/config/migration.go
index 51f21e4f4..24d567f85 100644
--- a/pkg/config/migration.go
+++ b/pkg/config/migration.go
@@ -14,11 +14,11 @@ import (
 // If the model already contains a "/" (indicating it has a protocol prefix), it is returned as-is.
 // Otherwise, the protocol prefix is added.
 func buildModelWithProtocol(protocol, model string) string {
-	if strings.Contains(model, "/") {
-		// Model already has a protocol prefix, return as-is
+	prefix := protocol + "/"
+	if strings.HasPrefix(model, prefix) {
 		return model
 	}
-	return protocol + "/" + model
+	return prefix + model
 }
 
 // providerMigrationConfig defines how to migrate a provider from old config to new format.