slidemind 0.0.28 → 0.0.29

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "slidemind",
-  "version": "0.0.28",
+  "version": "0.0.29",
   "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/src/bridge/bundle.mjs

@@ -358,16 +358,27 @@ export function loadEnginesForDeck(deckDir) {
   return readEnginesFromDir(resolveBundleDirForDeck(deckDir));
 }
 
-// Pick the default engine to search: the first image/icon engine, else the
-// first engine. Single source of truth for the heuristic shared by the CLI
-// dispatcher (`slidemind search`) and the MCP `slidemind_search_engine` tool.
+// Image-producing engine kinds — the modality "image" plus the finer image
+// content kinds an authored bundle may carry. Excludes "slide" (a slide-search
+// engine returns whole slides, not insertable bytes). Mirrors the add-in's
+// local-bridge `engine_not_image` gate so the CLI never picks an engine the
+// bridge would reject.
+const IMAGE_ENGINE_KIND_RE = /^(image|icon|photo|illustration|vector)$/i;
+
+// True when an engine produces insertable image bytes. Gates on the engine's
+// real `kind` (its modality from the design bundle) — NOT a guess over its
+// label/description.
+export function isImageEngine(engine) {
+  return IMAGE_ENGINE_KIND_RE.test(engine?.kind ?? "");
+}
+
+// Pick the default engine to search: the first image engine, by its real
+// `kind`. Returns null when the CD exposes no image engine (e.g. only a
+// slide-search engine) — callers surface a "wire an image engine" message
+// rather than falling back to an engine the bridge can't insert. Single source
+// of truth for the heuristic shared by the CLI dispatcher (`slidemind search`)
+// and the MCP `slidemind_search_engine` tool.
 export function pickImageEngine(engines) {
   const list = Array.isArray(engines) ? engines : [];
-  return (
-    list.find((e) =>
-      /image|icon/i.test(`${e.kind ?? ""} ${e.label ?? ""} ${e.description ?? ""}`),
-    ) ||
-    list[0] ||
-    null
-  );
+  return list.find(isImageEngine) ?? null;
 }

package/src/builders/resolve-skill.mjs

@@ -44,6 +44,26 @@ function firstExisting(candidates) {
   return null;
 }
 
+// Does a design root actually carry brand files? A root qualifies if it has the
+// corporate-design CSS or a layouts dir that holds at least one real layout
+// (a `*.jsx` or `layouts.css`). A bare/junk-only `layouts/` — e.g. one left
+// behind on disk holding only a macOS `.DS_Store` — must NOT count, or it would
+// mask the "materialize the bundle first" guidance for a deck that hasn't
+// downloaded one yet.
+function carriesDesign(root, cssName) {
+  if (!root) return false;
+  if (fs.existsSync(path.join(root, cssName))) return true;
+  const layoutsDir = path.join(root, "layouts");
+  if (!fs.existsSync(layoutsDir)) return false;
+  try {
+    return fs
+      .readdirSync(layoutsDir)
+      .some((f) => f.endsWith(".jsx") || f === "layouts.css");
+  } catch {
+    return false;
+  }
+}
+
 // First existing candidate, or `fallback` when none exist. Encodes the
 // deck-override → bundle → skill precedence in one place so every resolved path
 // shares the same lookup rule.
@@ -73,12 +93,7 @@ export function resolveSkill(skillName, planMeta = {}, opts = {}) {
   const designRoots = [
     bundleDir,
     fs.existsSync(skillDir) ? skillDir : null,
-  ].filter(
-    (r) =>
-      r &&
-      (fs.existsSync(path.join(r, cssName)) ||
-        fs.existsSync(path.join(r, "layouts"))),
-  );
+  ].filter((r) => carriesDesign(r, cssName));
   if (designRoots.length === 0) {
     throw new Error(
       `No design source for skill "${skillName}". The corporate design now lives ` +