@koda-sl/baker-cli 0.382.0-dev.a410b0488 → 0.383.0-dev.2caa8a15e

package/README.md

@@ -892,7 +892,7 @@ baker ads meta insights --object act_123 --level adset \
 baker ads meta activities --account-id act_123 --days 7
 
 # "Which Page should this ad run from?"
-baker ads meta pages                                                 # Facebook Pages the user manages (tasks shows which can ADVERTISE)
+baker ads meta pages                                                 # Facebook Pages the selected ad accounts can promote
 
 # "What is the lead form collecting, and how many came in?"
 baker ads meta lead-forms --page-id 100635183117758                  # instant forms on a Page + lead counts
@@ -5474,7 +5474,7 @@ Before the deconstruct it runs a **local shot-cut pass** on the source file with
 
 A shot longer than the video model's per-clip ceiling (Seedance's 15s, passed as `video_deconstruct`'s `max_clip_s`) is split into equal **continuation sub-scenes** that share their splice boundary exactly — so a long shot is reproduced in **full** (no truncation) and joins seamlessly. Each sub-scene carries `continues_previous`.
 
-It then scaffolds the full pipeline like an **editing timeline**: each clip gets a **static-ad-grade start AND end keyframe** (`image_generate`, each with its **own `params.prompt`** regenerated from that scene's `scenes/sNN.json` — edit the scene file to change that frame, never the node prompt by hand; the slim `prompt.style.json` wired as the **shared `target_blueprint`** style reference, plus a per-element reference legend). Each keyframe is **fully recast** to the dropped `el_*` reference images. The original extracted frame is kept LAST as a **pure composition anchor** (framing / camera angle / shot size / pose) only in an ad with **nobody in it** — no deconstructed cast and no recurring person. In any ad with people, no generated frame takes a source frame as a reference: an image model copies the face of whoever is in its references, whatever the prompt says, and which scenes a person is in is itself a model's guess — so framing is carried by the frame prompt instead. **People are cast from company avatars** (`--avatar <handle>`): each recurring person grounds on that avatar's identity sheet, used as-is (no sheet is built from it); a person with no avatar gets a `[TODO]` asking for one, never for a found photo. Both keyframes feed `video_generate` (`first_frame`+`last_frame`, so Seedance interpolates real in-shot motion; ultra-detailed motion brief; duration snapped to the nearest allowed clip length). Every keyframe grounds **only on its own extracted frame + `el_*` slots** — no reference to any other generated frame — so all images render **in parallel** (no cascade). Source-frame URLs are **deduped** (each ingested once). `--frames reuse` wires the real source frame straight in.
+It then scaffolds the full pipeline like an **editing timeline**: each clip gets a **static-ad-grade start AND end keyframe** (`image_generate`, each with its **own `params.prompt`** regenerated from that scene's `scenes/sNN.json` — edit the scene file to change that frame, never the node prompt by hand; the slim `prompt.style.json` wired as the **shared `target_blueprint`** style reference, plus a per-element reference legend). Each keyframe is **fully recast** to the dropped `el_*` reference images. The original extracted frame is kept LAST as a **pure composition anchor** (framing / camera angle / shot size / pose) only in an ad with **nobody in it** — no deconstructed cast and no recurring person. In any ad with people, no generated frame takes a source frame as a reference: an image model copies the face of whoever is in its references, whatever the prompt says, and which scenes a person is in is itself a model's guess — so framing is carried by the frame prompt instead. **People are cast from company avatars** (`--avatar <handle>`): each recurring person grounds on that avatar's identity sheet, used as-is (no sheet is built from it); a person with no avatar gets a `[TODO]` asking for one, never for a found photo. Both keyframes feed `video_generate` (`first_frame`+`last_frame`, so Seedance interpolates real in-shot motion; ultra-detailed motion brief; duration snapped to the nearest allowed clip length). Every keyframe grounds **only on its own extracted frame + `el_*` slots** — no reference to any other generated frame — so all images render **in parallel** (no cascade). Source-frame URLs are **deduped** (each ingested once). `--frames reuse` wires the real source frame straight in — only on a video with nobody in it: when the deconstruct found a cast or a recurring person, reuse is refused (`frames_reuse_refused`) and every frame is generated, so the source's actors never reach the ad.
 
 **Composited scenes (split-screen / picture-in-picture / keyed presenter).** Real ads aren't always one full-frame shot — a frame can be **persistently divided** (b-roll on top, a presenter talking on the bottom) or **layer a presenter** over background footage (boxed in a corner, or green-screen keyed). The deconstruct now reports this per scene as `scene.composition` (`layout: split_screen | pip | keyed_overlay`, with one `region` per stream — each its own clean-plate frame + motion brief, the talking-head region flagged `is_presenter`). The scaffold reproduces a composited scene by building **one clip per region** (`s<i>_r0_*`, `s<i>_r1_*`, …) and compositing them with ffmpeg: a split-screen `vstack`/`hstack` (stack direction read from the region **panels**, so a top/bottom split always stacks vertically), or a picture-in-picture `overlay` of the presenter inset at its corner. A **keyed** presenter is first cut to transparency by `video_background_remove` (`s<i>_key`), then overlaid. The presenter region carries the native lip-synced voice; b-roll/render panels stay silent. To change a layout, edit `composition` in that scene's `scenes/sNN.json` and re-scaffold, or hand-edit the `s<i>_composite` ffmpeg args. Plain full-frame scenes (the default) are unaffected.
 
@@ -5528,7 +5528,7 @@ baker canvas run ./reference-ad.video.canvas.json
 |---|---|---|
 | `--out <path>` | `<video-dir>/<name>.video.canvas.json` | Where to write the canvas (composition is copied alongside). |
 | `--slug <slug>` | — | Creative slug (lowercase kebab): writes the canvas to `src/creatives/<slug>/<slug>.canvas.json` — the repo convention that attaches every run to the creative's dashboard generation history. `--out` wins over `--slug`. |
-| `--frames <mode>` | `generate` | `generate` emits ONE recast keyframe per scene (the original frame is dropped so the dropped `el_*` assets drive identity); `reuse` wires the real extracted first+last frames straight into the clips (faithful, cheaper, no recast). |
+| `--frames <mode>` | `generate` | `generate` emits ONE recast keyframe per scene (the original frame is dropped so the dropped `el_*` assets drive identity); `reuse` wires the real extracted first+last frames straight into the clips (faithful, cheaper, no recast) — only on a video with nobody in it; on one showing people it is refused and every frame is generated. |
 | `--ambient` | off | Give silent **b-roll** scenes native diegetic ambient (Seedance `generate_audio`), mixed deep under the music bed. Talking scenes already carry voice; check levels don't muddy the mix before keeping it. |
 | `--seam-dedup <mode>` | `head` | How to dedup the boundary frame two clips SHARE when a long shot is split for length (the second clip's first frame IS the first clip's last frame, so a plain concat freezes on it for a frame). `head` drops the second clip's first frame, `tail` drops the first clip's last frame, `off` keeps both. Only touches shared-frame continuation joins — a hard cut between two shots shares no frame. |
 | `--max-scenes <n>` | all source scenes | **Cost lever that reduces fidelity** — caps the deconstruct, MERGING away every scene beyond the cap (fewer cuts, lost beats). Prints a warning when set; omit it to reproduce every scene. |

package/dist/cli.js

@@ -299,6 +299,7 @@ var chatActionDraftOpViewSchema = z.object({
   targetActionId: z.string().optional(),
   tempId: z.string().optional(),
   tags: z.array(z.string()).optional(),
+  priority: actionPrioritySchema.nullable().optional(),
   reason: z.string().optional(),
   note: z.string().optional(),
   name: z.string().optional(),
@@ -28397,19 +28398,10 @@ var metaMediaCommand = defineCommand68({
 
 // src/commands/ads/meta/pages.ts
 import { defineCommand as defineCommand69 } from "citty";
-function advertisablePages(data) {
-  return data.filter((p) => Array.isArray(p.tasks) && p.tasks.includes("ADVERTISE"));
-}
 function pageSelectionHints(data) {
-  const advertisable = advertisablePages(data);
-  if (advertisable.length > 1) {
+  if (data.length > 1) {
     return [
-      "MULTIPLE Pages can run ads \u2014 do NOT guess. Confirm with the user which Page an ad should run from before passing --page to `creatives create`; a wrong Page publishes the ad under the wrong brand identity and staging will not catch it."
-    ];
-  }
-  if (data.length > 0 && advertisable.length === 0) {
-    return [
-      "NO Page can run ads \u2014 none of the connected user's Pages carry the ADVERTISE task. Ask the user to grant Advertise access to the right Page before creating an ad; a readable-but-non-advertising Page still passes staging and would fail or run under the wrong identity."
+      "MULTIPLE Pages can run ads from the selected Meta ad accounts \u2014 do NOT guess. Confirm with the user which Page an ad should run from before passing --page to `creatives create`; a wrong Page publishes the ad under the wrong brand identity and staging will not catch it."
     ];
   }
   return [];
@@ -28417,7 +28409,7 @@ function pageSelectionHints(data) {
 var pagesCommand = defineCommand69({
   meta: {
     name: "pages",
-    description: `List the Facebook Pages the connected user manages. Every Meta ad publishes from a Page, so use this to identify the Page an ad should run from. The 'tasks' field shows what the user can do on each Page (ADVERTISE means it can back ads). If exactly one Page can ADVERTISE, use it; if more than one can, confirm with the user which Page to use; if none can, ask the user to grant Advertise access \u2014 a wrong or non-advertising Page publishes under the wrong brand identity.
+    description: `List the Facebook Pages that the selected Meta ad accounts can promote. Every Meta ad publishes from a Page, so use this to identify the Page an ad should run from. If more than one Page is listed, confirm with the user which Page to use \u2014 a wrong Page publishes under the wrong brand identity.
 
 Examples:
   baker ads meta pages`
@@ -28678,7 +28670,7 @@ Discovery:
   baker ads meta accounts                                \u2014 accounts in this company's connected scope
   baker ads meta accounts --include-all                  \u2014 every account the token can see
   baker ads meta businesses                              \u2014 Business Manager IDs
-  baker ads meta pages                                   \u2014 Facebook Pages the user manages (an ad's identity)
+  baker ads meta pages                                   \u2014 Facebook Pages the selected ad accounts can promote (an ad's identity)
   baker ads meta lead-forms --page-id 100635183117758    \u2014 instant (lead) forms on a Page, with lead counts
   baker ads meta leads --form-id 987654321               \u2014 the leads one form collected (contact details masked)
 
@@ -40790,10 +40782,18 @@ function dropUncastVoices(nodes) {
     if (node && node.type === "voice_select" && !referenced.has(node.id)) nodes.splice(i, 1);
   }
 }
-function scaffoldVideoCanvas(input, elementsInput, opts) {
+function framesReuseRefusal(blueprint, elements, requested) {
+  if (requested !== "reuse") return void 0;
+  const showsPeople = (blueprint.global?.cast ?? []).length > 0 || elements.some((e) => e.type.toLowerCase() === "person");
+  if (!showsPeople) return void 0;
+  return "--frames reuse was not applied: this video shows people, and reusing its frames would put the source's actors into our ad. Every frame is generated instead, recast to the el_* references. Reuse applies only to a video with nobody in it.";
+}
+function scaffoldVideoCanvas(input, elementsInput, requested) {
   const blueprint = VideoBlueprint.parse(input);
   injectHookPhysicality(blueprint);
   const elements = RecurringElements.parse(elementsInput);
+  const reuseRefused = framesReuseRefusal(blueprint, elements, requested.frames);
+  const opts = reuseRefused ? { ...requested, frames: "generate" } : requested;
   const clock = spineClockFor(blueprint);
   const nodes = [];
   emitBlueprintIngests(opts, nodes);
@@ -40896,6 +40896,7 @@ function scaffoldVideoCanvas(input, elementsInput, opts) {
   }
   const todo = {
     ...buildVideoTodo(videoReport(input, elementsInput), overlays.length, floating.length, opts, blueprint),
+    ...reuseRefused ? { frames_reuse_refused: reuseRefused } : {},
     ...aspectRemapTodo(resolveAspect(blueprint.source?.aspect_ratio, opts.aspect, genAspectsFor(opts.videoModel))),
     model_constraints: buildModelConstraints()
   };
@@ -42021,12 +42022,6 @@ async function restoreRunSnapshot(manifest, targetDir, opts = {}) {
   return { canvasPath: path14.resolve(resolvedTarget, manifest.canvas.path), restored, upToDate };
 }
 
-// src/commands/canvas/unchanged-run.ts
-function unchangedRunHint(runId, stats) {
-  if (stats.total_nodes === 0 || stats.cached_nodes < stats.total_nodes) return null;
-  return `Nothing changed since the last render: every step came from the cache, so this run is no new version. To read what a render produced, use \`baker canvas inspect ${runId}\` instead of re-running; re-run a canvas only after editing it or the files it reads.`;
-}
-
 // src/engine/nodes/local/lib/render-review.ts
 import { execFile as execFile2 } from "child_process";
 import { mkdtemp, readdir as readdir6, readFile as readFile11, rm as rm5 } from "fs/promises";
@@ -42910,8 +42905,6 @@ ${describeRewrites(healed.rewrites)}
         `Anything about the mark or the closing card is a composition fix, not a re-render: both are drawn as HTML by the caption composition, so editing it and re-running re-renders only that pass \u2014 the clips are cached and no generation is paid for again. Fix them, then re-run: edit the \`show\` of the scene files named above \u2014 a shot with a person handling equipment is where limbs and objects go wrong, so widen it, move the hands out of frame, or drop the person from that beat. Re-running re-renders only the scenes you edited; everything else comes from the cache (this run: ${result.stats?.cached_nodes ?? 0} of ${result.stats?.total_nodes ?? 0} nodes were cached).`
       ] : []
     ] : [];
-    const unchanged = result.stats ? unchangedRunHint(result.run_id, result.stats) : null;
-    if (unchanged) hints2.push(unchanged);
     if (poster) {
       await poster.flush(
         // `review` is null only when nobody looked (no key, no ffmpeg, unreadable video);
@@ -45830,6 +45823,12 @@ function videoDefinitionDescription(blueprint) {
   const product = g.branding?.product;
   return typeof product === "string" && product.trim() ? product.trim() : void 0;
 }
+function framesModeReport(canvas, requested) {
+  const todo = canvas.metadata?.todo;
+  const mode = typeof todo?.frames_mode === "string" ? todo.frames_mode : requested;
+  const refused = typeof todo?.frames_reuse_refused === "string" ? todo.frames_reuse_refused : void 0;
+  return { frames_mode: mode, ...refused ? { frames_reuse_refused: refused } : {} };
+}
 async function materializeReferenceVideo(fileArg2) {
   if (!/^https?:\/\//i.test(fileArg2)) return path26.resolve(fileArg2);
   let bytes;
@@ -45987,7 +45986,10 @@ var scaffoldVideoCommand = defineCommand116({
       type: "string",
       description: "Creative slug \u2014 writes the canvas to src/creatives/<slug>/<slug>.canvas.json (repo convention)"
     },
-    frames: { type: "string", description: '"generate" (default, anchored regen) or "reuse" (wire real frames in)' },
+    frames: {
+      type: "string",
+      description: `"generate" (default, every frame regenerated for our ad) or "reuse" (wire the source's real frames in). Reuse applies only to a video with nobody in it \u2014 on a video showing people it is refused and every frame is generated, so the source's actors never reach our ad.`
+    },
     ambient: {
       type: "boolean",
       description: "Give silent b-roll scenes native diegetic ambient mixed deep under the music bed (off by default)"
@@ -46190,6 +46192,7 @@ var scaffoldVideoCommand = defineCommand116({
             error: { code: "validation", issues: validation.issues },
             canvas_path: outPath,
             prompt_path: blueprintPath,
+            ...framesModeReport(canvas, frames),
             note: "The canvas WAS written despite the blocking issue(s) \u2014 fix them in place, then `baker canvas validate` before running.",
             ...uncastHint
           },
@@ -46241,7 +46244,7 @@ var scaffoldVideoCommand = defineCommand116({
           composition_dir: compositionDest,
           output: canvas.output,
           ...uncastHint,
-          frames_mode: frames,
+          ...framesModeReport(canvas, frames),
           models: {
             deconstruct: deconstructModel,
             select: selectModel,