@olegkoval/agent-skills 1.9.2 → 1.10.0

package/scripts/build-adapters.sh

@@ -16,14 +16,10 @@ const skillPathFor = (pkg) => path.join(root, pkg.path, 'SKILL.md')
 const promptNameFor = (pkg) => `${pkg.name}.prompt.md`
 const generatedHeader = '<!-- Generated by scripts/build-adapters.sh. Do not edit directly. -->'
 const stripFrontmatter = (content) => content.replace(/^---\n[\s\S]*?\n---\n?/, '')
-const injectTargets = (content, targets) => {
-  const targetsYaml = targets.length > 0
-    ? `  targets: [${targets.map(t => JSON.stringify(t)).join(', ')}]`
-    : '  targets: []'
-  if (/^  targets:/m.test(content)) {
-    return content.replace(/^  targets:.*$/m, targetsYaml)
-  }
-  return content.replace(/^(metadata:)$/m, `$1\n${targetsYaml}`)
+const extractFrontmatter = (content) => {
+  const m = content.match(/^(---\n[\s\S]*?\n---\n?)/)
+  if (!m) return ''
+  return m[1].endsWith('\n') ? m[1] : m[1] + '\n'
 }
 const stripTrailingLineWhitespace = (content) => content.replace(/[ \t]+$/gm, '')
 const copySkillResources = (pkg, destDir) => {
@@ -193,6 +189,9 @@ for (const pkg of catalog.packages) {
 
 for (const pkg of catalog.packages) {
   const canonical = fs.readFileSync(skillPathFor(pkg), 'utf8').trimEnd()
+  const fm = extractFrontmatter(canonical)
+  if (!fm) throw new Error(`canonical SKILL.md has no frontmatter: ${skillPathFor(pkg)}`)
+  const body = stripTrailingLineWhitespace(stripFrontmatter(canonical).trimStart().trimEnd())
 
   if (pkg.adapters.includes('claude')) {
     const claudeDestDir = path.join(root, pkg.path, 'adapters', 'claude', 'skills', pkg.name)
@@ -209,7 +208,7 @@ for (const pkg of catalog.packages) {
         2,
       ) + '\n',
     )
-    fs.writeFileSync(path.join(claudeDestDir, 'SKILL.md'), [generatedHeader, '', injectTargets(canonical, ['claude']), ''].join('\n'))
+    fs.writeFileSync(path.join(claudeDestDir, 'SKILL.md'), [fm + generatedHeader, '', body, ''].join('\n'))
     copySkillResources(pkg, claudeDestDir)
   }
 
@@ -230,12 +229,12 @@ for (const pkg of catalog.packages) {
         2,
       ) + '\n',
     )
-    fs.writeFileSync(path.join(destDir, 'SKILL.md'), [generatedHeader, '', injectTargets(canonical, ['cursor']), ''].join('\n'))
+    fs.writeFileSync(path.join(destDir, 'SKILL.md'), [fm + generatedHeader, '', body, ''].join('\n'))
     copySkillResources(pkg, destDir)
   }
 
   if (pkg.adapters.includes('windsurf')) {
-    const skillBody = stripFrontmatter(canonical).trimEnd()
+    const skillBody = stripFrontmatter(canonical).trimStart().trimEnd()
     const content = stripTrailingLineWhitespace([
       generatedHeader,
       '',
@@ -252,7 +251,7 @@ for (const pkg of catalog.packages) {
   }
 
   if (pkg.adapters.includes('kiro')) {
-    const skillBody = stripFrontmatter(canonical).trimEnd()
+    const skillBody = stripFrontmatter(canonical).trimStart().trimEnd()
     const content = stripTrailingLineWhitespace([
       generatedHeader,
       '',

package/scripts/validate-catalog.sh

@@ -118,6 +118,22 @@ for (const pkg of catalog.packages) {
     throw new Error(`stale flat package path exists: ${oldFlatPath}`)
   }
 }
+
+// Assert every SKILL.md (canonical + adapter wrappers) has a valid closed frontmatter block
+// Codex adapter files use README.md and are validated separately above
+const walk = (dir) => {
+  for (const entry of fs.readdirSync(dir, { withFileTypes: true })) {
+    const full = path.join(dir, entry.name)
+    if (entry.isDirectory()) walk(full)
+    else if (entry.name === 'SKILL.md') {
+      const content = fs.readFileSync(full, 'utf8')
+      if (!content.match(/^---\n[\s\S]*?\n---\n?/)) {
+        throw new Error(`SKILL.md does not have valid closed frontmatter: ${full}`)
+      }
+    }
+  }
+}
+walk('packages')
 EOF
 
 echo "catalog validation passed"