ali-skills 0.0.19 → 0.0.21

package/node_modules/@ali/cli-skills/dist/cli.mjs

@@ -937,8 +937,8 @@ const agents = {
 	qoder: {
 		name: "qoder",
 		displayName: "Qoder",
-		skillsDir: ".qoder/skills",
-		globalSkillsDir: join(home, ".qoder/skills"),
+		skillsDir: ".agents/skills",
+		globalSkillsDir: join(home, ".agents/skills"),
 		detectInstalled: async () => {
 			return existsSync(join(home, ".qoder"));
 		}
@@ -2205,7 +2205,7 @@ function logSkillParseFailures(failures) {
 		if (f.hint) M.message(import_picocolors.default.yellow(`  提示: ${f.hint}`));
 	}
 }
-var version$1 = "2.0.19";
+var version$1 = "2.0.21";
 const isCancelled$1 = (value) => typeof value === "symbol";
 function redactSensitiveText(text) {
 	return text.replace(/(https?:\/\/)([^/\s:@]+)(?::([^@/\s]*))?@/gi, "$1").replace(/([?&](?:private_token|access_token|token|api_key|access_key|secret|password)=)[^&\s]+/gi, "$1***");
@@ -5283,6 +5283,16 @@ async function runConfig(args) {
 		}
 	}
 }
+function getUpdateComparisonStrategy(entry) {
+	if (entry.sourceType === "internal") return {
+		field: "commitHash",
+		value: entry.commitHash
+	};
+	return {
+		field: "skillFolderHash",
+		value: entry.skillFolderHash
+	};
+}
 const __dirname = dirname(fileURLToPath(import.meta.url));
 function getVersion() {
 	try {
@@ -5299,6 +5309,18 @@ const BOLD = "\x1B[1m";
 const DIM = "\x1B[38;5;102m";
 const TEXT = "\x1B[38;5;145m";
 const YELLOW = "\x1B[33m";
+function isTruthyEnv(value) {
+	if (!value) return false;
+	const normalized = value.trim().toLowerCase();
+	return normalized !== "" && normalized !== "0" && normalized !== "false" && normalized !== "off";
+}
+function isUpdateDebugEnabled() {
+	return isTruthyEnv(process.env.ALI_SKILLS_DEBUG_UPDATE) || isTruthyEnv(process.env.DEBUG);
+}
+function updateDebugLog(enabled, message) {
+	if (!enabled) return;
+	console.log(`${DIM}[update-debug]${RESET} ${message}`);
+}
 const LOGO_LINES = [
 	" █████╗ ██╗     ██╗    ███████╗██╗  ██╗██╗██╗     ██╗     ███████╗",
 	"██╔══██╗██║     ██║    ██╔════╝██║ ██╔╝██║██║     ██║     ██╔════╝",
@@ -5405,6 +5427,9 @@ ${BOLD}Options:${RESET}
   --help, -h        Show this help message
   --version, -v     Show version number
 
+${BOLD}Debug:${RESET}
+  ALI_SKILLS_DEBUG_UPDATE=1  Print detailed logs for check/update diagnosis
+
 ${BOLD}Examples:${RESET}
   ${DIM}$${RESET} ali-skills add group/project
   ${DIM}$${RESET} ali-skills add group/project -g
@@ -5697,7 +5722,7 @@ function readSkillLock() {
 function getSkipReason(entry) {
 	if (entry.sourceType === "local") return "Local path";
 	if (entry.sourceType === "git") return "Git URL (hash tracking not supported)";
-	if (!(entry.sourceType === "internal" ? entry.commitHash : entry.skillFolderHash)) return "No version hash available";
+	if (!getUpdateComparisonStrategy(entry).value) return "No version hash available";
 	if (!entry.skillPath) return "No skill path recorded";
 	return "No version tracking";
 }
@@ -5962,8 +5987,10 @@ async function runUpdate(args = []) {
 	const isGlobal = args.includes("-g") || args.includes("--global");
 	const yesFlag = args.includes("-y") || args.includes("--yes");
 	const skillArgs = args.filter((arg) => !arg.startsWith("-"));
+	const debugUpdate = isUpdateDebugEnabled();
 	console.log(`${TEXT}Checking for skill updates...${RESET}`);
 	console.log();
+	updateDebugLog(debugUpdate, `scope=${isGlobal ? "global" : "project"}, yes=${yesFlag}, requestedSkills=${skillArgs.length > 0 ? skillArgs.join(",") : "(all)"}`);
 	const token = getGitHubToken();
 	let skillsToCheck = {};
 	let scopeLabel = "project";
@@ -5983,6 +6010,7 @@ async function runUpdate(args = []) {
 				sourceType: entry.sourceType,
 				sourceUrl,
 				skillPath: entry.skillPath || "",
+				commitHash: entry.commitHash,
 				skillFolderHash: entry.skillFolderHash || entry.computedHash,
 				installedAt: (/* @__PURE__ */ new Date()).toISOString(),
 				updatedAt: (/* @__PURE__ */ new Date()).toISOString()
@@ -5990,6 +6018,7 @@ async function runUpdate(args = []) {
 		}
 	}
 	const skillNames = Object.keys(skillsToCheck);
+	updateDebugLog(debugUpdate, `loaded ${skillNames.length} skill(s) from ${isGlobal ? "~/.agents/.skill-lock.json" : "skills-lock.json"}`);
 	if (skillNames.length === 0) {
 		console.log(`${DIM}No ${scopeLabel}-level skills found.${RESET}`);
 		console.log();
@@ -6002,9 +6031,12 @@ async function runUpdate(args = []) {
 	for (const skillName of skillNames) {
 		const entry = skillsToCheck[skillName];
 		if (!entry) continue;
-		if (entry.sourceType === "internal" && projectLock) {
-			const localEntry = projectLock.skills[skillName];
-			if (!localEntry?.commitHash) {
+		if (entry.sourceType === "internal") {
+			const localEntry = projectLock?.skills[skillName];
+			const commitHash = getUpdateComparisonStrategy(entry).value || getUpdateComparisonStrategy(localEntry || {}).value;
+			const storedSkillPath = entry.skillPath || localEntry?.skillPath || "";
+			updateDebugLog(debugUpdate, `checking internal skill=${skillName}, repo=${entry.source}, skillPath=${storedSkillPath || "(root)"}, hasCommitHash=${commitHash ? "yes" : "no"}`);
+			if (!commitHash) {
 				skipped.push({
 					name: skillName,
 					reason: "Reinstall required to enable update checking (missing commit hash)",
@@ -6013,14 +6045,18 @@ async function runUpdate(args = []) {
 				continue;
 			}
 			try {
-				const skillPath = (localEntry.skillPath || "").replace(/\/?SKILL\.md$/i, "");
+				const skillPath = storedSkillPath.replace(/\/?SKILL\.md$/i, "");
 				const latestCommit = await fetchAliInternalCommitHash(entry.source, skillPath, null);
-				if (latestCommit && latestCommit !== localEntry.commitHash) availableUpdates.push({
-					name: skillName,
-					source: entry.source,
-					entry
-				});
+				if (latestCommit && latestCommit !== commitHash) {
+					availableUpdates.push({
+						name: skillName,
+						source: entry.source,
+						entry
+					});
+					updateDebugLog(debugUpdate, `update available for ${skillName}: local=${commitHash?.slice(0, 12)} remote=${latestCommit.slice(0, 12)}`);
+				} else updateDebugLog(debugUpdate, `no update for ${skillName}: local=${commitHash?.slice(0, 12)} remote=${latestCommit?.slice(0, 12) || "(empty)"}`);
 			} catch {
+				updateDebugLog(debugUpdate, `failed to check internal skill=${skillName}`);
 				skipped.push({
 					name: skillName,
 					reason: "Ali internal repository (API check failed)",
@@ -6108,10 +6144,17 @@ async function runUpdate(args = []) {
 	console.log(`${TEXT}Updating ${selectedUpdates.length} ${scopeLabel} skill(s)...${RESET}`);
 	console.log();
 	const installedSkills = await listInstalledSkills({ global: isGlobal });
+	updateDebugLog(debugUpdate, `listInstalledSkills(global=${isGlobal}) returned ${installedSkills.length} skill record(s)`);
 	const installedAgentsBySkillName = /* @__PURE__ */ new Map();
-	for (const skill of installedSkills) installedAgentsBySkillName.set(skill.name.toLowerCase(), [...skill.agents]);
+	for (const skill of installedSkills) {
+		installedAgentsBySkillName.set(skill.name.toLowerCase(), [...skill.agents]);
+		updateDebugLog(debugUpdate, `installed skill=${skill.name}, scope=${skill.scope}, agents=${skill.agents.length > 0 ? skill.agents.join(",") : "(none)"}, path=${skill.path}`);
+	}
 	let successCount = 0;
 	let failCount = 0;
+	let safeSkipCount = 0;
+	const listCommand = isGlobal ? "npx ali-skills list -g" : "npx ali-skills list";
+	const canonicalSkillDir = isGlobal ? "~/.agents/skills/<skill-name>" : ".agents/skills/<skill-name>";
 	for (const update of selectedUpdates) {
 		console.log(`${TEXT}Updating ${update.name}...${RESET}`);
 		let skillFolder = update.entry.skillPath || "";
@@ -6130,7 +6173,14 @@ async function runUpdate(args = []) {
 			const installedAgents = installedAgentsBySkillName.get(update.name.toLowerCase());
 			if (!installedAgents || installedAgents.length === 0) {
 				failCount++;
+				safeSkipCount++;
+				const nearMatches = installedSkills.filter((s) => s.name.toLowerCase().includes(update.name.toLowerCase())).map((s) => s.name);
+				updateDebugLog(debugUpdate, `skip ${update.name}: no installed agents found in current scope. nearMatches=${nearMatches.length > 0 ? nearMatches.join(",") : "(none)"}`);
 				console.log(`  ${DIM}✗ Skipped ${update.name}: unable to determine installed agents for safe update${RESET}`);
+				console.log(`    ${DIM}Tips:${RESET}`);
+				console.log(`      ${DIM}- Run ${TEXT}${listCommand}${RESET}${DIM} to verify whether ${update.name} was accidentally removed${RESET}`);
+				console.log(`      ${DIM}- In symlink mode, deleting ${canonicalSkillDir} removes the skill entity and can remove it from all agents${RESET}`);
+				console.log(`      ${DIM}- If you see duplicated skills, delete the one under agent-specific skills dir first, then report duplicate detection to ali-skills maintainers${RESET}`);
 				continue;
 			}
 			await runAdd([installUrl], {
@@ -6138,9 +6188,11 @@ async function runUpdate(args = []) {
 				global: isGlobal,
 				agent: installedAgents
 			});
+			updateDebugLog(debugUpdate, `updated ${update.name} with installUrl=${installUrl} targetAgents=${installedAgents.join(",")}`);
 			successCount++;
 			console.log(`  ${TEXT}✓${RESET} Updated ${update.name}`);
 		} catch {
+			updateDebugLog(debugUpdate, `runAdd failed for ${update.name}`);
 			failCount++;
 			console.log(`  ${DIM}✗ Failed to update ${update.name}${RESET}`);
 		}
@@ -6148,6 +6200,7 @@ async function runUpdate(args = []) {
 	console.log();
 	if (successCount > 0) console.log(`${TEXT}✓ Updated ${successCount} skill(s)${RESET}`);
 	if (failCount > 0) console.log(`${DIM}Failed to update ${failCount} skill(s)${RESET}`);
+	if (safeSkipCount > 0 && !debugUpdate) console.log(`${DIM}Tip: re-run with ALI_SKILLS_DEBUG_UPDATE=1 to inspect installed-agent detection details${RESET}`);
 	track({
 		event: "update",
 		skillCount: String(selectedUpdates.length),

package/node_modules/@ali/cli-skills/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ali/cli-skills",
-  "version": "2.0.19",
+  "version": "2.0.21",
   "description": "The open agent skills ecosystem",
   "type": "module",
   "bin": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ali-skills",
-  "version": "0.0.19",
+  "version": "0.0.21",
   "description": "The open agent skills ecosystem",
   "type": "module",
   "bin": {
@@ -31,7 +31,7 @@
   "author": "",
   "license": "MIT",
   "dependencies": {
-    "@ali/cli-skills": "^2.0.19"
+    "@ali/cli-skills": "^2.0.21"
   },
   "bundleDependencies": [
     "@ali/cli-skills"