@koda-sl/baker-cli 0.254.0 → 0.255.0-dev.8030cfcf9

package/dist/{chunk-6MAVG6LO.js → chunk-KSOGK7SJ.js}

@@ -2,10 +2,10 @@ import {
   handleConnectionError,
   needsConnectionFix,
   writeAdsJson
-} from "./chunk-5JHSX3ZB.js";
+} from "./chunk-6F52WYB7.js";
 import {
   ApiError
-} from "./chunk-JKSMLAQN.js";
+} from "./chunk-GU7IWEBC.js";
 import {
   getEnv
 } from "./chunk-DZUVUGEP.js";
@@ -108,4 +108,4 @@ export {
   csvOrJson,
   resolveEffectiveStatus
 };
-//# sourceMappingURL=chunk-6MAVG6LO.js.map
+//# sourceMappingURL=chunk-KSOGK7SJ.js.map

package/dist/cli.js

@@ -58,7 +58,7 @@ import {
   ulid,
   validateCanvasDeep,
   ytDlpBlockSignal
-} from "./chunk-SJPRTSGY.js";
+} from "./chunk-HBDGUVUH.js";
 import {
   csvOrJson,
   daysAgoIso,
@@ -67,7 +67,7 @@ import {
   resolveAccountIdArg,
   resolveEffectiveStatus,
   todayIso
-} from "./chunk-6MAVG6LO.js";
+} from "./chunk-KSOGK7SJ.js";
 import {
   buildQueryCacheKey,
   cacheGet,
@@ -83,13 +83,14 @@ import {
   writeAdsJson,
   writeAdsOutput,
   writeJsonEnvelope
-} from "./chunk-5JHSX3ZB.js";
+} from "./chunk-6F52WYB7.js";
 import {
   ApiError,
   apiGet,
+  apiGetWithHeaders,
   apiPost,
   validateConvexId
-} from "./chunk-JKSMLAQN.js";
+} from "./chunk-GU7IWEBC.js";
 import {
   installStreamTaps,
   logInvocation
@@ -2112,7 +2113,14 @@ var GOOGLE_ADS_LIMITS = {
     // Google's published headers (`structured-snippet-headers.ts`), the longest
     // of which is 29 characters. Only the values are capped.
     structuredSnippetValuesMin: 3,
-    structuredSnippetValuesMax: 10
+    structuredSnippetValuesMax: 10,
+    /**
+     * Per VALUE, not per asset. Google rejects a 26-character value with a bare "Too long." at
+     * `structured_snippet_asset.values[i]` — one red row inside an otherwise clean publish, plus
+     * every link that depends on the asset. Confirmed live on v23: "Bases de datos vectoriales"
+     * (26) rejected, its four neighbours accepted.
+     */
+    structuredSnippetValueMax: 25
   },
   conversionAction: {
     nameMax: 255
@@ -2272,9 +2280,24 @@ var ASSET_FIELD_TYPES = [
   "HEADLINE",
   "DESCRIPTION",
   "LOGO",
+  /**
+   * Kept because a draft staged before {@link IMAGE_ASSET_LINK_MESSAGE} shipped may still carry one,
+   * and because it is the field type an asset-group image occupies. It is refused on an
+   * `assetLink.attach` — see that message for what Google actually does with it.
+   */
   "MARKETING_IMAGE",
   "BUSINESS_NAME"
 ];
+var IMAGE_ASSET_LINK_FIELD_TYPES = /* @__PURE__ */ new Set([
+  "MARKETING_IMAGE",
+  "SQUARE_MARKETING_IMAGE",
+  "PORTRAIT_MARKETING_IMAGE",
+  "AD_IMAGE"
+]);
+var IMAGE_ASSET_LINK_MESSAGE = "Google does not accept an image through an asset link. MARKETING_IMAGE is a Performance Max asset-group slot (use `asset-groups attach`), and AD_IMAGE \u2014 the field type Google names for a Search image extension \u2014 comes back UNSUPPORTED_FIELD_TYPE at campaign and ad group level alike. Image extensions are not reachable through the API: tell the user to add them in the Google Ads interface, and carry on with the rest of the job.";
+function isImageAssetLinkFieldType(fieldType) {
+  return IMAGE_ASSET_LINK_FIELD_TYPES.has(fieldType);
+}
 var ASSET_GROUP_ASSET_FIELD_TYPES = [
   "MARKETING_IMAGE",
   "SQUARE_MARKETING_IMAGE",
@@ -3122,7 +3145,7 @@ var structuredSnippetHeaderSchema = z4.string().min(1).transform((header, ctx) =
 var structuredSnippetAssetSchema = z4.object({
   type: z4.literal("structuredSnippet"),
   header: structuredSnippetHeaderSchema,
-  values: z4.array(z4.string().min(1)).min(GOOGLE_ADS_LIMITS.asset.structuredSnippetValuesMin).max(GOOGLE_ADS_LIMITS.asset.structuredSnippetValuesMax)
+  values: z4.array(z4.string().min(1).max(GOOGLE_ADS_LIMITS.asset.structuredSnippetValueMax)).min(GOOGLE_ADS_LIMITS.asset.structuredSnippetValuesMin).max(GOOGLE_ADS_LIMITS.asset.structuredSnippetValuesMax)
 });
 var callToActionAssetSchema = z4.object({ type: z4.literal("callToAction"), callToAction: z4.string().min(1) });
 var assetCreateSchema = z4.discriminatedUnion("type", [
@@ -3142,7 +3165,7 @@ var assetUpdateSchema = z4.object({
   finalUrls: z4.array(httpsUrlSchema2).min(1).optional(),
   calloutText: z4.string().min(1).max(GOOGLE_ADS_LIMITS.asset.calloutTextMax).optional(),
   header: structuredSnippetHeaderSchema.optional(),
-  values: z4.array(z4.string().min(1)).min(GOOGLE_ADS_LIMITS.asset.structuredSnippetValuesMin).max(GOOGLE_ADS_LIMITS.asset.structuredSnippetValuesMax).optional(),
+  values: z4.array(z4.string().min(1).max(GOOGLE_ADS_LIMITS.asset.structuredSnippetValueMax)).min(GOOGLE_ADS_LIMITS.asset.structuredSnippetValuesMin).max(GOOGLE_ADS_LIMITS.asset.structuredSnippetValuesMax).optional(),
   callToAction: z4.string().min(1).optional(),
   text: z4.string().min(1).optional()
 }).refine((p) => Object.values(p).some((v) => v !== void 0), "update needs at least one field");
@@ -3150,7 +3173,11 @@ var assetLinkAttachSchema = z4.object({
   level: z4.enum(["campaign", "adGroup", "customer"]),
   parent: refSchema.optional(),
   asset: refSchema,
-  fieldType: z4.enum(ASSET_FIELD_TYPES)
+  // A plain string, not `z.enum(ASSET_FIELD_TYPES)`, so the image refusal below can answer with
+  // the reason instead of an enum listing. An agent reaching for AD_IMAGE is following Google's
+  // own documentation; "Invalid option: expected one of …" sends it to guess another field type,
+  // which is the failure this whole refusal exists to end.
+  fieldType: z4.string().min(1)
 }).superRefine((value, ctx) => {
   if (value.level !== "customer" && !value.parent) {
     ctx.addIssue({
@@ -3159,6 +3186,15 @@ var assetLinkAttachSchema = z4.object({
       message: `parent is required for a ${value.level}-level asset link (--parent-ref)`
     });
   }
+  if (isImageAssetLinkFieldType(value.fieldType)) {
+    ctx.addIssue({ code: z4.ZodIssueCode.custom, path: ["fieldType"], message: IMAGE_ASSET_LINK_MESSAGE });
+  } else if (!ASSET_FIELD_TYPES.includes(value.fieldType)) {
+    ctx.addIssue({
+      code: z4.ZodIssueCode.custom,
+      path: ["fieldType"],
+      message: `expected one of ${ASSET_FIELD_TYPES.join(" | ")}`
+    });
+  }
 });
 var assetGroupCreateSchema = z4.object({
   campaign: refSchema,
@@ -6183,7 +6219,10 @@ var testimonialsListRequestSchema = z20.object({
   status: testimonialStatusSchema.optional(),
   sentiment: testimonialSentimentSchema.optional(),
   language: z20.string().min(2).max(5).optional(),
-  limit: z20.coerce.number().int().positive().max(200).optional()
+  limit: z20.coerce.number().int().positive().max(200).optional(),
+  // Opaque, from the previous response's `X-Baker-Next-Cursor`. Absent on the
+  // first page; its absence on a response means the corpus is exhausted.
+  cursor: z20.string().optional()
 });
 var testimonialsListResponseSchema = z20.array(testimonialDocSchema);
 var testimonialsGetRequestSchema = z20.object({ id: z20.string().min(1, "Missing id parameter") });
@@ -14780,7 +14819,7 @@ var assetsUpdateCommand = defineCommand30({
     "final-urls": { type: "string", description: "Comma-separated sitelink final URLs" },
     "callout-text": { type: "string", description: "Callout text" },
     header: { type: "string", description: "Structured snippet header" },
-    values: { type: "string", description: "Comma-separated structured snippet values" },
+    values: { type: "string", description: "Comma-separated structured snippet values (3-10, max 25 characters each)" },
     cta: { type: "string", description: "Call-to-action text" },
     text: { type: "string", description: "Text asset content" }
   },
@@ -14814,7 +14853,9 @@ var assetsCommand = defineCommand30({
       "Stage an asset (text/image/sitelink/callout/structuredSnippet/\u2026)"
     ),
     update: assetsUpdateCommand,
-    attach: fileCreateCommand("attach", "google.assetLink.attach", "Attach an asset to a campaign/adGroup/customer"),
+    attach: fileCreateCommand("attach", "google.assetLink.attach", "Attach an asset to a campaign/adGroup/customer", [
+      "Images are not attachable here at all \u2014 Google refuses every image field type through an asset link, at every level. Image extensions have to be added in the Google Ads interface; say so and carry on. Performance Max asset-group images are a different mechanism and do work (`asset-groups attach`)."
+    ]),
     detach: statusCommand2("google.assetLink.detach", "asset link")
   }
 });
@@ -21014,10 +21055,10 @@ function duplicateCommand2(entity, label) {
       replace: { type: "boolean", description: "Pause the original once the copy publishes" }
     },
     run: async ({ args }) => {
-      const { apiPost: apiPost2 } = await import("./client-3TNO43FO.js");
+      const { apiPost: apiPost2 } = await import("./client-VDCCDEHE.js");
       const { requireChatId: requireChatId2 } = await import("./env-FWMZXMQK.js");
-      const { writeJsonEnvelope: writeJsonEnvelope2 } = await import("./output-B3OCLLS5.js");
-      const { handleMetaError: handleMetaError2 } = await import("./shared-J24NXBW4.js");
+      const { writeJsonEnvelope: writeJsonEnvelope2 } = await import("./output-JBYI5IVX.js");
+      const { handleMetaError: handleMetaError2 } = await import("./shared-GWFXJXKJ.js");
       try {
         const accountId = bareAccountId2(args);
         const chatId = requireChatId2();
@@ -50105,14 +50146,30 @@ function emptyReasonDetail(reason, sourceCount, activeFilters) {
   }
   return `${sourceCount} review source${sourceCount === 1 ? " is" : "s are"} connected but hold no testimonials yet \u2014 the source is there, the rows are not. Ingestion may still be running.`;
 }
+function morePagesHint(resultCount, cursor) {
+  const found = resultCount === 0 ? "This page held no matches" : `This page held ${resultCount}`;
+  return `${found}, and the library continues past it \u2014 so this is one page, not the client's whole set. Do not report it as the corpus and do not conclude anything is missing from it. Read the next page with \`baker testimonials list --cursor ${cursor}\` (same filters), or go straight at the rows you want with \`baker testimonials search <query>\`.`;
+}
+function endOfWalkHint(command) {
+  const noun = command === "search" ? "results" : "rows";
+  return `That was the end of the library \u2014 no more ${noun} past the page before this one. Nothing here says the corpus is empty (the earlier pages held it), so answer from what those pages returned rather than re-reading.`;
+}
 async function measureEmptyCorpus({
   command,
   resultCount,
-  activeFilters
+  activeFilters,
+  nextCursor = null,
+  continued = false
 }) {
+  if (nextCursor) {
+    return { hints: [morePagesHint(resultCount, nextCursor)] };
+  }
   if (resultCount > 0) {
     return { hints: [] };
   }
+  if (continued) {
+    return { hints: [endOfWalkHint(command)] };
+  }
   if (activeFilters.length > 0) {
     const reason2 = "filtered_out";
     return {
@@ -50191,7 +50248,12 @@ registerSchema({
     },
     language: { type: "string", description: "Filter by language code (e.g. en, es)", required: false },
     tags: { type: "string", description: "Comma-separated tags to filter by", required: false },
-    limit: { type: "number", description: "Max results (default 50)", required: false, default: 50 }
+    limit: { type: "number", description: "Max results per page (default 50)", required: false, default: 50 },
+    cursor: {
+      type: "string",
+      description: "Read the next page: pass meta.next_cursor from the previous call, with the same filters.",
+      required: false
+    }
   }
 });
 var PARAM_NAMES = {
@@ -50202,7 +50264,8 @@ var PARAM_NAMES = {
   sentiment: "sentiment",
   language: "language",
   tags: "tags",
-  limit: "limit"
+  limit: "limit",
+  cursor: "cursor"
 };
 function buildListParams(args) {
   const params = {};
@@ -50227,22 +50290,30 @@ var listCommand18 = defineCommand205({
     sentiment: { type: "string", description: "Filter: positive|neutral|negative", required: false },
     language: { type: "string", description: "Filter by language code (e.g. en, es)", required: false },
     tags: { type: "string", description: "Comma-separated tags to filter by", required: false },
-    limit: { type: "string", description: "Max results (default 50)", required: false },
+    limit: { type: "string", description: "Max results per page (default 50)", required: false },
+    cursor: { type: "string", description: "meta.next_cursor from the previous page", required: false },
     output: { type: "string", description: "Output format: json|files|md", required: false, default: "json" },
     fields: { type: "string", description: "Comma-separated field names to include", required: false },
     full: { type: "boolean", description: "Include full metadata", required: false, default: false }
   },
   run: async ({ args }) => {
     try {
-      const data = await apiGet("/api/testimonials", buildListParams(args));
+      const { data, headers } = await apiGetWithHeaders(
+        "/api/testimonials",
+        buildListParams(args)
+      );
+      const nextCursor = headers.get("x-baker-next-cursor");
       const { hints: hints2, empty } = await measureEmptyCorpus({
         command: "list",
         resultCount: data.length,
-        activeFilters: activeFilterFlags(args, FILTER_FLAGS)
+        activeFilters: activeFilterFlags(args, FILTER_FLAGS),
+        nextCursor,
+        continued: Boolean(args.cursor)
       });
       const outputFormat = args.output || "json";
+      const meta = { ...empty ? { empty } : {}, ...nextCursor ? { next_cursor: nextCursor } : {} };
       writeOutput(
-        { ok: true, data, ...empty ? { meta: { empty } } : {}, ...hints2.length > 0 ? { hints: hints2 } : {} },
+        { ok: true, data, ...Object.keys(meta).length > 0 ? { meta } : {}, ...hints2.length > 0 ? { hints: hints2 } : {} },
         outputFormat,
         args.fields ? args.fields.split(",") : void 0,
         args.full,