slidemind 0.3.16 → 0.3.17

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "slidemind",
-  "version": "0.3.16",
+  "version": "0.3.17",
   "description": "Connect Claude Code to the slidemind PowerPoint add-in — build and edit native, on-brand PowerPoint decks against the active corporate design, from the terminal.",
   "author": {
     "name": "statworx",

package/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "slidemind",
-  "version": "0.3.16",
+  "version": "0.3.17",
   "description": "Connect Codex to the slidemind PowerPoint add-in — build and edit native, on-brand PowerPoint decks against the active corporate design. Requires the slidemind CLI (npm install -g slidemind); the skills explain the install when it is missing.",
   "author": {
     "name": "statworx",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "slidemind",
-  "version": "0.3.16",
+  "version": "0.3.17",
   "description": "Connect Claude Code, Codex & opencode to the slidemind PowerPoint add-in — let your coding agent build and edit native, on-brand PowerPoint decks against your active corporate design, from the terminal.",
   "type": "module",
   "bin": {

package/skills/create-slides/SKILL.md

@@ -652,7 +652,9 @@ From here:
    the gate will open). Two things come back, and both bind:
    - **Measured lines** under each slide — `[fill:warn]` (a stretched region whose organisms pool
      at its top, with the region and the dead band) and `[inventory:warn]` (connectors listed in
-     `asset-plan.md` but not drawn, an empty inventory, a fetched asset no slide references) —
+     `asset-plan.md` but not drawn — counted per route kind too, so five `curved` rows drawn as
+     elbows warn even when the total matches; an empty inventory, a fetched asset no slide
+     references) —
      and a closing `DRAFT CHECK: N measured defects` line. Fix every one in the JSX and re-run;
      the review gate refuses to open while they stand for an unchanged slide. A fill band is
      fixed by distributing the organisms (`grid-template-rows` with one track per organism,

package/src/bridge/build-and-push.mjs

@@ -621,7 +621,10 @@ export async function viewDraft(
 
   let captured;
   try {
-    captured = await captureDeckLocally({ deckDir, bundleDir, onProgress, only });
+    // Converted slides never reach the local engine — it would lint each one
+    // as an unregistered layout and the noise would ride the translate gate.
+    const capturable = all.filter((f) => f.endsWith(".jsx") && !conversions[f]);
+    captured = await captureDeckLocally({ deckDir, bundleDir, onProgress, only: capturable });
   } catch (err) {
     throw new Error(
       `local draft capture failed (${err instanceof Error ? err.message : String(err)}). ` +

package/src/builders/build.mjs

@@ -38,6 +38,7 @@ import { loadChartPreview, loadConnectorPreview } from "../bridge/capture.mjs";
 import { FAVICON_LINK } from "./gate-server.mjs";
 import { lintSlidesOrThrow } from "./lint.mjs";
 import { resolveSkillForDeck } from "./resolve-skill.mjs";
+import { loadConversions } from "../lib/conversions.mjs";
 import { slideOrderWarning, sortSlideFilenames } from "../lib/slide-order.mjs";
 
 const __filename = fileURLToPath(import.meta.url);
@@ -87,6 +88,29 @@ function inlineLayout(file) {
   );
 }
 
+// Preview stand-ins for the ConvertedHost layouts converted slides invoke. The
+// host's name carries its canvas (`ConvertedHost<cx>x<cy>`, EMU), so the stub
+// sizes itself the way the minted host does (EMU / 9525, 3 decimals) instead of
+// assuming 16:9 — a 4:3 source previews at 960x720, not stretched to 1280.
+// Real geometry still comes from the server render.
+const HOST_NAME_RE = /<(ConvertedHost(\d+)x(\d+))/;
+const emuToCssPx = (emu) => `${Math.round((Number(emu) / 9525) * 1000) / 1000}px`;
+
+function convertedHostStubs(slideFiles, conversions) {
+  const hosts = new Map();
+  for (const file of slideFiles) {
+    if (!conversions[path.basename(file)]) continue;
+    const m = HOST_NAME_RE.exec(fs.readFileSync(file, "utf8"));
+    if (m) hosts.set(m[1], { w: emuToCssPx(m[2]), h: emuToCssPx(m[3]) });
+  }
+  return [...hosts].map(
+    ([name, { w, h }]) =>
+      `<script type="text/jsx-template" data-kind="layout" data-name="${name}" data-binding="${name}">\n` +
+      `function ${name}({ background, children }) { return <div style={{ position: "relative", width: "${w}", height: "${h}", overflow: "hidden", background: background || "#FFFFFF" }}>{children}</div>; }\n` +
+      `</script>`,
+  );
+}
+
 function layoutBindingNameForStem(stem) {
   return "Slide" + stem.split("-").map((p) => (
     p ? p[0].toUpperCase() + p.slice(1) : ""
@@ -354,7 +378,15 @@ export async function buildDeck(deckDir, opts = {}) {
       fs.readFileSync(file, "utf8"),
     ]),
   );
-  await lintSlidesOrThrow({ deckDir, slideFiles, layoutSources });
+  // Converted slides (`slidemind adopt`) invoke a synthetic ConvertedHost
+  // layout the server mints per source canvas; no bundle carries it, so the
+  // local engine would fail every one of them as an unregistered layout. They
+  // are linted server-side on render — skip them here, lint the authored rest.
+  const conversions = loadConversions(deckDir, { includeSource: false });
+  const authoredFiles = slideFiles.filter((f) => !conversions[path.basename(f)]);
+  if (authoredFiles.length > 0) {
+    await lintSlidesOrThrow({ deckDir, slideFiles: authoredFiles, layoutSources });
+  }
 
   // The <Chart> stand-in runtime rides in the same bundle cache (see
   // loadChartPreview) — resolved after lint, which already proved the bundle
@@ -364,7 +396,10 @@ export async function buildDeck(deckDir, opts = {}) {
 
   copyDesignSystem(deckDir, skill);
 
-  const layoutsBlock = layoutFiles.map(inlineLayout).join("\n");
+  const layoutsBlock = layoutFiles
+    .map(inlineLayout)
+    .concat(convertedHostStubs(slideFiles, conversions))
+    .join("\n");
   const slidesBlock = slideFiles
     .map((file, i) => inlineSlide(file, i))
     .join("\n");

package/src/builders/draft-check.mjs

@@ -30,6 +30,20 @@ const NONE_RE = /^\(?\s*(none|n\/a|no\b)/i;
 
 const pad2 = (n) => String(n).padStart(2, "0");
 
+// Route kind per connector row / directive — part of the mockup contract: a
+// loop the concept draws as curves that lands as elbows is a different figure,
+// and the count alone (5 = 5) never fires. Port of inventory.ts (gotcha 84).
+const ROUTE_KINDS = ["straight", "elbow", "curved"];
+const emptyRoutes = () => ({ straight: 0, elbow: 0, curved: 0 });
+function routeKindOf(text) {
+  for (const token of text.split(",")) {
+    const t = token.trim().toLowerCase();
+    if (ROUTE_KINDS.includes(t)) return t;
+  }
+  return null;
+}
+const describeRoutes = (r) => ROUTE_KINDS.map((k) => `${r[k]} ${k}`).join(", ");
+
 /** Seed the inventory form once (never overwrites a filled one). `images`: [{slideNumber,label}]. */
 export function seedAssetPlan(deckDir, images) {
   const file = path.join(deckDir, ASSET_PLAN_FILE);
@@ -91,7 +105,7 @@ function parseAssetPlan(text) {
     const line = raw.trim();
     const heading = SLIDE_HEADING_RE.exec(line);
     if (heading) {
-      current = { filled: false, connectors: 0, organisms: 0, icons: 0, iconsResolved: 0, verdict: null, ticked: 0, unticked: 0 };
+      current = { filled: false, connectors: 0, routes: emptyRoutes(), organisms: 0, icons: 0, iconsResolved: 0, verdict: null, ticked: 0, unticked: 0 };
       out.set(Number(heading[1]), current);
       section = "";
       continue;
@@ -119,7 +133,12 @@ function parseAssetPlan(text) {
       if (inline && !NONE_RE.test(inline)) {
         if (section === "organisms" || section === "regions") current.filled = true;
         if (section === "connectors") {
-          current.connectors += inline.split(";").filter((x) => x.trim()).length;
+          for (const row of inline.split(";")) {
+            if (!row.trim()) continue;
+            current.connectors += 1;
+            const kind = routeKindOf(row);
+            if (kind) current.routes[kind] += 1;
+          }
         }
       }
       continue;
@@ -129,7 +148,11 @@ function parseAssetPlan(text) {
     if (!body || NONE_RE.test(body)) continue;
     if (section === "organisms" || section === "regions") current.filled = true;
     if (section === "organisms") current.organisms += 1;
-    if (section === "connectors") current.connectors += 1;
+    if (section === "connectors") {
+      current.connectors += 1;
+      const kind = routeKindOf(body);
+      if (kind) current.routes[kind] += 1;
+    }
     if (section === "icons") {
       current.icons += 1;
       // Resolved = a fetched file appended, or an explicit "(no result)".
@@ -167,6 +190,19 @@ function countConnectors(jsx) {
   return attached + markers;
 }
 
+// Route kind per `data-connector` directive; a missing data-route is straight.
+// Imported native connectors carry their own preset and are not authored
+// against the mockup, so they are left out of the kind tally.
+function countRoutes(jsx) {
+  const routes = emptyRoutes();
+  for (const tag of jsx.match(/<[^<>]*\bdata-connector\b[^<>]*>/g) ?? []) {
+    if (/\bdata-native-connector\b/.test(tag)) continue;
+    const kind = /\bdata-route\s*=\s*["']([a-z]+)["']/.exec(tag)?.[1] ?? "straight";
+    if (ROUTE_KINDS.includes(kind)) routes[kind] += 1;
+  }
+  return routes;
+}
+
 /**
  * `[inventory:warn]` lines for the deck: per authored slide (inventory empty,
  * connectors listed > drawn) plus fetched assets no slide references. Empty
@@ -212,6 +248,14 @@ export function inventoryWarnings(deckDir, jsxFiles = null) {
         `[inventory:warn] ${jsxPath} — asset-plan.md lists ${inv.connectors} connector${inv.connectors === 1 ? "" : "s"} for this slide, the JSX draws ${drawn} (data-connector / marker-end). Draw the missing ones per workflow/authoring.md → Arrow contract; a mockup connector is never dropped.`,
       );
     }
+    const drawnRoutes = countRoutes(jsx);
+    for (const kind of ROUTE_KINDS) {
+      if (inv.routes[kind] > drawnRoutes[kind]) {
+        lines.push(
+          `[inventory:warn] ${jsxPath} — asset-plan.md lists ${inv.routes[kind]} ${kind} connector${inv.routes[kind] === 1 ? "" : "s"} for this slide, the JSX draws ${drawnRoutes[kind]} (${describeRoutes(drawnRoutes)}). The mockup's route shape is the contract: keep data-route="${kind}" and fix the endpoint sides or node positions instead of changing the route kind.`,
+        );
+      }
+    }
   }
   // Fetched assets nobody references: the mockup staged them, the deck lost them.
   const referenced = allJsx

package/src/lib/reconcile.mjs

@@ -166,8 +166,34 @@ export async function collectLiveReconcileInputs({
     const before = Object.keys(exported).length;
     const ack = await exportRound(exportSlides, pending);
     deck ??= ack?.deck ?? null;
+    // PowerPoint web (package fallback on) may hand back the SAVED file's bytes
+    // (`provenance.source` graph/saved) when the host export fails. Saved bytes
+    // cannot show unsaved manual edits, so projecting them would read "no
+    // drift" for a slide the user is editing right now — treat them as not
+    // exported, same failure mode as a slide that never arrived.
+    const stale = [];
     for (const [guid, slide] of Object.entries(ack?.slides ?? {})) {
-      if (slide?.base64 && slide?.tags) exported[guid] = slide;
+      if (!slide?.base64 || !slide?.tags) continue;
+      if (slide.provenance && slide.provenance.source !== "host") stale.push(guid);
+      else exported[guid] = slide;
+    }
+    // `failed` names slides neither the host nor the stored file could supply
+    // (web fallback); they never come back, so they fail closed at once.
+    const failed = Object.keys(ack?.failed ?? {}).filter(
+      (guid) => !exported[guid] && requested.includes(guid),
+    );
+    const unverifiable = [...new Set([...failed, ...stale])];
+    if (unverifiable.length > 0) {
+      const err = new Error(
+        `the pane could not export ${unverifiable.length} live slide(s) from the ` +
+          `open presentation (${failed.length} unavailable, ${stale.length} only as ` +
+          `the saved file), so their manual edits cannot be checked. Save the ` +
+          `presentation and retry, or re-run with --force to push anyway and ` +
+          `discard any manual edits.`,
+      );
+      err.code = "export_incomplete";
+      err.failed = ack?.failed ?? {};
+      throw err;
     }
     const next = (Array.isArray(ack?.truncated) ? ack.truncated : []).filter(
       (guid) => !exported[guid] && requested.includes(guid),