@f5xc-salesdemos/xcsh 18.27.0 → 18.28.0

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@f5xc-salesdemos/xcsh",
-	"version": "18.27.0",
+	"version": "18.28.0",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://github.com/f5xc-salesdemos/xcsh",
 	"author": "Can Boluk",
@@ -47,12 +47,12 @@
 	"dependencies": {
 		"@agentclientprotocol/sdk": "0.16.1",
 		"@mozilla/readability": "^0.6",
-		"@f5xc-salesdemos/xcsh-stats": "18.27.0",
-		"@f5xc-salesdemos/pi-agent-core": "18.27.0",
-		"@f5xc-salesdemos/pi-ai": "18.27.0",
-		"@f5xc-salesdemos/pi-natives": "18.27.0",
-		"@f5xc-salesdemos/pi-tui": "18.27.0",
-		"@f5xc-salesdemos/pi-utils": "18.27.0",
+		"@f5xc-salesdemos/xcsh-stats": "18.28.0",
+		"@f5xc-salesdemos/pi-agent-core": "18.28.0",
+		"@f5xc-salesdemos/pi-ai": "18.28.0",
+		"@f5xc-salesdemos/pi-natives": "18.28.0",
+		"@f5xc-salesdemos/pi-tui": "18.28.0",
+		"@f5xc-salesdemos/pi-utils": "18.28.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.27.0",
-	"commit": "5c365003b8532b2252bde64eeb9cc751e927752d",
-	"shortCommit": "5c36500",
+	"version": "18.28.0",
+	"commit": "aefe3468aa6c23cfe1f6ac0575e011b2416389cf",
+	"shortCommit": "aefe346",
 	"branch": "main",
-	"tag": "v18.27.0",
-	"commitDate": "2026-04-29T19:40:39Z",
-	"buildDate": "2026-04-29T20:01:37.365Z",
+	"tag": "v18.28.0",
+	"commitDate": "2026-04-30T03:44:13Z",
+	"buildDate": "2026-04-30T04:02:30.628Z",
 	"dirty": false,
 	"prNumber": "",
 	"repoUrl": "https://github.com/f5xc-salesdemos/xcsh",
 	"repoSlug": "f5xc-salesdemos/xcsh",
-	"commitUrl": "https://github.com/f5xc-salesdemos/xcsh/commit/5c365003b8532b2252bde64eeb9cc751e927752d",
-	"releaseUrl": "https://github.com/f5xc-salesdemos/xcsh/releases/tag/v18.27.0"
+	"commitUrl": "https://github.com/f5xc-salesdemos/xcsh/commit/aefe3468aa6c23cfe1f6ac0575e011b2416389cf",
+	"releaseUrl": "https://github.com/f5xc-salesdemos/xcsh/releases/tag/v18.28.0"
 };

package/src/modes/components/welcome-checks.ts

@@ -14,7 +14,7 @@ const STARTUP_RETRY_DELAY_MS = 500;
 
 type ContextValidator = (opts: {
 	timeoutMs: number;
-}) => Promise<{ status: AuthStatus; latencyMs?: number; errorClass?: "network" | "credential" }>;
+}) => Promise<{ status: AuthStatus; latencyMs?: number; errorClass?: "network" | "credential" | "url_not_found" }>;
 
 /**
  * Runs the context validator once with a startup-sized timeout; if the result is `offline`
@@ -28,7 +28,7 @@ export async function validateContextWithStartupRetry(
 		retryTimeoutMs?: number;
 		retryDelayMs?: number;
 	},
-): Promise<{ status: AuthStatus; latencyMs?: number }> {
+): Promise<{ status: AuthStatus; latencyMs?: number; errorClass?: "network" | "credential" | "url_not_found" }> {
 	const firstTimeoutMs = options?.firstTimeoutMs ?? STARTUP_FIRST_TIMEOUT_MS;
 	const retryTimeoutMs = options?.retryTimeoutMs ?? STARTUP_RETRY_TIMEOUT_MS;
 	const retryDelayMs = options?.retryDelayMs ?? STARTUP_RETRY_DELAY_MS;
@@ -56,6 +56,7 @@ export interface WelcomeContextStatus {
 	state: ContextCheckState;
 	name?: string;
 	latencyMs?: number;
+	errorClass?: "network" | "credential" | "url_not_found";
 }
 
 export interface WelcomeCheckResult {
@@ -167,7 +168,7 @@ async function checkContextStatus(): Promise<WelcomeContextStatus> {
 			case "auth_error":
 				return { state: "auth_error", name };
 			case "offline":
-				return { state: "offline", name };
+				return { state: "offline", name, errorClass: result.errorClass };
 			default:
 				return { state: "no_context" };
 		}

package/src/modes/components/welcome.ts

@@ -232,6 +232,12 @@ export class WelcomeComponent implements Component {
 					`   ${theme.fg("dim", "Run /context to update")}`,
 				];
 			case "offline":
+				if (this.contextStatus?.errorClass === "url_not_found") {
+					return [
+						` ${formatStatusIcon("error")} ${theme.fg("muted", n)} ${theme.fg("error", "\u2014 tenant not found")}`,
+						`   ${theme.fg("dim", "Recreate with /context create or check with /context show")}`,
+					];
+				}
 				return [
 					` ${formatStatusIcon("warning")} ${theme.fg("muted", n)} ${theme.fg("warning", "\u2014 unreachable")}`,
 					`   ${theme.fg("dim", "Check network, /context")}`,

package/src/modes/controllers/context-command-controller.ts

@@ -1,3 +1,4 @@
+import { renderContextMessage } from "../../services/f5xc-table";
 import { ContextAddWizard } from "../components/context-add-wizard";
 import type { InteractiveModeContext } from "../types";
 
@@ -31,10 +32,11 @@ export class ContextCommandController {
 					const { ContextService } = await import("../../services/f5xc-context");
 					const service = await ContextService.getOrInit();
 					await service.createContext(context);
-					this.#ctx.showStatus(`Context '${context.name}' created.`);
 					if (shouldActivate) {
 						await service.activate(context.name);
-						this.#ctx.showStatus(`Context '${context.name}' activated.`);
+						this.#ctx.showStatus(renderContextMessage(context.name, "Created and activated."), { dim: false });
+					} else {
+						this.#ctx.showStatus(renderContextMessage(context.name, "Created."), { dim: false });
 					}
 					this.#ctx.statusLine?.invalidate();
 					this.#ctx.updateEditorTopBorder?.();

package/src/services/f5xc-context-command.ts

@@ -2,6 +2,7 @@ import * as fs from "node:fs";
 import { SECRET_ENV_PATTERNS } from "../secrets/index";
 import { expandTilde } from "../tools/path-utils";
 import { ContextError, ContextService, CURRENT_SCHEMA_VERSION } from "./f5xc-context";
+import { formatStatusIcon } from "./f5xc-context-indicators";
 import {
 	deriveTenantFromUrl,
 	F5XC_API_TOKEN,
@@ -16,12 +17,13 @@ import {
 	formatExpiration,
 	formatRelativeTime,
 	formatRotation,
+	renderContextMessage,
 	renderF5XCTable,
 	type TableRow,
 } from "./f5xc-table";
 
 interface CommandContext {
-	showStatus(msg: string): void;
+	showStatus(msg: string, options?: { dim?: boolean }): void;
 	showError(msg: string): void;
 	editor: { setText(text: string): void };
 	statusLine?: { invalidate(): void };
@@ -130,20 +132,30 @@ async function handleList(ctx: CommandContext, service: ContextService): Promise
 		const status = service.getStatus();
 		if (status.credentialSource === "environment" && status.activeContextUrl) {
 			const label = deriveTenantFromUrl(status.activeContextUrl) ?? "(environment)";
-			ctx.showStatus(`  * ${sanitize(label).padEnd(20)} ${sanitize(status.activeContextUrl)}  (via env vars)`);
+			ctx.showStatus(
+				renderContextMessage(
+					"contexts",
+					`  * ${sanitize(label)}  ${sanitize(status.activeContextUrl)}  (via env vars)`,
+				),
+				{ dim: false },
+			);
 			return;
 		}
-		ctx.showStatus("No F5 XC contexts found. Use /context create or ask me to help set one up.");
+		ctx.showStatus(
+			renderContextMessage("contexts", "No F5 XC contexts found. Use /context create or ask me to help set one up."),
+			{ dim: false },
+		);
 		return;
 	}
 	const status = service.getStatus();
-	const lines = contexts.map(p => {
-		const marker = p.name === status.activeContextName ? "*" : " ";
+	const rows: TableRow[] = contexts.map(p => {
+		const isActive = p.name === status.activeContextName;
+		const marker = isActive ? `${formatStatusIcon("connected")} ` : "  ";
 		const versionSuffix =
 			p.version !== undefined && p.version > CURRENT_SCHEMA_VERSION ? ` (v${p.version} — upgrade required)` : "";
-		return `  ${marker} ${sanitize(p.name).padEnd(20)} ${sanitize(p.apiUrl)}${versionSuffix}`;
+		return { key: `${marker}${sanitize(p.name)}`, value: `${sanitize(p.apiUrl)}${versionSuffix}` };
 	});
-	ctx.showStatus(lines.join("\n"));
+	ctx.showStatus(renderF5XCTable("contexts", rows), { dim: false });
 }
 
 async function handleActivate(ctx: CommandContext, service: ContextService, name: string): Promise<void> {
@@ -286,7 +298,7 @@ async function handleShow(ctx: CommandContext, service: ContextService, name?: s
 		rows.push(...metaRows);
 	}
 
-	ctx.showStatus(renderF5XCTable(context.name, rows, { dividers }));
+	ctx.showStatus(renderF5XCTable(context.name, rows, { dividers }), { dim: false });
 }
 
 async function handleValidate(ctx: CommandContext, service: ContextService, name: string): Promise<void> {
@@ -305,7 +317,7 @@ async function handleValidate(ctx: CommandContext, service: ContextService, name
 			{ key: F5XC_API_TOKEN, value: service.maskToken(result.context.apiToken) },
 			{ key: "Status", value: formatAuthIndicator(result.status, result.latencyMs, result.errorClass) },
 		];
-		ctx.showStatus(renderF5XCTable(`${result.context.name} (validation only)`, rows));
+		ctx.showStatus(renderF5XCTable(`${result.context.name} (validation only)`, rows), { dim: false });
 	} catch (err) {
 		ctx.showError(err instanceof ContextError ? err.message : String(err));
 	}
@@ -314,7 +326,10 @@ async function handleValidate(ctx: CommandContext, service: ContextService, name
 async function handleStatus(ctx: CommandContext, service: ContextService): Promise<void> {
 	const status = service.getStatus();
 	if (!status.isConfigured) {
-		ctx.showStatus("F5 XC: not configured. Use /context create or ask me to help set one up.");
+		ctx.showStatus(
+			renderContextMessage("status", "Not configured. Use /context create or ask me to help set one up."),
+			{ dim: false },
+		);
 		return;
 	}
 	const auth = await service.validateToken({ timeoutMs: 3000 });
@@ -325,7 +340,7 @@ async function handleStatus(ctx: CommandContext, service: ContextService): Promi
 		{ key: "Namespace", value: status.activeContextNamespace ?? "(not set)" },
 		{ key: "Status", value: formatAuthIndicator(auth.status, auth.latencyMs, auth.errorClass) },
 	];
-	ctx.showStatus(renderF5XCTable(status.activeContextName ?? "status", rows));
+	ctx.showStatus(renderF5XCTable(status.activeContextName ?? "status", rows), { dim: false });
 }
 
 async function handleCreate(ctx: CommandContext, service: ContextService, args: string[]): Promise<void> {
@@ -355,7 +370,9 @@ async function handleCreate(ctx: CommandContext, service: ContextService, args:
 			apiToken: token,
 			defaultNamespace: namespace ?? "default",
 		});
-		ctx.showStatus(`Context '${name}' created. Use /context activate ${name} to switch to it.`);
+		ctx.showStatus(renderContextMessage(name, `Created. Use /context activate ${name} to switch to it.`), {
+			dim: false,
+		});
 	} catch (err) {
 		ctx.showError(err instanceof ContextError ? err.message : String(err));
 	}
@@ -369,7 +386,7 @@ async function handleRename(ctx: CommandContext, service: ContextService, args:
 	}
 	try {
 		await service.renameContext(oldName, newName);
-		ctx.showStatus(`Context '${oldName}' renamed to '${newName}'.`);
+		ctx.showStatus(renderContextMessage(newName, `Renamed from '${oldName}'.`), { dim: false });
 		ctx.statusLine?.invalidate();
 		ctx.updateEditorTopBorder?.();
 		ctx.ui?.requestRender();
@@ -452,13 +469,13 @@ async function handleImport(ctx: CommandContext, service: ContextService, rawArg
 
 	try {
 		const result = await service.importContexts(parsed, { overwrite });
-		const lines: string[] = [];
-		lines.push(`Imported ${result.imported.length} context${result.imported.length === 1 ? "" : "s"}:`);
-		for (const name of result.imported) lines.push(`  + ${name}`);
+		const bodyLines: string[] = [];
+		bodyLines.push(`Imported ${result.imported.length} context${result.imported.length === 1 ? "" : "s"}:`);
+		for (const name of result.imported) bodyLines.push(`  + ${name}`);
 		if (result.overwritten.length > 0) {
-			lines.push(`Overwrote ${result.overwritten.length}: ${result.overwritten.join(", ")}`);
+			bodyLines.push(`Overwrote ${result.overwritten.length}: ${result.overwritten.join(", ")}`);
 		}
-		ctx.showStatus(lines.join("\n"));
+		ctx.showStatus(renderContextMessage("import", bodyLines.join("\n")), { dim: false });
 		// Invalidate TUI chrome IF the active context was overwritten. The
 		// service's importContexts re-activates the active context when an
 		// overwrite touches it, which means #activeContext, bash.environment,
@@ -491,13 +508,17 @@ async function handleDelete(ctx: CommandContext, service: ContextService, args:
 	}
 	if (!confirmed) {
 		ctx.showStatus(
-			`This will permanently delete context '${name}' from ~/.config/f5xc/contexts/.\nRun /context delete ${name} --confirm to proceed.`,
+			renderContextMessage(
+				name,
+				`This will permanently delete context '${name}' from ~/.config/f5xc/contexts/.\nRun /context delete ${name} --confirm to proceed.`,
+			),
+			{ dim: false },
 		);
 		return;
 	}
 	try {
 		await service.deleteContext(name);
-		ctx.showStatus(`Context '${name}' deleted.`);
+		ctx.showStatus(renderContextMessage(name, "Deleted."), { dim: false });
 	} catch (err) {
 		ctx.showError(err instanceof ContextError ? err.message : String(err));
 	}
@@ -512,7 +533,7 @@ async function handleNamespace(ctx: CommandContext, service: ContextService, nam
 	}
 	try {
 		service.setNamespace(namespace);
-		ctx.showStatus(`Namespace switched to: ${namespace}`);
+		ctx.showStatus(renderContextMessage("namespace", `Namespace → ${namespace}`), { dim: false });
 		ctx.statusLine?.invalidate();
 		ctx.updateEditorTopBorder?.();
 		ctx.ui?.requestRender();
@@ -597,7 +618,7 @@ async function handleEnvList(ctx: CommandContext, service: ContextService): Prom
 	const contexts = await service.listContexts();
 	const context = contexts.find(p => p.name === contextName);
 	if (!context?.env || Object.keys(context.env).length === 0) {
-		ctx.showStatus(`Context '${contextName}' has no custom environment variables.`);
+		ctx.showStatus(renderContextMessage(`${contextName} env`, "No custom environment variables."), { dim: false });
 		return;
 	}
 	const rows: TableRow[] = [];
@@ -605,7 +626,7 @@ async function handleEnvList(ctx: CommandContext, service: ContextService): Prom
 		const sensitive = isSensitiveKey(key) || (context.sensitiveKeys ?? []).includes(key);
 		rows.push({ key: sanitize(key), value: sensitive ? service.maskToken(value) : sanitize(value) });
 	}
-	ctx.showStatus(renderF5XCTable(`${contextName} env`, rows));
+	ctx.showStatus(renderF5XCTable(`${contextName} env`, rows), { dim: false });
 }
 
 async function handleEnvSet(ctx: CommandContext, service: ContextService, args: string): Promise<void> {
@@ -623,15 +644,14 @@ async function handleEnvSet(ctx: CommandContext, service: ContextService, args:
 	}
 	try {
 		const result = await service.setEnvVars(contextName, vars);
-		const lines: string[] = [];
+		const bodyLines: string[] = [];
+		bodyLines.push(`Set ${keys.length} variable${keys.length > 1 ? "s" : ""} on '${contextName}':`);
 		for (const key of keys) {
 			const lock = result.sensitive.includes(key) ? " (auto-sensitive)" : "";
 			const displayValue = isSensitiveKey(key) ? "***" : vars[key];
-			lines.push(`  ${key}=${displayValue}${lock}`);
+			bodyLines.push(`  ${key}=${displayValue}${lock}`);
 		}
-		ctx.showStatus(
-			`Set ${keys.length} variable${keys.length > 1 ? "s" : ""} on '${contextName}':\n${lines.join("\n")}`,
-		);
+		ctx.showStatus(renderContextMessage(contextName, bodyLines.join("\n")), { dim: false });
 		ctx.statusLine?.invalidate();
 	} catch (err) {
 		ctx.showError(err instanceof ContextError ? err.message : String(err));
@@ -653,11 +673,15 @@ async function handleEnvUnset(ctx: CommandContext, service: ContextService, args
 	try {
 		const result = await service.unsetEnvVars(contextName, keys);
 		if (result.removed.length === 0) {
-			ctx.showStatus(`No matching variables found on '${contextName}'.`);
+			ctx.showStatus(renderContextMessage(contextName, "No matching variables found."), { dim: false });
 			return;
 		}
 		ctx.showStatus(
-			`Removed ${result.removed.length} variable${result.removed.length > 1 ? "s" : ""} from '${contextName}':\n${result.removed.map(k => `  ${k}`).join("\n")}`,
+			renderContextMessage(
+				contextName,
+				`Removed ${result.removed.length} variable${result.removed.length > 1 ? "s" : ""} from '${contextName}':\n${result.removed.map(k => `  ${k}`).join("\n")}`,
+			),
+			{ dim: false },
 		);
 		ctx.statusLine?.invalidate();
 	} catch (err) {

package/src/services/f5xc-context.ts

@@ -114,7 +114,7 @@ export interface ValidationResult {
 	context: F5XCContext;
 	status: AuthStatus;
 	latencyMs?: number;
-	errorClass?: "network" | "credential";
+	errorClass?: "network" | "credential" | "url_not_found";
 }
 
 export class ContextError extends Error {
@@ -915,7 +915,7 @@ export class ContextService {
 		timeoutMs?: number;
 		apiUrl?: string;
 		apiToken?: string;
-	}): Promise<{ status: AuthStatus; latencyMs?: number; errorClass?: "network" | "credential" }> {
+	}): Promise<{ status: AuthStatus; latencyMs?: number; errorClass?: "network" | "credential" | "url_not_found" }> {
 		// Use explicit credentials if provided (for non-active contexts or env-backed sessions),
 		// otherwise fall back to effective credentials (env override > active context)
 		const effectiveUrl = options?.apiUrl ?? process.env[F5XC_API_URL] ?? this.#activeContext?.apiUrl;
@@ -946,13 +946,23 @@ export class ContextService {
 				method: "GET",
 				headers: { Authorization: `APIToken ${effectiveToken}`, Accept: "application/json" },
 				signal: AbortSignal.timeout(timeout),
+				redirect: "manual",
 			});
 			const latencyMs = Math.round(performance.now() - start);
 			if (!adHoc) {
 				this.#lastAuthLatencyMs = latencyMs;
 				this.#lastAuthCheckedAt = checkedAt;
 			}
+			if (response.type === "opaqueredirect" || (response.status >= 300 && response.status < 400)) {
+				if (!adHoc) this.#authStatus = "offline";
+				return { status: "offline", latencyMs, errorClass: "url_not_found" };
+			}
 			if (response.ok) {
+				const contentType = response.headers.get("content-type") ?? "";
+				if (!contentType.includes("application/json")) {
+					if (!adHoc) this.#authStatus = "offline";
+					return { status: "offline", latencyMs, errorClass: "url_not_found" };
+				}
 				if (!adHoc) this.#authStatus = "connected";
 				return { status: "connected", latencyMs };
 			}

package/src/services/f5xc-table.ts

@@ -23,7 +23,7 @@ const r = (s: string) => `${F5_RED}${s}${RESET}`;
 export function formatAuthIndicator(
 	status: AuthStatus,
 	latencyMs?: number,
-	errorClass?: "network" | "credential",
+	errorClass?: "network" | "credential" | "url_not_found",
 ): string {
 	const ms = latencyMs !== undefined ? ` (${latencyMs}ms)` : "";
 	switch (status) {
@@ -32,6 +32,9 @@ export function formatAuthIndicator(
 		case "auth_error":
 			return `${formatStatusIcon("error")} Auth Error — check token${ms}`;
 		case "offline":
+			if (errorClass === "url_not_found") {
+				return `${formatStatusIcon("error")} Offline — tenant URL not found${ms}`;
+			}
 			return `${formatStatusIcon("warning")} Offline — ${errorClass === "credential" ? "auth issue" : "network issue"}${ms}`;
 		default:
 			return `${formatStatusIcon("unknown")} Unknown`;
@@ -138,3 +141,24 @@ export function renderF5XCTable(title: string, rows: TableRow[], options?: Table
 
 	return lines.join("\n");
 }
+
+export function renderContextMessage(title: string, body: string): string {
+	const bodyLines = body.split("\n");
+	const maxLine = Math.max(...bodyLines.map(l => visibleWidth(l)), 0);
+	const innerWidth = Math.max(maxLine + 2, visibleWidth(title) + 3, 40);
+
+	const lines: string[] = [];
+
+	const titleText = ` ${title} `;
+	const titlePad = innerWidth - visibleWidth(titleText) - 1;
+	lines.push(`${r(BOX.tl + BOX.h)}${BOLD}${titleText}${RESET}${r(BOX.h.repeat(Math.max(0, titlePad)) + BOX.tr)}`);
+
+	for (const bodyLine of bodyLines) {
+		const pad = innerWidth - visibleWidth(bodyLine) - 2;
+		lines.push(`${r(BOX.v)} ${bodyLine}${" ".repeat(Math.max(0, pad))} ${r(BOX.v)}`);
+	}
+
+	lines.push(r(BOX.bl + BOX.h.repeat(innerWidth) + BOX.br));
+
+	return lines.join("\n");
+}