agent-skills-standard 2.4.3 → 2.4.5

package/README.md

@@ -5,13 +5,15 @@
 
 **Sync 242 AI coding standards to any project in one command.** Works with Cursor, Claude Code, GitHub Copilot, Gemini, Windsurf, Trae, Kiro, and Roo.
 
-**Current release:** `v2.4.3` — packaged `ags -V` now matches the published CLI version.
+**Current release:** `v2.4.5` — packaged `ags -V` now detects stale pnpm shims and guides repair.
 
 ```bash
 npx agent-skills-standard@latest init   # detect your stack
 npx agent-skills-standard@latest sync   # install skills
 ```
 
+If `ags -V` still shows an old version after reinstalling, check your PATH order. `~/Library/pnpm` must come before `~/Library/pnpm/bin`, then run `hash -r` and verify with `ags -V` again.
+
 ---
 
 ## What It Does

package/dist/index.js

@@ -24,10 +24,10 @@ var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__ge
 let commander = require("commander");
 let dotenv = require("dotenv");
 dotenv = __toESM(dotenv);
-let inquirer = require("inquirer");
-inquirer = __toESM(inquirer);
 let picocolors = require("picocolors");
 picocolors = __toESM(picocolors);
+let inquirer = require("inquirer");
+inquirer = __toESM(inquirer);
 let fs_extra = require("fs-extra");
 fs_extra = __toESM(fs_extra);
 let js_yaml = require("js-yaml");
@@ -40,6 +40,60 @@ cross_fetch = __toESM(cross_fetch);
 let os = require("os");
 os = __toESM(os);
 let child_process = require("child_process");
+let fs = require("fs");
+var package_default = {
+	name: "agent-skills-standard",
+	version: "2.4.5",
+	description: "A CLI to manage and sync AI agent skills standard for Cursor, Claude, Copilot, Windsurf, and more.",
+	main: "dist/index.js",
+	bin: {
+		"agent-skills-standard": "dist/index.js",
+		"ags": "dist/index.js"
+	},
+	files: ["dist"],
+	scripts: {
+		"build": "vite build",
+		"start": "node dist/index.js",
+		"dev": "tsx src/index.ts",
+		"validate": "tsx src/index.ts validate",
+		"validate:all": "tsx src/index.ts validate --all",
+		"format": "prettier --write \"src/**/*.ts\"",
+		"format:check": "prettier --check \"src/**/*.ts\"",
+		"lint": "eslint \"src/**/*.ts\" --fix",
+		"lint:check": "eslint \"src/**/*.ts\"",
+		"test": "vitest run",
+		"test:watch": "vitest",
+		"test:cov": "vitest run --coverage",
+		"prepublishOnly": "pnpm build"
+	},
+	keywords: [
+		"ai",
+		"agent",
+		"cursor",
+		"claude",
+		"copilot",
+		"cli",
+		"antigravity"
+	],
+	author: "Hoang Nguyen",
+	license: "MIT",
+	dependencies: {
+		"commander": "^11.1.0",
+		"cross-fetch": "^4.1.0",
+		"dotenv": "^17.2.3",
+		"fs-extra": "^11.2.0",
+		"inquirer": "^8.2.5",
+		"js-yaml": "^4.1.0",
+		"picocolors": "^1.0.0",
+		"zod": "^3.22.4"
+	},
+	devDependencies: {
+		"@vitest/coverage-v8": "^4.0.17",
+		"vite": "^7.0.0",
+		"vitest": "^4.0.17"
+	}
+};
+//#endregion
 //#region src/services/FeedbackService.ts
 /**
 * Service to handle automated feedback reporting via Proxy Backend.
@@ -4485,58 +4539,6 @@ var SyncCommand = class {
 		}
 	}
 };
-var package_default = {
-	name: "agent-skills-standard",
-	version: "2.4.3",
-	description: "A CLI to manage and sync AI agent skills standard for Cursor, Claude, Copilot, Windsurf, and more.",
-	main: "dist/index.js",
-	bin: {
-		"agent-skills-standard": "dist/index.js",
-		"ags": "dist/index.js"
-	},
-	files: ["dist"],
-	scripts: {
-		"build": "vite build",
-		"start": "node dist/index.js",
-		"dev": "tsx src/index.ts",
-		"validate": "tsx src/index.ts validate",
-		"validate:all": "tsx src/index.ts validate --all",
-		"format": "prettier --write \"src/**/*.ts\"",
-		"format:check": "prettier --check \"src/**/*.ts\"",
-		"lint": "eslint \"src/**/*.ts\" --fix",
-		"lint:check": "eslint \"src/**/*.ts\"",
-		"test": "vitest run",
-		"test:watch": "vitest",
-		"test:cov": "vitest run --coverage",
-		"prepublishOnly": "pnpm build"
-	},
-	keywords: [
-		"ai",
-		"agent",
-		"cursor",
-		"claude",
-		"copilot",
-		"cli",
-		"antigravity"
-	],
-	author: "Hoang Nguyen",
-	license: "MIT",
-	dependencies: {
-		"commander": "^11.1.0",
-		"cross-fetch": "^4.1.0",
-		"dotenv": "^17.2.3",
-		"fs-extra": "^11.2.0",
-		"inquirer": "^8.2.5",
-		"js-yaml": "^4.1.0",
-		"picocolors": "^1.0.0",
-		"zod": "^3.22.4"
-	},
-	devDependencies: {
-		"@vitest/coverage-v8": "^4.0.17",
-		"vite": "^7.0.0",
-		"vitest": "^4.0.17"
-	}
-};
 //#endregion
 //#region src/commands/upgrade.ts
 /**
@@ -4588,7 +4590,31 @@ var UpgradeCommand = class {
 				console.log(picocolors.default.yellow(`⚠️  Failed to install v${latestVersion}. Retrying with @latest...`));
 				(0, child_process.execSync)(this.getUpgradeCommand(pm, "latest"), { stdio: "inherit" });
 			}
-			try {
+			if (pm === "pnpm") {
+				const installedVersion = this.getInstalledVersionFromPnpm(latestVersion);
+				if (installedVersion !== latestVersion) {
+					console.log("\n" + picocolors.default.red(`❌ pnpm install completed, but the installed package still reports v${installedVersion ?? "unknown"}.`));
+					console.log(picocolors.default.gray("   This means the global install itself is stale or incomplete, not just the shell shim."));
+					this.printShellMismatchHint();
+					return;
+				}
+				try {
+					const lines = (0, child_process.execSync)("ags -V", {
+						encoding: "utf8",
+						stdio: "pipe"
+					}).trim().split("\n");
+					const shellVersion = lines[lines.length - 1].trim();
+					if (shellVersion.includes(latestVersion)) console.log(picocolors.default.green(`✅ Successfully upgraded to v${latestVersion}!`));
+					else {
+						console.log("\n" + picocolors.default.yellow(`⚠️  pnpm installed v${latestVersion}, but 'ags -V' still reports v${shellVersion}.`));
+						console.log(picocolors.default.gray("   Your shell is still resolving a stale shim or PATH entry."));
+						await this.detectAndFixStaleShim(latestVersion);
+					}
+				} catch {
+					console.log(picocolors.default.green("\n✅ Upgrade command finished."));
+					console.log(picocolors.default.gray("   (Installed version verified via pnpm; please check the shell command with 'ags -V')"));
+				}
+			} else try {
 				const lines = (0, child_process.execSync)("ags -V", {
 					encoding: "utf8",
 					stdio: "pipe"
@@ -4596,9 +4622,14 @@ var UpgradeCommand = class {
 				const installedVersion = lines[lines.length - 1].trim();
 				if (installedVersion.includes(latestVersion)) console.log(picocolors.default.green(`✅ Successfully upgraded to v${latestVersion}!`));
 				else {
+					const resolvedBinary = (0, child_process.execSync)("type -a ags", {
+						encoding: "utf8",
+						stdio: "pipe"
+					}).trim();
 					console.log("\n" + picocolors.default.yellow(`⚠️  Installation complete, but 'ags -V' still reports v${installedVersion}.`));
-					console.log(picocolors.default.gray("   This is common if multiple versions are installed or your shell needs a restart."));
-					console.log(picocolors.default.cyan(`👉 Recommendation: Restart your terminal or run: ${picocolors.default.bold(this.getUpgradeCommand(pm, latestVersion) + " --force")}`));
+					console.log(picocolors.default.gray("   This usually means your shell is still resolving a stale shim or PATH entry."));
+					console.log(picocolors.default.gray(`   Resolved binary:\n${resolvedBinary}`));
+					console.log(picocolors.default.cyan(`👉 Recommendation: make sure ${picocolors.default.bold("~/Library/pnpm")} comes before ${picocolors.default.bold("~/Library/pnpm/bin")} in PATH, then run ${picocolors.default.bold("hash -r")} and recheck with 'ags -V'.`));
 				}
 			} catch {
 				console.log(picocolors.default.green("\n✅ Upgrade command finished."));
@@ -4628,6 +4659,92 @@ var UpgradeCommand = class {
 			default: return `npm install -g agent-skills-standard@${version}`;
 		}
 	}
+	getInstalledVersionFromPnpm(version) {
+		try {
+			const globalRoot = (0, child_process.execSync)("pnpm root -g", {
+				encoding: "utf8",
+				stdio: "pipe"
+			}).trim();
+			const packageJsonPath = path.default.join(globalRoot, ".pnpm", `agent-skills-standard@${version}`, "node_modules", "agent-skills-standard", "package.json");
+			return (0, child_process.execSync)(`node -p "require(${JSON.stringify(packageJsonPath)}).version"`, {
+				encoding: "utf8",
+				stdio: "pipe"
+			}).trim() || null;
+		} catch {
+			return null;
+		}
+	}
+	/**
+	* Shown when the pnpm package store itself is stale/incomplete (Case 1).
+	* Falls back to a resolved-binary hint and PATH guidance.
+	*/
+	printShellMismatchHint() {
+		try {
+			const resolvedBinary = (0, child_process.execSync)("type -a ags", {
+				encoding: "utf8",
+				stdio: "pipe"
+			}).trim();
+			console.log(picocolors.default.gray(`   Resolved binary:\n${resolvedBinary}`));
+		} catch {}
+		this.printShimPathHint();
+	}
+	/**
+	* Prints the standard PATH-order fix recommendation.
+	*/
+	printShimPathHint() {
+		console.log(picocolors.default.cyan(`👉 Make sure ${picocolors.default.bold("~/Library/pnpm")} comes before ${picocolors.default.bold("~/Library/pnpm/bin")} in PATH, then run ${picocolors.default.bold("hash -r")} and recheck with ${picocolors.default.bold("ags -V")}.`));
+	}
+	/**
+	* Called when the pnpm store is correct but `ags -V` still reports an old
+	* version (Case 2 — stale shim).  Attempts to locate and remove the stale
+	* shim file at <pnpm-bin>/bin/ags, which is created by older pnpm versions
+	* using a content-addressed hash path that is never updated on reinstall.
+	*/
+	async detectAndFixStaleShim(latestVersion) {
+		try {
+			const resolvedBinary = (0, child_process.execSync)("type -a ags", {
+				encoding: "utf8",
+				stdio: "pipe"
+			}).trim();
+			console.log(picocolors.default.gray(`   Resolved binary:\n${resolvedBinary}`));
+		} catch {}
+		try {
+			const pnpmBin = (0, child_process.execSync)("pnpm bin -g", {
+				encoding: "utf8",
+				stdio: "pipe"
+			}).trim();
+			const staleShimPath = path.default.join(pnpmBin, "bin", "ags");
+			if (!(0, fs.existsSync)(staleShimPath)) {
+				this.printShimPathHint();
+				return;
+			}
+			if ((0, fs.readFileSync)(staleShimPath, "utf8").includes(`agent-skills-standard@${latestVersion}`)) {
+				this.printShimPathHint();
+				return;
+			}
+			const correctShimPath = path.default.join(pnpmBin, "ags");
+			if (!(0, fs.existsSync)(correctShimPath)) {
+				console.log(picocolors.default.yellow(`⚠️  Correct shim not found at ${correctShimPath}. Re-run: ${picocolors.default.bold("pnpm add -g agent-skills-standard@latest")}`));
+				this.printShimPathHint();
+				return;
+			}
+			console.log("\n" + picocolors.default.yellow(`🔍 Stale shim detected: ${staleShimPath}`));
+			console.log(picocolors.default.gray("   It points to an old install path and will always report the wrong version."));
+			const { remove } = await inquirer.default.prompt([{
+				type: "confirm",
+				name: "remove",
+				message: `Remove stale shim at ${staleShimPath}?`,
+				default: true
+			}]);
+			if (remove) {
+				(0, fs.unlinkSync)(staleShimPath);
+				console.log(picocolors.default.green("✅ Stale shim removed."));
+				console.log(picocolors.default.cyan(`   Run ${picocolors.default.bold("hash -r")} then ${picocolors.default.bold("ags -V")} to confirm the correct version.`));
+			} else this.printShimPathHint();
+		} catch {
+			this.printShimPathHint();
+		}
+	}
 	printManualInstructions(pm, version) {
 		const sudoPrefix = process.platform === "win32" ? "" : "sudo ";
 		console.log(picocolors.default.yellow("Possible reasons:"));
@@ -4972,8 +5089,10 @@ var ValidateCommand = class {
 //#endregion
 //#region src/index.ts
 dotenv.default.config();
+var _agsExecPath = process.argv[1] ?? "";
+if (_agsExecPath.includes("/pnpm/") && _agsExecPath.includes("/node_modules/agent-skills-standard/") && !_agsExecPath.includes(`agent-skills-standard@${package_default.version}`)) process.stderr.write(`\n${picocolors.default.yellow("⚠️  ags is running from a stale global shim — reported version will be wrong.")}\n   ${picocolors.default.cyan("Quickfix:")} ${picocolors.default.bold("npx agent-skills-standard@latest upgrade")} ` + picocolors.default.gray("(auto-detects & removes the stale shim)") + "\n\n");
 var program = new commander.Command();
-program.name("agent-skills-standard").description("A CLI to manage and sync AI agent skills for Cursor, Claude, Copilot, Windsurf, and more.").version("2.4.3");
+program.name("agent-skills-standard").description("A CLI to manage and sync AI agent skills for Cursor, Claude, Copilot, Windsurf, and more.").version(package_default.version);
 program.command("init").description("Initialize a .skillsrc configuration file interactively").action(async () => {
 	await new InitCommand().run();
 });