@koda-sl/baker-cli 0.191.0 → 0.192.1

package/dist/cli.js

@@ -47,7 +47,7 @@ import {
   toModelSafeImage,
   ulid,
   validateCanvasDeep
-} from "./chunk-OWDTSPY4.js";
+} from "./chunk-ISZWNERZ.js";
 import {
   csvOrJson,
   daysAgoIso,
@@ -11142,7 +11142,7 @@ var assetsUpdateCommand = defineCommand30({
 var assetsCommand = defineCommand30({
   meta: {
     name: "assets",
-    description: "Stage asset create/update + asset-link attach/detach (via --file). For a high-performance Search campaign add \u22654 sitelinks, \u22653 callouts, and \u22651 structured snippet per campaign."
+    description: "Stage asset create/update + asset-link attach/detach (via --file). For a high-performance Search campaign add \u22654 sitelinks, \u22653 callouts, \u22651 structured snippet, and \u22653 image assets per campaign. Build the images with the creative-canvas-ads flow (platform preset `google_search`) \u2014 product or team in action, never stock, and never a logo on a plain background."
   },
   subCommands: {
     create: fileCreateCommand(
@@ -20906,7 +20906,7 @@ function todoPath(el, label) {
   const t = el.type.toLowerCase();
   const fresh = t === "person" || t === "animal" || t === "location" ? " [SOURCE FRESH \u2014 a DIFFERENT person/animal/set than the original ad; do not reuse the source's individual]" : "";
   const same = el.same_as ? ` [SAME INDIVIDUAL as ${el.same_as} \u2014 a different wardrobe/look of the same person; reuse that cast person, change only the outfit]` : "";
-  return `[TODO: drop one real source image for ${label} (${el.type})${desc}${expr} \u2014 reused across every frame it appears in${fresh}${same}]`;
+  return `[TODO: drop one real source image for ${label} (${el.type})${desc}${expr} \u2014 reused across every frame it appears in${fresh}${same}; use a path relative to this canvas file, never an absolute path]`;
 }
 function buildElementSlots(elements) {
   const usedIds = /* @__PURE__ */ new Set([
@@ -21753,7 +21753,7 @@ function emitScreenScene(i, scene, lengths, out, surfaceIngests, nodes, clips) {
       type: "ingest",
       params: {
         source: "path",
-        path: `[TODO: supply the REAL screen for "${label}" \u2014 NEVER AI-generate a UI. Capture a clean, text-free screenshot with \`baker images screenshot https://<brand-domain>/<path>\` (image-library skill); spoken/overlay text rides the overlay layer, not the screenshot]`,
+        path: `[TODO: supply the REAL screen for "${label}" \u2014 NEVER AI-generate a UI. Capture a clean, text-free screenshot with \`baker images screenshot https://<brand-domain>/<path>\` (image-library skill); spoken/overlay text rides the overlay layer, not the screenshot. Wire it in as a path relative to this canvas file, never an absolute path]`,
         expect: "image"
       }
     });
@@ -21805,7 +21805,7 @@ function emitGraphicScene(i, scene, lengths, out, surfaceIngests, nodes, clips)
       type: "ingest",
       params: {
         source: "path",
-        path: `[TODO: supply the REAL designed graphic for "${label}" \u2014 NEVER AI-generate a designed panel (its typography/wordmarks garble). Rebuild it as brand HTML in video-overlay-composition (the scene's text/logos are already seeded there), export the design from the brand kit, or source stock art; text rides the overlay layer, not this plate]`,
+        path: `[TODO: supply the REAL designed graphic for "${label}" \u2014 NEVER AI-generate a designed panel (its typography/wordmarks garble). Rebuild it as brand HTML in video-overlay-composition (the scene's text/logos are already seeded there), export the design from the brand kit, or source stock art; text rides the overlay layer, not this plate. Wire it in as a path relative to this canvas file, never an absolute path]`,
         expect: "image"
       }
     });
@@ -24082,6 +24082,10 @@ function mimeForFile(filePath) {
 function toPosix(p) {
   return p.split(path12.sep).join("/");
 }
+function isInside(dir, target) {
+  const rel = path12.relative(dir, target);
+  return !rel.startsWith("..") && !path12.isAbsolute(rel);
+}
 function localPathRefsFromCanvas(parsed) {
   const nodes = parsed?.nodes;
   if (!Array.isArray(nodes)) return [];
@@ -24120,6 +24124,13 @@ async function uploadRunSnapshot(client, opts) {
     const skipped = [];
     for (const refPath of await sourceRefsToSnapshot(canvasDir, opts.parsed)) {
       const abs = path12.isAbsolute(refPath) ? refPath : path12.resolve(canvasDir, refPath);
+      if (!isInside(canvasDir, abs)) {
+        skipped.push({
+          path: toPosix(path12.relative(canvasDir, abs)),
+          reason: "outside the creative folder \u2014 read from the workspace on rerun"
+        });
+        continue;
+      }
       let st;
       try {
         st = await stat2(abs);
@@ -24776,7 +24787,7 @@ function todoPath2(el, label) {
   const expr = el.expression ? `, with a ${el.expression} expression` : "";
   const t = el.type.toLowerCase();
   const lock = t === "person" || t === "animal" ? " \u2014 REQUIRED: this is the emotional hero; ground it in a real reference (image-library / video-library / Pinterest), do not delete this slot and free-generate" : "";
-  return `[TODO: drop a real image for ${label} (${el.type})${desc}${expr}${lock}]`;
+  return `[TODO: drop a real image for ${label} (${el.type})${desc}${expr}${lock} \u2014 replace with a path relative to this canvas file, never an absolute path]`;
 }
 function sheetSubjectDescription(el) {
   const desc = el.description ?? `the ${el.type}`;
@@ -24867,7 +24878,11 @@ function scaffoldStaticAd(input, elementsInput, opts) {
     nodes.push({
       id: "brandfont",
       type: "ingest",
-      params: { source: "path", path: "[TODO: path to brand font .otf/.ttf]", expect: "font" }
+      params: {
+        source: "path",
+        path: "[TODO: brand font .otf/.ttf, as a path relative to this canvas file (e.g. ../../brand/fonts/Brand-Bold.otf) \u2014 never an absolute path]",
+        expect: "font"
+      }
     });
     nodes.push({
       id: "type_ref",
@@ -35559,20 +35574,29 @@ async function captureMobileShots(args) {
     await mobile.context.close();
   }
 }
+var MOTION_CONCURRENCY = 3;
 async function captureMotionTakes(args) {
   const { browser, sections, sectionsDir, outDir, pageUrl, log } = args;
   const moving = sections.filter((section) => isWorthFilming(section.motion));
   if (moving.length === 0) return;
   log(`filming ${moving.length} moving sections`);
-  for (const section of moving) {
-    const take = await captureMotionTake(browser, pageUrl, section.selector);
-    if (!take) continue;
+  const filmOne = async (section) => {
+    const take = await captureMotionTake(browser, pageUrl, section.selector).catch(() => null);
+    if (!take) return;
     const dir = path31.join(sectionsDir, String(section.index).padStart(2, "0"));
     const file = path31.join(dir, "motion-filmstrip.png");
     await writeFile15(file, take.filmstrip);
     section.motionFilmstrip = path31.relative(outDir, file);
     log(`  [${section.index}] ${section.motion.summary}`);
-  }
+  };
+  const queue = [...moving];
+  await Promise.all(
+    Array.from({ length: Math.min(MOTION_CONCURRENCY, queue.length) }, async () => {
+      for (let next = queue.shift(); next; next = queue.shift()) {
+        await filmOne(next);
+      }
+    })
+  );
 }
 async function scrapeLanding(options) {
   const timeoutMs = options.timeoutMs ?? 45e3;
@@ -35666,6 +35690,54 @@ async function scrapeLanding(options) {
   }
 }
 
+// src/engine/landing-library/unreachable.ts
+var NET_ERROR_REASONS = [
+  {
+    prefixes: ["ERR_NAME_NOT_RESOLVED", "ERR_NAME_RESOLUTION_FAILED"],
+    message: "We couldn't find this site \u2014 check the address is spelled right."
+  },
+  {
+    prefixes: ["ERR_SSL_", "ERR_CERT_", "ERR_TLS_", "ERR_BAD_SSL_"],
+    message: "This site's secure connection is broken, so no browser can open it \u2014 including ours. That's a fault on their side, not something you can fix here."
+  },
+  {
+    prefixes: ["ERR_TOO_MANY_REDIRECTS"],
+    message: "This site kept redirecting us and never settled on a page."
+  },
+  {
+    prefixes: ["ERR_BLOCKED_BY_", "ERR_ACCESS_DENIED"],
+    message: "This site refused our visit, so we couldn't read the page."
+  },
+  {
+    prefixes: [
+      "ERR_CONNECTION_",
+      "ERR_ADDRESS_UNREACHABLE",
+      "ERR_EMPTY_RESPONSE",
+      "ERR_NETWORK_CHANGED",
+      "ERR_INTERNET_DISCONNECTED",
+      "ERR_TIMED_OUT"
+    ],
+    message: "This site didn't respond, so we couldn't read the page. It may be down \u2014 try again later."
+  }
+];
+var GENERIC_UNREACHABLE = "We couldn't open this site, so there was no page to study.";
+var TOO_SLOW = "This site took too long to load, so we stopped waiting. Try again later.";
+function describeUnreachableSite(error) {
+  const raw = error instanceof Error ? error.message : typeof error === "string" ? error : "";
+  if (!raw) return null;
+  const netError = /net::(ERR_[A-Z0-9_]+)/.exec(raw)?.[1];
+  if (netError) {
+    const reason = NET_ERROR_REASONS.find(
+      (candidate) => candidate.prefixes.some((prefix) => netError.startsWith(prefix))
+    );
+    return { code: "SITE_UNREACHABLE", message: reason?.message ?? GENERIC_UNREACHABLE };
+  }
+  if (/Timeout \d+ms exceeded/.test(raw) || raw.includes("page.goto: Timeout")) {
+    return { code: "SITE_UNREACHABLE", message: TOO_SLOW };
+  }
+  return null;
+}
+
 // src/commands/landing/inspiration/scrape.ts
 async function recordCapture(manifest, outDir) {
   const consultedAt = (/* @__PURE__ */ new Date()).toISOString();
@@ -35701,7 +35773,7 @@ registerSchema({
     },
     motion: {
       type: "boolean",
-      description: "Film sections that move. `--no-motion` to skip \u2014 roughly halves runtime",
+      description: "Film sections that move. `--no-motion` to skip \u2014 the single biggest lever on runtime",
       required: false
     },
     report: { type: "boolean", description: "Write report.html. `--no-report` to skip", required: false }
@@ -35773,6 +35845,11 @@ var scrapeCommand = defineCommand147({
         writeJson({ ok: false, error: { code: error.code, message: error.message } });
         process.exit(1);
       }
+      const unreachable = describeUnreachableSite(error);
+      if (unreachable) {
+        writeJson({ ok: false, error: { code: unreachable.code, message: unreachable.message } });
+        process.exit(1);
+      }
       reportError(error);
     }
   }