wand-decks-kit 0.8.4 → 0.8.7

package/PROMPT.md

@@ -421,13 +421,15 @@ DejaVu Sans / Carlito / Liberation Sans, never Geist. Consequences:
 **Garbled digits in exported PDFs.** The reference PDFs show `$ǿǹT+` where the deck says `$60T+`.
 Cause: Google Slides subsets the embedded Geist and writes a `ToUnicode` map offset by +457, so
 digits decode to Latin Extended-B in the PDF text layer. **The `.pptx` is correct — do not "fix"
-these strings.** Our LibreOffice export is clean. To confirm any deck you ship:
+these strings.** Our own export is clean, and `wand.js ship` confirms it on every run —
+the `text layer` line and the `pdf text` gate. To check a PDF by hand, use the portable
+form (sandbox grep has no `-P`, so the old `grep -P` command errors there):
 
 ```bash
-pdftotext output.pdf - | grep -P '[\x{01C9}-\x{0250}]'
+python3 -c "import subprocess;t=subprocess.run(['pdftotext','output.pdf','-'],capture_output=True,text=True).stdout;print([c for c in t if 0x01C9<=ord(c)<=0x024F] or 'clean')"
 ```
 
-Empty output is a pass. Hits mean subsetted fonts got in — set `saveSubsetFonts="0"` and re-embed.
+`clean` is a pass. Listed characters mean subsetted fonts got in — set `saveSubsetFonts="0"` and re-embed.
 
 **Other:** the reference decks use `roundRect` with per-shape `adj`, so copying a card and resizing
 it changes its corner radius — recompute. Icon glyphs are separate PNGs layered over tile shapes;

package/SKILL.md

@@ -4,7 +4,7 @@ description: Rebrand, merge, edit or author a PowerPoint deck on the Wand design
 compatibility: Requires Node.js 18+ and code execution enabled in claude.ai (Settings > Features). No network access needed - dependencies are bundled. Python 3 is used to read a source .pptx and to embed fonts; without it the kit can still author and check a deck, but the output will render in Arial on machines without Geist.
 license: Proprietary - Wand AI
 metadata:
-  version: "0.7.0"
+  version: "0.8.7"
 ---
 
 
@@ -61,7 +61,16 @@ node wand.js doctor
 
 If `node_modules` is somehow missing, `npm install` in this directory.
 
-## The four jobs
+## Before any build: show the plan, then wait
+
+Write the slide-by-slide mapping in the chat **before** writing a line of JSON:
+`S07 (timeline, 6 phases) → phaseCards`, one line per slide, with the reason
+where it is not obvious. Then build. A field run skipped this, built the wrong
+thing, and paid a full second pass. This is not optional for a rebuild, a merge
+or an adaptation; only "author from a brief" with fewer than eight slides may
+go straight to the spec.
+
+## The five jobs
 
 ### Rebrand a deck they attached
 
@@ -89,6 +98,24 @@ to cover — and it says so.
 spec. Ask which deck leads if they have not said. Flag duplicates rather than
 silently dropping them, and list every merge in the review note.
 
+### Adapt a deck and keep its graphics
+
+The ask sounds like a rebrand but is not: *"take our standard deck, drop in an
+agenda, a team slide and a demo placeholder, keep everything else as is."* They
+want the source's own slides and art preserved, with a few slides retexted or
+added. **This kit re-authors from a spec; it cannot copy a source slide's
+shapes.** Say that in one line, then edit the source `.pptx` directly with the
+pptx tools — and still run this kit's gates on the result:
+
+```bash
+node wand.js audit <edited-deck>.pptx              # file, pdf text, renders + contact sheet
+node wand.js audit <edited-deck>.pptx --dump work/visible_dump.txt   # + coverage
+```
+
+`audit` runs every gate that does not need a spec. A deck delivered without it
+has been checked by nobody. Say which slides were lifted unchanged, which were
+retexted, and that the source's fonts (not Geist) carry through.
+
 ### Edit a deck this kit built
 
 ```bash
@@ -139,6 +166,14 @@ repeatedly.
 4. **Icons are named.** `node wand.js glyphs` lists all 75. Pick by meaning.
    They recolour to the theme, so there is no polarity to get wrong.
 5. **Never invent content.** If the source is ambiguous, ask.
+6. **A FAIL in the gate table is not shippable.** Not "acceptable two-line
+   wraps", not "probably fine" — either fix it and re-run, or list each failing
+   item with its measurement in the review note and let the user decide. A
+   field run shipped on `FAIL fit + bounds` with a reassurance instead of a
+   list; the reassurance was not checked against anything.
+7. **Verbatim applies to rebuilds, merges and lifted slides.** When the job is
+   to author or retext a slide, say which slides were retexted — the review
+   note separates *lifted verbatim* from *rewritten*.
 
 ## Useful commands
 
@@ -150,6 +185,7 @@ repeatedly.
 | `node wand.js check <spec>` | fit in real Geist, bounds, assets |
 | `node wand.js preview <spec>` | writes `preview/trace.js` for the browser preview |
 | `node wand.js inspect <deck>` | parts, relationships, embedded fonts |
+| `node wand.js audit <deck> [--dump f]` | every spec-free gate on any .pptx — use on decks edited outside the kit |
 
 ## Deliver
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wand-decks-kit",
-  "version": "0.8.4",
+  "version": "0.8.7",
   "description": "Rebuilds or authors a client-ready .pptx on the Wand design system: measured fit, embedded Geist, verbatim content.",
   "license": "SEE LICENSE IN PROMPT.md",
   "bin": {

package/wand.js

@@ -56,6 +56,7 @@ function runPython(args) {
 function doctor() {
   const rows = [];
   const py = python();
+  rows.push(["wand-decks-kit", require("./wand_kit").VERSION, "ok"]);
   rows.push(["node", process.version, "ok"]);
   rows.push(["python3", py ? execFileSync(py, ["-V"], { encoding: "utf8" }).trim() : "MISSING",
              py ? "ok" : "extract and font embedding unavailable"]);
@@ -203,6 +204,24 @@ function ship(specPath, outArg) {
   console.log(rule("7 file"));
   const fileErrs = inspect(out);
 
+  // Garbled-digit check, automated. The manual command in PROMPT.md used
+  // grep -P, which the sandbox's grep does not have — a field run had to
+  // improvise around it. A gate nobody types cannot be mistyped.
+  let garbled = null;
+  if (has("pdftotext") && fs.existsSync(pdfOut)) {
+    const r = spawnSync("pdftotext", [pdfOut, "-"], { encoding: "utf8" });
+    if (r.status === 0) {
+      garbled = 0;
+      for (const c of r.stdout || "") {
+        const p = c.codePointAt(0);
+        if (p >= 0x01c9 && p <= 0x024f) garbled++;
+      }
+      console.log(garbled
+        ? `text layer  : ${garbled} garbled glyph(s) — subsetted fonts got in; set saveSubsetFonts="0" and re-embed`
+        : "text layer  : digits decode clean");
+    }
+  }
+
   console.log(rule("8 visual"));
   node([path.join(KIT, "build.js"), path.resolve(specPath), "--preview"]);
   let shots = 0;
@@ -227,13 +246,78 @@ function ship(specPath, outArg) {
   }
 
   const gates = [["fit + bounds", bad === 0], ["coverage", cov === null ? null : cov === 0],
-                 ["file", fileErrs === 0], ["renders", shots > 0 ? true : null]];
+                 ["file", fileErrs === 0], ["pdf text", garbled === null ? null : garbled === 0],
+                 ["renders", shots > 0 ? true : null]];
   console.log(rule("gates"));
   for (const [n, ok] of gates) console.log(`  ${ok === null ? "skip" : ok ? " ok " : "FAIL"}  ${n}`);
   console.log(`\n${rel(out)}${fs.existsSync(pdfOut) ? "\n" + rel(pdfOut) : ""}\n`);
   return gates.some(([, ok]) => ok === false) ? 1 : 0;
 }
 
+/**
+ * Every gate that does not need a spec, on any .pptx.
+ *
+ * A field run adapted a source deck by editing it directly — the right call,
+ * this kit cannot copy a source slide's shapes — and then delivered it with
+ * no gate but a generic validator, because `ship` wants a spec. Now the
+ * spec-free gates run on their own: file structure and fonts, the PDF text
+ * layer, a render of every slide with the contact sheet, and coverage when a
+ * verbatim dump is given. LibreOffice does the render, so fit is *not* judged
+ * here (§9 — DejaVu is wider than Geist); geometry, overlap and contrast are.
+ */
+function audit(file, dump) {
+  const work = path.join(path.dirname(file), "audit");
+  fs.mkdirSync(work, { recursive: true });
+  console.log(rule("1 file"));
+  const fileErrs = inspect(file);
+
+  console.log(rule("2 render"));
+  let shots = 0, garbled = null, pdf = null;
+  // on PATH in a sandbox; an app bundle on a Mac
+  const soffice = ["soffice", "libreoffice"].find((c) => has(c))
+    || ["/Applications/LibreOffice.app/Contents/MacOS/soffice"].find((p) => fs.existsSync(p)) || null;
+  if (soffice) {
+    spawnSync(soffice, ["--headless", "--convert-to", "pdf", "--outdir", work, file], { stdio: "ignore" });
+    pdf = path.join(work, path.basename(file).replace(/\.pptx$/i, ".pdf"));
+  }
+  if (pdf && fs.existsSync(pdf) && has("pdftoppm")) {
+    const dir = path.join(work, "slides");
+    fs.mkdirSync(dir, { recursive: true });
+    spawnSync("pdftoppm", ["-r", "96", "-png", pdf, path.join(dir, "slide")], { stdio: "ignore" });
+    shots = fs.readdirSync(dir).filter((f) => f.startsWith("slide")).length;
+    const small = fs.mkdtempSync(path.join(require("os").tmpdir(), "wand-sheet-"));
+    spawnSync("pdftoppm", ["-r", "26", "-png", pdf, path.join(small, "s")], { stdio: "ignore" });
+    const sheet = path.join(dir, "sheet.png");
+    runPython([path.join(KIT, "tools", "sheet.py"), small, sheet]);
+    fs.rmSync(small, { recursive: true, force: true });
+    console.log(`rendered    : ${shots} slide(s) — open ${rel(sheet)} (LibreOffice: trust geometry,`);
+    console.log(`              overlap and contrast; do NOT judge fit or weight from it)`);
+    if (has("pdftotext")) {
+      const r = spawnSync("pdftotext", [pdf, "-"], { encoding: "utf8" });
+      if (r.status === 0) {
+        garbled = 0;
+        for (const c of r.stdout || "") { const cp = c.codePointAt(0); if (cp >= 0x01c9 && cp <= 0x024f) garbled++; }
+        console.log(garbled ? `text layer  : ${garbled} garbled glyph(s) — fonts are subsetted` : "text layer  : digits decode clean");
+      }
+    }
+  } else {
+    console.log(soffice ? "pdftoppm absent — no renders" : "no LibreOffice — no renders; open the deck and look at every slide");
+  }
+
+  console.log(rule("3 coverage"));
+  let cov = null;
+  if (dump && fs.existsSync(dump)) cov = runPython([path.join(KIT, "qa_coverage.py"), dump, file]);
+  else console.log("no dump given — skipped. Pass --dump work/visible_dump.txt from `rebuild` to check\n" +
+                   "every source string survived the edit.");
+
+  const gates = [["file", fileErrs === 0], ["pdf text", garbled === null ? null : garbled === 0],
+                 ["renders", shots > 0 ? true : null], ["coverage", cov === null ? null : cov === 0]];
+  console.log(rule("gates"));
+  for (const [n, ok] of gates) console.log(`  ${ok === null ? "skip" : ok ? " ok " : "FAIL"}  ${n}`);
+  console.log("  (fit is not gated on a deck built outside the kit — say so in the review note)\n");
+  return gates.some(([, ok]) => ok === false) ? 1 : 0;
+}
+
 /**
  * File-level check, in-process. `validate.py` from the pptx skill is not in the
  * zip and LibreOffice may not be installed, so the kit carries its own: the
@@ -396,6 +480,12 @@ to.
     if (!positional[0]) die("usage: node wand.js inspect <deck.pptx>");
     process.exit(inspect(path.resolve(positional[0])) ? 1 : 0);
 
+  case "audit": {
+    if (!positional[0]) die("usage: node wand.js audit <deck.pptx> [--dump visible_dump.txt]");
+    const di = rest.indexOf("--dump");
+    process.exit(audit(path.resolve(positional[0]), di >= 0 ? path.resolve(rest[di + 1]) : null));
+  }
+
   case "ship":
     if (!positional[0]) die("usage: node wand.js ship <spec.json> [out.pptx]");
     process.exit(ship(positional[0], positional[1]));

package/wand_kit.js

@@ -999,7 +999,7 @@ function productMap(pres, { tag, titleText, subtitleText, bandRow, cards, split,
 
 
 module.exports = {
-  VERSION: "0.8.4",
+  VERSION: "0.8.7",
   T, F, COLS, ASSETS, radius, theme, autoSize, measure: M,
   background, chrome, footer, title, subtitle, kicker, contentTop,
   card, iconTile, pill,