slidemind 0.2.1 → 0.2.3

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

@@ -1,5 +1,5 @@
 // The fill-template deck builder — the host-free `build-pptx` path for
-// proposal-style packs (mode: "fill-template" in the pack's skill.json).
+// template packs (mode: "fill-template" in the pack's skill.json).
 //
 // Assembly direction: the template .pptx is cloned WHOLE (masters, layouts,
 // media, tags, embeddings), then only declared shapes inside the clone are
@@ -74,7 +74,7 @@ export async function buildFillTemplateDeck({ deckDir, skill, outPath, onLog })
       problems.push(err.message);
       continue;
     }
-    const layout = layoutNumber(parsed.component, locale);
+    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 ` +
@@ -190,6 +190,9 @@ function loadPack(skill) {
     fields,
     repeats: read("repeats.json", {}),
     policy: skill.manifest.policy ?? {},
+    // The pack's layout-component naming contract (see parse-slides.mjs).
+    // skillDir rides along so a bad pattern names the pack that shipped it.
+    components: { ...(skill.manifest.components ?? {}), skillDir: skill.skillDir },
   };
 }
 
@@ -283,7 +286,7 @@ function checkIdentities(sequence, warnings, problems) {
     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 proposal, not a style choice.
+  // 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]);

package/src/pptx/fill/freeform.mjs

@@ -1,12 +1,12 @@
 // In-deck freeform slides for fill-template decks.
 //
-// Contract: a proposal deck's slides/ may carry create-slides freeform JSX
+// Contract: a fill-template deck's slides/ may carry create-slides freeform JSX
 // (NN-name.jsx composing the design bundle's layouts) directly next to the
 // master-backed slides — no sibling deck. The rendered native artifact lives
 // beside its source as NN-name.pptx; the fill engine inserts that artifact at
 // the filename position like any native slide. This module keeps the
 // artifacts fresh: stale or missing renders go through the add-in's render
-// path (the ONLY host-dependent step of a proposal build) and are folded to
+// path (the ONLY host-dependent step of such a build) and are folded to
 // pruned one-slide packages. Unchanged freeform slides rebuild fully offline.
 
 import crypto from "node:crypto";
@@ -22,22 +22,26 @@ import { layoutNumber } from "./parse-slides.mjs";
 
 const SIDE_CAR = ".freeform-renders.json";
 
-// The template's Challenge/Solution slides carry FIXED titles; freeform
-// replacements must keep them so the deck reads like the master.
-const FIXED_TITLES = {
-  en: [
-    { stem: /challenge/i, title: "Challenge" },
-    { stem: /solution/i, title: "Solution" },
-  ],
-  de: [
-    { stem: /challenge|herausforderung/i, title: "Herausforderung" },
-    { stem: /solution|loesung|lösung/i, title: "Lösung" },
-  ],
-};
+// Some templates fix a slide's title (the master prints it, so a freeform
+// replacement must keep it or the deck stops reading like the master). WHICH
+// slides, and in which locale, is the pack's business — it declares them in
+// skill.json as policy.fixedTitles:
+//
+//   "fixedTitles": {
+//     "en": [ { "stem": "agenda", "title": "Agenda" } ],
+//     "de": [ { "stem": "agenda|ablauf", "title": "Ablauf" } ]
+//   }
+//
+// `stem` is a case-insensitive regex matched against the slide file's stem.
+// A pack that declares none simply gets no title warnings.
+function fixedTitleRules(policy, locale) {
+  const all = policy?.fixedTitles ?? {};
+  return all[locale] ?? all.en ?? [];
+}
 
-function isFreeformSource(src) {
+function isFreeformSource(src, components) {
   const comp = src.match(/<([A-Z][A-Za-z0-9]*)/)?.[1];
-  return Boolean(comp) && layoutNumber(comp, "en") === null;
+  return Boolean(comp) && layoutNumber(comp, "en", components) === null;
 }
 
 function contentHash(parts) {
@@ -53,9 +57,10 @@ function slideText(bytes) {
   return [...(xml ?? "").matchAll(/<a:t>([^<]*)<\/a:t>/g)].map((m) => m[1]).join("\n");
 }
 
-function fixedTitleWarning(stem, bytes, locale) {
-  const rules = FIXED_TITLES[locale] ?? FIXED_TITLES.en;
-  const rule = rules.find((r) => r.stem.test(stem));
+function fixedTitleWarning(stem, bytes, locale, policy) {
+  const rule = fixedTitleRules(policy, locale).find((r) =>
+    new RegExp(r.stem, "i").test(stem),
+  );
   if (!rule) return null;
   if (slideText(bytes).includes(rule.title)) return null;
   return (
@@ -69,7 +74,14 @@ function fixedTitleWarning(stem, bytes, locale) {
  * `{ rendered, warnings }`; throws with the offending file named when a stale
  * render is needed but the paired backend cannot render.
  */
-export async function ensureFreeformRenders({ deckDir, locale = "en", srcDir, onLog = () => {} }) {
+export async function ensureFreeformRenders({
+  deckDir,
+  locale = "en",
+  srcDir,
+  components,
+  policy = {},
+  onLog = () => {},
+}) {
   const slidesDir = path.join(deckDir, "slides");
   const jsxFiles = fs.existsSync(slidesDir)
     ? fs.readdirSync(slidesDir).filter((f) => f.endsWith(".jsx")).sort()
@@ -77,7 +89,9 @@ export async function ensureFreeformRenders({ deckDir, locale = "en", srcDir, on
   const items = [];
   for (const file of jsxFiles) {
     const src = fs.readFileSync(path.join(slidesDir, file), "utf8");
-    if (isFreeformSource(src)) items.push({ file, stem: file.replace(/\.jsx$/, ""), src });
+    if (isFreeformSource(src, components)) {
+      items.push({ file, stem: file.replace(/\.jsx$/, ""), src });
+    }
   }
   const warnings = [];
   if (items.length === 0) return { rendered: 0, warnings };
@@ -149,7 +163,7 @@ export async function ensureFreeformRenders({ deckDir, locale = "en", srcDir, on
   for (const it of items) {
     const target = path.join(slidesDir, `${it.stem}.pptx`);
     if (!fs.existsSync(target)) continue;
-    const warn = fixedTitleWarning(it.stem, fs.readFileSync(target), locale);
+    const warn = fixedTitleWarning(it.stem, fs.readFileSync(target), locale, policy);
     if (warn) warnings.push(warn);
   }
   return { rendered: stale.length, warnings };

package/src/pptx/fill/parse-slides.mjs

@@ -1,27 +1,61 @@
-// Parser for proposal slide files. Proposal JSX is declarative by contract —
-// one default export returning exactly one <SlideProposalLayoutNN …/> element
-// whose props are data literals. That constraint is what makes a real JSX
-// runtime unnecessary: attributes are scanned textually and brace expressions
-// are evaluated as plain JS literals in an empty vm sandbox.
+// Parser for fill-template slide files. Their JSX is declarative by contract —
+// one default export returning exactly one <SomeLayout …/> element whose props
+// are data literals. That constraint is what makes a real JSX runtime
+// unnecessary: attributes are scanned textually and brace expressions are
+// evaluated as plain JS literals in an empty vm sandbox.
+//
+// The component NAMES are the pack's business, not this engine's: a pack ships
+// its own layouts under its own naming scheme, so it declares the scheme in
+// skill.json and this file only applies it.
 
 import vm from "node:vm";
 
-// Accepted component spellings → template slide number.
-//   SlideProposalLayout07           (canonical, locale-neutral)
-//   SlideProposalEn07Slide7 / SlideProposalDe07Slide7 (generated concrete names)
-export function layoutNumber(component, locale) {
-  let m = component.match(/^SlideProposalLayout(\d{2})$/);
+// Compiled once per distinct pattern — layoutNumber runs per slide file.
+const patternCache = new Map();
+function compile(pattern, which, skillDir) {
+  let re = patternCache.get(pattern);
+  if (re) return re;
+  try {
+    re = new RegExp(pattern);
+  } catch (err) {
+    throw new Error(
+      `pack ${skillDir ?? "(unknown)"}: skill.json components.${which} is not a ` +
+        `valid regular expression (${err.message})`,
+    );
+  }
+  patternCache.set(pattern, re);
+  return re;
+}
+
+// Map a component name → the template slide number it fills, or null when the
+// component is not master-backed at all (i.e. a freeform slide).
+//
+// `components` comes from the pack's skill.json and declares two patterns:
+//   canonical  — locale-neutral spelling; capture group 1 is the slide number
+//   localized  — optional generated spelling; group 1 is the locale, group 2
+//                the slide number (so a locale mismatch can be reported)
+export function layoutNumber(component, locale, components) {
+  if (!components || !components.canonical) {
+    throw new Error(
+      "fill-template pack declares no components.canonical pattern in " +
+        "skill.json — the pack owns its layout naming contract and must " +
+        "declare it, e.g. { \"components\": { \"canonical\": \"^MyLayout(\\\\d{2})$\" } }",
+    );
+  }
+  let m = component.match(compile(components.canonical, "canonical", components.skillDir));
   if (m) return { no: Number(m[1]) };
-  m = component.match(/^SlideProposal(En|De)(\d{2})Slide\d+$/);
-  if (m) {
-    const compLocale = m[1].toLowerCase();
-    return {
-      no: Number(m[2]),
-      warn:
-        compLocale === locale
-          ? `use the canonical <SlideProposalLayout${m[2]}> alias`
-          : `component locale "${compLocale}" does not match deck locale "${locale}"`,
-    };
+  if (components.localized) {
+    m = component.match(compile(components.localized, "localized", components.skillDir));
+    if (m) {
+      const compLocale = String(m[1]).toLowerCase();
+      return {
+        no: Number(m[2]),
+        warn:
+          compLocale === locale
+            ? `use the canonical alias for slide ${Number(m[2])}`
+            : `component locale "${compLocale}" does not match deck locale "${locale}"`,
+      };
+    }
   }
   return null;
 }
@@ -81,7 +115,7 @@ function evalLiteral(expr, file) {
     return value === undefined ? undefined : JSON.parse(JSON.stringify(value));
   } catch (err) {
     throw new Error(`${file}: cannot evaluate prop expression (${err.message}). ` +
-      `Proposal props must be data literals — strings, numbers, arrays, objects.`);
+      `Slide props must be data literals — strings, numbers, arrays, objects.`);
   }
 }
 
@@ -136,7 +170,7 @@ export function parseSlideFile(source, file) {
     const inner = closeIdx > 0 ? src.slice(i + 1, closeIdx).trim() : "";
     if (inner) {
       throw new Error(
-        `${file}: <${component}> has children. Master-backed proposal slides are ` +
+        `${file}: <${component}> has children. Master-backed slides are ` +
           `self-closing props-only elements; freeform content belongs to create-slides.`,
       );
     }

package/src/pptx/fill/xml.mjs

@@ -30,7 +30,7 @@ const stripHighlight = (rPr) =>
 
 const SHAPE_TAGS = ["sp", "pic", "cxnSp", "graphicFrame"];
 
-// Top-level shape blocks of one kind. The proposal masters keep all editable
+// Top-level shape blocks of one kind. Template masters keep all editable
 // shapes at spTree top level (verified by inventory); grouped shapes are not
 // addressed by the pack metadata.
 export function* blocks(xml, tag) {