agent-skillboard 0.2.16 → 0.2.18

package/src/cli.mjs

@@ -142,8 +142,8 @@ async function run(argv, stdout, stderr, stdin) {
       return await sources(argv.slice(1), options, stdout);
     case "import":
       return await importProfile(options, stdout);
-    case "scan":
-      return await scan(options, stdout);
+    case "scan":
+      return await scan(options, stdout);
     case "check":
       return await check(options, stdout, stderr);
     case "doctor":
@@ -192,10 +192,10 @@ async function run(argv, stdout, stderr, stdin) {
       return await remove(argv.slice(1), options, stdout);
     case "dashboard":
       return await dashboard(options, stdout);
-    case "reconcile":
-      return await reconcile(options, stdout);
-    case "impact":
-      return await impact(argv.slice(1), options, stdout);
+    case "reconcile":
+      return await reconcile(options, stdout);
+    case "impact":
+      return await impact(argv.slice(1), options, stdout);
     case "help":
     case "--help":
     case "-h":
@@ -224,30 +224,30 @@ function selectedHelpText(command, args, options) {
   }
   return null;
 }
-
+
 async function importProfile(options, stdout) {
-  const profileRef = options.get("profile");
-  const sourceRoot = options.get("source-root");
-  if (profileRef === undefined || sourceRoot === undefined) {
-    throw new Error("Usage: skillboard import --profile <id-or-path> --source-root <dir>");
-  }
-  const profile = await loadSourceProfile(profileRef, { profileDirs: readCsv(options.get("profile-dirs")) });
-  const imported = await importSource({ profile, sourceRoot });
-  if (options.get("merge") === "true") {
-    return await mergeImport(options, imported, stdout);
-  }
-  const fragment = renderImportFragment(imported);
-  const out = options.get("out");
-  if (out === undefined) {
-    stdout.write(fragment);
-    return 0;
-  }
-  await mkdir(dirname(out), { recursive: true });
-  await writeFile(out, fragment, "utf8");
-  stdout.write(`Import fragment written: ${out}\n`);
-  return 0;
-}
-
+  const profileRef = options.get("profile");
+  const sourceRoot = options.get("source-root");
+  if (profileRef === undefined || sourceRoot === undefined) {
+    throw new Error("Usage: skillboard import --profile <id-or-path> --source-root <dir>");
+  }
+  const profile = await loadSourceProfile(profileRef, { profileDirs: readCsv(options.get("profile-dirs")) });
+  const imported = await importSource({ profile, sourceRoot });
+  if (options.get("merge") === "true") {
+    return await mergeImport(options, imported, stdout);
+  }
+  const fragment = renderImportFragment(imported);
+  const out = options.get("out");
+  if (out === undefined) {
+    stdout.write(fragment);
+    return 0;
+  }
+  await mkdir(dirname(out), { recursive: true });
+  await writeFile(out, fragment, "utf8");
+  stdout.write(`Import fragment written: ${out}\n`);
+  return 0;
+}
+
 async function mergeImport(options, imported, stdout) {
   const path = configPath(options);
   const originalText = await readFile(path, "utf8");
@@ -337,21 +337,21 @@ async function sources(argv, options, stdout) {
   writeOutput(stdout, result, options, () => renderSourceRefresh(result));
   return 0;
 }
-
+
 async function scan(options, stdout) {
-  const workspace = await loadWorkspace({ configPath: configPath(options), skillsRoot: skillsRoot(options) });
-  stdout.write(`${JSON.stringify(workspace, null, 2)}\n`);
-  return 0;
-}
-
+  const workspace = await loadWorkspace({ configPath: configPath(options), skillsRoot: skillsRoot(options) });
+  stdout.write(`${JSON.stringify(workspace, null, 2)}\n`);
+  return 0;
+}
+
 async function check(options, stdout, stderr) {
   const workspace = await loadWorkspace({ configPath: configPath(options), skillsRoot: skillsRoot(options) });
   const result = checkPolicy(workspace);
-  const output = [...result.errors, ...result.warnings].join("\n");
-  if (output.length > 0) {
-    (result.ok ? stdout : stderr).write(`${output}\n`);
-  }
-  stdout.write(result.ok ? "Policy check passed\n" : "Policy check failed\n");
+  const output = [...result.errors, ...result.warnings].join("\n");
+  if (output.length > 0) {
+    (result.ok ? stdout : stderr).write(`${output}\n`);
+  }
+  stdout.write(result.ok ? "Policy check passed\n" : "Policy check failed\n");
   return result.ok ? 0 : 1;
 }
 
@@ -983,37 +983,37 @@ async function remove(argv, options, stdout) {
 }
 
 async function dashboard(options, stdout) {
-  const workspace = await loadWorkspace({ configPath: configPath(options), skillsRoot: skillsRoot(options) });
-  const markdown = renderDashboard(workspace);
-  const out = options.get("out");
-  if (out === undefined) {
-    stdout.write(markdown);
-    return 0;
-  }
-  await mkdir(dirname(out), { recursive: true });
-  await writeFile(out, markdown, "utf8");
-  stdout.write(`Dashboard written: ${out}\n`);
-  return 0;
-}
-
-async function reconcile(options, stdout) {
-  const workspace = await loadWorkspace({ configPath: configPath(options), skillsRoot: skillsRoot(options) });
-  const plan = reconcileWorkspace(workspace, { actualHarnesses: readCsv(options.get("actual-harnesses")) });
-  const markdown = renderReconcilePlan(plan);
-  const out = options.get("out");
-  if (out === undefined) {
-    stdout.write(markdown);
-    return 0;
-  }
-  await mkdir(dirname(out), { recursive: true });
-  await writeFile(out, markdown, "utf8");
-  stdout.write(`Reconcile plan written: ${out}\n`);
-  return 0;
-}
-
-async function impact(argv, options, stdout) {
-  if (argv[0] !== "disable" || argv[1] === undefined) {
-    throw new Error("Usage: skillboard impact disable <skill-id>");
+  const workspace = await loadWorkspace({ configPath: configPath(options), skillsRoot: skillsRoot(options) });
+  const markdown = renderDashboard(workspace);
+  const out = options.get("out");
+  if (out === undefined) {
+    stdout.write(markdown);
+    return 0;
+  }
+  await mkdir(dirname(out), { recursive: true });
+  await writeFile(out, markdown, "utf8");
+  stdout.write(`Dashboard written: ${out}\n`);
+  return 0;
+}
+
+async function reconcile(options, stdout) {
+  const workspace = await loadWorkspace({ configPath: configPath(options), skillsRoot: skillsRoot(options) });
+  const plan = reconcileWorkspace(workspace, { actualHarnesses: readCsv(options.get("actual-harnesses")) });
+  const markdown = renderReconcilePlan(plan);
+  const out = options.get("out");
+  if (out === undefined) {
+    stdout.write(markdown);
+    return 0;
+  }
+  await mkdir(dirname(out), { recursive: true });
+  await writeFile(out, markdown, "utf8");
+  stdout.write(`Reconcile plan written: ${out}\n`);
+  return 0;
+}
+
+async function impact(argv, options, stdout) {
+  if (argv[0] !== "disable" || argv[1] === undefined) {
+    throw new Error("Usage: skillboard impact disable <skill-id>");
   }
   const workspace = await loadWorkspace({ configPath: configPath(options), skillsRoot: skillsRoot(options) });
   const result = impactDisable(workspace, argv[1]);
@@ -1023,22 +1023,22 @@ async function impact(argv, options, stdout) {
   }
   const markdown = renderImpact(result);
   const out = options.get("out");
-  if (out === undefined) {
-    stdout.write(markdown);
-    return 0;
-  }
-  await mkdir(dirname(out), { recursive: true });
-  await writeFile(out, markdown, "utf8");
-  stdout.write(`Impact report written: ${out}\n`);
-  return 0;
-}
-
+  if (out === undefined) {
+    stdout.write(markdown);
+    return 0;
+  }
+  await mkdir(dirname(out), { recursive: true });
+  await writeFile(out, markdown, "utf8");
+  stdout.write(`Impact report written: ${out}\n`);
+  return 0;
+}
+
 function renderImpact(result) {
-  return [
-    `# Disable Impact: ${result.skillId}`,
-    "",
-    `- Found: \`${result.exists}\``,
-    `- Risk: ${result.risk}`,
+  return [
+    `# Disable Impact: ${result.skillId}`,
+    "",
+    `- Found: \`${result.exists}\``,
+    `- Risk: ${result.risk}`,
     `- Affected workflows: ${formatList(result.affectedWorkflows)}`,
     `- Affected required outputs: ${formatList(result.affectedOutputs)}`,
     `- Alternatives: ${formatList(result.alternatives)}`,
@@ -1075,24 +1075,24 @@ function commandRoot(options) {
 }
 
 function parseOptions(args) {
-  const options = new Map();
-  for (let index = 0; index < args.length; index += 1) {
-    const arg = args[index];
-    if (!arg.startsWith("--")) {
-      continue;
-    }
-    const key = arg.slice(2);
-    const value = args[index + 1];
-    if (value === undefined || value.startsWith("--")) {
-      options.set(key, "true");
-      continue;
-    }
-    options.set(key, value);
-    index += 1;
-  }
-  return options;
-}
-
+  const options = new Map();
+  for (let index = 0; index < args.length; index += 1) {
+    const arg = args[index];
+    if (!arg.startsWith("--")) {
+      continue;
+    }
+    const key = arg.slice(2);
+    const value = args[index + 1];
+    if (value === undefined || value.startsWith("--")) {
+      options.set(key, "true");
+      continue;
+    }
+    options.set(key, value);
+    index += 1;
+  }
+  return options;
+}
+
 function formatList(values) {
   return values.length === 0 ? "none" : values.map((value) => `\`${value}\``).join(", ");
 }
@@ -1105,7 +1105,7 @@ function readCsv(value) {
   if (value === undefined || value.trim() === "") {
     return [];
   }
-  return value.split(",").map((item) => item.trim()).filter((item) => item.length > 0);
+  return value.split(",").map((item) => item.trim()).filter((item) => item.length > 0);
 }
 
 function renderImportMerge(result) {
@@ -1516,7 +1516,6 @@ function helpText() {
     "AI/automation operations:",
     "  setup [--yes] [--agent codex[,claude,opencode,hermes]]",
     "  import-skill --from <agent> --to <agent> --skill <id-or-dir> [--target-skill <id-or-dir>] [--adapted-file <path>] [--dry-run] [--yes] [--replace] [--json]",
-    "  init [--dir <path>] [--scan-root <dir>[,<dir>]] [--no-scan-installed]",
     "  uninstall [--dir <path>] [--dry-run] [--keep-settings] [--purge] [--remove-config|--reset-config] [--remove-reports] [--remove-hooks] [--keep-empty-dirs] [--agent-layer] [--agent codex[,claude,opencode,hermes]]",
     "  inventory refresh [--dir <path>] [--config <path>] [--scan-root <dir>[,<dir>]] [--dry-run] [--json]",
     "  inventory detect --unit <id> --config <path> [--install-output <path>] [--config-file a,b] [--source <value>] [--kind <kind>] [--scope <scope>] [--dry-run] [--json]",
@@ -1556,6 +1555,10 @@ function helpText() {
     "  reconcile --config <path> --skills <dir> [--actual-harnesses a,b] [--out <path>]",
     "  impact disable <skill-id> --config <path> --skills <dir> [--out <path>] [--json]",
     "",
+    "Legacy project policy mode:",
+    "  init [--dir <path>] [--scan-root <dir>[,<dir>]] [--no-scan-installed]",
+    "  Deprecated project-local policy bootstrap; not needed for normal use.",
+    "",
     "AI/automation control loop:",
     "  Goal: keep skills broadly available while routing overlaps consistently; see docs/ai-skill-routing-goal.md.",
     "  Development loop: observe → route → work → explain briefly → ask after → remember policy.",
@@ -1618,8 +1621,9 @@ function initHelpText() {
   return [
     "Usage: skillboard init [--dir <path>] [--scan-root <dir>[,<dir>]] [--no-scan-installed]",
     "",
-    "Creates the local SkillBoard control files for a project.",
-    "Use it once per project before asking the AI which skills it can use.",
+    "Deprecated project-local policy bootstrap. This command is not needed for normal use.",
+    "Normal flow: install agent-skillboard globally, let postinstall/setup refresh agent-layer guidance, then ask your AI normal work requests.",
+    "Use init only for an existing workspace that intentionally maintains local SkillBoard policy files.",
     "",
     "Options:",
     "  --dir <path>              Project root to initialize; defaults to the current directory.",
@@ -1631,7 +1635,7 @@ function initHelpText() {
     "  Imports trusted user-local skills as on-request skills.",
     "  Keeps runtime, plugin, system, and external skills behind source review before manual activation.",
     "",
-    "Next:",
+    "Legacy next steps:",
     "  Run skillboard doctor --summary.",
     "  Ask your AI: \"What skills can you use in this project?\"",
     ""
@@ -1645,7 +1649,7 @@ function setupHelpText() {
     "Installs or refreshes SkillBoard at the agent layer, not into a project.",
     "A normal global package install already runs this automatically for detected supported agents.",
     "Use setup later after adding another supported agent, enabling a new agent home, or skipping install scripts.",
-    "You do not need skillboard init for this install-time setup; init is only for a workspace where you want project-local policy files.",
+    "skillboard init is deprecated project-local policy bootstrap and is not needed for normal use.",
     "Without --yes, setup explains the user agent skill files it will write and asks before installing when run in a TTY.",
     "In non-interactive automation, rerun with --yes after choosing the target agents.",
     "",
@@ -1736,11 +1740,11 @@ function doctorHelpText() {
   return [
     "Usage: skillboard doctor [--dir <path>] [--config <path>] [--skills <dir>] [--verify] [--strict] [--summary] [--json]",
     "",
-    "Checks whether a SkillBoard project is ready to use.",
+    "Checks legacy local policy workspace health and source-review readiness.",
     "The status command is an alias for doctor.",
     "",
     "Options:",
-    "  --dir <path>       Project root to check; defaults to the current directory.",
+    "  --dir <path>       Workspace root to check; defaults to the current directory.",
     "  --config <path>    Use a specific skillboard.config.yaml.",
     "  --skills <dir>     Use a specific skills directory.",
     "  --verify           Verify local source/cache digests when available.",
@@ -1748,7 +1752,8 @@ function doctorHelpText() {
     "  --summary          Print a short human summary.",
     "  --json             Print an agent-readable payload.",
     "",
-    "Use this after init and before trusting new skill sources.",
+    "Use this for existing workspaces that intentionally keep local SkillBoard policy files.",
+    "It is read-only and is not part of agent-layer setup.",
     ""
   ].join("\n");
 }

package/src/doctor.mjs

@@ -121,10 +121,10 @@ function finalizeDoctor(result, recommendations) {
 function doctorRecommendations(result) {
   const recommendations = [];
   if (!result.bridges.some((bridge) => bridge.status === "installed")) {
-    recommendations.push("run skillboard init to install agent bridge blocks");
+    recommendations.push("legacy project bridge blocks are absent; run skillboard init only if maintaining deprecated project-local policy");
   }
   if (result.bridges.some((bridge) => bridge.status === "unmanaged")) {
-    recommendations.push("run skillboard init to add SkillBoard bridge blocks to unmanaged agent files");
+    recommendations.push("legacy project bridge is unmanaged; run skillboard init only if maintaining deprecated project-local policy");
   }
   if (result.bridges.some((bridge) => bridge.status === "broken")) {
     recommendations.push("repair AGENTS.md or CLAUDE.md SkillBoard bridge markers");