skills 1.3.4 → 1.3.5

package/README.md

@@ -215,14 +215,14 @@ Skills can be installed to any of these agents:
 | OpenClaw | `openclaw` | `skills/` | `~/.moltbot/skills/` |
 | Cline | `cline` | `.cline/skills/` | `~/.cline/skills/` |
 | CodeBuddy | `codebuddy` | `.codebuddy/skills/` | `~/.codebuddy/skills/` |
-| Codex | `codex` | `.codex/skills/` | `~/.codex/skills/` |
+| Codex | `codex` | `.agents/skills/` | `~/.codex/skills/` |
 | Command Code | `command-code` | `.commandcode/skills/` | `~/.commandcode/skills/` |
 | Continue | `continue` | `.continue/skills/` | `~/.continue/skills/` |
 | Crush | `crush` | `.crush/skills/` | `~/.config/crush/skills/` |
 | Cursor | `cursor` | `.cursor/skills/` | `~/.cursor/skills/` |
 | Droid | `droid` | `.factory/skills/` | `~/.factory/skills/` |
-| Gemini CLI | `gemini-cli` | `.gemini/skills/` | `~/.gemini/skills/` |
-| GitHub Copilot | `github-copilot` | `.github/skills/` | `~/.copilot/skills/` |
+| Gemini CLI | `gemini-cli` | `.agents/skills/` | `~/.gemini/skills/` |
+| GitHub Copilot | `github-copilot` | `.agents/skills/` | `~/.copilot/skills/` |
 | Goose | `goose` | `.goose/skills/` | `~/.config/goose/skills/` |
 | Junie | `junie` | `.junie/skills/` | `~/.junie/skills/` |
 | iFlow CLI | `iflow-cli` | `.iflow/skills/` | `~/.iflow/skills/` |
@@ -232,12 +232,12 @@ Skills can be installed to any of these agents:
 | MCPJam | `mcpjam` | `.mcpjam/skills/` | `~/.mcpjam/skills/` |
 | Mistral Vibe | `mistral-vibe` | `.vibe/skills/` | `~/.vibe/skills/` |
 | Mux | `mux` | `.mux/skills/` | `~/.mux/skills/` |
-| OpenCode | `opencode` | `.opencode/skills/` | `~/.config/opencode/skills/` |
+| OpenCode | `opencode` | `.agents/skills/` | `~/.config/opencode/skills/` |
 | OpenHands | `openhands` | `.openhands/skills/` | `~/.openhands/skills/` |
 | Pi | `pi` | `.pi/skills/` | `~/.pi/agent/skills/` |
 | Qoder | `qoder` | `.qoder/skills/` | `~/.qoder/skills/` |
 | Qwen Code | `qwen-code` | `.qwen/skills/` | `~/.qwen/skills/` |
-| Replit | `replit` | `.agent/skills/` | N/A (project-only) |
+| Replit | `replit` | `.agents/skills/` | N/A (project-only) |
 | Roo Code | `roo` | `.roo/skills/` | `~/.roo/skills/` |
 | Trae | `trae` | `.trae/skills/` | `~/.trae/skills/` |
 | Trae CN | `trae-cn` | `.trae/skills/` | `~/.trae-cn/skills/` |
@@ -322,14 +322,11 @@ The CLI searches for skills in these locations within a repository:
 - `./skills/`
 - `.cline/skills/`
 - `.codebuddy/skills/`
-- `.codex/skills/`
 - `.commandcode/skills/`
 - `.continue/skills/`
 - `.crush/skills/`
 - `.cursor/skills/`
 - `.factory/skills/`
-- `.gemini/skills/`
-- `.github/skills/`
 - `.goose/skills/`
 - `.junie/skills/`
 - `.iflow/skills/`
@@ -339,7 +336,6 @@ The CLI searches for skills in these locations within a repository:
 - `.mcpjam/skills/`
 - `.vibe/skills/`
 - `.mux/skills/`
-- `.opencode/skills/`
 - `.openhands/skills/`
 - `.pi/skills/`
 - `.qoder/skills/`

package/dist/cli.mjs

@@ -173,10 +173,12 @@ const S_STEP_CANCEL = import_picocolors.default.red("■");
 const S_STEP_SUBMIT = import_picocolors.default.green("◇");
 const S_RADIO_ACTIVE = import_picocolors.default.green("●");
 const S_RADIO_INACTIVE = import_picocolors.default.dim("○");
+const S_CHECKBOX_LOCKED = import_picocolors.default.green("✓");
 const S_BAR = import_picocolors.default.dim("│");
+const S_BAR_H = import_picocolors.default.dim("─");
 const cancelSymbol = Symbol("cancel");
 async function searchMultiselect(options) {
-	const { message, items, maxVisible = 8, initialSelected = [], required = false } = options;
+	const { message, items, maxVisible = 8, initialSelected = [], required = false, lockedSection } = options;
 	return new Promise((resolve) => {
 		const rl = readline.createInterface({
 			input: process.stdin,
@@ -189,6 +191,7 @@ async function searchMultiselect(options) {
 		let cursor = 0;
 		const selected = new Set(initialSelected);
 		let lastRenderHeight = 0;
+		const lockedValues = lockedSection ? lockedSection.items.map((i) => i.value) : [];
 		const filter = (item, q) => {
 			if (!q) return true;
 			const lowerQ = q.toLowerCase();
@@ -211,6 +214,13 @@ async function searchMultiselect(options) {
 			const icon = state === "active" ? S_STEP_ACTIVE : state === "cancel" ? S_STEP_CANCEL : S_STEP_SUBMIT;
 			lines.push(`${icon}  ${import_picocolors.default.bold(message)}`);
 			if (state === "active") {
+				if (lockedSection && lockedSection.items.length > 0) {
+					lines.push(`${S_BAR}`);
+					lines.push(`${S_BAR}  ${S_BAR_H}${S_BAR_H} ${import_picocolors.default.bold(lockedSection.title)} ${S_BAR_H.repeat(30)}`);
+					for (const item of lockedSection.items) lines.push(`${S_BAR}    ${S_CHECKBOX_LOCKED} ${item.label}`);
+					lines.push(`${S_BAR}`);
+					lines.push(`${S_BAR}  ${S_BAR_H}${S_BAR_H} ${import_picocolors.default.bold("Other agents")} ${S_BAR_H.repeat(34)}`);
+				}
 				const searchLine = `${S_BAR}  ${import_picocolors.default.dim("Search:")} ${query}${import_picocolors.default.inverse(" ")}`;
 				lines.push(searchLine);
 				lines.push(`${S_BAR}  ${import_picocolors.default.dim("↑↓ move, space select, enter confirm")}`);
@@ -241,16 +251,16 @@ async function searchMultiselect(options) {
 					}
 				}
 				lines.push(`${S_BAR}`);
-				if (selected.size === 0) lines.push(`${S_BAR}  ${import_picocolors.default.dim("Selected: (none)")}`);
+				const allSelectedLabels = [...lockedSection ? lockedSection.items.map((i) => i.label) : [], ...items.filter((item) => selected.has(item.value)).map((item) => item.label)];
+				if (allSelectedLabels.length === 0) lines.push(`${S_BAR}  ${import_picocolors.default.dim("Selected: (none)")}`);
 				else {
-					const selectedLabels = items.filter((item) => selected.has(item.value)).map((item) => item.label);
-					const summary = selectedLabels.length <= 3 ? selectedLabels.join(", ") : `${selectedLabels.slice(0, 3).join(", ")} +${selectedLabels.length - 3} more`;
+					const summary = allSelectedLabels.length <= 3 ? allSelectedLabels.join(", ") : `${allSelectedLabels.slice(0, 3).join(", ")} +${allSelectedLabels.length - 3} more`;
 					lines.push(`${S_BAR}  ${import_picocolors.default.green("Selected:")} ${summary}`);
 				}
 				lines.push(`${import_picocolors.default.dim("└")}`);
 			} else if (state === "submit") {
-				const selectedLabels = items.filter((item) => selected.has(item.value)).map((item) => item.label);
-				lines.push(`${S_BAR}  ${import_picocolors.default.dim(selectedLabels.join(", "))}`);
+				const allSelectedLabels = [...lockedSection ? lockedSection.items.map((i) => i.label) : [], ...items.filter((item) => selected.has(item.value)).map((item) => item.label)];
+				lines.push(`${S_BAR}  ${import_picocolors.default.dim(allSelectedLabels.join(", "))}`);
 			} else if (state === "cancel") lines.push(`${S_BAR}  ${import_picocolors.default.strikethrough(import_picocolors.default.dim("Cancelled"))}`);
 			process.stdout.write(lines.join("\n") + "\n");
 			lastRenderHeight = lines.length;
@@ -261,10 +271,10 @@ async function searchMultiselect(options) {
 			rl.close();
 		};
 		const submit = () => {
-			if (required && selected.size === 0) return;
+			if (required && selected.size === 0 && lockedValues.length === 0) return;
 			render("submit");
 			cleanup();
-			resolve(Array.from(selected));
+			resolve([...lockedValues, ...Array.from(selected)]);
 		};
 		const cancel = () => {
 			render("cancel");
@@ -597,7 +607,7 @@ const agents = {
 	codex: {
 		name: "codex",
 		displayName: "Codex",
-		skillsDir: ".codex/skills",
+		skillsDir: ".agents/skills",
 		globalSkillsDir: join(codexHome, "skills"),
 		detectInstalled: async () => {
 			return existsSync(codexHome) || existsSync("/etc/codex");
@@ -651,7 +661,7 @@ const agents = {
 	"gemini-cli": {
 		name: "gemini-cli",
 		displayName: "Gemini CLI",
-		skillsDir: ".gemini/skills",
+		skillsDir: ".agents/skills",
 		globalSkillsDir: join(home, ".gemini/skills"),
 		detectInstalled: async () => {
 			return existsSync(join(home, ".gemini"));
@@ -660,7 +670,7 @@ const agents = {
 	"github-copilot": {
 		name: "github-copilot",
 		displayName: "GitHub Copilot",
-		skillsDir: ".github/skills",
+		skillsDir: ".agents/skills",
 		globalSkillsDir: join(home, ".copilot/skills"),
 		detectInstalled: async () => {
 			return existsSync(join(process.cwd(), ".github")) || existsSync(join(home, ".copilot"));
@@ -759,7 +769,7 @@ const agents = {
 	opencode: {
 		name: "opencode",
 		displayName: "OpenCode",
-		skillsDir: ".opencode/skills",
+		skillsDir: ".agents/skills",
 		globalSkillsDir: join(configHome, "opencode/skills"),
 		detectInstalled: async () => {
 			return existsSync(join(configHome, "opencode")) || existsSync(join(claudeHome, "skills"));
@@ -804,10 +814,10 @@ const agents = {
 	replit: {
 		name: "replit",
 		displayName: "Replit",
-		skillsDir: ".agent/skills",
+		skillsDir: ".agents/skills",
 		globalSkillsDir: void 0,
 		detectInstalled: async () => {
-			return existsSync(join(process.cwd(), ".agent"));
+			return existsSync(join(process.cwd(), ".agents"));
 		}
 	},
 	roo: {
@@ -889,6 +899,15 @@ async function detectInstalledAgents() {
 		installed: await config.detectInstalled()
 	})))).filter((r) => r.installed).map((r) => r.type);
 }
+function getUniversalAgents() {
+	return Object.entries(agents).filter(([_, config]) => config.skillsDir === ".agents/skills").map(([type]) => type);
+}
+function getNonUniversalAgents() {
+	return Object.entries(agents).filter(([_, config]) => config.skillsDir !== ".agents/skills").map(([type]) => type);
+}
+function isUniversalAgent(type) {
+	return agents[type].skillsDir === ".agents/skills";
+}
 const AGENTS_DIR$2 = ".agents";
 const SKILLS_SUBDIR = "skills";
 function sanitizeName(name) {
@@ -1196,22 +1215,30 @@ async function installWellKnownSkillForAgent(skill, agentType, options = {}) {
 }
 async function listInstalledSkills(options = {}) {
 	const cwd = options.cwd || process.cwd();
-	const installedSkills = [];
+	const skillsMap = /* @__PURE__ */ new Map();
 	const scopes = [];
 	const detectedAgents = await detectInstalledAgents();
-	if (options.global === void 0) {
-		scopes.push({
-			global: false,
-			path: getCanonicalSkillsDir(false, cwd)
-		});
+	const agentFilter = options.agentFilter;
+	const agentsToCheck = agentFilter ? detectedAgents.filter((a) => agentFilter.includes(a)) : detectedAgents;
+	const scopeTypes = [];
+	if (options.global === void 0) scopeTypes.push({ global: false }, { global: true });
+	else scopeTypes.push({ global: options.global });
+	for (const { global: isGlobal } of scopeTypes) {
 		scopes.push({
-			global: true,
-			path: getCanonicalSkillsDir(true, cwd)
+			global: isGlobal,
+			path: getCanonicalSkillsDir(isGlobal, cwd)
 		});
-	} else scopes.push({
-		global: options.global,
-		path: getCanonicalSkillsDir(options.global, cwd)
-	});
+		for (const agentType of agentsToCheck) {
+			const agent = agents[agentType];
+			if (isGlobal && agent.globalSkillsDir === void 0) continue;
+			const agentDir = isGlobal ? agent.globalSkillsDir : join(cwd, agent.skillsDir);
+			if (!scopes.some((s) => s.path === agentDir && s.global === isGlobal)) scopes.push({
+				global: isGlobal,
+				path: agentDir,
+				agentType
+			});
+		}
+	}
 	for (const scope of scopes) try {
 		const entries = await readdir(scope.path, { withFileTypes: true });
 		for (const entry of entries) {
@@ -1225,22 +1252,35 @@ async function listInstalledSkills(options = {}) {
 			}
 			const skill = await parseSkillMd(skillMdPath);
 			if (!skill) continue;
+			const scopeKey = scope.global ? "global" : "project";
+			const skillKey = `${scopeKey}:${skill.name}`;
+			if (scope.agentType) {
+				if (skillsMap.has(skillKey)) {
+					const existing = skillsMap.get(skillKey);
+					if (!existing.agents.includes(scope.agentType)) existing.agents.push(scope.agentType);
+				} else skillsMap.set(skillKey, {
+					name: skill.name,
+					description: skill.description,
+					path: skillDir,
+					canonicalPath: skillDir,
+					scope: scopeKey,
+					agents: [scope.agentType]
+				});
+				continue;
+			}
 			const sanitizedSkillName = sanitizeName(skill.name);
 			const installedAgents = [];
-			const agentFilter = options.agentFilter;
-			const agentsToCheck = agentFilter ? detectedAgents.filter((a) => agentFilter.includes(a)) : detectedAgents;
 			for (const agentType of agentsToCheck) {
 				const agent = agents[agentType];
 				if (scope.global && agent.globalSkillsDir === void 0) continue;
 				const agentBase = scope.global ? agent.globalSkillsDir : join(cwd, agent.skillsDir);
 				let found = false;
-				const possibleNames = [
+				const possibleNames = Array.from(new Set([
 					entry.name,
 					sanitizedSkillName,
 					skill.name.toLowerCase().replace(/\s+/g, "-").replace(/[\/\\:\0]/g, "")
-				];
-				const uniqueNames = Array.from(new Set(possibleNames));
-				for (const possibleName of uniqueNames) {
+				]));
+				for (const possibleName of possibleNames) {
 					const agentSkillDir = join(agentBase, possibleName);
 					if (!isPathSafe(agentBase, agentSkillDir)) continue;
 					try {
@@ -1268,17 +1308,20 @@ async function listInstalledSkills(options = {}) {
 				} catch {}
 				if (found) installedAgents.push(agentType);
 			}
-			installedSkills.push({
+			if (skillsMap.has(skillKey)) {
+				const existing = skillsMap.get(skillKey);
+				for (const agent of installedAgents) if (!existing.agents.includes(agent)) existing.agents.push(agent);
+			} else skillsMap.set(skillKey, {
 				name: skill.name,
 				description: skill.description,
 				path: skillDir,
 				canonicalPath: skillDir,
-				scope: scope.global ? "global" : "project",
+				scope: scopeKey,
 				agents: installedAgents
 			});
 		}
 	} catch {}
-	return installedSkills;
+	return Array.from(skillsMap.values());
 }
 const TELEMETRY_URL = "https://add-skill.vercel.sh/t";
 let cliVersion = null;
@@ -1700,7 +1743,7 @@ async function saveSelectedAgents(agents) {
 	lock.lastSelectedAgents = agents;
 	await writeSkillLock(lock);
 }
-var version$1 = "1.3.4";
+var version$1 = "1.3.5";
 const isCancelled = (value) => typeof value === "symbol";
 async function isSourcePrivate(source) {
 	const ownerRepo = parseOwnerRepo(source);
@@ -1722,6 +1765,44 @@ function formatList$1(items, maxShow = 5) {
 	const remaining = items.length - maxShow;
 	return `${shown.join(", ")} +${remaining} more`;
 }
+function splitAgentsByType(agentTypes) {
+	const universal = [];
+	const symlinked = [];
+	for (const a of agentTypes) if (isUniversalAgent(a)) universal.push(agents[a].displayName);
+	else symlinked.push(agents[a].displayName);
+	return {
+		universal,
+		symlinked
+	};
+}
+function buildAgentSummaryLines(targetAgents, installMode) {
+	const lines = [];
+	const { universal, symlinked } = splitAgentsByType(targetAgents);
+	if (installMode === "symlink") {
+		if (universal.length > 0) lines.push(`  ${import_picocolors.default.green("universal:")} ${formatList$1(universal)}`);
+		if (symlinked.length > 0) lines.push(`  ${import_picocolors.default.dim("symlink →")} ${formatList$1(symlinked)}`);
+	} else {
+		const allNames = targetAgents.map((a) => agents[a].displayName);
+		lines.push(`  ${import_picocolors.default.dim("copy →")} ${formatList$1(allNames)}`);
+	}
+	return lines;
+}
+function ensureUniversalAgents(targetAgents) {
+	const universalAgents = getUniversalAgents();
+	const result = [...targetAgents];
+	for (const ua of universalAgents) if (!result.includes(ua)) result.push(ua);
+	return result;
+}
+function buildResultLines(results, targetAgents) {
+	const lines = [];
+	const { universal, symlinked: symlinkAgents } = splitAgentsByType(targetAgents);
+	const successfulSymlinks = results.filter((r) => !r.symlinkFailed && !universal.includes(r.agent)).map((r) => r.agent);
+	const failedSymlinks = results.filter((r) => r.symlinkFailed).map((r) => r.agent);
+	if (universal.length > 0) lines.push(`  ${import_picocolors.default.green("universal:")} ${formatList$1(universal)}`);
+	if (successfulSymlinks.length > 0) lines.push(`  ${import_picocolors.default.dim("symlinked:")} ${formatList$1(successfulSymlinks)}`);
+	if (failedSymlinks.length > 0) lines.push(`  ${import_picocolors.default.yellow("copied:")} ${formatList$1(failedSymlinks)}`);
+	return lines;
+}
 function multiselect(opts) {
 	return fe({
 		...opts,
@@ -1755,11 +1836,35 @@ async function promptForAgents(message, choices) {
 	return selected;
 }
 async function selectAgentsInteractive(options) {
-	return promptForAgents("Which agents do you want to install to?", Object.keys(agents).map((a) => ({
+	const supportsGlobalFilter = (a) => !options.global || agents[a].globalSkillsDir;
+	const universalAgents = getUniversalAgents().filter(supportsGlobalFilter);
+	const otherAgents = getNonUniversalAgents().filter(supportsGlobalFilter);
+	const universalSection = {
+		title: "Universal (.agents/skills)",
+		items: universalAgents.map((a) => ({
+			value: a,
+			label: agents[a].displayName
+		}))
+	};
+	const otherChoices = otherAgents.map((a) => ({
 		value: a,
 		label: agents[a].displayName,
-		hint: `${options.global ? agents[a].globalSkillsDir : agents[a].skillsDir}`
-	})));
+		hint: options.global ? agents[a].globalSkillsDir : agents[a].skillsDir
+	}));
+	let lastSelected;
+	try {
+		lastSelected = await getLastSelectedAgents();
+	} catch {}
+	const selected = await searchMultiselect({
+		message: "Which agents do you want to install to?",
+		items: otherChoices,
+		initialSelected: lastSelected ? lastSelected.filter((a) => otherAgents.includes(a) && !universalAgents.includes(a)) : [],
+		lockedSection: universalSection
+	});
+	if (!isCancelled(selected)) try {
+		await saveSelectedAgents(selected);
+	} catch {}
+	return selected;
 }
 setVersion(version$1);
 async function handleRemoteSkill(source, url, options, spinner) {
@@ -1802,6 +1907,7 @@ async function handleRemoteSkill(source, url, options, spinner) {
 	}
 	let targetAgents;
 	const validAgents = Object.keys(agents);
+	const universalAgents = getUniversalAgents();
 	if (options.agent?.includes("*")) {
 		targetAgents = validAgents;
 		M.info(`Installing to all ${targetAgents.length} agents`);
@@ -1812,21 +1918,17 @@ async function handleRemoteSkill(source, url, options, spinner) {
 			M.info(`Valid agents: ${validAgents.join(", ")}`);
 			process.exit(1);
 		}
-		targetAgents = options.agent;
+		targetAgents = ensureUniversalAgents(options.agent);
 	} else {
 		spinner.start("Loading agents...");
 		const installedAgents = await detectInstalledAgents();
 		const totalAgents = Object.keys(agents).length;
 		spinner.stop(`${totalAgents} agents`);
 		if (installedAgents.length === 0) if (options.yes) {
-			targetAgents = validAgents;
-			M.info("Installing to all agents");
+			targetAgents = universalAgents;
+			M.info(`Installing to universal agents`);
 		} else {
-			M.info("Select agents to install skills to");
-			const selected = await promptForAgents("Which agents do you want to install to?", Object.entries(agents).map(([key, config]) => ({
-				value: key,
-				label: config.displayName
-			})));
+			const selected = await selectAgentsInteractive({ global: options.global });
 			if (pD(selected)) {
 				xe("Installation cancelled");
 				process.exit(0);
@@ -1834,11 +1936,10 @@ async function handleRemoteSkill(source, url, options, spinner) {
 			targetAgents = selected;
 		}
 		else if (installedAgents.length === 1 || options.yes) {
-			targetAgents = installedAgents;
-			if (installedAgents.length === 1) {
-				const firstAgent = installedAgents[0];
-				M.info(`Installing to: ${import_picocolors.default.cyan(agents[firstAgent].displayName)}`);
-			} else M.info(`Installing to: ${installedAgents.map((a) => import_picocolors.default.cyan(agents[a].displayName)).join(", ")}`);
+			targetAgents = ensureUniversalAgents(installedAgents);
+			const { universal, symlinked } = splitAgentsByType(targetAgents);
+			if (symlinked.length > 0) M.info(`Installing to: ${import_picocolors.default.green("universal")} + ${symlinked.map((a) => import_picocolors.default.cyan(a)).join(", ")}`);
+			else M.info(`Installing to: ${import_picocolors.default.green("universal agents")}`);
 		} else {
 			const selected = await selectAgentsInteractive({ global: options.global });
 			if (pD(selected)) {
@@ -1896,15 +1997,9 @@ async function handleRemoteSkill(source, url, options, spinner) {
 	})));
 	const overwriteStatus = new Map(overwriteChecks.map(({ agent, installed }) => [agent, installed]));
 	const summaryLines = [];
-	const agentNames = targetAgents.map((a) => agents[a].displayName);
-	if (installMode === "symlink") {
-		const shortCanonical = shortenPath$1(getCanonicalPath(remoteSkill.installName, { global: installGlobally }), cwd);
-		summaryLines.push(`${import_picocolors.default.cyan(shortCanonical)}`);
-		summaryLines.push(`  ${import_picocolors.default.dim("symlink →")} ${formatList$1(agentNames)}`);
-	} else {
-		summaryLines.push(`${import_picocolors.default.cyan(remoteSkill.installName)}`);
-		summaryLines.push(`  ${import_picocolors.default.dim("copy →")} ${formatList$1(agentNames)}`);
-	}
+	const shortCanonical = shortenPath$1(getCanonicalPath(remoteSkill.installName, { global: installGlobally }), cwd);
+	summaryLines.push(`${import_picocolors.default.cyan(shortCanonical)}`);
+	summaryLines.push(...buildAgentSummaryLines(targetAgents, installMode));
 	const overwriteAgents = targetAgents.filter((a) => overwriteStatus.get(a)).map((a) => agents[a].displayName);
 	if (overwriteAgents.length > 0) summaryLines.push(`  ${import_picocolors.default.yellow("overwrites:")} ${formatList$1(overwriteAgents)}`);
 	console.log();
@@ -1969,12 +2064,9 @@ async function handleRemoteSkill(source, url, options, spinner) {
 				const shortPath = shortenPath$1(firstResult.canonicalPath, cwd);
 				resultLines.push(`${import_picocolors.default.green("✓")} ${shortPath}`);
 			} else resultLines.push(`${import_picocolors.default.green("✓")} ${remoteSkill.installName}`);
-			const symlinked = successful.filter((r) => !r.symlinkFailed).map((r) => r.agent);
-			const copied = successful.filter((r) => r.symlinkFailed).map((r) => r.agent);
-			if (symlinked.length > 0) resultLines.push(`  ${import_picocolors.default.dim("symlink →")} ${formatList$1(symlinked)}`);
-			if (copied.length > 0) resultLines.push(`  ${import_picocolors.default.yellow("copied →")} ${formatList$1(copied)}`);
+			resultLines.push(...buildResultLines(successful, targetAgents));
 		}
-		const title = import_picocolors.default.green(`Installed 1 skill to ${successful.length} agent${successful.length !== 1 ? "s" : ""}`);
+		const title = import_picocolors.default.green("Installed 1 skill");
 		Me(resultLines.join("\n"), title);
 		const symlinkFailures = successful.filter((r) => r.mode === "symlink" && r.symlinkFailed);
 		if (symlinkFailures.length > 0) {
@@ -2145,7 +2237,7 @@ async function handleWellKnownSkills(source, url, options, spinner) {
 	}
 	const cwd = process.cwd();
 	const summaryLines = [];
-	const agentNames = targetAgents.map((a) => agents[a].displayName);
+	targetAgents.map((a) => agents[a].displayName);
 	const overwriteChecks = await Promise.all(selectedSkills.flatMap((skill) => targetAgents.map(async (agent) => ({
 		skillName: skill.installName,
 		agent,
@@ -2158,15 +2250,10 @@ async function handleWellKnownSkills(source, url, options, spinner) {
 	}
 	for (const skill of selectedSkills) {
 		if (summaryLines.length > 0) summaryLines.push("");
-		if (installMode === "symlink") {
-			const shortCanonical = shortenPath$1(getCanonicalPath(skill.installName, { global: installGlobally }), cwd);
-			summaryLines.push(`${import_picocolors.default.cyan(shortCanonical)}`);
-			summaryLines.push(`  ${import_picocolors.default.dim("symlink →")} ${formatList$1(agentNames)}`);
-			if (skill.files.size > 1) summaryLines.push(`  ${import_picocolors.default.dim("files:")} ${skill.files.size}`);
-		} else {
-			summaryLines.push(`${import_picocolors.default.cyan(skill.installName)}`);
-			summaryLines.push(`  ${import_picocolors.default.dim("copy →")} ${formatList$1(agentNames)}`);
-		}
+		const shortCanonical = shortenPath$1(getCanonicalPath(skill.installName, { global: installGlobally }), cwd);
+		summaryLines.push(`${import_picocolors.default.cyan(shortCanonical)}`);
+		summaryLines.push(...buildAgentSummaryLines(targetAgents, installMode));
+		if (skill.files.size > 1) summaryLines.push(`  ${import_picocolors.default.dim("files:")} ${skill.files.size}`);
 		const skillOverwrites = overwriteStatus.get(skill.installName);
 		const overwriteAgents = targetAgents.filter((a) => skillOverwrites?.get(a)).map((a) => agents[a].displayName);
 		if (overwriteAgents.length > 0) summaryLines.push(`  ${import_picocolors.default.yellow("overwrites:")} ${formatList$1(overwriteAgents)}`);
@@ -2228,7 +2315,6 @@ async function handleWellKnownSkills(source, url, options, spinner) {
 			bySkill.set(r.skill, skillResults);
 		}
 		const skillCount = bySkill.size;
-		const agentCount = new Set(successful.map((r) => r.agent)).size;
 		const symlinkFailures = successful.filter((r) => r.mode === "symlink" && r.symlinkFailed);
 		const copiedAgents = symlinkFailures.map((r) => r.agent);
 		const resultLines = [];
@@ -2245,13 +2331,10 @@ async function handleWellKnownSkills(source, url, options, spinner) {
 					const shortPath = shortenPath$1(firstResult.canonicalPath, cwd);
 					resultLines.push(`${import_picocolors.default.green("✓")} ${shortPath}`);
 				} else resultLines.push(`${import_picocolors.default.green("✓")} ${skillName}`);
-				const symlinked = skillResults.filter((r) => !r.symlinkFailed).map((r) => r.agent);
-				const copied = skillResults.filter((r) => r.symlinkFailed).map((r) => r.agent);
-				if (symlinked.length > 0) resultLines.push(`  ${import_picocolors.default.dim("symlink →")} ${formatList$1(symlinked)}`);
-				if (copied.length > 0) resultLines.push(`  ${import_picocolors.default.yellow("copied →")} ${formatList$1(copied)}`);
+				resultLines.push(...buildResultLines(skillResults, targetAgents));
 			}
 		}
-		const title = import_picocolors.default.green(`Installed ${skillCount} skill${skillCount !== 1 ? "s" : ""} to ${agentCount} agent${agentCount !== 1 ? "s" : ""}`);
+		const title = import_picocolors.default.green(`Installed ${skillCount} skill${skillCount !== 1 ? "s" : ""}`);
 		Me(resultLines.join("\n"), title);
 		if (symlinkFailures.length > 0) {
 			M.warn(import_picocolors.default.yellow(`Symlinks failed for: ${formatList$1(copiedAgents)}`));
@@ -2374,10 +2457,10 @@ async function handleDirectUrlSkillLegacy(source, url, options, spinner) {
 	})));
 	const overwriteStatus = new Map(overwriteChecks.map(({ agent, installed }) => [agent, installed]));
 	const summaryLines = [];
-	const agentNames = targetAgents.map((a) => agents[a].displayName);
+	targetAgents.map((a) => agents[a].displayName);
 	const shortCanonical = shortenPath$1(getCanonicalPath(remoteSkill.installName, { global: installGlobally }), cwd);
 	summaryLines.push(`${import_picocolors.default.cyan(shortCanonical)}`);
-	summaryLines.push(`  ${import_picocolors.default.dim("symlink →")} ${formatList$1(agentNames)}`);
+	summaryLines.push(...buildAgentSummaryLines(targetAgents, installMode));
 	const overwriteAgents = targetAgents.filter((a) => overwriteStatus.get(a)).map((a) => agents[a].displayName);
 	if (overwriteAgents.length > 0) summaryLines.push(`  ${import_picocolors.default.yellow("overwrites:")} ${formatList$1(overwriteAgents)}`);
 	console.log();
@@ -2430,11 +2513,8 @@ async function handleDirectUrlSkillLegacy(source, url, options, spinner) {
 			const shortPath = shortenPath$1(firstResult.canonicalPath, cwd);
 			resultLines.push(`${import_picocolors.default.green("✓")} ${shortPath}`);
 		} else resultLines.push(`${import_picocolors.default.green("✓")} ${remoteSkill.installName}`);
-		const symlinked = successful.filter((r) => !r.symlinkFailed).map((r) => r.agent);
-		const copied = successful.filter((r) => r.symlinkFailed).map((r) => r.agent);
-		if (symlinked.length > 0) resultLines.push(`  ${import_picocolors.default.dim("symlink →")} ${formatList$1(symlinked)}`);
-		if (copied.length > 0) resultLines.push(`  ${import_picocolors.default.yellow("copied →")} ${formatList$1(copied)}`);
-		const title = import_picocolors.default.green(`Installed 1 skill to ${successful.length} agent${successful.length !== 1 ? "s" : ""}`);
+		resultLines.push(...buildResultLines(successful, targetAgents));
+		const title = import_picocolors.default.green("Installed 1 skill");
 		Me(resultLines.join("\n"), title);
 		const symlinkFailures = successful.filter((r) => r.mode === "symlink" && r.symlinkFailed);
 		if (symlinkFailures.length > 0) {
@@ -2674,7 +2754,7 @@ async function runAdd(args, options = {}) {
 		}
 		const cwd = process.cwd();
 		const summaryLines = [];
-		const agentNames = targetAgents.map((a) => agents[a].displayName);
+		targetAgents.map((a) => agents[a].displayName);
 		const overwriteChecks = await Promise.all(selectedSkills.flatMap((skill) => targetAgents.map(async (agent) => ({
 			skillName: skill.name,
 			agent,
@@ -2687,14 +2767,9 @@ async function runAdd(args, options = {}) {
 		}
 		for (const skill of selectedSkills) {
 			if (summaryLines.length > 0) summaryLines.push("");
-			if (installMode === "symlink") {
-				const shortCanonical = shortenPath$1(getCanonicalPath(skill.name, { global: installGlobally }), cwd);
-				summaryLines.push(`${import_picocolors.default.cyan(shortCanonical)}`);
-				summaryLines.push(`  ${import_picocolors.default.dim("symlink →")} ${formatList$1(agentNames)}`);
-			} else {
-				summaryLines.push(`${import_picocolors.default.cyan(getSkillDisplayName(skill))}`);
-				summaryLines.push(`  ${import_picocolors.default.dim("copy →")} ${formatList$1(agentNames)}`);
-			}
+			const shortCanonical = shortenPath$1(getCanonicalPath(skill.name, { global: installGlobally }), cwd);
+			summaryLines.push(`${import_picocolors.default.cyan(shortCanonical)}`);
+			summaryLines.push(...buildAgentSummaryLines(targetAgents, installMode));
 			const skillOverwrites = overwriteStatus.get(skill.name);
 			const overwriteAgents = targetAgents.filter((a) => skillOverwrites?.get(a)).map((a) => agents[a].displayName);
 			if (overwriteAgents.length > 0) summaryLines.push(`  ${import_picocolors.default.yellow("overwrites:")} ${formatList$1(overwriteAgents)}`);
@@ -2784,7 +2859,6 @@ async function runAdd(args, options = {}) {
 				bySkill.set(r.skill, skillResults);
 			}
 			const skillCount = bySkill.size;
-			const agentCount = new Set(successful.map((r) => r.agent)).size;
 			const symlinkFailures = successful.filter((r) => r.mode === "symlink" && r.symlinkFailed);
 			const copiedAgents = symlinkFailures.map((r) => r.agent);
 			const resultLines = [];
@@ -2801,13 +2875,10 @@ async function runAdd(args, options = {}) {
 						const shortPath = shortenPath$1(firstResult.canonicalPath, cwd);
 						resultLines.push(`${import_picocolors.default.green("✓")} ${shortPath}`);
 					} else resultLines.push(`${import_picocolors.default.green("✓")} ${skillName}`);
-					const symlinked = skillResults.filter((r) => !r.symlinkFailed).map((r) => r.agent);
-					const copied = skillResults.filter((r) => r.symlinkFailed).map((r) => r.agent);
-					if (symlinked.length > 0) resultLines.push(`  ${import_picocolors.default.dim("symlink →")} ${formatList$1(symlinked)}`);
-					if (copied.length > 0) resultLines.push(`  ${import_picocolors.default.yellow("copied →")} ${formatList$1(copied)}`);
+					resultLines.push(...buildResultLines(skillResults, targetAgents));
 				}
 			}
-			const title = import_picocolors.default.green(`Installed ${skillCount} skill${skillCount !== 1 ? "s" : ""} to ${agentCount} agent${agentCount !== 1 ? "s" : ""}`);
+			const title = import_picocolors.default.green(`Installed ${skillCount} skill${skillCount !== 1 ? "s" : ""}`);
 			Me(resultLines.join("\n"), title);
 			if (symlinkFailures.length > 0) {
 				M.warn(import_picocolors.default.yellow(`Symlinks failed for: ${formatList$1(copiedAgents)}`));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skills",
-  "version": "1.3.4",
+  "version": "1.3.5",
   "description": "The open agent skills ecosystem",
   "type": "module",
   "bin": {