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

package/dist/cli.js

@@ -67,7 +67,7 @@ import {
   validateCanvasDeep,
   withoutCastName,
   ytDlpBlockSignal
-} from "./chunk-D5P4COPD.js";
+} from "./chunk-FFOURYP5.js";
 import {
   csvOrJson,
   daysAgoIso,
@@ -17043,6 +17043,113 @@ 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 refuseDashLedValue(rawArgs, flag) {
+  for (let i = 0; i < rawArgs.length - 1; i++) {
+    if (rawArgs[i] === "--") return;
+    const value = rawArgs[i + 1];
+    if (rawArgs[i] === `--${flag}` && value?.startsWith("-")) {
+      failWriteValidation(
+        `--${flag} is followed by "${value}", which starts with a dash and is read as another flag, not as copy. Join copy that starts with a dash to its flag with =: --${flag}="${value}"`
+      );
+    }
+  }
+}
+function withRepeatedCopyFlags(args5, rawArgs) {
+  const out = { ...args5 };
+  for (const flag of AD_COPY_LIST_FLAGS) {
+    refuseDashLedValue(rawArgs ?? [], flag);
+    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 +18205,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 +18228,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 +18283,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 +18320,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 +18358,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 +18464,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 +18481,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 +18532,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 +18556,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 +18571,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 +18601,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 +18617,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 +18637,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({