yeknal 1.3.3 → 1.4.1

package/README.md

@@ -41,7 +41,7 @@ Behavior:
 - If GitHub API rate limit is hit, it automatically falls back to `git clone` (Git must be installed).
 - Top-level folders are included only if they contain `SKILL.md`.
 - Installs each synced skill with a `yeknal-` folder prefix, for example `taste-skill` installs as `yeknal-taste-skill`.
-- Only the matching `yeknal-*` managed destination folders are overwritten during sync.
+- Matching `yeknal-*` managed destination folders are overwritten during sync, and stale `yeknal-*` folders removed from an earlier repository version are cleaned up. Unprefixed personal skill folders are never touched.
 - Sync targets (if parent folder exists):
 - Gemini: `~/.gemini/antigravity` or `~/.antigravity`
 - Codex: `~/.codex`

package/bin/yeknal.js

@@ -195,6 +195,26 @@ async function copyDirRecursive(sourceDir, targetDir) {
   }
 }
 
+async function removeStaleManagedSkillFolders(skillsPath, expectedFolders) {
+  const entries = await fsp.readdir(skillsPath, { withFileTypes: true });
+  const removed = [];
+
+  for (const entry of entries) {
+    if (!entry.isDirectory() || !entry.name.startsWith(MANAGED_SKILL_FOLDER_PREFIX)) {
+      continue;
+    }
+
+    if (expectedFolders.has(entry.name)) {
+      continue;
+    }
+
+    await fsp.rm(path.join(skillsPath, entry.name), { recursive: true, force: true });
+    removed.push(entry.name);
+  }
+
+  return removed;
+}
+
 function execCommand(command, options = {}) {
   return new Promise((resolve, reject) => {
     exec(command, options, (error, stdout, stderr) => {
@@ -408,6 +428,17 @@ async function runSkillsCommand() {
       try {
         await fsp.mkdir(target.skillsPath, { recursive: true });
         let copiedCount = 0;
+        const expectedFolders = new Set(
+          skillFolders.map((folder) => getManagedSkillFolderName(folder)),
+        );
+        const removedFolders = await removeStaleManagedSkillFolders(
+          target.skillsPath,
+          expectedFolders,
+        );
+
+        for (const removedFolder of removedFolders) {
+          console.log(`  [removed] ${target.label}: stale managed skill ${removedFolder}`);
+        }
 
         for (const folder of skillFolders) {
           const sourceFolder = path.join(tempRoot, folder);
@@ -446,9 +477,36 @@ const SCAN_IGNORE_DIRS = new Set([
   "node_modules", ".git", ".next", "dist", "build", ".cache", ".output",
   "coverage", ".nyc_output", "__pycache__", ".venv", "venv", "env",
   ".terraform", "vendor", "target", "bin", "obj", ".nuxt", ".svelte-kit",
-  ".vercel", ".netlify", ".turbo", "out", ".parcel-cache", "tmp",
+  ".astro", ".vercel", ".netlify", ".turbo", "out", ".parcel-cache", "tmp",
 ]);
 
+// Dependency backups and similarly named generated/build directories are
+// commonly left beside the working tree after reinstalls or builds.
+const SCAN_IGNORE_DIR_PATTERNS = [
+  /^node_modules(?:\.|[-_])(?:bak|backup)(?:[-_.]|$)/i,
+  /^(?:vendor|bower_components|\.venv|venv|env)(?:\.|[-_])(?:bak|backup)(?:[-_.]|$)/i,
+  /^(?:build|dist|out|coverage|generated|__generated__)(?:\.|[-_])(?:bak|backup)(?:[-_.]|$)/i,
+];
+
+// Generated source/config files can contain copied secrets or noisy bundled
+// content that is not the project's authored implementation.
+const SCAN_IGNORE_FILE_PATTERNS = [
+  /\.min\.(?:js|css|mjs|cjs)$/i,
+  /\.(?:bundle|chunk|generated|gen)\.[^.]+$/i,
+  /(?:^|[._-])generated(?:[._-]|$)/i,
+  /\.d\.ts$/i,
+  /\.map$/i,
+];
+
+function isScanIgnoredDirectory(name) {
+  const nameLC = name.toLowerCase();
+  return SCAN_IGNORE_DIRS.has(nameLC) || SCAN_IGNORE_DIR_PATTERNS.some((pattern) => pattern.test(name));
+}
+
+function isScanIgnoredFile(name) {
+  return SCAN_IGNORE_FILE_PATTERNS.some((pattern) => pattern.test(name));
+}
+
 const SCAN_SOURCE_EXTENSIONS = new Set([
   ".js", ".ts", ".jsx", ".tsx", ".mjs", ".cjs",
   ".py", ".rb", ".php", ".go", ".rs", ".java", ".cs",
@@ -524,13 +582,16 @@ async function walkProjectFiles(rootDir) {
       const fullPath = path.join(dir, entry.name);
 
       if (entry.isDirectory()) {
-        if (!SCAN_IGNORE_DIRS.has(entry.name)) {
+        if (!isScanIgnoredDirectory(entry.name)) {
           await walk(fullPath, depth + 1);
         }
         continue;
       }
 
       if (entry.isFile()) {
+        if (isScanIgnoredFile(entry.name)) {
+          continue;
+        }
         const ext = path.extname(entry.name).toLowerCase();
         const nameLC = entry.name.toLowerCase();
         if (SCAN_ALL_EXTENSIONS.has(ext) || nameLC.startsWith(".env")) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "yeknal",
-  "version": "1.3.3",
+  "version": "1.4.1",
   "description": "CLI to fetch markdown templates and sync AI agent skills",
   "main": "bin/yeknal.js",
   "bin": {