@koda-sl/baker-cli 0.147.0 → 0.148.0

package/README.md

@@ -492,6 +492,8 @@ Command groups: `budgets`, `campaigns`, `ad-groups`, `keywords` (add/update/remo
 
 **Amending a staged op** — `draft amend <ref> (--file patch.json | --patch '<json>')` merges a JSON patch into an already-staged op's payload (objects deep-merge, `null` deletes a key, arrays/scalars replace) and re-validates it in full. Always prefer this over removing and re-staging — it preserves the op's ref (so dependents don't break) and its position in the draft. `draft show <ref>` prints the full staged payload (including warnings/annotations) so you can verify a change looks right before publish.
 
+**Already-satisfied writes aren't staged** — an update or pause/resume whose every field already holds the requested value on the live account is dropped instead of staged: the envelope returns `"staged": false, "noop": true` with a `reason` ("this ad is already Paused on Google Ads"), plus a hint. This is a success, not an error — there is nothing to publish. Only a positively-read live snapshot can trigger it; a failed or empty read always stages. Batch responses list the dropped ops under `skipped`, and `count` reflects what actually entered the draft.
+
 **Batch keyword adds** — `keywords add`, `negative-keywords add`, and `keyword-lists add` take a whole batch in one command: comma-separate `--text` entries and/or pass `--file <list.txt>` (one keyword per line). A `:EXACT`/`:PHRASE`/`:BROAD` suffix per entry overrides the `--match-type` default. Batches stage all-or-nothing as one request (limit 500); each keyword still lands as its own draft op, so it stays individually removable/amendable.
 
 ---

package/dist/cli.js

@@ -5395,17 +5395,27 @@ var googleDraftStageRequestSchema = z13.object({
   chatId: z13.string(),
   op: googleDraftOpInputSchema
 });
-var googleDraftStageResponseSchema = z13.object({
-  staged: z13.literal(true),
-  ref: z13.string(),
-  kind: googleDraftOpKindSchema,
-  mode: googleWriteModeSchema,
-  dependsOn: z13.array(z13.string()),
-  summary: z13.string(),
-  warnings: z13.array(z13.string()),
-  /** True when the op amended an already-staged op in place instead of appending a new one. */
-  amended: z13.boolean().optional()
-});
+var googleDraftStageResponseSchema = z13.discriminatedUnion("staged", [
+  z13.object({
+    staged: z13.literal(true),
+    ref: z13.string(),
+    kind: googleDraftOpKindSchema,
+    mode: googleWriteModeSchema,
+    dependsOn: z13.array(z13.string()),
+    summary: z13.string(),
+    warnings: z13.array(z13.string()),
+    /** True when the op amended an already-staged op in place instead of appending a new one. */
+    amended: z13.boolean().optional()
+  }),
+  z13.object({
+    staged: z13.literal(false),
+    noop: z13.literal(true),
+    kind: googleDraftOpKindSchema,
+    mode: googleWriteModeSchema,
+    summary: z13.string(),
+    reason: z13.string()
+  })
+]);
 var googleDraftAmendRequestSchema = z13.object({
   chatId: z13.string(),
   ref: z13.string(),
@@ -5432,7 +5442,8 @@ var googleDraftStageBatchResponseSchema = z13.object({
       summary: z13.string(),
       warnings: z13.array(z13.string())
     })
-  )
+  ),
+  skipped: z13.array(z13.object({ kind: googleDraftOpKindSchema, summary: z13.string(), reason: z13.string() })).optional()
 });
 var googleDraftOpViewSchema = z13.object({
   ref: z13.string(),
@@ -5691,7 +5702,7 @@ function keywordEntries(args) {
   const seen = /* @__PURE__ */ new Set();
   for (const item of raw) {
     const entry = parseKeywordEntry(item, defaultMatch);
-    const key = `${entry.text}\0${entry.matchType}`;
+    const key = JSON.stringify([entry.text, entry.matchType]);
     if (!seen.has(key)) {
       seen.add(key);
       entries.push(entry);
@@ -5758,6 +5769,9 @@ function handleGoogleError(err) {
   });
   process.exit(1);
 }
+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.`];
+}
 async function stageGoogleOp(raw, hints) {
   const preflight = googleDraftOpInputSchema.safeParse(raw);
   if (!preflight.success) {
@@ -5775,7 +5789,8 @@ async function stageGoogleOp(raw, hints) {
   try {
     const chatId = requireChatId();
     const response = await apiPost("/api/ads/google/draft/stage", { chatId, op: preflight.data });
-    writeJsonEnvelope(hints && hints.length > 0 ? { ...response, hints } : response);
+    const allHints = [...noopHints(response.data), ...hints ?? []];
+    writeJsonEnvelope(allHints.length > 0 ? { ...response, hints: allHints } : response);
   } catch (err) {
     handleGoogleError(err);
   }
@@ -5811,7 +5826,14 @@ async function stageGoogleOps(rawOps, hints) {
   try {
     const chatId = requireChatId();
     const response = await apiPost("/api/ads/google/draft/stage-batch", { chatId, ops });
-    writeJsonEnvelope(hints && hints.length > 0 ? { ...response, hints } : response);
+    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 ?? []
+    ];
+    writeJsonEnvelope(allHints.length > 0 ? { ...response, hints: allHints } : response);
   } catch (err) {
     handleGoogleError(err);
   }