Updated the IsError determination for a list of AIContent to require that all items in contentItems are instances of ErrorContent and that the collection is not empty

Andy Tagg · Andy Tagg · commit 803c8824afb8 · 2025-05-08T14:11:22.000+01:00
diff --git a/src/ModelContextProtocol/Server/AIFunctionMcpServerTool.cs b/src/ModelContextProtocol/Server/AIFunctionMcpServerTool.cs
@@ -280,7 +280,7 @@ public override async ValueTask<CallToolResponse> InvokeAsync(
             IEnumerable<AIContent> contentItems => new()
             {
                 Content = [.. contentItems.Select(static item => item.ToContent())],
-                IsError = contentItems.Any(item => item is ErrorContent)
+                IsError = contentItems.All(item => item is ErrorContent) && contentItems.Any()
             },
             
             IEnumerable<Content> contents => new()

Original file line number	Diff line number	Diff line change
`@@ -280,7 +280,7 @@ public override async ValueTask<CallToolResponse> InvokeAsync(`
`280`	`280`	`IEnumerable<AIContent> contentItems => new()`
`281`	`281`	`{`
`282`	`282`	`Content = [.. contentItems.Select(static item => item.ToContent())],`
`283`		`- IsError = contentItems.Any(item => item is ErrorContent)`
	`283`	`+ IsError = contentItems.All(item => item is ErrorContent) && contentItems.Any()`
`284`	`284`	`},`
`285`	`285`
`286`	`286`	`IEnumerable<Content> contents => new()`