@f5-sales-demo/xcsh 21.30.1 → 21.32.0

package/package.json

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

package/src/browser/office-pane-server.ts

@@ -155,7 +155,7 @@ export function paneUnavailableMessage(distDir: string, source: PaneSource): str
 	return (
 		"The Office pane is not included in the npm package — it ships as a build-time asset of the " +
 		"compiled binary, which is also what provides the sideload and the trusted certificate. " +
-		"Install it with `brew install f5-sales-demo/tap/xcsh` and run `xcsh office` from there."
+		"Install it with `brew install --cask f5-sales-demo/tap/xcsh` and run `xcsh office` from there."
 	);
 }
 

package/src/cli/update-cli.ts

@@ -75,7 +75,7 @@ type UpdateTarget =
  * Detection order:
  * 1. bun  — binary is inside bun's global bin directory
  * 2. npm  — binary is a symlink whose resolution chain contains "node_modules"
- * 3. brew — binary path or realpath contains "Cellar" or "homebrew"
+ * 3. brew — binary path or realpath contains "Cellar", "Caskroom", or "homebrew"
  * 4. binary — fallback for standalone installs
  */
 function detectInstallMethod(binPath: string, bunBinDir: string | undefined): InstallMethod {
@@ -106,14 +106,18 @@ function detectInstallMethod(binPath: string, bunBinDir: string | undefined): In
 		// lstat/readlink may fail; fall through
 	}
 
-	// 3. brew: path or realpath contains Cellar or homebrew
+	// 3. brew: path or realpath contains Cellar, Caskroom, or homebrew
 	const lowerBinPath = binPath.toLowerCase();
-	if (lowerBinPath.includes("/cellar/") || lowerBinPath.includes("/homebrew/")) {
+	if (
+		lowerBinPath.includes("/cellar/") ||
+		lowerBinPath.includes("/caskroom/") ||
+		lowerBinPath.includes("/homebrew/")
+	) {
 		return "brew";
 	}
 	try {
 		const realPath = fs.realpathSync(binPath).toLowerCase();
-		if (realPath.includes("/cellar/") || realPath.includes("/homebrew/")) {
+		if (realPath.includes("/cellar/") || realPath.includes("/caskroom/") || realPath.includes("/homebrew/")) {
 			return "brew";
 		}
 	} catch {
@@ -322,7 +326,7 @@ async function updateViaNpm(expectedVersion: string): Promise<void> {
 function updateViaBrew(targetPath: string, expectedVersion: string): void {
 	console.log(chalk.yellow(`\n${APP_NAME} at ${targetPath} was installed via Homebrew.`));
 	console.log(chalk.yellow(`To update to ${expectedVersion}, run:`));
-	console.log(chalk.cyan(`  brew upgrade ${APP_NAME}`));
+	console.log(chalk.cyan(`  ${getBrewUpgradeCommand()}`));
 	console.log(chalk.dim("\nThis ensures the update goes through your organization's Homebrew tap."));
 	// #1874 Task 7: brew upgrade runs out-of-band, so we can't recycle automatically.
 	// Nudge the one command that applies it to the Chrome extension immediately.
@@ -330,6 +334,10 @@ function updateViaBrew(targetPath: string, expectedVersion: string): void {
 	console.log(chalk.dim("(otherwise it takes effect the next time you open Chrome)."));
 }
 
+export function getBrewUpgradeCommand(): string {
+	return `brew upgrade --cask ${APP_NAME}`;
+}
+
 /**
  * Download a release binary to a target path, replacing an existing file.
  */

package/src/cli.ts

@@ -78,6 +78,7 @@ const commands: CommandEntry[] = [
 	{ name: "manager", load: () => import("./commands/manager").then(m => m.default) },
 	{ name: "office", load: () => import("./commands/office").then(m => m.default) },
 	{ name: "plugin", load: () => import("./commands/plugin").then(m => m.default) },
+	{ name: "profile", load: () => import("./commands/profile").then(m => m.default) },
 	{ name: "setup", load: () => import("./commands/setup").then(m => m.default) },
 	{ name: "shell", load: () => import("./commands/shell").then(m => m.default) },
 	{ name: "ssh", load: () => import("./commands/ssh").then(m => m.default) },

package/src/commands/profile.ts

@@ -0,0 +1,72 @@
+import { createInterface } from "node:readline/promises";
+import { Args, Command, Flags } from "@f5-sales-demo/pi-utils/cli";
+import { machineProfileService } from "../person-profile/machine-profile";
+import { resetProfileTargets } from "../person-profile/private-store";
+import { personProfileService } from "../person-profile/service";
+
+type ProfileTarget = "person" | "computer" | "all";
+
+export default class Profile extends Command {
+	static description = "Inspect or reset private person and computer profiles";
+	static args = {
+		action: Args.string({ description: "Profile action", options: ["status", "reset"], required: true }),
+		target: Args.string({ description: "Reset target", options: ["person", "computer", "all"] }),
+	};
+	static flags = {
+		json: Flags.boolean({ description: "Output machine-readable status", default: false }),
+		yes: Flags.boolean({ description: "Confirm permanent reset", default: false }),
+	};
+
+	async run(): Promise<void> {
+		const { args, flags } = await this.parse(Profile);
+		if (args.action === "status") {
+			if (flags.yes) throw new Error("profile status does not accept --yes");
+			if (args.target) throw new Error("profile status does not accept a target");
+			const result = {
+				person: await personProfileService.status(),
+				computer: await machineProfileService.status(),
+			};
+			if (flags.json) {
+				process.stdout.write(`${JSON.stringify(result)}\n`);
+				return;
+			}
+			for (const [name, status] of Object.entries(result)) {
+				const details = [
+					status.schemaVersion === undefined ? undefined : `schema v${status.schemaVersion}`,
+					status.permissions?.file ? `file ${status.permissions.file}` : undefined,
+					status.permissions?.directory ? `directory ${status.permissions.directory}` : undefined,
+					status.reason,
+				].filter(Boolean);
+				process.stdout.write(`${name}: ${status.status}${details.length ? ` (${details.join(", ")})` : ""}\n`);
+				if (status.remedy) process.stdout.write(`  remedy: ${status.remedy}\n`);
+			}
+			return;
+		}
+
+		const target = args.target as ProfileTarget | undefined;
+		if (!target) throw new Error("profile reset requires person, computer, or all");
+		if (flags.json) throw new Error("profile reset does not accept --json");
+		const services =
+			target === "all"
+				? [personProfileService, machineProfileService]
+				: [target === "person" ? personProfileService : machineProfileService];
+		process.stdout.write(
+			`Profiles selected for permanent reset:\n${services.map(service => `- ${service.path}`).join("\n")}\n`,
+		);
+		if (!flags.yes) {
+			if (!process.stdin.isTTY || !process.stdout.isTTY)
+				throw new Error("Non-interactive profile reset requires --yes");
+			const prompt = createInterface({ input: process.stdin, output: process.stdout });
+			try {
+				const answer = await prompt.question('Type "reset" to continue: ');
+				if (answer !== "reset") throw new Error("Profile reset cancelled");
+			} finally {
+				prompt.close();
+			}
+		}
+		const results = await resetProfileTargets(services);
+		for (const service of services) {
+			process.stdout.write(`${service.path}: ${results.get(service.path) ? "reset" : "already missing"}\n`);
+		}
+	}
+}

package/src/extensibility/extensions/loader.ts

@@ -15,7 +15,7 @@ import { loadCapability } from "../../discovery";
 import { getExtensionNameFromPath, getPreloadedPluginRoots } from "../../discovery/helpers";
 import type { ExecOptions } from "../../exec/exec";
 import { execCommand } from "../../exec/exec";
-import { type ProfileCollector, personProfileService } from "../../person-profile/service";
+import { personProfileService } from "../../person-profile/service";
 import type { CustomMessage } from "../../session/messages";
 import { EventBus } from "../../utils/event-bus";
 import { getAllPluginExtensionPaths } from "../plugins/loader";
@@ -121,6 +121,7 @@ class ConcreteExtensionAPI implements ExtensionAPI, IExtensionRuntime {
 		config: import("./types").ProviderConfig;
 		sourceId: string;
 	}> = [];
+	readonly personProfile: ExtensionAPI["personProfile"];
 
 	constructor(
 		public readonly pi: typeof import("@f5-sales-demo/xcsh"),
@@ -128,7 +129,18 @@ class ConcreteExtensionAPI implements ExtensionAPI, IExtensionRuntime {
 		private readonly runtime: IExtensionRuntime,
 		private readonly cwd: string,
 		public readonly events: EventBus,
-	) {}
+	) {
+		const registrant = extension.resolvedPath;
+		this.personProfile = Object.freeze({
+			get: () => personProfileService.get(),
+			registerCollector: collector => {
+				if (!collector || typeof collector.collect !== "function")
+					throw new Error("Invalid person profile collector");
+				personProfileService.registerProfileCollector(collector, registrant);
+			},
+			unregisterCollector: (id: string) => personProfileService.unregisterProfileCollector(id, registrant),
+		});
+	}
 
 	on<F extends HandlerFn>(event: string, handler: F): void {
 		const list = this.extension.handlers.get(event) ?? [];
@@ -249,35 +261,6 @@ class ConcreteExtensionAPI implements ExtensionAPI, IExtensionRuntime {
 		return this.runtime.setSessionName(name);
 	}
 
-	registerProfileCollector(collector: ProfileCollector): void {
-		if (!collector || typeof collector.collect !== "function") throw new Error("Invalid person profile collector");
-		personProfileService.registerProfileCollector(
-			{
-				...collector,
-				async collect(signal) {
-					const result = await collector.collect(signal);
-					if ("facts" in result) return result;
-					// Legacy flat output does not distinguish queried values from heuristic inference.
-					// Keep it as candidates; modern collectors can return explicit facts/observations.
-					return {
-						facts: {},
-						observations: Object.entries(result).map(([field, value]) => ({
-							field: field as keyof typeof result,
-							value,
-							source: collector.id,
-							kind: "inferred" as const,
-							observedAt: new Date().toISOString(),
-						})),
-					};
-				},
-			},
-			this.extension.resolvedPath,
-		);
-	}
-	unregisterProfileCollector(id: string): boolean {
-		return personProfileService.unregisterProfileCollector(id, this.extension.resolvedPath);
-	}
-
 	registerProvider(name: string, config: import("./types").ProviderConfig): void {
 		this.runtime.pendingProviderRegistrations.push({ name, config, sourceId: this.extension.path });
 	}

package/src/extensibility/extensions/types.ts

@@ -1013,6 +1013,13 @@ export interface ExtensionAPI {
 	/** Injected pi-coding-agent exports for accessing SDK utilities */
 	pi: typeof piCodingAgent;
 
+	/** Canonical, provenance-aware person profile integration. */
+	readonly personProfile: {
+		get(): Promise<import("../../person-profile/schema").PersonProfile>;
+		registerCollector(collector: import("../../person-profile/service").ExtensionProfileCollector): void;
+		unregisterCollector(id: string): boolean;
+	};
+
 	// =========================================================================
 	// Event Subscription
 	// =========================================================================
@@ -1222,9 +1229,6 @@ export interface ExtensionAPI {
 	 * });
 	 */
 	registerProvider(name: string, config: ProviderConfig): void;
-	registerProfileCollector(collector: import("../../person-profile/service").ProfileCollector): void;
-	unregisterProfileCollector(id: string): boolean;
-
 	/** Shared event bus for extension communication. */
 	events: EventBus;
 }

package/src/index.ts

@@ -40,7 +40,7 @@ export * from "./modes/components";
 export * from "./modes/theme/theme";
 export type { PersonProfile, UserProfile, UserProfileObservation } from "./person-profile/schema";
 export type { ProfileCollection, ProfileCollector } from "./person-profile/service";
-export { loadProfile, PersonProfileService } from "./person-profile/service";
+export { PersonProfileService } from "./person-profile/service";
 export * from "./routing/classifier";
 export * from "./routing/coordinator";
 export * from "./routing/delegation";

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

@@ -17,17 +17,17 @@ export interface BuildInfo {
 }
 
 export const BUILD_INFO: BuildInfo = {
-	"version": "21.30.1",
-	"commit": "22d9767e595b2a3d2e57a62b192bd582fc9b00eb",
-	"shortCommit": "22d9767",
-	"branch": "main",
-	"tag": "v21.30.1",
-	"commitDate": "2026-09-17T15:05:21+00:00",
-	"buildDate": "2026-09-17T16:16:04.516Z",
+	"version": "21.32.0",
+	"commit": "318283a955aea8725f8119d68e14db6ea2d8b3e7",
+	"shortCommit": "318283a",
+	"branch": "unknown",
+	"tag": "v21.32.0",
+	"commitDate": "2026-09-17T18:23:21Z",
+	"buildDate": "2026-09-17T23:24:55.633Z",
 	"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/22d9767e595b2a3d2e57a62b192bd582fc9b00eb",
-	"releaseUrl": "https://github.com/f5-sales-demo/xcsh/releases/tag/v21.30.1"
+	"commitUrl": "https://github.com/f5-sales-demo/xcsh/commit/318283a955aea8725f8119d68e14db6ea2d8b3e7",
+	"releaseUrl": "https://github.com/f5-sales-demo/xcsh/releases/tag/v21.32.0"
 };