playbooks 0.1.2 → 0.1.3

package/dist/index.js

@@ -7,7 +7,7 @@ import { program } from "commander";
 // package.json
 var package_default = {
   name: "playbooks",
-  version: "0.1.2",
+  version: "0.1.3",
   description: "Install agent skills, MCPs and docs into your coding agents from any git repository.",
   type: "module",
   bin: {
@@ -950,12 +950,47 @@ async function isSkillInstalled(skillName, agentType, options = {}) {
 }
 
 // src/skills.ts
+import { existsSync as existsSync2 } from "fs";
 import { readFile, readdir as readdir2, stat } from "fs/promises";
 import { basename as basename2, dirname, join as join6 } from "path";
 import matter from "gray-matter";
-var SKIP_DIRS = ["node_modules", ".git", "dist", "build", "__pycache__"];
+var SKIP_DIRS = ["node_modules", ".git", ".github", "dist", "build", "__pycache__"];
+var DENIED_SEGMENTS = /* @__PURE__ */ new Set([
+  ".git",
+  "node_modules",
+  ".github",
+  ".codex",
+  "playbooks",
+  "context",
+  "prompts",
+  "backups",
+  "backup",
+  "dist",
+  "deprecated",
+  ".opencode"
+]);
+var normalizePath = (value) => value.replace(/^\/+/, "");
+var normalizeRoot = (value) => normalizePath(value).replace(/^\.\//, "").replace(/^\/+/, "").replace(/\/+$/, "");
+var isDeniedPath = (path) => {
+  const cleaned = normalizePath(path);
+  const segments = cleaned.split("/").map((segment) => segment.toLowerCase());
+  for (const segment of segments) {
+    if (!segment) continue;
+    if (segment === ".claude-plugin") {
+      continue;
+    }
+    if (segment.startsWith(".claude")) {
+      return true;
+    }
+    if (DENIED_SEGMENTS.has(segment)) {
+      return true;
+    }
+  }
+  return false;
+};
 async function hasSkillMd(dir) {
   try {
+    if (isDeniedPath(dir)) return false;
     const skillPath = join6(dir, "SKILL.md");
     const stats = await stat(skillPath);
     return stats.isFile();
@@ -965,6 +1000,7 @@ async function hasSkillMd(dir) {
 }
 async function parseSkillMd(skillMdPath) {
   try {
+    if (isDeniedPath(skillMdPath)) return null;
     const content = await readFile(skillMdPath, "utf-8");
     const { data } = matter(content);
     if (!data.name || !data.description) {
@@ -984,6 +1020,7 @@ async function parseSkillMd(skillMdPath) {
 async function findSkillDirs(dir, depth = 0, maxDepth = 5) {
   const skillDirs = [];
   if (depth > maxDepth) return skillDirs;
+  if (isDeniedPath(dir)) return skillDirs;
   try {
     if (await hasSkillMd(dir)) {
       skillDirs.push(dir);
@@ -999,9 +1036,51 @@ async function findSkillDirs(dir, depth = 0, maxDepth = 5) {
   }
   return skillDirs;
 }
+async function readMarketplacePluginRoots(basePath) {
+  const filePath = join6(basePath, ".claude-plugin", "marketplace.json");
+  if (!existsSync2(filePath)) return [];
+  try {
+    const raw = await readFile(filePath, "utf-8");
+    const parsed = JSON.parse(raw);
+    const roots = (parsed.plugins ?? []).map((plugin) => typeof plugin.source === "string" ? plugin.source : null).filter(Boolean);
+    return roots.map((root) => normalizeRoot(root)).filter(Boolean);
+  } catch {
+    return [];
+  }
+}
+async function collectSkillsFromRoot(root, seenSlugs, skills) {
+  if (!root || !existsSync2(root) || isDeniedPath(root)) return;
+  const skillDirs = await findSkillDirs(root);
+  for (const skillDir of skillDirs) {
+    const skill = await parseSkillMd(join6(skillDir, "SKILL.md"));
+    if (!skill) continue;
+    const slug = basename2(skill.path).toLowerCase();
+    if (seenSlugs.has(slug)) continue;
+    skills.push(skill);
+    seenSlugs.add(slug);
+  }
+}
+async function listPluginSkillRoots(basePath) {
+  const pluginsDir = join6(basePath, "plugins");
+  if (!existsSync2(pluginsDir)) return [];
+  try {
+    const entries = await readdir2(pluginsDir, { withFileTypes: true });
+    const roots = [];
+    for (const entry of entries) {
+      if (!entry.isDirectory()) continue;
+      const candidate = join6(pluginsDir, entry.name, "skills");
+      if (existsSync2(candidate)) {
+        roots.push(candidate);
+      }
+    }
+    return roots;
+  } catch {
+    return [];
+  }
+}
 async function discoverSkills(basePath, subpath) {
   const skills = [];
-  const seenNames = /* @__PURE__ */ new Set();
+  const seenSlugs = /* @__PURE__ */ new Set();
   const searchPath = subpath ? join6(basePath, subpath) : basePath;
   if (await hasSkillMd(searchPath)) {
     const skill = await parseSkillMd(join6(searchPath, "SKILL.md"));
@@ -1010,15 +1089,20 @@ async function discoverSkills(basePath, subpath) {
       return skills;
     }
   }
-  const prioritySearchDirs = [
-    searchPath,
-    join6(searchPath, "skills"),
-    join6(searchPath, "skills/.curated"),
-    join6(searchPath, "skills/.experimental"),
-    join6(searchPath, "skills/.system"),
+  const marketplaceRoots = await readMarketplacePluginRoots(searchPath);
+  for (const root of marketplaceRoots) {
+    const skillsRoot = root.toLowerCase().endsWith("/skills") ? root : `${root}/skills`;
+    await collectSkillsFromRoot(join6(searchPath, skillsRoot), seenSlugs, skills);
+  }
+  await collectSkillsFromRoot(join6(searchPath, "skills"), seenSlugs, skills);
+  const pluginRoots = await listPluginSkillRoots(searchPath);
+  for (const root of pluginRoots) {
+    await collectSkillsFromRoot(root, seenSlugs, skills);
+  }
+  await collectSkillsFromRoot(join6(searchPath, ".claude-plugin"), seenSlugs, skills);
+  const agentRoots = [
     join6(searchPath, ".agent/skills"),
     join6(searchPath, ".agents/skills"),
-    join6(searchPath, ".claude/skills"),
     join6(searchPath, ".cline/skills"),
     join6(searchPath, ".codex/skills"),
     join6(searchPath, ".commandcode/skills"),
@@ -1030,7 +1114,6 @@ async function discoverSkills(basePath, subpath) {
     join6(searchPath, ".kilocode/skills"),
     join6(searchPath, ".kiro/skills"),
     join6(searchPath, ".neovate/skills"),
-    join6(searchPath, ".opencode/skills"),
     join6(searchPath, ".openhands/skills"),
     join6(searchPath, ".pi/skills"),
     join6(searchPath, ".qoder/skills"),
@@ -1039,32 +1122,18 @@ async function discoverSkills(basePath, subpath) {
     join6(searchPath, ".windsurf/skills"),
     join6(searchPath, ".zencoder/skills")
   ];
-  for (const dir of prioritySearchDirs) {
-    try {
-      const entries = await readdir2(dir, { withFileTypes: true });
-      for (const entry of entries) {
-        if (entry.isDirectory()) {
-          const skillDir = join6(dir, entry.name);
-          if (await hasSkillMd(skillDir)) {
-            const skill = await parseSkillMd(join6(skillDir, "SKILL.md"));
-            if (skill && !seenNames.has(skill.name)) {
-              skills.push(skill);
-              seenNames.add(skill.name);
-            }
-          }
-        }
-      }
-    } catch {
-    }
+  for (const root of agentRoots) {
+    await collectSkillsFromRoot(root, seenSlugs, skills);
   }
   if (skills.length === 0) {
     const allSkillDirs = await findSkillDirs(searchPath);
     for (const skillDir of allSkillDirs) {
       const skill = await parseSkillMd(join6(skillDir, "SKILL.md"));
-      if (skill && !seenNames.has(skill.name)) {
-        skills.push(skill);
-        seenNames.add(skill.name);
-      }
+      if (!skill) continue;
+      const slug = basename2(skill.path).toLowerCase();
+      if (seenSlugs.has(slug)) continue;
+      skills.push(skill);
+      seenSlugs.add(slug);
     }
   }
   return skills;
@@ -2034,10 +2103,10 @@ function AddScopeScreen() {
 }
 
 // src/tui/screens/AddSkillSelect.tsx
-import { existsSync as existsSync3 } from "fs";
+import { existsSync as existsSync4 } from "fs";
 import { mkdir as mkdir4, mkdtemp as mkdtemp2, writeFile as writeFile3 } from "fs/promises";
 import { tmpdir as tmpdir3 } from "os";
-import { join as join11 } from "path";
+import { basename as basename3, join as join11 } from "path";
 import { Box as Box15, Text as Text13 } from "ink";
 import React11 from "react";
 
@@ -2351,7 +2420,7 @@ async function resolveRemoteSkill(url) {
 }
 
 // src/marketplace.ts
-import { existsSync as existsSync2, statSync } from "fs";
+import { existsSync as existsSync3, statSync } from "fs";
 import { readFile as readFile3 } from "fs/promises";
 import { dirname as dirname3, join as join10, posix } from "path";
 function toRecord(value) {
@@ -2418,14 +2487,14 @@ function isMarketplaceInput(input) {
   return input.toLowerCase().endsWith("marketplace.json");
 }
 function resolveLocalMarketplacePath(input) {
-  if (!existsSync2(input)) return null;
+  if (!existsSync3(input)) return null;
   const stats = statSync(input);
   if (stats.isFile() && input.toLowerCase().endsWith("marketplace.json")) {
     return input;
   }
   if (stats.isDirectory()) {
     const candidate = join10(input, ".claude-plugin", "marketplace.json");
-    if (existsSync2(candidate)) return candidate;
+    if (existsSync3(candidate)) return candidate;
   }
   return null;
 }
@@ -2799,7 +2868,7 @@ function AddSkillSelectScreen() {
             throw new Error("Local path is missing.");
           }
           skillsDir = parsed.localPath;
-          if (!existsSync3(skillsDir)) {
+          if (!existsSync4(skillsDir)) {
             throw new Error(`Local path does not exist: ${skillsDir}`);
           }
         } else {
@@ -2843,6 +2912,9 @@ function AddSkillSelectScreen() {
         if (autoSelection.status === "error") {
           throw new Error(autoSelection.message);
         }
+        if (autoSelection.status === "prompt" && autoSelection.message) {
+          setFlash(autoSelection.message);
+        }
         if (tempDir) {
           keepTempDir = true;
         }
@@ -2864,7 +2936,7 @@ function AddSkillSelectScreen() {
     return () => {
       cancelled = true;
     };
-  }, [source, addSkill.skills, updateAddSkill, navigateTo, options]);
+  }, [source, addSkill.skills, updateAddSkill, navigateTo, options, setFlash]);
   React11.useEffect(() => {
     if (invocation.source) {
       setBackHandler(() => {
@@ -2963,16 +3035,18 @@ function AddSkillSelectScreen() {
     )
   ] });
 }
+function matchesSkillName(skill, input) {
+  const normalized = input.toLowerCase();
+  const byName = skill.name.toLowerCase() === normalized;
+  const byPath = basename3(skill.path).toLowerCase() === normalized;
+  return byName || byPath;
+}
 function autoSelect(skills, options) {
   if (options.skill && options.skill.length > 0) {
-    const selected = skills.filter(
-      (s) => options.skill?.some(
-        (name) => s.name.toLowerCase() === name.toLowerCase() || getSkillDisplayName(s).toLowerCase() === name.toLowerCase()
-      )
-    );
+    const selected = skills.filter((s) => options.skill?.some((name) => matchesSkillName(s, name)));
     if (selected.length === 0) {
       return {
-        status: "error",
+        status: "prompt",
         message: `No matching skills found for: ${options.skill.join(", ")}`
       };
     }
@@ -3140,7 +3214,7 @@ import React14 from "react";
 
 // src/installed-skills.ts
 import { lstat as lstat2, readFile as readFile4, readdir as readdir3, stat as stat2 } from "fs/promises";
-import { basename as basename3, join as join12 } from "path";
+import { basename as basename4, join as join12 } from "path";
 import matter6 from "gray-matter";
 function getAgentSkillsDir(agent, scope, cwd) {
   const config = agents[agent];
@@ -3215,7 +3289,7 @@ async function listSkillsForAgent(agent, scope, cwd = process.cwd()) {
 }
 async function findSkillInstallations(skillName, scope, cwd = process.cwd()) {
   const installs = [];
-  const sanitized = basename3(getCanonicalPath(skillName, { global: scope === "global", cwd }));
+  const sanitized = basename4(getCanonicalPath(skillName, { global: scope === "global", cwd }));
   if (!sanitized) {
     return installs;
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "playbooks",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "Install agent skills, MCPs and docs into your coding agents from any git repository.",
   "type": "module",
   "bin": {