@koda-sl/baker-cli 0.369.0 → 0.371.0

package/dist/cli.js

@@ -55,6 +55,7 @@ import {
   promptMaxLength,
   requireCredentialsFromEnv,
   resolveConcurrency,
+  runFailureCode,
   runYtDlp,
   sha256Hex,
   spineInputFlags,
@@ -69,7 +70,7 @@ import {
   validateCanvasDeep,
   withoutCastName,
   ytDlpBlockSignal
-} from "./chunk-AI4HWQVN.js";
+} from "./chunk-3NIBJ3GL.js";
 import {
   csvOrJson,
   daysAgoIso,
@@ -12051,6 +12052,41 @@ function adCopyHints(payload) {
   return tells.slice(0, MAX_HINTS).map(hintFor);
 }
 
+// src/commands/ads/google/copy-list-usage.ts
+var AD_COPY_LIST_USAGE = `repeat the flag once per item, or pass one comma-separated list and wrap an item that contains a comma in double quotes: '"Fast, free quotes",Book today'. A flag given once is read as a list, so a lone item with a comma needs its double quotes too: '"Fast, free quotes"'`;
+var COPY_LIST_FLAG = {
+  headlines: "--headlines",
+  descriptions: "--descriptions",
+  longHeadlines: "--long-headlines"
+};
+function valueAt(root, path48) {
+  let node = root;
+  for (const key of path48) {
+    if (node === null || typeof node !== "object") return void 0;
+    node = node[key];
+  }
+  return node;
+}
+function countRefusal(op, issue) {
+  if (issue.code !== "too_big" && issue.code !== "too_small" || issue.origin !== "array") return void 0;
+  const flag = COPY_LIST_FLAG[String(issue.path.at(-1))];
+  if (!flag) return void 0;
+  const value = valueAt(op, issue.path);
+  const count2 = Array.isArray(value) ? value.length : void 0;
+  return issue.code === "too_big" ? { flag, count: count2, bound: `at most ${issue.maximum}`, tooMany: true } : { flag, count: count2, bound: `at least ${issue.minimum}`, tooMany: false };
+}
+function copyListCountFix(op, issues) {
+  const refusals = issues.map((issue) => countRefusal(op, issue));
+  if (refusals.length === 0 || refusals.some((refusal) => refusal === void 0)) return void 0;
+  const counted = refusals.filter((refusal) => refusal !== void 0);
+  const limits = counted.map(({ flag, count: count2, bound }) => `${flag} has ${count2 ?? "the wrong number of"} items; this ad takes ${bound}.`).join(" ");
+  const commaCause = counted.some((refusal) => refusal.tooMany) ? ` If you meant fewer items, the copy was cut at its commas: a flag given once is read as a comma-separated list, so an item with a comma in it ("Fast, free quotes", a decimal like "4,5/5") becomes two. To keep a comma, ${AD_COPY_LIST_USAGE} \u2014 or put the content in --file as JSON.` : "";
+  return {
+    action: "fix_copy_list_count",
+    explanation: `${limits}${commaCause} ${commaCause ? "Otherwise add" : "Add"} or drop items to fit the limit, then stage again; the field names are right.`
+  };
+}
+
 // src/commands/ads/google/draft-status.ts
 var OPERATION_LABELS = {
   create: "Creating",
@@ -12371,7 +12407,7 @@ async function stageGoogleOp(raw, hints2) {
         code: "VALIDATION_ERROR",
         message: fields.map((f) => f.path ? `${f.path}: ${f.message}` : f.message).join("; "),
         fields,
-        fix: capabilityFix("google-ads")
+        fix: copyListCountFix(raw, preflight.error.issues) ?? capabilityFix("google-ads")
       }
     });
     process.exit(1);
@@ -12407,7 +12443,7 @@ async function stageGoogleOps(rawOps, hints2) {
           code: "VALIDATION_ERROR",
           message: fields.map((f) => `${f.path}: ${f.message}`).join("; "),
           fields,
-          fix: capabilityFix("google-ads")
+          fix: copyListCountFix(raw, preflight.error.issues) ?? capabilityFix("google-ads")
         }
       });
       process.exit(1);
@@ -17858,7 +17894,6 @@ import { defineCommand as defineCommand31 } from "citty";
 
 // src/commands/ads/google/ad-copy-flags.ts
 var AD_COPY_LIST_FLAGS = ["headlines", "descriptions", "long-headlines"];
-var AD_COPY_LIST_USAGE = `repeat the flag once per item, or pass one comma-separated list and wrap an item that contains a comma in double quotes: '"Fast, free quotes",Book today'. A flag given once is read as a list, so a lone item with a comma needs its double quotes too: '"Fast, free quotes"'`;
 function skipBlanks(value, at) {
   let i = at;
   while (value[i] === " " || value[i] === "	") i++;
@@ -42434,14 +42469,14 @@ ${describeRewrites(healed.rewrites)}
       const failures = e.failures.map((f) => ({ node_id: f.nodeId, message: describeFailureReason(f.reason) }));
       if (poster) await poster.flush(failedPayload(e.message));
       process.stderr.write(
-        `${JSON.stringify({ ok: false, error: { code: "runtime", message: e.message, failures } }, null, 2)}
+        `${JSON.stringify({ ok: false, error: { code: runFailureCode(e), message: e.message, failures } }, null, 2)}
 `
       );
       process.exit(1);
     }
     const msg = e instanceof Error ? e.message : String(e);
     if (poster) await poster.flush(failedPayload(msg));
-    process.stderr.write(`${JSON.stringify({ ok: false, error: { code: "runtime", message: msg } }, null, 2)}
+    process.stderr.write(`${JSON.stringify({ ok: false, error: { code: runFailureCode(e), message: msg } }, null, 2)}
 `);
     process.exit(1);
   }
@@ -43278,6 +43313,15 @@ function resolvePlacements(placementsFlag, definitionPlatform) {
     message: `--placements "${flag}" is not a valid placement set. Use one of: ${AD_FORMAT_PLATFORMS.join(", ")}, or "none" for a single base ad.`
   };
 }
+async function validateScaffoldedStaticAd(canvas, canvasDir) {
+  const validation = await validateCanvasDeep(resolveRelativeCanvasPaths(canvas, canvasDir), defaultRegistry());
+  if (validation.ok) return { blocking: [], hints: [], estimatedCredits: validation.estimatedCredits };
+  const overCap = validation.issues.filter((i) => i.code === "PROMPT_OVER_MODEL_LIMIT");
+  return {
+    blocking: validation.issues.filter((i) => i.code !== "PROMPT_OVER_MODEL_LIMIT"),
+    hints: overCap.map((i) => `${i.code}: ${i.message} Trim prompt.json, then \`baker canvas validate\`.`)
+  };
+}
 function regionCount(layout) {
   const regions = layout?.regions;
   return Array.isArray(regions) ? regions.length : 0;
@@ -43469,10 +43513,10 @@ var scaffoldStaticAdCommand = defineCommand114({
     } catch (e) {
       return fail3("scaffold", e instanceof Error ? e.message : String(e));
     }
-    const validation = await validateCanvasDeep(canvas, defaultRegistry());
-    if (!validation.ok) {
+    const validation = await validateScaffoldedStaticAd(canvas, path21.dirname(outPath));
+    if (validation.blocking.length > 0) {
       process.stderr.write(
-        `${JSON.stringify({ ok: false, error: { code: "validation", issues: validation.issues } }, null, 2)}
+        `${JSON.stringify({ ok: false, error: { code: "validation", issues: validation.blocking } }, null, 2)}
 `
       );
       process.exit(2);
@@ -43506,6 +43550,12 @@ var scaffoldStaticAdCommand = defineCommand114({
         sourceSha: await computeSourceSha(outPath)
       });
     }
+    const hints2 = [
+      ...uncast.length ? [
+        `NOT CAST: ${uncast.map((h) => `\`${h}\``).join(", ")} \u2014 the element selection found fewer people than avatars passed, so ${uncast.length === 1 ? "this avatar plays" : "these avatars play"} nobody in this ad. Tell the user who is cast and who is not.`
+      ] : [],
+      ...validation.hints
+    ];
     process.stdout.write(
       `${JSON.stringify(
         {
@@ -43515,11 +43565,7 @@ var scaffoldStaticAdCommand = defineCommand114({
           definition_path: definitionPath ?? void 0,
           source_reference: durableSourceUrl ?? void 0,
           output: canvas.output,
-          ...uncast.length ? {
-            hints: [
-              `NOT CAST: ${uncast.map((h) => `\`${h}\``).join(", ")} \u2014 the element selection found fewer people than avatars passed, so ${uncast.length === 1 ? "this avatar plays" : "these avatars play"} nobody in this ad. Tell the user who is cast and who is not.`
-            ]
-          } : {},
+          ...hints2.length ? { hints: hints2 } : {},
           models: { describe: describeModel, select: selectModel, layout: layoutModel, gen: opts.genModel },
           aspect_ratio: report.aspect_ratio,
           placements: report.placements,
@@ -45804,7 +45850,7 @@ import { defineCommand as defineCommand118 } from "citty";
 var validateCommand = defineCommand118({
   meta: {
     name: "validate",
-    description: "Validate a canvas JSON file (no execution). Includes a per-node cost preview and runs each node's deep validators (composition meta checks for hyperframe_render/_snapshot)."
+    description: "Validate a canvas JSON file (no execution). Includes a per-node cost preview and runs each node's deep validators (composition meta checks for hyperframe_render/_snapshot), and measures each generator prompt with its {{slots}} filled against the model's character cap (PROMPT_OVER_MODEL_LIMIT)."
   },
   args: { file: { type: "positional", required: true, description: "Path to canvas JSON" } },
   async run({ args: args5 }) {