agentpacks 0.9.0 → 1.0.0

package/dist/importers/cursor.js

@@ -129,14 +129,104 @@ function serializeFrontmatter(data, content) {
   return matter.stringify(content, filtered);
 }
 
+// src/features/skills.ts
+import { readFileSync as readFileSync2, existsSync as existsSync2 } from "fs";
+import { basename, join as join2 } from "path";
+var SKILL_NAME_PATTERN = /^[a-z0-9]+(?:-[a-z0-9]+)*$/;
+var SKILL_NAME_MAX_LENGTH = 64;
+function parseSkills(skillsDir, packName) {
+  const dirs = listDirs(skillsDir);
+  const skills = [];
+  for (const dir of dirs) {
+    const skillMd = join2(dir, "SKILL.md");
+    if (existsSync2(skillMd)) {
+      skills.push(parseSkillFile(skillMd, dir, packName));
+    }
+  }
+  return skills;
+}
+function parseSkillFile(filepath, skillDir, packName) {
+  const raw = readFileSync2(filepath, "utf-8");
+  const { data, content } = parseFrontmatter(raw);
+  return {
+    name: data.name ?? basename(skillDir),
+    sourcePath: filepath,
+    sourceDir: skillDir,
+    packName,
+    meta: data,
+    content
+  };
+}
+function buildSkillFrontmatter(skill) {
+  return {
+    ...skill.meta,
+    name: skill.name
+  };
+}
+function serializeSkill(skill) {
+  return serializeFrontmatter(buildSkillFrontmatter(skill), skill.content);
+}
+function normalizeImportedSkillMarkdown(source, skillName) {
+  const { data, content } = parseFrontmatter(source);
+  const normalized = {
+    ...data,
+    name: skillName
+  };
+  let addedDescription = false;
+  const description = normalized.description;
+  if (typeof description !== "string" || description.trim().length === 0) {
+    normalized.description = `Imported skill: ${skillName}`;
+    addedDescription = true;
+  }
+  return {
+    content: serializeFrontmatter(normalized, content),
+    addedDescription
+  };
+}
+function validateAgentSkillsFrontmatter(skill) {
+  const errors = [];
+  const dirName = basename(skill.sourceDir);
+  const declaredName = skill.meta.name;
+  if (typeof declaredName !== "string" || declaredName.trim().length === 0) {
+    errors.push('Missing required frontmatter field "name".');
+  } else {
+    if (declaredName.length > SKILL_NAME_MAX_LENGTH) {
+      errors.push(`Invalid "name": must be at most ${SKILL_NAME_MAX_LENGTH} characters.`);
+    }
+    if (!SKILL_NAME_PATTERN.test(declaredName)) {
+      errors.push('Invalid "name": use lowercase letters, numbers, and single hyphens only.');
+    }
+    if (declaredName !== dirName) {
+      errors.push(`Invalid "name": must match containing directory "${dirName}".`);
+    }
+  }
+  const description = skill.meta.description;
+  if (typeof description !== "string" || description.trim().length === 0) {
+    errors.push('Missing required frontmatter field "description".');
+  }
+  const allowedTools = skill.meta["allowed-tools"];
+  if (allowedTools !== undefined && (!Array.isArray(allowedTools) || allowedTools.some((tool) => typeof tool !== "string" || tool.length === 0))) {
+    errors.push('Invalid "allowed-tools": expected an array of non-empty strings.');
+  }
+  return errors;
+}
+function skillMatchesTarget(skill, targetId) {
+  const { targets } = skill.meta;
+  if (!targets || targets === "*")
+    return true;
+  if (Array.isArray(targets) && targets.includes("*"))
+    return true;
+  return Array.isArray(targets) && targets.includes(targetId);
+}
+
 // src/importers/cursor.ts
-import { existsSync as existsSync2, readFileSync as readFileSync2, writeFileSync as writeFileSync2, copyFileSync } from "fs";
-import { resolve, join as join2, basename } from "path";
+import { existsSync as existsSync3, readFileSync as readFileSync3, writeFileSync as writeFileSync2, copyFileSync } from "fs";
+import { resolve, join as join3, basename as basename2 } from "path";
 function importFromCursor(projectRoot, outputPackDir) {
   const cursorDir = resolve(projectRoot, ".cursor");
   const warnings = [];
   const filesImported = [];
-  if (!existsSync2(cursorDir)) {
+  if (!existsSync3(cursorDir)) {
     return {
       packDir: "",
       filesImported: [],
@@ -147,12 +237,12 @@ function importFromCursor(projectRoot, outputPackDir) {
   const packDir = outputPackDir ?? resolve(projectRoot, "packs", "cursor-import");
   ensureDir(packDir);
   const rulesDir = resolve(cursorDir, "rules");
-  if (existsSync2(rulesDir)) {
+  if (existsSync3(rulesDir)) {
     const outRulesDir = resolve(packDir, "rules");
     ensureDir(outRulesDir);
     const files = listFiles(rulesDir, { extension: ".mdc" });
     for (const file of files) {
-      const raw = readFileSync2(file, "utf-8");
+      const raw = readFileSync3(file, "utf-8");
       const { data, content } = parseFrontmatter(raw);
       const meta = {};
       if (data.description)
@@ -163,65 +253,71 @@ function importFromCursor(projectRoot, outputPackDir) {
         meta.globs = data.globs;
       meta.cursor = { ...data };
       const mdContent = buildAgentpacksRule(meta, content);
-      const name = basename(file, ".mdc");
-      const dest = join2(outRulesDir, `${name}.md`);
+      const name = basename2(file, ".mdc");
+      const dest = join3(outRulesDir, `${name}.md`);
       writeFileSync2(dest, mdContent);
       filesImported.push(dest);
     }
     const mdFiles = listFiles(rulesDir, { extension: ".md" });
     for (const file of mdFiles) {
-      const dest = join2(outRulesDir, basename(file));
+      const dest = join3(outRulesDir, basename2(file));
       copyFileSync(file, dest);
       filesImported.push(dest);
     }
   }
   const agentsDir = resolve(cursorDir, "agents");
-  if (existsSync2(agentsDir)) {
+  if (existsSync3(agentsDir)) {
     const outDir = resolve(packDir, "agents");
     ensureDir(outDir);
     const files = listFiles(agentsDir, { extension: ".md" });
     for (const file of files) {
-      const dest = join2(outDir, basename(file));
+      const dest = join3(outDir, basename2(file));
       copyFileSync(file, dest);
       filesImported.push(dest);
     }
   }
   const skillsDir = resolve(cursorDir, "skills");
-  if (existsSync2(skillsDir)) {
+  if (existsSync3(skillsDir)) {
     const outDir = resolve(packDir, "skills");
     ensureDir(outDir);
     const dirs = listDirs(skillsDir);
     for (const dir of dirs) {
-      const name = basename(dir);
-      const skillMd = join2(dir, "SKILL.md");
-      if (existsSync2(skillMd)) {
-        const outSkillDir = join2(outDir, name);
+      const name = basename2(dir);
+      const skillMd = join3(dir, "SKILL.md");
+      if (existsSync3(skillMd)) {
+        const outSkillDir = join3(outDir, name);
         ensureDir(outSkillDir);
-        copyFileSync(skillMd, join2(outSkillDir, "SKILL.md"));
-        filesImported.push(join2(outSkillDir, "SKILL.md"));
+        const rawSkill = readFileSync3(skillMd, "utf-8");
+        const normalized = normalizeImportedSkillMarkdown(rawSkill, name);
+        const dest = join3(outSkillDir, "SKILL.md");
+        writeFileSync2(dest, normalized.content);
+        filesImported.push(dest);
+        if (normalized.addedDescription) {
+          warnings.push(`skills/${name}/SKILL.md missing description; added import placeholder.`);
+        }
       }
     }
   }
   const commandsDir = resolve(cursorDir, "commands");
-  if (existsSync2(commandsDir)) {
+  if (existsSync3(commandsDir)) {
     const outDir = resolve(packDir, "commands");
     ensureDir(outDir);
     const files = listFiles(commandsDir, { extension: ".md" });
     for (const file of files) {
-      const dest = join2(outDir, basename(file));
+      const dest = join3(outDir, basename2(file));
       copyFileSync(file, dest);
       filesImported.push(dest);
     }
   }
   const mcpJson = resolve(cursorDir, "mcp.json");
-  if (existsSync2(mcpJson)) {
-    copyFileSync(mcpJson, join2(packDir, "mcp.json"));
-    filesImported.push(join2(packDir, "mcp.json"));
+  if (existsSync3(mcpJson)) {
+    copyFileSync(mcpJson, join3(packDir, "mcp.json"));
+    filesImported.push(join3(packDir, "mcp.json"));
   }
   const cursorIgnore = resolve(projectRoot, ".cursorignore");
-  if (existsSync2(cursorIgnore)) {
-    copyFileSync(cursorIgnore, join2(packDir, "ignore"));
-    filesImported.push(join2(packDir, "ignore"));
+  if (existsSync3(cursorIgnore)) {
+    copyFileSync(cursorIgnore, join3(packDir, "ignore"));
+    filesImported.push(join3(packDir, "ignore"));
   }
   writePackJson(packDir, "cursor-import", filesImported);
   return { packDir, filesImported, warnings, configGenerated: false };
@@ -250,7 +346,7 @@ function writePackJson(packDir, name, filesImported) {
     targets: "*",
     features: "*"
   };
-  const dest = join2(packDir, "pack.json");
+  const dest = join3(packDir, "pack.json");
   writeFileSync2(dest, JSON.stringify(packJson, null, 2) + `
 `);
   filesImported.push(dest);

package/dist/importers/opencode.js

@@ -111,14 +111,122 @@ function getHeader(type) {
   }
 }
 
+// src/utils/frontmatter.ts
+import matter from "gray-matter";
+function parseFrontmatter(source) {
+  const { data, content } = matter(source);
+  return {
+    data,
+    content: content.trim(),
+    raw: source
+  };
+}
+function serializeFrontmatter(data, content) {
+  const filtered = Object.fromEntries(Object.entries(data).filter(([, v]) => v !== undefined));
+  if (Object.keys(filtered).length === 0) {
+    return content;
+  }
+  return matter.stringify(content, filtered);
+}
+
+// src/features/skills.ts
+import { readFileSync as readFileSync2, existsSync as existsSync2 } from "fs";
+import { basename, join as join2 } from "path";
+var SKILL_NAME_PATTERN = /^[a-z0-9]+(?:-[a-z0-9]+)*$/;
+var SKILL_NAME_MAX_LENGTH = 64;
+function parseSkills(skillsDir, packName) {
+  const dirs = listDirs(skillsDir);
+  const skills = [];
+  for (const dir of dirs) {
+    const skillMd = join2(dir, "SKILL.md");
+    if (existsSync2(skillMd)) {
+      skills.push(parseSkillFile(skillMd, dir, packName));
+    }
+  }
+  return skills;
+}
+function parseSkillFile(filepath, skillDir, packName) {
+  const raw = readFileSync2(filepath, "utf-8");
+  const { data, content } = parseFrontmatter(raw);
+  return {
+    name: data.name ?? basename(skillDir),
+    sourcePath: filepath,
+    sourceDir: skillDir,
+    packName,
+    meta: data,
+    content
+  };
+}
+function buildSkillFrontmatter(skill) {
+  return {
+    ...skill.meta,
+    name: skill.name
+  };
+}
+function serializeSkill(skill) {
+  return serializeFrontmatter(buildSkillFrontmatter(skill), skill.content);
+}
+function normalizeImportedSkillMarkdown(source, skillName) {
+  const { data, content } = parseFrontmatter(source);
+  const normalized = {
+    ...data,
+    name: skillName
+  };
+  let addedDescription = false;
+  const description = normalized.description;
+  if (typeof description !== "string" || description.trim().length === 0) {
+    normalized.description = `Imported skill: ${skillName}`;
+    addedDescription = true;
+  }
+  return {
+    content: serializeFrontmatter(normalized, content),
+    addedDescription
+  };
+}
+function validateAgentSkillsFrontmatter(skill) {
+  const errors = [];
+  const dirName = basename(skill.sourceDir);
+  const declaredName = skill.meta.name;
+  if (typeof declaredName !== "string" || declaredName.trim().length === 0) {
+    errors.push('Missing required frontmatter field "name".');
+  } else {
+    if (declaredName.length > SKILL_NAME_MAX_LENGTH) {
+      errors.push(`Invalid "name": must be at most ${SKILL_NAME_MAX_LENGTH} characters.`);
+    }
+    if (!SKILL_NAME_PATTERN.test(declaredName)) {
+      errors.push('Invalid "name": use lowercase letters, numbers, and single hyphens only.');
+    }
+    if (declaredName !== dirName) {
+      errors.push(`Invalid "name": must match containing directory "${dirName}".`);
+    }
+  }
+  const description = skill.meta.description;
+  if (typeof description !== "string" || description.trim().length === 0) {
+    errors.push('Missing required frontmatter field "description".');
+  }
+  const allowedTools = skill.meta["allowed-tools"];
+  if (allowedTools !== undefined && (!Array.isArray(allowedTools) || allowedTools.some((tool) => typeof tool !== "string" || tool.length === 0))) {
+    errors.push('Invalid "allowed-tools": expected an array of non-empty strings.');
+  }
+  return errors;
+}
+function skillMatchesTarget(skill, targetId) {
+  const { targets } = skill.meta;
+  if (!targets || targets === "*")
+    return true;
+  if (Array.isArray(targets) && targets.includes("*"))
+    return true;
+  return Array.isArray(targets) && targets.includes(targetId);
+}
+
 // src/importers/opencode.ts
-import { existsSync as existsSync2, readFileSync as readFileSync2, writeFileSync as writeFileSync2, copyFileSync } from "fs";
-import { resolve, join as join2, basename } from "path";
+import { existsSync as existsSync3, readFileSync as readFileSync3, writeFileSync as writeFileSync2, copyFileSync } from "fs";
+import { resolve, join as join3, basename as basename2 } from "path";
 function importFromOpenCode(projectRoot, outputPackDir) {
   const warnings = [];
   const filesImported = [];
   const ocDir = resolve(projectRoot, ".opencode");
-  if (!existsSync2(ocDir)) {
+  if (!existsSync3(ocDir)) {
     return {
       packDir: "",
       filesImported: [],
@@ -132,41 +240,47 @@ function importFromOpenCode(projectRoot, outputPackDir) {
   importDirMd(resolve(ocDir, "commands"), resolve(packDir, "commands"), filesImported);
   importDirMd(resolve(ocDir, "agents"), resolve(packDir, "agents"), filesImported);
   const skillDir = resolve(ocDir, "skill");
-  if (existsSync2(skillDir)) {
+  if (existsSync3(skillDir)) {
     const outSkillDir = resolve(packDir, "skills");
     ensureDir(outSkillDir);
     const dirs = listDirs(skillDir);
     for (const dir of dirs) {
-      const name = basename(dir);
+      const name = basename2(dir);
       if (name.startsWith("."))
         continue;
-      const skillMd = join2(dir, "SKILL.md");
-      if (existsSync2(skillMd)) {
-        const outDir = join2(outSkillDir, name);
+      const skillMd = join3(dir, "SKILL.md");
+      if (existsSync3(skillMd)) {
+        const outDir = join3(outSkillDir, name);
         ensureDir(outDir);
-        copyFileSync(skillMd, join2(outDir, "SKILL.md"));
-        filesImported.push(join2(outDir, "SKILL.md"));
+        const rawSkill = readFileSync3(skillMd, "utf-8");
+        const normalized = normalizeImportedSkillMarkdown(rawSkill, name);
+        const dest2 = join3(outDir, "SKILL.md");
+        writeFileSync2(dest2, normalized.content);
+        filesImported.push(dest2);
+        if (normalized.addedDescription) {
+          warnings.push(`skills/${name}/SKILL.md missing description; added import placeholder.`);
+        }
       }
     }
   }
   const pluginsDir = resolve(ocDir, "plugins");
-  if (existsSync2(pluginsDir)) {
+  if (existsSync3(pluginsDir)) {
     const outPluginsDir = resolve(packDir, "plugins");
     ensureDir(outPluginsDir);
     const files = listFiles(pluginsDir);
     for (const file of files) {
       if (file.endsWith(".ts") || file.endsWith(".js")) {
-        const dest2 = join2(outPluginsDir, basename(file));
+        const dest2 = join3(outPluginsDir, basename2(file));
         copyFileSync(file, dest2);
         filesImported.push(dest2);
       }
     }
   }
   const agentsMd = resolve(projectRoot, "AGENTS.md");
-  if (existsSync2(agentsMd)) {
+  if (existsSync3(agentsMd)) {
     const outRulesDir = resolve(packDir, "rules");
     ensureDir(outRulesDir);
-    const raw = readFileSync2(agentsMd, "utf-8");
+    const raw = readFileSync3(agentsMd, "utf-8");
     const ruleContent = [
       "---",
       "root: true",
@@ -176,18 +290,18 @@ function importFromOpenCode(projectRoot, outputPackDir) {
       raw
     ].join(`
 `);
-    const dest2 = join2(outRulesDir, "agents-md-root.md");
+    const dest2 = join3(outRulesDir, "agents-md-root.md");
     writeFileSync2(dest2, ruleContent);
     filesImported.push(dest2);
   }
   const ocJson = resolve(projectRoot, "opencode.json");
-  if (existsSync2(ocJson)) {
+  if (existsSync3(ocJson)) {
     try {
-      const raw = readFileSync2(ocJson, "utf-8");
+      const raw = readFileSync3(ocJson, "utf-8");
       const config = JSON.parse(raw);
       const mcpObj = config.mcp;
       if (mcpObj) {
-        const dest2 = join2(packDir, "mcp.json");
+        const dest2 = join3(packDir, "mcp.json");
         writeFileSync2(dest2, JSON.stringify({ servers: mcpObj }, null, 2) + `
 `);
         filesImported.push(dest2);
@@ -197,9 +311,9 @@ function importFromOpenCode(projectRoot, outputPackDir) {
     }
   }
   const ocIgnore = resolve(projectRoot, ".opencodeignore");
-  if (existsSync2(ocIgnore)) {
-    copyFileSync(ocIgnore, join2(packDir, "ignore"));
-    filesImported.push(join2(packDir, "ignore"));
+  if (existsSync3(ocIgnore)) {
+    copyFileSync(ocIgnore, join3(packDir, "ignore"));
+    filesImported.push(join3(packDir, "ignore"));
   }
   const packJson = {
     name: "opencode-import",
@@ -211,19 +325,19 @@ function importFromOpenCode(projectRoot, outputPackDir) {
     targets: "*",
     features: "*"
   };
-  const dest = join2(packDir, "pack.json");
+  const dest = join3(packDir, "pack.json");
   writeFileSync2(dest, JSON.stringify(packJson, null, 2) + `
 `);
   filesImported.push(dest);
   return { packDir, filesImported, warnings, configGenerated: false };
 }
 function importDirMd(srcDir, outDir, filesImported) {
-  if (!existsSync2(srcDir))
+  if (!existsSync3(srcDir))
     return;
   ensureDir(outDir);
   const files = listFiles(srcDir, { extension: ".md" });
   for (const file of files) {
-    const dest = join2(outDir, basename(file));
+    const dest = join3(outDir, basename2(file));
     copyFileSync(file, dest);
     filesImported.push(dest);
   }