@koda-sl/baker-cli 0.105.0 → 0.105.1

package/README.md

@@ -236,7 +236,7 @@ The CLI rejects or automatically corrects common GAQL mistakes before hitting th
 | Pattern | Behavior | Warning/error emitted |
 |---------|----------|-----------------------|
 | Missing LIMIT | Adds `LIMIT 200` | `"Added LIMIT 200. Use --limit to override."` |
-| `keyword.text` | Rewrites to `ad_group_criterion.keyword.text` | `"keyword.text → ad_group_criterion.keyword.text"` |
+| Bare `keyword.text` / `keyword.match_type` | Rewrites to `ad_group_criterion.*` (resource-scoped forms like `shared_criterion.keyword.text` pass through untouched) | `"keyword.text → ad_group_criterion.keyword.text"` |
 | `shopping_performance_view.product_*` | Rewrites to `segments.product_*` | Field renamed |
 | `campaign.status = 'ACTIVE'` | Rewrites to `= 'ENABLED'` | Enum corrected |
 | `segments.conversion_action_name` with `metrics.cost_micros` | Rejects; split into separate conversion-action and spend queries | `INCOMPATIBLE_FIELDS` with `fix.action: "split_query"` |

package/dist/cli.js

@@ -2623,12 +2623,12 @@ function buildCommand(query, ctx) {
   return `baker ads google query "${query}" --customer-id ${ctx.customerId}`;
 }
 var CORRECTION_RULES = [
-  // 1. keyword.text → ad_group_criterion.keyword.text
+  // 1. bare keyword.text → ad_group_criterion.keyword.text (leaves <resource>.keyword.text untouched)
   {
-    matchQuery: /\bkeyword\.text\b/,
+    matchQuery: /(?<![\w.])keyword\.text\b/,
     matchApiError: /keyword\.text/i,
     fix: (ctx) => {
-      const corrected = ctx.originalQuery.replace(/\bkeyword\.text\b/g, "ad_group_criterion.keyword.text");
+      const corrected = ctx.originalQuery.replace(/(?<![\w.])keyword\.text\b/g, "ad_group_criterion.keyword.text");
       return {
         action: "retry_with_modified_query",
         correctedCommand: buildCommand(corrected, ctx),
@@ -2636,12 +2636,15 @@ var CORRECTION_RULES = [
       };
     }
   },
-  // 2. keyword.match_type → ad_group_criterion.keyword.match_type
+  // 2. bare keyword.match_type → ad_group_criterion.keyword.match_type (leaves <resource>.keyword.match_type untouched)
   {
-    matchQuery: /\bkeyword\.match_type\b/,
+    matchQuery: /(?<![\w.])keyword\.match_type\b/,
     matchApiError: /keyword\.match_type/i,
     fix: (ctx) => {
-      const corrected = ctx.originalQuery.replace(/\bkeyword\.match_type\b/g, "ad_group_criterion.keyword.match_type");
+      const corrected = ctx.originalQuery.replace(
+        /(?<![\w.])keyword\.match_type\b/g,
+        "ad_group_criterion.keyword.match_type"
+      );
       return {
         action: "retry_with_modified_query",
         correctedCommand: buildCommand(corrected, ctx),
@@ -3993,12 +3996,12 @@ function buildCommand2(query, customerId) {
 function applyAutoFixes(query, limit) {
   let corrected = query;
   const warnings = [];
-  if (/\bkeyword\.text\b/.test(corrected) && !/ad_group_criterion\.keyword\.text/.test(corrected)) {
-    corrected = corrected.replace(/\bkeyword\.text\b/g, "ad_group_criterion.keyword.text");
+  if (/(?<![\w.])keyword\.text\b/.test(corrected)) {
+    corrected = corrected.replace(/(?<![\w.])keyword\.text\b/g, "ad_group_criterion.keyword.text");
     warnings.push({ code: "FIELD_RENAMED", message: "keyword.text \u2192 ad_group_criterion.keyword.text" });
   }
-  if (/\bkeyword\.match_type\b/.test(corrected) && !/ad_group_criterion\.keyword\.match_type/.test(corrected)) {
-    corrected = corrected.replace(/\bkeyword\.match_type\b/g, "ad_group_criterion.keyword.match_type");
+  if (/(?<![\w.])keyword\.match_type\b/.test(corrected)) {
+    corrected = corrected.replace(/(?<![\w.])keyword\.match_type\b/g, "ad_group_criterion.keyword.match_type");
     warnings.push({ code: "FIELD_RENAMED", message: "keyword.match_type \u2192 ad_group_criterion.keyword.match_type" });
   }
   if (/shopping_performance_view\.product_\w+/.test(corrected)) {