Function parseSkill

src/skills/loader.ts:106–146

(
  raw: string,
  name: string,
  dir: string,
  origin: SkillSpec['origin'],
)

/** Parse a SKILL.md into a SkillSpec. Returns null if the spec is unusable. */
export function parseSkill(
  raw: string,
  name: string,
  dir: string,
  origin: SkillSpec['origin'],
): SkillSpec | null {
  let body = raw;
  const fmMatch = raw.match(/^---\s*\n([\s\S]*?)\n---\s*\r?\n?([\s\S]*)$/);
  const fm = fmMatch ? fmMatch[1] ?? '' : '';
  if (fmMatch) body = fmMatch[2] ?? '';

  const fields = parseFrontmatter(fm);

  const description = (fields.description as string | undefined) ?? '';
  if (!description) {
    logger.warn('Skill missing description, skipping', { name, dir });
    return null;
  }

  // name in frontmatter is advisory — directory name wins so installer collisions stay safe.
  return {
    name,
    dir,
    origin,
    description,
    version: fields.version as string | undefined,
    body: body.trim(),
    allowedTools: asStringArray(fields['allowed-tools'] ?? fields['allowed_tools']),
    triggers: asStringArray(fields.triggers),
    slashAliases: asStringArray(fields['slash-aliases'] ?? fields['slash_aliases']),
    model: fields.model as string | undefined,
    files: asStringArray(fields.files),
    author: fields.author as string | undefined,
    source: fields.source as string | undefined,
    // Anti-self-congratulation: an ABSENT provenance means a human authored/installed
    // it → protected. Only the capture loop writes `provenance: machine`.
    provenance: fields.provenance === 'machine' ? 'machine' : 'user',
    humanEdited: String(fields['humanedited'] ?? fields['human-edited'] ?? '').toLowerCase() === 'true',
    status: fields.status === 'candidate' ? 'candidate' : 'active',
  };
}

/** Minimal YAML-ish parser. Same shape as custom-commands.ts for consistency. */
function parseFrontmatter(fm: string): Record<string, string | string[]> {