@koda-sl/baker-cli 0.444.0-dev.cac1750ec → 0.447.0-dev.397af7869

package/dist/cli.js

@@ -45,6 +45,7 @@ import {
   SEEDANCE_DURATIONS,
   SEEDANCE_PROFILE,
   SOUND_EFFECT_MODEL,
+  SPEC_SHAPES_HINT,
   SPOKEN_CLIP_DURATION_S,
   SPOKEN_WORDS_PER_SECOND,
   STATIC_ADAPT_FORMAT_ROLE,
@@ -69,6 +70,7 @@ import {
   ValidationError,
   VideoBlueprintSchema,
   YtDlpError,
+  adSpecErrorText,
   adSpecFromBlueprint,
   adSpecJsonSchema,
   adaptBrief,
@@ -171,6 +173,7 @@ import {
   samePictureSizes,
   sanitizeId,
   sceneWindowsBeside,
+  sceneWordProblems,
   sha256Hex,
   shapeListRows,
   specimenText,
@@ -198,7 +201,7 @@ import {
   withoutCastName,
   wordCount,
   ytDlpBlockSignal
-} from "./chunk-GY352NRN.js";
+} from "./chunk-WMAHVZMY.js";
 import {
   NAME_CONTAINS_ARG,
   csvOrJson,
@@ -71038,7 +71041,7 @@ function toPosix(p) {
 }
 
 // src/commands/creatives/ideas-model.ts
-import { parse as parseYaml2 } from "yaml";
+import { parse as parseYaml2, YAMLParseError } from "yaml";
 var IDEA_ANGLES = [
   "pain_relief",
   "aspiration",
@@ -71087,6 +71090,18 @@ function frontmatterOf(markdown) {
     return null;
   }
 }
+function frontmatterError(markdown) {
+  const block2 = FRONTMATTER.exec(markdown)?.[1];
+  if (block2 === void 0) return null;
+  try {
+    parseYaml2(block2);
+    return null;
+  } catch (e) {
+    const line = e instanceof YAMLParseError ? e.linePos?.[0]?.line : void 0;
+    const at = line ? ` at line ${line} (\`${block2.split(/\r?\n/)[line - 1]?.trim() ?? ""}\`)` : "";
+    return `the frontmatter is not valid YAML${at}: put each text value wholly in double quotes, with any quote inside it as \\" or as single quotes`;
+  }
+}
 function text2(value2) {
   if (typeof value2 === "string") return value2.trim() || void 0;
   if (typeof value2 === "number") return String(value2);
@@ -71141,6 +71156,8 @@ function fieldProblems(f) {
   return problems;
 }
 function parseIdea(id, markdown) {
+  const unparsable = frontmatterError(markdown);
+  if (unparsable) return { problems: [unparsable] };
   const fm = frontmatterOf(markdown) ?? {};
   const f = {
     who: text2(fm.who),
@@ -71714,6 +71731,14 @@ async function staticSpecGate(canvas, canvasDir) {
       fix: "Write the ad in prompt.json: the scene the viewer sees and each line of copy as it prints. Then re-run."
     };
   }
+  const sceneWords = spec.native ? [] : sceneWordProblems(spec);
+  if (sceneWords.length > 0) {
+    return {
+      code: "scene_words_not_in_copy",
+      problems: sceneWords,
+      fix: "Edit prompt.json: every word or figure the picture shows is a `copy` line (`in_scene`, with `on`); describe any other surface without words. Then re-run."
+    };
+  }
   const concept = conceptProblems(spec);
   if (concept.length > 0) {
     return {
@@ -71809,7 +71834,8 @@ async function prepareCanvas(canvasPath, opts = {}) {
     staticCopy = native === "not_applicable" ? await syncStaticAdBrief(canvas, log) : native;
     if (logo === "regenerated" || pictures === "regenerated") staticCopy = "regenerated";
   } catch (e) {
-    throw new CanvasPrepareError("style_projection", `prompt.json is not readable or not a valid ad: ${errorMessage(e)}`, {
+    throw new CanvasPrepareError("style_projection", `prompt.json is not readable or not a valid ad:
+${adSpecErrorText(e)}`, {
       fix: `Fix ${path14.join(canvasDir, "prompt.json")} so it parses as JSON (and, for a static ad, matches the ad spec the message names), then re-run.`
     });
   }
@@ -73579,15 +73605,17 @@ function correctionOf(finding, band) {
   if (misread2) return `Re-letter ${misread2[1]} exactly as written, letter for letter (it now reads ${misread2[2]}).`;
   const missing = what.match(MISSING);
   if (missing) return `Add the line ${missing[1]}, spelled exactly, where it belongs in the layout.`;
+  if (SIZES_FINDING.test(what)) return FOLLOW_PICTURE;
   const described2 = what.replace(CATEGORY, "").replace(/\.$/, "");
   return `Fix this: ${described2.charAt(0).toLowerCase()}${described2.slice(1)}.`;
 }
 function autoFixStep(canvas, review, opts = {}) {
   if ((opts.round ?? 0) >= AUTO_FIX_ROUNDS) return null;
-  const blocking = (review ?? []).filter((f) => f.severity === "blocking");
-  if (blocking.length === 0) return null;
-  if (isNativeCanvas(canvas)) return { kind: "respec", findings: unique(blocking.map((f) => f.what)) };
+  const flagged = (review ?? []).filter((f) => f.severity === "blocking");
+  if (flagged.length === 0) return null;
+  if (isNativeCanvas(canvas)) return { kind: "respec", findings: unique(flagged.map((f) => f.what)) };
   const relaid = relaidSizes(canvas);
+  const blocking = onDerivedSizes(flagged, relaid);
   const unplaceable = new Set((opts.tooFar ?? []).filter((format) => relaid.some((r) => r.format === format)));
   const misplaced = (f) => {
     const what = withoutSize(f.what);
@@ -73631,8 +73659,8 @@ function autoFixStep(canvas, review, opts = {}) {
     pictureFindings.filter((f) => !BRIEF.test(withoutSize(f.what))).flatMap((f) => correctionOf(f.what) ?? [])
   );
   if (changes.length === 0) return sizesOnly();
-  const flagged = new Set(blocking.flatMap((f) => sizesOf(f.what)));
-  const keep = relaid.map((r) => r.format).filter((format) => !flagged.has(format));
+  const named = new Set(blocking.flatMap((f) => sizesOf(f.what)));
+  const keep = relaid.map((r) => r.format).filter((format) => !named.has(format));
   const remade = every.filter((size) => !keep.includes(size));
   const kept = {
     ...keep.length > 0 ? { keep } : {},
@@ -73645,6 +73673,17 @@ function autoFixStep(canvas, review, opts = {}) {
   return { kind: "fix", instruction, ...Object.keys(edits).length > 0 ? { relay: edits } : {}, ...kept };
 }
 var unique = (lines) => [...new Set(lines)];
+var SIZES_FINDING = /^SIZES:/i;
+var FOLLOW_PICTURE = "Show exactly what the finished ad (Image 2) shows: its people, products, panels and their order, and nothing it does not show.";
+function onDerivedSizes(findings, relaid) {
+  if (relaid.length === 0) return [...findings];
+  return findings.flatMap((f) => {
+    const sizes = sizesOf(f.what);
+    const onPicture = sizes.length > 0 && sizes.every((size) => !relaid.some((r) => r.format === size));
+    if (!onPicture || !SIZES_FINDING.test(withoutSize(f.what))) return [f];
+    return relaid.map((r) => ({ ...f, what: `${r.format}: ${withoutSize(f.what)}` }));
+  });
+}
 function deliveredSizes(canvas) {
   const nodes = canvas?.nodes;
   const labels = Array.isArray(nodes) ? nodes.find((n) => n.id === "final" && n.type === "collect")?.params?.labels : void 0;
@@ -74322,7 +74361,8 @@ function viewerCheckFailure(outcome2, slug) {
     ...outcome2.verdict.missed.map(missLine),
     ...missing ? [`They needed: ${missing}`] : [],
     "Rewrite the copy in prompt.json so the words on the ad say it plainly (name the offer and who it is for; a stranger reads only what is printed), then re-run.",
-    `Check new copy before rendering with \`baker canvas viewer-check ${slug}\`.`,
+    // The run asks the viewer itself, from the same cache: a separate check is one more wait (dev, Oct 2026).
+    `Then run the same \`baker canvas run\` again: it checks the new copy before it spends (or \`baker canvas viewer-check ${slug}\` alone).`,
     "If this ad will run to people who already know the brand, record that viewer in _definition.md (`targetAdSet` and `viewer`, from `baker ads meta adsets --audience`) instead of rewriting."
   ].join(" ");
   return {
@@ -75252,7 +75292,7 @@ var runCommand = defineCommand139({
     },
     concurrency: {
       type: "string",
-      description: "Max nodes per layer in flight at once (default 8; env BAKER_CANVAS_CONCURRENCY)"
+      description: "Max nodes running at once (default 8; env BAKER_CANVAS_CONCURRENCY)"
     },
     parallel: {
       type: "string",
@@ -76036,7 +76076,7 @@ var rerunCommand = defineCommand140({
     },
     fresh: { type: "boolean", default: false, description: "Start a new run id instead of resuming an interrupted one" },
     regenerate: { type: "string", description: "Comma-separated node ids to force fresh this run" },
-    concurrency: { type: "string", description: "Max nodes per layer in flight at once" }
+    concurrency: { type: "string", description: "Max nodes running at once" }
   },
   async run({ args: args8 }) {
     process.stderr.write("[deprecated] `baker canvas rerun <slug>` is now `baker canvas run --slug <slug>`.\n");
@@ -81054,7 +81094,7 @@ function ideasHints(report) {
   if (next.length === 1) hints2.push(`Build: \`baker creatives make ${next[0]}\`.`);
   if (next.length > 1)
     hints2.push(
-      `Build ${next.join(", ")} at once: one \`creative-producer\` per idea, all spawned in one message in the background, each given its idea id, and handle each as it returns. Never build them one after another: each render takes minutes.`
+      `Build ${next.join(", ")} at once: one \`creative-producer\` per idea, all spawned in one message in the background, each given its idea id, and handle each as it returns. Never build them one after another: each render takes minutes. Edited an idea after this check? Run \`baker creatives ideas\` again first: a producer cannot build an idea with a problem.`
     );
   return hints2;
 }
@@ -81927,7 +81967,7 @@ var learnCommand = defineCommand158({
 
 // src/commands/creatives/make.ts
 import { execFile as execFile3 } from "child_process";
-import { mkdtemp as mkdtemp2, readFile as readFile38, rm as rm7, writeFile as writeFile24 } from "fs/promises";
+import { mkdtemp as mkdtemp2, readFile as readFile38, rm as rm7, stat as stat7, writeFile as writeFile24 } from "fs/promises";
 import { tmpdir as tmpdir3 } from "os";
 import path46 from "path";
 import { promisify as promisify3 } from "util";
@@ -82025,8 +82065,16 @@ async function findCandidates(searches, deps) {
   }
   return [];
 }
-async function pickReference(root, searches, deps = liveReferenceDeps(root)) {
+var candidateOf = (c) => ({
+  adId: String(c.ad_id),
+  platform: asPlatform(c.platform),
+  ...c.advertiser_id ? { advertiserId: c.advertiser_id } : {}
+});
+async function pickReference(root, searches, deps = liveReferenceDeps(root), prepare) {
   const candidates = (await findCandidates(searches, deps)).slice(0, FITS_TRIED);
+  const first = candidates[0];
+  const building = first && prepare ? prepare(candidateOf(first)) : void 0;
+  void building?.catch(() => void 0);
   const fits = await Promise.all(candidates.map((c) => deps.fit(asPlatform(c.platform), String(c.ad_id))));
   let unchecked = null;
   let checked = 0;
@@ -82038,14 +82086,13 @@ async function pickReference(root, searches, deps = liveReferenceDeps(root)) {
     if (!(fit.ok && fitsIdea(fit.fit))) continue;
     const layers = fit.fit.why_it_wins;
     return {
-      adId: String(candidate.ad_id),
-      platform: asPlatform(candidate.platform),
-      ...candidate.advertiser_id ? { advertiserId: candidate.advertiser_id } : {},
+      ...candidateOf(candidate),
       why: [layers.disguise, layers.trigger, layers.logic].filter(Boolean).join("; "),
-      tried: index + 1
+      tried: index + 1,
+      ...building ? index === 0 ? { prepared: building } : { discarded: building } : {}
     };
   }
-  return noReference(candidates.length, checked, unchecked);
+  return { ...noReference(candidates.length, checked, unchecked), ...building ? { discarded: building } : {} };
 }
 function noReference(found, checked, unchecked) {
   if (found > 0 && checked === 0 && unchecked) {
@@ -82113,24 +82160,37 @@ async function scaffoldIdea(root, idea, args8, slug, temperature) {
       reference: null
     };
   }
-  const reference = args8.reference ? { adId: args8.reference, platform: asPlatform(args8.platform), why: "chosen by hand", tried: 0 } : args8.fromIdea ? { code: "FROM_IDEA", message: "Asked to start from the idea." } : await pickReference(root, referenceSearchesFor(idea, temperature).map(buildSearchBody));
-  if ("code" in reference) {
-    return { scaffold: await scaffoldFromIdea(idea, slug, viewerFlags), reference: null, fromIdea: reference.message };
-  }
-  const scaffold = await baker([
+  const onReference = (c) => baker([
     "canvas",
     "scaffold-static-ad",
     "--source-ad-id",
-    reference.adId,
+    c.adId,
     "--source-platform",
-    reference.platform,
-    ...reference.advertiserId ? ["--source-advertiser-id", reference.advertiserId] : [],
+    c.platform,
+    ...c.advertiserId ? ["--source-advertiser-id", c.advertiserId] : [],
     "--slug",
     slug,
     "--title",
     idea.message.slice(0, 120),
     ...viewerFlags
   ]);
+  const dir = path46.join(root, "src", "creatives", slug);
+  const fresh = !await stat7(dir).catch(() => null);
+  const reference = args8.reference ? { adId: args8.reference, platform: asPlatform(args8.platform), why: "chosen by hand", tried: 0 } : args8.fromIdea ? { code: "FROM_IDEA", message: "Asked to start from the idea." } : await pickReference(
+    root,
+    referenceSearchesFor(idea, temperature).map(buildSearchBody),
+    void 0,
+    fresh ? onReference : void 0
+  );
+  if ("discarded" in reference && reference.discarded) {
+    await reference.discarded.catch(() => void 0);
+    await rm7(dir, { recursive: true, force: true });
+  }
+  if ("code" in reference) {
+    return { scaffold: await scaffoldFromIdea(idea, slug, viewerFlags), reference: null, fromIdea: reference.message };
+  }
+  const prepared = "prepared" in reference ? await reference.prepared?.catch(() => null) : null;
+  const scaffold = prepared?.ok === true ? prepared : await onReference(reference);
   return { scaffold, reference };
 }
 async function scaffoldFromIdea(idea, slug, viewerFlags) {
@@ -82227,6 +82287,7 @@ function makeEnvelope(made) {
         `Rewrite before running: ${made.unsourcedCopy.map((l) => JSON.stringify(l)).join(", ")} claim a figure nothing the company wrote holds (the reference's promise, not ours).`
       ] : [],
       `Write the copy lines and the scene in src/creatives/${made.slug}/prompt.json${made.reference ? ": the drafted lines are the reference's, so keep one only if the Knowledge says it (badges, ratings and guarantees too)" : ""}. Say the message in the customer's words${quote ? ` ("${quote.slice(0, 80)}")` : ""}; keep \`message\` as it is.`,
+      SPEC_SHAPES_HINT,
       `Then: baker canvas run ${made.canvasPath} (it validates and asks the viewer itself: nothing to run before it).`,
       // The scaffold's own next steps name a separate validate; the run does it (dev, Oct 2026: five validates before one run).
       ...made.scaffoldHints.filter((h) => !/canvas (?:validate|viewer-check)/.test(h)).slice(0, 4)
@@ -84004,7 +84065,7 @@ import { readdir as readdir19, readFile as readFile42 } from "fs/promises";
 import path48 from "path";
 
 // src/commands/landing/import-graph.ts
-import { readdir as readdir18, readFile as readFile41, stat as stat7 } from "fs/promises";
+import { readdir as readdir18, readFile as readFile41, stat as stat8 } from "fs/promises";
 import path47 from "path";
 var SOURCE_EXT = /\.(astro|ts|tsx|js|jsx|mjs|cjs|vue|svelte)$/;
 var RESOLVE_CANDIDATES = [
@@ -84024,7 +84085,7 @@ var IMPORT_RE = /(?<![A-Za-z0-9_$])(?:import|export)\s*(?:[\s\S]*?\bfrom\s*)?["'
 var DYNAMIC_IMPORT_RE = /\bimport\s*\(\s*["']([^"']+)["']\s*\)/g;
 async function isFile(p) {
   try {
-    return (await stat7(p)).isFile();
+    return (await stat8(p)).isFile();
   } catch {
     return false;
   }
@@ -84156,7 +84217,7 @@ async function readForms(root, slug) {
 
 // src/commands/experiment/fold.ts
 import { statSync } from "fs";
-import { cp as cp3, mkdir as mkdir15, readdir as readdir20, readFile as readFile43, rm as rm8, stat as stat8, writeFile as writeFile25 } from "fs/promises";
+import { cp as cp3, mkdir as mkdir15, readdir as readdir20, readFile as readFile43, rm as rm8, stat as stat9, writeFile as writeFile25 } from "fs/promises";
 import path50 from "path";
 
 // ../api/src/experiments/variants.ts
@@ -84223,7 +84284,7 @@ function posix2(p) {
 }
 async function isDir(p) {
   try {
-    return (await stat8(p)).isDirectory();
+    return (await stat9(p)).isDirectory();
   } catch {
     return false;
   }
@@ -91625,7 +91686,7 @@ function cropSprite(input, region) {
 
 // src/lib/image/io.ts
 import { randomBytes } from "crypto";
-import { glob as fsGlob, readFile as readFile46, rename as rename2, stat as stat9, writeFile as writeFile26 } from "fs/promises";
+import { glob as fsGlob, readFile as readFile46, rename as rename2, stat as stat10, writeFile as writeFile26 } from "fs/promises";
 import { dirname as dirname2, extname as extname3, join as join5, resolve as resolve4 } from "path";
 var REMOTE_RE = /^https?:\/\//i;
 var GLOB_RE = /[*?[\]{}]/;
@@ -91662,7 +91723,7 @@ async function readImageBuffer(pathOrUrl) {
 }
 async function isDirectory(path68) {
   try {
-    const s = await stat9(path68);
+    const s = await stat10(path68);
     return s.isDirectory();
   } catch {
     return false;
@@ -95120,7 +95181,7 @@ Full guide: __tooling__/docs/tools/baker/images.md`
 import { defineCommand as defineCommand226 } from "citty";
 
 // src/commands/landing/critique.ts
-import { readdir as readdir22, readFile as readFile48, stat as stat11 } from "fs/promises";
+import { readdir as readdir22, readFile as readFile48, stat as stat12 } from "fs/promises";
 import path55 from "path";
 import { defineCommand as defineCommand213 } from "citty";
 
@@ -96331,7 +96392,7 @@ async function writeCritiqueSnapshot(projectRoot, snapshot) {
 }
 
 // src/commands/landing/source-version.ts
-import { readdir as readdir21, readFile as readFile47, stat as stat10 } from "fs/promises";
+import { readdir as readdir21, readFile as readFile47, stat as stat11 } from "fs/promises";
 import path54 from "path";
 var CRITIQUED_ROOTS = ["src/pages/", "src/components/"];
 async function landingSourceRelPaths(root, slug) {
@@ -96379,7 +96440,7 @@ async function computeLegacyLandingSourceSha(landingDir) {
 }
 async function isFile2(p) {
   try {
-    return (await stat10(p)).isFile();
+    return (await stat11(p)).isFile();
   } catch {
     return false;
   }
@@ -96557,7 +96618,7 @@ function present(f) {
 }
 async function isDir2(p) {
   try {
-    return (await stat11(p)).isDirectory();
+    return (await stat12(p)).isDirectory();
   } catch {
     return false;
   }
@@ -99875,7 +99936,7 @@ Full guide: __tooling__/docs/tools/baker/landing.md`
 
 // src/commands/landing/variant.ts
 import { randomUUID as randomUUID4 } from "crypto";
-import { mkdir as mkdir20, readdir as readdir23, readFile as readFile49, stat as stat12, writeFile as writeFile32 } from "fs/promises";
+import { mkdir as mkdir20, readdir as readdir23, readFile as readFile49, stat as stat13, writeFile as writeFile32 } from "fs/promises";
 import path63 from "path";
 import { defineCommand as defineCommand225 } from "citty";
 registerSchema({
@@ -99927,14 +99988,14 @@ function fail11(code, message, fix) {
 }
 async function isDir3(p) {
   try {
-    return (await stat12(p)).isDirectory();
+    return (await stat13(p)).isDirectory();
   } catch {
     return false;
   }
 }
 async function exists2(p) {
   try {
-    await stat12(p);
+    await stat13(p);
     return true;
   } catch {
     return false;
@@ -108003,7 +108064,7 @@ var groupCommand2 = defineCommand295({
 });
 
 // src/commands/videos/ingest.ts
-import { mkdtemp as mkdtemp4, rm as rm10, stat as stat13 } from "fs/promises";
+import { mkdtemp as mkdtemp4, rm as rm10, stat as stat14 } from "fs/promises";
 import { tmpdir as tmpdir5 } from "os";
 import path67 from "path";
 import { defineCommand as defineCommand296 } from "citty";
@@ -108276,7 +108337,7 @@ async function downloadThenIngest(args8, country) {
       // we allow to fetch it cannot drift apart.
       timeoutMs: downloadTimeoutMs(durationSeconds(probe.info))
     });
-    const stats = await stat13(filePath);
+    const stats = await stat14(filePath);
     if (stats.size > MAX_VIDEO_INGEST_BYTES) {
       throw new ApiError(
         "VALIDATION_ERROR",
@@ -108437,7 +108498,7 @@ var searchCommand6 = defineCommand298({
 var tagsCommand6 = makeTagsCommand("videos", "video", "/api/videos/tags");
 
 // src/commands/videos/upload.ts
-import { stat as stat14 } from "fs/promises";
+import { stat as stat15 } from "fs/promises";
 import { basename as basename4 } from "path";
 import { defineCommand as defineCommand299 } from "citty";
 registerSchema({
@@ -108505,7 +108566,7 @@ var uploadCommand2 = defineCommand299({
       const contentType = args8["content-type"] || detectVideoContentType(filePath);
       const descriptionContext = args8.context;
       if (args8["dry-run"]) {
-        const fileStats = await stat14(filePath);
+        const fileStats = await stat15(filePath);
         const originalFilename = basename4(filePath);
         writeJson({
           ok: true,