@f5-sales-demo/xcsh 19.57.0 → 19.57.2

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@f5-sales-demo/xcsh",
-	"version": "19.57.0",
+	"version": "19.57.2",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://github.com/f5-sales-demo/xcsh",
 	"author": "Can Boluk",
@@ -55,13 +55,13 @@
 	"dependencies": {
 		"@agentclientprotocol/sdk": "0.16.1",
 		"@mozilla/readability": "^0.6",
-		"@f5-sales-demo/xcsh-stats": "19.57.0",
-		"@f5-sales-demo/pi-agent-core": "19.57.0",
-		"@f5-sales-demo/pi-ai": "19.57.0",
-		"@f5-sales-demo/pi-natives": "19.57.0",
-		"@f5-sales-demo/pi-resource-management": "19.57.0",
-		"@f5-sales-demo/pi-tui": "19.57.0",
-		"@f5-sales-demo/pi-utils": "19.57.0",
+		"@f5-sales-demo/xcsh-stats": "19.57.2",
+		"@f5-sales-demo/pi-agent-core": "19.57.2",
+		"@f5-sales-demo/pi-ai": "19.57.2",
+		"@f5-sales-demo/pi-natives": "19.57.2",
+		"@f5-sales-demo/pi-resource-management": "19.57.2",
+		"@f5-sales-demo/pi-tui": "19.57.2",
+		"@f5-sales-demo/pi-utils": "19.57.2",
 		"@sinclair/typebox": "^0.34",
 		"@xterm/headless": "^6.0",
 		"ajv": "^8.20",

package/src/commands/worker.ts

@@ -13,7 +13,7 @@
  * `XCSH_SESSION_TENANT` so it advertises its assigned tenant even before a context
  * is bound. This lets the extension panel lock onto the right tenant immediately.
  */
-import { getProjectDir, getXCSHConfigDir } from "@f5-sales-demo/pi-utils";
+import { getProjectDir, getXCSHConfigDir, logger } from "@f5-sales-demo/pi-utils";
 import { Command } from "@f5-sales-demo/pi-utils/cli";
 import { ChatHandler } from "../browser/chat-handler";
 import { startBridgeServer } from "../browser/extension-bridge";
@@ -90,6 +90,12 @@ export default class Worker extends Command {
 	static description = "Run a headless extension-bridge worker (no TUI); blocks until SIGTERM";
 
 	async run(): Promise<void> {
+		// Record the per-tab session-boot timeline (parity with main.ts:runRootCommand).
+		// Spans only accumulate while recording; nothing prints unless PI_TIMING is set,
+		// and each logger.time() returns its wrapped value unchanged — so a normal
+		// `xcsh worker` run is behaviorally identical to before.
+		logger.startTiming();
+
 		process.env.XCSH_BROWSER_PROVIDER = "extension";
 
 		const cwd = getProjectDir();
@@ -113,7 +119,9 @@ export default class Worker extends Command {
 
 		// INSTANT-ON: start the bridge before the heavy session init so the extension
 		// can connect immediately. Honors XCSH_BRIDGE_PORT (forced) or auto-selects.
-		const bridge = await startBridgeServer();
+		// session:bridgeListen — time-to-"bridge-ready": the extension can connect and
+		// complete the hello/hello_ack handshake once this resolves (INSTANT-ON path).
+		const bridge = await logger.time("session:bridgeListen", startBridgeServer);
 		console.error(`[xcsh worker] extension bridge listening on ws://127.0.0.1:${bridge.port}`);
 		setSharedBridgeServer(bridge);
 		bridge.setSessionInfo(sessionInfoForWorker);
@@ -124,7 +132,10 @@ export default class Worker extends Command {
 		// the browser (without this the agent only has catalog_workflow_runner and
 		// merely narrates "Navigating…"). Include their names in the tool scope.
 		const extensionTools = createExtensionBridgeTools(bridge);
-		const { session } = await createAgentSession({
+		// session:createAgentSession — the heavy step between bridge-ready and
+		// session-ready (model registry, tools, context bootstrap). Wrapped as a span
+		// (parity with main.ts:892) so PI_TIMING reveals the per-tab session-load split.
+		const { session } = await logger.time("session:createAgentSession", createAgentSession, {
 			cwd,
 			hasUI: false,
 			toolNames: [...new Set([...BROWSER_TOOL_NAMES, ...EXTENSION_AGENT_TOOL_NAMES])],
@@ -139,6 +150,17 @@ export default class Worker extends Command {
 		const chatHandler = new ChatHandler(bridge, session);
 		chatHandler.attach();
 
+		// session-ready. Emit the per-tab boot breakdown when requested (parity with
+		// main.ts:1002-1009). PI_TIMING=x prints then exits — used by bench/extension-session.ts
+		// to measure total worker cold-start; otherwise this is a no-op.
+		if (process.env.PI_TIMING) {
+			logger.printTimings();
+			if (process.env.PI_TIMING === "x") {
+				process.exit(0);
+			}
+		}
+		logger.endTiming();
+
 		let shuttingDown = false;
 		const shutdown = () => {
 			if (shuttingDown) return;

package/src/extensibility/extensions/loader.ts

@@ -284,7 +284,7 @@ async function loadExtension(
 	const resolvedPath = resolvePath(extensionPath, cwd);
 
 	try {
-		const module = await import(resolvedPath);
+		const module = await logger.time("ext:loadModule", () => import(resolvedPath));
 		const factory = (module.default ?? module) as ExtensionFactory;
 
 		if (typeof factory !== "function") {
@@ -295,7 +295,13 @@ async function loadExtension(
 		}
 
 		const extension = createExtension(extensionPath, resolvedPath);
-		const api = new ConcreteExtensionAPI(await import("@f5-sales-demo/xcsh"), extension, runtime, cwd, eventBus);
+		const api = new ConcreteExtensionAPI(
+			await logger.time("ext:barrelImport", () => import("@f5-sales-demo/xcsh")),
+			extension,
+			runtime,
+			cwd,
+			eventBus,
+		);
 		await factory(api);
 
 		return { extension, error: null };
@@ -316,7 +322,13 @@ export async function loadExtensionFromFactory(
 	name = "<inline>",
 ): Promise<Extension> {
 	const extension = createExtension(name, name);
-	const api = new ConcreteExtensionAPI(await import("@f5-sales-demo/xcsh"), extension, runtime, cwd, eventBus);
+	const api = new ConcreteExtensionAPI(
+		await logger.time("ext:barrelImport", () => import("@f5-sales-demo/xcsh")),
+		extension,
+		runtime,
+		cwd,
+		eventBus,
+	);
 	await factory(api);
 	return extension;
 }
@@ -540,7 +552,9 @@ export async function discoverAndLoadExtensions(
 	};
 
 	// 1. Discover extension modules via capability API (native .omp/.pi only)
-	const discovered = await loadCapability<ExtensionModule>(extensionModuleCapability.id, { cwd });
+	const discovered = await logger.time("ext:discoverCapability", () =>
+		loadCapability<ExtensionModule>(extensionModuleCapability.id, { cwd }),
+	);
 	for (const ext of discovered.items) {
 		if (ext._source.provider !== "native") continue;
 		if (isDisabledName(ext.name)) continue;
@@ -548,29 +562,31 @@ export async function discoverAndLoadExtensions(
 	}
 
 	// 2. Discover extension entry points from installed plugins (node_modules path)
-	addPaths(await getAllPluginExtensionPaths(cwd));
+	addPaths(await logger.time("ext:pluginPaths", () => getAllPluginExtensionPaths(cwd)));
 
 	// 2b. Discover extension entry points from marketplace-cached plugins
-	for (const root of getPreloadedPluginRoots()) {
-		try {
-			const pkgPath = path.join(root.path, "package.json");
-			const pkg = await Bun.file(pkgPath).json();
-			const manifest = pkg?.xcsh ?? pkg?.pi;
-			const extensions = manifest?.extensions;
-			if (Array.isArray(extensions)) {
-				for (const entry of extensions) {
-					if (typeof entry !== "string") continue;
-					if (path.isAbsolute(entry) || entry.includes("..")) continue;
-					const resolved = path.resolve(root.path, entry);
-					if (!resolved.startsWith(root.path + path.sep) && resolved !== root.path) continue;
-					if (isDisabledName(getExtensionNameFromPath(resolved))) continue;
-					addPath(resolved);
+	await logger.time("ext:marketplaceRoots", async () => {
+		for (const root of getPreloadedPluginRoots()) {
+			try {
+				const pkgPath = path.join(root.path, "package.json");
+				const pkg = await Bun.file(pkgPath).json();
+				const manifest = pkg?.xcsh ?? pkg?.pi;
+				const extensions = manifest?.extensions;
+				if (Array.isArray(extensions)) {
+					for (const entry of extensions) {
+						if (typeof entry !== "string") continue;
+						if (path.isAbsolute(entry) || entry.includes("..")) continue;
+						const resolved = path.resolve(root.path, entry);
+						if (!resolved.startsWith(root.path + path.sep) && resolved !== root.path) continue;
+						if (isDisabledName(getExtensionNameFromPath(resolved))) continue;
+						addPath(resolved);
+					}
 				}
+			} catch {
+				// No package.json or invalid — skip
 			}
-		} catch {
-			// No package.json or invalid — skip
 		}
-	}
+	});
 
 	// 3. Explicitly configured paths
 	for (const configuredPath of configuredPaths) {
@@ -601,7 +617,7 @@ export async function discoverAndLoadExtensions(
 	}
 
 	const resolvedEventBus = eventBus ?? new EventBus();
-	const result = await loadExtensions(allPaths, cwd, resolvedEventBus);
-	await loadBundledExtensions(result, cwd, resolvedEventBus, isDisabledName);
+	const result = await logger.time("ext:loadLoop", () => loadExtensions(allPaths, cwd, resolvedEventBus));
+	await logger.time("ext:loadBundled", () => loadBundledExtensions(result, cwd, resolvedEventBus, isDisabledName));
 	return result;
 }

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

@@ -17,17 +17,17 @@ export interface BuildInfo {
 }
 
 export const BUILD_INFO: BuildInfo = {
-	"version": "19.57.0",
-	"commit": "9b508d8af3633290ce801f2ca9956dd22df78e30",
-	"shortCommit": "9b508d8",
+	"version": "19.57.2",
+	"commit": "642fb826019640261cf1e3c1ec0206581fd9c1fc",
+	"shortCommit": "642fb82",
 	"branch": "main",
-	"tag": "v19.57.0",
-	"commitDate": "2026-07-04T20:19:08Z",
-	"buildDate": "2026-07-04T20:54:40.287Z",
+	"tag": "v19.57.2",
+	"commitDate": "2026-07-04T22:01:28Z",
+	"buildDate": "2026-07-04T22:29:54.204Z",
 	"dirty": true,
 	"prNumber": "",
 	"repoUrl": "https://github.com/f5-sales-demo/xcsh",
 	"repoSlug": "f5-sales-demo/xcsh",
-	"commitUrl": "https://github.com/f5-sales-demo/xcsh/commit/9b508d8af3633290ce801f2ca9956dd22df78e30",
-	"releaseUrl": "https://github.com/f5-sales-demo/xcsh/releases/tag/v19.57.0"
+	"commitUrl": "https://github.com/f5-sales-demo/xcsh/commit/642fb826019640261cf1e3c1ec0206581fd9c1fc",
+	"releaseUrl": "https://github.com/f5-sales-demo/xcsh/releases/tag/v19.57.2"
 };