@fluid-app/fluid-cli-theme-dev 0.1.23 → 0.1.25

package/dist/index.mjs

@@ -2370,7 +2370,32 @@ function createNavigateCommand() {
 	});
 }
 //#endregion
-//#region src/fs/replace-directory.ts
+//#region src/skills/install.ts
+function listSkillNames(skillsDir) {
+	if (!existsSync(skillsDir)) return [];
+	return readdirSync(skillsDir, { withFileTypes: true }).filter((entry) => entry.isDirectory()).map((entry) => entry.name).filter((name) => existsSync(join(skillsDir, name, "SKILL.md"))).sort();
+}
+async function installSkills(options) {
+	const { sourceDir, targetRoot, force, confirmOverwrite, onLeftover } = options;
+	const installed = [];
+	const skipped = [];
+	mkdirSync(targetRoot, { recursive: true });
+	for (const name of listSkillNames(sourceDir)) {
+		const from = join(sourceDir, name);
+		const to = join(targetRoot, name);
+		if (existsSync(to) && !force && !await confirmOverwrite(name)) {
+			skipped.push(name);
+			continue;
+		}
+		const leftover = replaceDirectory(from, to);
+		if (leftover !== null) onLeftover(leftover);
+		installed.push(name);
+	}
+	return {
+		installed,
+		skipped
+	};
+}
 /**
 * Replace `target` with a fresh copy of `source` without ever leaving `target`
 * missing or partially written.
@@ -2388,49 +2413,39 @@ function createNavigateCommand() {
 *
 * Not safe against a second process racing on the same `target`; intended for
 * single-process CLI use.
+*
+* @returns the path of a leftover backup directory that could not be removed
+* after an otherwise-successful replace (the previous contents are retained
+* there for manual cleanup), or `null` when nothing was left behind. The caller
+* should surface a non-null result so the leftover isn't silently hidden.
 */
 function replaceDirectory(source, target) {
 	const staging = reserveSiblingPath(target, "staging");
 	try {
 		cpSync(source, staging, { recursive: true });
 	} catch (error) {
-		rmSync(staging, {
-			recursive: true,
-			force: true
-		});
+		removeQuietly(staging);
 		throw error;
 	}
-	if (!existsSync(target)) {
-		swapIntoPlace(staging, target, null);
-		return;
-	}
+	if (!existsSync(target)) return swapIntoPlace(staging, target, null);
 	const backup = reserveSiblingPath(target, "backup");
 	try {
 		renameSync(target, backup);
 	} catch (error) {
-		rmSync(staging, {
-			recursive: true,
-			force: true
-		});
+		removeQuietly(staging);
 		throw error;
 	}
-	swapIntoPlace(staging, target, backup);
+	return swapIntoPlace(staging, target, backup);
 }
 function swapIntoPlace(staging, target, backup) {
 	try {
 		renameSync(staging, target);
 	} catch (error) {
-		rmSync(staging, {
-			recursive: true,
-			force: true
-		});
 		if (backup !== null) restoreBackup(backup, target, error);
+		removeQuietly(staging);
 		throw error;
 	}
-	if (backup !== null) rmSync(backup, {
-		recursive: true,
-		force: true
-	});
+	return removeQuietly(backup);
 }
 function restoreBackup(backup, target, cause) {
 	try {
@@ -2439,38 +2454,24 @@ function restoreBackup(backup, target, cause) {
 		throw new Error(`Failed to replace ${target}; its previous contents are preserved at ${backup}.`, { cause });
 	}
 }
+function removeQuietly(path) {
+	if (path === null) return null;
+	try {
+		rmSync(path, {
+			recursive: true,
+			force: true
+		});
+		return null;
+	} catch {
+		return path;
+	}
+}
 function reserveSiblingPath(basePath, label) {
 	let candidate = `${basePath}.${label}`;
 	for (let n = 1; existsSync(candidate); n += 1) candidate = `${basePath}.${label}.${n}`;
 	return candidate;
 }
 //#endregion
-//#region src/skills/install.ts
-function listSkillNames(skillsDir) {
-	if (!existsSync(skillsDir)) return [];
-	return readdirSync(skillsDir, { withFileTypes: true }).filter((entry) => entry.isDirectory()).map((entry) => entry.name).filter((name) => existsSync(join(skillsDir, name, "SKILL.md"))).sort();
-}
-async function installSkills(options) {
-	const { sourceDir, targetRoot, force, confirmOverwrite } = options;
-	const installed = [];
-	const skipped = [];
-	mkdirSync(targetRoot, { recursive: true });
-	for (const name of listSkillNames(sourceDir)) {
-		const from = join(sourceDir, name);
-		const to = join(targetRoot, name);
-		if (existsSync(to) && !force && !await confirmOverwrite(name)) {
-			skipped.push(name);
-			continue;
-		}
-		replaceDirectory(from, to);
-		installed.push(name);
-	}
-	return {
-		installed,
-		skipped
-	};
-}
-//#endregion
 //#region src/commands/skills.ts
 const DEFAULT_TARGET_DIR = ".agents/skills";
 function resolveBundledSkillsDir() {
@@ -2510,6 +2511,9 @@ function createSkillsCommand() {
 					initial: false
 				}, { onCancel: () => process.exit(130) });
 				return Boolean(res.overwrite);
+			},
+			onLeftover: (path) => {
+				console.log(`${chalk.yellow("⚠")} kept the previous copy at ${path} (couldn't remove it — delete it manually)`);
 			}
 		});
 		for (const name of installed) console.log(`${chalk.green("✓")} ${name} → ${join(opts.dir, name)}`);