@kurokeita/add-skill 1.7.0 → 1.9.0

package/dist/skills/skill-creator/scripts/init_skill.cjs

@@ -12,8 +12,8 @@
  *     node init_skill.cjs my-new-skill --path skills/public
  */
 
-const fs = require("node:fs");
-const path = require("node:path");
+const fs = require("node:fs")
+const path = require("node:path")
 
 const SKILL_TEMPLATE = `---
 name: {skill_name}
@@ -100,7 +100,7 @@ Files not intended to be loaded into context, but rather used within the output
 ---
 
 **Any unneeded directories can be deleted.** Not every skill requires all three types of resources.
-`;
+`
 
 const EXAMPLE_SCRIPT = `#!/usr/bin/env node
 
@@ -135,7 +135,7 @@ async function main() {
 }
 
 main();
-`;
+`
 
 const EXAMPLE_REFERENCE = `# Reference Documentation for {skill_title}
 
@@ -154,24 +154,24 @@ Replace with actual reference content or delete if not needed.
 - Prerequisites
 - Step-by-step instructions
 - Best practices
-`;
+`
 
 function titleCase(name) {
 	return name
 		.split("-")
 		.map((word) => word.charAt(0).toUpperCase() + word.slice(1))
-		.join(" ");
+		.join(" ")
 }
 
 async function main() {
-	const args = process.argv.slice(2);
+	const args = process.argv.slice(2)
 	if (args.length < 3 || args[1] !== "--path") {
-		console.log("Usage: node init_skill.cjs <skill-name> --path <path>");
-		process.exit(1);
+		console.log("Usage: node init_skill.cjs <skill-name> --path <path>")
+		process.exit(1)
 	}
 
-	const skillName = args[0];
-	const basePath = path.resolve(args[2]);
+	const skillName = args[0]
+	const basePath = path.resolve(args[2])
 
 	// Prevent path traversal
 	if (
@@ -179,30 +179,30 @@ async function main() {
 		skillName.includes("/") ||
 		skillName.includes("\\")
 	) {
-		console.error("❌ Error: Skill name cannot contain path separators.");
-		process.exit(1);
+		console.error("❌ Error: Skill name cannot contain path separators.")
+		process.exit(1)
 	}
 
-	const skillDir = path.join(basePath, skillName);
+	const skillDir = path.join(basePath, skillName)
 
 	// Additional check to ensure the resolved skillDir is actually inside basePath
 	if (!skillDir.startsWith(basePath)) {
-		console.error("❌ Error: Invalid skill name or path.");
-		process.exit(1);
+		console.error("❌ Error: Invalid skill name or path.")
+		process.exit(1)
 	}
 
 	if (fs.existsSync(skillDir)) {
-		console.error(`❌ Error: Skill directory already exists: ${skillDir}`);
-		process.exit(1);
+		console.error(`❌ Error: Skill directory already exists: ${skillDir}`)
+		process.exit(1)
 	}
 
-	const skillTitle = titleCase(skillName);
+	const skillTitle = titleCase(skillName)
 
 	try {
-		fs.mkdirSync(skillDir, { recursive: true });
-		fs.mkdirSync(path.join(skillDir, "scripts"));
-		fs.mkdirSync(path.join(skillDir, "references"));
-		fs.mkdirSync(path.join(skillDir, "assets"));
+		fs.mkdirSync(skillDir, { recursive: true })
+		fs.mkdirSync(path.join(skillDir, "scripts"))
+		fs.mkdirSync(path.join(skillDir, "references"))
+		fs.mkdirSync(path.join(skillDir, "assets"))
 
 		fs.writeFileSync(
 			path.join(skillDir, "SKILL.md"),
@@ -210,26 +210,26 @@ async function main() {
 				/{skill_title}/g,
 				skillTitle,
 			),
-		);
+		)
 		fs.writeFileSync(
 			path.join(skillDir, "scripts/example_script.cjs"),
 			EXAMPLE_SCRIPT.replace(/{skill_name}/g, skillName),
 			{ mode: 0o755 },
-		);
+		)
 		fs.writeFileSync(
 			path.join(skillDir, "references/example_reference.md"),
 			EXAMPLE_REFERENCE.replace(/{skill_title}/g, skillTitle),
-		);
+		)
 		fs.writeFileSync(
 			path.join(skillDir, "assets/example_asset.txt"),
 			"Placeholder for assets.",
-		);
+		)
 
-		console.log(`✅ Skill '${skillName}' initialized at ${skillDir}`);
+		console.log(`✅ Skill '${skillName}' initialized at ${skillDir}`)
 	} catch (err) {
-		console.error(`❌ Error: ${err.message}`);
-		process.exit(1);
+		console.error(`❌ Error: ${err.message}`)
+		process.exit(1)
 	}
 }
 
-main();
+main()

package/dist/skills/skill-creator/scripts/package_skill.cjs

@@ -9,48 +9,48 @@
  *     node package_skill.js <path/to/skill-folder> [output-directory]
  */
 
-const path = require("node:path");
-const { spawnSync } = require("node:child_process");
-const { validateSkill } = require("./validate_skill.cjs");
+const path = require("node:path")
+const { spawnSync } = require("node:child_process")
+const { validateSkill } = require("./validate_skill.cjs")
 
 async function main() {
-	const args = process.argv.slice(2);
+	const args = process.argv.slice(2)
 	if (args.length < 1) {
 		console.log(
 			"Usage: node package_skill.js <path/to/skill-folder> [output-directory]",
-		);
-		process.exit(1);
+		)
+		process.exit(1)
 	}
 
-	const skillPathArg = args[0];
-	const outputDirArg = args[1];
+	const skillPathArg = args[0]
+	const outputDirArg = args[1]
 
 	if (skillPathArg.includes("..") || outputDirArg?.includes("..")) {
-		console.error("❌ Error: Path traversal detected in arguments.");
-		process.exit(1);
+		console.error("❌ Error: Path traversal detected in arguments.")
+		process.exit(1)
 	}
 
-	const skillPath = path.resolve(skillPathArg);
-	const outputDir = outputDirArg ? path.resolve(outputDirArg) : process.cwd();
-	const skillName = path.basename(skillPath);
+	const skillPath = path.resolve(skillPathArg)
+	const outputDir = outputDirArg ? path.resolve(outputDirArg) : process.cwd()
+	const skillName = path.basename(skillPath)
 
 	// 1. Validate first
-	console.log("🔍 Validating skill...");
-	const result = validateSkill(skillPath);
+	console.log("🔍 Validating skill...")
+	const result = validateSkill(skillPath)
 	if (!result.valid) {
-		console.error(`❌ Validation failed: ${result.message}`);
-		process.exit(1);
+		console.error(`❌ Validation failed: ${result.message}`)
+		process.exit(1)
 	}
 
 	if (result.warning) {
-		console.warn(`⚠️  ${result.warning}`);
-		console.log("Please resolve all TODOs before packaging.");
-		process.exit(1);
+		console.warn(`⚠️  ${result.warning}`)
+		console.log("Please resolve all TODOs before packaging.")
+		process.exit(1)
 	}
-	console.log("✅ Skill is valid!");
+	console.log("✅ Skill is valid!")
 
 	// 2. Package
-	const outputFilename = path.join(outputDir, `${skillName}.skill`);
+	const outputFilename = path.join(outputDir, `${skillName}.skill`)
 
 	try {
 		// Zip everything except junk, keeping the folder structure
@@ -64,21 +64,21 @@ async function main() {
 		const zipProcess = spawnSync("zip", ["-r", outputFilename, "."], {
 			cwd: skillPath,
 			stdio: "inherit",
-		});
+		})
 
 		if (zipProcess.error) {
-			throw zipProcess.error;
+			throw zipProcess.error
 		}
 
 		if (zipProcess.status !== 0) {
-			throw new Error(`zip command failed with exit code ${zipProcess.status}`);
+			throw new Error(`zip command failed with exit code ${zipProcess.status}`)
 		}
 
-		console.log(`✅ Successfully packaged skill to: ${outputFilename}`);
+		console.log(`✅ Successfully packaged skill to: ${outputFilename}`)
 	} catch (err) {
-		console.error(`❌ Error packaging: ${err.message}`);
-		process.exit(1);
+		console.error(`❌ Error packaging: ${err.message}`)
+		process.exit(1)
 	}
 }
 
-main();
+main()

package/dist/skills/skill-creator/scripts/validate_skill.cjs

@@ -5,123 +5,123 @@
  * Leveraging existing dependencies when possible or providing a zero-dep fallback.
  */
 
-const fs = require("node:fs");
-const path = require("node:path");
+const fs = require("node:fs")
+const path = require("node:path")
 
 function validateSkill(skillPath) {
 	if (!fs.existsSync(skillPath) || !fs.statSync(skillPath).isDirectory()) {
-		return { valid: false, message: `Path is not a directory: ${skillPath}` };
+		return { valid: false, message: `Path is not a directory: ${skillPath}` }
 	}
 
-	const skillMdPath = path.join(skillPath, "SKILL.md");
+	const skillMdPath = path.join(skillPath, "SKILL.md")
 	if (!fs.existsSync(skillMdPath)) {
-		return { valid: false, message: "SKILL.md not found" };
+		return { valid: false, message: "SKILL.md not found" }
 	}
 
-	const content = fs.readFileSync(skillMdPath, "utf8");
+	const content = fs.readFileSync(skillMdPath, "utf8")
 	if (!content.startsWith("---")) {
-		return { valid: false, message: "No YAML frontmatter found" };
+		return { valid: false, message: "No YAML frontmatter found" }
 	}
 
-	const parts = content.split("---");
+	const parts = content.split("---")
 	if (parts.length < 3) {
-		return { valid: false, message: "Invalid frontmatter format" };
+		return { valid: false, message: "Invalid frontmatter format" }
 	}
 
-	const frontmatterText = parts[1];
+	const frontmatterText = parts[1]
 
-	const nameMatch = frontmatterText.match(/^name:\s*(.+)$/m);
+	const nameMatch = frontmatterText.match(/^name:\s*(.+)$/m)
 	// Match description: "text" or description: 'text' or description: text
 	const descMatch = frontmatterText.match(
 		/^description:\s*(?:'([^']*)'|"([^"]*)"|(.+))$/m,
-	);
+	)
 
 	if (!nameMatch)
-		return { valid: false, message: 'Missing "name" in frontmatter' };
+		return { valid: false, message: 'Missing "name" in frontmatter' }
 	if (!descMatch)
 		return {
 			valid: false,
 			message: "Description must be a single-line string: description: ...",
-		};
+		}
 
-	const name = nameMatch[1].trim();
+	const name = nameMatch[1].trim()
 	const description = (
 		descMatch[1] !== undefined
 			? descMatch[1]
 			: descMatch[2] !== undefined
 				? descMatch[2]
 				: descMatch[3] || ""
-	).trim();
+	).trim()
 
 	if (description.includes("\n")) {
 		return {
 			valid: false,
 			message: "Description must be a single line (no newlines)",
-		};
+		}
 	}
 
 	if (!/^[a-z0-9-]+$/.test(name)) {
-		return { valid: false, message: `Name "${name}" should be hyphen-case` };
+		return { valid: false, message: `Name "${name}" should be hyphen-case` }
 	}
 
 	if (description.length > 1024) {
-		return { valid: false, message: "Description is too long (max 1024)" };
+		return { valid: false, message: "Description is too long (max 1024)" }
 	}
 
 	// Check for TODOs
-	const files = getAllFiles(skillPath);
+	const files = getAllFiles(skillPath)
 	for (const file of files) {
-		const fileContent = fs.readFileSync(file, "utf8");
+		const fileContent = fs.readFileSync(file, "utf8")
 		if (fileContent.includes("TODO:")) {
 			return {
 				valid: true,
 				message: "Skill has unresolved TODOs",
 				warning: `Found unresolved TODO in ${path.relative(skillPath, file)}`,
-			};
+			}
 		}
 	}
 
-	return { valid: true, message: "Skill is valid!" };
+	return { valid: true, message: "Skill is valid!" }
 }
 
 function getAllFiles(dir, fileList = []) {
-	const files = fs.readdirSync(dir);
+	const files = fs.readdirSync(dir)
 	files.forEach((file) => {
-		const name = path.join(dir, file);
+		const name = path.join(dir, file)
 		if (fs.statSync(name).isDirectory()) {
 			if (!["node_modules", ".git", "__pycache__"].includes(file)) {
-				getAllFiles(name, fileList);
+				getAllFiles(name, fileList)
 			}
 		} else {
-			fileList.push(name);
+			fileList.push(name)
 		}
-	});
-	return fileList;
+	})
+	return fileList
 }
 
 if (require.main === module) {
-	const args = process.argv.slice(2);
+	const args = process.argv.slice(2)
 	if (args.length !== 1) {
-		console.log("Usage: node validate_skill.js <skill_directory>");
-		process.exit(1);
+		console.log("Usage: node validate_skill.js <skill_directory>")
+		process.exit(1)
 	}
 
-	const skillDirArg = args[0];
+	const skillDirArg = args[0]
 	if (skillDirArg.includes("..")) {
-		console.error("❌ Error: Path traversal detected in skill directory path.");
-		process.exit(1);
+		console.error("❌ Error: Path traversal detected in skill directory path.")
+		process.exit(1)
 	}
 
-	const result = validateSkill(path.resolve(skillDirArg));
+	const result = validateSkill(path.resolve(skillDirArg))
 	if (result.warning) {
-		console.warn(`⚠️  ${result.warning}`);
+		console.warn(`⚠️  ${result.warning}`)
 	}
 	if (result.valid) {
-		console.log(`✅ ${result.message}`);
+		console.log(`✅ ${result.message}`)
 	} else {
-		console.error(`❌ ${result.message}`);
-		process.exit(1);
+		console.error(`❌ ${result.message}`)
+		process.exit(1)
 	}
 }
 
-module.exports = { validateSkill };
+module.exports = { validateSkill }

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@kurokeita/add-skill",
-	"version": "1.7.0",
+	"version": "1.9.0",
 	"description": "CLI to install AI agent skills to various platforms",
 	"type": "module",
 	"bin": {
@@ -13,10 +13,12 @@
 	"scripts": {
 		"dev": "tsx bin/cli.ts",
 		"update:readme": "tsx scripts/update-readme.ts",
-		"build": "tsc && tsx scripts/copy-assets.ts",
+		"build": "tsc --noEmit && esbuild bin/cli.ts --bundle --platform=node --format=esm --outfile=dist/bin/cli.js --packages=external && tsx scripts/copy-assets.ts",
 		"prepublishOnly": "pnpm run build",
+		"test": "vitest run",
+		"test:coverage": "vitest run --coverage",
 		"lint": "pnpm run lint:ts && pnpm run lint:md '**/*.md'",
-		"lint:ts": "biome check .",
+		"lint:ts": "biome check --error-on-warnings .",
 		"lint:md": "markdownlint --ignore node_modules '**/*.md'",
 		"format": "biome check --write . && pnpm run lint:md --fix",
 		"release": "semantic-release"
@@ -38,17 +40,22 @@
 		"@clack/prompts": "^0.11.0",
 		"commander": "^14.0.2",
 		"fs-extra": "^11.3.3",
+		"js-yaml": "^4.1.1",
 		"picocolors": "^1.1.1"
 	},
 	"devDependencies": {
 		"@biomejs/biome": "^2.3.11",
 		"@semantic-release/git": "^10.0.1",
 		"@types/fs-extra": "^11.0.4",
+		"@types/js-yaml": "^4.0.9",
 		"@types/node": "^25.0.9",
+		"@vitest/coverage-v8": "^4.0.18",
+		"esbuild": "^0.25.0",
 		"markdownlint-cli": "^0.47.0",
 		"semantic-release": "^25.0.2",
 		"tsx": "^4.21.0",
-		"typescript": "^5.9.3"
+		"typescript": "^5.9.3",
+		"vitest": "^4.0.18"
 	},
 	"publishConfig": {
 		"access": "public"