@koda-sl/baker-cli 0.201.0 → 0.202.0

package/README.md

@@ -2797,8 +2797,8 @@ baker actions create --name "Build hero" --priority high --tags landing,creative
 # surface — doing the work is the default, an action is the exception. One advisory hint per call: the
 # scheduled redirect wins when both match.
 # `draftCreateCount` is how many creates this chat has staged (counted after this op, so it drops again
-# after `draft remove`). At 3 or more, create emits a fan-out hint: re-read the draft and merge actions
-# that touch the same system into one.
+# after `draft remove`). At exactly 2, create emits a one-line merge nudge; at 3 or more it emits the
+# fuller fan-out hint: re-read the draft and merge actions that touch the same system into one.
 # If no --tags are passed, create emits a MISSING --tags hint — the action is invisible to the backlog's tag filter.
 # If no --priority is passed, create emits a MISSING --priority hint — the action ranks as normal/medium in the do-first order.
 baker actions update <action-id> --name "Better name" --tags google-ads,strategy  # --tags REPLACES the set; '' clears

package/dist/cli.js

@@ -587,6 +587,9 @@ function parseTagList(value) {
   const tags = value.split(",").map((t) => t.trim()).filter((t) => t.length > 0);
   return [...new Set(tags)];
 }
+function fold(text2) {
+  return text2.normalize("NFD").replace(/[̀-ͯ]/g, "");
+}
 var SCHEDULE_SIGNALS = [
   /\brecurr(?:ing|ence)\b/i,
   /\bcadence\b/i,
@@ -626,28 +629,95 @@ function parsePriority(value, { allowClear }) {
   );
 }
 function looksScheduled(name, description) {
-  const haystack = `${name}
-${description}`;
-  return SCHEDULE_SIGNALS.some((re) => re.test(haystack));
-}
+  const haystack = fold(`${name}
+${description}`);
+  return SCHEDULE_SIGNALS.some((re2) => re2.test(haystack));
+}
+var CAMPAIGN_ENTITY = String.raw`(?:campaigns?|campanas?|ad\s*sets?|adsets?|ad\s*groups?|adgroups?|ads?|anuncios?|conjuntos?\s+de\s+anuncios|grupos?\s+de\s+anuncios)`;
+var LIFECYCLE_VERB = "(?:pause|unpause|enable|disable|launch|restructure|duplicate|clone|rebuild|prune|reweight|split|consolidate|pausar|pausa|activar|podar|poda|reponderar|duplicar|reestructurar|consolidar)";
+var BUDGET_VERB = "(?:cap|raise|lower|increase|decrease|adjust|change|set|update|reallocate|shift|limitar|subir|bajar|ajustar|aplicar|reasignar)";
+var re = (...parts) => new RegExp(parts.join(""), "i");
 var SURFACE_RULES = [
+  {
+    surface: "ask-user",
+    signals: [
+      /\bproduction\s+(?:domain|url|site|hostname)\b/i,
+      /\b(?:confirm|check|verify|confirmar|verificar)\s+(?:with|w\/|con)\s+(?:the\s+|el\s+|la\s+)?(?:client|user|customer|team|cliente|usuario|equipo)\b/i,
+      /\bmissing\s+(?:the\s+)?(?:value|values|id|ids|url|budget|domain|credentials?|account\s+id|measurement\s+id)\b/i,
+      /\b(?:ask|get)\s+(?:the\s+)?(?:client|user)\s+for\b/i,
+      /\b(?:pedir|solicitar)\s+(?:al\s+)?cliente\b/i
+    ],
+    hint: "This is a missing input, not blocked work. Ask for it with AskUserQuestion (one question), then do the work in the same turn \u2014 filing the Task AND asking the question is the same request twice."
+  },
   {
     surface: "analysis",
     nameOnly: true,
     signals: [
-      /^\s*(?:re-?)?(?:investigate|analyse|analyze|audit|review|document|assess|evaluate|research|map|inventory|examine|diagnose)\b/i
+      /^\s*(?:re-?)?(?:investigat|analy[sz]|audit|review|document|assess|evaluat|research|map|inventor|examin|diagnos|verif|validat|benchmark)\w*\b/i,
+      /^\s*(?:auditar|auditoria|investigar|investigacion|revisar|revision|analizar|analisis|documentar|evaluar|evaluacion|diagnosticar|diagnostico|mapear|inventariar|verificar|verificacion|comprobar|estudiar|estudio)\b/i
     ],
     hint: "This names the analysis itself \u2014 investigating, auditing and documenting are work you do now, not work you file. Run it in this chat and deliver the finding; a Task is for a fix something genuinely blocks (bar: `__tooling__/docs/tools/baker/actions.md`)."
   },
   {
-    surface: "ask-user",
+    surface: "ads-write",
+    // "Approve X" is a human decision, not a staged write — and the budget
+    // signals are broad enough to swallow it otherwise. Name-anchored so a
+    // description recalling an earlier approval doesn't veto real work.
+    exceptName: [/^\s*(?:approve|approval|sign[-\s]?off|aprobar|aprobacion)\b/i],
+    // Bare `keywords` is the work when it's the subject of the Task, but plain
+    // context when a description happens to discuss search terms.
+    nameOnlySignals: [/\b(?:keywords?|palabras\s+clave)\b/i],
     signals: [
-      /\bproduction\s+(?:domain|url|site|hostname)\b/i,
-      /\b(?:confirm|check|verify)\s+(?:with|w\/)\s+(?:the\s+)?(?:client|user|customer|team)\b/i,
-      /\bmissing\s+(?:the\s+)?(?:value|values|id|ids|url|budget|domain|credentials?|account\s+id|measurement\s+id)\b/i,
-      /\b(?:ask|get)\s+(?:the\s+)?(?:client|user)\s+for\b/i
+      // Targeting.
+      /\b(?:geo|location|geographic|geografic\w*)[\s-]*(?:targeting|target|segmentacion)\b/i,
+      /\bsegmentacion\s+(?:geografica|por\s+ubicacion)\b/i,
+      /\bpresence[-\s_]?(?:only|or[-\s_]?interest)\b/i,
+      /\b(?:ad\s*schedul\w*|dayparting|calendario\s+de\s+anuncios)\b/i,
+      // Extensions / assets.
+      /\b(?:callouts?|sitelinks?|site\s*links?|structured\s+snippets?|price\s+extensions?|lead\s+form\s+extensions?|extensiones|asset\s+groups?)\b/i,
+      // Ad copy. Compound forms only — bare `headline` and `copy` belong to
+      // `landing`, and an RSA headline has to say so.
+      /\b(?:rsas?|responsive\s+search\s+ads?|ad\s+copy|copy\s+de\s+(?:los\s+)?anuncios)\b/i,
+      /\b(?:headline|titular|description)\s+(?:pool|set|slots?)\b/i,
+      /\bdisplay\s+paths?\b|\brutas?\s+visibles?\b/i,
+      // Keywords. `negatives` is bare in English because in this product's
+      // vocabulary the noun is always negative keywords; the Spanish side is
+      // feminine-only, because "resultados negativos" is a real phrase.
+      /\bmatch\s+types?\b|\bconcordancias?\b/i,
+      /\bnegatives?\b/i,
+      /\b(?:palabras\s+clave\s+)?negativas\b/i,
+      /\bkeyword\s+lists?\b|\blistas?\s+de\s+palabras\s+clave\b/i,
+      // Audiences. `LAL` is the operators' own shorthand for a lookalike and
+      // has no other reading in this corpus.
+      /\b(?:in-?market|affinity|audience\s+signals?|senales?\s+de\s+audiencia)\b/i,
+      /\b(?:audiencias?|lal)\b/i,
+      /\blistas?\s+de\s+(?:exclusion|remarketing)\b|\bexclusion\s+lists?\b/i,
+      // URLs.
+      /\btracking\s+(?:url\s+)?templates?\b|\bfinal\s+urls?\b|\bfinal\s+url\s+suffix\b/i,
+      // Bidding.
+      /\bbid\s+(?:adjustments?|modifiers?|strateg\w*)\b|\bajustes?\s+de\s+puja\b/i,
+      /\b(?:tcpa|troas|target\s+cpa|target\s+roas)\b/i,
+      // Lifecycle. Bare `campaign` is deliberately NOT a signal — it appears
+      // everywhere and would strip tag-manager and landing of correct
+      // routings, so every campaign-shaped rule needs a verb or a specific
+      // setting noun next to it.
+      // Bounded, and never across the name/description boundary. `ads` is an
+      // ordinary word in this domain, so an unbounded gap lets a verb in the
+      // name pair with a passing mention in the description — and since
+      // ads-write is evaluated first, that spurious match outranks the surface
+      // which should have answered.
+      re(String.raw`\b`, LIFECYCLE_VERB, String.raw`\b[^\n]{0,40}\b`, CAMPAIGN_ENTITY, String.raw`\b`),
+      // Budget. Two complementary shapes — verb before, quantity after — so a
+      // change stated either way lands, guarded by the `exceptName` veto above.
+      /\b(?:daily|campaign|lifetime|ad\s*set|minimum|min|competitor)\s+budgets?\b/i,
+      /\bpresupuestos?\b/i,
+      re(String.raw`\b`, BUDGET_VERB, String.raw`\b[\s\S]{0,24}\bbudgets?\b`),
+      /\bbudgets?\b[\s\S]{0,24}\b(?:at|to|cap|down|up|\d|%|€|\$)/i,
+      /\bnegative\s+keywords?\b/i,
+      /\b(?:lookalike|custom)\s+audience\b/i,
+      /\bswap\s+(?:the\s+)?creative\b/i
     ],
-    hint: "This is a missing input, not blocked work. Ask for it with AskUserQuestion (one question), then do the work in the same turn \u2014 filing the Task AND asking the question is the same request twice."
+    hint: "This is an ad-platform change the write surface covers \u2014 stage it now with `baker ads google|meta|linkedin` and it applies at publish (guides: `__tooling__/docs/tools/baker/ads-google.md`, `__tooling__/docs/tools/baker/ads-meta.md`, `__tooling__/docs/tools/baker/ads-linkedin.md`). Staging is not going live."
   },
   {
     surface: "tag-manager",
@@ -661,7 +731,10 @@ var SURFACE_RULES = [
       /\bga4\s+(?:config|configuration|event)\b/i,
       /\bcontainer\s+(?:tag|trigger|variable)s?\b/i,
       /\b(?:conversion|form|submit|submission|click|purchase|lead|custom)\s+event\b/i,
-      /\btrack\w*\b[\s\S]*\bevent\b/i
+      /\btrack\w*\b[\s\S]*\bevent\b/i,
+      /\beventos?\s+de\s+(?:formulario|conversion|clic|compra)\b/i,
+      /\bconsentimiento\b/i,
+      /\bmedicion\b/i
     ],
     except: [/\bserver-?\s?side\b/i, /\bsgtm\b/i, /\bhosting\b/i],
     hint: "This is a change inside the GTM container \u2014 `baker tag-manager` stages it now and it applies at publish (guide: `__tooling__/docs/tools/baker/tag-manager.md`). Stage it instead of filing it, and if several container fixes really are blocked, they belong in ONE Task, not one per finding."
@@ -674,30 +747,20 @@ var SURFACE_RULES = [
       /\bclarity\b/i,
       /\bhotjar\b/i,
       /\bgtm\s+snippet\b/i,
-      /\b(?:install|add|remove|swap)\b[\s\S]*\b(?:snippet|script|tag)\b/i
+      /\bcapi\b/i,
+      /\b(?:install|add|remove|swap|instalar|colocar)\b[\s\S]*\b(?:snippet|script|tag)\b/i
     ],
+    // Two vetoes. A structured snippet is a Google Ads extension, not a script
+    // on the page — `ads-write` claims it above, and this stops both rules
+    // being able to answer. Server-side delivery is infrastructure nobody here
+    // can provision, the same call `tag-manager` already makes for sGTM.
+    except: [/\bstructured\s+snippets?\b/i, /\bserver-?\s?side\b/i, /\bhosting\b/i],
     hint: "This is a tag or script on the site \u2014 it goes through the `request_tag_input` approval form in this chat, which also collects any secret values (guide: `__tooling__/docs/tools/baker/tags.md`). Show the form instead of filing a Task."
   },
-  {
-    surface: "ads-write",
-    signals: [
-      /\b(?:pause|unpause|enable|disable|launch|restructure|duplicate|clone|rebuild)\b[\s\S]*\b(?:campaign|ad\s*set|ad\s*group|adgroup)\b/i,
-      /\b(?:tcpa|troas|target\s+cpa|target\s+roas)\b/i,
-      /\b(?:daily|campaign|lifetime|ad\s*set)\s+budget\b/i,
-      /\b(?:raise|lower|increase|decrease|adjust|change|set|update)\s+(?:the\s+)?budget\b/i,
-      /\bbid\s+strateg/i,
-      /\bnegative\s+keywords?\b/i,
-      /\b(?:lookalike|custom)\s+audience\b/i,
-      /\bswap\s+(?:the\s+)?creative\b/i
-    ],
-    hint: "This is an ad-platform change the write surface covers \u2014 stage it now with `baker ads google|meta|linkedin` and it applies at publish (guides: `__tooling__/docs/tools/baker/ads-google.md`, `__tooling__/docs/tools/baker/ads-meta.md`, `__tooling__/docs/tools/baker/ads-linkedin.md`). Staging is not going live."
-  },
   {
     surface: "landing",
-    signals: [
-      /\b(?:landing\s+page|hero|headline|above\s+the\s+fold)\b/i,
-      /\b(?:rewrite|tighten|restyle)\b[\s\S]*\b(?:copy|page|section)\b/i
-    ],
+    nameOnlySignals: [/\b(?:landing\s+pages?|landings?|hero|headline|above\s+the\s+fold|pagina)\b/i],
+    signals: [/\b(?:rewrite|tighten|restyle)\b[\s\S]*\b(?:copy|page|section)\b/i],
     hint: "This is a page change \u2014 build it in this chat with the `/landing` skill. Only file it if something outside the page blocks the work."
   },
   {
@@ -706,17 +769,25 @@ var SURFACE_RULES = [
     hint: "This is form behaviour \u2014 change it in this chat with the `/flow-builder` skill. Only file it if something outside the form blocks the work."
   }
 ];
+var FAN_OUT_NUDGE_THRESHOLD = 2;
 var FAN_OUT_THRESHOLD = 3;
 var SCHEDULED_HINT = 'This reads as recurring/scheduled work. If it should run on a cadence or a future date, create a Scheduled Action instead \u2014 baker scheduled-actions create --cron "0 9 * * MON" (or --run-at; guide: `__tooling__/docs/tools/baker/scheduled-actions.md`). Do NOT capture "set up a scheduled action" as a Work Action.';
 function looksExecutable(name, description) {
-  const full = `${name}
-${description}`;
+  const foldedName = fold(name);
+  const full = `${foldedName}
+${fold(description)}`;
   for (const rule of SURFACE_RULES) {
-    const haystack = rule.nameOnly ? name : full;
-    if (rule.except?.some((re) => re.test(full))) {
+    const haystack = rule.nameOnly ? foldedName : full;
+    if (rule.except?.some((re2) => re2.test(full))) {
+      continue;
+    }
+    if (rule.exceptName?.some((re2) => re2.test(foldedName))) {
       continue;
     }
-    if (rule.signals.some((re) => re.test(haystack))) {
+    if (rule.signals.some((re2) => re2.test(haystack))) {
+      return { surface: rule.surface, hint: rule.hint };
+    }
+    if (rule.nameOnlySignals?.some((re2) => re2.test(foldedName))) {
       return { surface: rule.surface, hint: rule.hint };
     }
   }
@@ -741,6 +812,10 @@ function buildCreateHints({
     hints.push(
       `${draftCreateCount} Tasks are staged in this chat. Re-read them with \`baker actions draft\`: can any be done now with a surface you already have, and do any two touch the same system (same container, same campaign, same page)? Those belong in ONE Task \u2014 fold them together with \`baker actions update\` and drop the extras with \`baker actions draft remove <tempId>\`.`
     );
+  } else if (draftCreateCount === FAN_OUT_NUDGE_THRESHOLD) {
+    hints.push(
+      `Second Task staged in this chat. If these two land on the same system (same container, same campaign, same page) they are ONE Task \u2014 fold them with \`baker actions update\` and drop the extra with \`baker actions draft remove ${tempId}\`.`
+    );
   }
   hints.push(`Link dependencies: baker actions link --blocker <id> --blocked ${tempId}`);
   if (!description) {
@@ -8357,8 +8432,8 @@ var FALSE_RANGE_PATTERNS = [
 function countFalseRanges(text2) {
   let count = 0;
   let first = "";
-  for (const re of FALSE_RANGE_PATTERNS) {
-    for (const m of text2.matchAll(re)) {
+  for (const re2 of FALSE_RANGE_PATTERNS) {
+    for (const m of text2.matchAll(re2)) {
       const left = m[1] ?? "";
       const right = m[2] ?? "";
       if (/\d/.test(left) || /\d/.test(right)) continue;
@@ -8421,8 +8496,8 @@ function countPhraseTell(text2, id, phrases) {
   }
   return { count, first };
 }
-function countPattern(text2, re) {
-  const matches = text2.match(re) ?? [];
+function countPattern(text2, re2) {
+  const matches = text2.match(re2) ?? [];
   return { count: matches.length, first: matches.join("") };
 }
 function detectCopyTells(text2) {
@@ -20048,8 +20123,8 @@ import { defineCommand as defineCommand90 } from "citty";
 var GOOGLE_CSS2 = "https://fonts.googleapis.com/css2";
 function parseGoogleFontFaces(css) {
   const faces = [];
-  const re = /\/\*\s*([\w-]+)\s*\*\/\s*@font-face\s*\{([^}]*)\}/g;
-  for (const match of String(css).matchAll(re)) {
+  const re2 = /\/\*\s*([\w-]+)\s*\*\/\s*@font-face\s*\{([^}]*)\}/g;
+  for (const match of String(css).matchAll(re2)) {
     const subset = match[1] ?? "";
     const body = match[2] ?? "";
     const family = body.match(/font-family\s*:\s*['"]?([^'";]+)['"]?\s*;/i)?.[1]?.trim();
@@ -22645,8 +22720,8 @@ function parseVoiceTraits(description) {
   const out = {};
   if (/\b(female|woman|women|girl|lady)\b/.test(d)) out.gender = "female";
   else if (/\b(male|man|men|guy|boy)\b/.test(d)) out.gender = "male";
-  for (const [re, name] of LANGUAGE_WORDS) {
-    if (re.test(d)) {
+  for (const [re2, name] of LANGUAGE_WORDS) {
+    if (re2.test(d)) {
       out.language = name;
       break;
     }