mcp-google-ads 1.0.6 → 1.0.8

package/dist/build-info.json

@@ -1 +1 @@
-{"sha":"5a28978","builtAt":"2026-04-09T20:25:53.245Z"}
+{"sha":"bdd46f2","builtAt":"2026-04-09T21:28:07.536Z"}

package/dist/index.js

@@ -8,29 +8,30 @@ import { CallToolRequestSchema, ListToolsRequestSchema, } from "@modelcontextpro
 import { tools } from "./tools.js";
 import { GoogleAdsApi, enums } from "google-ads-api";
 import { readFileSync, existsSync } from "fs";
+// CLI package info
+const __cliPkg = JSON.parse(readFileSync(join(dirname(new URL(import.meta.url).pathname), "..", "package.json"), "utf-8"));
 // Log build fingerprint at startup
 try {
     const __buildInfoDir = dirname(new URL(import.meta.url).pathname);
     const buildInfo = JSON.parse(readFileSync(join(__buildInfoDir, "build-info.json"), "utf-8"));
-    logger.info({ sha: buildInfo.sha, builtAt: buildInfo.builtAt }, "Build fingerprint");
+    console.error(`[build] SHA: ${buildInfo.sha} (${buildInfo.builtAt})`);
 }
 catch {
-    // build-info.json not present (dev mode)
+    console.error(`[build] ${__cliPkg.name}@${__cliPkg.version} (dev mode)`);
 }
 // CLI flags
-const __cliPkg = JSON.parse(readFileSync(join(dirname(new URL(import.meta.url).pathname), "..", "package.json"), "utf-8"));
 if (process.argv.includes("--help") || process.argv.includes("-h")) {
-    console.log(`${__cliPkg.name} v${__cliPkg.version}\n`);
-    console.log(`Usage: ${__cliPkg.name} [options]\n`);
-    console.log("MCP server communicating via stdio. Configure in your .mcp.json.\n");
-    console.log("Options:");
-    console.log("  --help, -h       Show this help message");
-    console.log("  --version, -v    Show version number");
-    console.log(`\nDocumentation: https://github.com/mharnett/mcp-google-ads`);
+    console.error(`${__cliPkg.name} v${__cliPkg.version}\n`);
+    console.error(`Usage: ${__cliPkg.name} [options]\n`);
+    console.error("MCP server communicating via stdio. Configure in your .mcp.json.\n");
+    console.error("Options:");
+    console.error("  --help, -h       Show this help message");
+    console.error("  --version, -v    Show version number");
+    console.error(`\nDocumentation: https://github.com/mharnett/mcp-google-ads`);
     process.exit(0);
 }
 if (process.argv.includes("--version") || process.argv.includes("-v")) {
-    console.log(__cliPkg.version);
+    console.error(__cliPkg.version);
     process.exit(0);
 }
 function loadConfig() {
@@ -1151,8 +1152,8 @@ class GoogleAdsManager {
 const config = loadConfig();
 const adsManager = new GoogleAdsManager(config);
 const server = new Server({
-    name: "mcp-google-ads",
-    version: "1.0.0",
+    name: __cliPkg.name,
+    version: __cliPkg.version,
 }, {
     capabilities: {
         tools: {},
@@ -1806,4 +1807,15 @@ async function main() {
     await server.connect(transport);
     logger.info("MCP Google Ads server running");
 }
+process.on("SIGTERM", () => {
+    console.error("[shutdown] SIGTERM received, exiting");
+    process.exit(0);
+});
+process.on("SIGINT", () => {
+    console.error("[shutdown] SIGINT received, exiting");
+    process.exit(0);
+});
+process.on("SIGPIPE", () => {
+    // Client disconnected -- expected during shutdown
+});
 main().catch((err) => logger.error({ error: err.message, stack: err.stack }, "Fatal startup error"));

package/dist/resilience.js

@@ -1,4 +1,4 @@
-import { retry, circuitBreaker, wrap, handleAll, timeout, TimeoutStrategy, ExponentialBackoff, ConsecutiveBreaker, } from "cockatiel";
+import { retry, circuitBreaker, wrap, handleWhen, timeout, TimeoutStrategy, ExponentialBackoff, ConsecutiveBreaker, } from "cockatiel";
 import pino from "pino";
 // ============================================
 // LOGGER
@@ -12,36 +12,51 @@ export const logger = pino({
                 colorize: true,
                 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) : undefined);
 // ============================================
 // SAFE RESPONSE (Response Size Limiting)
 // ============================================
 const MAX_RESPONSE_SIZE = 200_000; // 200KB
 export function safeResponse(data, context) {
-    const jsonStr = JSON.stringify(data);
-    const sizeBytes = Buffer.byteLength(jsonStr, "utf-8");
-    if (sizeBytes > MAX_RESPONSE_SIZE) {
-        logger.warn({ sizeBytes, maxSize: MAX_RESPONSE_SIZE, context }, `Response exceeds size limit, truncating`);
-        // If it's an array, truncate
-        if (Array.isArray(data)) {
-            const truncated = data.slice(0, Math.max(1, Math.floor(data.length * 0.5)));
-            return truncated;
+    let current = data;
+    for (let pass = 0; pass < 10; pass++) {
+        const jsonStr = JSON.stringify(current);
+        const sizeBytes = Buffer.byteLength(jsonStr, "utf-8");
+        if (sizeBytes <= MAX_RESPONSE_SIZE)
+            return current;
+        logger.warn({ sizeBytes, maxSize: MAX_RESPONSE_SIZE, context, pass }, "Response exceeds size limit, truncating");
+        if (Array.isArray(current)) {
+            current = current.slice(0, Math.max(1, Math.floor(current.length * 0.5)));
+            continue;
         }
-        // If it's an object with items/results, truncate those
-        if (typeof data === "object" && data !== null) {
-            const obj = data;
-            for (const key of ["items", "results", "data", "rows"]) {
-                if (Array.isArray(obj[key])) {
+        if (typeof current === "object" && current !== null) {
+            const obj = current;
+            let truncated = false;
+            for (const key of ["items", "results", "data", "rows", "tags", "triggers", "variables"]) {
+                if (Array.isArray(obj[key]) && obj[key].length > 1) {
                     obj[key] = obj[key].slice(0, Math.max(1, Math.floor(obj[key].length * 0.5)));
-                    return obj;
+                    if ("count" in obj)
+                        obj.count = obj[key].length;
+                    if ("row_count" in obj)
+                        obj.row_count = obj[key].length;
+                    obj.truncated = true;
+                    truncated = true;
+                    break;
                 }
             }
+            if (truncated)
+                continue;
         }
+        // Can't truncate further (not an array/object with known keys)
+        break;
     }
-    return data;
+    return current;
 }
 // ============================================
 // RETRY + CIRCUIT BREAKER + TIMEOUT
@@ -50,12 +65,28 @@ const backoff = new ExponentialBackoff({
     initialDelay: 100,
     maxDelay: 5_000,
 });
+const isTransient = handleWhen((err) => {
+    const msg = (err?.message || "").toLowerCase();
+    const code = err?.code || err?.status;
+    // Don't retry auth errors
+    if (code === 401 || code === 403 || code === 7 || code === 16)
+        return false;
+    if (msg.includes("unauthenticated") || msg.includes("permission_denied") || msg.includes("invalid_grant"))
+        return false;
+    // Don't retry client errors (except rate limits)
+    if (code === 429 || msg.includes("rate"))
+        return true;
+    if (code >= 400 && code < 500)
+        return false;
+    // Retry everything else (5xx, timeouts, network errors)
+    return true;
+});
 // Individual policies
-const retryPolicy = retry(handleAll, {
+const retryPolicy = retry(isTransient, {
     maxAttempts: 3,
     backoff,
 });
-const circuitBreakerPolicy = circuitBreaker(handleAll, {
+const circuitBreakerPolicy = circuitBreaker(isTransient, {
     halfOpenAfter: 60_000, // 60s to attempt recovery
     breaker: new ConsecutiveBreaker(5), // Open after 5 consecutive failures
 });

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "mcp-google-ads",
   "mcpName": "io.github.mharnett/google-ads",
-  "version": "1.0.6",
+  "version": "1.0.8",
   "description": "MCP server for Google Ads API with MCC support, 35 tools for campaign management, reporting, and optimization. Safe by default — all changes created PAUSED.",
   "main": "dist/index.js",
   "bin": {