@getnoan/wizard 0.1.4 → 0.1.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@getnoan/wizard",
-  "version": "0.1.4",
+  "version": "0.1.5",
   "description": "One command from nothing to a grounded agent: sign in to NOAN, wire your coding assistant, install the skill, and set up the agent pack.",
   "license": "MIT",
   "type": "module",

package/src/agents.mjs

@@ -6,7 +6,7 @@
  * the pack's documented ones. Safe mode stays on: the wizard never sets DRY_RUN to 0.
  */
 import { spawnSync } from "node:child_process";
-import { existsSync } from "node:fs";
+import { existsSync, readFileSync } from "node:fs";
 import path from "node:path";
 import { randomBytes } from "node:crypto";
 import { writeEnv } from "./env-file.mjs";
@@ -128,15 +128,36 @@ export function setVariable(dest, name, value, dryRun) {
   return { name, action: r.status === 0 ? "set" : "failed", error: r.status === 0 ? undefined : (r.stderr || "").trim().split("\n")[0] };
 }
 
+/** Where a pack file lives in the clone, named as the pack names it (`seed-deck.mjs`).
+ *
+ *  The pack groups agents/ by agent — agents/<agent>/ for one agent's files, agents/shared/
+ *  for the rest — and writes agents/pack-layout.json mapping each name to its path. A pack
+ *  cloned before that grouping has neither the folders nor the map, and every file sits
+ *  directly in agents/. Both are live at once (a fork keeps whatever it was cloned from), so
+ *  both are read.
+ *
+ *  A layout file that exists but cannot be read THROWS. Falling back to the flat path would
+ *  report every seed "missing" and seed nothing — the silent failure the map exists to stop. */
+export function packFile(dest, name) {
+  const layoutPath = path.join(dest, "agents", "pack-layout.json");
+  if (existsSync(layoutPath)) {
+    let files;
+    try { files = JSON.parse(readFileSync(layoutPath, "utf8")).files; }
+    catch (e) { throw new Error(`agents/pack-layout.json in ${dest} is not readable JSON: ${e.message}`); }
+    if (files?.[name]) return path.join(dest, "agents", files[name]);
+  }
+  return path.join(dest, "agents", name);
+}
+
 /** Run every seed script the pack ships; collect the block slugs they print. */
 export function runSeeds(dest, env, { dryRun = false } = {}) {
   const seeds = ["seed-weekly-activity-report.mjs", "seed-fact-alignment.mjs", "seed-market-research-refresh.mjs", "seed-customer-support.mjs", "seed-deck.mjs"];
   const slugs = {}; const rows = [];
   for (const s of seeds) {
-    const p = path.join(dest, "agents", s);
+    const p = packFile(dest, s);
     if (!existsSync(p)) { rows.push({ seed: s, action: "missing" }); continue; }
     if (dryRun) { rows.push({ seed: s, action: "would run" }); continue; }
-    const r = spawnSync(process.execPath, [p], { cwd: path.join(dest, "agents"), encoding: "utf8", env: { ...process.env, ...env } });
+    const r = spawnSync(process.execPath, [p], { cwd: path.dirname(p), encoding: "utf8", env: { ...process.env, ...env } });
     const found = parseSeedOutput((r.stdout || "") + "\n" + (r.stderr || ""));
     Object.assign(slugs, found);
     rows.push({ seed: s, action: r.status === 0 ? "ran" : "failed", slugs: Object.keys(found).length, error: r.status === 0 ? undefined : (r.stderr || r.stdout || "").trim().split("\n").slice(-1)[0] });
@@ -146,9 +167,9 @@ export function runSeeds(dest, env, { dryRun = false } = {}) {
 
 /** The pack's grounding check: which blocks the agents read hold no fact. Runs in the clone; absent in an old pack. */
 export function runGroundingCheck(dest, env, { dryRun = false } = {}) {
-  const p = path.join(dest, "agents", "grounding-check.mjs");
+  const p = packFile(dest, "grounding-check.mjs");
   if (!existsSync(p)) return { available: false };
-  const r = spawnSync(process.execPath, [p, "--json", ...(dryRun ? ["--dry-run"] : [])], { cwd: path.join(dest, "agents"), encoding: "utf8", env: { ...process.env, ...env } });
+  const r = spawnSync(process.execPath, [p, "--json", ...(dryRun ? ["--dry-run"] : [])], { cwd: path.dirname(p), encoding: "utf8", env: { ...process.env, ...env } });
   return { available: true, ...parseGroundingOutput(r.stdout), error: r.status === 0 ? undefined : (r.stderr || "").trim().split("\n").slice(-1)[0] };
 }
 export function parseGroundingOutput(text) {

package/src/run.mjs

@@ -168,7 +168,8 @@ async function setupAgents({ args, ui, dir, key, me, report }) {
   pack.writePackEnv(fork.dest, { NOAN_PERSONAL_API_KEY: key, ...(anthropic && { [mName]: anthropic }), ...(mBase && { ANTHROPIC_BASE_URL: mBase }), ...(resend && { RESEND_API_KEY: resend }) }, args.dryRun);
   const seeds = pack.runSeeds(fork.dest, { NOAN_PERSONAL_API_KEY: key }, { dryRun: args.dryRun });
   out.seeds = seeds.rows;
-  for (const r of seeds.rows) ui.ok(`${r.seed}: ${r.action}${r.slugs != null ? ` (${r.slugs} slug(s))` : ""}${r.error ? ` — ${r.error}` : ""}`);
+  // A missing or failed seed is a warning, not a tick: its agent has no Config fact and no slug variable.
+  for (const r of seeds.rows) (/^(missing|failed)$/.test(r.action) ? ui.warn : ui.ok)(`${r.seed}: ${r.action}${r.slugs != null ? ` (${r.slugs} slug(s))` : ""}${r.error ? ` — ${r.error}` : ""}`);
   Object.assign(vars, seeds.slugs);
   // The blocks the agents read that hold no fact: one task each on the user's board, and a line in the report.
   const grounding = args.dryRun ? { available: false } : pack.runGroundingCheck(fork.dest, { NOAN_PERSONAL_API_KEY: key });