@f5xc-salesdemos/xcsh 18.77.3 → 18.77.5

package/package.json

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

package/src/internal-urls/api-catalog-resolve.ts

@@ -249,6 +249,12 @@ function renderCatalogDetail(cat: ApiCatalogCategory, index: ApiCatalogIndex, op
 			sections.push(`Required fields: ${op.minimumPayload.requiredFields.join(", ")}`);
 			sections.push("", "```json", JSON.stringify(op.minimumPayload.json, null, 2), "```");
 		}
+		// Universal reference format hint — shown even in compact mode
+		if (op.method.toUpperCase() === "POST" || op.method.toUpperCase() === "PUT") {
+			sections.push(
+				'Object references: `{"namespace": "$F5XC_NAMESPACE", "name": "<name>"}`. Pool arrays: `[{"pool": {"namespace": "$F5XC_NAMESPACE", "name": "<name>"}, "weight": 1, "priority": 1}]`.',
+			);
+		}
 
 		// Field Constraints (Tier 2) — skipped in compact mode, deduped when identical across operations
 		if (op.fieldMetadata && Object.keys(op.fieldMetadata).length > 0 && !options?.compact) {

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

@@ -17,17 +17,17 @@ export interface BuildInfo {
 }
 
 export const BUILD_INFO: BuildInfo = {
-	"version": "18.77.3",
-	"commit": "97623491edfb9ed7a9da8c71c074d24d51a17a9d",
-	"shortCommit": "9762349",
+	"version": "18.77.5",
+	"commit": "9d5df6ccedc2df7a8dfd59c21489a8b208804e27",
+	"shortCommit": "9d5df6c",
 	"branch": "main",
-	"tag": "v18.77.3",
-	"commitDate": "2026-05-23T22:55:12Z",
-	"buildDate": "2026-05-23T23:14:49.552Z",
+	"tag": "v18.77.5",
+	"commitDate": "2026-05-24T00:50:34Z",
+	"buildDate": "2026-05-24T01:18:57.755Z",
 	"dirty": true,
 	"prNumber": "",
 	"repoUrl": "https://github.com/f5xc-salesdemos/xcsh",
 	"repoSlug": "f5xc-salesdemos/xcsh",
-	"commitUrl": "https://github.com/f5xc-salesdemos/xcsh/commit/97623491edfb9ed7a9da8c71c074d24d51a17a9d",
-	"releaseUrl": "https://github.com/f5xc-salesdemos/xcsh/releases/tag/v18.77.3"
+	"commitUrl": "https://github.com/f5xc-salesdemos/xcsh/commit/9d5df6ccedc2df7a8dfd59c21489a8b208804e27",
+	"releaseUrl": "https://github.com/f5xc-salesdemos/xcsh/releases/tag/v18.77.5"
 };

package/src/modes/components/assistant-message.ts

@@ -146,10 +146,13 @@ export class AssistantMessageComponent extends Container {
 					.some(c => (c.type === "text" && c.text.trim()) || (c.type === "thinking" && c.thinking.trim()));
 
 				if (this.hideThinkingBlock) {
-					// Show static "Thinking..." label when hidden
-					this.#contentContainer.addChild(new Text(theme.italic(theme.fg("thinkingText", "Thinking...")), 0, 0));
-					if (hasVisibleContentAfter) {
-						this.#contentContainer.addChild(new Spacer(1));
+					if (!message.stopReason) {
+						this.#contentContainer.addChild(
+							new Text(theme.italic(theme.fg("thinkingText", "Thinking...")), 0, 0),
+						);
+						if (hasVisibleContentAfter) {
+							this.#contentContainer.addChild(new Spacer(1));
+						}
 					}
 				} else {
 					// Thinking traces in thinkingText color, italic

package/src/modes/controllers/event-controller.ts

@@ -240,6 +240,9 @@ export class EventController {
 								? { ...content.arguments, __partialJson: content.partialJson }
 								: content.arguments;
 						if (!this.ctx.pendingTools.has(content.id)) {
+							if (content.name === "todo_write" && !settings.get("todo.verbose")) {
+								continue;
+							}
 							this.#resetReadGroup();
 							this.ctx.chatContainer.addChild(new Spacer(1));
 							const tool = this.ctx.session.getToolByName(content.name);

package/src/prompts/system/system-prompt.md

@@ -263,6 +263,8 @@ Most tools resolve custom protocol URLs to internal resources (not web URLs):
   Do not issue a follow-up GET to verify — the response body is the verification.
   Only issue a GET if the user explicitly asks to read current state, or if the
   initial call returned a non-2xx status.
+  For xcsh_api mutations, the 200 response satisfies the "verify the effect" requirement — do not GET the resource again.
+  For CREATE, you **MUST NOT** GET referenced dependencies (origin pools, firewalls) to verify they exist — include them by name. For UPDATE, GET the target resource for its current spec, but you **MUST NOT** GET other referenced resources.
 
   **Namespace discovery** — when the user asks what resources exist in a namespace
   (e.g. "what's in my namespace", "list everything configured", "show all resources"),

package/src/tools/xcsh-api.ts

@@ -502,7 +502,7 @@ export class XcshApiTool implements AgentTool<typeof xcshApiSchema, XcshApiToolD
 		}
 
 		sections.push(
-			"\nInventory complete. Resource summary above contains all relationship data. No further API calls needed.",
+			"\nInventory complete. All listed resources exist and are valid references for mutations. No further API calls needed.",
 		);
 		const batchTotalItems = relevantData.reduce((sum, r) => sum + (r.itemCount ?? 0), 0);
 		const text = sections.join("\n");
@@ -546,7 +546,7 @@ export class XcshApiTool implements AgentTool<typeof xcshApiSchema, XcshApiToolD
 				return `Access denied${ctxHint}. The API token may lack the required role or permission for this operation. Check the token's role assignments in the F5 XC console.`;
 			case 404: {
 				const ns = process.env.F5XC_NAMESPACE ?? this.#contextEnv.get("F5XC_NAMESPACE") ?? "default";
-				return `Resource not found in namespace \`${ns}\`${ctxHint}. Use a GET list operation to verify existing resources.`;
+				return `Resource not found in namespace \`${ns}\`${ctxHint}. Verify the resource name, or use POST to create it.`;
 			}
 			case 409:
 				return `Resource already exists${ctxHint}. Use PUT to replace the existing resource, or DELETE it first before creating a new one.`;
@@ -701,6 +701,33 @@ export class XcshApiTool implements AgentTool<typeof xcshApiSchema, XcshApiToolD
 				}
 				return this.#errorResult(`${statusLine}\n\n${bodyText}\n\n${errorCodePrefix}${guidance}`, detail);
 			}
+			// Mutation stop signals + cache invalidation: reduce post-mutation verification GETs
+			if (response.status >= 200 && response.status < 300 && params.method !== "GET") {
+				// Invalidate batch cache so subsequent namespace queries reflect the mutation
+				const nsMatch = resolvedPath.match(/\/api\/config\/namespaces\/([^/]+)\//);
+				if (nsMatch?.[1]) {
+					const nsCachePath = `${os.tmpdir()}/xcsh-batch-${nsMatch[1]}.json`;
+					await Bun.write(nsCachePath, JSON.stringify({ ts: 0 })).catch(() => {});
+				}
+				// Append stop signal to prevent unnecessary verification GETs
+				const verb = params.method === "DELETE" ? "Deleted" : params.method === "POST" ? "Created" : "Updated";
+				// POST returns the full resource; PUT/DELETE return {}.
+				// Only claim response contains the resource for POST to avoid misleading the model.
+				const resourceHint =
+					params.method === "POST"
+						? "The response above contains the complete resource. No verification GET is needed. Reference this resource by name in subsequent mutations immediately."
+						: "No verification GET is needed.";
+
+				return {
+					content: [
+						{
+							type: "text",
+							text: `${statusLine}\n\n${bodyText}\n\n${verb} ${resolvedPath} successfully. ${resourceHint}`,
+						},
+					],
+					details: detail,
+				};
+			}
 
 			return {
 				content: [{ type: "text", text: `${statusLine}\n\n${bodyText}` }],