mcp-google-ads 1.2.2 → 1.2.4

package/dist/auth-cli.js

@@ -264,33 +264,39 @@ async function pickAccount(accounts, presetCustomerId) {
     return match;
   }
   if (accounts.length === 1) {
+    const only = accounts[0];
+    if (only.isManager) {
+      throw new Error(
+        `Only one accessible account and it is a Manager (MCC): ${only.name} (${only.id}). Most tools need a child (leaf) customer account. Grant your Google user direct access to a specific client account under that MCC and re-run the auth helper.`
+      );
+    }
     process.stderr.write(
       `
-Only one account accessible: ${accounts[0].name} (${accounts[0].id}). Auto-selecting.
+Only one account accessible: ${only.name} (${only.id}). Auto-selecting.
 `
     );
-    return accounts[0];
+    return only;
   }
-  const sorted = [...accounts].sort((a, b) => {
-    const aKey = a.parentMccId ? `${a.parentMccId}:1:${a.name}` : `${a.id}:0:${a.name}`;
-    const bKey = b.parentMccId ? `${b.parentMccId}:1:${b.name}` : `${b.id}:0:${b.name}`;
-    return aKey.localeCompare(bKey);
-  });
-  const choices = sorted.map((acct) => {
-    const prefix = acct.parentMccId ? "    \u21B3 " : acct.isManager ? "\u{1F4C1} " : "\u2022 ";
-    const mccSuffix = acct.parentMccId ? "" : acct.isManager ? " (MCC)" : "";
-    return {
-      title: `${prefix}${acct.name} \u2014 ${acct.id}${mccSuffix}`,
-      value: acct,
-      disabled: acct.isManager && accounts.some((a) => a.parentMccId === acct.id) ? false : false
-    };
+  const leafChildren = (mccId) => accounts.filter((a) => a.parentMccId === mccId);
+  const topLevel = accounts.filter((a) => {
+    if (a.parentMccId) return false;
+    if (a.isManager && leafChildren(a.id).length > 0) return false;
+    return true;
   });
-  const response = await prompts(
+  const topLevelChoices = topLevel.map((acct) => ({
+    title: acct.isManager ? `\u{1F4C1} ${acct.name} \u2014 ${acct.id} (MCC, no enumerated children)` : `\u2022 ${acct.name} \u2014 ${acct.id}`,
+    value: { kind: "leaf", account: acct }
+  }));
+  const mccChoices = accounts.filter((a) => a.isManager && leafChildren(a.id).length > 0).map((mcc2) => ({
+    title: `\u{1F4C1} ${mcc2.name} \u2014 ${mcc2.id}  (${leafChildren(mcc2.id).length} children)`,
+    value: { kind: "mcc", account: mcc2 }
+  }));
+  const first = await prompts(
     {
       type: "select",
-      name: "account",
+      name: "choice",
       message: "Which Google Ads account should Claude use?",
-      choices,
+      choices: [...topLevelChoices, ...mccChoices],
       initial: 0
     },
     {
@@ -299,10 +305,32 @@ Only one account accessible: ${accounts[0].name} (${accounts[0].id}). Auto-selec
       }
     }
   );
-  if (!response.account) {
-    throw new Error("No account selected");
+  if (!first.choice) throw new Error("No account selected");
+  if (first.choice.kind === "leaf") {
+    return first.choice.account;
   }
-  return response.account;
+  const mcc = first.choice.account;
+  const children = leafChildren(mcc.id);
+  const childChoices = children.map((child) => ({
+    title: `\u2022 ${child.name} \u2014 ${child.id}`,
+    value: child
+  }));
+  const second = await prompts(
+    {
+      type: "select",
+      name: "account",
+      message: `Pick the client account under ${mcc.name}:`,
+      choices: childChoices,
+      initial: 0
+    },
+    {
+      onCancel: () => {
+        throw new Error("Cancelled by user");
+      }
+    }
+  );
+  if (!second.account) throw new Error("No account selected");
+  return second.account;
 }
 async function run(argv = process.argv.slice(2)) {
   const args = parseArgs(argv);

package/dist/build-info.json

@@ -1,5 +1,5 @@
 {
-  "sha": "d3b40c7",
-  "builtAt": "2026-04-17T20:02:21.153Z",
+  "sha": "ce9ba28",
+  "builtAt": "2026-04-17T21:01:16.463Z",
   "embeddedSecrets": true
 }

package/dist/resilience.js

@@ -21,13 +21,11 @@ const logger = pino(
           singleLine: true,
           translateTime: "SYS:standard",
           destination: 2
-          // stderr -- stdout is reserved for MCP JSON-RPC
         }
       }
     }
   },
-  // When no transport (test mode), write to stderr directly
-  process.env.NODE_ENV === "test" ? pino.destination(2) : void 0
+  pino.destination(2)
 );
 const MAX_RESPONSE_SIZE = 2e5;
 function safeResponse(data, context) {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "mcp-google-ads",
   "mcpName": "io.github.mharnett/google-ads",
-  "version": "1.2.2",
+  "version": "1.2.4",
   "description": "MCP server for Google Ads API with MCC support, 36 tools for campaign management, reporting, and optimization -- now includes Demand Gen campaign creation end-to-end. Safe by default -- all changes created PAUSED.",
   "main": "dist/index.js",
   "bin": {
@@ -33,7 +33,7 @@
     "dev:auth": "tsx src/auth-cli.ts",
     "test": "vitest run",
     "test:portability": "vitest run src/portability.test.ts",
-    "prepublishOnly": "npm run build && npm test"
+    "prepublishOnly": "node scripts/check-embedded.mjs && npm run build && npm test"
   },
   "keywords": [
     "mcp",