mcp-google-ads 1.0.8 → 1.0.10

package/dist/build-info.json

@@ -1 +1 @@
-{"sha":"bdd46f2","builtAt":"2026-04-09T21:28:07.536Z"}
+{"sha":"fd2b513","builtAt":"2026-04-09T21:47:18.505Z"}

package/dist/errors.js

@@ -12,7 +12,7 @@ export class GoogleAdsAuthError extends Error {
 export class GoogleAdsRateLimitError extends Error {
     retryAfterMs;
     constructor(retryAfterMs, cause) {
-        super(`Rate limited, retry after ${retryAfterMs}ms`);
+        super(`Google Ads rate limited, retry after ${retryAfterMs}ms`);
         this.retryAfterMs = retryAfterMs;
         this.name = "GoogleAdsRateLimitError";
         this.cause = cause;
@@ -89,7 +89,7 @@ export function classifyError(error) {
         message.includes("refresh token") ||
         code?.authentication_error ||
         code?.authorization_error) {
-        return new GoogleAdsAuthError(`Auth failed: ${message}. Check your refresh token and OAuth credentials.`, error);
+        return new GoogleAdsAuthError(`Google Ads auth failed: ${message}. Check your refresh token and OAuth credentials.`, error);
     }
     // Rate limiting
     if (status === 429 ||

package/dist/index.js

@@ -34,6 +34,10 @@ if (process.argv.includes("--version") || process.argv.includes("-v")) {
     console.error(__cliPkg.version);
     process.exit(0);
 }
+// ============================================
+// ENV VAR TRIMMING
+// ============================================
+const envTrimmed = (key) => (process.env[key] || "").trim().replace(/^["']|["']$/g, "");
 function loadConfig() {
     // Try config.json first (for multi-client setups)
     const configPath = join(dirname(new URL(import.meta.url).pathname), "..", "config.json");
@@ -41,12 +45,12 @@ function loadConfig() {
         return JSON.parse(readFileSync(configPath, "utf-8"));
     }
     // Fall back to env vars (single-client mode)
-    const clientId = process.env.GOOGLE_ADS_CLIENT_ID;
-    const clientSecret = process.env.GOOGLE_ADS_CLIENT_SECRET;
-    const developerToken = process.env.GOOGLE_ADS_DEVELOPER_TOKEN;
-    const refreshToken = process.env.GOOGLE_ADS_REFRESH_TOKEN;
-    const customerId = process.env.GOOGLE_ADS_CUSTOMER_ID;
-    const mccId = process.env.GOOGLE_ADS_MCC_CUSTOMER_ID;
+    const clientId = envTrimmed("GOOGLE_ADS_CLIENT_ID");
+    const clientSecret = envTrimmed("GOOGLE_ADS_CLIENT_SECRET");
+    const developerToken = envTrimmed("GOOGLE_ADS_DEVELOPER_TOKEN");
+    const refreshToken = envTrimmed("GOOGLE_ADS_REFRESH_TOKEN");
+    const customerId = envTrimmed("GOOGLE_ADS_CUSTOMER_ID");
+    const mccId = envTrimmed("GOOGLE_ADS_MCC_CUSTOMER_ID");
     if (!clientId || !clientSecret || !developerToken || !refreshToken) {
         throw new Error("No configuration found. Either:\n" +
             "  1. Set env vars: GOOGLE_ADS_CLIENT_ID, GOOGLE_ADS_CLIENT_SECRET, GOOGLE_ADS_DEVELOPER_TOKEN, GOOGLE_ADS_REFRESH_TOKEN, GOOGLE_ADS_CUSTOMER_ID\n" +
@@ -113,11 +117,11 @@ class GoogleAdsManager {
             throw new GoogleAdsAuthError(msg);
         }
         logger.info("Credentials validated: all required env vars present");
-        this.defaultRefreshToken = process.env.GOOGLE_ADS_REFRESH_TOKEN;
+        this.defaultRefreshToken = envTrimmed("GOOGLE_ADS_REFRESH_TOKEN");
         this.api = new GoogleAdsApi({
-            client_id: process.env.GOOGLE_ADS_CLIENT_ID,
-            client_secret: process.env.GOOGLE_ADS_CLIENT_SECRET,
-            developer_token: process.env.GOOGLE_ADS_DEVELOPER_TOKEN,
+            client_id: envTrimmed("GOOGLE_ADS_CLIENT_ID"),
+            client_secret: envTrimmed("GOOGLE_ADS_CLIENT_SECRET"),
+            developer_token: envTrimmed("GOOGLE_ADS_DEVELOPER_TOKEN"),
         });
     }
     getClientForCustomerId(customerId) {
@@ -1759,6 +1763,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
             error: true,
             error_type: error.name,
             message: error.message,
+            server: __cliPkg.name,
         };
         if (error instanceof GoogleAdsAuthError) {
             response.action_required = "Re-authenticate: check refresh token in macOS Keychain. Token may be expired or revoked.";
@@ -1775,11 +1780,11 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
             response.details = rawError.errors || rawError.stack;
         }
         return {
+            isError: true,
             content: [{
                     type: "text",
                     text: JSON.stringify(response, null, 2),
                 }],
-            isError: true,
         };
     }
 });
@@ -1818,4 +1823,7 @@ process.on("SIGINT", () => {
 process.on("SIGPIPE", () => {
     // Client disconnected -- expected during shutdown
 });
+process.on("unhandledRejection", (reason) => {
+    console.error("[error] Unhandled promise rejection:", reason);
+});
 main().catch((err) => logger.error({ error: err.message, stack: err.stack }, "Fatal startup error"));

package/dist/resilience.js

@@ -5,7 +5,7 @@ import pino from "pino";
 // ============================================
 export const logger = pino({
     level: process.env.LOG_LEVEL || "info",
-    ...(process.env.NODE_ENV !== "test" && {
+    ...(process.env.NODE_ENV !== "test" && process.stderr.isTTY && {
         transport: {
             target: "pino-pretty",
             options: {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "mcp-google-ads",
   "mcpName": "io.github.mharnett/google-ads",
-  "version": "1.0.8",
+  "version": "1.0.10",
   "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": {
@@ -53,7 +53,7 @@
   },
   "homepage": "https://github.com/mharnett/mcp-google-ads#readme",
   "engines": {
-    "node": ">=18.0.0"
+    "node": ">=18.18.0"
   },
   "dependencies": {
     "@anthropic-ai/sdk": "^0.24.0",