slidemind 0.2.2 → 0.2.4

package/src/installers/lib.mjs

@@ -113,7 +113,7 @@ const CURSOR_SHOW = "\x1b[?25h";
 // uniform.
 function startWordmarkAnimation() {
   const lines = wordmarkFrame(0);
-  process.stdout.write("\n" + lines.join("\n") + "\n\n");
+  tui.out("\n" + lines.join("\n") + "\n\n");
   return () => {};
 }
 
@@ -125,6 +125,30 @@ export function isForce(flags) {
   return flags.includes("--force") || flags.includes("-F");
 }
 
+// Is `name` an executable on the user's PATH? Used as the second half of
+// harness detection: a coding agent's config dir (~/.claude, ~/.codex,
+// ~/.config/opencode) is created by USE, not by install, so a freshly installed
+// agent that has never been launched has no directory at all — its binary is
+// the only evidence it exists. Resolved by walking PATH rather than shelling
+// out to which/where, so it costs nothing and behaves the same on Windows.
+export function binaryOnPath(name) {
+  const exts =
+    process.platform === "win32"
+      ? (process.env.PATHEXT || ".COM;.EXE;.BAT;.CMD").split(";")
+      : [""];
+  for (const dir of (process.env.PATH || "").split(path.delimiter)) {
+    if (!dir) continue;
+    for (const ext of exts) {
+      try {
+        if (fs.statSync(path.join(dir, name + ext)).isFile()) return true;
+      } catch {
+        /* not there / unreadable — keep looking */
+      }
+    }
+  }
+  return false;
+}
+
 export function expandHome(p) {
   if (p.startsWith("~/")) return path.join(homeDir(), p.slice(2));
   return p;
@@ -234,6 +258,12 @@ let _startedAt = 0;
 let _stopWordmark = null;
 
 export const ui = {
+  // Suppress ALL decorative output. `slidemind setup --json` runs the real
+  // installers but owns stdout for its machine-readable payload, so the rail
+  // would corrupt the very thing the caller is parsing.
+  silence(value = true) {
+    tui.setSilent(value);
+  },
   // `wordmark: false` skips painting the wordmark — used when the caller
   // (e.g. the install.sh bootstrap) already printed it, so we only emit the
   // `v… · action` intro line and continue the rail.
@@ -276,12 +306,12 @@ export const ui = {
     // Manual emission so we can animate the title in place. Symbol
     // sits at col 1; "  " padding lands the title at col 4.
     tui.bumpRow(1);
-    process.stdout.write(tui.rail() + "\n");
+    tui.out(tui.rail() + "\n");
     tui.bumpRow(1);
-    process.stdout.write(SYM.step + "  ");
+    tui.out(SYM.step + "  ");
     await tui.shimmer(title, { startCol: 4, restColor: (s) => s });
-    if (sub) process.stdout.write("  " + brand.slate(tildify(sub)));
-    process.stdout.write("\n");
+    if (sub) tui.out("  " + brand.slate(tildify(sub)));
+    tui.out("\n");
   },
   step({ status = "ok", label, value, note }) {
     const symbol =
@@ -328,15 +358,15 @@ export const ui = {
 
     const boxLine = (content) => {
       const pad = Math.max(0, inner - visibleLength(content));
-      process.stdout.write(slate("│") + content + " ".repeat(pad) + slate("│") + "\n");
+      tui.out(slate("│") + content + " ".repeat(pad) + slate("│") + "\n");
     };
 
-    process.stdout.write("\n");
+    tui.out("\n");
 
     // Top: ╭─ installed ────╮
     const labelText = " installed ";
     const fillerLen = Math.max(2, inner - 1 - labelText.length);
-    process.stdout.write(
+    tui.out(
       slate("╭─") +
         dim(labelText) +
         slate("─".repeat(fillerLen)) +
@@ -359,10 +389,10 @@ export const ui = {
     boxLine("  " + dim(stats));
 
     // Bottom: ╰────────╯
-    process.stdout.write(slate("╰" + "─".repeat(inner) + "╯") + "\n");
+    tui.out(slate("╰" + "─".repeat(inner) + "╯") + "\n");
 
-    process.stdout.write("\n");
-    process.stdout.write(`${SYM.success}  ${brand.bold(action)}\n`);
+    tui.out("\n");
+    tui.out(`${SYM.success}  ${brand.bold(action)}\n`);
 
     // Stop the wordmark animation last — that resets the scroll region
     // and shows the cursor. The wordmark itself stays painted at the
@@ -401,16 +431,16 @@ export const ui = {
     // Reserve two rows: rail spacer + the active spinner line. Both
     // count toward the gradient row counter.
     tui.bumpRow(1);
-    process.stdout.write(tui.rail() + "\n");
+    tui.out(tui.rail() + "\n");
     tui.bumpRow(1);
-    process.stdout.write(
+    tui.out(
       `${brand.techBlue40(SPINNER_FRAMES[0])}  ${fmtStep({ label, value, note })}`,
     );
     let frame = 0;
     const timer = setInterval(() => {
       frame = (frame + 1) % SPINNER_FRAMES.length;
-      process.stdout.write("\r\x1b[2K");
-      process.stdout.write(
+      tui.out("\r\x1b[2K");
+      tui.out(
         `${brand.techBlue40(SPINNER_FRAMES[frame])}  ${fmtStep({ label, value, note })}`,
       );
     }, 80);
@@ -418,15 +448,15 @@ export const ui = {
       const result = await fn();
       const time = Date.now() - start;
       clearInterval(timer);
-      process.stdout.write("\r\x1b[2K");
-      process.stdout.write(
+      tui.out("\r\x1b[2K");
+      tui.out(
         `${SYM.success}  ${fmtStep({ label, value, note: finalNote ?? note, time })}\n`,
       );
       return result;
     } catch (err) {
       clearInterval(timer);
-      process.stdout.write("\r\x1b[2K");
-      process.stdout.write(
+      tui.out("\r\x1b[2K");
+      tui.out(
         `${SYM.error}  ${fmtStep({ label, value, note: "failed" })}\n`,
       );
       throw err;

package/src/installers/opencode.mjs

@@ -17,6 +17,7 @@
 import fs from "node:fs";
 import path from "node:path";
 import {
+  binaryOnPath,
   expandHome,
   installDir,
   isForce,
@@ -117,6 +118,21 @@ function removeLegacyArtifacts() {
   }
 }
 
+// See claude.mjs `detect()`. opencode has no slidemind plugin channel, so the
+// skills can only ever come from here.
+export function detect() {
+  const root = opencodeRoot();
+  const configured = fs.existsSync(root);
+  const onPath = binaryOnPath("opencode");
+  return {
+    root,
+    configured,
+    onPath,
+    present: configured || onPath,
+    pluginInstalled: false,
+  };
+}
+
 export async function install({ packageRoot, flags }) {
   const link = isLinkMode(flags);
   const force = isForce(flags);

package/src/installers/tui.mjs

@@ -23,6 +23,17 @@ const FADE_ROWS = 28; // rail finishes fading after this many emitted rows
 
 let _row = 0;
 
+// Every terminal write in the installer UI funnels through here, so a caller
+// whose stdout carries machine-readable output (`slidemind setup --json`) can
+// silence the decorative rail without each writer knowing about it.
+let _silent = false;
+export function setSilent(value) {
+  _silent = Boolean(value);
+}
+export function out(text) {
+  if (!_silent) process.stdout.write(text);
+}
+
 // Linear interpolate between #9999FF (techBlue40) and #6C7D8C (slate).
 // Top of the rail is bright tech-blue, fading to brand slate as the
 // install progresses — feels like the deck's tech-blue gradation card.
@@ -52,7 +63,7 @@ export function bumpRow(n = 1) {
 
 export function intro(text) {
   _row = 0;
-  process.stdout.write("\n" + paint(_row, "┌") + "  " + text + "\n");
+  out("\n" + paint(_row, "┌") + "  " + text + "\n");
 }
 
 // Each message starts with its own gradient-coloured rail spacer, then
@@ -60,28 +71,28 @@ export function intro(text) {
 // (◆ ● ▲ ■ etc.); when omitted we fall back to a continuation rail.
 export function message(text, { symbol } = {}) {
   _row += 1;
-  process.stdout.write(paint(_row, "│") + "\n");
+  out(paint(_row, "│") + "\n");
   _row += 1;
   const sym = symbol ?? paint(_row, "│");
-  process.stdout.write(sym + "  " + text + "\n");
+  out(sym + "  " + text + "\n");
 }
 
 export function outro(text) {
   _row += 1;
-  process.stdout.write(paint(_row, "│") + "\n");
+  out(paint(_row, "│") + "\n");
   _row += 1;
-  process.stdout.write(paint(_row, "└") + "  " + text + "\n\n");
+  out(paint(_row, "└") + "  " + text + "\n\n");
   _row = 0;
 }
 
 // Reserve the next two rows for a spin step: emits the rail spacer,
 // returns the row index the spinner is written on. Callers should
-// `process.stdout.write(...)` the spinner content themselves (without a
+// `out(...)` the spinner content themselves (without a
 // trailing `\n`) so they can overwrite it via `\r`. When the operation
 // completes they call `commitSpinRow()` after writing the final line.
 export function reserveSpinRows() {
   _row += 1;
-  process.stdout.write(paint(_row, "│") + "\n");
+  out(paint(_row, "│") + "\n");
   _row += 1;
   return _row;
 }
@@ -96,7 +107,7 @@ const ANIMATE = !NO_COLOR && process.stdout.isTTY;
 export async function shimmer(text, { startCol = 1, restColor }) {
   const final = brand.bold(restColor(text));
   if (!ANIMATE) {
-    process.stdout.write(final);
+    out(final);
     return;
   }
   const len = text.length;
@@ -117,9 +128,9 @@ export async function shimmer(text, { startCol = 1, restColor }) {
       else painter = (s) => brand.dim(brand.slate(s));
       line += brand.bold(painter(text[i]));
     }
-    process.stdout.write(`\x1b[${startCol}G\x1b[K` + line);
+    out(`\x1b[${startCol}G\x1b[K` + line);
     if (f < frames) await new Promise((r) => setTimeout(r, frameMs));
   }
   // Final settle.
-  process.stdout.write(`\x1b[${startCol}G\x1b[K` + final);
+  out(`\x1b[${startCol}G\x1b[K` + final);
 }

package/src/pptx/fill/engine.mjs

@@ -35,69 +35,11 @@ export async function buildFillTemplateDeck({ deckDir, skill, outPath, onLog })
   const locale = skill.locale || "en";
   const fieldsByNo = pack.fields[locale] ?? {};
   const repeatsByNo = pack.repeats[locale] ?? {};
-
-  if (!skill.templatePath || !fs.existsSync(skill.templatePath)) {
-    throw new Error(
-      `template "${skill.templateKey}" not found in pack ${skill.skillDir} — check plan.md frontmatter and skill.json templates`,
-    );
-  }
-
-  // ---- deck reading ---------------------------------------------------------
-  const slidesDir = path.join(deckDir, "slides");
-  const entries = fs.readdirSync(slidesDir)
-    .filter((f) => f.endsWith(".jsx") || f.endsWith(".pptx"))
-    .sort((a, b) => a.localeCompare(b, "en", { numeric: true }));
-  if (entries.length === 0) throw new Error(`no .jsx or .pptx slides in ${slidesDir}`);
-
-  const problems = [];
-  const warnings = [];
-  const sequence = []; // { kind: "master", no, props, file } | { kind: "insert", file, buf }
-
-  // In-deck freeform pairing: NN-name.jsx (create-slides source) renders to a
-  // sibling NN-name.pptx artifact — when both exist, the .pptx carries the
-  // slide and the .jsx is source only.
-  const renderedStems = new Set(
-    entries.filter((f) => f.endsWith(".pptx")).map((f) => f.replace(/\.pptx$/, "")),
-  );
-
-  for (const file of entries) {
-    if (file.endsWith(".pptx")) {
-      sequence.push({ kind: "insert", file, buf: fs.readFileSync(path.join(slidesDir, file)) });
-      continue;
-    }
-    if (renderedStems.has(file.replace(/\.jsx$/, ""))) continue;
-    const src = fs.readFileSync(path.join(slidesDir, file), "utf8");
-    let parsed;
-    try {
-      parsed = parseSlideFile(src, file);
-    } catch (err) {
-      problems.push(err.message);
-      continue;
-    }
-    const layout = layoutNumber(parsed.component, locale, pack.components);
-    if (!layout) {
-      problems.push(`${file}: <${parsed.component}> is a freeform (create-slides) slide with no ` +
-        `rendered artifact next to it — 'slidemind build-pptx' renders slides/${file.replace(/\.jsx$/, "")}.pptx ` +
-        `automatically when a paired add-in backend is reachable`);
-      continue;
-    }
-    if (layout.warn) warnings.push(`${file}: ${layout.warn}`);
-    const no = String(layout.no);
-
-    if ((pack.policy.freeform ?? []).includes(no)) {
-      if (Object.keys(parsed.props).length > 0) {
-        problems.push(`${file}: slide ${no} is a freeform placeholder — it takes no props. ` +
-          `Keep it bare to keep the master placeholder, or delete the file to drop the slide.`);
-        continue;
-      }
-      warnings.push(`${file}: keeping the master's slide-${no} placeholder (freeform replacement pending)`);
-    }
-
-    validateProps(parsed.props, fieldsByNo[no] ?? {}, repeatsByNo[no] ?? {}, pack.policy, no, file, problems);
-    sequence.push({ kind: "master", no: Number(no), props: parsed.props, file });
-  }
-
-  checkIdentities(sequence, warnings, problems);
+  const { sequence, warnings, problems } = inspectFillTemplateDeck({
+    deckDir,
+    skill,
+    includeBuffers: true,
+  });
   if (problems.length) {
     const err = new Error(`deck has ${problems.length} authoring error(s):\n  - ${problems.join("\n  - ")}`);
     err.problems = problems;
@@ -174,6 +116,113 @@ export async function buildFillTemplateDeck({ deckDir, skill, outPath, onLog })
   return { outPath: finalOut, slideCount: output.length, warnings };
 }
 
+// Parse and structurally validate a fill-template deck without mutating it.
+// External packs use this generic inspection surface to run their own domain
+// checks before invoking build-pptx. Buffers are opt-in because JSON callers
+// must receive a compact, serializable sequence.
+export function inspectFillTemplateDeck({
+  deckDir,
+  skill,
+  allowUnrenderedFreeform = false,
+  includeBuffers = false,
+}) {
+  const pack = loadPack(skill);
+  const locale = skill.locale || "en";
+  const fieldsByNo = pack.fields[locale] ?? {};
+  const repeatsByNo = pack.repeats[locale] ?? {};
+  const slidesDir = path.join(deckDir, "slides");
+  const problems = [];
+  const warnings = [];
+  const sequence = [];
+
+  if (!skill.templatePath || !fs.existsSync(skill.templatePath)) {
+    problems.push(
+      `template "${skill.templateKey}" not found in pack ${skill.skillDir} — check the pack manifest templates`,
+    );
+  }
+  if (!fs.existsSync(slidesDir) || !fs.statSync(slidesDir).isDirectory()) {
+    problems.push(`expected 'slides/' folder in ${deckDir}`);
+    return { sequence, warnings, problems, locale, templateKey: skill.templateKey };
+  }
+
+  const entries = fs.readdirSync(slidesDir)
+    .filter((f) => f.endsWith(".jsx") || f.endsWith(".pptx"))
+    .sort((a, b) => a.localeCompare(b, "en", { numeric: true }));
+  if (entries.length === 0) {
+    problems.push(`no .jsx or .pptx slides in ${slidesDir}`);
+    return { sequence, warnings, problems, locale, templateKey: skill.templateKey };
+  }
+
+  const renderedStems = new Set(
+    entries.filter((f) => f.endsWith(".pptx")).map((f) => f.replace(/\.pptx$/, "")),
+  );
+
+  for (const file of entries) {
+    const full = path.join(slidesDir, file);
+    if (file.endsWith(".pptx")) {
+      sequence.push({
+        kind: "insert",
+        file,
+        ...(includeBuffers ? { buf: fs.readFileSync(full) } : {}),
+      });
+      continue;
+    }
+    if (renderedStems.has(file.replace(/\.jsx$/, ""))) continue;
+
+    const src = fs.readFileSync(full, "utf8");
+    const component = src.match(/<([A-Z][A-Za-z0-9]*)/)?.[1] ?? null;
+    let layout = null;
+    if (component) {
+      try {
+        layout = layoutNumber(component, locale, pack.components);
+      } catch (err) {
+        problems.push(err.message);
+        continue;
+      }
+    }
+
+    // A pack validator needs to see freeform source positions even before the
+    // paired add-in artifact exists. The build path still rejects such a
+    // source until ensureFreeformRenders has produced its .pptx sibling.
+    if (!layout && allowUnrenderedFreeform) {
+      sequence.push({ kind: "freeform", file, component });
+      warnings.push(`${file}: freeform source has no rendered .pptx artifact yet`);
+      continue;
+    }
+
+    let parsed;
+    try {
+      parsed = parseSlideFile(src, file);
+    } catch (err) {
+      problems.push(err.message);
+      continue;
+    }
+    layout = layoutNumber(parsed.component, locale, pack.components);
+    if (!layout) {
+      problems.push(`${file}: <${parsed.component}> is a freeform (create-slides) slide with no ` +
+        `rendered artifact next to it — render slides/${file.replace(/\.jsx$/, "")}.pptx ` +
+        `through the paired add-in before building`);
+      continue;
+    }
+    if (layout.warn) warnings.push(`${file}: ${layout.warn}`);
+    const no = String(layout.no);
+
+    if ((pack.policy.freeform ?? []).includes(no)) {
+      if (Object.keys(parsed.props).length > 0) {
+        problems.push(`${file}: slide ${no} is a freeform placeholder — it takes no props. ` +
+          `Keep it bare to keep the master placeholder, or delete the file to drop the slide.`);
+        continue;
+      }
+      warnings.push(`${file}: keeping the master's slide-${no} placeholder (freeform replacement pending)`);
+    }
+
+    validateProps(parsed.props, fieldsByNo[no] ?? {}, repeatsByNo[no] ?? {}, pack.policy, no, file, problems);
+    sequence.push({ kind: "master", no: Number(no), props: parsed.props, file });
+  }
+
+  return { sequence, warnings, problems, locale, templateKey: skill.templateKey };
+}
+
 // ---------------------------------------------------------------------------
 // Pack
 // ---------------------------------------------------------------------------
@@ -266,77 +315,6 @@ function editDistance(a, b) {
   return dp[a.length][b.length];
 }
 
-// ---------------------------------------------------------------------------
-// Cross-slide identities (the catalog's consistency table, as code)
-// ---------------------------------------------------------------------------
-
-const asLines = (v) => (Array.isArray(v) ? v : String(v ?? "").split("\n")).map((s) => s.trim()).filter(Boolean);
-
-function checkIdentities(sequence, warnings, problems) {
-  const slide = (no) => sequence.filter((s) => s.kind === "master" && s.no === no);
-  const approach = slide(10)[0];
-  const details = slide(11);
-  const gantt = slide(13)[0];
-  const calc = slide(16)[0];
-
-  const wpItems = approach?.props?.repeat?.workPackages ?? null;
-  const ganttItems = gantt?.props?.repeat?.workPackages ?? null;
-
-  if (wpItems && details.length && wpItems.length !== details.length) {
-    problems.push(`WP count mismatch: slide 10 declares ${wpItems.length} work packages but there are ${details.length} WP-detail files (slide 11)`);
-  }
-  // Every WP detail must carry FULL bullet lists — a section left empty (or a
-  // one-liner) is a defect in a client deliverable, not a style choice.
-  details.forEach((d, i) => {
-    for (const section of ["services", "assumptions", "deliverables"]) {
-      const lines = asLines(d.props?.[section]);
-      if (lines.length === 0) {
-        problems.push(`${d.file}: WP detail ${i + 1} has no "${section}" — every WP detail needs all three bullet lists filled`);
-      } else if (lines.length < 3) {
-        warnings.push(`${d.file}: "${section}" has only ${lines.length} bullet(s) — WP details expect full lists (3–5)`);
-      }
-    }
-  });
-  if (wpItems && ganttItems && wpItems.length !== ganttItems.length) {
-    problems.push(`WP count mismatch: slide 10 has ${wpItems.length} work packages, the project plan (13) has ${ganttItems.length} rows`);
-  }
-  if (ganttItems && details.length === ganttItems.length) {
-    ganttItems.forEach((item, i) => {
-      const canonical = asLines(details[i]?.props?.deliverables);
-      const row = asLines(item.deliverable);
-      if (canonical.length && row.length && canonical.join("\n") !== row.join("\n")) {
-        problems.push(`deliverables drift: gantt row ${i + 1} must repeat WP-${i + 1}'s deliverables verbatim (slide 11 owns them)`);
-      }
-    });
-  }
-  if (wpItems && ganttItems) {
-    ganttItems.forEach((item, i) => {
-      if (item.title && wpItems[i]?.title && item.title !== wpItems[i].title) {
-        problems.push(`WP name drift: slide 10 card ${i + 1} is "${wpItems[i].title}" but gantt row ${i + 1} is "${item.title}"`);
-      }
-    });
-  }
-  for (const [frameId, rows] of Object.entries(calc?.props?.tables ?? {})) {
-    if (!Array.isArray(rows) || !wpItems) continue;
-    const labels = rows.slice(1, -1).map((r) => String(r[0] ?? ""));
-    labels.forEach((label, i) => {
-      if (wpItems[i]?.title && label && label !== wpItems[i].title) {
-        problems.push(`WP name drift: calculation table ${frameId} row ${i + 1} is "${label}" but slide 10 card ${i + 1} is "${wpItems[i].title}"`);
-      }
-    });
-  }
-  // Price echo: the contracted total should appear in the management summary.
-  const costRows = calc?.props?.tables?.["15"];
-  if (Array.isArray(costRows)) {
-    const grand = String(costRows[costRows.length - 1]?.slice(-1)[0] ?? "");
-    const digits = grand.replace(/[^\d.,]/g, "");
-    const summary = slide(7)[0];
-    if (digits && summary?.props?.approach && !String(summary.props.approach).includes(digits)) {
-      warnings.push(`slide 7's approach paragraph does not mention the contracted total ${grand} from table 15`);
-    }
-  }
-}
-
 // ---------------------------------------------------------------------------
 // Per-slide ops
 // ---------------------------------------------------------------------------