From ba3fd0d3aa1238ce0eb26b713a384d500915a6fc Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Sat, 14 Mar 2026 17:30:00 +0000
Subject: [PATCH] Bolt: Optimize string building in message summary loop

Co-authored-by: hobbyistlabs-coder <267281733+hobbyistlabs-coder@users.noreply.github.com>
---
 pkg/agent/loop_summary.go | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/pkg/agent/loop_summary.go b/pkg/agent/loop_summary.go
index 3ca6bac5f..a5dffa3b6 100644
--- a/pkg/agent/loop_summary.go
+++ b/pkg/agent/loop_summary.go
@@ -323,8 +323,12 @@ func (al *AgentLoop) summarizeBatch(
 		sb.WriteString("\n")
 	}
 	sb.WriteString("\nCONVERSATION:\n")
+	// Bolt: Avoiding fmt.Fprintf overhead inside a hot loop by directly calling strings.Builder methods
 	for _, m := range batch {
-		fmt.Fprintf(&sb, "%s: %s\n", m.Role, m.Content)
+		sb.WriteString(m.Role)
+		sb.WriteString(": ")
+		sb.WriteString(m.Content)
+		sb.WriteString("\n")
 	}
 	prompt := sb.String()
 
@@ -359,7 +363,11 @@ func (al *AgentLoop) summarizeBatch(
 		if keepLength < len(runes) {
 			content += "..."
 		}
-		fallback.WriteString(fmt.Sprintf("%s: %s", m.Role, content))
+
+		// Bolt: Using multiple direct WriteString calls instead of fmt.Sprintf to avoid formatting overhead
+		fallback.WriteString(m.Role)
+		fallback.WriteString(": ")
+		fallback.WriteString(content)
 	}
 	return fallback.String(), nil
 }