@f5xc-salesdemos/xcsh 18.79.0 → 18.81.0

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@f5xc-salesdemos/xcsh",
-	"version": "18.79.0",
+	"version": "18.81.0",
 	"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.79.0",
-		"@f5xc-salesdemos/pi-agent-core": "18.79.0",
-		"@f5xc-salesdemos/pi-ai": "18.79.0",
-		"@f5xc-salesdemos/pi-natives": "18.79.0",
-		"@f5xc-salesdemos/pi-tui": "18.79.0",
-		"@f5xc-salesdemos/pi-utils": "18.79.0",
+		"@f5xc-salesdemos/xcsh-stats": "18.81.0",
+		"@f5xc-salesdemos/pi-agent-core": "18.81.0",
+		"@f5xc-salesdemos/pi-ai": "18.81.0",
+		"@f5xc-salesdemos/pi-natives": "18.81.0",
+		"@f5xc-salesdemos/pi-tui": "18.81.0",
+		"@f5xc-salesdemos/pi-utils": "18.81.0",
 		"@sinclair/typebox": "^0.34",
 		"@xterm/headless": "^6.0",
 		"ajv": "^8.18",

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

@@ -17,17 +17,17 @@ export interface BuildInfo {
 }
 
 export const BUILD_INFO: BuildInfo = {
-	"version": "18.79.0",
-	"commit": "cc280f31122399fcfa7cde248777d0430b721bc7",
-	"shortCommit": "cc280f3",
+	"version": "18.81.0",
+	"commit": "042ddfbad1fad8132b3dc9a1eb0190ac17b2d22d",
+	"shortCommit": "042ddfb",
 	"branch": "main",
-	"tag": "v18.79.0",
-	"commitDate": "2026-05-24T21:32:29Z",
-	"buildDate": "2026-05-24T21:50:49.956Z",
+	"tag": "v18.81.0",
+	"commitDate": "2026-05-25T15:40:42Z",
+	"buildDate": "2026-05-25T16:05:58.171Z",
 	"dirty": true,
 	"prNumber": "",
 	"repoUrl": "https://github.com/f5xc-salesdemos/xcsh",
 	"repoSlug": "f5xc-salesdemos/xcsh",
-	"commitUrl": "https://github.com/f5xc-salesdemos/xcsh/commit/cc280f31122399fcfa7cde248777d0430b721bc7",
-	"releaseUrl": "https://github.com/f5xc-salesdemos/xcsh/releases/tag/v18.79.0"
+	"commitUrl": "https://github.com/f5xc-salesdemos/xcsh/commit/042ddfbad1fad8132b3dc9a1eb0190ac17b2d22d",
+	"releaseUrl": "https://github.com/f5xc-salesdemos/xcsh/releases/tag/v18.81.0"
 };

package/src/tools/xcsh-api.ts

@@ -6,9 +6,7 @@ import xcshApiDescription from "../prompts/tools/xcsh-api.md" with { type: "text
 import { type ContextEnv, createContextEnv } from "../services/context-env";
 import type { ToolSession } from ".";
 
-// ── Spec-driven namespace filtering ────────────────────────────────
-// Replaces hardcoded startsWith("ves-io")/startsWith("system")/startsWith("shared")
-// with data-driven classification aligned to x-f5xc-namespace-profile extension.
+// Namespace filtering driven by x-f5xc-namespace-profile from enriched API specs.
 
 type NamespaceType = "system" | "shared" | "default" | "custom";
 
@@ -133,6 +131,7 @@ export class XcshApiTool implements AgentTool<typeof xcshApiSchema, XcshApiToolD
 	#contextEnv: ContextEnv;
 	#lastApiBase = "";
 	#listablePathsCache: string[] | null = null;
+	#autoExpandPathsCache: string[] | null = null;
 	#expandedNamespaces = new Set<string>();
 
 	constructor(session: ToolSession) {
@@ -180,7 +179,11 @@ export class XcshApiTool implements AgentTool<typeof xcshApiSchema, XcshApiToolD
 			const CONFIG_PREFIX = "/api/config/namespaces/{namespace}/";
 			// Only include app/security types (keyword filter). Reduces batch from ~136 to ~42 paths,
 			// cutting expansion time by ~3× and eliminating infrastructure noise from the response.
+			// Healthcheck is included in batch content (for HC labels) but NOT in the auto-expand
+			// trigger list — direct GET to /healthchecks should not trigger a full namespace expansion.
 			const APP_KW =
+				/loadbalancer|pool|firewall|healthcheck|_policys|setting|type|mitigation|identification|network|route|host|definition|rate_limiter|prefix_set|cdn|waf|api_/i;
+			const AUTO_EXPAND_KW =
 				/loadbalancer|pool|firewall|_policys|setting|type|mitigation|identification|network|route|host|definition|rate_limiter|prefix_set|cdn|waf|api_/i;
 			const META_EXCL = /policy_set|policy_rule|data_polic/i;
 			for (const summary of summaries) {
@@ -200,6 +203,11 @@ export class XcshApiTool implements AgentTool<typeof xcshApiSchema, XcshApiToolD
 					}
 				}
 			}
+			// Build separate auto-expand trigger list (excludes healthcheck)
+			this.#autoExpandPathsCache = paths.filter(p => {
+				const last = p.split("/").filter(Boolean).at(-1) ?? "";
+				return AUTO_EXPAND_KW.test(last);
+			});
 			this.#listablePathsCache = paths;
 			return paths;
 		} catch {
@@ -419,7 +427,7 @@ export class XcshApiTool implements AgentTool<typeof xcshApiSchema, XcshApiToolD
 		// Only fetch specs for types that carry relationship data (LBs, pools, firewalls,
 		// healthchecks). Excludes system-generated objects (routes, virtual_hosts, etc.)
 		// that inflate the item count and don't add relationship info.
-		const SPEC_TYPES = /loadbalancer|origin_pool|app_firewall|healthcheck/i;
+		const SPEC_TYPES = /loadbalancer|origin_pool|app_firewall|healthcheck|rate_limiter/i;
 		for (const r of relevantData) {
 			const typeName = r.path.split("/").pop() ?? "";
 			if (!SPEC_TYPES.test(typeName)) continue;
@@ -459,15 +467,23 @@ export class XcshApiTool implements AgentTool<typeof xcshApiSchema, XcshApiToolD
 		// Core types get detailed per-resource output with spec summaries.
 		// Secondary types get a compact count to reduce batch response noise.
 		// Shorter, focused output helps the model find relationship data directly.
-		const getTypeName = (r: BatchEntry) => r.path.split("/").pop() ?? r.path;
-		const coreTypes = relevantData.filter(r => SPEC_TYPES.test(getTypeName(r)));
-		const secondaryTypes = relevantData.filter(r => !SPEC_TYPES.test(getTypeName(r)));
+		const getRawTypeName = (r: BatchEntry) => r.path.split("/").pop() ?? r.path;
+		// Human-readable type name for display: http_loadbalancers → load balancers, origin_pools → origin pools
+		const humanizeType = (raw: string): string =>
+			raw
+				.replace(/^http_/, "")
+				.replace(/^tcp_/, "TCP ")
+				.replace(/_/g, " ")
+				.replace(/([a-z])([A-Z])/g, "$1 $2")
+				.replace(/([a-z])(balancer|checker)/gi, "$1 $2");
+		const coreTypes = relevantData.filter(r => SPEC_TYPES.test(getRawTypeName(r)));
+		const secondaryTypes = relevantData.filter(r => !SPEC_TYPES.test(getRawTypeName(r)));
 
 		if (coreTypes.length > 0) {
 			sections.push(`Namespace resource inventory (${coreTypes.length} core types):\n`);
 			let idx = 1;
 			for (const r of coreTypes) {
-				const typeName = getTypeName(r);
+				const typeName = humanizeType(getRawTypeName(r));
 				const items = (r.parsed?.items as Array<Record<string, unknown>> | undefined) ?? [];
 				if (items.length === 0) {
 					sections.push(`${idx}. ${typeName}: ${r.itemCount} item(s)`);
@@ -486,7 +502,7 @@ export class XcshApiTool implements AgentTool<typeof xcshApiSchema, XcshApiToolD
 		if (secondaryTypes.length > 0) {
 			const secondaryCount = secondaryTypes.reduce((sum, r) => sum + (r.itemCount ?? 0), 0);
 			sections.push(
-				`\n(+${secondaryTypes.length} other types with ${secondaryCount} items: ${secondaryTypes.map(r => getTypeName(r)).join(", ")})`,
+				`\n(+${secondaryTypes.length} other types with ${secondaryCount} items: ${secondaryTypes.map(r => humanizeType(getRawTypeName(r))).join(", ")})`,
 			);
 		}
 
@@ -516,7 +532,13 @@ export class XcshApiTool implements AgentTool<typeof xcshApiSchema, XcshApiToolD
 			const rName = parts.at(-1) ?? "";
 			const rType = parts.at(-2) ?? "";
 			const labels: string[] = [];
-			if (/loadbalancer/i.test(rType)) {
+			if (/tcp_loadbalancer/i.test(rType)) {
+				const pools = extractPoolRefs(spec.origin_pools_weights);
+				if (pools.length > 0) labels.push(`pools=[${pools.join(",")}]`);
+				if (typeof spec.listen_port === "number") labels.push(`port=${spec.listen_port}`);
+				const domains = Array.isArray(spec.domains) ? (spec.domains as string[]) : [];
+				if (domains.length > 0) labels.push(`domains=[${domains.join(",")}]`);
+			} else if (/loadbalancer/i.test(rType)) {
 				const waf = extractRef(spec.app_firewall);
 				labels.push(waf ? `WAF=${waf}` : "no-WAF");
 				const pools = extractPoolRefs(spec.default_route_pools);
@@ -540,6 +562,22 @@ export class XcshApiTool implements AgentTool<typeof xcshApiSchema, XcshApiToolD
 				if (spec.monitoring != null) labels.push("monitoring-mode");
 				else if (spec.blocking != null) labels.push("blocking-mode");
 			}
+			if (/healthcheck/i.test(rType)) {
+				if (spec.http_health_check != null) {
+					labels.push("http");
+					const httpHC = spec.http_health_check as Record<string, unknown>;
+					if (typeof httpHC.path === "string") labels.push(`path=${httpHC.path}`);
+				} else if (spec.tcp_health_check != null) {
+					labels.push("tcp");
+				}
+			}
+			if (/rate_limiter/i.test(rType)) {
+				if (typeof spec.total_number === "number") labels.push(`threshold=${spec.total_number}`);
+				if (typeof spec.burst_size === "number") labels.push(`burst=${spec.burst_size}`);
+				if (typeof spec.committed_information_rate === "number")
+					labels.push(`rate=${spec.committed_information_rate}`);
+				if (typeof spec.unit === "string") labels.push(`unit=${spec.unit}`);
+			}
 			if (labels.length > 0) {
 				summaryLines.push(`${rName}: ${labels.join(", ")}`);
 			}
@@ -642,12 +680,15 @@ export class XcshApiTool implements AgentTool<typeof xcshApiSchema, XcshApiToolD
 		// File-based cache in #executeBatch prevents redundant API calls across sessions.
 		if (params.method === "GET" && !params.payload) {
 			const listablePaths = this.#loadListablePaths();
-			if (listablePaths.length > 0) {
+			// Use the auto-expand trigger list (excludes healthcheck) to decide WHETHER to expand.
+			// The batch itself uses the full listablePaths (includes healthcheck) for content.
+			const triggerPaths = this.#autoExpandPathsCache ?? listablePaths;
+			if (triggerPaths.length > 0) {
 				const normalized = params.path.replace(
 					/\/api\/config\/namespaces\/[^/]+\//,
 					"/api/config/namespaces/{namespace}/",
 				);
-				if (listablePaths.includes(params.path) || listablePaths.includes(normalized)) {
+				if (triggerPaths.includes(params.path) || triggerPaths.includes(normalized)) {
 					const nsMatch = params.path.match(/\/api\/config\/namespaces\/([^/]+)\//);
 					const ns = params.params?.namespace ?? (nsMatch?.[1] && nsMatch[1] !== "{namespace}" ? nsMatch[1] : "");
 					if (ns && !this.#expandedNamespaces.has(ns)) {