@f5xc-salesdemos/xcsh 17.0.2 → 17.1.1

package/package.json

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

package/src/config/model-registry.ts

@@ -831,6 +831,29 @@ export class ModelRegistry {
 		this.#backgroundRefresh = refreshPromise;
 	}
 
+	/**
+	 * Await the in-flight background refresh if one is running.
+	 * Returns immediately if no background refresh is in progress.
+	 */
+	async awaitBackgroundRefresh(): Promise<void> {
+		if (this.#backgroundRefresh) {
+			await this.#backgroundRefresh;
+		}
+	}
+
+	/**
+	 * Check if any non-optional discoverable provider has no cached models yet.
+	 * Returns true on first run when the model cache is empty.
+	 */
+	hasUncachedDiscoverableProviders(): boolean {
+		for (const [, state] of this.#providerDiscoveryStates) {
+			if (state.status === "idle" && !state.optional) {
+				return true;
+			}
+		}
+		return false;
+	}
+
 	async refreshProvider(providerId: string, strategy: ModelRefreshStrategy = "online"): Promise<void> {
 		this.#reloadStaticModels();
 		for (const selector of this.#suppressedSelectors.keys()) {

package/src/main.ts

@@ -624,6 +624,19 @@ export async function runRootCommand(parsed: Args, rawArgs: string[]): Promise<v
 
 	const cwd = getProjectDir();
 	await logger.time("settings:init", Settings.init, { cwd });
+
+	// F5 XC profile loading — optional, never blocks startup.
+	// NOTE: This runs in the CLI path only. SDK consumers using createAgentSession()
+	// directly must call ProfileService.init(configDir).loadActive() themselves.
+	try {
+		const { ProfileService } = await import("./services/f5xc-profile");
+		const { getF5XCConfigDir } = await import("@f5xc-salesdemos/pi-utils");
+		const profileService = ProfileService.init(getF5XCConfigDir());
+		await profileService.loadActive();
+	} catch {
+		// F5 XC auth is optional — silently continue if anything fails
+	}
+
 	if (parsedArgs.mode === "rpc") {
 		applyRpcDefaultSettingOverrides();
 	}

package/src/sdk.ts

@@ -23,11 +23,11 @@ import {
 	prompt,
 	Snowflake,
 } from "@f5xc-salesdemos/pi-utils";
-import chalk from "chalk";
 import { AsyncJobManager, isBackgroundJobSupportEnabled } from "./async";
 import { createAutoresearchExtension } from "./autoresearch";
 import { loadCapability } from "./capability";
 import { type Rule, ruleCapability } from "./capability/rule";
+import { hasLiteLLMEnv } from "./config/auto-config";
 import { ModelRegistry } from "./config/model-registry";
 import { formatModelString, parseModelPattern, parseModelString, resolveModelRoleValue } from "./config/model-resolver";
 import { loadPromptTemplates as loadPromptTemplatesInternal, type PromptTemplate } from "./config/prompt-templates";
@@ -780,6 +780,13 @@ export async function createAgentSession(options: CreateAgentSessionOptions = {}
 	const modelMatchPreferences = {
 		usageOrder: settings.getStorage()?.getModelUsageOrder(),
 	};
+	// When LiteLLM is configured and no model cache exists yet (first run),
+	// await the background refresh so model discovery from the proxy completes
+	// before we select a default model. Bounded by the 3s probe timeout.
+	if (!options.modelRegistry && hasLiteLLMEnv() && modelRegistry.hasUncachedDiscoverableProviders()) {
+		await logger.time("awaitLiteLLMDiscovery", () => modelRegistry.awaitBackgroundRefresh());
+	}
+
 	const defaultRoleSpec = logger.time("resolveDefaultModelRole", () =>
 		resolveModelRoleValue(settings.getModelRole("default"), modelRegistry.getAvailable(), {
 			settings,
@@ -1069,8 +1076,8 @@ export async function createAgentSession(options: CreateAgentSessionOptions = {}
 		if (enableMCP) {
 			const mcpResult = await logger.time("discoverAndLoadMCPTools", discoverAndLoadMCPTools, cwd, {
 				onConnecting: serverNames => {
-					if (options.hasUI && serverNames.length > 0) {
-						process.stderr.write(`${chalk.gray(`Connecting to MCP servers: ${serverNames.join(", ")}…`)}\n`);
+					if (serverNames.length > 0) {
+						logger.debug("Connecting to MCP servers", { servers: serverNames });
 					}
 				},
 				enableProjectConfig: settings.get("mcp.enableProjectConfig") ?? true,

package/src/tools/vim.ts

@@ -644,7 +644,19 @@ export class VimTool implements AgentTool<typeof vimSchema, VimToolDetails> {
 
 				await executeVimSteps(engine, steps, {
 					pauseLastStep: params.pause === true,
-					onKbdStep: emitUpdate ? () => emitUpdate() : undefined,
+					onKbdStep: emitUpdate
+						? () => {
+								// Force update in prompt modes (command/search) so every keystroke
+								// is reported to onUpdate — users need to see each character of
+								// their ex-command input, and throttling can cause intermediate
+								// states to be skipped under heavy event-loop load (CI).
+								const forcePrompt =
+									engine.inputMode === "command" ||
+									engine.inputMode === "search-forward" ||
+									engine.inputMode === "search-backward";
+								return emitUpdate(forcePrompt);
+							}
+						: undefined,
 					onInsertStep: emitUpdate ? () => emitUpdate(true) : undefined,
 				});
 

package/src/web/search/provider.ts

@@ -71,8 +71,12 @@ export async function resolveProviderChain(
 	const providers: SearchProvider[] = [];
 
 	if (preferredProvider !== "auto") {
-		if (await getSearchProvider(preferredProvider).isAvailable()) {
-			providers.push(getSearchProvider(preferredProvider));
+		try {
+			if (await getSearchProvider(preferredProvider).isAvailable()) {
+				providers.push(getSearchProvider(preferredProvider));
+			}
+		} catch {
+			// Preferred provider check failed; continue with fallback chain
 		}
 	}
 
@@ -80,8 +84,12 @@ export async function resolveProviderChain(
 		if (id === preferredProvider) continue;
 
 		const provider = getSearchProvider(id);
-		if (await provider.isAvailable()) {
-			providers.push(provider);
+		try {
+			if (await provider.isAvailable()) {
+				providers.push(provider);
+			}
+		} catch {
+			// Provider availability check failed; skip and continue
 		}
 	}