Merge pull request #229 from mqasimsarfraz/fix/empty-messages

rumpl · web-flow · commit 40a1b4752c48 · 2025-09-17T17:05:59.000+02:00
Fix: Handle empty tool messages
diff --git a/pkg/tools/mcp/client.go b/pkg/tools/mcp/client.go
@@ -204,6 +204,11 @@ func processMCPContent(toolResult *mcp.CallToolResult) *tools.ToolCallResult {
 		}
 	}
 
+	// Handle an empty response. This can happen if the MCP tool does not return any content.
+	if finalContent == "" {
+		finalContent = "no output"
+	}
+
 	return &tools.ToolCallResult{
 		Output: finalContent,
 	}

Original file line number	Diff line number	Diff line change
`@@ -204,6 +204,11 @@ func processMCPContent(toolResult mcp.CallToolResult) tools.ToolCallResult {`
`204`	`204`	`}`
`205`	`205`	`}`
`206`	`206`
	`207`	`+ // Handle an empty response. This can happen if the MCP tool does not return any content.`
	`208`	`+ if finalContent == "" {`
	`209`	`+ finalContent = "no output"`
	`210`	`+ }`
	`211`	`+`
`207`	`212`	`return &tools.ToolCallResult{`
`208`	`213`	`Output: finalContent,`
`209`	`214`	`}`