agent-skillboard 0.2.17 → 0.2.18

package/src/source-profiles.mjs

@@ -1,26 +1,26 @@
-import { readdir, readFile } from "node:fs/promises";
-import { basename, dirname, join, relative, resolve } from "node:path";
-import YAML from "yaml";
+import { readdir, readFile } from "node:fs/promises";
+import { basename, dirname, join, relative, resolve } from "node:path";
+import YAML from "yaml";
 import { readString, requireRecord } from "./config-helpers.mjs";
 import { textChangePlan } from "./change-plan.mjs";
 import { loadSourceProfile } from "./source-profile-loader.mjs";
-
-export { loadSourceProfile };
-
-export async function importSource(options) {
-  const sourceRoot = resolve(options.sourceRoot);
-  const profile = options.profile;
-  const files = await findSkillFiles(sourceRoot);
-  const matchedFiles = files
-    .map((file) => ({
-      file,
+
+export { loadSourceProfile };
+
+export async function importSource(options) {
+  const sourceRoot = resolve(options.sourceRoot);
+  const profile = options.profile;
+  const files = await findSkillFiles(sourceRoot);
+  const matchedFiles = files
+    .map((file) => ({
+      file,
       relativeFile: relative(sourceRoot, file).replace(/\\/g, "/")
-    }))
-    .filter((entry) => matchesAnyProfilePattern(entry.relativeFile, profile.skillPaths))
-    .sort((left, right) => left.relativeFile.localeCompare(right.relativeFile));
-  const skills = [];
-  const warnings = [];
-
+    }))
+    .filter((entry) => matchesAnyProfilePattern(entry.relativeFile, profile.skillPaths))
+    .sort((left, right) => left.relativeFile.localeCompare(right.relativeFile));
+  const skills = [];
+  const warnings = [];
+
   for (const entry of matchedFiles) {
     const frontmatter = parseSkillFrontmatter(await readFile(entry.file, "utf8"));
     const pathRule = matchingPathRule(entry.relativeFile, profile);
@@ -37,17 +37,17 @@ export async function importSource(options) {
       ownerInstallUnit: profile.id,
       description: frontmatter.description
     });
-  }
-
-  if (skills.length === 0) {
-    warnings.push(`No SKILL.md files matched profile ${profile.id} under ${sourceRoot}`);
-  }
-
-  return {
-    profile,
-    sourceRoot,
-    skills,
-    installUnit: {
+  }
+
+  if (skills.length === 0) {
+    warnings.push(`No SKILL.md files matched profile ${profile.id} under ${sourceRoot}`);
+  }
+
+  return {
+    profile,
+    sourceRoot,
+    skills,
+    installUnit: {
       id: profile.id,
       kind: profile.kind,
       sourceClass: profile.sourceClass,
@@ -58,31 +58,31 @@ export async function importSource(options) {
       publicKey: profile.publicKey,
       verifiedAt: profile.verifiedAt,
       source: profile.source || sourceRoot,
-      scope: profile.scope,
-      manifestPath: profile.manifestPath,
-      cachePath: profile.cachePath,
-      providedComponents: profile.providedComponents,
-      components: {
-        skills: skills.map((skill) => skill.id),
-        commands: profile.components.commands,
-        hooks: profile.components.hooks,
-        mcpServers: profile.components.mcpServers
-      },
-      modifiedConfigFiles: profile.modifiedConfigFiles,
-      autoUpdate: profile.autoUpdate,
-      enabled: profile.enabled,
-      workflowDependencies: profile.workflowDependencies,
-      permissionRisk: profile.permissionRisk,
-      rollback: profile.rollback
-    },
-    warnings
-  };
-}
-
-export function renderImportFragment(imported) {
-  return YAML.stringify(importFragment(imported));
-}
-
+      scope: profile.scope,
+      manifestPath: profile.manifestPath,
+      cachePath: profile.cachePath,
+      providedComponents: profile.providedComponents,
+      components: {
+        skills: skills.map((skill) => skill.id),
+        commands: profile.components.commands,
+        hooks: profile.components.hooks,
+        mcpServers: profile.components.mcpServers
+      },
+      modifiedConfigFiles: profile.modifiedConfigFiles,
+      autoUpdate: profile.autoUpdate,
+      enabled: profile.enabled,
+      workflowDependencies: profile.workflowDependencies,
+      permissionRisk: profile.permissionRisk,
+      rollback: profile.rollback
+    },
+    warnings
+  };
+}
+
+export function renderImportFragment(imported) {
+  return YAML.stringify(importFragment(imported));
+}
+
 export function mergeImportFragment(configText, imported, options = {}) {
   const document = YAML.parseDocument(configText);
   if (document.errors.length > 0) {
@@ -117,7 +117,7 @@ export function mergeImportFragment(configText, imported, options = {}) {
     addedSkills: skillIds,
     addedInstallUnits: unitIds,
     replacedSkills: duplicateSkills,
-    replacedInstallUnits: duplicateUnits
+    replacedInstallUnits: duplicateUnits
   };
 }
 
@@ -141,20 +141,20 @@ function requireYamlMap(value, label) {
 function preserveLineEndings(text, reference) {
   return reference.includes("\r\n") ? text.replace(/\n/g, "\r\n") : text;
 }
-
+
 function importFragment(imported) {
-  const skills = {};
-  for (const skill of imported.skills) {
-    skills[skill.id] = {
-      path: skill.path,
-      status: skill.status,
-      invocation: skill.invocation,
-      exposure: skill.exposure,
-      category: skill.category,
-      owner_install_unit: skill.ownerInstallUnit
-    };
-  }
-  const unit = imported.installUnit;
+  const skills = {};
+  for (const skill of imported.skills) {
+    skills[skill.id] = {
+      path: skill.path,
+      status: skill.status,
+      invocation: skill.invocation,
+      exposure: skill.exposure,
+      category: skill.category,
+      owner_install_unit: skill.ownerInstallUnit
+    };
+  }
+  const unit = imported.installUnit;
   return stripUndefined({
     skills,
     install_units: {
@@ -168,64 +168,64 @@ function importFragment(imported) {
         public_key: unit.publicKey,
         verified_at: unit.verifiedAt,
         source: unit.source,
-        scope: unit.scope,
-        manifest_path: unit.manifestPath,
-        cache_path: unit.cachePath,
-        provided_components: unit.providedComponents,
-        components: {
-          skills: unit.components.skills,
-          commands: unit.components.commands,
-          hooks: unit.components.hooks,
-          mcp_servers: unit.components.mcpServers
-        },
-        modified_config_files: unit.modifiedConfigFiles,
-        auto_update: unit.autoUpdate,
-        enabled: unit.enabled,
-        workflow_dependencies: unit.workflowDependencies,
-        permission_risk: unit.permissionRisk,
+        scope: unit.scope,
+        manifest_path: unit.manifestPath,
+        cache_path: unit.cachePath,
+        provided_components: unit.providedComponents,
+        components: {
+          skills: unit.components.skills,
+          commands: unit.components.commands,
+          hooks: unit.components.hooks,
+          mcp_servers: unit.components.mcpServers
+        },
+        modified_config_files: unit.modifiedConfigFiles,
+        auto_update: unit.autoUpdate,
+        enabled: unit.enabled,
+        workflow_dependencies: unit.workflowDependencies,
+        permission_risk: unit.permissionRisk,
         rollback: unit.rollback
       }
     }
   });
 }
-
-function duplicateMessage(skillIds, unitIds) {
-  const parts = [];
-  if (skillIds.length > 0) {
-    parts.push(`skills already exist: ${skillIds.join(", ")}`);
-  }
-  if (unitIds.length > 0) {
-    parts.push(`install units already exist: ${unitIds.join(", ")}`);
-  }
-  return `${parts.join("; ")}. Re-run with --replace to overwrite.`;
-}
-
-async function findSkillFiles(root) {
-  const files = [];
-  const entries = await readdir(root, { withFileTypes: true }).catch(() => []);
-  for (const entry of entries) {
-    const path = join(root, entry.name);
-    if (entry.isDirectory()) {
-      files.push(...(await findSkillFiles(path)));
-    } else if (entry.isFile() && entry.name === "SKILL.md") {
-      files.push(path);
-    }
-  }
-  return files;
-}
-
-function parseSkillFrontmatter(text) {
-  const match = /^---[ \t]*\r?\n(?<body>[\s\S]*?)\r?\n---[ \t]*(?:\r?\n|$)/.exec(text);
-  if (match?.groups === undefined) {
-    throw new Error("SKILL.md is missing YAML frontmatter");
-  }
-  const raw = requireRecord(YAML.parse(match.groups.body), "SKILL.md frontmatter");
-  return {
-    name: readString(raw, "name", ""),
-    description: readString(raw, "description", "")
-  };
-}
-
+
+function duplicateMessage(skillIds, unitIds) {
+  const parts = [];
+  if (skillIds.length > 0) {
+    parts.push(`skills already exist: ${skillIds.join(", ")}`);
+  }
+  if (unitIds.length > 0) {
+    parts.push(`install units already exist: ${unitIds.join(", ")}`);
+  }
+  return `${parts.join("; ")}. Re-run with --replace to overwrite.`;
+}
+
+async function findSkillFiles(root) {
+  const files = [];
+  const entries = await readdir(root, { withFileTypes: true }).catch(() => []);
+  for (const entry of entries) {
+    const path = join(root, entry.name);
+    if (entry.isDirectory()) {
+      files.push(...(await findSkillFiles(path)));
+    } else if (entry.isFile() && entry.name === "SKILL.md") {
+      files.push(path);
+    }
+  }
+  return files;
+}
+
+function parseSkillFrontmatter(text) {
+  const match = /^---[ \t]*\r?\n(?<body>[\s\S]*?)\r?\n---[ \t]*(?:\r?\n|$)/.exec(text);
+  if (match?.groups === undefined) {
+    throw new Error("SKILL.md is missing YAML frontmatter");
+  }
+  const raw = requireRecord(YAML.parse(match.groups.body), "SKILL.md frontmatter");
+  return {
+    name: readString(raw, "name", ""),
+    description: readString(raw, "description", "")
+  };
+}
+
 function matchesAnyProfilePattern(file, patterns) {
   const activePatterns = patterns.length === 0 ? ["**/SKILL.md"] : patterns;
   return activePatterns.some((pattern) => patternToRegExp(pattern).test(file));
@@ -234,16 +234,16 @@ function matchesAnyProfilePattern(file, patterns) {
 function matchingPathRule(file, profile) {
   return (profile.pathRules ?? []).find((rule) => patternToRegExp(rule.pattern).test(file));
 }
-
-function patternToRegExp(pattern) {
-  const escaped = pattern
-    .replace(/[.+^${}()|[\]\\]/g, "\\$&")
+
+function patternToRegExp(pattern) {
+  const escaped = pattern
+    .replace(/[.+^${}()|[\]\\]/g, "\\$&")
     .split("**").join("\0")
     .split("*").join("[^/]*")
     .split("\0").join(".*");
-  return new RegExp(`^${escaped}$`);
-}
-
+  return new RegExp(`^${escaped}$`);
+}
+
 function skillSlug(relativeFile, frontmatter, profile) {
   if (profile.idStrategy === "frontmatter-name" && frontmatter.name.length > 0) {
     const segments = frontmatter.name.split(".");
@@ -251,14 +251,14 @@ function skillSlug(relativeFile, frontmatter, profile) {
   }
   return normalizeSlug(basename(dirname(relativeFile)));
 }
-
-function skillId(slug, frontmatter, profile) {
-  if (profile.idStrategy === "frontmatter-name" && frontmatter.name.includes(".")) {
-    return frontmatter.name;
-  }
-  return profile.namespace.length === 0 ? slug : `${profile.namespace}.${slug}`;
-}
-
+
+function skillId(slug, frontmatter, profile) {
+  if (profile.idStrategy === "frontmatter-name" && frontmatter.name.includes(".")) {
+    return frontmatter.name;
+  }
+  return profile.namespace.length === 0 ? slug : `${profile.namespace}.${slug}`;
+}
+
 function skillTargetPath(slug, profile) {
   return profile.targetPathPrefix.length === 0 ? slug : `${profile.targetPathPrefix}/${slug}`;
 }
@@ -276,13 +276,13 @@ function categoryFromPath(relativeFile, profile) {
 }
 
 function normalizeSlug(value) {
-  return value
-    .trim()
-    .toLowerCase()
-    .replace(/[^a-z0-9._-]+/g, "-")
-    .replace(/^-+|-+$/g, "");
-}
-
+  return value
+    .trim()
+    .toLowerCase()
+    .replace(/[^a-z0-9._-]+/g, "-")
+    .replace(/^-+|-+$/g, "");
+}
+
 function safeDefaultInvocation(invocation) {
   return invocation === "global-auto" ? "blocked" : invocation;
 }

package/src/workspace.mjs

@@ -1,7 +1,7 @@
 import { readdir, readFile, realpath, stat } from "node:fs/promises";
 import { homedir } from "node:os";
 import { dirname, isAbsolute, join, relative, resolve } from "node:path";
-import YAML from "yaml";
+import YAML from "yaml";
 import {
   readBoolean,
   readOptionalRecord,
@@ -21,8 +21,8 @@ import { parseInstallUnits } from "./install-units.mjs";
 import { normalizeSkillPath } from "./skill-paths.mjs";
 
 export async function loadWorkspace(options) {
-  const configText = await readFile(options.configPath, "utf8");
-  const parsed = YAML.parse(configText);
+  const configText = await readFile(options.configPath, "utf8");
+  const parsed = YAML.parse(configText);
   const config = requireRecord(parsed, "config root");
   const version = parseVersion(config.version);
   const skills = parseSkills(config.skills);
@@ -123,7 +123,7 @@ function resolveStoredPath(value, options) {
   }
   return resolve(dirname(options.configPath ?? "."), value);
 }
-
+
 async function findSkillFiles(root, seen = new Set()) {
   const files = [];
   const resolvedRoot = await realpath(root).catch(() => root);
@@ -148,8 +148,8 @@ async function findSkillFiles(root, seen = new Set()) {
     }
   }
   return files;
-}
-
+}
+
 function parseSkillFrontmatter(text) {
   const match = /^---[ \t]*\r?\n(?<body>[\s\S]*?)\r?\n---[ \t]*(?:\r?\n|$)/.exec(text);
   if (match?.groups === undefined) {
@@ -170,49 +170,49 @@ function parseSkillFrontmatter(text) {
     description: readString(raw, "description", "")
   };
 }
-
-function parseVersion(value) {
-  if (value === undefined) {
-    return 1;
-  }
-  if (value !== 1) {
-    throw new Error(`Unsupported config version: ${value}`);
-  }
-  return value;
-}
-
-function parseDefaults(value) {
-  const raw = requireRecord(value ?? {}, "defaults");
-  return {
-    invocationPolicy: readString(raw, "invocation_policy", "deny-by-default"),
-    allowModelInvocation: readBoolean(raw, "allow_model_invocation", false),
-    requireExplicitWorkflow: readBoolean(raw, "require_explicit_workflow", true)
-  };
-}
-
-function parseSkills(value) {
-  const raw = requireRecord(value ?? {}, "skills");
-  return Object.entries(raw).map(([id, entry]) => {
-    const skill = requireRecord(entry, `skills.${id}`);
-    const status = readString(skill, "status", "vendor");
-    const invocation = readString(skill, "invocation", "manual-only");
-    const exposure = readString(skill, "exposure", "exported");
-    if (!STATUS_VALUES.has(status)) {
-      throw new Error(`Unsupported status for ${id}: ${status}`);
-    }
-    if (!INVOCATION_VALUES.has(invocation)) {
-      throw new Error(`Unsupported invocation for ${id}: ${invocation}`);
-    }
-    if (!EXPOSURE_VALUES.has(exposure)) {
-      throw new Error(`Unsupported exposure for ${id}: ${exposure}`);
-    }
+
+function parseVersion(value) {
+  if (value === undefined) {
+    return 1;
+  }
+  if (value !== 1) {
+    throw new Error(`Unsupported config version: ${value}`);
+  }
+  return value;
+}
+
+function parseDefaults(value) {
+  const raw = requireRecord(value ?? {}, "defaults");
+  return {
+    invocationPolicy: readString(raw, "invocation_policy", "deny-by-default"),
+    allowModelInvocation: readBoolean(raw, "allow_model_invocation", false),
+    requireExplicitWorkflow: readBoolean(raw, "require_explicit_workflow", true)
+  };
+}
+
+function parseSkills(value) {
+  const raw = requireRecord(value ?? {}, "skills");
+  return Object.entries(raw).map(([id, entry]) => {
+    const skill = requireRecord(entry, `skills.${id}`);
+    const status = readString(skill, "status", "vendor");
+    const invocation = readString(skill, "invocation", "manual-only");
+    const exposure = readString(skill, "exposure", "exported");
+    if (!STATUS_VALUES.has(status)) {
+      throw new Error(`Unsupported status for ${id}: ${status}`);
+    }
+    if (!INVOCATION_VALUES.has(invocation)) {
+      throw new Error(`Unsupported invocation for ${id}: ${invocation}`);
+    }
+    if (!EXPOSURE_VALUES.has(exposure)) {
+      throw new Error(`Unsupported exposure for ${id}: ${exposure}`);
+    }
     return {
       id,
       path: normalizeSkillPath(readString(skill, "path", id), `skills.${id}.path`),
-      status,
-      invocation,
-      exposure,
-      category: readString(skill, "category", "uncategorized"),
+      status,
+      invocation,
+      exposure,
+      category: readString(skill, "category", "uncategorized"),
       canonicalFor: readStringList(skill, "canonical_for"),
       conflictsWith: readStringList(skill, "conflicts_with"),
       replacedBy: readOptionalString(skill, "replaced_by"),
@@ -266,39 +266,39 @@ function parseCapabilities(value) {
     };
   });
 }
-
-function parseHarnesses(value) {
-  const raw = requireRecord(value ?? {}, "harnesses");
-  return Object.entries(raw).map(([name, entry]) => {
-    const harness = requireRecord(entry, `harnesses.${name}`);
-    const status = readString(harness, "status", "available");
-    if (!HARNESS_STATUS_VALUES.has(status)) {
-      throw new Error(`Unsupported harness status for ${name}: ${status}`);
-    }
-    return {
-      name,
-      status,
-      workflows: readStringList(harness, "workflows"),
-      commands: readStringList(harness, "commands")
-    };
-  });
-}
-
-function parseWorkflows(value) {
-  const raw = requireRecord(value ?? {}, "workflows");
-  return Object.entries(raw).map(([name, entry]) => {
-    const workflow = requireRecord(entry, `workflows.${name}`);
-    return {
-      name,
-      harness: readString(workflow, "harness", "unspecified"),
-      activeSkills: readStringList(workflow, "active_skills"),
-      blockedSkills: readStringList(workflow, "blocked_skills"),
-      requiredOutputs: readStringList(workflow, "required_outputs"),
-      requiredCapabilities: parseRequiredCapabilities(workflow.required_capabilities, `workflows.${name}.required_capabilities`)
-    };
-  });
-}
-
+
+function parseHarnesses(value) {
+  const raw = requireRecord(value ?? {}, "harnesses");
+  return Object.entries(raw).map(([name, entry]) => {
+    const harness = requireRecord(entry, `harnesses.${name}`);
+    const status = readString(harness, "status", "available");
+    if (!HARNESS_STATUS_VALUES.has(status)) {
+      throw new Error(`Unsupported harness status for ${name}: ${status}`);
+    }
+    return {
+      name,
+      status,
+      workflows: readStringList(harness, "workflows"),
+      commands: readStringList(harness, "commands")
+    };
+  });
+}
+
+function parseWorkflows(value) {
+  const raw = requireRecord(value ?? {}, "workflows");
+  return Object.entries(raw).map(([name, entry]) => {
+    const workflow = requireRecord(entry, `workflows.${name}`);
+    return {
+      name,
+      harness: readString(workflow, "harness", "unspecified"),
+      activeSkills: readStringList(workflow, "active_skills"),
+      blockedSkills: readStringList(workflow, "blocked_skills"),
+      requiredOutputs: readStringList(workflow, "required_outputs"),
+      requiredCapabilities: parseRequiredCapabilities(workflow.required_capabilities, `workflows.${name}.required_capabilities`)
+    };
+  });
+}
+
 function parseRequiredCapabilities(value, label) {
   const raw = requireRecord(value ?? {}, label);
   return Object.entries(raw).map(([name, entry]) => {