@f5xc-salesdemos/xcsh 19.28.2 → 19.29.1

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@f5xc-salesdemos/xcsh",
-	"version": "19.28.2",
+	"version": "19.29.1",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://github.com/f5xc-salesdemos/xcsh",
 	"author": "Can Boluk",
@@ -50,12 +50,12 @@
 	"dependencies": {
 		"@agentclientprotocol/sdk": "0.16.1",
 		"@mozilla/readability": "^0.6",
-		"@f5xc-salesdemos/xcsh-stats": "19.28.2",
-		"@f5xc-salesdemos/pi-agent-core": "19.28.2",
-		"@f5xc-salesdemos/pi-ai": "19.28.2",
-		"@f5xc-salesdemos/pi-natives": "19.28.2",
-		"@f5xc-salesdemos/pi-tui": "19.28.2",
-		"@f5xc-salesdemos/pi-utils": "19.28.2",
+		"@f5xc-salesdemos/xcsh-stats": "19.29.1",
+		"@f5xc-salesdemos/pi-agent-core": "19.29.1",
+		"@f5xc-salesdemos/pi-ai": "19.29.1",
+		"@f5xc-salesdemos/pi-natives": "19.29.1",
+		"@f5xc-salesdemos/pi-tui": "19.29.1",
+		"@f5xc-salesdemos/pi-utils": "19.29.1",
 		"@sinclair/typebox": "^0.34",
 		"@xterm/headless": "^6.0",
 		"ajv": "^8.20",

package/src/internal-urls/build-info.generated.ts

@@ -17,17 +17,17 @@ export interface BuildInfo {
 }
 
 export const BUILD_INFO: BuildInfo = {
-	"version": "19.28.2",
-	"commit": "ebd6c0383d5c0e5bde390d93e4b6346d311efdae",
-	"shortCommit": "ebd6c03",
+	"version": "19.29.1",
+	"commit": "87b69f04b6efe97f6f5d1f849b61ee0f01e448e0",
+	"shortCommit": "87b69f0",
 	"branch": "main",
-	"tag": "v19.28.2",
-	"commitDate": "2026-06-12T12:35:42Z",
-	"buildDate": "2026-06-12T13:06:01.155Z",
+	"tag": "v19.29.1",
+	"commitDate": "2026-06-12T18:13:55Z",
+	"buildDate": "2026-06-12T18:33:56.944Z",
 	"dirty": true,
 	"prNumber": "",
 	"repoUrl": "https://github.com/f5xc-salesdemos/xcsh",
 	"repoSlug": "f5xc-salesdemos/xcsh",
-	"commitUrl": "https://github.com/f5xc-salesdemos/xcsh/commit/ebd6c0383d5c0e5bde390d93e4b6346d311efdae",
-	"releaseUrl": "https://github.com/f5xc-salesdemos/xcsh/releases/tag/v19.28.2"
+	"commitUrl": "https://github.com/f5xc-salesdemos/xcsh/commit/87b69f04b6efe97f6f5d1f849b61ee0f01e448e0",
+	"releaseUrl": "https://github.com/f5xc-salesdemos/xcsh/releases/tag/v19.29.1"
 };

package/src/tools/render-utils.ts

@@ -666,3 +666,12 @@ export function addSection(
 		sections.push({ label: titled, lines });
 	}
 }
+
+export function humanizeResourceType(raw: string): string {
+	return raw
+		.replace(/^http_/, "")
+		.replace(/^tcp_/, "TCP ")
+		.replace(/_/g, " ")
+		.replace(/([a-z])(balancer|checker)/gi, "$1 $2")
+		.replace(/s$/, "");
+}

package/src/tools/xcsh-api-renderer.ts

@@ -8,6 +8,7 @@ import { CachedOutputBlock, F5_TOOL_BORDER_COLOR, renderStatusLine } from "../tu
 import {
 	formatErrorMessage,
 	formatTimestamp,
+	humanizeResourceType,
 	addSection as pushSection,
 	replaceTabs,
 	stripEmpty,
@@ -91,7 +92,15 @@ function splitResultContent(textContent: string, isError: boolean): { json?: str
 
 	if (!isError) {
 		const pretty = tryPrettyJson(body);
-		return pretty ? { json: pretty, raw: body } : { raw: body };
+		if (pretty) return { json: pretty, raw: body };
+		// Body may contain "compactJSON\n\nAI guidance text" — split and discard guidance for display
+		const split = body.indexOf("\n\n");
+		if (split >= 0) {
+			const jsonPart = body.slice(0, split);
+			const prettyPart = tryPrettyJson(jsonPart);
+			if (prettyPart) return { json: prettyPart, raw: body };
+		}
+		return { raw: body };
 	}
 
 	// Error: body is "compactJSON\n\nguidanceText"
@@ -263,15 +272,10 @@ export const xcshApiToolRenderer = {
 		const emptyList = Array.isArray(parsed?.items) && (parsed!.items as unknown[]).length === 0;
 
 		if ((emptyBody || emptyList) && !guidance) {
-			// Contextual success message based on HTTP method and response shape
-			let successMessage = emptyList ? "No items found." : "Empty response";
-			const rn = resourceName ? ` \u2018${resourceName}\u2019` : "";
-			if (!emptyList && status >= 200 && status < 300) {
-				if (method === "DELETE") successMessage = `Resource${rn} deleted successfully.`;
-				else if (method === "POST") successMessage = `Resource${rn} created successfully.`;
-				else if (method === "PUT" || method === "PATCH") successMessage = `Resource${rn} updated successfully.`;
+			if (emptyList) {
+				addSection("Response", [uiTheme.fg("dim", "No items found.")]);
 			}
-			addSection("Response", [uiTheme.fg("dim", successMessage)]);
+			// Empty mutation bodies ({}) are handled by the Result section below \u2014 no Response section needed
 		} else if (json && parsed) {
 			// Branch 1: List response with named items — compact summary
 			const items = parsed.items;
@@ -343,15 +347,38 @@ export const xcshApiToolRenderer = {
 			);
 		}
 
-		// Section 4: Error guidance (for HTTP error responses)
-		if (guidance) {
-			// Extract the API's specific error message from JSON body for prominent display
-			const guidanceLines: string[] = [];
-			const apiMessage =
-				parsed && typeof parsed.message === "string" ? stripProtobufPrefix(parsed.message) : undefined;
-			if (apiMessage) guidanceLines.push(uiTheme.fg("error", apiMessage));
-			guidanceLines.push(uiTheme.fg("warning", guidance));
-			addSection("Guidance", guidanceLines);
+		// Section: Result — human-readable glyph status at the bottom of the panel
+		{
+			const resultLines: string[] = [];
+			const infoIcon = uiTheme.styledSymbol("status.success", "chromeAccent");
+			const errorIcon = uiTheme.styledSymbol("status.error", "error");
+
+			if (isError) {
+				const apiMessage =
+					parsed && typeof parsed.message === "string" ? stripProtobufPrefix(parsed.message) : undefined;
+				const errorLabel = apiMessage ?? (status > 0 ? `Request failed (${status})` : "Request failed");
+				resultLines.push(`${errorIcon} ${uiTheme.fg("warning", errorLabel)}`);
+				if (guidance) resultLines.push(`  ${uiTheme.fg("dim", `Try: ${guidance}`)}`);
+			} else if (details?.mutationVerb && details?.resourceLabel) {
+				resultLines.push(`${infoIcon} ${details.mutationVerb} ${details.resourceLabel}`);
+			} else if (method === "GET" && parsed) {
+				const items = parsed.items;
+				if (Array.isArray(items)) {
+					const rawType = pathParts.at(-1) ?? "items";
+					const typeLabel = humanizeResourceType(rawType);
+					const plural = items.length === 1 ? typeLabel : `${typeLabel}s`;
+					resultLines.push(`${infoIcon} ${items.length} ${plural}`);
+				} else {
+					const rawType = pathParts.at(-2) ?? "";
+					const name = resourceName ?? "";
+					const label = rawType && name ? `${humanizeResourceType(rawType)} '${name}'` : displayPath;
+					resultLines.push(`${infoIcon} Loaded ${label}`);
+				}
+			} else if (status >= 200 && status < 300) {
+				resultLines.push(`${infoIcon} OK`);
+			}
+
+			if (resultLines.length > 0) addSection("Result", resultLines);
 		}
 
 		// --- Render with CachedOutputBlock ---

package/src/tools/xcsh-api.ts

@@ -5,6 +5,7 @@ import { type Static, Type } from "@sinclair/typebox";
 import xcshApiDescription from "../prompts/tools/xcsh-api.md" with { type: "text" };
 import { type ContextEnv, createContextEnv } from "../services/context-env";
 import type { ToolSession } from ".";
+import { humanizeResourceType } from "./render-utils";
 
 // Namespace filtering driven by x-f5xc-namespace-profile from enriched API specs.
 
@@ -108,6 +109,10 @@ export interface XcshApiToolDetails {
 	batchTotalItems?: number;
 	/** The resolved JSON body string sent to the API (after $F5XC_* expansion). */
 	resolvedPayload?: string;
+	/** Human-readable verb for mutation results. */
+	mutationVerb?: "Created" | "Updated" | "Deleted";
+	/** Human-readable resource label, e.g. "load balancer 'rm-headers'". */
+	resourceLabel?: string;
 }
 
 type XcshApiResult = AgentToolResult<XcshApiToolDetails> & { isError?: boolean };
@@ -638,6 +643,10 @@ export class XcshApiTool implements AgentTool<typeof xcshApiSchema, XcshApiToolD
 			}
 			case 409:
 				return `Resource already exists${ctxHint}. Use PUT to replace the existing resource, or DELETE it first before creating a new one.`;
+			case 400:
+				return `Bad request${ctxHint}. Check the payload for missing required fields or invalid values.`;
+			case 422:
+				return `Validation failed${ctxHint}. The request payload has field-level errors — check the message above.`;
 			case 429:
 				return `API rate limit exceeded${ctxHint}. Wait briefly and retry the request.`;
 			default:
@@ -817,13 +826,6 @@ export class XcshApiTool implements AgentTool<typeof xcshApiSchema, XcshApiToolD
 				// Label changes help the model echo type names in its response (Finding 13).
 				// INTERACTION EFFECT: this change + TCP protocol label are synergistic (Finding 31).
 				const pathSegs = resolvedPath.split("/").filter(Boolean);
-				const humanizeResourceType = (raw: string): string =>
-					raw
-						.replace(/^http_/, "")
-						.replace(/^tcp_/, "TCP ")
-						.replace(/_/g, " ")
-						.replace(/([a-z])(balancer|checker)/gi, "$1 $2")
-						.replace(/s$/, "");
 				let resourceLabel: string;
 				if (params.method === "POST") {
 					const rawType = pathSegs.at(-1) ?? "";
@@ -835,6 +837,9 @@ export class XcshApiTool implements AgentTool<typeof xcshApiSchema, XcshApiToolD
 					const rawType = pathSegs.at(-2) ?? "";
 					resourceLabel = name && rawType ? `${humanizeResourceType(rawType)} ${name}` : resolvedPath;
 				}
+				detail.mutationVerb = verb as "Created" | "Updated" | "Deleted";
+				detail.resourceLabel = resourceLabel;
+
 				// POST returns the full resource; PUT/DELETE return {}.
 				// Only claim response contains the resource for POST to avoid misleading the model.
 				const resourceHint =