bip-skills 1.4.12 → 1.4.14

package/README.md

@@ -20,8 +20,6 @@ After a global install, you can run:
 
 ```bash
 bip-skills --help
-skills --help
-add-skill --help
 ```
 
 ## Install a Skill

package/dist/cli.mjs

@@ -692,6 +692,7 @@ function getYonClawBaseDir(options = {}) {
 	const home = options.homeDir ?? homedir();
 	const currentPlatform = options.platform ?? platform();
 	const env = options.env ?? process.env;
+	if (env.YONCLAW_HOME) return env.YONCLAW_HOME;
 	if (currentPlatform === "darwin") return join(home, "Library", "Application Support", "yonclaw");
 	if (currentPlatform === "win32") return join(env.APPDATA || join(home, "AppData", "Roaming"), "yonclaw");
 	return join(options.xdgConfigDir || env.XDG_CONFIG_HOME || xdgConfig || join(home, ".config"), "yonclaw");
@@ -2239,7 +2240,7 @@ function createEmptyLocalLock() {
 		skills: {}
 	};
 }
-var version$1 = "1.4.12";
+var version$1 = "1.4.14";
 const isCancelled$1 = (value) => typeof value === "symbol";
 async function isSourcePrivate(source) {
 	const ownerRepo = parseOwnerRepo(source);
@@ -2385,22 +2386,38 @@ function agentSupportsScope(agent, global) {
 	if (global === false) return agents[agent].supportsProjectInstall !== false;
 	return true;
 }
-function filterProjectUnsupportedAgents(targetAgents, installGlobally, explicitlyRequestedAgents) {
-	if (installGlobally) return targetAgents;
+function resolveAgentScopes(targetAgents, installGlobally, explicitlyRequestedAgents) {
+	if (installGlobally) return {
+		targetAgents,
+		autoGlobalAgents: []
+	};
 	const unsupported = targetAgents.filter((agent) => agents[agent].supportsProjectInstall === false);
-	if (unsupported.length === 0) return targetAgents;
-	const explicitUnsupported = explicitlyRequestedAgents?.filter((agent) => agent !== "*" && unsupported.includes(agent));
-	if (explicitUnsupported && explicitUnsupported.length > 0) {
-		M.error(`${formatList$1(unsupported.map((agent) => agents[agent].displayName))} only supports global skill installation. Use --global.`);
-		process.exit(1);
-	}
-	M.warn(`Skipping ${formatList$1(unsupported.map((agent) => agents[agent].displayName))}: global-only agent`);
-	const filtered = targetAgents.filter((agent) => agents[agent].supportsProjectInstall !== false);
-	if (filtered.length === 0) {
-		M.error("No selected agents support project skill installation.");
-		process.exit(1);
+	if (unsupported.length === 0) return {
+		targetAgents,
+		autoGlobalAgents: []
+	};
+	if (explicitlyRequestedAgents?.includes("*")) {
+		const filtered = targetAgents.filter((agent) => agents[agent].supportsProjectInstall !== false);
+		if (filtered.length === 0) return {
+			targetAgents: unsupported,
+			autoGlobalAgents: unsupported
+		};
+		return {
+			targetAgents: filtered,
+			autoGlobalAgents: []
+		};
 	}
-	return filtered;
+	return {
+		targetAgents,
+		autoGlobalAgents: unsupported
+	};
+}
+function shouldInstallAgentGlobally(agent, installGlobally, autoGlobalAgents) {
+	return installGlobally || autoGlobalAgents.includes(agent);
+}
+function warnAutoGlobalAgents(autoGlobalAgents) {
+	if (autoGlobalAgents.length === 0) return;
+	M.warn(`${formatList$1(autoGlobalAgents.map((agent) => agents[agent].displayName))} only supports global skill installation and was installed globally.`);
 }
 function buildResultLines(results, targetAgents) {
 	const lines = [];
@@ -2658,7 +2675,9 @@ async function handleWellKnownSkills(source, url, options, spinner, allowFallbac
 			}
 			installGlobally = scope;
 		}
-		targetAgents = filterProjectUnsupportedAgents(targetAgents, installGlobally, options.agent);
+		const scopeResolution = resolveAgentScopes(targetAgents, installGlobally, options.agent);
+		targetAgents = scopeResolution.targetAgents;
+		const autoGlobalAgents = scopeResolution.autoGlobalAgents;
 		let installMode = options.copy ? "copy" : "symlink";
 		if (!options.copy && !options.yes) {
 			const modeChoice = await ve({
@@ -2685,7 +2704,7 @@ async function handleWellKnownSkills(source, url, options, spinner, allowFallbac
 		const overwriteChecks = await Promise.all(selectedSkills.flatMap((skill) => targetAgents.map(async (agent) => ({
 			skillName: skill.installName,
 			agent,
-			installed: await isSkillInstalled(skill.installName, agent, { global: installGlobally })
+			installed: await isSkillInstalled(skill.installName, agent, { global: shouldInstallAgentGlobally(agent, installGlobally, autoGlobalAgents) })
 		}))));
 		const overwriteStatus = /* @__PURE__ */ new Map();
 		for (const { skillName, agent, installed } of overwriteChecks) {
@@ -2694,7 +2713,7 @@ async function handleWellKnownSkills(source, url, options, spinner, allowFallbac
 		}
 		for (const skill of selectedSkills) {
 			if (summaryLines.length > 0) summaryLines.push("");
-			const shortCanonical = shortenPath$2(getCanonicalPath(skill.installName, { global: installGlobally }), cwd);
+			const shortCanonical = shortenPath$2(getCanonicalPath(skill.installName, { global: installGlobally || targetAgents.every((agent) => autoGlobalAgents.includes(agent)) }), cwd);
 			summaryLines.push(`${import_picocolors.default.cyan(shortCanonical)}`);
 			summaryLines.push(...buildAgentSummaryLines(targetAgents, installMode));
 			const fileCount = getWellKnownFileCount(skill);
@@ -2747,11 +2766,11 @@ async function handleWellKnownSkills(source, url, options, spinner, allowFallbac
 					error: "Repository-backed skill was not prepared"
 				};
 				else installOutcome = await installSkillForAgent(preparedSkill, agent, {
-					global: installGlobally,
+					global: shouldInstallAgentGlobally(agent, installGlobally, autoGlobalAgents),
 					mode: installMode
 				});
 			} else installOutcome = await installWellKnownSkillForAgent(skill, agent, {
-				global: installGlobally,
+				global: shouldInstallAgentGlobally(agent, installGlobally, autoGlobalAgents),
 				mode: installMode
 			});
 			results.push({
@@ -2762,9 +2781,11 @@ async function handleWellKnownSkills(source, url, options, spinner, allowFallbac
 		}
 		await Promise.all(repositoryTempDirs.map((dir) => cleanupTempDir(dir).catch(() => {})));
 		spinner.stop("Installation complete");
+		warnAutoGlobalAgents(autoGlobalAgents);
 		console.log();
 		const successful = results.filter((r) => r.success);
 		const failed = results.filter((r) => !r.success);
+		const hasAnyGlobalInstall = installGlobally || autoGlobalAgents.length > 0;
 		const sourceIdentifier = wellKnownProvider.getSourceIdentifier(url);
 		const skillFiles = {};
 		for (const skill of selectedSkills) skillFiles[skill.installName] = skill.sourceUrl;
@@ -2773,7 +2794,7 @@ async function handleWellKnownSkills(source, url, options, spinner, allowFallbac
 			source: sourceIdentifier,
 			skills: selectedSkills.map((s) => s.installName).join(","),
 			agents: targetAgents.join(","),
-			...installGlobally && { global: "1" },
+			...hasAnyGlobalInstall && { global: "1" },
 			skillFiles: JSON.stringify(skillFiles),
 			sourceType: "well-known"
 		});
@@ -2782,14 +2803,14 @@ async function handleWellKnownSkills(source, url, options, spinner, allowFallbac
 			sourceUrl: url,
 			skills: selectedSkills.map((s) => s.installName),
 			agents: targetAgents,
-			installGlobally,
+			installGlobally: hasAnyGlobalInstall,
 			skillFiles,
 			sourceType: "well-known",
 			successfulCount: successful.length,
 			failedCount: failed.length,
 			failedDetails: buildFailedDetails(results)
 		});
-		if (successful.length > 0 && installGlobally) {
+		if (successful.length > 0 && hasAnyGlobalInstall) {
 			const successfulSkillNames = new Set(successful.map((r) => r.skill));
 			for (const skill of selectedSkills) if (successfulSkillNames.has(skill.installName)) try {
 				await addSkillToLock(skill.installName, {
@@ -2803,18 +2824,21 @@ async function handleWellKnownSkills(source, url, options, spinner, allowFallbac
 		}
 		if (successful.length > 0 && !installGlobally) {
 			const successfulSkillNames = new Set(successful.map((r) => r.skill));
-			for (const skill of selectedSkills) if (successfulSkillNames.has(skill.installName)) try {
-				const matchingResult = successful.find((r) => r.skill === skill.installName);
-				const installDir = matchingResult?.canonicalPath || matchingResult?.path;
-				if (installDir) {
-					const computedHash = await computeSkillFolderHash(installDir);
-					await addSkillToLocalLock(skill.installName, {
-						source: sourceIdentifier,
-						sourceType: "well-known",
-						computedHash
-					}, cwd);
-				}
-			} catch {}
+			for (const skill of selectedSkills) if (successfulSkillNames.has(skill.installName)) {
+				if (!successful.some((r) => r.skill === skill.installName && targetAgents.some((agent) => agents[agent].displayName === r.agent && !shouldInstallAgentGlobally(agent, installGlobally, autoGlobalAgents)))) continue;
+				try {
+					const matchingResult = successful.find((r) => r.skill === skill.installName);
+					const installDir = matchingResult?.canonicalPath || matchingResult?.path;
+					if (installDir) {
+						const computedHash = await computeSkillFolderHash(installDir);
+						await addSkillToLocalLock(skill.installName, {
+							source: sourceIdentifier,
+							sourceType: "well-known",
+							computedHash
+						}, cwd);
+					}
+				} catch {}
+			}
 		}
 		if (successful.length > 0) {
 			const bySkill = /* @__PURE__ */ new Map();
@@ -3119,7 +3143,9 @@ async function runAdd(args, options = {}) {
 			}
 			installGlobally = scope;
 		}
-		targetAgents = filterProjectUnsupportedAgents(targetAgents, installGlobally, options.agent);
+		const scopeResolution = resolveAgentScopes(targetAgents, installGlobally, options.agent);
+		targetAgents = scopeResolution.targetAgents;
+		const autoGlobalAgents = scopeResolution.autoGlobalAgents;
 		let installMode = options.copy ? "copy" : "symlink";
 		if (!options.copy && !options.yes) {
 			const modeChoice = await ve({
@@ -3147,7 +3173,7 @@ async function runAdd(args, options = {}) {
 		const overwriteChecks = await Promise.all(selectedSkills.flatMap((skill) => targetAgents.map(async (agent) => ({
 			skillName: skill.name,
 			agent,
-			installed: await isSkillInstalled(skill.name, agent, { global: installGlobally })
+			installed: await isSkillInstalled(skill.name, agent, { global: shouldInstallAgentGlobally(agent, installGlobally, autoGlobalAgents) })
 		}))));
 		const overwriteStatus = /* @__PURE__ */ new Map();
 		for (const { skillName, agent, installed } of overwriteChecks) {
@@ -3164,7 +3190,7 @@ async function runAdd(args, options = {}) {
 		const printSkillSummary = (skills) => {
 			for (const skill of skills) {
 				if (summaryLines.length > 0) summaryLines.push("");
-				const shortCanonical = shortenPath$2(getCanonicalPath(skill.name, { global: installGlobally }), cwd);
+				const shortCanonical = shortenPath$2(getCanonicalPath(skill.name, { global: installGlobally || targetAgents.every((agent) => autoGlobalAgents.includes(agent)) }), cwd);
 				summaryLines.push(`${import_picocolors.default.cyan(shortCanonical)}`);
 				summaryLines.push(...buildAgentSummaryLines(targetAgents, installMode));
 				const skillOverwrites = overwriteStatus.get(skill.name);
@@ -3210,7 +3236,7 @@ async function runAdd(args, options = {}) {
 		const results = [];
 		for (const skill of selectedSkills) for (const agent of targetAgents) {
 			const result = await installSkillForAgent(skill, agent, {
-				global: installGlobally,
+				global: shouldInstallAgentGlobally(agent, installGlobally, autoGlobalAgents),
 				mode: installMode
 			});
 			results.push({
@@ -3221,9 +3247,11 @@ async function runAdd(args, options = {}) {
 			});
 		}
 		spinner.stop("Installation complete");
+		warnAutoGlobalAgents(autoGlobalAgents);
 		console.log();
 		const successful = results.filter((r) => r.success);
 		const failed = results.filter((r) => !r.success);
+		const hasAnyGlobalInstall = installGlobally || autoGlobalAgents.length > 0;
 		const skillFiles = {};
 		for (const skill of selectedSkills) {
 			let relativePath;
@@ -3238,7 +3266,7 @@ async function runAdd(args, options = {}) {
 			sourceUrl: parsed.url,
 			skills: selectedSkills.map((s) => s.name),
 			agents: targetAgents,
-			installGlobally,
+			installGlobally: hasAnyGlobalInstall,
 			sourceType: parsed.type,
 			skillFiles
 		};
@@ -3250,7 +3278,7 @@ async function runAdd(args, options = {}) {
 					source: normalizedSource,
 					skills: selectedSkills.map((s) => s.name).join(","),
 					agents: targetAgents.join(","),
-					...installGlobally && { global: "1" },
+					...hasAnyGlobalInstall && { global: "1" },
 					skillFiles: JSON.stringify(skillFiles)
 				});
 			} else track({
@@ -3258,7 +3286,7 @@ async function runAdd(args, options = {}) {
 				source: normalizedSource,
 				skills: selectedSkills.map((s) => s.name).join(","),
 				agents: targetAgents.join(","),
-				...installGlobally && { global: "1" },
+				...hasAnyGlobalInstall && { global: "1" },
 				skillFiles: JSON.stringify(skillFiles)
 			});
 		}
@@ -3274,7 +3302,7 @@ async function runAdd(args, options = {}) {
 			failedCount: failed.length,
 			failedDetails: buildFailedDetails(results)
 		});
-		if (successful.length > 0 && installGlobally && normalizedSource) {
+		if (successful.length > 0 && (installGlobally || autoGlobalAgents.length > 0) && normalizedSource) {
 			const successfulSkillNames = new Set(successful.map((r) => r.skill));
 			for (const skill of selectedSkills) {
 				const skillDisplayName = getSkillDisplayName(skill);
@@ -3300,14 +3328,17 @@ async function runAdd(args, options = {}) {
 			const successfulSkillNames = new Set(successful.map((r) => r.skill));
 			for (const skill of selectedSkills) {
 				const skillDisplayName = getSkillDisplayName(skill);
-				if (successfulSkillNames.has(skillDisplayName)) try {
-					const computedHash = await computeSkillFolderHash(skill.path);
-					await addSkillToLocalLock(skill.name, {
-						source: normalizedSource || parsed.url,
-						sourceType: parsed.type,
-						computedHash
-					}, cwd);
-				} catch {}
+				if (successfulSkillNames.has(skillDisplayName)) {
+					if (!successful.some((r) => r.skill === skillDisplayName && targetAgents.some((agent) => agents[agent].displayName === r.agent && !shouldInstallAgentGlobally(agent, installGlobally, autoGlobalAgents)))) continue;
+					try {
+						const computedHash = await computeSkillFolderHash(skill.path);
+						await addSkillToLocalLock(skill.name, {
+							source: normalizedSource || parsed.url,
+							sourceType: parsed.type,
+							computedHash
+						}, cwd);
+					} catch {}
+				}
 			}
 		}
 		if (successful.length > 0) {
@@ -3475,6 +3506,12 @@ function parseAddOptions(args) {
 			i--;
 		} else if (arg === "--full-depth") options.fullDepth = true;
 		else if (arg === "--copy") options.copy = true;
+		else if (arg && /^-[ygld]+$/.test(arg)) for (const flag of arg.slice(1)) {
+			if (flag === "y") options.yes = true;
+			if (flag === "g") options.global = true;
+			if (flag === "l") options.list = true;
+			if (flag === "d") options.fullDepth = true;
+		}
 		else if (arg && !arg.startsWith("-")) source.push(arg);
 	}
 	return {

package/package.json

@@ -1,12 +1,10 @@
 {
   "name": "bip-skills",
-  "version": "1.4.12",
+  "version": "1.4.14",
   "description": "The open agent skills ecosystem",
   "type": "module",
   "bin": {
-    "bip-skills": "bin/cli.mjs",
-    "skills": "bin/cli.mjs",
-    "add-skill": "bin/cli.mjs"
+    "bip-skills": "bin/cli.mjs"
   },
   "files": [
     "dist",