@koda-sl/baker-cli 0.343.0-dev.acf96fa0f → 0.343.0-dev.c28f2e0b3

package/README.md

@@ -609,11 +609,16 @@ baker ads google keywords update <criterion-id> --customer-id 1234567890 --clear
 baker ads google ads create --customer-id 1234567890 --ad-group-ref g_temp_<adgroup> \
   --headlines "Fast Widgets,Buy Online,Free Shipping" --descriptions "Best widgets around.,Ships tomorrow." \
   --final-url https://example.com
+# Copy with commas in it — repeat the flag once per item (or quote the item: '"Fast, free quotes",Book today')
+baker ads google ads create --customer-id 1234567890 --ad-group-ref g_temp_<adgroup> \
+  --headlines "Fast Widgets,Buy Online,Free Shipping" \
+  --descriptions "Calls, messages and bookings in one place." --descriptions "Ships tomorrow, free." \
+  --final-url https://example.com
 # Responsive display ad — images are fields on the ad; register each image as an asset first (returns a g_temp_* ref)
 baker ads google assets create --customer-id 1234567890 --file '{"type":"image","imageId":"<baker_image_id>","name":"Hero"}'
 baker ads google ads create --customer-id 1234567890 --ad-group-ref g_temp_<adgroup> --format responsiveDisplay \
   --headlines "Come back and save" --long-headline "Finish signing up and get 20% off" \
-  --descriptions "Fast setup, no card required" --business-name "Acme" --final-url https://example.com \
+  --descriptions '"Fast setup, no card required"' --business-name "Acme" --final-url https://example.com \
   --marketing-images g_temp_<img_landscape> --square-marketing-images g_temp_<img_square> --logo-images g_temp_<img_logo>
 # Video ad — the YouTube video is its own asset first, then referenced by --video-assets
 baker ads google assets create --customer-id 1234567890 --file '{"type":"youtubeVideo","youtubeVideoId":"dQw4w9WgXcQ"}'

package/canvas/end-card-composition/meta.json

@@ -8,11 +8,7 @@
   "default_duration": 3,
   "inputs": {},
   "params": {
-    "logo_svg": {
-      "kind": "string",
-      "default": "",
-      "description": "Raw inline <svg>…</svg> wordmark with fill set to text_color (optional)."
-    },
+    "logo_svg": { "kind": "string", "default": "", "description": "Raw inline <svg>…</svg> wordmark with fill set to text_color (optional)." },
     "headline": { "kind": "string", "required": true, "description": "Closing line." },
     "cta_text": { "kind": "string", "default": "", "description": "Call-to-action pill label (optional)." },
     "bg_color": { "kind": "color", "default": "#0b1020" },

package/canvas/feature-reveal-composition/meta.json

@@ -9,20 +9,10 @@
   "inputs": {},
   "params": {
     "title": { "kind": "string", "default": "", "description": "Optional heading above the list." },
-    "features": {
-      "kind": "json",
-      "required": true,
-      "description": "Array of 1–6 short labels, e.g. [\"No setup fees\", \"Cancel anytime\"]."
-    },
+    "features": { "kind": "json", "required": true, "description": "Array of 1–6 short labels, e.g. [\"No setup fees\", \"Cancel anytime\"]." },
     "text_color": { "kind": "color", "default": "#0b1020" },
     "accent_color": { "kind": "color", "default": "#1f9d57", "description": "Check-circle fill." },
     "check_color": { "kind": "color", "default": "#ffffff", "description": "The check mark inside the circle." },
-    "stagger_s": {
-      "kind": "number",
-      "default": 0.18,
-      "min": 0.02,
-      "max": 1,
-      "description": "Seconds between each item."
-    }
+    "stagger_s": { "kind": "number", "default": 0.18, "min": 0.02, "max": 1, "description": "Seconds between each item." }
   }
 }

package/canvas/lower-third-composition/meta.json

@@ -13,12 +13,6 @@
     "bar_color": { "kind": "color", "default": "#0b1020" },
     "text_color": { "kind": "color", "default": "#ffffff" },
     "accent_color": { "kind": "color", "default": "#5b8cff", "description": "The leading accent block." },
-    "from_bottom_px": {
-      "kind": "integer",
-      "default": 220,
-      "min": 0,
-      "max": 1600,
-      "description": "Distance of the bar from the bottom edge."
-    }
+    "from_bottom_px": { "kind": "integer", "default": 220, "min": 0, "max": 1600, "description": "Distance of the bar from the bottom edge." }
   }
 }

package/canvas/video-call-composition/meta.json

@@ -15,34 +15,12 @@
     }
   },
   "params": {
-    "headline": {
-      "kind": "string",
-      "default": "",
-      "description": "Big line on the content stage (what the presenter is showing)."
-    },
-    "claim": {
-      "kind": "string",
-      "default": "",
-      "description": "Short proof/claim on the floating glass card (optional)."
-    },
+    "headline": { "kind": "string", "default": "", "description": "Big line on the content stage (what the presenter is showing)." },
+    "claim": { "kind": "string", "default": "", "description": "Short proof/claim on the floating glass card (optional)." },
     "caption": { "kind": "string", "default": "", "description": "A caption line in the lower band (optional)." },
-    "pip_at_s": {
-      "kind": "number",
-      "default": 2,
-      "min": 0,
-      "max": 30,
-      "description": "When the presenter shrinks to the corner pill."
-    },
-    "bg_color": {
-      "kind": "color",
-      "default": "#0b1020",
-      "description": "Content-stage background (behind the presenter)."
-    },
+    "pip_at_s": { "kind": "number", "default": 2, "min": 0, "max": 30, "description": "When the presenter shrinks to the corner pill." },
+    "bg_color": { "kind": "color", "default": "#0b1020", "description": "Content-stage background (behind the presenter)." },
     "text_color": { "kind": "color", "default": "#ffffff" },
-    "accent_color": {
-      "kind": "color",
-      "default": "#5b8cff",
-      "description": "Headline accent + caption underline + pill ring tint."
-    }
+    "accent_color": { "kind": "color", "default": "#5b8cff", "description": "Headline accent + caption underline + pill ring tint." }
   }
 }

package/dist/cli.js

@@ -17043,6 +17043,101 @@ Examples:
 // src/commands/ads/google/write-commands.ts
 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++;
+  return i;
+}
+function readQuotedField(value, open, flag) {
+  let text2 = "";
+  let j = open + 1;
+  while (!(value[j] === '"' && value[j + 1] !== '"')) {
+    if (j >= value.length) {
+      failWriteValidation(`--${flag} has a quote with no closing quote: ${value} \u2014 ${AD_COPY_LIST_USAGE}`);
+    }
+    text2 += value[j];
+    j += value[j] === '"' ? 2 : 1;
+  }
+  const after = skipBlanks(value, j + 1);
+  if (after < value.length && value[after] !== ",") {
+    failWriteValidation(
+      `--${flag}: text follows the closing quote of "${text2}" \u2014 a quoted item must end at a comma. To put a quote inside an item, double it ("") or ${AD_COPY_LIST_USAGE}`
+    );
+  }
+  return { text: text2, next: after + 1 };
+}
+function splitQuotedList(value, flag) {
+  const fields = [];
+  let i = 0;
+  while (i <= value.length) {
+    const start = skipBlanks(value, i);
+    if (value[start] === '"') {
+      const { text: text2, next } = readQuotedField(value, start, flag);
+      fields.push({ text: text2, quoted: true });
+      i = next;
+    } else {
+      const comma = value.indexOf(",", start);
+      const end = comma === -1 ? value.length : comma;
+      fields.push({ text: value.slice(start, end), quoted: false });
+      i = end + 1;
+    }
+  }
+  const several = fields.length > 1;
+  return fields.map((field) => ({ text: field.text.trim(), splitAtComma: several && !field.quoted }));
+}
+function jsonStringArray(value) {
+  if (!value.startsWith("[")) return void 0;
+  try {
+    const parsed = JSON.parse(value);
+    return Array.isArray(parsed) && parsed.every((v) => typeof v === "string") ? parsed : void 0;
+  } catch {
+    return void 0;
+  }
+}
+function parseCopyList(value, flag) {
+  let items;
+  if (Array.isArray(value)) {
+    items = value.filter((v) => typeof v === "string").map((v) => ({ text: v.trim(), splitAtComma: false }));
+  } else if (typeof value === "string") {
+    const trimmed = value.trim();
+    const json = jsonStringArray(trimmed);
+    items = json ? json.map((v) => ({ text: v.trim(), splitAtComma: false })) : splitQuotedList(trimmed, flag);
+  } else {
+    return void 0;
+  }
+  const kept = items.filter((item) => item.text.length > 0);
+  return kept.length > 0 ? kept : void 0;
+}
+function copyListFlag(value, flag) {
+  return parseCopyList(value, flag)?.map((item) => item.text);
+}
+function withRepeatedCopyFlags(args5, rawArgs) {
+  const out = { ...args5 };
+  for (const flag of AD_COPY_LIST_FLAGS) {
+    const values = repeatedFlagValues(rawArgs ?? [], flag);
+    if (values.length > 1) {
+      out[flag] = values;
+    }
+  }
+  return out;
+}
+function copyListSplitHints(args5) {
+  const hints2 = [];
+  for (const flag of AD_COPY_LIST_FLAGS) {
+    const items = parseCopyList(args5[flag], flag) ?? [];
+    const fragments = items.filter((item, index) => index > 0 && item.splitAtComma && new RegExp("^\\p{Ll}", "u").test(item.text));
+    if (fragments.length > 0) {
+      hints2.push(
+        `--${flag}: ${fragments.map((f) => `"${f.text}"`).join(", ")} starts in lowercase \u2014 it looks like the end of one item cut at a comma. Check the staged copy; if it was one item, fix this op in place with \`draft amend <ref>\` (the ref this response returned) rather than staging it again, which would leave the cut copy staged too \u2014 ${AD_COPY_LIST_USAGE}.`
+      );
+    }
+  }
+  return hints2;
+}
+
 // src/commands/ads/google/conversion-upload-command.ts
 import { readFileSync as readFileSync4 } from "fs";
 import { defineCommand as defineCommand30 } from "citty";
@@ -18098,10 +18193,10 @@ var keywordListsCommand = defineCommand31({
   }
 });
 function rsaContentFromFlags(args5) {
-  const headlines = listFlag(args5.headlines);
-  const descriptions = listFlag(args5.descriptions);
+  const headlines = copyListFlag(args5.headlines, "headlines");
+  const descriptions = copyListFlag(args5.descriptions, "descriptions");
   if (!headlines || !descriptions) {
-    failWriteValidation("responsive search ads need --headlines and --descriptions (comma-separated), or use --file");
+    failWriteValidation("responsive search ads need --headlines and --descriptions, or use --file");
   }
   const finalUrls = listFlag(args5["final-url"]);
   if (!finalUrls) {
@@ -18121,11 +18216,11 @@ function rdaContentFromFlags(args5, base) {
     format: "responsiveDisplay",
     ...base !== null && typeof base === "object" && !Array.isArray(base) ? base : {}
   };
-  const headlines = listFlag(args5.headlines);
+  const headlines = copyListFlag(args5.headlines, "headlines");
   if (headlines) {
     content.headlines = headlines.map((text2) => ({ text: text2 }));
   }
-  const descriptions = listFlag(args5.descriptions);
+  const descriptions = copyListFlag(args5.descriptions, "descriptions");
   if (descriptions) {
     content.descriptions = descriptions.map((text2) => ({ text: text2 }));
   }
@@ -18176,11 +18271,11 @@ function demandGenContentFromFlags(args5, base) {
   if (typeof args5["ad-name"] === "string") {
     content.name = args5["ad-name"];
   }
-  const headlines = listFlag(args5.headlines);
+  const headlines = copyListFlag(args5.headlines, "headlines");
   if (headlines) {
     content.headlines = headlines.map((text2) => ({ text: text2 }));
   }
-  const descriptions = listFlag(args5.descriptions);
+  const descriptions = copyListFlag(args5.descriptions, "descriptions");
   if (descriptions) {
     content.descriptions = descriptions.map((text2) => ({ text: text2 }));
   }
@@ -18213,15 +18308,15 @@ function demandGenVideoContentFromFlags(args5, base) {
   if (typeof args5["ad-name"] === "string") {
     content.name = args5["ad-name"];
   }
-  const headlines = listFlag(args5.headlines);
+  const headlines = copyListFlag(args5.headlines, "headlines");
   if (headlines) {
     content.headlines = headlines.map((text2) => ({ text: text2 }));
   }
-  const longHeadlines = listFlag(args5["long-headlines"]);
+  const longHeadlines = copyListFlag(args5["long-headlines"], "long-headlines");
   if (longHeadlines) {
     content.longHeadlines = longHeadlines.map((text2) => ({ text: text2 }));
   }
-  const descriptions = listFlag(args5.descriptions);
+  const descriptions = copyListFlag(args5.descriptions, "descriptions");
   if (descriptions) {
     content.descriptions = descriptions.map((text2) => ({ text: text2 }));
   }
@@ -18251,11 +18346,11 @@ function appContentFromFlags(args5, base) {
     format: "app",
     ...base !== null && typeof base === "object" && !Array.isArray(base) ? base : {}
   };
-  const headlines = listFlag(args5.headlines);
+  const headlines = copyListFlag(args5.headlines, "headlines");
   if (headlines) {
     content.headlines = headlines.map((text2) => ({ text: text2 }));
   }
-  const descriptions = listFlag(args5.descriptions);
+  const descriptions = copyListFlag(args5.descriptions, "descriptions");
   if (descriptions) {
     content.descriptions = descriptions.map((text2) => ({ text: text2 }));
   }
@@ -18357,10 +18452,13 @@ var adContentArgs = {
     type: "string",
     description: "responsiveSearch (default) | responsiveDisplay | call | app | video | demandGen | demandGenVideo"
   },
-  headlines: { type: "string", description: "Comma-separated headlines (RSA/RDA/demandGen/demandGenVideo/app)" },
+  headlines: {
+    type: "string",
+    description: `Headlines (RSA/RDA/demandGen/demandGenVideo/app) \u2014 ${AD_COPY_LIST_USAGE}`
+  },
   descriptions: {
     type: "string",
-    description: "Comma-separated descriptions (RSA/RDA/demandGen/demandGenVideo/app)"
+    description: `Descriptions (RSA/RDA/demandGen/demandGenVideo/app) \u2014 ${AD_COPY_LIST_USAGE}`
   },
   path1: { type: "string" },
   path2: { type: "string" },
@@ -18371,7 +18469,7 @@ var adContentArgs = {
   },
   "long-headlines": {
     type: "string",
-    description: "Comma-separated long headlines, up to 5 of 90 chars (demandGenVideo \u2014 no other format has them)"
+    description: `Long headlines, up to 5 of 90 chars (demandGenVideo \u2014 no other format has them) \u2014 ${AD_COPY_LIST_USAGE}`
   },
   "business-name": { type: "string", description: "Business name (responsiveDisplay/demandGen/demandGenVideo)" },
   "marketing-images": {
@@ -18422,16 +18520,22 @@ var adsCommand = defineCommand31({
         ...adContentArgs,
         status: { type: "string" }
       },
-      run: async ({ args: args5 }) => {
+      run: async ({ args: parsedArgs, rawArgs }) => {
+        const args5 = withRepeatedCopyFlags(parsedArgs, rawArgs);
         const customerId = requireCustomerId(args5);
         const file = loadJsonFileArg(args5.file);
         const format = args5.format ?? "responsiveSearch";
         const content = adContentFromFlags(args5, format, file.content);
-        await stageCreate("google.ad.create", customerId, {
-          adGroup: requireStringFlag(args5["ad-group-ref"] ?? file.adGroup, "--ad-group-ref"),
-          status: args5.status ?? file.status,
-          content
-        });
+        await stageCreate(
+          "google.ad.create",
+          customerId,
+          {
+            adGroup: requireStringFlag(args5["ad-group-ref"] ?? file.adGroup, "--ad-group-ref"),
+            status: args5.status ?? file.status,
+            content
+          },
+          copyListSplitHints(args5)
+        );
       }
     }),
     update: defineCommand31({
@@ -18440,7 +18544,8 @@ var adsCommand = defineCommand31({
         description: "Stage an ad update. Any content change needs --format plus EVERY required field of that format (or --file with the complete content) \u2014 a field you send replaces its current value wholesale, so a list you pass must already include the entries you want to keep. Optional asset lists you omit entirely are left untouched on the ad. Status-only changes need neither --format nor content flags."
       },
       args: { ...customerIdArg, ...fileArg, ...adContentArgs, status: { type: "string" } },
-      run: async ({ args: args5 }) => {
+      run: async ({ args: parsedArgs, rawArgs }) => {
+        const args5 = withRepeatedCopyFlags(parsedArgs, rawArgs);
         const customerId = requireCustomerId(args5);
         const file = loadJsonFileArg(args5.file);
         const format = args5.format;
@@ -18454,7 +18559,8 @@ var adsCommand = defineCommand31({
         await stageUpdate("google.ad.update", customerId, requireTarget(args5, "ad"), payload, [
           ...payload.content !== void 0 ? [
             "Each content field you sent replaces its current value wholesale at publish \u2014 re-check that every headline, description, final URL and asset you want to KEEP is in this payload, not just the ones you changed."
-          ] : []
+          ] : [],
+          ...format ? copyListSplitHints(args5) : []
         ]);
       }
     }),
@@ -18483,9 +18589,9 @@ var assetGroupsCommand = defineCommand31({
         },
         name: { type: "string", description: "Asset group name" },
         "final-urls": { type: "string", description: "Comma-separated final URLs" },
-        headlines: { type: "string", description: "Comma-separated headlines (3\u201315, \u226430 chars)" },
-        "long-headlines": { type: "string", description: "Comma-separated long headlines (1\u20135, \u226490 chars)" },
-        descriptions: { type: "string", description: "Comma-separated descriptions (2\u20135, \u226490 chars)" },
+        headlines: { type: "string", description: `Headlines (3\u201315, \u226430 chars) \u2014 ${AD_COPY_LIST_USAGE}` },
+        "long-headlines": { type: "string", description: `Long headlines (1\u20135, \u226490 chars) \u2014 ${AD_COPY_LIST_USAGE}` },
+        descriptions: { type: "string", description: `Descriptions (2\u20135, \u226490 chars) \u2014 ${AD_COPY_LIST_USAGE}` },
         "business-name": { type: "string" },
         "image-assets": { type: "string", description: "Comma-separated marketing image asset refs (1.91:1)" },
         "square-image-assets": {
@@ -18499,11 +18605,12 @@ var assetGroupsCommand = defineCommand31({
         "logo-assets": { type: "string", description: "Comma-separated logo image asset refs" },
         status: { type: "string", description: "ENABLED | PAUSED (default PAUSED)" }
       },
-      run: async ({ args: args5 }) => {
+      run: async ({ args: parsedArgs, rawArgs }) => {
+        const args5 = withRepeatedCopyFlags(parsedArgs, rawArgs);
         const customerId = requireCustomerId(args5);
-        const headlines = listFlag(args5.headlines);
-        const longHeadlines = listFlag(args5["long-headlines"]);
-        const descriptions = listFlag(args5.descriptions);
+        const headlines = copyListFlag(args5.headlines, "headlines");
+        const longHeadlines = copyListFlag(args5["long-headlines"], "long-headlines");
+        const descriptions = copyListFlag(args5.descriptions, "descriptions");
         const payload = mergePayload(loadJsonFileArg(args5.file), {
           campaign: args5["campaign-ref"],
           name: args5.name,
@@ -18518,7 +18625,7 @@ var assetGroupsCommand = defineCommand31({
           logoAssets: listFlag(args5["logo-assets"]),
           status: args5.status
         });
-        await stageCreate("google.assetGroup.create", customerId, payload);
+        await stageCreate("google.assetGroup.create", customerId, payload, copyListSplitHints(args5));
       }
     }),
     update: defineCommand31({