fix(mcp): append to output instead of replacing (#658)

caarlos0 · web-flow · commit ae3b3d746fa4 · 2025-08-08T16:18:41.000-03:00
Signed-off-by: Carlos Alexandro Becker &lt;caarlos0@users.noreply.github.com&gt;
diff --git a/internal/llm/agent/mcp-tools.go b/internal/llm/agent/mcp-tools.go
@@ -6,6 +6,7 @@ import (
 	"fmt"
 	"log/slog"
 	"slices"
+	"strings"
 	"sync"
 	"time"
 
@@ -122,16 +123,16 @@ func runTool(ctx context.Context, name, toolName string, input string) (tools.To
 		return tools.NewTextErrorResponse(err.Error()), nil
 	}
 
-	output := ""
+	var output strings.Builder
 	for _, v := range result.Content {
 		if v, ok := v.(mcp.TextContent); ok {
-			output = v.Text
+			output.WriteString(v.Text)
 		} else {
-			output = fmt.Sprintf("%v", v)
+			_, _ = fmt.Fprintf(&output, "%v: ", v)
 		}
 	}
 
-	return tools.NewTextResponse(output), nil
+	return tools.NewTextResponse(output.String()), nil
 }
 
 func (b *McpTool) Run(ctx context.Context, params tools.ToolCall) (tools.ToolResponse, error) {

Original file line number	Diff line number	Diff line change
`@@ -6,6 +6,7 @@ import (`
`6`	`6`	`"fmt"`
`7`	`7`	`"log/slog"`
`8`	`8`	`"slices"`
	`9`	`+ "strings"`
`9`	`10`	`"sync"`
`10`	`11`	`"time"`
`11`	`12`
`@@ -122,16 +123,16 @@ func runTool(ctx context.Context, name, toolName string, input string) (tools.To`
`122`	`123`	`return tools.NewTextErrorResponse(err.Error()), nil`
`123`	`124`	`}`
`124`	`125`
`125`		`- output := ""`
	`126`	`+ var output strings.Builder`
`126`	`127`	`for _, v := range result.Content {`
`127`	`128`	`if v, ok := v.(mcp.TextContent); ok {`
`128`		`- output = v.Text`
	`129`	`+ output.WriteString(v.Text)`
`129`	`130`	`} else {`
`130`		`- output = fmt.Sprintf("%v", v)`
	`131`	`+ _, _ = fmt.Fprintf(&output, "%v: ", v)`
`131`	`132`	`}`
`132`	`133`	`}`
`133`	`134`
`134`		`- return tools.NewTextResponse(output), nil`
	`135`	`+ return tools.NewTextResponse(output.String()), nil`
`135`	`136`	`}`
`136`	`137`
`137`	`138`	`func (b *McpTool) Run(ctx context.Context, params tools.ToolCall) (tools.ToolResponse, error) {`