opencode/picoclaw
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #29 from hobbyistlabs-coder/bolt-optimize-loop-summary-13063486100100534764

Browse source 
⚡ Bolt: Optimize string building in message summary loop
This commit is contained in:
hobbyistlabs-coder 2026-03-15 01:31:16 -04:00 • committed by GitHub
commit e45b3d3d1f
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 10 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
pkg/agent/loop_summary.go
View file

@ -323,8 +323,12 @@ func (al *AgentLoop) summarizeBatch(
sb.WriteString("\n") sb.WriteString("\n")
} }
sb.WriteString("\nCONVERSATION:\n") sb.WriteString("\nCONVERSATION:\n")
// Bolt: Avoiding fmt.Fprintf overhead inside a hot loop by directly calling strings.Builder methods
for _, m := range batch { 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() prompt := sb.String()
@ -359,7 +363,11 @@ func (al *AgentLoop) summarizeBatch(
if keepLength < len(runes) { if keepLength < len(runes) {
content += "..." 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 return fallback.String(), nil
} }