@f5-sales-demo/pi-agent-core 20.2.7 → 20.3.1

package/CHANGELOG.md

@@ -1,6 +1,12 @@
+<!-- markdownlint-configure-file { "MD024": { "siblings_only": true } } -->
+
 # Changelog
 
-## [Unreleased]
+## [20.3.0] - 2026-08-04
+
+### Fixed
+
+- Fixed successful tools with empty output being indistinguishable from missing results to the model ([#2874](https://github.com/f5-sales-demo/xcsh/issues/2874))
 
 ## [14.0.1] - 2026-04-08
 

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@f5-sales-demo/pi-agent-core",
-	"version": "20.2.7",
+	"version": "20.3.1",
 	"description": "General-purpose agent with transport abstraction, state management, and attachment support",
 	"homepage": "https://github.com/f5-sales-demo/xcsh",
 	"author": "Can Boluk",
@@ -35,8 +35,8 @@
 		"fmt": "biome format --write ."
 	},
 	"dependencies": {
-		"@f5-sales-demo/pi-ai": "20.2.7",
-		"@f5-sales-demo/pi-utils": "20.2.7"
+		"@f5-sales-demo/pi-ai": "20.3.1",
+		"@f5-sales-demo/pi-utils": "20.3.1"
 	},
 	"devDependencies": {
 		"@sinclair/typebox": "^0.34",

package/src/agent-loop.ts

@@ -11,6 +11,7 @@ import {
 	validateToolArguments,
 } from "@f5-sales-demo/pi-ai";
 import { logger } from "@f5-sales-demo/pi-utils";
+import emptyToolSuccess from "./prompts/empty-tool-success.md" with { type: "text" };
 import type {
 	AgentContext,
 	AgentEvent,
@@ -500,6 +501,7 @@ async function executeToolCalls(
 
 	const emitToolResult = (record: (typeof records)[number], result: AgentToolResult<any>, isError: boolean): void => {
 		if (record.resultEmitted) return;
+		const normalizedResult = normalizeToolResult(result, isError);
 		const { toolCall } = record;
 		if (!record.started) {
 			stream.push({
@@ -510,12 +512,12 @@ async function executeToolCalls(
 				intent: toolCall.intent,
 			});
 		}
-		const isWarning = Boolean(result.isWarning);
+		const isWarning = Boolean(normalizedResult.isWarning);
 		stream.push({
 			type: "tool_execution_end",
 			toolCallId: toolCall.id,
 			toolName: toolCall.name,
-			result,
+			result: normalizedResult,
 			isError,
 			isWarning,
 		});
@@ -524,13 +526,13 @@ async function executeToolCalls(
 			role: "toolResult",
 			toolCallId: toolCall.id,
 			toolName: toolCall.name,
-			content: result.content,
-			details: result.details,
+			content: normalizedResult.content,
+			details: normalizedResult.details,
 			isError,
 			isWarning,
 			timestamp: Date.now(),
 		};
-		record.result = result;
+		record.result = normalizedResult;
 		record.isError = isError;
 		record.toolResultMessage = toolResultMessage;
 		record.resultEmitted = true;
@@ -705,3 +707,14 @@ function createSkippedToolResult(): AgentToolResult<any> {
 		details: {},
 	};
 }
+
+function normalizeToolResult<T>(result: AgentToolResult<T>, isError: boolean): AgentToolResult<T> {
+	if (isError || result.content.some(content => content.type !== "text" || content.text.trim().length > 0)) {
+		return result;
+	}
+
+	return {
+		...result,
+		content: [{ type: "text", text: emptyToolSuccess.trim() }],
+	};
+}

package/src/prompts/empty-tool-success.md

@@ -0,0 +1 @@
+Tool completed successfully and produced no output.