@f5xc-salesdemos/xcsh 18.90.0 → 18.91.1

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## [Unreleased]
 
+## [18.91.0] - 2026-06-01
+
+### Changed
+
+- Welcome screen fully plugin-driven: removed all hardcoded cloud service stubs (`getFixableServices` empty stub, `checkProfileStatus` dead code). Only F5 XC Context remains as a built-in service. All cloud connector status checks now come exclusively from the Extension API `registerServiceStatus()`. ([#1076](https://github.com/f5xc-salesdemos/xcsh/issues/1076))
+
 ## [18.90.0] - 2026-06-01
 
 ### Added

package/package.json

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

package/src/extensibility/extensions/types.ts

@@ -939,6 +939,7 @@ export interface RegisteredCommand {
 
 export interface ServiceStatusContribution {
 	name: string;
+	group?: string;
 	check: () => Promise<{ state: "connected" | "unauthenticated" | "unavailable"; hint?: string }>;
 	fix?: {
 		prompt: string;

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

@@ -17,17 +17,17 @@ export interface BuildInfo {
 }
 
 export const BUILD_INFO: BuildInfo = {
-	"version": "18.90.0",
-	"commit": "b39669e1b23f62c71f7feb98e1881444f415a783",
-	"shortCommit": "b39669e",
+	"version": "18.91.1",
+	"commit": "722eac01533203da9bd0bd3ddafb10c38f851d5c",
+	"shortCommit": "722eac0",
 	"branch": "main",
-	"tag": "v18.90.0",
-	"commitDate": "2026-06-01T02:24:32Z",
-	"buildDate": "2026-06-01T02:46:05.746Z",
+	"tag": "v18.91.1",
+	"commitDate": "2026-06-01T21:58:38Z",
+	"buildDate": "2026-06-01T22:19:50.728Z",
 	"dirty": true,
 	"prNumber": "",
 	"repoUrl": "https://github.com/f5xc-salesdemos/xcsh",
 	"repoSlug": "f5xc-salesdemos/xcsh",
-	"commitUrl": "https://github.com/f5xc-salesdemos/xcsh/commit/b39669e1b23f62c71f7feb98e1881444f415a783",
-	"releaseUrl": "https://github.com/f5xc-salesdemos/xcsh/releases/tag/v18.90.0"
+	"commitUrl": "https://github.com/f5xc-salesdemos/xcsh/commit/722eac01533203da9bd0bd3ddafb10c38f851d5c",
+	"releaseUrl": "https://github.com/f5xc-salesdemos/xcsh/releases/tag/v18.91.1"
 };

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

@@ -1,7 +1,6 @@
 import type { Model } from "@f5xc-salesdemos/pi-ai";
 import { validateApiKeyAgainstModelsEndpoint } from "@f5xc-salesdemos/pi-ai/utils/oauth/api-key-validation";
 import { logger } from "@f5xc-salesdemos/pi-utils";
-import { loadProfile } from "../../internal-urls/user-profile";
 import { type AuthStatus, ContextService } from "../../services/f5xc-context";
 import { deriveTenantFromUrl } from "../../services/f5xc-env";
 import type { AuthStorage } from "../../session/auth-storage";
@@ -185,6 +184,8 @@ export interface ServiceStatus {
 	name: string;
 	state: ServiceState;
 	hint?: string;
+	_isPlugin?: boolean;
+	_group?: string;
 }
 
 export function mapContextStatus(status: WelcomeContextStatus): ServiceStatus {
@@ -207,54 +208,9 @@ export function mapContextStatus(status: WelcomeContextStatus): ServiceStatus {
 	}
 }
 
-export type ProfileCheckState = "current" | "stale" | "missing";
-
-export interface WelcomeProfileStatus {
-	state: ProfileCheckState;
-	name?: string;
-	updatedAt?: string;
-	staleDays?: number;
-}
-
-const PROFILE_STALE_HOURS = 24;
-
-/** Check user profile freshness. Returns undefined only on unexpected error. */
-export async function checkProfileStatus(): Promise<WelcomeProfileStatus | undefined> {
-	try {
-		const profile = await loadProfile();
-		if (!profile.givenName && !profile.familyName) {
-			return { state: "missing" };
-		}
-		const name = [profile.givenName, profile.familyName].filter(Boolean).join(" ");
-		const updatedAt = profile.updatedAt;
-
-		if (!updatedAt) {
-			return { state: "stale", name };
-		}
-
-		const ageHours = (Date.now() - new Date(updatedAt).getTime()) / (1000 * 60 * 60);
-		if (ageHours > PROFILE_STALE_HOURS) {
-			return {
-				state: "stale",
-				name,
-				updatedAt,
-				staleDays: Math.floor(ageHours / 24),
-			};
-		}
-
-		return { state: "current", name, updatedAt };
-	} catch {
-		return { state: "missing" };
-	}
-}
-
 export interface FixableService {
 	name: string;
 	prompt: string;
 	command: string[];
 	recheck: () => Promise<ServiceStatus>;
 }
-
-export function getFixableServices(): FixableService[] {
-	return [];
-}

package/src/modes/components/welcome.ts

@@ -122,15 +122,37 @@ export class WelcomeComponent implements Component {
 
 	#measureStatusWidth(): number {
 		const lines: string[] = [" Model Provider", ...this.#renderModelStatus()];
-		for (const svc of this.services) {
+		const coreServices = this.services.filter(s => !s._isPlugin);
+		const pluginServices = this.services.filter(s => s._isPlugin);
+		for (const svc of coreServices) {
 			lines.push(this.#renderServiceLine(svc));
 		}
+		if (pluginServices.length > 0) {
+			const groups = this.#groupPluginServices(pluginServices);
+			for (const [groupName] of groups) {
+				lines.push(` ${groupName}`);
+			}
+			for (const svc of pluginServices) {
+				lines.push(this.#renderServiceLine(svc));
+			}
+		}
 		if (this.#showUpdateSection()) {
 			lines.push(this.#renderUpdateLine());
 		}
 		return Math.max(...lines.map(l => visibleWidth(l)));
 	}
 
+	#groupPluginServices(plugins: ServiceStatus[]): Map<string, ServiceStatus[]> {
+		const groups = new Map<string, ServiceStatus[]>();
+		for (const svc of plugins) {
+			const groupName = svc._group ?? "Plugins";
+			const list = groups.get(groupName) ?? [];
+			list.push(svc);
+			groups.set(groupName, list);
+		}
+		return groups;
+	}
+
 	#buildStatusLines(rightCol: number): string[] {
 		const lines: string[] = [];
 		const separatorWidth = Math.max(0, rightCol - 2);
@@ -139,11 +161,24 @@ export class WelcomeComponent implements Component {
 		lines.push(` ${theme.bold(theme.fg("contentAccent", "Model Provider"))}`);
 		lines.push(...this.#renderModelStatus());
 		lines.push("");
-		if (this.services.length > 0 || this.#showUpdateSection()) {
+		const coreServices = this.services.filter(s => !s._isPlugin);
+		const pluginServices = this.services.filter(s => s._isPlugin);
+		const hasContent = coreServices.length > 0 || pluginServices.length > 0 || this.#showUpdateSection();
+		if (hasContent) {
 			lines.push(separator);
-			for (const svc of this.services) {
+			for (const svc of coreServices) {
 				lines.push(this.#renderServiceLine(svc));
 			}
+			if (pluginServices.length > 0) {
+				const groups = this.#groupPluginServices(pluginServices);
+				for (const [groupName, groupServices] of groups) {
+					lines.push("");
+					lines.push(` ${theme.fg("dim", groupName)}`);
+					for (const svc of groupServices) {
+						lines.push(this.#renderServiceLine(svc));
+					}
+				}
+			}
 			if (this.#showUpdateSection()) {
 				lines.push(this.#renderUpdateLine());
 			}

package/src/modes/interactive-mode.ts

@@ -54,7 +54,6 @@ import type { ToolExecutionHandle } from "./components/tool-execution";
 import { type UpdateStatus, WelcomeComponent } from "./components/welcome";
 import {
 	type FixableService,
-	getFixableServices,
 	mapContextStatus,
 	runWelcomeChecks,
 	type ServiceStatus,
@@ -346,15 +345,20 @@ export class InteractiveMode implements InteractiveModeContext {
 			for (const contribution of pluginContributions) {
 				try {
 					const status = await contribution.check();
-					services.push({ name: contribution.name, ...status });
+					services.push({ name: contribution.name, ...status, _isPlugin: true, _group: contribution.group });
 				} catch {
-					services.push({ name: contribution.name, state: "unavailable", hint: "check failed" });
+					services.push({
+						name: contribution.name,
+						state: "unavailable",
+						hint: "check failed",
+						_isPlugin: true,
+						_group: contribution.group,
+					});
 				}
 			}
 		}
 
-		const fixableServices: FixableService[] =
-			!startupQuiet && welcomeResult.model.state === "connected" ? getFixableServices() : [];
+		const fixableServices: FixableService[] = [];
 
 		// Add fixable services from plugins
 		if (this.session.extensionRunner) {
@@ -559,6 +563,12 @@ export class InteractiveMode implements InteractiveModeContext {
 	finishPendingSubmission(input: SubmittedUserInput): void {
 		if (this.#pendingSubmittedInput === input) {
 			this.#pendingSubmittedInput = undefined;
+			this.#pendingWorkingMessage = undefined;
+			if (this.loadingAnimation) {
+				this.loadingAnimation.stop();
+				this.loadingAnimation = undefined;
+				this.statusContainer.clear();
+			}
 		}
 	}
 

package/src/modes/rpc/rpc-mode.ts

@@ -625,9 +625,15 @@ export async function runRpcMode(session: AgentSession): Promise<never> {
 					for (const contribution of pluginContributions) {
 						try {
 							const status = await contribution.check();
-							services.push({ name: contribution.name, ...status });
+							services.push({ name: contribution.name, ...status, _isPlugin: true, _group: contribution.group });
 						} catch {
-							services.push({ name: contribution.name, state: "unavailable", hint: "check failed" });
+							services.push({
+								name: contribution.name,
+								state: "unavailable",
+								hint: "check failed",
+								_isPlugin: true,
+								_group: contribution.group,
+							});
 						}
 					}
 				}