@f5-sales-demo/xcsh 21.40.0 → 21.41.1

package/package.json

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

package/src/capability/fs.ts

@@ -8,40 +8,47 @@ function resolvePath(filePath: string): string {
 	return path.resolve(filePath);
 }
 
-export async function readFile(filePath: string): Promise<string | null> {
+export async function readFile(filePath: string, signal?: AbortSignal): Promise<string | null> {
 	const abs = resolvePath(filePath);
 	if (contentCache.has(abs)) {
 		return contentCache.get(abs) ?? null;
 	}
 
 	try {
-		const content = await Bun.file(abs).text();
+		signal?.throwIfAborted();
+		const stat = await fs.promises.stat(abs);
+		if (!stat.isFile()) return null;
+		const content = await fs.promises.readFile(abs, { encoding: "utf8", signal });
 		contentCache.set(abs, content);
 		return content;
-	} catch {
+	} catch (error) {
+		if (signal?.aborted) throw error;
 		contentCache.set(abs, null);
 		return null;
 	}
 }
 
-export async function readDirEntries(dirPath: string): Promise<fs.Dirent[]> {
+export async function readDirEntries(dirPath: string, signal?: AbortSignal): Promise<fs.Dirent[]> {
 	const abs = resolvePath(dirPath);
 	if (dirCache.has(abs)) {
 		return dirCache.get(abs) ?? [];
 	}
 
 	try {
+		signal?.throwIfAborted();
 		const entries = await fs.promises.readdir(abs, { withFileTypes: true });
+		signal?.throwIfAborted();
 		dirCache.set(abs, entries);
 		return entries;
-	} catch {
+	} catch (error) {
+		if (signal?.aborted) throw error;
 		dirCache.set(abs, []);
 		return [];
 	}
 }
 
-export async function readDir(dirPath: string): Promise<string[]> {
-	const entries = await readDirEntries(dirPath);
+export async function readDir(dirPath: string, signal?: AbortSignal): Promise<string[]> {
+	const entries = await readDirEntries(dirPath, signal);
 	return entries.map(entry => entry.name);
 }
 
@@ -49,12 +56,14 @@ export async function walkUp(
 	startDir: string,
 	name: string,
 	opts: { file?: boolean; dir?: boolean } = {},
+	signal?: AbortSignal,
 ): Promise<string | null> {
 	const { file = true, dir = true } = opts;
 	let current = resolvePath(startDir);
 
 	while (true) {
-		const entries = await readDirEntries(current);
+		signal?.throwIfAborted();
+		const entries = await readDirEntries(current, signal);
 		const entry = entries.find(e => e.name === name);
 		if (entry) {
 			if (file && entry.isFile()) return path.join(current, name);
@@ -71,10 +80,11 @@ export async function walkUp(
  * Returns the directory containing `.git` (the repo root), or null if not in a git repo.
  * Results are based on the cached readDirEntries, so repeated calls are cheap.
  */
-export async function findRepoRoot(startDir: string): Promise<string | null> {
+export async function findRepoRoot(startDir: string, signal?: AbortSignal): Promise<string | null> {
 	let current = resolvePath(startDir);
 	while (true) {
-		const entries = await readDirEntries(current);
+		signal?.throwIfAborted();
+		const entries = await readDirEntries(current, signal);
 		if (entries.some(e => e.name === ".git")) {
 			return current;
 		}

package/src/capability/index.ts

@@ -114,6 +114,7 @@ async function loadImpl<T>(
 	const results = await Promise.all(
 		providers.map(async provider => {
 			try {
+				ctx.signal?.throwIfAborted();
 				const result = await logger.time(
 					`capability:${capability.id}:${provider.id}`,
 					provider.load.bind(provider),
@@ -121,6 +122,7 @@ async function loadImpl<T>(
 				);
 				return { provider, result };
 			} catch (error) {
+				if (ctx.signal?.aborted) throw error;
 				logger.debug(`capability:${capability.id}:${provider.id}:error`);
 				return { provider, error };
 			}
@@ -235,8 +237,9 @@ export async function loadCapability<T>(capabilityId: string, options: LoadOptio
 
 	const cwd = options.cwd ?? getProjectDir();
 	const home = os.homedir();
-	const repoRoot = await findRepoRoot(cwd);
-	const ctx: LoadContext = { cwd, home, repoRoot };
+	options.signal?.throwIfAborted();
+	const repoRoot = await findRepoRoot(cwd, options.signal);
+	const ctx: LoadContext = { cwd, home, repoRoot, signal: options.signal };
 	const providers = filterProviders(capability, options);
 
 	return await loadImpl(capability, providers, ctx, options);

package/src/capability/types.ts

@@ -16,6 +16,8 @@ export interface LoadContext {
 	home: string;
 	/** Git repository root (directory containing .git), or null if not in a repo */
 	repoRoot: string | null;
+	/** Cancellation for bounded discovery. */
+	signal?: AbortSignal;
 }
 
 /**
@@ -72,6 +74,8 @@ export interface LoadOptions {
 	includeDisabled?: boolean;
 	/** Explicit disabled extension IDs to apply instead of settings. */
 	disabledExtensions?: string[];
+	/** Cancellation for bounded discovery. */
+	signal?: AbortSignal;
 }
 
 /**

package/src/deprecations.ts

@@ -17,9 +17,9 @@ interface DeprecationEntry {
 const DEPRECATIONS = BRANDING_DEPRECATIONS as unknown as Record<string, DeprecationEntry>;
 
 /** Markers that identify a command as targeting the F5 XC API. */
-const F5XC_API_MARKERS = [
-	"f5xc_api_url",
-	"f5xc_api_token",
+const XCSH_API_MARKERS = [
+	"xcsh_api_url",
+	"xcsh_api_token",
 	"apitoken",
 	".volterra.io",
 	".volterra.us",
@@ -57,7 +57,7 @@ export function isDisallowedCliCommand(command: string): boolean {
 	if (!lower) return false;
 	const leadingToken = lower.split(/\s+/)[0] ?? "";
 	if (getDeprecatedClis().includes(leadingToken)) return true;
-	if (leadingToken === "curl" && F5XC_API_MARKERS.some(marker => lower.includes(marker))) return true;
+	if (leadingToken === "curl" && XCSH_API_MARKERS.some(marker => lower.includes(marker))) return true;
 	return false;
 }
 

package/src/discovery/agents-md.ts

@@ -26,8 +26,9 @@ async function loadAgentsMd(ctx: LoadContext): Promise<LoadResult<ContextFile>>
 	let current = ctx.cwd;
 
 	while (true) {
+		ctx.signal?.throwIfAborted();
 		const candidate = path.join(current, "XCSH.md");
-		const content = await readFile(candidate);
+		const content = await readFile(candidate, ctx.signal);
 
 		if (content !== null) {
 			const parent = path.dirname(candidate);

package/src/discovery/agents.ts

@@ -52,10 +52,10 @@ function getProjectPathCandidates(ctx: LoadContext, ...segments: string[]): stri
 // Skills
 async function loadSkills(ctx: LoadContext): Promise<LoadResult<Skill>> {
 	const projectScans = getProjectPathCandidates(ctx, "skills").map(dir =>
-		scanSkillsFromDir(ctx, { dir, providerId: PROVIDER_ID, level: "project" }),
+		scanSkillsFromDir(ctx, { dir, providerId: PROVIDER_ID, level: "project", signal: ctx.signal }),
 	);
 	const userScans = getUserPathCandidates(ctx, "skills").map(dir =>
-		scanSkillsFromDir(ctx, { dir, providerId: PROVIDER_ID, level: "user" }),
+		scanSkillsFromDir(ctx, { dir, providerId: PROVIDER_ID, level: "user", signal: ctx.signal }),
 	);
 
 	const results = await Promise.all([...projectScans, ...userScans]);
@@ -170,7 +170,7 @@ registerProvider<SlashCommand>(slashCommandCapability.id, {
 // Context Files (AGENTS.md)
 async function loadContextFiles(ctx: LoadContext): Promise<LoadResult<ContextFile>> {
 	const load = async (filePath: string, level: "user" | "project"): Promise<ContextFile | null> => {
-		const content = await readFile(filePath);
+		const content = await readFile(filePath, ctx.signal);
 		if (!content) return null;
 		// filePath is <ancestor>/.agent(s)/AGENTS.md — go up past the config dir to the ancestor
 		const ancestorDir = path.dirname(path.dirname(filePath));
@@ -197,7 +197,7 @@ registerProvider<ContextFile>(contextFileCapability.id, {
 // System Prompt (SYSTEM.md)
 async function loadSystemPrompt(ctx: LoadContext): Promise<LoadResult<SystemPrompt>> {
 	const load = async (filePath: string, level: "user" | "project"): Promise<SystemPrompt | null> => {
-		const content = await readFile(filePath);
+		const content = await readFile(filePath, ctx.signal);
 		if (!content) return null;
 		return { path: filePath, content, level, _source: createSourceMeta(PROVIDER_ID, filePath, level) };
 	};

package/src/discovery/builtin.ts

@@ -225,7 +225,7 @@ async function loadSystemPrompt(ctx: LoadContext): Promise<LoadResult<SystemProm
 	const items: SystemPrompt[] = [];
 
 	const userPath = path.join(ctx.home, PATHS.userAgent, "SYSTEM.md");
-	const userContent = await readFile(userPath);
+	const userContent = await readFile(userPath, ctx.signal);
 	if (userContent) {
 		items.push({
 			path: userPath,
@@ -238,7 +238,7 @@ async function loadSystemPrompt(ctx: LoadContext): Promise<LoadResult<SystemProm
 	const nearestProjectConfigDir = await findNearestProjectConfigDir(ctx.cwd, ctx.repoRoot);
 	if (nearestProjectConfigDir) {
 		const projectPath = path.join(nearestProjectConfigDir.dir, "SYSTEM.md");
-		const projectContent = await readFile(projectPath);
+		const projectContent = await readFile(projectPath, ctx.signal);
 		if (projectContent) {
 			items.push({
 				path: projectPath,
@@ -270,6 +270,7 @@ async function loadSkills(ctx: LoadContext): Promise<LoadResult<Skill>> {
 			providerId: PROVIDER_ID,
 			level: "project",
 			requireDescription: true,
+			signal: ctx.signal,
 		}),
 	);
 
@@ -279,6 +280,7 @@ async function loadSkills(ctx: LoadContext): Promise<LoadResult<Skill>> {
 		providerId: PROVIDER_ID,
 		level: "user",
 		requireDescription: true,
+		signal: ctx.signal,
 	});
 
 	const results = await Promise.all([...projectScans, userScan]);
@@ -814,7 +816,7 @@ async function loadContextFiles(ctx: LoadContext): Promise<LoadResult<ContextFil
 	const warnings: string[] = [];
 
 	const userPath = path.join(ctx.home, PATHS.userAgent, "XCSH.md");
-	const userContent = await readFile(userPath);
+	const userContent = await readFile(userPath, ctx.signal);
 	if (userContent) {
 		items.push({
 			path: userPath,
@@ -827,7 +829,7 @@ async function loadContextFiles(ctx: LoadContext): Promise<LoadResult<ContextFil
 	const nearestProjectConfigDir = await findNearestProjectConfigDir(ctx.cwd, ctx.repoRoot);
 	if (nearestProjectConfigDir) {
 		const projectPath = path.join(nearestProjectConfigDir.dir, "XCSH.md");
-		const projectContent = await readFile(projectPath);
+		const projectContent = await readFile(projectPath, ctx.signal);
 		if (projectContent) {
 			items.push({
 				path: projectPath,

package/src/discovery/claude-plugins.ts

@@ -37,7 +37,7 @@ async function loadSkills(ctx: LoadContext): Promise<LoadResult<Skill>> {
 	const items: Skill[] = [];
 	const warnings: string[] = [];
 
-	const { roots, warnings: rootWarnings } = await listXcshPluginRoots(ctx.home, ctx.cwd);
+	const { roots, warnings: rootWarnings } = await listXcshPluginRoots(ctx.home, ctx.cwd, ctx.signal);
 	warnings.push(...rootWarnings);
 
 	const results = await Promise.all(
@@ -47,6 +47,7 @@ async function loadSkills(ctx: LoadContext): Promise<LoadResult<Skill>> {
 				dir: skillsDir,
 				providerId: PROVIDER_ID,
 				level: scopeToLevel(root.scope),
+				signal: ctx.signal,
 			});
 			return { root, result };
 		}),

package/src/discovery/claude.ts

@@ -132,7 +132,7 @@ async function loadContextFiles(ctx: LoadContext): Promise<LoadResult<ContextFil
 	const userBase = getUserClaude(ctx);
 	const userClaudeMd = path.join(userBase, "CLAUDE.md");
 
-	const userContent = await readFile(userClaudeMd);
+	const userContent = await readFile(userClaudeMd, ctx.signal);
 	if (userContent !== null) {
 		items.push({
 			path: userClaudeMd,
@@ -144,7 +144,7 @@ async function loadContextFiles(ctx: LoadContext): Promise<LoadResult<ContextFil
 
 	const projectBase = getProjectClaude(ctx);
 	const projectClaudeMd = path.join(projectBase, "CLAUDE.md");
-	const projectContent = await readFile(projectClaudeMd);
+	const projectContent = await readFile(projectClaudeMd, ctx.signal);
 	if (projectContent !== null) {
 		const depth = calculateDepth(ctx.cwd, path.dirname(projectBase), path.sep);
 		items.push({
@@ -175,6 +175,7 @@ async function loadSkills(ctx: LoadContext): Promise<LoadResult<Skill>> {
 				dir: path.join(current, CONFIG_DIR, "skills"),
 				providerId: PROVIDER_ID,
 				level: "project",
+				signal: ctx.signal,
 			}),
 		);
 		if (current === (ctx.repoRoot ?? ctx.home)) break;
@@ -184,7 +185,7 @@ async function loadSkills(ctx: LoadContext): Promise<LoadResult<Skill>> {
 	}
 
 	const [userResult, ...projectResults] = await Promise.allSettled([
-		scanSkillsFromDir(ctx, { dir: userSkillsDir, providerId: PROVIDER_ID, level: "user" }),
+		scanSkillsFromDir(ctx, { dir: userSkillsDir, providerId: PROVIDER_ID, level: "user", signal: ctx.signal }),
 		...projectScans,
 	]);
 
@@ -407,7 +408,7 @@ async function loadSystemPrompts(ctx: LoadContext): Promise<LoadResult<SystemPro
 	const userBase = getUserClaude(ctx);
 	const userSystemMd = path.join(userBase, "SYSTEM.md");
 
-	const content = await readFile(userSystemMd);
+	const content = await readFile(userSystemMd, ctx.signal);
 	if (content !== null) {
 		items.push({
 			path: userSystemMd,

package/src/discovery/codex.ts

@@ -56,7 +56,7 @@ async function loadContextFiles(ctx: LoadContext): Promise<LoadResult<ContextFil
 
 	// User level only: ~/.codex/AGENTS.md
 	const agentsMd = path.join(ctx.home, SOURCE_PATHS.codex.userBase, "AGENTS.md");
-	const agentsContent = await readFile(agentsMd);
+	const agentsContent = await readFile(agentsMd, ctx.signal);
 	if (agentsContent) {
 		items.push({
 			path: agentsMd,
@@ -221,11 +221,13 @@ async function loadSkills(ctx: LoadContext): Promise<LoadResult<Skill>> {
 			dir: userSkillsDir,
 			providerId: PROVIDER_ID,
 			level: "user",
+			signal: ctx.signal,
 		}),
 		scanSkillsFromDir(ctx, {
 			dir: projectSkillsDir,
 			providerId: PROVIDER_ID,
 			level: "project",
+			signal: ctx.signal,
 		}),
 	]);
 

package/src/discovery/gemini.ts

@@ -129,7 +129,7 @@ async function loadContextFiles(ctx: LoadContext): Promise<LoadResult<ContextFil
 	// User-level: ~/.gemini/GEMINI.md
 	const userGeminiMd = getUserPath(ctx, "gemini", "GEMINI.md");
 	if (userGeminiMd) {
-		const content = await readFile(userGeminiMd);
+		const content = await readFile(userGeminiMd, ctx.signal);
 		if (content) {
 			items.push({
 				path: userGeminiMd,
@@ -143,7 +143,7 @@ async function loadContextFiles(ctx: LoadContext): Promise<LoadResult<ContextFil
 	// Project-level: .gemini/GEMINI.md
 	const projectGeminiMd = getProjectPath(ctx, "gemini", "GEMINI.md");
 	if (projectGeminiMd) {
-		const content = await readFile(projectGeminiMd);
+		const content = await readFile(projectGeminiMd, ctx.signal);
 		if (content) {
 			const projectBase = getProjectPath(ctx, "gemini", "");
 			const depth = projectBase ? calculateDepth(ctx.cwd, projectBase, path.sep) : 0;
@@ -335,7 +335,7 @@ async function loadSystemPrompt(ctx: LoadContext): Promise<LoadResult<SystemProm
 	// User-level: ~/.gemini/system.md
 	const userSystemMd = getUserPath(ctx, "gemini", "system.md");
 	if (userSystemMd) {
-		const content = await readFile(userSystemMd);
+		const content = await readFile(userSystemMd, ctx.signal);
 		if (content) {
 			items.push({
 				path: userSystemMd,
@@ -349,7 +349,7 @@ async function loadSystemPrompt(ctx: LoadContext): Promise<LoadResult<SystemProm
 	// Project-level: .gemini/system.md
 	const projectSystemMd = getProjectPath(ctx, "gemini", "system.md");
 	if (projectSystemMd) {
-		const content = await readFile(projectSystemMd);
+		const content = await readFile(projectSystemMd, ctx.signal);
 		if (content) {
 			items.push({
 				path: projectSystemMd,

package/src/discovery/github.ts

@@ -35,7 +35,7 @@ async function loadContextFiles(ctx: LoadContext): Promise<LoadResult<ContextFil
 
 	const copilotInstructionsPath = getProjectPath(ctx, "github", "copilot-instructions.md");
 	if (copilotInstructionsPath) {
-		const content = await readFile(copilotInstructionsPath);
+		const content = await readFile(copilotInstructionsPath, ctx.signal);
 		if (content) {
 			const fileDir = path.dirname(copilotInstructionsPath);
 			const depth = calculateDepth(ctx.cwd, fileDir, path.sep);