@koda-sl/baker-cli 0.349.0-dev.c05788ee3 → 0.349.0-dev.fe6a04f5a

package/README.md

@@ -742,7 +742,7 @@ Use `--no-cache` on any command to bypass **both** the local file cache and the
 |------|-----------|---------|
 | `FIELD_NOT_FOUND` | No | Invalid field in GAQL query |
 | `WRONG_RESOURCE` | No | Field queried from wrong resource |
-| `INVALID_OPERATOR` | No | Unsupported operator (e.g. CONTAINS(), or IS NOT NULL on a list field such as `final_urls` — the fix names the operators Google accepts) |
+| `INVALID_OPERATOR` | No | Unsupported operator (e.g. CONTAINS) |
 | `CUSTOMER_NOT_FOUND` | No | Invalid customer ID — run `baker ads google accounts` to find valid IDs |
 | `MISSING_MANAGER_ID` | No | Account not accessible (may need MCC login-customer-id) |
 | `INCOMPATIBLE_FIELDS` | No | Fields can't be in same query |
@@ -3422,6 +3422,7 @@ baker actions list                         # default: bucketed (claimable, myCla
 baker actions list --bucketed=false --status pending
 baker actions list --bucketed=false --sort priority   # flat list, do-first order (default); use --sort recent for newest-first
 baker actions list --output md             # id-first Markdown; works for both the bucketed and flat shapes
+baker actions list --search "conversion goal" --output md   # find Tasks by words in name/description/tags (flat, best match first)
 # With BAKER_CHAT_ID, the bucketed list folds in THIS chat's draft: staged creates appear in
 # `draftCreates`; published actions being completed/discarded/updated carry a `draftStatus` marker.
 # Only the caller's own chat draft is reflected — never another chat's staged work.

package/dist/cli.js

@@ -7880,7 +7880,7 @@ function buildListHints({ returned, limit }) {
     return [];
   }
   return [
-    `This list is capped at ${limit} Tasks and came back full, so older Tasks exist that are NOT shown. Do not treat it as the whole backlog \u2014 re-run with --limit ${limit * 2} (or narrow with --status / --q).`
+    `This list is capped at ${limit} Tasks and came back full, so older Tasks exist that are NOT shown. Do not treat it as the whole backlog \u2014 re-run with --limit ${limit * 2} (or narrow with --status / --search).`
   ];
 }
 
@@ -8490,8 +8490,13 @@ var linkCommand = defineCommand7({
 import { defineCommand as defineCommand8 } from "citty";
 registerSchema({
   command: "actions.list",
-  description: "List actions for the current company. Default: pre-bucketed (claimable/myClaims/blocked/claimedByOthers/completed/discarded), each bucket ordered do-first (blockers before blocked, then highest-leverage). The bucketed view reads the whole open backlog, so nothing pending is ever hidden from it. Set --bucketed=false for a flat list filterable by --status, orderable by --sort, and windowed by --limit.",
+  description: 'List actions for the current company. Default: pre-bucketed (claimable/myClaims/blocked/claimedByOthers/completed/discarded), each bucket ordered do-first (blockers before blocked, then highest-leverage). The bucketed view reads the whole open backlog, so nothing pending is ever hidden from it. Set --bucketed=false for a flat list filterable by --status, orderable by --sort, and windowed by --limit. Looking for one Task by name? --search "conversion goal" --output md returns only the matching Tasks instead of the whole backlog.',
   args: {
+    search: {
+      type: "string",
+      description: 'Find Tasks by words in their name, description or tags, searched on the server (e.g. --search "conversion goal"). Returns a flat list, best match first; by default it covers pending, in-progress and completed Tasks. Add --status to search one status instead, including discarded Tasks (--status discarded). Implies --bucketed=false.',
+      required: false
+    },
     bucketed: {
       type: "boolean",
       description: "Pre-bucket the results based on the current chat (default: true)",
@@ -8519,9 +8524,10 @@ registerSchema({
 var listCommand2 = defineCommand8({
   meta: {
     name: "list",
-    description: "List actions. Default: bucketed view (claimable, myClaims, blocked, claimedByOthers). Use --bucketed=false for raw filter by --status."
+    description: "List actions. Default: bucketed view (claimable, myClaims, blocked, claimedByOthers). Use --bucketed=false for raw filter by --status, or --search <text> to find Tasks by name."
   },
   args: {
+    search: { type: "string", description: "Find Tasks by words in name/description/tags", required: false },
     bucketed: { type: "boolean", description: "Pre-bucket by chat", required: false, default: true },
     status: { type: "string", description: "Status filter (raw mode)", required: false },
     sort: { type: "string", description: "Order (raw mode): priority|recent", required: false },
@@ -8530,12 +8536,9 @@ var listCommand2 = defineCommand8({
   },
   run: async ({ args: args5 }) => {
     try {
-      const bucketed = args5.bucketed !== false;
-      const parsedLimit = args5.limit === void 0 ? Number.NaN : Number.parseInt(String(args5.limit), 10);
-      const limit = Number.isFinite(parsedLimit) && parsedLimit > 0 ? parsedLimit : ACTIONS_LIST_DEFAULT_LIMIT;
-      const body = buildRequestBody(args5, bucketed, limit);
+      const body = buildListRequest(args5, getEnv().BAKER_CHAT_ID);
       const response = await apiPost("/api/actions/list", body);
-      const hints2 = !bucketed && response.ok && Array.isArray(response.data) ? buildListHints({ returned: response.data.length, limit }) : [];
+      const hints2 = body.bucketed === false && body.limit !== void 0 && response.ok && Array.isArray(response.data) ? buildListHints({ returned: response.data.length, limit: body.limit }) : [];
       const envelope = hints2.length > 0 ? { ...response, hints: hints2 } : response;
       if (args5.output === "md" && envelope.ok) {
         process.stdout.write(renderMarkdown(envelope.data, hints2));
@@ -8547,20 +8550,25 @@ var listCommand2 = defineCommand8({
     }
   }
 });
-function buildRequestBody(args5, bucketed, limit) {
-  const env = getEnv();
-  const body = { bucketed };
-  if (bucketed) {
-    if (env.BAKER_CHAT_ID) {
-      body.chatId = env.BAKER_CHAT_ID;
-    }
-    return body;
+function buildListRequest(args5, chatId) {
+  const q = args5.search?.trim();
+  if (args5.bucketed !== false && !q) {
+    return chatId ? { bucketed: true, chatId } : { bucketed: true };
+  }
+  const parsedLimit = args5.limit === void 0 ? Number.NaN : Number.parseInt(String(args5.limit), 10);
+  const body = {
+    bucketed: false,
+    // Default to do-first ordering so the flat list matches its documented
+    // contract; only "recent" opts out. A search keeps relevance order.
+    sort: args5.sort === "recent" ? "recent" : "priority",
+    limit: Number.isFinite(parsedLimit) && parsedLimit > 0 ? parsedLimit : ACTIONS_LIST_DEFAULT_LIMIT
+  };
+  if (q) {
+    body.q = q;
   }
   if (args5.status) {
     body.status = args5.status;
   }
-  body.sort = args5.sort === "recent" ? "recent" : "priority";
-  body.limit = limit;
   return body;
 }
 function toRow(entry) {
@@ -9848,27 +9856,6 @@ var CORRECTION_RULES = [
       };
     }
   },
-  // 0b. OPERATOR_FIELD_MISMATCH — the field refuses this operator, and Google names the
-  // ones it takes. Kept ahead of the field-specific rules for the same reason as rule 0.
-  // A repeated field (final_urls, labels, …) takes only CONTAINS ALL/ANY/NONE, so it has
-  // no null test at all: read it unfiltered and filter the rows instead.
-  {
-    matchApiError: /OPERATOR_FIELD_MISMATCH|is not a valid operator to use with/i,
-    fix: (ctx) => {
-      const message = ctx.apiErrorMessage ?? "";
-      const field = message.match(/valid operator to use with '([\w.]+)'/i)?.[1] ?? "this field";
-      const operator = message.match(/([A-Z][A-Z_ =!<>]*?) is not a valid operator/)?.[1]?.trim();
-      const valid = message.match(/Valid operators: (.+?)\.?(?:"|$)/i)?.[1]?.trim();
-      const refused = operator ? `${operator} cannot be used on ${field}` : `${field} does not take that operator`;
-      const allowed = valid ? `; its valid operators are ${valid}` : "";
-      const repeated = valid?.split(",").every((op) => /^CONTAINS (ALL|ANY|NONE)$/i.test(op.trim())) ?? false;
-      const guidance = repeated ? `. ${field} is a list field, so it cannot be tested for null or compared with =: to find rows that have it, drop that condition, keep ${field} in SELECT and filter the rows (rows without it omit the key); to match values, use ${field} CONTAINS ANY ('value')` : ". Rewrite that condition with one of them, or drop it and filter the rows";
-      return {
-        action: "change_operator",
-        explanation: `${refused}${allowed}${guidance}`
-      };
-    }
-  },
   // 1. bare keyword.text → ad_group_criterion.keyword.text (leaves <resource>.keyword.text untouched)
   {
     matchQuery: /(?<![\w.])keyword\.text\b/,
@@ -9911,11 +9898,10 @@ var CORRECTION_RULES = [
       };
     }
   },
-  // 4. CONTAINS → LIKE. Keyed on the query alone: Google's error envelope ("Request contains an
-  // invalid argument … Valid operators: CONTAINS ANY") matches any loose error pattern, and this
-  // advice only makes sense when the query actually wrote CONTAINS(…).
+  // 4. CONTAINS → LIKE
   {
     matchQuery: /CONTAINS\s*\(/i,
+    matchApiError: /CONTAINS.*not.*supported|invalid.*operator.*CONTAINS/i,
     fix: (ctx) => {
       const match = ctx.originalQuery.match(/CONTAINS\s*\(\s*([^,]+),\s*'([^']+)'\s*\)/i);
       if (match) {
@@ -50013,7 +49999,7 @@ function outputRows(rows, args5, response, cached) {
   }
   writeJsonEnvelope({ ...response, ...cached && { cached: true } });
 }
-function buildRequestBody2(args5, propertyId, useCache) {
+function buildRequestBody(args5, propertyId, useCache) {
   const body = { propertyId };
   if (args5.preset) body.preset = args5.preset;
   if (args5.dimensions) body.dimensions = args5.dimensions.split(",").map((d) => d.trim());
@@ -50088,7 +50074,7 @@ Free-form (escape hatch):
     }
     const propertyId = await resolvePropertyId(args5);
     const useCache = !args5["no-cache"];
-    const body = buildRequestBody2(args5, propertyId, useCache);
+    const body = buildRequestBody(args5, propertyId, useCache);
     const cacheKey = buildQueryCacheKey(propertyId, JSON.stringify(body));
     if (useCache) {
       const cached = cacheGet("ga4-queries", cacheKey);
@@ -51013,7 +50999,7 @@ function outputRows2(rows, args5, response, cached) {
   }
   writeJsonEnvelope({ ...response, ...cached && { cached: true } });
 }
-function buildRequestBody3(args5, siteUrl, useCache) {
+function buildRequestBody2(args5, siteUrl, useCache) {
   const body = { siteUrl };
   if (args5.preset) body.preset = args5.preset;
   if (args5.brand) body.brand = args5.brand;
@@ -51097,7 +51083,7 @@ Free-form (escape hatch):
     }
     const siteUrl = await resolveSiteUrl(args5);
     const useCache = !args5["no-cache"];
-    const body = buildRequestBody3(args5, siteUrl, useCache);
+    const body = buildRequestBody2(args5, siteUrl, useCache);
     const cacheKey = buildQueryCacheKey(siteUrl, JSON.stringify(body));
     if (useCache) {
       const cached = cacheGet("gsc-queries", cacheKey);