@koda-sl/baker-cli 0.169.0 → 0.169.2

package/dist/cli.js

@@ -7229,6 +7229,15 @@ function handleGoogleError(err) {
 function noopHints(data) {
   return data.staged ? [] : [`NOT STAGED \u2014 ${data.reason}. Nothing to publish for this change; say so rather than reporting it as done.`];
 }
+function skippedHints(skipped, total) {
+  if (skipped.length === 0) {
+    return [];
+  }
+  const detail = skipped.map((s) => `${s.summary} \u2014 ${s.reason}`).join("; ");
+  return [
+    `${skipped.length} of ${total} op(s) NOT STAGED: ${detail}. Don't re-stage them; tell the user which changes couldn't be made and why, then carry on with the rest.`
+  ];
+}
 async function stageGoogleOp(raw, hints) {
   const preflight = googleDraftOpInputSchema.safeParse(raw);
   if (!preflight.success) {
@@ -7285,14 +7294,7 @@ async function stageGoogleOps(rawOps, hints) {
   try {
     const chatId = requireChatId();
     const response = await apiPost("/api/ads/google/draft/stage-batch", { chatId, ops });
-    const skipped = response.data.skipped ?? [];
-    const allHints = [
-      ...skipped.length > 0 ? [
-        `${skipped.length} of ${ops.length} op(s) NOT STAGED \u2014 already in the requested state: ${skipped.map((s) => s.reason).join("; ")}`
-      ] : [],
-      ...hints ?? [],
-      ...adCopyHints(ops)
-    ];
+    const allHints = [...skippedHints(response.data.skipped ?? [], ops.length), ...hints ?? [], ...adCopyHints(ops)];
     writeJsonEnvelope(allHints.length > 0 ? { ...response, hints: allHints } : response);
   } catch (err) {
     handleGoogleError(err);
@@ -15954,10 +15956,27 @@ var metaMediaCommand = defineCommand64({
 
 // src/commands/ads/meta/pages.ts
 import { defineCommand as defineCommand65 } 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) {
+    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."
+    ];
+  }
+  return [];
+}
 var pagesCommand = defineCommand65({
   meta: {
     name: "pages",
-    description: `List the Facebook Pages the connected user manages. Every Meta ad publishes from a Page, so use this to find the Page id an ad should run from instead of asking the user for it. The 'tasks' field shows what the user can do on each Page (ADVERTISE means it can back ads).
+    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.
 
 Examples:
   baker ads meta pages`
@@ -15976,7 +15995,8 @@ Examples:
         writeAdsOutput(data, fmt);
         return;
       }
-      writeAdsJson({ ok: true, data });
+      const hints = pageSelectionHints(data);
+      writeJsonEnvelope(hints.length > 0 ? { ok: true, data, hints } : { ok: true, data });
     } catch (err) {
       handleMetaError(err);
     }