@tankpkg/cli 0.16.3 → 0.16.5

package/dist/bin/tank.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { a as VERSION, i as USER_AGENT, l as setConfig, n as flushLogs, o as getConfig, s as getConfigDir, t as authFlowLog } from "../debug-logger-Ds5BG5rk.js";
+import { a as VERSION, i as USER_AGENT, l as setConfig, n as flushLogs, o as getConfig, s as getConfigDir, t as authFlowLog } from "../debug-logger-BpF5Gl-T.js";
 import { t as logger } from "../logger-BhULz3Uz.js";
 import { createRequire } from "node:module";
 import { Command } from "commander";
@@ -21,6 +21,7 @@ import { pipeline } from "node:stream/promises";
 import open from "open";
 import ignore from "ignore";
 import { fileURLToPath } from "node:url";
+import { inspect } from "node:util";
 //#region \0rolldown/runtime.js
 var __defProp$1 = Object.defineProperty;
 var __exportAll = (all, no_symbols) => {
@@ -9067,18 +9068,31 @@ function writeLockfileWithResolvedGraph(lock, nodes, downloaded, onProgress) {
 	lock.skills = sortedSkills;
 	return lock;
 }
-function mkdirSafeNoSymlinks(targetDir) {
-	const resolved = path.resolve(targetDir);
-	const segments = resolved.split(path.sep).filter(Boolean);
-	let current = resolved.startsWith("/") ? "/" : "";
-	for (const seg of segments) {
-		current = path.join(current, seg);
-		if (fs.existsSync(current)) {
-			const stat = fs.lstatSync(current);
-			if (stat.isSymbolicLink()) throw new TankIntegrityError(`Symlink in install path: ${current}`);
-			if (!stat.isDirectory()) throw new TankIntegrityError(`Non-directory in install path: ${current}`);
-		} else fs.mkdirSync(current);
+/**
+* Plan the ordered list of ancestor directories to create for `targetDir`,
+* from just below the filesystem root down to the leaf (the root itself is
+* never returned — it always exists and must not be `mkdir`'d).
+*
+* The path module is injectable so Windows drive/UNC root handling can be
+* verified on any host via `path.win32` without a real Windows machine.
+*/
+function planMkdirSegments(targetDir, pathMod = path) {
+	const resolved = pathMod.resolve(targetDir);
+	const { root } = pathMod.parse(resolved);
+	const dirs = [];
+	let current = root;
+	for (const seg of resolved.slice(root.length).split(pathMod.sep).filter(Boolean)) {
+		current = pathMod.join(current, seg);
+		dirs.push(current);
 	}
+	return dirs;
+}
+function mkdirSafeNoSymlinks(targetDir) {
+	for (const current of planMkdirSegments(targetDir)) if (fs.existsSync(current)) {
+		const stat = fs.lstatSync(current);
+		if (stat.isSymbolicLink()) throw new TankIntegrityError(`Symlink in install path: ${current}`);
+		if (!stat.isDirectory()) throw new TankIntegrityError(`Non-directory in install path: ${current}`);
+	} else fs.mkdirSync(current);
 }
 function isDirNonEmpty(dir) {
 	try {
@@ -9965,11 +9979,11 @@ function wrapMcpServerForSkill(options) {
 		logger.warn(`MCP proxy wrapping skipped for ${options.skillName}: ${msg}`);
 	}
 }
-async function linkInstalledRoots(options) {
-	const { rootSkillNames, resolvedNodeByName, extractDirForSkill, directory, global, resolvedHome, homedir, dangerouslyNoTankProxy } = options;
+async function linkInstalledSkills(options) {
+	const { skillNames, resolvedNodeByName, extractDirForSkill, directory, global, resolvedHome, homedir, dangerouslyNoTankProxy } = options;
 	const agentSkillsBaseDir = global ? getGlobalAgentSkillsDir(resolvedHome) : path.join(directory, ".tank", "agent-skills");
 	const linksDir = global ? path.join(resolvedHome, ".tank") : path.join(directory, ".tank");
-	for (const skillName of rootSkillNames) try {
+	for (const skillName of skillNames) try {
 		const node = resolvedNodeByName.get(skillName);
 		if (!node) continue;
 		const linkResult = linkSkillToAgents({
@@ -9993,7 +10007,7 @@ async function linkInstalledRoots(options) {
 			dangerouslyNoTankProxy
 		});
 	} catch {
-		if (rootSkillNames.length === 1) logger.warn("Agent linking skipped (non-fatal)");
+		if (skillNames.length === 1) logger.warn("Agent linking skipped (non-fatal)");
 		else logger.warn(`Agent linking skipped for ${skillName} (non-fatal)`);
 	}
 	const detectedAgents = detectInstalledAgents(homedir);
@@ -10006,7 +10020,7 @@ async function linkInstalledRoots(options) {
 		openclaw: "opencode"
 	};
 	const platforms = new Set(detectedAgents.map((a) => agentToPlatform[a.id]).filter(Boolean));
-	for (const skillName of rootSkillNames) {
+	for (const skillName of skillNames) {
 		if (!resolvedNodeByName.get(skillName)) continue;
 		const skillDir = extractDirForSkill(skillName);
 		const manifestPath = path.join(skillDir, "tank.json");
@@ -10057,8 +10071,8 @@ async function executeInstallPipeline(options) {
 		global,
 		homedir
 	});
-	await linkInstalledRoots({
-		rootSkillNames,
+	await linkInstalledSkills({
+		skillNames: nodesToInstall.map((node) => node.name),
 		resolvedNodeByName,
 		extractDirForSkill,
 		directory,
@@ -12084,6 +12098,37 @@ function printUserInfo(user) {
 	logger.info(`Email: ${user.email ?? "unknown"}`);
 }
 //#endregion
+//#region src/lib/describe-error.ts
+/** Format a real Error, surfacing Node errno fields (code/syscall/path) when present. */
+function describeErrorObject(err) {
+	const e = err;
+	const base = err.message?.trim() || err.name || "Error";
+	const tags = [
+		e.code != null ? `code=${String(e.code)}` : null,
+		e.syscall != null ? `syscall=${String(e.syscall)}` : null,
+		e.path != null ? `path=${String(e.path)}` : null
+	].filter((tag) => tag !== null);
+	return tags.length > 0 ? `${base} (${tags.join(", ")})` : base;
+}
+/**
+* Render an unknown thrown value as a non-empty, actionable string.
+*
+* Guarantees a useful message for every input: errno errors surface their
+* `code`/`syscall`/`path`, non-Error objects are inspected, and the result
+* never collapses to an empty string or a bare placeholder like `<none>`
+* (the symptom Windows users hit during `tank install`).
+*/
+function describeError(err) {
+	if (err instanceof Error) return describeErrorObject(err);
+	if (err === null || err === void 0) return `${String(err)} (non-error thrown)`;
+	if (typeof err === "object") try {
+		return `${err.constructor?.name ?? "object"}: ${inspect(err, { depth: 2 })}`;
+	} catch {
+		return Object.prototype.toString.call(err);
+	}
+	return `${typeof err} ${String(err)}`;
+}
+//#endregion
 //#region src/lib/install-suggestions.ts
 /**
 * Best-effort fuzzy lookup of similar skill names. Hits the public /api/v1/search
@@ -12304,7 +12349,7 @@ program.command("install").alias("i").description("Install one or more skills fr
 				...proxyOpt
 			});
 		} catch (err) {
-			const msg = err instanceof Error ? err.message : String(err);
+			const msg = describeError(err);
 			console.error(`Install failed: ${msg}`);
 			process.exit(1);
 		}
@@ -12327,7 +12372,7 @@ program.command("install").alias("i").description("Install one or more skills fr
 				...proxyOpt
 			});
 		} catch (err) {
-			const msg = err instanceof Error ? err.message : String(err);
+			const msg = describeError(err);
 			failures.push({
 				target,
 				error: msg,